New package handling -- stacked with previous PR tests #108

adamm · 2026-01-12T15:13:31+01:00

adamm commented

2026-01-12 15:13:31 +01:00

Depends on #106

Depends on https://src.opensuse.org/git-workflow/autogits/pulls/106

adamm added 21 commits 2026-01-12 15:13:32 +01:00

pr: fix unit tests 362e481a09

add PRProcessor tests 37c9cc7a57

pr: repo_check unit tests 09001ce01b

pr: add error handling unit tests b7f5c97de1

pr: move common test helpers to dedicated area 2f18adaa67

pr: add tests for RebaseAndSkipSubmoduleCommits e8738c9585

pr: add some tests for UpdatePrjGitPR 5f5e7d98b5

pr: add test cases for PRProcessor corner cases 86a7fd072e

- Add scenarios for closed/merged project PRs that trigger
   submodule checks and downstream PR updates (manual merge vs close).
- Test the "Consistency check" logic where submodules are reset
  if they don't match the PR set.
- Test the "superfluous PR" check (no-op PRs that should be closed).

pr: test verifyRepositoryConfiguration 4f132ec154

pr: test PRProcessor that is triggered by webhook abf8aa58fc

- PullRequestWebhookEvent: Verified that PR events correctly
  trigger processing with all necessary Gitea and Git mocks.
- IssueCommentWebhookEvent: Verified that issue comment events
  (which Gitea often uses for PR comments) are handled correctly.
- Recursion Limit: Verified that the recursion protection logic
  correctly terminates and cleans up when the limit is reached.
- Invalid Data Format: Verified that non-event data types return
  appropriate errors.

pr: revive PRProcessor sync tests e806d6ad0d

- Uncomment and fix the existing tests for `synchronized` actions.
- Ensure it uses the new `PullRequestProcessor` interface and mocked dependencies.

pr: fix PR lists to check packages not just project PRs c866303696

Also,
- Add simple unit tests to verify mapping of `models.StateType`
  to internal event strings.
- Verify it correctly wraps `ProcesPullRequest` and handles panics
  via the deferred recovery block.
- Add tests for scenarios where `GetRecentPullRequests` fails.
- Verify the random sleep interval logic (can be tested by mocking
  `time.Sleep` if refactored, or verifying behavior with interval=0).

pr: Add additional unit tests c05fa236d1

- Add a test case specifically verifying that `Gitea.SetLabels`
  is called with `staging/Auto` when a *new* project PR is created
  for submodules.
- Verify `PrjGitDescription` and `SetSubmodulesToMatchPRSet` behave
  correctly when a single `PRSet` contains 5+ different package
  repositories.

pr: move interfaces and mocks to parent package

go-generate-check / go-generate-check (pull_request) Failing after 22s

Details

18f7ed658a

pr: interfaces moved to main package

go-generate-check / go-generate-check (pull_request) Successful in 7s

Details

f959684540

wip: process issues 39b8f6ad55

pr: implement first part of issue processing 88528586fe

pr: merge new package a445bd8236

pr: fix more unit tests 39928c405a

common: more unit tests fixes 79da6cf262

TZ needs to be defined, otherwise it was assumed to be local which
then resulted in unpredictable commit hashes. We define it to UTC
for unit tests

PR have state "open" not "opened"

pr: make sure new repos have fork/parent relationship

go-generate-check / go-generate-check (pull_request) Successful in 23s

Details

b689aafa7f

If new target repo is "reparented", it will have correct relationship
here. Otherwise PR creation will fail

adamm added 2 commits 2026-01-18 23:33:21 +01:00

pr: new package handling 74e8d83bfa

pr: fix tests

go-generate-check / go-generate-check (pull_request) Successful in 25s

Details

a0d554caaf

go-generate-check / go-generate-check (pull_request) Successful in 25s

Details

This pull request can be merged automatically.

This branch is out-of-date with the base branch

You are not authorized to merge this pull request.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin new-packages:new-packages

git checkout new-packages

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: git-workflow/autogits#108