Updated tests to work on GitHub Actions

Author: JamesMessinger

karma.conf.js

@@ -3,30 +3,37 @@
 // https://jstools.dev/karma-config/
 
 "use strict";
+
 const { karmaConfig } = require("@jsdevtools/karma-config");
-let exclude = [];
+const { host } = require("@jsdevtools/host-environment");
+
+// let exclude = [];
 
-if (process.env.WINDOWS && process.env.CI) {
-  // We're running in a Windows CI/CD environment, so Karma-Config will use SauceLabs.
-  // The following tests tend to fail on SauceLabs, probably due to zero-byte files
-  // and special characters in the paths. So, exclude them.
-  exclude.push(
-    "test/specs/__*/**",
-    "test/specs/blank/**/*.spec.js",
-    "test/specs/circular*/**/*.spec.js",
-    "test/specs/empty/**/*.spec.js",
-    "test/specs/invalid/**/*.spec.js",
-    "test/specs/parsers/**/*.spec.js"
-  );
-}
+// if (host.os.windows && host.ci) {
+//   // We're running in a Windows CI/CD environment, so Karma-Config will use SauceLabs.
+//   // The following tests tend to fail on SauceLabs, probably due to zero-byte files
+//   // and special characters in the paths. So, exclude them.
+//   exclude.push(
+//     "test/specs/__*/**",
+//     "test/specs/blank/**/*.spec.js",
+//     "test/specs/circular*/**/*.spec.js",
+//     "test/specs/empty/**/*.spec.js",
+//     "test/specs/invalid/**/*.spec.js",
+//     "test/specs/parsers/**/*.spec.js"
+//   );
+// }
 
 module.exports = karmaConfig({
   sourceDir: "lib",
   fixtures: "test/fixtures/**/*.js",
   browsers: {
-    ie: true,
+    chrome: !host.os.windows,
+    firefox: host.os.linux,
+    safari: host.os.linux,    // SauceLabs
+    edge: host.os.linux,      // SauceLabs
+    ie: host.os.windows,
   },
-  config: {
-    exclude,
-  }
+  // config: {
+  //   exclude,
+  // }
 });

test/fixtures/mocha.js

@@ -13,7 +13,7 @@ if (host.browser) {
 
 beforeEach(function () {
   // Flag TravisCI and SauceLabs as being very slow environments
-  let isSlowEnvironment = host.env.CI || host.karma;
+  let isSlowEnvironment = host.ci || host.karma;
 
   // Most of our tests perform multiple AJAX requests,
   // so we need to increase the timeouts to allow for that

test/fixtures/polyfill.js

@@ -5,6 +5,6 @@ const { host } = require("@jsdevtools/host-environment");
 // Load the Babel Polyfills for old browsers.
 // NOTE: It's important that we ONLY do this when needed,
 // to ensure that our code works _without_ polyfills everywhere else
-if (host.os.windows) {
+if (host.browser.IE) {
   require("@babel/polyfill");
 }

test/specs/http.spec.js

@@ -50,7 +50,7 @@ describe("HTTP options", () => {
   });
 
   describe("http.redirect", () => {
-    if (host.karma && host.env.CI) {
+    if (host.browser.safari && host.karma && host.karma.ci) {
       // These tests fail in Safari when running on Sauce Labs (they pass when running on Safari locally).
       // It gets an XHR error when trying to reach httpbin.org.
       // TODO: Only skip these tests on Safari on Sauce Labs