UpdatePullRequest() (Gitea mock) is never called here. This is not mocked in this instance and the other mocked function calls are all reading only, not updating anything (except for the…
Why no more panic()?
This one should probably be reviewed as one chunk instead of per-commit. If there are issues, I'll add new commits that fix these issues in review on top and I will not be rewriting history.
Thanks for the review. Quite a bit of functionality for PR processing happens in the /common code which have actual tests there. In the code here we have either a lot of git operations or it's…
This looks like a smoke test that different message formats either do something or return an error. The last test here expects an error if the message format is unknown.
This is LLM comment why prset is unused. It's not used because it's re-created internally from the PR body text. So this prset here is not returned by any of the mocks or used by them.
LLM…