Skip to content

test: simplify tests code - #28065

Closed
himself65 wants to merge 1 commit into
nodejs:masterfrom
himself65:update-test
Closed

test: simplify tests code#28065
himself65 wants to merge 1 commit into
nodejs:masterfrom
himself65:update-test

Conversation

@himself65

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

@nodejs-github-bot nodejs-github-bot added the test Issues and PRs related to Node.js core tests and test infrastructure. label Jun 5, 2019
Comment thread test/parallel/test-accessor-properties.js Outdated
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@ZYSzys ZYSzys added the author ready PRs with CI started, the required approvals, and no outstanding review comments. label Jun 5, 2019
Comment thread test/parallel/test-accessor-properties.js Outdated
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@ZYSzys

ZYSzys commented Jun 9, 2019

Copy link
Copy Markdown
Member

Landed in fefc275 🎉

@ZYSzys ZYSzys closed this Jun 9, 2019
ZYSzys pushed a commit that referenced this pull request Jun 9, 2019
PR-URL: #28065
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
BridgeAR pushed a commit that referenced this pull request Jun 17, 2019
PR-URL: #28065
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
@BridgeAR BridgeAR mentioned this pull request Jun 17, 2019
@himself65
himself65 deleted the update-test branch June 28, 2019 03:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs with CI started, the required approvals, and no outstanding review comments. test Issues and PRs related to Node.js core tests and test infrastructure.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants