Fix regressions from the json_dumps sorting change.

Review Request #10484 — Created April 1, 2019 and submitted

Information

Review Board
release-4.0.x

Reviewers

The recent change to json_dumps in Djblets to sort keys broke a couple
of unit tests in Review Board, which hard-coded results using the old
ordering. This is a simple change that updates the tests to compare
against the sorted payloads.

Unit tests pass on Django 1.6 and 1.11 (with other in-progress changes).

Summary ID
Fix regressions from the json_dumps sorting change.
The recent change to `json_dumps` in Djblets to sort keys broke a couple of unit tests in Review Board, which hard-coded results using the old ordering. This is a simple change that updates the tests to compare against the sorted payloads.
96eb0aa577d833ead48f3d27bd91cb38f5f1dc48
Description From Last Updated

E501 line too long (137 > 79 characters)

reviewbotreviewbot

E501 line too long (120 > 79 characters)

reviewbotreviewbot

E501 line too long (120 > 79 characters)

reviewbotreviewbot
Checks run (1 failed, 1 succeeded)
flake8 failed.
JSHint passed.

flake8

david
  1. Ship It!
  2. 
      
chipx86
Review request changed
Status:
Completed
Change Summary:
Pushed to release-4.0.x (7f99bd1)