Re: [Django] #29226: @modify_settings append/remove/prepend is affected by dictionary order

[Django]

#29226: @modify_settings append/remove/prepend is affected by dictionary order
-+-
 Reporter:  Manuel Kaufmann  |Owner:
 |  benjaoming
 Type:  Bug  |   Status:  closed
Component:  Testing framework|  Version:  2.0
 Severity:  Normal   |   Resolution:  fixed
 Keywords:  tests,   | Triage Stage:  Accepted
  modify_settings|
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Tim Graham ):

 * status:  assigned => closed
 * resolution:   => fixed


Comment:

 In [changeset:"08f788b169a30d26f50433083aca253a4e4031b2" 08f788b1]:
 {{{
 #!CommitTicketReference repository=""
 revision="08f788b169a30d26f50433083aca253a4e4031b2"
 Fixed #29226 -- Doc'd modify_settings() ordering considerations for Python
 < 3.6.
 }}}

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.ed8fb0e7f45b8809bb307ea707472a6d%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29226: @modify_settings append/remove/prepend is affected by dictionary order

[Django]

#29226: @modify_settings append/remove/prepend is affected by dictionary order
-+-
 Reporter:  Manuel Kaufmann  |Owner:
 |  benjaoming
 Type:  Bug  |   Status:  closed
Component:  Testing framework|  Version:  2.0
 Severity:  Normal   |   Resolution:  fixed
 Keywords:  tests,   | Triage Stage:  Accepted
  modify_settings|
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-

Comment (by Tim Graham ):

 In [changeset:"fb9c1f4b46623208ed8dd587a020ae5fc1b2a87e" fb9c1f4]:
 {{{
 #!CommitTicketReference repository=""
 revision="fb9c1f4b46623208ed8dd587a020ae5fc1b2a87e"
 [2.1.x] Fixed #29226 -- Doc'd modify_settings() ordering considerations
 for Python < 3.6.

 Backport of 08f788b169a30d26f50433083aca253a4e4031b2 from master
 }}}

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.e41d9867837492328f2345ace78eff8e%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29226: @modify_settings append/remove/prepend is affected by dictionary order

[Django]

#29226: @modify_settings append/remove/prepend is affected by dictionary order
-+-
 Reporter:  Manuel Kaufmann  |Owner:
 |  benjaoming
 Type:  Bug  |   Status:  assigned
Component:  Testing framework|  Version:  2.0
 Severity:  Normal   |   Resolution:
 Keywords:  tests,   | Triage Stage:  Accepted
  modify_settings|
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Tim Graham):

 * type:  Uncategorized => Bug


-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.8c902a7a5b0af9619a3b72131f250be6%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29226: @modify_settings append/remove/prepend is affected by dictionary order

[Django]

#29226: @modify_settings append/remove/prepend is affected by dictionary order
-+-
 Reporter:  Manuel Kaufmann  |Owner:
 |  benjaoming
 Type:  Uncategorized|   Status:  assigned
Component:  Testing framework|  Version:  2.0
 Severity:  Normal   |   Resolution:
 Keywords:  tests,   | Triage Stage:  Accepted
  modify_settings|
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Carlton Gibson):

 * needs_better_patch:  0 => 1


-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.ae242b85ba744c1416ac63d929632083%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29226: @modify_settings append/remove/prepend is affected by dictionary order

[Django]

#29226: @modify_settings append/remove/prepend is affected by dictionary order
-+-
 Reporter:  Manuel Kaufmann  |Owner:
 |  benjaoming
 Type:  Uncategorized|   Status:  assigned
Component:  Testing framework|  Version:  2.0
 Severity:  Normal   |   Resolution:
 Keywords:  tests,   | Triage Stage:  Accepted
  modify_settings|
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by benjaoming):

 * has_patch:  0 => 1


Comment:

 @Windson yang
 This issue itself is evidence that it's easy to get wrong, and that it
 will cost a lot of time to understand.. especially if the docs promote a
 pattern that doesn't work in Python <3.6.

 There's a PR now with a note about it, and examples are updated.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.df98b2e53c0bf41e79443ec173d29e6d%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29226: @modify_settings append/remove/prepend is affected by dictionary order

[Django]

#29226: @modify_settings append/remove/prepend is affected by dictionary order
-+-
 Reporter:  Manuel Kaufmann  |Owner:
 |  benjaoming
 Type:  Uncategorized|   Status:  assigned
Component:  Testing framework|  Version:  2.0
 Severity:  Normal   |   Resolution:
 Keywords:  tests,   | Triage Stage:  Accepted
  modify_settings|
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-

Comment (by Windson yang):

 I think users should know dict is not ordered before
 [https://docs.python.org/3.6/whatsnew/3.6.html#new-dict-implementation/
 python3.6]. I don't think we should docs this in django

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.23a2f0a04773193b863b35c2f1840ace%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29226: @modify_settings append/remove/prepend is affected by dictionary order

[Django]

#29226: @modify_settings append/remove/prepend is affected by dictionary order
-+-
 Reporter:  Manuel Kaufmann  |Owner:
 |  benjaoming
 Type:  Uncategorized|   Status:  assigned
Component:  Testing framework|  Version:  2.0
 Severity:  Normal   |   Resolution:
 Keywords:  tests,   | Triage Stage:  Accepted
  modify_settings|
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-

Comment (by Chris Jerdonek):

 Another option could simply be to sort anytime iterating over a dict. I
 use this practice in personal projects so that behavior will always be
 deterministic. Also, I personally don't think relying on dict's implicit
 ordering should be encouraged. It seems too tricky, and there are other
 patterns whose behavior is more obvious (e.g. the nesting approach).

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.593d3b6efdccda5f0cbbed468c46bd36%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29226: @modify_settings append/remove/prepend is affected by dictionary order

[Django]

#29226: @modify_settings append/remove/prepend is affected by dictionary order
-+-
 Reporter:  Manuel Kaufmann  |Owner:
 |  benjaoming
 Type:  Uncategorized|   Status:  assigned
Component:  Testing framework|  Version:  2.0
 Severity:  Normal   |   Resolution:
 Keywords:  tests,   | Triage Stage:  Accepted
  modify_settings|
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by benjaoming):

 * owner:  nobody => benjaoming
 * status:  new => assigned
 * type:  Cleanup/optimization => Uncategorized


Comment:

 Hi!

 Sounds like a time-consuming issue for those who might encounter it. I
 agree with Tim that hopefully someone would think twice about this, but
 having dealt with issues related to dict key ordering myself, I have to
 admit that you get it wrong sometimes.

 Also, the example in the docs would directly lead to the issue, except
 that it does append/remove/prepend the same items, but it stills seems
 like it would easily misguide someone to do that:
 
https://docs.djangoproject.com/en/2.0/topics/testing/tools/#django.test.SimpleTestCase.modify_settings

 The solution that Markus and I just discussed was that you can A) use
 `OrderedDict` or B) add `modify_settings` twice like this:

 {{{#!python

 # ...
 @modify_settings(MIDDLEWARES={
 'remove': [
 '4',
 '5',
 ],
 })
 @modify_settings(MIDDLEWARES={
 'append': [
 'MyOwnMiddlewareHere',
 '4',
 '5',
 ],
 })

 }}}

 Two suggestions:

 1. Updating the (misguiding) example that does the above without
 elaborating lots about why it's applied several times:
 
https://docs.djangoproject.com/en/2.0/topics/testing/tools/#django.test.SimpleTestCase.modify_settings
 1. Add a warning box about doing append+reject on the same items (re-
 ordering)

 As I understand Tim's comment, the latter would be a bit over the top, but
 the former could be preferable as it wouldn't complicate the docs much?

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.b2ef6d91d1f9e6c3c8de09a2c5ed98d5%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29226: @modify_settings append/remove/prepend is affected by dictionary order

[Django]

#29226: @modify_settings append/remove/prepend is affected by dictionary order
-+-
 Reporter:  Manuel Kaufmann  |Owner:  nobody
 Type:   |   Status:  new
  Cleanup/optimization   |
Component:  Testing framework|  Version:  2.0
 Severity:  Normal   |   Resolution:
 Keywords:  tests,   | Triage Stage:  Accepted
  modify_settings|
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Tim Graham):

 * type:  Uncategorized => Cleanup/optimization


Comment:

 If I understand correctly, the requirement to nest `@modify_settings`
 isn't present if targeting Python 3.6+ because dicts are ordered there.
 I'm not sure what would give an expectation of some order of operations in
 older versions of Python. I'm not sure it's worth documenting anything
 about this.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.c502160ac9a07bd5c70c386fbff833c2%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29226: @modify_settings append/remove/prepend is affected by dictionary order

[Django]

#29226: @modify_settings append/remove/prepend is affected by dictionary order
-+-
 Reporter:  Manuel Kaufmann  |Owner:  nobody
 Type:  Uncategorized|   Status:  new
Component:  Testing framework|  Version:  2.0
 Severity:  Normal   |   Resolution:
 Keywords:  tests,   | Triage Stage:  Accepted
  modify_settings|
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-

Comment (by Claude Paroz):

 I'm not sure `append`/`removed` were designed to run in the same
 `modify_settings` and the same setting.
 What about documenting to nest `modify_settings` calls for this use case?

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.1ccd417b4258a0f3f0ee2bcb7ee57e74%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29226: @modify_settings append/remove/prepend is affected by dictionary order

[Django]

#29226: @modify_settings append/remove/prepend is affected by dictionary order
-+-
 Reporter:  Manuel Kaufmann  |Owner:  nobody
 Type:  Uncategorized|   Status:  new
Component:  Testing framework|  Version:  2.0
 Severity:  Normal   |   Resolution:
 Keywords:  tests,   | Triage Stage:  Accepted
  modify_settings|
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Markus Holtermann):

 * stage:  Unreviewed => Accepted


Comment:

 Sounds like a legitimate problem.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.d43ea71f33cf65977b96bfa8ebc1c4da%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29226: @modify_settings append/remove/prepend is affected by dictionary order

[Django]

#29226: @modify_settings append/remove/prepend is affected by dictionary order
-+-
 Reporter:  Manuel Kaufmann  |Owner:  nobody
 Type:  Uncategorized|   Status:  new
Component:  Testing framework|  Version:  2.0
 Severity:  Normal   |   Resolution:
 Keywords:  tests,   | Triage Stage:
  modify_settings|  Unreviewed
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Manuel Kaufmann):

 * Attachment "test_modify_settings.patch" added.

 Tests to demostrate the inconsisten behaviour of @modify_settings

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.c900a7764cfdb6a1e973970eb131e85e%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.