stable
Clone or download
Read-only
refactor: Stop relying on the the god Factory class in PullRequestUpdatedNotificationToProcessBuilder
Part of story #32316 Update comments No functionnal change expected Note: I also take the opportunity to remove the mock of the `UserManager` class Change-Id: I6857b794a427d99d8d3fa7f5ddddb7440280d9b0
Modified Files
Name | ||||
---|---|---|---|---|
M | plugins/pullrequest/include/PullRequest/BranchUpdate/PullRequestUpdatedNotificationToProcessBuilder.php | +63 | −107 | Go to diff View file |
M | plugins/pullrequest/include/PullRequest/Notification/PullRequestNotificationSupport.php | +2 | −2 | Go to diff View file |
M | plugins/pullrequest/tests/unit/BranchUpdate/PullRequestUpdatedNotificationToProcessBuilderTest.php | +56 | −99 | Go to diff View file |
M | plugins/pullrequest/tests/unit/Tests/Builders/PullRequestTestBuilder.php | +6 | −0 | Go to diff View file |