-
Notifications
You must be signed in to change notification settings - Fork 676
npm vulnerabilities fixes #8511
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
Changes from all commits
3f2cda0
9f24a1a
1a4eb7a
b4ace87
b32ea3b
9e426af
cdd4128
16db2db
75b066a
babc627
76e3ac2
b0f360a
5dd5276
d2f009c
22541f6
2f31f62
7c8784c
2344d02
bfc76f7
94d72a3
3277000
b6a67ba
be46059
b13efed
c2c252e
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
This file was deleted.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,19 @@ | ||
| const childProcess = require('child_process'); | ||
|
|
||
| async function runCommands (commands) { | ||
| for (const command of commands) { | ||
| const commandExitCode = await new Promise((resolve, reject) => { | ||
| const child = childProcess.spawn(command, { shell: true, stdio: 'inherit' }); | ||
|
|
||
| child.on('error', reject); | ||
| child.on('close', code => { | ||
| resolve(code ?? 1); | ||
| }); | ||
| }); | ||
|
|
||
| if (commandExitCode !== 0) | ||
| throw new Error(`Command "${command}" exited with code ${commandExitCode}`); | ||
| } | ||
| } | ||
|
|
||
| exports.runCommands = runCommands; |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -51,8 +51,7 @@ module.exports = async function testFunctional (src, testingEnvironmentName, { n | |
| tests.unshift(SETUP_TESTS_GLOB); | ||
|
|
||
| const opts = { | ||
| reporter: 'mocha-reporter-spec-with-retries', | ||
| timeout: getTimeout(3 * 60 * 1000), | ||
| timeout: getTimeout(3 * 60 * 1000), | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. But without these
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. yes, but it doesn't look for me that we really use this feature. I can do this if you insist. |
||
| }; | ||
|
|
||
| if (process.env.RETRY_FAILED_TESTS === 'true') | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.