mirror of
https://github.com/actions/toolkit
synced 2025-05-09 00:22:56 +00:00
Add test for large stdline output (#827)
* Add test for large stdline output * Format/Lint * Update stdlineoutput.js * Update stdlineoutput.js
This commit is contained in:
parent
c5035362ab
commit
8df94d9879
2 changed files with 30 additions and 0 deletions
|
@ -286,6 +286,31 @@ describe('@actions/exec', () => {
|
||||||
expect(stderrCalled).toBeTruthy()
|
expect(stderrCalled).toBeTruthy()
|
||||||
})
|
})
|
||||||
|
|
||||||
|
it('Handles large stdline', async () => {
|
||||||
|
const stdlinePath: string = path.join(
|
||||||
|
__dirname,
|
||||||
|
'scripts',
|
||||||
|
'stdlineoutput.js'
|
||||||
|
)
|
||||||
|
const nodePath: string = await io.which('node', true)
|
||||||
|
|
||||||
|
const _testExecOptions = getExecOptions()
|
||||||
|
let largeLine = ''
|
||||||
|
_testExecOptions.listeners = {
|
||||||
|
stdline: (line: string) => {
|
||||||
|
largeLine = line
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
const exitCode = await exec.exec(
|
||||||
|
`"${nodePath}"`,
|
||||||
|
[stdlinePath],
|
||||||
|
_testExecOptions
|
||||||
|
)
|
||||||
|
expect(exitCode).toBe(0)
|
||||||
|
expect(Buffer.byteLength(largeLine)).toEqual(2 ** 16 + 1)
|
||||||
|
})
|
||||||
|
|
||||||
it('Handles stdin shell', async () => {
|
it('Handles stdin shell', async () => {
|
||||||
let command: string
|
let command: string
|
||||||
if (IS_WINDOWS) {
|
if (IS_WINDOWS) {
|
||||||
|
|
5
packages/exec/__tests__/scripts/stdlineoutput.js
Normal file
5
packages/exec/__tests__/scripts/stdlineoutput.js
Normal file
|
@ -0,0 +1,5 @@
|
||||||
|
//Default highWaterMark for readable stream buffers us 64K (2^16)
|
||||||
|
//so we go over that to get more than a buffer's worth
|
||||||
|
const os = require('os')
|
||||||
|
|
||||||
|
process.stdout.write('a'.repeat(2**16 + 1) + os.EOL);
|
Loading…
Add table
Add a link
Reference in a new issue