• 
      

    Use display_id when forming URLs in search tests.

    Review Request #12016 — Created Jan. 27, 2022 and submitted — Latest diff uploaded

    Information

    Review Board
    master

    Reviewers

    The search tests have a bunch of test cases that were querying URLs
    based on the local_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, the local_id field can be None,
    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.

    Commits

    Files