Skip to content

test: add test-benchmark-zlib - #14763

Closed
Trott wants to merge 1 commit into
nodejs:masterfrom
Trott:benchmark-zlib
Closed

test: add test-benchmark-zlib#14763
Trott wants to merge 1 commit into
nodejs:masterfrom
Trott:benchmark-zlib

Conversation

@Trott

Copy link
Copy Markdown
Member

Add minimal test to confirm that zlib benchmarks run.

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

test zlib benchmark

Add minimal test to confirm that zlib benchmarks run.
@TrottTrott added benchmark Issues and PRs related to the benchmark subsystem. test Issues and PRs related to the tests. zlib Issues and PRs related to the zlib subsystem. labels Aug 11, 2017
@nodejs-github-botnodejs-github-bot added the test Issues and PRs related to the tests. label Aug 11, 2017
@Trott

Copy link
Copy Markdown
MemberAuthor

Trott added a commit to Trott/io.js that referenced this pull request Aug 14, 2017
Add minimal test to confirm that zlib benchmarks run.
PR-URL: nodejs#14763
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@Trott

Copy link
Copy Markdown
MemberAuthor

Landed in f417add

@MylesBorins

Copy link
Copy Markdown
Contributor

Setting dont-land-on-v8.x as the zlib benchmarks don't exist on 8.x

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

benchmarkIssues and PRs related to the benchmark subsystem.testIssues and PRs related to the tests.zlibIssues and PRs related to the zlib subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants

@Trott@MylesBorins@refack@jasnell@cjihrig@aqrln@gibfahn@nodejs-github-bot