Skip to content

Conversation

@mdboom
Copy link
Contributor

@mdboom mdboom commented Apr 22, 2014

As reported by @cdeil in #1822.

@astrofrog
Copy link
Member

@cdeil - does this fix it for you?

@cdeil
Copy link
Member

cdeil commented Apr 22, 2014

@astrofrog Yes, with this branch the test error I got with python -c 'import astropy; astropy.test() listed here doesn't show up.

@embray
Copy link
Member

embray commented Apr 22, 2014

I feel like I had an issue open for this a long time ago, but I'm not sure...

@embray
Copy link
Member

embray commented Apr 22, 2014

Ah, the original issue I'm thinking of was #1284. In that, I decided to let config values modify the behavior of astropy.test(). But I think now it's better to write the tests in such a way that the config values have predictable defaults. So if this was easy to fix then I'm fine with that.

@astrofrog
Copy link
Member

@embray - so just to be clear, are you ok with us merging this?

@embray
Copy link
Member

embray commented Apr 23, 2014

Fine by me.

@astrofrog astrofrog added this to the v0.4.0 milestone Apr 23, 2014
astrofrog added a commit that referenced this pull request Apr 23, 2014
Ignore any user config file, even when running astropy.test()
@astrofrog astrofrog merged commit d184e49 into astropy:master Apr 23, 2014
@astrofrog
Copy link
Member

This doesn't sound like a critical bug, so tagging as 0.4.0 - but feel free to re-tag as 0.3.2 if you think it should be backported.

@mdboom mdboom deleted the test/ignore-config-file branch May 21, 2014 23:56
astrofrog added a commit to astrofrog/astropy-helpers that referenced this pull request May 22, 2014
astrofrog added a commit to astrofrog/astropy-helpers that referenced this pull request May 22, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants