Use display_id when forming URLs in search tests.
Review Request #12016 — Created Jan. 27, 2022 and submitted — Latest diff uploaded
The search tests have a bunch of test cases that were querying URLs
based on thelocal_idfield. For some of these, the test is checking
that a review request on the global site does not show up when searching
within a local site. In these cases, thelocal_idfield can beNone,
which now causes a reverse lookup error (before it ended up just
sticking "None" in for the ID, which used to give us a URL but didn't
make sense).I've changed all the test cases to instead use the
display_idfield
when creating API URLs within the tests.
Ran unit tests.