Skip to content

test(integration): add AllTargetsFiles e2e test for BUG-015 - #313

Open
xytan0056 wants to merge 1 commit into
mainfrom
xytan0056/atf-integration-test
Open

test(integration): add AllTargetsFiles e2e test for BUG-015#313
xytan0056 wants to merge 1 commit into
mainfrom
xytan0056/atf-integration-test

Conversation

@xytan0056

@xytan0056 xytan0056 commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add a separate integration test suite exercising the AllTargetsFiles code path end-to-end against bazel-fixture PR #3, which changes .bazelrc (a configured all_targets_files trigger), adds pkg/printer, and deletes pkg/version in the same revision.
  • Verify that NEW and DELETED change types are preserved when the global trigger fires, while targets present in both revisions are promoted to CHANGED at distance 0.
  • Split integration env vars so each suite only receives its own SHAs/URLs.

Test Plan

CI

@xytan0056
xytan0056 requested review from a team as code owners September 11, 2026 15:05
@xytan0056
xytan0056 force-pushed the xytan0056/atf-integration-test branch 3 times, most recently from 0e63950 to 083e033 Compare September 11, 2026 15:43
Add a separate integration test suite that exercises the AllTargetsFiles
code path end-to-end against bazel-fixture PR #3, which changes .bazelrc
(a configured all_targets_files trigger), adds pkg/printer, and deletes
pkg/version in the same revision.

The test verifies that NEW and DELETED change types are preserved when
the global trigger fires, while targets present in both revisions are
promoted to CHANGED at distance 0.

Changes:
- integration/atf_test.go: dedicated test file with sha and PR subtests
- integration/BUILD.bazel: separate atf_test Bazel target, shared deps
- integration/testdata/tango-config.yaml.tmpl: add all_targets_files
- Makefile: split INTEGRATION_ENV/ATF_ENV, add test-integration-atf
@xytan0056
xytan0056 force-pushed the xytan0056/atf-integration-test branch from 083e033 to 0382afe Compare September 11, 2026 15:49
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.

2 participants