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.

Sign up for GitHub

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

Jump to bottom

test: add test unknown credential error of process.setgroups #22368

Closed
wants to merge 1 commit into from
Closed

test: add test unknown credential error of process.setgroups #22368

wants to merge 1 commit into from

Conversation

shisama
Copy link
Contributor

@shisama shisama commented Aug 17, 2018

Added test to check ERR_UNKNOWN_CREDENTIAL of process.setgroups to increase
coverage.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

@nodejs-github-bot nodejs-github-bot added the test Issues and PRs related to the tests. label Aug 17, 2018
@BridgeAR
Copy link
Member

CI https://ci.nodejs.org/job/node-test-pull-request/16514/

@BridgeAR BridgeAR added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. fast-track PRs that do not need to wait for 48 hours to land. labels Aug 17, 2018
@joyeecheung joyeecheung mentioned this pull request Aug 17, 2018
Closed
cjihrig
cjihrig approved these changes Aug 17, 2018
View reviewed changes
Copy link
Contributor

@cjihrig cjihrig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but the last CI run didn't go well (infrastructure issues).

@joyeecheung joyeecheung mentioned this pull request Aug 17, 2018
Closed
@joyeecheung
Copy link
Member

This PR needs a rebase against master to avoid the git failure in the CI.

test: add test unknown credential error of process.setgroups
92c29f2 
Added test to check ERR_UNKNOWN_CREDENTIAL of process.setgroups to increase
coverage.
@shisama shisama force-pushed the test-process-setgroups-error branch from 5f55a29 to 92c29f2 Compare August 20, 2018 01:20
@shisama
Copy link
Contributor Author

shisama commented Aug 20, 2018

@joyeecheung Thank you for your comment. I rebased this.

@joyeecheung
Copy link
Member

https://ci.nodejs.org/job/node-test-pull-request/16593/

@shisama
Copy link
Contributor Author

shisama commented Aug 20, 2018

Maybe node-test-commit-linux failure is a flaky test (#20628).

12:21:48 not ok 2287 sequential/test-http2-session-timeout
12:21:48   ---
12:21:48   duration_ms: 0.926
12:21:48   severity: fail
12:21:48   exitcode: 1
12:21:48   stack: |-
12:21:48     (node:19667) ExperimentalWarning: The http2 module is an experimental API.
12:21:48     assert.js:84
12:21:48       throw new AssertionError(obj);
12:21:48       ^
12:21:48     
12:21:48     AssertionError [ERR_ASSERTION]: function should not have been called at /home/iojs/build/workspace/node-test-commit-linux/nodes/ubuntu1404-64/test/sequential/test-http2-session-timeout.js:9
12:21:48         at Http2Server.mustNotCall (/home/iojs/build/workspace/node-test-commit-linux/nodes/ubuntu1404-64/test/common/index.js:517:12)
12:21:48         at Http2Server.emit (events.js:182:13)
12:21:48         at ServerHttp2Session.sessionOnTimeout (internal/http2/core.js:2545:15)
12:21:48         at Object.onceWrapper (events.js:273:13)
12:21:48         at ServerHttp2Session.emit (events.js:182:13)
12:21:48         at ServerHttp2Session._onTimeout (internal/http2/core.js:1270:10)
12:21:48         at ontimeout (timers.js:454:11)
12:21:48         at tryOnTimeout (timers.js:326:5)
12:21:48         at listOnTimeout (timers.js:300:5)
12:21:48         at processTimers (timers.js:257:5)

@addaleax
Copy link
Member

Resume CI: https://ci.nodejs.org/job/node-test-pull-request/16708/

@addaleax
Copy link
Member

Landed in 4dc8467 🎉

@addaleax addaleax closed this Aug 24, 2018
addaleax pushed a commit that referenced this pull request Aug 24, 2018
@addaleax
test: add test unknown credential error of process.setgroups
4dc8467 
Added test to check ERR_UNKNOWN_CREDENTIAL of process.setgroups to
increase coverage.

PR-URL: #22368
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
targos pushed a commit that referenced this pull request Aug 24, 2018
@targos
test: add test unknown credential error of process.setgroups
0d30848 
Added test to check ERR_UNKNOWN_CREDENTIAL of process.setgroups to
increase coverage.

PR-URL: #22368
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
targos pushed a commit that referenced this pull request Sep 3, 2018
@targos
test: add test unknown credential error of process.setgroups
fcf059a 
Added test to check ERR_UNKNOWN_CREDENTIAL of process.setgroups to
increase coverage.

PR-URL: #22368
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
@targos targos mentioned this pull request Sep 5, 2018
Merged
This was referenced Sep 6, 2018
Closed
Closed
@snyk-bot snyk-bot mentioned this pull request Aug 11, 2020
Open
@snyk-bot snyk-bot mentioned this pull request Aug 5, 2021
Open
This was referenced Aug 5, 2021
Open
Open
Open
Open
Open
Open
Open
@snyk-bot snyk-bot mentioned this pull request Aug 15, 2021
Open
@molllyn1 molllyn1 mentioned this pull request Mar 22, 2024
Open
@137717unity 137717unity mentioned this pull request Mar 22, 2024
Open
@kissofire kissofire mentioned this pull request Mar 22, 2024
Open
@aliceUnhinged613 aliceUnhinged613 mentioned this pull request Mar 22, 2024
Open
@ron-a20 ron-a20 mentioned this pull request Mar 22, 2024
Open
@wedataintelligence wedataintelligence mentioned this pull request Mar 23, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jasnell jasnell jasnell approved these changes

@addaleax addaleax addaleax approved these changes

@lpinca lpinca lpinca approved these changes

@cjihrig cjihrig cjihrig approved these changes

@BridgeAR BridgeAR BridgeAR approved these changes

@trivikr trivikr trivikr approved these changes

Assignees
No one assigned
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. fast-track PRs that do not need to wait for 48 hours to land. test Issues and PRs related to the tests.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@shisama @BridgeAR @joyeecheung @addaleax @jasnell @lpinca @cjihrig @trivikr @nodejs-github-bot