Properly classify model removals as breaking (#21)

Author: bauer-andreas

Resources/ExampleDocuments/migration_guide.json

@@ -705,7 +705,7 @@
         {
             "type": "removal",
             "id": "CategoryStatus",
-            "breaking": false,
+            "breaking": true,
             "solvable": false
         },
         {

Sources/ApodiniMigratorCompare/Comparators/Model/ModelsComparator.swift

@@ -51,8 +51,7 @@ struct ModelsComparator: Comparator {
 
         for removal in removalCandidates where !pairs.contains(where: { $0.contains(removal.deltaIdentifier) }) {
             results.append(.removal(
-                id: removal.deltaIdentifier,
-                breaking: false
+                id: removal.deltaIdentifier
             ))
         }
 

Tests/ApodiniMigratorTests/ApodiniMigratorCompareTests/ModelsComparatorTests.swift

@@ -63,7 +63,7 @@ final class ModelsComparatorTests: ApodiniMigratorXCTestCase {
         let change = try XCTUnwrap(modelChanges.first)
         XCTAssertEqual(change.id, programmingLanguages.deltaIdentifier)
         XCTAssertEqual(change.type, .removal)
-        XCTAssertEqual(change.breaking, false)
+        XCTAssertEqual(change.breaking, true)
         XCTAssertEqual(change.solvable, false)
 
         let removalChange = try XCTUnwrap(change.modeledRemovalChange)