Merge pull request #28 from greglittlefield-wf/js_tests

UIP-1903 Run unit tests in dart2js on CI

smithy.yaml

@@ -6,6 +6,7 @@ runner_image: drydock-prod.workiva.org/workiva/smithy-runner-dart:74173
 
 script:
   - pub get
+  - ./tool/smithy_dart2js_tests.sh
 
 artifacts:
   build:

test/over_react/component/dom_components_test.dart

@@ -17,7 +17,7 @@ library dom_components_test;
 // Tell dart2js that this library only needs to reflect the specified types.
 // This speeds up compilation and makes JS output much smaller.
 @MirrorsUsed(targets: const [
-  'over_react.Dom'
+  'over_react.dom_components.Dom'
 ])
 import 'dart:mirrors';
 

tool/dev.dart

@@ -30,7 +30,11 @@ main(List<String> args) async {
     ..pubServe = true
     ..platforms = [
       'vm',
-      'content-shell'
+      'content-shell',
+      // Can't run tests in dart2js on Travis since the suite takes too long to load and times out.
+      // Run on Smithy instead.
+      // See https://github.com/Workiva/over_react/issues/36
+      // 'chrome',
     ]
     // Prevent test load timeouts on Smithy.
     ..concurrency = 1

tool/smithy_dart2js_tests.sh

@@ -0,0 +1,18 @@
+#!/usr/bin/env bash
+
+# Can't run tests in dart2js on Travis since the suite takes too long to load and times out.
+# Run on Smithy instead.
+# See https://github.com/Workiva/over_react/issues/36
+
+set -e
+
+# Trick the test package into using Chromium instead of Chrome
+TMP_BIN=$(mktemp -d)
+ln -s "$(which chromium-browser)" "$TMP_BIN/google-chrome"
+export PATH="$PATH:$TMP_BIN"
+
+# Run the tests
+DART_FLAGS=--checked  xvfb-run -s '-screen 0 1024x768x24' pub run dart_dev test -p chrome
+
+# Be sneaky and clean up our tricks
+rm -rf "$TMP_BIN"