Fix a data race in unit test

stadiamaps · Nov 4, 2024 · 27dab0b · 27dab0b
1 parent a0639c7
commit 27dab0b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/.github/workflows/android.yml b/.github/workflows/android.yml
@@ -52,7 +52,6 @@ jobs:
       working-directory: android
 
   test:
-
     runs-on: ubuntu-latest
     concurrency:
       group: ${{ github.workflow }}-${{ github.ref }}-android-test

diff --git a/apple/Tests/FerrostarCoreTests/FerrostarCoreTests.swift b/apple/Tests/FerrostarCoreTests/FerrostarCoreTests.swift
@@ -395,6 +395,11 @@ final class FerrostarCoreTests: XCTestCase {
         await fulfillment(of: [routeDeviationCallbackExp], timeout: 1.0)
         await fulfillment(of: [loadedAltRoutesExp], timeout: 1.0)
 
+        // At this point, there is a brief window where the delegate call loading alternate routes has completed,
+        // but the state is still updating. This is a quick hack to fix the tests
+        // in the absence of something more reliable.
+        try? await Task.sleep(nanoseconds: NSEC_PER_SEC / 10)
+
         XCTAssert(core.state?.isCalculatingNewRoute == false, "Expected to no longer be calculating a new route")
     }