Merge pull request #49 from msgpack/fix_tests

fix tests on Edge for ReadableStream

Author: gfx

.travis.yml

@@ -42,9 +42,9 @@ script: |
   if [ "${BROWSER}" = "" ]
     then npm run test:cover
   else
-    travis_wait 300 npm run test:browser -- --browsers "$BROWSER"
+    travis_wait 600 npm run test:browser -- --browsers "$BROWSER"
   fi
 after_success: |
   if [ "${BROWSER}" = "" ]
-    then nyc report --reporter=json > coverage/coverage.json && codecov
+    then codecov -f coverage/*.json
   fi

package.json

@@ -22,7 +22,7 @@
     "test:cover:wasm": "npx nyc --no-clean npm run test:wasm",
     "test:cover:td": "npx nyc --no-clean npm run test:td",
     "cover:clean": "rimraf .nyc_output coverage/",
-    "cover:report": "nyc report --reporter=lcov --reporter=text-summary --reporter=html",
+    "cover:report": "nyc report --reporter=text-summary --reporter=html --reporter=json",
     "test:browser": "karma start --single-run",
     "test:browser:firefox": "karma start --single-run --browsers FirefoxHeadless",
     "test:browser:chrome": "karma start --single-run --browsers ChromeHeadless",

sauceLabs.ts

@@ -77,6 +77,6 @@ export const sauceLaunchers: Record<string, SauceLauncher> = {
     browserName: "Chrome",
     platformName: "Android",
     platformVersion: "latest",
-    deviceName: "Android GoogleAPI Emulator",
+    deviceName: "Android Emulator",
   },
 };

test/whatwg-streams.test.ts

@@ -1,9 +1,21 @@
 import { deepStrictEqual } from "assert";
 import { decodeAsync, encode, decodeArrayStream } from "@msgpack/msgpack";
 
+const isReadableStreamConstructorAvailable: boolean = (() => {
+  try {
+    // Edge <= 18 has ReadableStream but its constructor is not available
+    new ReadableStream({
+      start() {},
+    });
+    return true;
+  } catch {
+    return false;
+  }
+})();
+
 describe("whatwg streams", () => {
   before(function() {
-    if (typeof ReadableStream === "undefined") {
+    if (!isReadableStreamConstructorAvailable) {
       this.skip();
     }
   });