Tests introduced in patch for #17327 causing errors

2012-03-07 Thread Ryan Kaskel
Hi all.

I've pulled in the latest changes and it seems like ticket #17327 [1] 
introduces a few new tests that are raising errors when I run contrib.auth 
tests via my project:

- 
auth.MultiDBChangepasswordManagementCommandTestCase.test_that_changepassword_command_with_database_option_uses_given_db
- 
auth.MultiDBCreatesuperuserTestCase.test_createsuperuser_command_with_database_option

They raise: ConnectionDoesNotExist: The connection other doesn't exist.

The issue is that the tests assume there there is a database called 
"other," which my test settings didn't have. If I add a database "other" in 
my test settings, the tests pass.

I've played around with override_settings but I'm not sure if that would 
work here. What I've tried so far doesn't.

Any tips?

Thanks,
Ryan Kaskel

[1] https://code.djangoproject.com/ticket/17327

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers" group.
To view this discussion on the web visit 
https://groups.google.com/d/msg/django-developers/-/WBkMgg6L_1wJ.
To post to this group, send email to django-developers@googlegroups.com.
To unsubscribe from this group, send email to 
django-developers+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-developers?hl=en.



Re: Django participation in Google Summer of Code 2012

2012-03-07 Thread Kushagra Sinha
Thanks a lot for the reply.

On Wed, Mar 7, 2012 at 5:02 AM, Russell Keith-Magee  wrote:

>
> On 07/03/2012, at 5:35 AM, Kushagra Sinha wrote:
>
> > Are there plans for Django participating in this year's Google Summer of
> Code. The organization application deadline is on 9th March. I would love
> to participate as a student.
> >
>
> Yes, we are planning to apply as an organization.
>
> If you're interested in doing a Django-based project, it's never too early
> to start working on your proposal; historically, the applicants that have
> been the most successful have started on their proposal (and their
> projects) long before the "official" application period. Obviously, we
> haven't got our 2012 SOC page up yet, but last year's page [1] will give
> you a few ideas -- especially if you focus on projects that haven't been
> completed, or were started on last year but still need some work to bring
> them to completion.
>
> [1] https://code.djangoproject.com/wiki/SummerOfCode2011
>
> Yours,
> Russ Magee %-)
>
> --
> You received this message because you are subscribed to the Google Groups
> "Django developers" group.
> To post to this group, send email to django-developers@googlegroups.com.
> To unsubscribe from this group, send email to
> django-developers+unsubscr...@googlegroups.com.
> For more options, visit this group at
> http://groups.google.com/group/django-developers?hl=en.
>
>


-- 
Kushagra SInha
B. Tech. Part III (Pre-final year)
Indian Institute of Technology
Varanasi

Contact: +91 9415 125 215

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers" group.
To post to this group, send email to django-developers@googlegroups.com.
To unsubscribe from this group, send email to 
django-developers+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-developers?hl=en.



Re: Django participation in Google Summer of Code 2012

2012-03-07 Thread Russell Keith-Magee

One quick update -- Django's 2012 GSoC wiki page is now up, at the somewhat 
predictable URL:

https://code.djangoproject.com/wiki/SummerOfCode2012

Yours,
Russ Magee %-)

On 07/03/2012, at 5:03 PM, Kushagra Sinha wrote:

> Thanks a lot for the reply.
> 
> On Wed, Mar 7, 2012 at 5:02 AM, Russell Keith-Magee  
> wrote:
> 
> On 07/03/2012, at 5:35 AM, Kushagra Sinha wrote:
> 
> > Are there plans for Django participating in this year's Google Summer of 
> > Code. The organization application deadline is on 9th March. I would love 
> > to participate as a student.
> >
> 
> Yes, we are planning to apply as an organization.
> 
> If you're interested in doing a Django-based project, it's never too early to 
> start working on your proposal; historically, the applicants that have been 
> the most successful have started on their proposal (and their projects) long 
> before the "official" application period. Obviously, we haven't got our 2012 
> SOC page up yet, but last year's page [1] will give you a few ideas -- 
> especially if you focus on projects that haven't been completed, or were 
> started on last year but still need some work to bring them to completion.
> 
> [1] https://code.djangoproject.com/wiki/SummerOfCode2011
> 
> Yours,
> Russ Magee %-)
> 
> --
> You received this message because you are subscribed to the Google Groups 
> "Django developers" group.
> To post to this group, send email to django-developers@googlegroups.com.
> To unsubscribe from this group, send email to 
> django-developers+unsubscr...@googlegroups.com.
> For more options, visit this group at 
> http://groups.google.com/group/django-developers?hl=en.
> 
> 
> 
> 
> -- 
> Kushagra SInha
> B. Tech. Part III (Pre-final year)
> Indian Institute of Technology
> Varanasi
> 
> Contact: +91 9415 125 215
> 
> 
> -- 
> You received this message because you are subscribed to the Google Groups 
> "Django developers" group.
> To post to this group, send email to django-developers@googlegroups.com.
> To unsubscribe from this group, send email to 
> django-developers+unsubscr...@googlegroups.com.
> For more options, visit this group at 
> http://groups.google.com/group/django-developers?hl=en.

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers" group.
To post to this group, send email to django-developers@googlegroups.com.
To unsubscribe from this group, send email to 
django-developers+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-developers?hl=en.



Re: Tests introduced in patch for #17327 causing errors

2012-03-07 Thread Ryan Kaskel
In addition to the above (which I can confirm causes tests to fail in a new 
project), I also goofed up a patch related to a different set of auth 
tests. I've created #17848 (release blocker) to fix this. [1]

Ticket #16366 [2] required the template context processor cache to be 
cleared. The code resets the cache but fails to restore it. If the auth 
tests run first, this causes many other test failures in my project because 
the newly cached template context processors only include the Django 
defaults.

The general issue of having to reset this cache (and others) is in ticket 
#17787 which targets a later release. [3]

I'm sorry about that error and I hope it isn't too late to include the 
patch. I think it will cause many headaches for other people.

I know we shouldn't post about tickets we write but given 1.4 is going to 
be released in a few days, I think it's important to bring this up.

Best,
Ryan Kaskel

[1] https://code.djangoproject.com/ticket/17848
[2] https://code.djangoproject.com/ticket/16366
[3] https://code.djangoproject.com/ticket/17787

On Wednesday, 7 March 2012 08:40:56 UTC, Ryan Kaskel wrote:
>
> Hi all.
>
> I've pulled in the latest changes and it seems like ticket #17327 [1] 
> introduces a few new tests that are raising errors when I run contrib.auth 
> tests via my project:
>
> - 
> auth.MultiDBChangepasswordManagementCommandTestCase.test_that_changepassword_command_with_database_option_uses_given_db
> - 
> auth.MultiDBCreatesuperuserTestCase.test_createsuperuser_command_with_database_option
>
> They raise: ConnectionDoesNotExist: The connection other doesn't exist.
>
> The issue is that the tests assume there there is a database called 
> "other," which my test settings didn't have. If I add a database "other" in 
> my test settings, the tests pass.
>
> I've played around with override_settings but I'm not sure if that would 
> work here. What I've tried so far doesn't.
>
> Any tips?
>
> Thanks,
> Ryan Kaskel
>
> [1] https://code.djangoproject.com/ticket/17327
>

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers" group.
To view this discussion on the web visit 
https://groups.google.com/d/msg/django-developers/-/Kf3GUTSNUvYJ.
To post to this group, send email to django-developers@googlegroups.com.
To unsubscribe from this group, send email to 
django-developers+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-developers?hl=en.



Re: Tests introduced in patch for #17327 causing errors

2012-03-07 Thread Calvin Cheng

Thanks. I stumbled upon the same problem.


On Wednesday, March 7, 2012 4:40:56 PM UTC+8, Ryan Kaskel wrote:
>
> Hi all.
>
> I've pulled in the latest changes and it seems like ticket #17327 [1] 
> introduces a few new tests that are raising errors when I run contrib.auth 
> tests via my project:
>
> - 
> auth.MultiDBChangepasswordManagementCommandTestCase.test_that_changepassword_command_with_database_option_uses_given_db
> - 
> auth.MultiDBCreatesuperuserTestCase.test_createsuperuser_command_with_database_option
>
> They raise: ConnectionDoesNotExist: The connection other doesn't exist.
>
> The issue is that the tests assume there there is a database called 
> "other," which my test settings didn't have. If I add a database "other" in 
> my test settings, the tests pass.
>
> I've played around with override_settings but I'm not sure if that would 
> work here. What I've tried so far doesn't.
>
> Any tips?
>
> Thanks,
> Ryan Kaskel
>
> [1] https://code.djangoproject.com/ticket/17327
>

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers" group.
To view this discussion on the web visit 
https://groups.google.com/d/msg/django-developers/-/JXXZeiJ5SQgJ.
To post to this group, send email to django-developers@googlegroups.com.
To unsubscribe from this group, send email to 
django-developers+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-developers?hl=en.



Re: Tests introduced in patch for #17327 causing errors

2012-03-07 Thread Brian Riley
Thanks for catching this. I worked on that patch.

The problem is that those two test cases shouldn't be in the auth tests but 
rather somewhere in top-level tests folder (with the other multi DB tests). I'm 
not sure exactly where they belong though. If a core dev wants to chime in and 
suggest a location, I'd be happy to move them and submit the patch.

Brian

On Mar 7, 2012, at 7:11 AM, Calvin Cheng  wrote:

> 
> Thanks. I stumbled upon the same problem.
> 
> 
> On Wednesday, March 7, 2012 4:40:56 PM UTC+8, Ryan Kaskel wrote:
> Hi all.
> 
> I've pulled in the latest changes and it seems like ticket #17327 [1] 
> introduces a few new tests that are raising errors when I run contrib.auth 
> tests via my project:
> 
> - 
> auth.MultiDBChangepasswordManagementCommandTestCase.test_that_changepassword_command_with_database_option_uses_given_db
> - 
> auth.MultiDBCreatesuperuserTestCase.test_createsuperuser_command_with_database_option
> 
> They raise: ConnectionDoesNotExist: The connection other doesn't exist.
> 
> The issue is that the tests assume there there is a database called "other," 
> which my test settings didn't have. If I add a database "other" in my test 
> settings, the tests pass.
> 
> I've played around with override_settings but I'm not sure if that would work 
> here. What I've tried so far doesn't.
> 
> Any tips?
> 
> Thanks,
> Ryan Kaskel
> 
> [1] https://code.djangoproject.com/ticket/17327
> -- 
> You received this message because you are subscribed to the Google Groups 
> "Django developers" group.
> To view this discussion on the web visit 
> https://groups.google.com/d/msg/django-developers/-/JXXZeiJ5SQgJ.
> To post to this group, send email to django-developers@googlegroups.com.
> To unsubscribe from this group, send email to 
> django-developers+unsubscr...@googlegroups.com.
> For more options, visit this group at 
> http://groups.google.com/group/django-developers?hl=en.

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers" group.
To post to this group, send email to django-developers@googlegroups.com.
To unsubscribe from this group, send email to 
django-developers+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-developers?hl=en.



Re: Tests introduced in patch for #17327 causing errors

2012-03-07 Thread Brian Riley
I added a new patch to move these two failing tests to regressiontests--where 
they get run with the correct settings and pass. 

https://code.djangoproject.com/ticket/17327#comment:8

If this isn't the proper place for management command tests like this, I'll 
move them to wherever is more appropriate.

Brian 


On Wednesday, March 7, 2012 at 8:11 AM, Brian Riley wrote:

> Thanks for catching this. I worked on that patch.
> 
> The problem is that those two test cases shouldn't be in the auth tests but 
> rather somewhere in top-level tests folder (with the other multi DB tests). 
> I'm not sure exactly where they belong though. If a core dev wants to chime 
> in and suggest a location, I'd be happy to move them and submit the patch.
> 
> Brian
> 
> On Mar 7, 2012, at 7:11 AM, Calvin Cheng  (mailto:linchuan.ch...@gmail.com)> wrote:
> 
> > 
> > Thanks. I stumbled upon the same problem.
> > 
> > 
> > On Wednesday, March 7, 2012 4:40:56 PM UTC+8, Ryan Kaskel wrote:
> > > Hi all.
> > > 
> > > I've pulled in the latest changes and it seems like ticket #17327 [1] 
> > > introduces a few new tests that are raising errors when I run 
> > > contrib.auth tests via my project:
> > > 
> > > - 
> > > auth.MultiDBChangepasswordManagementCommandTestCase.test_that_changepassword_command_with_database_option_uses_given_db
> > > - 
> > > auth.MultiDBCreatesuperuserTestCase.test_createsuperuser_command_with_database_option
> > > 
> > > They raise: ConnectionDoesNotExist: The connection other doesn't exist.
> > > 
> > > The issue is that the tests assume there there is a database called 
> > > "other," which my test settings didn't have. If I add a database "other" 
> > > in my test settings, the tests pass.
> > > 
> > > I've played around with override_settings but I'm not sure if that would 
> > > work here. What I've tried so far doesn't.
> > > 
> > > Any tips?
> > > 
> > > Thanks,
> > > Ryan Kaskel
> > > 
> > > [1] https://code.djangoproject.com/ticket/17327
> > 
> > -- 
> > You received this message because you are subscribed to the Google Groups 
> > "Django developers" group.
> > To view this discussion on the web visit 
> > https://groups.google.com/d/msg/django-developers/-/JXXZeiJ5SQgJ.
> > To post to this group, send email to django-developers@googlegroups.com 
> > (mailto:django-developers@googlegroups.com).
> > To unsubscribe from this group, send email to 
> > django-developers+unsubscr...@googlegroups.com 
> > (mailto:django-developers+unsubscr...@googlegroups.com).
> > For more options, visit this group at 
> > http://groups.google.com/group/django-developers?hl=en.

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers" group.
To post to this group, send email to django-developers@googlegroups.com.
To unsubscribe from this group, send email to 
django-developers+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-developers?hl=en.