[HIGH] Unset forced-colors after running accessibility.liveRegion.activityStatus.sendFailed.contrast test (#4570)

* Reset forced-colors when test start

* Unset forced-colors on every run()

* Reset timezone on every run

Author: compulim

__tests__/html/accessibility.liveRegion.activityStatus.sendFailed.contrast.html

@@ -16,12 +16,8 @@
           const directLine = testHelpers.createDirectLineEmulator(store);
 
           await host.sendDevToolsCommand('Emulation.setEmulatedMedia', {
-          features: [
-            {
-              name: 'forced-colors',
-              value: 'active'
-            }
-          ]});
+            features: [{ name: 'forced-colors', value: 'active' }]
+          });
 
           WebChat.renderWebChat(
             {
@@ -37,29 +33,21 @@
 
           await pageConditions.uiConnected();
 
-          const { disconnect, flush } = pageObjects.observeLiveRegion();
-
-          try {
-            const sendMessage = await directLine.emulateOutgoingActivity('Hello, World!');
+          const sendMessage = await directLine.emulateOutgoingActivity('Hello, World!');
 
-            await directLine.emulateIncomingActivity('Aloha!');
+          await directLine.emulateIncomingActivity('Aloha!');
 
-            sendMessage.rejectPostActivity(new Error('artificial error'));
+          sendMessage.rejectPostActivity(new Error('artificial error'));
 
-            await pageConditions.became(
-              'failed to send message',
-              () => pageElements.activityStatuses()[0]?.innerText === 'Send failed. Retry.',
-              1000
-            );
-            
-          } finally {
-            
-            await host.snapshot();
+          await pageConditions.became(
+            'failed to send message',
+            () => pageElements.activityStatuses()[0]?.innerText === 'Send failed. Retry.',
+            1000
+          );
 
-            disconnect();
-          }
+          await host.snapshot();
         },
-        { ignoreErrors: true }
+        { ignoreErrors: 'artificial error' }
       );
     </script>
   </body>

packages/test/harness/src/browser/globals/run.js

@@ -10,11 +10,20 @@ export default function () {
 
       // Run the test, signal start by host.ready().
       // On success or failure, call host.done() or host.error() correspondingly.
-      return Promise.resolve()
-        .then(host.ready)
-        .then(fn)
-        .then(() => host.done(doneOptions))
-        .catch(host.error);
+      // This function will be executed in both Jest and "npm run browser".
+      return (
+        host
+          // Previous run may have enabled "forced-colors", we should unset it.
+          .sendDevToolsCommand('Emulation.setEmulatedMedia', {
+            features: [{ name: 'forced-colors', value: '' }]
+          })
+          // Some tests may have changed the time zone, we should unset it.
+          .then(() => host.sendDevToolsCommand('Emulation.setTimezoneOverride', { timezoneId: 'Etc/UTC' }))
+          .then(host.ready)
+          .then(fn)
+          .then(() => host.done(doneOptions))
+          .catch(host.error)
+      );
     })
   );
 }

packages/test/harness/src/host/dev/index.js

@@ -64,7 +64,6 @@ async function main() {
     process.once('SIGTERM', terminate);
 
     try {
-      await webDriver.sendDevToolsCommand('Emulation.setTimezoneOverride', { timezoneId: 'Etc/UTC' });
       // eslint-disable-next-line no-magic-numbers
       await webDriver.get(process.argv[2] || 'http://localhost:5080/');
 

packages/test/harness/src/host/jest/runHTML.js

@@ -52,7 +52,6 @@ global.runHTML = async function runHTML(url, options = DEFAULT_OPTIONS) {
 
     global.__operation__ = `loading URL ${absoluteURL.toString()}`;
 
-    await webDriver.sendDevToolsCommand('Emulation.setTimezoneOverride', { timezoneId: 'Etc/UTC' });
     await webDriver.get(absoluteURL);
 
     global.__operation__ = 'setting class name for body element';