Compare commits
5 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| 88d212f82b | |||
| b0f0516e10 | |||
| 14aaaaa7de | |||
| 6cf5fbc523 | |||
| f00e913aa0 |
@@ -484,7 +484,7 @@ describe('action', () => {
|
||||
|
||||
expect(setFailedMock).toHaveBeenCalledWith(
|
||||
new Error(
|
||||
'Too many subjects specified (1025). The maximum number of subjects is 1024.'
|
||||
'Too many subjects specified (>1024). The maximum number of subjects is 1024.'
|
||||
)
|
||||
)
|
||||
})
|
||||
|
||||
+12
-8
@@ -94,14 +94,18 @@ const getSubjectFromPath = async (
|
||||
// Expand the globbed paths to a list of actual paths
|
||||
const paths = await glob.create(subjectPaths).then(async g => g.glob())
|
||||
|
||||
// Filter path list to just the files (not directories)
|
||||
const stats = await Promise.all(paths.map(async p => fs.stat(p)))
|
||||
const files = paths.filter((_, i) => stats[i].isFile())
|
||||
|
||||
if (files.length > MAX_SUBJECT_COUNT) {
|
||||
throw new Error(
|
||||
`Too many subjects specified (${files.length}). The maximum number of subjects is ${MAX_SUBJECT_COUNT}.`
|
||||
)
|
||||
// Filter path list to just the files (not directories), enforcing the maximum
|
||||
const files: string[] = []
|
||||
for (const p of paths) {
|
||||
const stat = await fs.stat(p)
|
||||
if (stat.isFile()) {
|
||||
if (files.length >= MAX_SUBJECT_COUNT) {
|
||||
throw new Error(
|
||||
`Too many subjects specified (>${MAX_SUBJECT_COUNT}). The maximum number of subjects is ${MAX_SUBJECT_COUNT}.`
|
||||
)
|
||||
}
|
||||
files.push(p)
|
||||
}
|
||||
}
|
||||
|
||||
for (const file of files) {
|
||||
|
||||
Reference in New Issue
Block a user