git-workflow/autogits
SHA256
21
2
Fork 14
Code Issues 48 Pull Requests 15 Actions Projects 2 Activity

New package handling #108

Open
adamm wants to merge 25 commits from new-packages into main
pull from: new-packages
merge into: git-workflow:main
Conversation 0 Commits 25 Files Changed 41 +1974 -179
adamm commented 2026-01-12 15:13:31 +01:00
Owner
Copy Link

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
Some checks failed
go-generate-check / go-generate-check (pull_request) Failing after 22s
Details
18f7ed658a
pr: interfaces moved to main package
All checks were successful
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
All checks were successful
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
All checks were successful
go-generate-check / go-generate-check (pull_request) Successful in 25s
Details
a0d554caaf
adamm added 3 commits 2026-01-26 11:57:32 +01:00
pr: move functions around 7ec6420e20
pr: test updated issue ref 046c28aac2 
If issue is a SHA ref, and then it's updated, we need to make
sure that the branch is also updated.
common: add test for directory listing
All checks were successful
go-generate-check / go-generate-check (pull_request) Successful in 9s
Details
4c9c2a20f3
adamm added 3 commits 2026-01-27 14:07:55 +01:00
common: add test for cloning hashes 17a266121b
pr: use top 20 commits as base, if available d00f49ae0b
pr: fix tests
All checks were successful
go-generate-check / go-generate-check (pull_request) Successful in 8s
Details
80d4e83f30
adamm added 2 commits 2026-01-28 09:16:26 +01:00
pr: Only update issues if they are actually open 6b4be8e8e5
pr: no need to create PR if already exists.
All checks were successful
go-generate-check / go-generate-check (pull_request) Successful in 8s
Details
9922b05942
adamm changed title from New package handling -- stacked with previous PR tests to New package handling 2026-01-28 09:20:32 +01:00
adamm force-pushed new-packages from 9922b05942 to 98ac879aa6 2026-01-28 11:20:37 +01:00 Compare
adamm added 1 commit 2026-01-28 16:12:49 +01:00
pr: close associated PR if issue closed.
All checks were successful
go-generate-check / go-generate-check (pull_request) Successful in 8s
Details
1fd14b1904
adamm added 1 commit 2026-01-29 10:21:56 +01:00
pr: small refactor
All checks were successful
go-generate-check / go-generate-check (pull_request) Successful in 9s
Details
39ca3fcf42
adamm added 3 commits 2026-01-29 18:06:55 +01:00
pr: test coverage ca162dcbe5
pr: test coverage 9583e41117
pr: small refactor
All checks were successful
go-generate-check / go-generate-check (pull_request) Successful in 8s
Details
a010537a14
adamm force-pushed new-packages from a010537a14 to 4790ee3096 2026-01-30 16:44:20 +01:00 Compare
adamm added 2 commits 2026-01-31 10:18:55 +01:00
test: refactor loggin in unit tests 79e5345ac6 
Use test logger when running under a test.
common: replace legacy logger with standard impl
All checks were successful
go-generate-check / go-generate-check (pull_request) Successful in 27s
Details
f47da62481
adamm added 1 commit 2026-02-01 20:35:45 +01:00
pr: Always allow maintainer edits in new packages
All checks were successful
go-generate-check / go-generate-check (pull_request) Successful in 25s
Details
9b3d6e9a66
adamm requested review from jzerebecki 2026-02-04 14:21:10 +01:00
All checks were successful
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.
Reviewers
No Reviewers
jzerebecki
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
M0ses adamm adamm_super adrianSuSE andriinikitin atartamo bigironman dancermak darix dirkmueller dmach eroca jdsn jzerebecki mstrigl oertel smithfarm
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: git-workflow/autogits#108
Delete Branch "new-packages"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?