DartGroupEntry.listDartTests: fail more loudly when invariant is viol…

…ated
leancodepl · Sep 12, 2023 · 86b9b12 · 86b9b12
1 parent 04b41fb
commit 86b9b12
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/packages/patrol/android/src/main/kotlin/pl/leancode/patrol/ContractsExtensions.kt b/packages/patrol/android/src/main/kotlin/pl/leancode/patrol/ContractsExtensions.kt
@@ -195,7 +195,7 @@ fun DartGroupEntry.listTestsFlat(parentGroupName: String = ""): List<DartGroupEn
         if (test.type == DartGroupEntry.GroupEntryType.TEST) {
             if (parentGroupName.isEmpty()) {
                 // This case is invalid, because every test will have at least 1 named group - its filename.
-                throw IllegalStateException("Test $test has no named parent group")
+                throw IllegalStateException("Invariant violated: test $test has no named parent group")
             }
 
             tests.add(test.copy { name = "$parentGroupName ${test.name}" })

diff --git a/packages/patrol/ios/Classes/PatrolAppServiceClient.swift b/packages/patrol/ios/Classes/PatrolAppServiceClient.swift
@@ -71,7 +71,7 @@ extension Patrol_DartGroupEntry {
           // This case is invalid, because every test will have at least
           // 1 named group - its filename.
 
-          continue  // Ignore - what else can we do?
+          fatalError("Invariant violated: test \(test.name) has no named parent group")
         }
 
         test.name = "\(parentGroupName) \(test.name)"