Need a possiblity to skip some system #2

Closed
slepa opened this Issue Dec 21, 2012 · 8 comments

Comments

Projects
None yet
2 participants

slepa commented Dec 21, 2012

I guess it will be good to specify empty codes. Or ignore some of them. But unit tests fails in this case.

Owner

nkuttler commented Dec 21, 2012

I'm really not sure what you mean. If you don't want to use one of the service providers don't define them in the config dict.

slepa commented Dec 21, 2012

Yes, but it breaks unit testing.

I can't do
./manage test
anymore

So my trick is

WEBMASTER_VERIFICATION = {
'bing': '123',
'majestic': '12345678901234567890123456789012',
'yandex': '1234567890123456',
'google': 'b<my_real_google_code>3',
}

Owner

nkuttler commented Dec 21, 2012

Oh, are you trying to run the unit tests from your own project with your own settings file? Yeah, I haven't planned for that. The tests are supposed to be run from the source tree with the provided project mockup.

slepa commented Dec 21, 2012

I am newbie in django testing.
May be this way is wrong, but i can't guess how to test only "my own" apps. Django tests all apps founded in my setting. Or i need to specify it manually in test script. So i just do
./manage.py test
And its first time, i found a problem :)

slepa commented Dec 21, 2012

your app will work in MY environment, so its not so stupid idea to test it with MY settings file :)

Owner

nkuttler commented Dec 21, 2012

Can you have a look if 37fa415 fixes your problem? Thanks you for finding this, it really is a bug, I just hadn't thought about this.

You can run the tests for your own apps with ./manage.py test app1 app2 etc.

slepa commented Dec 21, 2012

Greate! Thanks a lot. Its what i want.
Do you plan to make new pip release?

PS: I know about ./manage.py test app1 app2 app3 app4 ... But not found it useful.

@nkuttler nkuttler closed this in e7ab49d Dec 21, 2012

Owner

nkuttler commented Dec 21, 2012

Just released 0.1.10, thanks again for reporting this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment