#17758: Do not depend on dict order in test suite
---------------------------------+------------------------------------
     Reporter:  lrekucki         |                    Owner:  Alex
         Type:  Bug              |                   Status:  new
    Component:  Core (Other)     |                  Version:  1.3
     Severity:  Release blocker  |               Resolution:
     Keywords:                   |             Triage Stage:  Accepted
    Has patch:  1                |      Needs documentation:  0
  Needs tests:  0                |  Patch needs improvement:  0
Easy pickings:  0                |                    UI/UX:  0
---------------------------------+------------------------------------

Comment (by lrekucki):

 Replying to [comment:24 aaugustin]:
 > The following tests still fail when hash randomization is enabled:
 > - `fixtures`, `fixtures_regress`, `m2m_through_regress`: they compare
 serialised JSON and XML representations. Possible solution: add
 `assertJSONEqual` to Django's `TestCase`.

 Uhmm, didn't I fix that already? :P -
 
https://github.com/lqc/django_old/commit/a9b52c101c2f9e902728c2816767971d86718f94

 > - `admin_filters`, `admin_views`: they compare query strings, whose
 order is no longer deterministic. Possible solution: add
 `assertQueryStringEqual` to Django's `TestCase`.
 
[https://github.com/django/django/blob/195bc37f1d251eb7b5bc4d5f916f3b08745a504f/tests/regressiontests/httpwrappers/tests.py#L152
 This technique] also works but it isn't practical.


 
https://github.com/lqc/django_old/commit/7fdeaa52db50de5bafaac4dc57f5c731c43b4fe0
 
https://github.com/lqc/django_old/commit/7ef7420ba051a981f9f7bd888fd6b5f65eecbf31

 > - `forms`: `<span class="foo bar">` and `<span class="bar foo">` are
 equivalent, but `assertHTMLEquals` doesn't knows.

 Ok, this one I don't have.

 > - `feedgenerator`: this test looks for a hard coded string in XML, which
 depends on attributes order.

 
https://github.com/lqc/django_old/commit/d16c3248a03e6785e6cea9187f906b81c85d6c89

 I guess my pull request against the old repo got messed up
 (https://github.com/django/django-old/pull/128), so I'll rebase the
 mentioned changes against the new repo.

-- 
Ticket URL: <https://code.djangoproject.com/ticket/17758#comment:25>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to django-updates@googlegroups.com.
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to