Prevents a client adding messages without calling error or info on
TestNotifier by modifying the Lists returned by getInfoMessages and
getErrorMessages.
More immediately usefully, saves a client from getting
ConcurrentModificationExceptions when iterating over the List returned
by those methods if other threads are concurrently altering them, which
should stop WebhooksAcceptanceViaPostServeActionTest.firesASingleWebhookWhenRequested
(and others?) being flaky.
Submitter checklist
[x] Recommended: Join WireMock Slack to get any help in #help-contributing or a project-specific channel like #wiremock-java
[x] The PR request is well described and justified, including the body and the references
[x] The PR title represents the desired changelog entry
[x] The repository's code style is followed (see the contributing guide)
[ ] Test coverage that demonstrates that the change works as expected
[ ] For new features, there's necessary documentation in this pull request or in a subsequent PR to wiremock.org
Prevents a client adding messages without calling
error
orinfo
onTestNotifier
by modifying the Lists returned bygetInfoMessages
andgetErrorMessages
.More immediately usefully, saves a client from getting
ConcurrentModificationException
s when iterating over the List returned by those methods if other threads are concurrently altering them, which should stopWebhooksAcceptanceViaPostServeActionTest.firesASingleWebhookWhenRequested
(and others?) being flaky.Submitter checklist
#help-contributing
or a project-specific channel like#wiremock-java