Skip to content

test: fix vacuous/incorrect indentation tests and remove 3 low-value yaml-only tests in compiler_tests#971

Draft
github-actions[bot] wants to merge 1 commit into
mainfrom
test-reducer-compiler-tests-cycle3-1b2af6c7ea381e42
Draft

test: fix vacuous/incorrect indentation tests and remove 3 low-value yaml-only tests in compiler_tests#971
github-actions[bot] wants to merge 1 commit into
mainfrom
test-reducer-compiler-tests-cycle3-1b2af6c7ea381e42

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Test Suite Reduction: tests/compiler_tests.rs

What was wrong

  • test_debug_pipeline_probe_step_indentation_standalone: The for loop made no assertion when the displayName line was absent — the test would silently pass even if the probe step was removed from the output entirely. Additionally, the search string used displayName: "Verify MCP backends" (double-quoted) but the compiled YAML emits the value unquoted (displayName: Verify MCP backends), so the line was never found. The expected indentation constant was also stale: 8 instead of the actual 4.

  • test_debug_pipeline_probe_step_indentation_1es: Same two bugs. Expected indent was 18; actual output uses 12.

  • test_standalone_complete_compiled_output_is_valid_yaml: Sole body was assert_valid_yaml(compile_fixture("complete-agent.md"), ...) — a single assertion on a fixture that is already compiled and more thoroughly checked by test_standalone_complete_agent_has_setup_and_teardown_jobs.

  • test_pr_filter_tier1_compiled_output_is_valid_yaml: Same pattern — only asserted YAML validity; test_pr_filter_tier1_has_evaluator_gate already compiles the same fixture with richer assertions.

  • test_pr_filter_tier2_compiled_output_is_valid_yaml: Same pattern.

Changes

Test Action Reason
test_debug_pipeline_probe_step_indentation_standalone Rewritten Vacuous loop + wrong search string + stale indent constant (8→4)
test_debug_pipeline_probe_step_indentation_1es Rewritten Same bugs; stale constant 18→12
test_standalone_complete_compiled_output_is_valid_yaml Removed assert_valid_yaml merged into companion test
test_pr_filter_tier1_compiled_output_is_valid_yaml Removed assert_valid_yaml merged into companion test
test_pr_filter_tier2_compiled_output_is_valid_yaml Removed assert_valid_yaml merged into companion test

Verification

  • cargo test: 129 passed, 0 failed
  • cargo clippy --all-targets --all-features: no warnings ✅

Warning

Firewall blocked 2 domains

The following domains were blocked by the firewall during workflow execution:

  • spsprodeus21.vssps.visualstudio.com
  • spsprodweu4.vssps.visualstudio.com

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "spsprodeus21.vssps.visualstudio.com"
    - "spsprodweu4.vssps.visualstudio.com"

See Network Configuration for more information.

Generated by Test Reducer · sonnet46 3.9M ·

…yaml-only tests in compiler_tests

- test_debug_pipeline_probe_step_indentation_standalone: the for-loop
  never asserted that the displayName line was actually found, so it
  silently passed even when the probe step was absent. Additionally the
  search string used escaped double-quotes ("Verify MCP backends") but
  the compiled YAML emits the value unquoted, and the expected indent
  constant was 8 but the actual output uses 4 spaces. Rewrote to use
  find()+expect() with the correct search string and indent.

- test_debug_pipeline_probe_step_indentation_1es: same two bugs.
  Expected indent was 18; actual is 12. Fixed search string and constant.

- test_standalone_complete_compiled_output_is_valid_yaml: sole body was
  assert_valid_yaml — merged into test_standalone_complete_agent_has_
  setup_and_teardown_jobs which already compiles the same fixture.

- test_pr_filter_tier1_compiled_output_is_valid_yaml: same pattern —
  merged assert_valid_yaml into test_pr_filter_tier1_has_evaluator_gate.

- test_pr_filter_tier2_compiled_output_is_valid_yaml: same pattern —
  merged assert_valid_yaml into test_pr_filter_tier2_has_extension_gate.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions github-actions Bot mentioned this pull request Jun 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants