actions/toolkit
1
0
Fork 0
mirror of https://git.mirrors.martin98.com/https://github.com/actions/toolkit synced 2026-01-03 14:42:56 +08:00
Code Issues Packages Projects Releases Wiki Activity

Concurrency has a min of 1

Browse Source
This commit is contained in:
Yang Cao 2025-01-08 18:14:04 +00:00
parent ede05b95d7
commit d4385a64a7
2 changed files with 9 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
packages/artifact/__tests__/config.test.ts
View File

@ -85,6 +85,14 @@ describe('uploadConcurrencyEnv', () => {
}).toThrow() }).toThrow()
}) })
it('should throw if ACTIONS_UPLOAD_CONCURRENCY is < 1', () => {
;(os.cpus as jest.Mock).mockReturnValue(new Array(4))
process.env.ACTIONS_UPLOAD_CONCURRENCY = '0'
expect(() => {
config.getConcurrency()
}).toThrow()
})
it('cannot go over currency cap when override value is greater', () => { it('cannot go over currency cap when override value is greater', () => {
;(os.cpus as jest.Mock).mockReturnValue(new Array(4)) ;(os.cpus as jest.Mock).mockReturnValue(new Array(4))
process.env.ACTIONS_UPLOAD_CONCURRENCY = '40' process.env.ACTIONS_UPLOAD_CONCURRENCY = '40'

2
packages/artifact/src/internal/shared/config.ts
View File

@ -61,7 +61,7 @@ export function getConcurrency(): number {
const concurrencyOverride = process.env['ACTIONS_UPLOAD_CONCURRENCY'] const concurrencyOverride = process.env['ACTIONS_UPLOAD_CONCURRENCY']
if (concurrencyOverride) { if (concurrencyOverride) {
const concurrency = parseInt(concurrencyOverride) const concurrency = parseInt(concurrencyOverride)
if (isNaN(concurrency)) { if (isNaN(concurrency) || concurrency < 1) {
throw new Error( throw new Error(
'Invalid value set for ACTIONS_UPLOAD_CONCURRENCY env variable' 'Invalid value set for ACTIONS_UPLOAD_CONCURRENCY env variable'
) )