Skip to content

Commit 5d9adab

Browse files
radicalCopilot
andcommitted
Expand audit selector rules to cover gaps surfaced by 14-day PR replay
Replayed the 161 PRs merged to main between 2026-05-23 and 2026-06-06 through the audit selector. v1 (the prior file) resolved 89/161 PRs to 'selective' and dropped 26/161 to 'fallback_unmatched' (any unmatched file forces RunAll). After this change all 161 PRs classify cleanly: 18 skip, 32 trigger_all, 111 selective, 0 fallback. Additions: * ignorePaths — files that never need a test gate but were forcing fallback today: .vscode/**, .mcp.json, localhive.{sh,ps1}, .github/{extensions,policies,aw}/**, .agents/** (CLI agent skill content, consumed only by humans/agents), eng/scripts/debug-*.{sh,ps1} (developer-loop helpers), eng/scripts/cli-starter-validation*.ps1 (consumed by the always-on cli_starter_validation_windows job, which isn't conditional-selector-gated). * sourceToTestMappings — scripts and infra files that have a specific test home but weren't wired up: eng/Publishing.props, eng/Signing.props, eng/scripts/{pack,stage,verify,validate}-cli-* and tools/TypeScriptApiCompat/** all map to Infrastructure.Tests (its Pipelines/ and PowerShellScripts/ folders test these scripts). eng/scripts/get-aspire-cli{,-pr}.{sh,ps1}, eng/scripts/verify-cli-archive.ps1, eng/homebrew/** and eng/winget/** map to Aspire.Acquisition.Tests (the script-suite that exercises them). eng/scripts/update-aspire- skills-bundle.ps1 + verify-aspire-skills-bundle.ps1 map to Aspire.Cli.Tests (AspireSkillsBundleTests). Two self-mappings for tests directories without a 'tests/Aspire.*.Tests/**' shape: tests/Infrastructure.Tests/** and tests/Aspire.Hosting.CodeGeneration.TypeScript.JsTests/**. * polyglot.triggerPaths — expanded beyond the workflow file itself to cover the polyglot fixture tree (tests/PolyglotAppHosts/**) and the per-language Aspire.Hosting.{JavaScript,TypeScript,Python,Go,Java,Rust} + CodeGeneration* projects. Without these, polyglot-impacting PRs (#17545, #17499, #17419, #17400, #17382) all fell into RunAll. Local verification: 251 TestSelector/PowerShellScripts/ConditionalSelection tests still pass. The CI-trigger-pattern coverage test (which loads this file from disk) is unchanged because we only added to ignorePaths and sourceToTestMappings — no existing entries were removed or narrowed. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 478d56c commit 5d9adab

1 file changed

Lines changed: 97 additions & 2 deletions

File tree

eng/scripts/test-selection-rules.audit.json

Lines changed: 97 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,19 @@
1313
".github/actions/**",
1414
".github/instructions/**",
1515
".github/skills/**",
16+
".github/extensions/**",
17+
".github/policies/**",
18+
".github/aw/**",
19+
".agents/**",
20+
".vscode/**",
21+
".mcp.json",
22+
"localhive.sh",
23+
"localhive.ps1",
1624
"tests/agent-scenarios/**",
17-
"eng/scripts/test-selection-rules.schema.json"
25+
"eng/scripts/test-selection-rules.schema.json",
26+
"eng/scripts/debug-*.ps1",
27+
"eng/scripts/debug-*.sh",
28+
"eng/scripts/cli-starter-validation*.ps1"
1829
],
1930

2031
"triggerAllPaths": [
@@ -78,6 +89,82 @@
7889
{
7990
"source": "global.json",
8091
"test": "tests/Aspire.Templates.Tests/Aspire.Templates.Tests.csproj"
92+
},
93+
94+
{
95+
"source": "tests/Infrastructure.Tests/**",
96+
"test": "tests/Infrastructure.Tests/Infrastructure.Tests.csproj"
97+
},
98+
{
99+
"source": "tests/Aspire.Hosting.CodeGeneration.TypeScript.JsTests/**",
100+
"test": "tests/Aspire.Hosting.CodeGeneration.TypeScript.JsTests/Aspire.Hosting.CodeGeneration.TypeScript.JsTests.csproj"
101+
},
102+
103+
{
104+
"source": "eng/Publishing.props",
105+
"test": "tests/Infrastructure.Tests/Infrastructure.Tests.csproj"
106+
},
107+
{
108+
"source": "eng/Signing.props",
109+
"test": "tests/Infrastructure.Tests/Infrastructure.Tests.csproj"
110+
},
111+
{
112+
"source": "eng/scripts/pack-cli-npm-package.ps1",
113+
"test": "tests/Infrastructure.Tests/Infrastructure.Tests.csproj"
114+
},
115+
{
116+
"source": "eng/scripts/stage-native-cli-tool-packages.ps1",
117+
"test": "tests/Infrastructure.Tests/Infrastructure.Tests.csproj"
118+
},
119+
{
120+
"source": "eng/scripts/verify-cli-npm-package.ps1",
121+
"test": "tests/Infrastructure.Tests/Infrastructure.Tests.csproj"
122+
},
123+
{
124+
"source": "eng/scripts/validate-cli-symbols.ps1",
125+
"test": "tests/Infrastructure.Tests/Infrastructure.Tests.csproj"
126+
},
127+
{
128+
"source": "tools/TypeScriptApiCompat/**",
129+
"test": "tests/Infrastructure.Tests/Infrastructure.Tests.csproj"
130+
},
131+
132+
{
133+
"source": "eng/scripts/get-aspire-cli.ps1",
134+
"test": "tests/Aspire.Acquisition.Tests/Aspire.Acquisition.Tests.csproj"
135+
},
136+
{
137+
"source": "eng/scripts/get-aspire-cli.sh",
138+
"test": "tests/Aspire.Acquisition.Tests/Aspire.Acquisition.Tests.csproj"
139+
},
140+
{
141+
"source": "eng/scripts/get-aspire-cli-pr.ps1",
142+
"test": "tests/Aspire.Acquisition.Tests/Aspire.Acquisition.Tests.csproj"
143+
},
144+
{
145+
"source": "eng/scripts/get-aspire-cli-pr.sh",
146+
"test": "tests/Aspire.Acquisition.Tests/Aspire.Acquisition.Tests.csproj"
147+
},
148+
{
149+
"source": "eng/scripts/verify-cli-archive.ps1",
150+
"test": "tests/Aspire.Acquisition.Tests/Aspire.Acquisition.Tests.csproj"
151+
},
152+
{
153+
"source": "eng/homebrew/**",
154+
"test": "tests/Aspire.Acquisition.Tests/Aspire.Acquisition.Tests.csproj"
155+
},
156+
{
157+
"source": "eng/winget/**",
158+
"test": "tests/Aspire.Acquisition.Tests/Aspire.Acquisition.Tests.csproj"
159+
},
160+
161+
{
162+
"source": "eng/scripts/update-aspire-skills-bundle.ps1",
163+
"test": "tests/Aspire.Cli.Tests/Aspire.Cli.Tests.csproj"
164+
},
165+
{
166+
"source": "eng/scripts/verify-aspire-skills-bundle.ps1",
167+
"test": "tests/Aspire.Cli.Tests/Aspire.Cli.Tests.csproj"
81168
}
82169
],
83170

@@ -106,7 +193,15 @@
106193
"description": "Polyglot SDK validation tests",
107194
"triggerPaths": [
108195
".github/workflows/polyglot-validation/**",
109-
".github/workflows/polyglot-validation.yml"
196+
".github/workflows/polyglot-validation.yml",
197+
"tests/PolyglotAppHosts/**",
198+
"src/Aspire.Hosting.JavaScript/**",
199+
"src/Aspire.Hosting.TypeScript/**",
200+
"src/Aspire.Hosting.Python/**",
201+
"src/Aspire.Hosting.Go/**",
202+
"src/Aspire.Hosting.Java/**",
203+
"src/Aspire.Hosting.Rust/**",
204+
"src/Aspire.Hosting.CodeGeneration*/**"
110205
]
111206
},
112207

0 commit comments

Comments
 (0)