From 4c3f9493af447bfd511c70e11477314365c73ac2 Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" <41898282+github-actions[bot]@users.noreply.github.com> Date: Tue, 12 May 2026 10:22:33 +0000 Subject: [PATCH] test(bash-lint): cover GateContext::PullRequest via pr-filter-tier1 fixture Add `pr-filter-tier1-agent.md` to the FIXTURES list so the bash-lint integration test compiles and shellchecks the gate step produced by `GateContext::PullRequest` in `src/compile/filter_ir.rs`. Add "Evaluate PR filters" to REQUIRED_STEP_DISPLAY_NAMES to ensure the generator is always exercised; the test will now fail loudly if the pr-filter fixtures are removed or the display name changes. Previously only the pipeline-completion variant ("Evaluate pipeline filters") was tracked; the PR-trigger variant shared the same generator but had no coverage guarantee. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- tests/bash_lint_tests.rs | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tests/bash_lint_tests.rs b/tests/bash_lint_tests.rs index ded51c0d..0db90bfa 100644 --- a/tests/bash_lint_tests.rs +++ b/tests/bash_lint_tests.rs @@ -76,6 +76,7 @@ const FIXTURES: &[&str] = &[ "azure-devops-mcp-agent.md", "pipeline-trigger-agent.md", "pipeline-filter-agent.md", + "pr-filter-tier1-agent.md", "runtime-coverage-agent.md", "runtime-coverage-1es-agent.md", ]; @@ -102,6 +103,7 @@ const REQUIRED_STEP_DISPLAY_NAMES: &[&str] = &[ "Initialize empty agent memory (clearMemory=true)", "Generate GITHUB_PATH file", // src/compile/common.rs (AWF path step) "Evaluate pipeline filters", // src/compile/extensions/trigger_filters.rs + src/compile/filter_ir.rs + "Evaluate PR filters", // src/compile/filter_ir.rs (GateContext::PullRequest) ]; fn ado_aw_binary() -> PathBuf {