Re: [Django] #22254: Testing documentation doesn't make it clear that django.setup() needs to be called with setup_test_environment

[Django]

#22254: Testing documentation doesn't make it clear that django.setup() needs 
to be
called with setup_test_environment
-+-
 Reporter:  peter.landry@…   |Owner:  Tim
 Type:   |  Graham 
  Cleanup/optimization   |   Status:  closed
Component:  Documentation|  Version:
 Severity:  Normal   |  1.7-alpha-2
 Keywords:  app-loading  |   Resolution:  fixed
Has patch:  0| Triage Stage:
  Needs tests:  0|  Unreviewed
Easy pickings:  0|  Needs documentation:  0
 |  Patch needs improvement:  0
 |UI/UX:  0
-+-

Comment (by Tim Graham ):

 In [changeset:"305ab00087a29ff400e815187b5568f84d2d8739"]:
 {{{
 #!CommitTicketReference repository=""
 revision="305ab00087a29ff400e815187b5568f84d2d8739"
 [1.7.x] Fixed #22254 -- Noted the requirement of calling django.setup()
 when running

 Thanks Peter Landry for the report.

 Backport of 26b03f2526 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/080.eeac7b68ee20fa5c04e8d49c48c58090%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #22254: Testing documentation doesn't make it clear that django.setup() needs to be called with setup_test_environment

[Django]

#22254: Testing documentation doesn't make it clear that django.setup() needs 
to be
called with setup_test_environment
-+-
 Reporter:  peter.landry@…   |Owner:  Tim
 Type:   |  Graham 
  Cleanup/optimization   |   Status:  closed
Component:  Documentation|  Version:
 Severity:  Normal   |  1.7-alpha-2
 Keywords:  app-loading  |   Resolution:  fixed
Has patch:  0| Triage Stage:
  Needs tests:  0|  Unreviewed
Easy pickings:  0|  Needs documentation:  0
 |  Patch needs improvement:  0
 |UI/UX:  0
-+-
Changes (by Tim Graham ):

 * owner:   => Tim Graham 
 * status:  new => closed
 * resolution:   => fixed


Comment:

 In [changeset:"26b03f25265227498a2559af807918fe2a1d246f"]:
 {{{
 #!CommitTicketReference repository=""
 revision="26b03f25265227498a2559af807918fe2a1d246f"
 Fixed #22254 -- Noted the requirement of calling django.setup() when
 running

 Thanks Peter Landry for the report.
 }}}

-- 
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/080.8e38106b84b9a669095b252e44708c9e%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #22254: Testing documentation doesn't make it clear that django.setup() needs to be called with setup_test_environment

[Django]

#22254: Testing documentation doesn't make it clear that django.setup() needs 
to be
called with setup_test_environment
-+-
 Reporter:  peter.landry@…   |Owner:
 Type:   |   Status:  new
  Cleanup/optimization   |  Version:
Component:  Documentation|  1.7-alpha-2
 Severity:  Normal   |   Resolution:
 Keywords:  app-loading  | Triage Stage:
Has patch:  0|  Unreviewed
  Needs tests:  0|  Needs documentation:  0
Easy pickings:  0|  Patch needs improvement:  0
 |UI/UX:  0
-+-
Changes (by aaugustin):

 * status:  assigned => new
 * owner:  aaugustin =>
 * has_patch:  1 => 0
 * severity:  Release blocker => Normal
 * stage:  Ready for checkin => Unreviewed


Comment:

 `django.setup()` is now properly documented, so we can re-evaluate if
 something needs to be done for `setup_test_environment()`.

-- 
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/080.250f40101fa7137ac6cf2fc284fcdd46%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #22254: Testing documentation doesn't make it clear that django.setup() needs to be called with setup_test_environment

[Django]

#22254: Testing documentation doesn't make it clear that django.setup() needs 
to be
called with setup_test_environment
-+-
 Reporter:  peter.landry@…   |Owner:  aaugustin
 Type:   |   Status:  assigned
  Cleanup/optimization   |  Version:
Component:  Documentation|  1.7-alpha-2
 Severity:  Release blocker  |   Resolution:
 Keywords:  app-loading  | Triage Stage:  Ready for
Has patch:  1|  checkin
  Needs tests:  0|  Needs documentation:  0
Easy pickings:  0|  Patch needs improvement:  0
 |UI/UX:  0
-+-
Changes (by timo):

 * needs_better_patch:  1 => 0
 * stage:  Accepted => Ready for checkin


-- 
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/080.e869818e1d8255b7926ba149a88aa237%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #22254: Testing documentation doesn't make it clear that django.setup() needs to be called with setup_test_environment

[Django]

#22254: Testing documentation doesn't make it clear that django.setup() needs 
to be
called with setup_test_environment
-+-
 Reporter:  peter.landry@…   |Owner:  aaugustin
 Type:   |   Status:  assigned
  Cleanup/optimization   |  Version:
Component:  Documentation|  1.7-alpha-2
 Severity:  Release blocker  |   Resolution:
 Keywords:  app-loading  | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-

Comment (by aaugustin):

 https://github.com/django/django/pull/2609 documents django.setup(). Can I
 haz review?

 I'm not convinced it's worth duplicating that information wherever
 setup_test_environment() is useful. If you hit `RuntimeError: App registry
 isn't ready yet.` Google will tell you how to fix it in 10 seconds. I'll
 let someone else deal with that part if you want to.

-- 
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/080.eadc7b4aca5414ac020bb13cd62586e9%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #22254: Testing documentation doesn't make it clear that django.setup() needs to be called with setup_test_environment

[Django]

#22254: Testing documentation doesn't make it clear that django.setup() needs 
to be
called with setup_test_environment
-+-
 Reporter:  peter.landry@…   |Owner:  aaugustin
 Type:   |   Status:  assigned
  Cleanup/optimization   |  Version:
Component:  Documentation|  1.7-alpha-2
 Severity:  Release blocker  |   Resolution:
 Keywords:  app-loading  | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-
Changes (by aaugustin):

 * status:  new => assigned
 * owner:  nobody => aaugustin
 * severity:  Normal => Release blocker


Comment:

 Indeed, django.setup() isn't correctly documented. Sorry about that.

-- 
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/080.bdba34e57596b9cfc2032ba00b1978f9%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #22254: Testing documentation doesn't make it clear that django.setup() needs to be called with setup_test_environment

[Django]

#22254: Testing documentation doesn't make it clear that django.setup() needs 
to be
called with setup_test_environment
-+-
 Reporter:  peter.landry@…   |Owner:  nobody
 Type:   |   Status:  new
  Cleanup/optimization   |  Version:
Component:  Documentation|  1.7-alpha-2
 Severity:  Normal   |   Resolution:
 Keywords:  app-loading  | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-
Changes (by timo):

 * needs_better_patch:  0 => 1
 * has_patch:  0 => 1
 * type:  Uncategorized => Cleanup/optimization


Comment:

 Yes, it seems like we should enhance the documentation of
 `django.setup()`. There's some mentions of it in the 1.7 release notes and
 a couple other places like `docs/ref/django-admin.txt` and
 `docs/intro/tutorial01.txt`.

-- 
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/080.e3e05b6eab21576f0c18f1d79c0cb0a3%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #22254: Testing documentation doesn't make it clear that django.setup() needs to be called with setup_test_environment

[Django]

#22254: Testing documentation doesn't make it clear that django.setup() needs 
to be
called with setup_test_environment
+---
 Reporter:  peter.landry@…  |Owner:  nobody
 Type:  Uncategorized   |   Status:  new
Component:  Documentation   |  Version:  1.7-alpha-2
 Severity:  Normal  |   Resolution:
 Keywords:  app-loading | Triage Stage:  Accepted
Has patch:  0   |  Needs documentation:  0
  Needs tests:  0   |  Patch needs improvement:  0
Easy pickings:  0   |UI/UX:  0
+---

Comment (by peter.landry@…):

 Perhaps it would be better to add instructions or information regarding
 `django.setup()` in
 https://docs.djangoproject.com/en/dev/ref/applications/, then just make a
 reference in the testing docs to it in a warning about using django
 outside of manage.py?

-- 
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/080.7828c984e3988730e97211404600e0f4%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #22254: Testing documentation doesn't make it clear that django.setup() needs to be called with setup_test_environment

[Django]

#22254: Testing documentation doesn't make it clear that django.setup() needs 
to be
called with setup_test_environment
+---
 Reporter:  peter.landry@…  |Owner:  nobody
 Type:  Uncategorized   |   Status:  new
Component:  Documentation   |  Version:  1.7-alpha-2
 Severity:  Normal  |   Resolution:
 Keywords:  app-loading | Triage Stage:  Accepted
Has patch:  0   |  Needs documentation:  0
  Needs tests:  0   |  Patch needs improvement:  0
Easy pickings:  0   |UI/UX:  0
+---
Changes (by aaugustin):

 * keywords:   => app-loading


-- 
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/080.2d517ca8f0d5c93f2e1c5f8f0a1dcd1a%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #22254: Testing documentation doesn't make it clear that django.setup() needs to be called with setup_test_environment

[Django]

#22254: Testing documentation doesn't make it clear that django.setup() needs 
to be
called with setup_test_environment
+---
 Reporter:  peter.landry@…  |Owner:  nobody
 Type:  Uncategorized   |   Status:  new
Component:  Documentation   |  Version:  1.7-alpha-2
 Severity:  Normal  |   Resolution:
 Keywords:  | Triage Stage:  Accepted
Has patch:  0   |  Needs documentation:  0
  Needs tests:  0   |  Patch needs improvement:  0
Easy pickings:  0   |UI/UX:  0
+---
Changes (by mjtamlyn):

 * needs_docs:   => 0
 * needs_better_patch:   => 0
 * needs_tests:   => 0
 * stage:  Unreviewed => Accepted


-- 
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/080.63cc837c1b22cee7482902740695e45a%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

[Django] #22254: Testing documentation doesn't make it clear that django.setup() needs to be called with setup_test_environment

[Django]

#22254: Testing documentation doesn't make it clear that django.setup() needs 
to be
called with setup_test_environment
+-
 Reporter:  peter.landry@…  |  Owner:  nobody
 Type:  Uncategorized   | Status:  new
Component:  Documentation   |Version:  1.7-alpha-2
 Severity:  Normal  |   Keywords:
 Triage Stage:  Unreviewed  |  Has patch:  0
Easy pickings:  0   |  UI/UX:  0
+-
 https://docs.djangoproject.com/en/dev/topics/testing/advanced/#running-
 tests-outside-the-test-runner

 It's not specific to testing, but it seems that we should mention the
 necessity of calling `django.setup()` in the directions for running tests
 outside of `manage.py`. I've submitted a pull request with a simple
 documentation change: https://github.com/django/django/pull/2419

 This may require some more discussion, though. As mjtamlyn points out,
 another solution could be to include `django.setup()` in
 `setup_test_environment()`. The call to `django.setup()` could also be
 moved into the example for using `setup_test_environment()` rather than a
 warning in its own.

-- 
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.779252dae9cea4613ba89f8bb133da3f%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.