chore(web): enable testing against legacy devices via BrowserStack

common/predictive-text/build.sh

@@ -27,7 +27,8 @@ builder_describe "Builds the lm-layer module" \
   "configure" \
   "build" \
   "test" \
-  "--ci        Sets $(builder_term test) action to use CI-based test configurations & reporting"
+  "--ci        Sets $(builder_term test) action to use CI-based test configurations & reporting" \
+  "--remote    Sets $(builder_term test) action to use a BrowserStack-based testing configuration"
 
 builder_describe_outputs \
   configure  /node_modules \
@@ -59,6 +60,8 @@ function do_test() {
   local TEST_OPTIONS=
   if builder_has_option --ci; then
     TEST_OPTIONS=--ci
+  elif builder_has_option --remote; then
+    TEST_OPTIONS=--remote
   fi
 
   # We'll test the included libraries here for now.  At some point, we may wish

common/predictive-text/unit_tests/in_browser/web-test-runner.remote.config.mjs

@@ -0,0 +1,5 @@
+import standardConfig from './web-test-runner.config.mjs';
+import { buildLegacyTestingConfig } from '@keymanapp/common-test-resources/wtr-browserstack-config.mjs';
+
+const config = buildLegacyTestingConfig(standardConfig, '@keymanapp/lexical-model-layer');
+export default config;

common/predictive-text/unit_tests/test.sh

@@ -23,7 +23,8 @@ builder_describe "Runs all tests for the language-modeling / predictive-text lay
   ":libraries  Runs unit tests for in-repo libraries used by this module"\
   ":headless   Runs this module's headless user tests" \
   ":browser    Runs this module's browser-based user tests" \
-  "--ci        Uses CI-based test configurations & emits CI-friendly test reports"
+  "--ci        Uses CI-based test configurations & emits CI-friendly test reports" \
+  "--remote    Uses a BrowserStack-based test configuration for legacy devices"
 
 # TODO: consider dependencies? ideally this will be test.inc.sh?
 
@@ -58,9 +59,14 @@ if builder_start_action test:libraries; then
   popd
 
   pushd "$KEYMAN_ROOT/common/web/lm-worker"
+  if builder_has_option --remote; then
+    WORKER_TEST_OPTS="$TEST_OPTS --remote"
+  else
+    WORKER_TEST_OPTS="$TEST_OPTS"
+  fi
   echo
   echo "### Running ${BUILDER_TERM_START}common/web/lm-worker${BUILDER_TERM_END} tests"
-  ./build.sh test $TEST_OPTS
+  ./build.sh test $WORKER_TEST_OPTS
   popd
 
   builder_finish_action success test:libraries
@@ -111,6 +117,8 @@ if builder_start_action test:browser; then
 
   if builder_has_option --ci; then
     WTR_CONFIG=.CI
+  elif builder_has_option --remote; then
+    WTR_CONFIG=.remote
   fi
 
   if builder_has_option --debug; then

common/test/resources/test-runner-TC-reporter.mjs

@@ -123,9 +123,18 @@ export default function teamcityReporter({ name="Web Test Runner JavaScript test
 
       for(const sessionName of sessionFailureMap.keys()) {
         const sessionTestData = testDefMap.get(sessionName);
+        const session = args.sessions.find((entry) => buildSessionName(entry) == sessionName);
 
         logger.log('');
         logger.log(`Failure data for '${sessionName}'`);
+
+        if(session.errors) {
+          for(let error of session.errors) {
+            // "Browser tests did not start after xxxx ms..."" errors appear here.
+            logger.error(`  ${error.message}`);
+          }
+        }
+
         for(const sessionFile of sessionFailureMap.get(sessionName)) {
           const testFile = sessionFile.testFile;
           const fileTestData = sessionTestData.get(testFile);

common/test/resources/test-timeouts-remote.mjs

@@ -0,0 +1,9 @@
+import * as baseTimeouts from './test-timeouts.mjs';
+
+// General pattern, in case more are defined.
+const scaledTimeouts = {
+  ...baseTimeouts
+};
+Object.keys(baseTimeouts).forEach((key) => scaledTimeouts[key] *= 10);
+
+export const DEFAULT_BROWSER_TIMEOUT = scaledTimeouts.DEFAULT_BROWSER_TIMEOUT; //ms