Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: increase coverage for internal/error_serdes.js #36628

Merged
merged 1 commit into from
Dec 28, 2020

Conversation

Lxxyx
Copy link
Member

@Lxxyx Lxxyx commented Dec 25, 2020

test serializeError with getter property

Refs: https://coverage.nodejs.org/coverage-73a21e4c06d5781d/lib/internal/error_serdes.js.html#L49

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 the tests. label Dec 25, 2020
@aduh95 aduh95 added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. request-ci Add this label to start a Jenkins CI on a PR. labels Dec 26, 2020
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Dec 26, 2020
@nodejs-github-bot
Copy link
Collaborator

@Trott Trott force-pushed the test-increase-error_serdes-coverage branch from 33796a9 to c27efd4 Compare December 28, 2020 05:04
@Trott
Copy link
Member

Trott commented Dec 28, 2020

Landed in c27efd4

@Trott Trott merged commit c27efd4 into nodejs:master Dec 28, 2020
@Lxxyx Lxxyx deleted the test-increase-error_serdes-coverage branch December 28, 2020 05:08
danielleadams pushed a commit that referenced this pull request Jan 12, 2021
@danielleadams danielleadams mentioned this pull request Jan 12, 2021
targos pushed a commit that referenced this pull request May 1, 2021
@danielleadams danielleadams mentioned this pull request May 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. test Issues and PRs related to the tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants