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_id
field. 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_id
field 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_id
field
when creating API URLs within the tests.
Ran unit tests.