This commit changes the name of two recently introduced methods in the
MockRestRequestMatchers
class for header and queryParam. These have
been found to cause false negatives in user tests, due to the new
overload taking precedence in some cases.
Namely, using a Matcher
factory method which can apply to both List
and String
will cause the compiler to select the newest list overload,
by instantiating a Matcher<Object>
.
This can cause false negatives in user tests, failing tests that used
to pass because the Matcher previously applied to the first String in
the header or queryParam value list. For instance, equalsTo("a")
.
The new overloads are recent enough and this has enough potential to
cause an arbitrary number of user tests to fail that we break the API
to eliminate the ambiguity, by renaming the methods with a *List
suffix.
Closes gh-30220 Closes gh-30238 See gh-29953 See gh-28660
Comment From: simonbasle
Backport issue: #30235