Skip to content

test: remove duplicate test in explicit_push_trigger codemod - #1913

Draft
github-actions[bot] wants to merge 1 commit into
mainfrom
test-reducer/codemods-0006-duplicate-cc5e448d804b41f7
Draft

test: remove duplicate test in explicit_push_trigger codemod#1913
github-actions[bot] wants to merge 1 commit into
mainfrom
test-reducer/codemods-0006-duplicate-cc5e448d804b41f7

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Test Suite Reduction: src/compile/codemods/0006_explicit_push_trigger.rs

What was wrong

  • injects_all_branches_push_when_on_absent_in_old_source was a duplicate of fires_for_the_release_immediately_before_the_cutover: both use the same effective context (compiler_version: INTRODUCED_IN / 0.48.0, source_compiler_version: Some("0.48.0")old_ctx() resolves to exactly this), the same input ("name: x\ndescription: y\n"), and the same assertions (apply_codemod returns true, push_include equals ["*"]). The second test provides strictly more documentation value (doc comment explaining the release-window boundary), so it was kept.

Changes

Test Action Reason
injects_all_branches_push_when_on_absent_in_old_source Removed Exact duplicate of fires_for_the_release_immediately_before_the_cutover

Verification

  • cargo test: all 3179 tests pass ✅ (one fewer than before, matching the removed duplicate)
  • cargo clippy --all-targets --all-features: no warnings ✅

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • spsprodeus21.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"

See Network Configuration for more information.

Generated by Test Reducer · auto · 105.6 AIC · ⌖ 12.6 AIC · ⊞ 13.4K ·

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
2 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

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