actions/toolkit
1
0
Fork 0
mirror of https://git.mirrors.martin98.com/https://github.com/actions/toolkit synced 2026-03-20 15:12:35 +08:00
Code Issues Packages Projects Releases Wiki Activity

Fix test timeouts (#235)

Browse Source 
* Fix test timeouts
This commit is contained in:
Ross Brodbeck
2019-12-03 13:55:39 -05:00
committed by GitHub
parent 5c894298f2
commit d98e55434d
1 changed files with 3 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
packages/exec/__tests__/exec.test.ts
View File

@@ -287,8 +287,6 @@ describe('@actions/exec', () => {
}) })
it('Handles child process holding streams open', async function() { it('Handles child process holding streams open', async function() {
// this was timing out on some slower hosted macOS runs at default 5s
jest.setTimeout(10000)
const semaphorePath = path.join( const semaphorePath = path.join(
getTestTemp(), getTestTemp(),
'child-process-semaphore.txt' 'child-process-semaphore.txt'
@@ -332,11 +330,9 @@ describe('@actions/exec', () => {
).toBe(1) ).toBe(1)
fs.unlinkSync(semaphorePath) fs.unlinkSync(semaphorePath)
}) }, 10000) // this was timing out on some slower hosted macOS runs at default 5s
it('Handles child process holding streams open and non-zero exit code', async function() { it('Handles child process holding streams open and non-zero exit code', async function() {
// this was timing out on some slower hosted macOS runs at default 5s
jest.setTimeout(10000)
const semaphorePath = path.join( const semaphorePath = path.join(
getTestTemp(), getTestTemp(),
'child-process-semaphore.txt' 'child-process-semaphore.txt'
@@ -388,7 +384,7 @@ describe('@actions/exec', () => {
).toBe(1) ).toBe(1)
fs.unlinkSync(semaphorePath) fs.unlinkSync(semaphorePath)
}) }, 10000) // this was timing out on some slower hosted macOS runs at default 5s
it('Handles child process holding streams open and stderr', async function() { it('Handles child process holding streams open and stderr', async function() {
const semaphorePath = path.join( const semaphorePath = path.join(
@@ -665,7 +661,7 @@ describe('@actions/exec', () => {
`[command]"${exePath}" myarg1 myarg2` `[command]"${exePath}" myarg1 myarg2`
) )
expect(output.trim()).toBe("args[0]: 'myarg1'\r\nargs[1]: 'myarg2'") expect(output.trim()).toBe("args[0]: 'myarg1'\r\nargs[1]: 'myarg2'")
}) }, 20000) // slower windows runs timeout, so upping timeout to 20s (from default of 5s)
it('execs .cmd with a space and with verbatim args (Windows)', async () => { it('execs .cmd with a space and with verbatim args (Windows)', async () => {
// this test validates the quoting that tool runner adds around the script path. // this test validates the quoting that tool runner adds around the script path.