• 
      

    Fix a unit test regression from the diff comment fragments cache change.

    Review Request #12316 — Created May 31, 2022 and submitted — Latest diff uploaded

    Information

    Review Board
    release-5.0.x

    Reviewers

    The unit test introduced in 5bbcc128c for altering comment diff fragment
    caching behavior on errors ended up being unstable. The Django function
    we used to generate the cache header, patch_cache_control, generates a
    dictionary of items and then turns it into a single string. This means
    that the order of items is not in a stable sort order, causing our
    checks to sometimes fail.

    To fix this, we now split the Cache-Control header value, convert to a
    set, and compare the items within. This avoids any sorting issues
    going forward.

    Unit tests pass.

    Commits

    Files