Skip to content

test: add test for validation for wasi.start() argument - #30919

Merged
Trott merged 0 commit into
nodejs:masterfrom
Trott:wasi-start-validation
Dec 14, 2019
Merged

test: add test for validation for wasi.start() argument#30919
Trott merged 0 commit into
nodejs:masterfrom
Trott:wasi-start-validation

Conversation

@Trott

Copy link
Copy Markdown
Member

Improve test coverage in lib/wasi.js.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • commit message follows commit guidelines

@nodejs-github-botnodejs-github-bot added the test Issues and PRs related to the tests. label Dec 12, 2019
@TrottTrott added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Dec 12, 2019
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@gengjiawengengjiawen added wasi Issues and PRs related to the WebAssembly System Interface. wasm Issues and PRs related to WebAssembly. labels Dec 13, 2019
@TrottTrott closed this Dec 14, 2019
@Trott
Trottforce-pushed the wasi-start-validation branch from eaab76d to decc5f5CompareDecember 14, 2019 21:35
@Trott
Trott merged commit decc5f5 into nodejs:masterDec 14, 2019
@Trott

Copy link
Copy Markdown
MemberAuthor

Landed in decc5f5

@Trott
Trott deleted the wasi-start-validation branch December 14, 2019 21:36
MylesBorins pushed a commit that referenced this pull request Dec 17, 2019
PR-URL: #30919
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
@MylesBorinsMylesBorins mentioned this pull request Dec 17, 2019
targos pushed a commit that referenced this pull request Jan 14, 2020
PR-URL: #30919
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
@targostargos mentioned this pull request Jan 15, 2020
BethGriggs pushed a commit that referenced this pull request Feb 6, 2020
PR-URL: #30919
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
@MylesBorinsMylesBorins mentioned this pull request Feb 8, 2020
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.testIssues and PRs related to the tests.wasiIssues and PRs related to the WebAssembly System Interface.wasmIssues and PRs related to WebAssembly.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@Trott@nodejs-github-bot@addaleax@lpinca@cjihrig@gengjiawen