Skip to content

Use a migration test instead of a wrong test which populated the meta test repositories and fix a migration bug#36160

Merged
lunny merged 5 commits into
go-gitea:mainfrom
lunny:lunny/remove_wrong_test
Dec 17, 2025
Merged

Use a migration test instead of a wrong test which populated the meta test repositories and fix a migration bug#36160
lunny merged 5 commits into
go-gitea:mainfrom
lunny:lunny/remove_wrong_test

Conversation

@lunny
Copy link
Copy Markdown
Member

@lunny lunny commented Dec 14, 2025

The test TestGiteaUploadUpdateGitForPullRequest modified the shared meta test repositories directly, so this PR removes that test and replaces it with an integration test that migrates a real repository from gitea.com into a local test instance.

This PR also fixes a bug where pull-request migrations were not correctly syncing head branches to the database.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Dec 14, 2025
@lunny lunny changed the title Use a migration test instead of a wrong test which populated the meta test repositories Use a migration test instead of a wrong test which populated the meta test repositories and fix a migration bug Dec 14, 2025
@lunny lunny added the type/bug label Dec 14, 2025
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Dec 15, 2025
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Dec 17, 2025
@lunny lunny added the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Dec 17, 2025
@lunny lunny merged commit ebf9b4d into go-gitea:main Dec 17, 2025
23 checks passed
@GiteaBot GiteaBot added this to the 1.26.0 milestone Dec 17, 2025
@lunny lunny deleted the lunny/remove_wrong_test branch December 17, 2025 20:00
@GiteaBot GiteaBot removed the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Dec 17, 2025
zjjhot added a commit to zjjhot/gitea that referenced this pull request Dec 19, 2025
* giteaofficial/main:
  Add JSON linting (go-gitea#36192)
  Bump setup-node to v6, re-enable cache (go-gitea#36207)
  [skip ci] Updated translations via Crowdin
  Update chroma to v2.21.1 (go-gitea#36201)
  Disable dependabot automatic labels (go-gitea#36203)
  Bump astral-sh/setup-uv from 6 to 7 (go-gitea#36198)
  Front port changelog (go-gitea#36193)
  Bump dev-hanz-ops/install-gh-cli-action from 0.1.0 to 0.2.1 (go-gitea#36195)
  Bump aws-actions/configure-aws-credentials from 4 to 5 (go-gitea#36196)
  Bump docker/build-push-action from 5 to 6 (go-gitea#36197)
  Enable dependabot for actions (go-gitea#36191)
  Bump alpine to 3.23 (go-gitea#36185)
  Use gitRepo as parameter instead of repopath when invoking sign functions (go-gitea#36162)
  Fix bug when creating pull request (go-gitea#36166)
  Bump golangci-lint to 2.7.2, enable modernize stringsbuilder (go-gitea#36180)
  Use a migration test instead of a wrong test which populated the meta test repositories and fix a migration bug (go-gitea#36160)
  Update JS deps and eslint enhancements (go-gitea#36147)
  Add git.DIFF_RENAME_SIMILARITY_THRESHOLD option (go-gitea#36164)
  Fix OrgAssignment opts (go-gitea#36174)

# Conflicts:
#	models/user/user.go
@go-gitea go-gitea locked as resolved and limited conversation to collaborators Mar 18, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/bug type/testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants