From 4bd8176788aeaad4e591abc252370b18ac375ae7 Mon Sep 17 00:00:00 2001
From: Martin Andonoski <mandonoski@paypal.com>
Date: Mon, 5 Aug 2024 10:01:34 +0200
Subject: [PATCH] Add test plan and explicitly disable tests instead of using
 an underscore

---
 .../xcshareddata/xcschemes/Flow.xcscheme      | 89 +++++++++++++++++++
 Flow.xctestplan                               | 34 +++++++
 FlowTests/FutureBasicTests.swift              |  8 +-
 FlowTests/FutureRepeatTests.swift             |  4 +-
 FlowTests/SignalProviderTests.swift           |  4 +-
 5 files changed, 131 insertions(+), 8 deletions(-)
 create mode 100644 .swiftpm/xcode/xcshareddata/xcschemes/Flow.xcscheme
 create mode 100644 Flow.xctestplan

diff --git a/.swiftpm/xcode/xcshareddata/xcschemes/Flow.xcscheme b/.swiftpm/xcode/xcshareddata/xcschemes/Flow.xcscheme
new file mode 100644
index 0000000..f5ffa7d
--- /dev/null
+++ b/.swiftpm/xcode/xcshareddata/xcschemes/Flow.xcscheme
@@ -0,0 +1,89 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<Scheme
+   LastUpgradeVersion = "1600"
+   version = "1.7">
+   <BuildAction
+      parallelizeBuildables = "YES"
+      buildImplicitDependencies = "YES"
+      buildArchitectures = "Automatic">
+      <BuildActionEntries>
+         <BuildActionEntry
+            buildForTesting = "YES"
+            buildForRunning = "YES"
+            buildForProfiling = "YES"
+            buildForArchiving = "YES"
+            buildForAnalyzing = "YES">
+            <BuildableReference
+               BuildableIdentifier = "primary"
+               BlueprintIdentifier = "Flow"
+               BuildableName = "Flow"
+               BlueprintName = "Flow"
+               ReferencedContainer = "container:">
+            </BuildableReference>
+         </BuildActionEntry>
+      </BuildActionEntries>
+   </BuildAction>
+   <TestAction
+      buildConfiguration = "Debug"
+      selectedDebuggerIdentifier = "Xcode.DebuggerFoundation.Debugger.LLDB"
+      selectedLauncherIdentifier = "Xcode.DebuggerFoundation.Launcher.LLDB"
+      shouldUseLaunchSchemeArgsEnv = "YES">
+      <TestPlans>
+         <TestPlanReference
+            reference = "container:Flow.xctestplan"
+            default = "YES">
+         </TestPlanReference>
+      </TestPlans>
+      <Testables>
+         <TestableReference
+            skipped = "NO">
+            <BuildableReference
+               BuildableIdentifier = "primary"
+               BlueprintIdentifier = "FlowTests"
+               BuildableName = "FlowTests"
+               BlueprintName = "FlowTests"
+               ReferencedContainer = "container:">
+            </BuildableReference>
+            <SkippedTests>
+               <Test
+                  Identifier = "FutureBasicTests/testRecursiveMapCancel()">
+               </Test>
+            </SkippedTests>
+         </TestableReference>
+      </Testables>
+   </TestAction>
+   <LaunchAction
+      buildConfiguration = "Debug"
+      selectedDebuggerIdentifier = "Xcode.DebuggerFoundation.Debugger.LLDB"
+      selectedLauncherIdentifier = "Xcode.DebuggerFoundation.Launcher.LLDB"
+      launchStyle = "0"
+      useCustomWorkingDirectory = "NO"
+      ignoresPersistentStateOnLaunch = "NO"
+      debugDocumentVersioning = "YES"
+      debugServiceExtension = "internal"
+      allowLocationSimulation = "YES">
+   </LaunchAction>
+   <ProfileAction
+      buildConfiguration = "Release"
+      shouldUseLaunchSchemeArgsEnv = "YES"
+      savedToolIdentifier = ""
+      useCustomWorkingDirectory = "NO"
+      debugDocumentVersioning = "YES">
+      <MacroExpansion>
+         <BuildableReference
+            BuildableIdentifier = "primary"
+            BlueprintIdentifier = "Flow"
+            BuildableName = "Flow"
+            BlueprintName = "Flow"
+            ReferencedContainer = "container:">
+         </BuildableReference>
+      </MacroExpansion>
+   </ProfileAction>
+   <AnalyzeAction
+      buildConfiguration = "Debug">
+   </AnalyzeAction>
+   <ArchiveAction
+      buildConfiguration = "Release"
+      revealArchiveInOrganizer = "YES">
+   </ArchiveAction>
+</Scheme>
diff --git a/Flow.xctestplan b/Flow.xctestplan
new file mode 100644
index 0000000..694dbdc
--- /dev/null
+++ b/Flow.xctestplan
@@ -0,0 +1,34 @@
+{
+  "configurations" : [
+    {
+      "id" : "CC1330D0-BFB1-4ED1-A5E3-361E83C2FEF9",
+      "name" : "Test Scheme Action",
+      "options" : {
+
+      }
+    }
+  ],
+  "defaultOptions" : {
+
+  },
+  "testTargets" : [
+    {
+      "skippedTests" : [
+        "FutureBasicTests\/testRecursiveFlatMapCancel()",
+        "FutureBasicTests\/testRecursiveImmediateFlatMap()",
+        "FutureBasicTests\/testRecursiveImmediateMap()",
+        "FutureBasicTests\/testRecursiveMapCancel()",
+        "FutureRepeatTests\/testRepeatForeverDelayStackOverflowMain()",
+        "FutureRepeatTests\/testRepeatForeverStackOverflowBackground()",
+        "SignalProviderTests\/testExclusiveMultiThread()",
+        "SignalProviderTests\/testStartWithTheRetriggerWithinOnEventMerge()"
+      ],
+      "target" : {
+        "containerPath" : "container:",
+        "identifier" : "FlowTests",
+        "name" : "FlowTests"
+      }
+    }
+  ],
+  "version" : 1
+}
diff --git a/FlowTests/FutureBasicTests.swift b/FlowTests/FutureBasicTests.swift
index 6623fb2..612a649 100644
--- a/FlowTests/FutureBasicTests.swift
+++ b/FlowTests/FutureBasicTests.swift
@@ -242,7 +242,7 @@ class FutureBasicTests: FutureTest {
         }
     }
 
-    func _testRecursiveImmediateFlatMap() {
+    func testRecursiveImmediateFlatMap() {
         testFuture(timeout: 1000) { () -> Future<Int> in
             func test(_ val: Int) -> Future<Int> {
                 if val > 0 {
@@ -256,7 +256,7 @@ class FutureBasicTests: FutureTest {
         }
     }
 
-    func _testRecursiveImmediateMap() {
+    func testRecursiveImmediateMap() {
         testFuture(timeout: 1000) { () -> Future<Int> in
             var f = Future(4711).delay(by: 1)
             for _ in 1...100000 {
@@ -294,7 +294,7 @@ class FutureBasicTests: FutureTest {
         }
     }
 
-    func _testRecursiveMapCancel() {
+    func testRecursiveMapCancel() {
         testFuture(timeout: 1000) { () -> Future<Int> in
             var f = Future(4711).delay(by: 1)
             for _ in 1...10000 {
@@ -305,7 +305,7 @@ class FutureBasicTests: FutureTest {
         }
     }
 
-    func _testRecursiveFlatMapCancel() {
+    func testRecursiveFlatMapCancel() {
         testFuture(timeout: 1000) { () -> Future<Int> in
             var f = Future(4711).delay(by: 1)
             for _ in 1...10000 {
diff --git a/FlowTests/FutureRepeatTests.swift b/FlowTests/FutureRepeatTests.swift
index 6372545..3a0732a 100644
--- a/FlowTests/FutureRepeatTests.swift
+++ b/FlowTests/FutureRepeatTests.swift
@@ -162,7 +162,7 @@ class FutureRepeatTests: FutureTest {
 
     // Not sure it we can always catch up?
     // With new immediate repeat handling this one will never come past repeatForever()
-    func _testRepeatForeverDelayStackOverflowMain() {
+    func testRepeatForeverDelayStackOverflowMain() {
         testFuture(repeatCount: 0, timeout: 2, allDoneDelay: 2, cancelAfterDelay: 0.1) { () -> Future<Int> in
             let f = Future(1).onResultRepeat()
             return f
@@ -177,7 +177,7 @@ class FutureRepeatTests: FutureTest {
     }
 
     // cancel seems to never be able to catch up
-    func _testRepeatForeverStackOverflowBackground() {
+    func testRepeatForeverStackOverflowBackground() {
         testFuture(repeatCount: 0, timeout: 2, allDoneDelay: 20, cancelAfterDelay: 0.1) { () -> Future<Int> in
             let f = Future(1).onValue(on: .concurrentBackground) { _ in }.onResultRepeat()
             return f
diff --git a/FlowTests/SignalProviderTests.swift b/FlowTests/SignalProviderTests.swift
index e2761c6..dfc127a 100644
--- a/FlowTests/SignalProviderTests.swift
+++ b/FlowTests/SignalProviderTests.swift
@@ -1504,7 +1504,7 @@ class SignalProviderTests: XCTestCase {
         XCTAssertTrue(vals[1] == (2, 3))
     }
 
-    func _testStartWithTheRetriggerWithinOnEventMerge() { /// Sometime fails sometime not.
+    func testStartWithTheRetriggerWithinOnEventMerge() { /// Sometime fails sometime not.
         for _ in 0..<1 {
             let bag = DisposeBag()
             let callbacker = Callbacker<Int>()
@@ -1710,7 +1710,7 @@ class SignalProviderTests: XCTestCase {
         XCTAssertEqual(result, [1, 2])
     }
 
-    func _testExclusiveMultiThread() {
+    func testExclusiveMultiThread() {
         for _ in 0..<1000 {
             let callbacker = Callbacker<Int>()
             var result = [Int]()