group()s work both on Android and iOS

leancodepl · Sep 12, 2023 · c9c68b2 · c9c68b2
1 parent 85aa81f
commit c9c68b2
Show file tree

Hide file tree

Showing 3 changed files with 41 additions and 4 deletions.
diff --git a/packages/patrol/example/ios/Runner.xcodeproj/project.pbxproj b/packages/patrol/example/ios/Runner.xcodeproj/project.pbxproj
@@ -265,7 +265,7 @@
 			isa = PBXProject;
 			attributes = {
 				LastSwiftUpdateCheck = 1430;
-				LastUpgradeCheck = 1300;
+				LastUpgradeCheck = 1430;
 				ORGANIZATIONNAME = "";
 				TargetAttributes = {
 					97C146ED1CF9000F007C117D = {

diff --git a/packages/patrol/example/ios/Runner.xcodeproj/xcshareddata/xcschemes/Runner.xcscheme b/packages/patrol/example/ios/Runner.xcodeproj/xcshareddata/xcschemes/Runner.xcscheme
@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <Scheme
-   LastUpgradeVersion = "1300"
+   LastUpgradeVersion = "1430"
    version = "1.3">
    <BuildAction
       parallelizeBuildables = "YES"

diff --git a/packages/patrol/ios/Classes/PatrolAppServiceClient.swift b/packages/patrol/ios/Classes/PatrolAppServiceClient.swift
@@ -41,10 +41,11 @@ import NIO
     let request = Patrol_Empty()
     let response = try await client.listDartTests(request)
 
-    return response.group.entries.map {
+    NSLog("RAW: Got tests: \(response.group)")
+
+    return response.group.listTestsFlat(parentGroupName: "").map {
       $0.name
     }
-    // return response.group.groups.map { $0.name }
   }
 
   @objc public func runDartTest(name: String) async throws -> RunDartTestResponse {
@@ -58,3 +59,39 @@ import NIO
     )
   }
 }
+
+extension Patrol_DartGroupEntry {
+
+  func listTestsFlat(parentGroupName: String) -> [Patrol_DartGroupEntry] {
+    var tests = [Patrol_DartGroupEntry]()
+
+    for test in self.entries {
+      if test.type == Patrol_DartGroupEntry.GroupEntryType.test {
+        if parentGroupName.isEmpty {
+          // This case is invalid, because every test will have at least
+          // 1 named group - its filename.
+
+          continue  // What else can we do?
+        }
+
+        // TODO: There has to be some copy() function
+        tests.append(
+          .with {
+            $0.name = "\(parentGroupName) \(test.name)"
+            $0.fullName = test.fullName
+            $0.type = test.type
+            $0.entries = test.entries
+          })
+      } else if test.type == Patrol_DartGroupEntry.GroupEntryType.group {
+        if parentGroupName.isEmpty {
+          tests.append(contentsOf: test.listTestsFlat(parentGroupName: test.name))
+        } else {
+          tests.append(
+            contentsOf: test.listTestsFlat(parentGroupName: "\(parentGroupName) \(test.name)"))
+        }
+      }
+    }
+
+    return tests
+  }
+}