Skip to content

fix(tests): Stop expecting an active repo to be reported as reactivated - #125350

Merged
vaind merged 1 commit into
masterfrom
ivan/fix/test-adopt-existing-row-reactivated
Sep 23, 2026
Merged

vaind merged 1 commit into
masterfrom
ivan/fix/test-adopt-existing-row-reactivated

Conversation

@vaind

@vaind vaind commented Sep 23, 2026

Copy link
Copy Markdown
Contributor

test_create_repositories__does_not_adopt_over_an_existing_row fails on master.

Two PRs landed on the same day without CI ever running them together. #125158 changed create_repositories so a repository that is already active gets its config refreshed but is no longer reported as "reactivated". The test added in #123820 was written before that and still expected the active repository in the reactivated list.

The test now expects an empty reactivated list. What it's actually there to check, that a hidden repository with the same external id is left alone, is unchanged.

#125158 changed create_repositories to refresh an already
active repository without reporting it as reactivated. The test added in
#123820 was written against the old behavior and the two
landed without a CI run on their combination.
@vaind
vaind requested a review from wedamija September 23, 2026 18:51
@vaind
vaind enabled auto-merge (squash) September 23, 2026 18:55
@vaind
vaind merged commit 07db0df into master Sep 23, 2026
65 checks passed
@vaind
vaind deleted the ivan/fix/test-adopt-existing-row-reactivated branch September 23, 2026 18:59
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.

3 participants