fix: gracefully handle missing test cases between two apps

src/libs/E2E/reactNativeLaunchingTest.ts

@@ -66,6 +66,7 @@ E2EClient.getTestConfig()
             console.error(`[E2E] Test '${config.name}' not found`);
             // instead of throwing, report the error to the server, which is better for DX
             return E2EClient.submitTestResults({
+                branch: Config.E2E_BRANCH,
                 name: config.name,
                 error: `Test '${config.name}' not found`,
                 isCritical: false, 

tests/e2e/server/index.ts

@@ -27,6 +27,7 @@ type ServerInstance = {
     addTestStartedListener: AddListener<TestStartedListener>;
     addTestResultListener: AddListener<TestResultListener>;
     addTestDoneListener: AddListener<TestDoneListener>;
+    forceTestCompletion: () => void;
     start: () => Promise<void>;
     stop: () => Promise<Error | undefined>;
 };
@@ -96,6 +97,12 @@ const createServerInstance = (): ServerInstance => {
     const [testResultListeners, addTestResultListener] = createListenerState<TestResultListener>();
     const [testDoneListeners, addTestDoneListener] = createListenerState<TestDoneListener>();
 
+    const forceTestCompletion = () => {
+        testDoneListeners.forEach((listener) => {
+            listener();
+        });
+    };
+
     let activeTestConfig: TestConfig | undefined;
     const networkCache: Record<string, NetworkCacheMap> = {};
 
@@ -131,9 +138,7 @@ const createServerInstance = (): ServerInstance => {
             }
 
             case Routes.testDone: {
-                testDoneListeners.forEach((listener) => {
-                    listener();
-                });
+                forceTestCompletion();
                 return res.end('ok');
             }
 
@@ -200,6 +205,7 @@ const createServerInstance = (): ServerInstance => {
         addTestStartedListener,
         addTestResultListener,
         addTestDoneListener,
+        forceTestCompletion,
         start: () =>
             new Promise<void>((resolve) => {
                 server.listen(PORT, resolve);

tests/e2e/testRunner.ts

@@ -99,6 +99,8 @@ const runTests = async (): Promise<void> => {
             throw new Error(`Test '${testResult.name}' failed with error: ${testResult.error}`);
         }
         if (testResult?.error != null && !isCritical) {
+            // force test completion, since we don't want to have timeout error for non being execute test
+            server.forceTestCompletion();
             Logger.warn(`Test '${testResult.name}' failed with error: ${testResult.error}`);
         }
         let result = 0;