Commit bed120e

authored

Merge pull request #49 from msgpack/fix_tests

fix tests on Edge for ReadableStream

2 parents 7513075 + c422db6 commit bed120eCopy full SHA for bed120e

File tree

+17

-5

lines changed

+17

-5

lines changed

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -42,9 +42,9 @@ script: \|`
`42`	`42`	`if [ "${BROWSER}" = "" ]`
`43`	`43`	`then npm run test:cover`
`44`	`44`	`else`
`45`		`- travis_wait 300 npm run test:browser -- --browsers "$BROWSER"`
	`45`	`+ travis_wait 600 npm run test:browser -- --browsers "$BROWSER"`
`46`	`46`	`fi`
`47`	`47`	`after_success: \|`
`48`	`48`	`if [ "${BROWSER}" = "" ]`
`49`		`- then nyc report --reporter=json > coverage/coverage.json && codecov`
	`49`	`+ then codecov -f coverage/*.json`
`50`	`50`	`fi`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@`
`22`	`22`	`"test:cover:wasm": "npx nyc --no-clean npm run test:wasm",`
`23`	`23`	`"test:cover:td": "npx nyc --no-clean npm run test:td",`
`24`	`24`	`"cover:clean": "rimraf .nyc_output coverage/",`
`25`		`- "cover:report": "nyc report --reporter=lcov --reporter=text-summary --reporter=html",`
	`25`	`+ "cover:report": "nyc report --reporter=text-summary --reporter=html --reporter=json",`
`26`	`26`	`"test:browser": "karma start --single-run",`
`27`	`27`	`"test:browser:firefox": "karma start --single-run --browsers FirefoxHeadless",`
`28`	`28`	`"test:browser:chrome": "karma start --single-run --browsers ChromeHeadless",`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -77,6 +77,6 @@ export const sauceLaunchers: Record<string, SauceLauncher> = {`
`77`	`77`	`browserName: "Chrome",`
`78`	`78`	`platformName: "Android",`
`79`	`79`	`platformVersion: "latest",`
`80`		`- deviceName: "Android GoogleAPI Emulator",`
	`80`	`+ deviceName: "Android Emulator",`
`81`	`81`	`},`
`82`	`82`	`};`

Lines changed: 13 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -1,9 +1,21 @@`
`1`	`1`	`import { deepStrictEqual } from "assert";`
`2`	`2`	`import { decodeAsync, encode, decodeArrayStream } from "@msgpack/msgpack";`
`3`	`3`
	`4`	`+const isReadableStreamConstructorAvailable: boolean = (() => {`
	`5`	`+ try {`
	`6`	`+ // Edge <= 18 has ReadableStream but its constructor is not available`
	`7`	`+ new ReadableStream({`
	`8`	`+ start() {},`
	`9`	`+ });`
	`10`	`+ return true;`
	`11`	`+ } catch {`
	`12`	`+ return false;`
	`13`	`+ }`
	`14`	`+})();`
	`15`	`+`
`4`	`16`	`describe("whatwg streams", () => {`
`5`	`17`	`before(function() {`
`6`		`- if (typeofReadableStream==="undefined") {`
	`18`	`+ if (!isReadableStreamConstructorAvailable) {`
`7`	`19`	`this.skip();`
`8`	`20`	`}`
`9`	`21`	`});`

Comments

(0)