fix: the test command's --tags option now includes matching CheckGroups

sorccu · sorccu · commit d52acfbe7762 · 2025-03-25T19:14:07.000+09:00
This is the intended behavior, but an oversight in the code effectively
disabled it.
diff --git a/packages/cli/src/commands/test.ts b/packages/cli/src/commands/test.ts
@@ -203,7 +203,7 @@ export default class Test extends AuthCommand {
         const checkGroup = this.getCheckGroup(project, check)
         if (checkGroup) {
           const checkGroupTags = checkGroup.tags ?? []
-          tags.concat(checkGroupTags)
+          tags.push(...checkGroupTags)
         }
         return filterByTags(targetTags?.map((tags: string) => tags.split(',')) ?? [], tags)
       })

Original file line number	Diff line number	Diff line change
`@@ -203,7 +203,7 @@ export default class Test extends AuthCommand {`
`203`	`203`	`const checkGroup = this.getCheckGroup(project, check)`
`204`	`204`	`if (checkGroup) {`
`205`	`205`	`const checkGroupTags = checkGroup.tags ?? []`
`206`		`- tags.concat(checkGroupTags)`
	`206`	`+ tags.push(...checkGroupTags)`
`207`	`207`	`}`
`208`	`208`	`return filterByTags(targetTags?.map((tags: string) => tags.split(',')) ?? [], tags)`
`209`	`209`	`})`