Skip to content

Commit

Permalink
Merge pull request #12373 from webpack/deps/wasm
Browse files Browse the repository at this point in the history
update `@webassemblyjs`
  • Loading branch information
sokra committed Jan 9, 2021
2 parents 88610d7 + d034c89 commit 492e5ec
Show file tree
Hide file tree
Showing 14 changed files with 137 additions and 158 deletions.
1 change: 1 addition & 0 deletions declarations.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -234,6 +234,7 @@ declare module "@webassemblyjs/ast" {
args: string[];
result: string[];
}
export function moduleContextFromModuleAST(ast: any): any;

// Node matcher
export function isGlobalType(n: Node): boolean;
Expand Down
4 changes: 1 addition & 3 deletions lib/wasm-sync/WebAssemblyGenerator.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,9 +10,7 @@ const Generator = require("../Generator");
const WebAssemblyUtils = require("./WebAssemblyUtils");

const t = require("@webassemblyjs/ast");
const {
moduleContextFromModuleAST
} = require("@webassemblyjs/helper-module-context");
const { moduleContextFromModuleAST } = require("@webassemblyjs/ast");
const { editWithAST, addWithAST } = require("@webassemblyjs/wasm-edit");
const { decode } = require("@webassemblyjs/wasm-parser");

Expand Down
4 changes: 1 addition & 3 deletions lib/wasm-sync/WebAssemblyParser.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,7 @@
"use strict";

const t = require("@webassemblyjs/ast");
const {
moduleContextFromModuleAST
} = require("@webassemblyjs/helper-module-context");
const { moduleContextFromModuleAST } = require("@webassemblyjs/ast");
const { decode } = require("@webassemblyjs/wasm-parser");
const Parser = require("../Parser");
const StaticExportsDependency = require("../dependencies/StaticExportsDependency");
Expand Down
9 changes: 4 additions & 5 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,9 @@
"dependencies": {
"@types/eslint-scope": "^3.7.0",
"@types/estree": "^0.0.45",
"@webassemblyjs/ast": "1.9.1",
"@webassemblyjs/helper-module-context": "1.9.1",
"@webassemblyjs/wasm-edit": "1.9.1",
"@webassemblyjs/wasm-parser": "1.9.1",
"@webassemblyjs/ast": "1.11.0",
"@webassemblyjs/wasm-edit": "1.11.0",
"@webassemblyjs/wasm-parser": "1.11.0",
"acorn": "^8.0.4",
"browserslist": "^4.14.5",
"chrome-trace-event": "^1.0.2",
Expand Down Expand Up @@ -96,7 +95,7 @@
"ts-loader": "^8.0.2",
"typescript": "^4.2.0-dev.20201130",
"url-loader": "^4.1.0",
"wast-loader": "^1.9.0",
"wast-loader": "^1.11.0",
"webassembly-feature": "1.3.0",
"webpack-cli": "^4.3.0",
"xxhashjs": "^0.2.2",
Expand Down
2 changes: 1 addition & 1 deletion test/cases/side-effects/issue-11673/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,6 @@ it("should compile", done => {
const worker = new Worker(new URL("worker.js", import.meta.url));
worker.once("message", value => {
expect(value).toBe(42);
done();
Promise.resolve(worker.terminate()).then(() => done(), done);
});
});
1 change: 1 addition & 0 deletions test/configCases/worker/custom-worker/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ it("should allow to create a WebWorker", async () => {
};
});
expect(result).toBe("data: OK, thanks");
await worker.terminate();
});

it("should allow to share chunks", async () => {
Expand Down
2 changes: 1 addition & 1 deletion test/configCases/worker/node-worker-named/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ it("should allow to create a WebWorker", async () => {
});
});
expect(result).toBe("data: OK, thanks");
worker.terminate();
await worker.terminate();
});

it("should allow to create another WebWorker", async () => {
Expand Down
2 changes: 1 addition & 1 deletion test/configCases/worker/node-worker/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ it("should allow to create a WebWorker", async () => {
});
});
expect(result).toBe("data: OK, thanks");
worker.terminate();
await worker.terminate();
});

it("should allow to share chunks", async () => {
Expand Down
1 change: 1 addition & 0 deletions test/configCases/worker/web-worker/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ it("should allow to create a WebWorker", async () => {
};
});
expect(result).toBe("data: OK, thanks");
await worker.terminate();
});

it("should allow to share chunks", async () => {
Expand Down
2 changes: 1 addition & 1 deletion test/configCases/worker/worker-contenthash/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ it("should allow to create a WebWorker", async () => {
});
});
expect(result).toBe("data: OK, thanks");
worker.terminate();
await worker.terminate();
});

it("should allow to share chunks", async () => {
Expand Down
4 changes: 4 additions & 0 deletions test/helpers/createFakeWorker.js
Original file line number Diff line number Diff line change
Expand Up @@ -70,4 +70,8 @@ require(${JSON.stringify(path.resolve(outputDirectory, file))});
postMessage(data) {
this.worker.postMessage(data);
}

terminate() {
return this.worker.terminate();
}
};
4 changes: 2 additions & 2 deletions test/helpers/supportsWorker.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
module.exports = function supportsWebAssembly() {
module.exports = function supportsWorker() {
try {
// eslint-disable-next-line node/no-unsupported-features/node-builtins
return require("worker_threads") !== "undefined";
return require("worker_threads") !== undefined;
} catch (e) {
return false;
}
Expand Down
1 change: 1 addition & 0 deletions test/hotCases/worker/update-in-worker/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -13,4 +13,5 @@ it("should support hot module replacement in WebWorkers", done => {
}
};
worker.postMessage("test");
Promise.resolve(worker.terminate()).then(done, done);
});
Loading

0 comments on commit 492e5ec

Please sign in to comment.