Skip to content
This repository was archived by the owner on Nov 11, 2024. It is now read-only.

Adds an option to set the base directory for temporary test files - #188

Merged
embray merged 1 commit into
astropy:masterfrom
embray:test-temp-root-master
Sep 22, 2015
Merged

embray merged 1 commit into
astropy:masterfrom
embray:test-temp-root-master

Conversation

@embray

@embray embray commented Sep 17, 2015

Copy link
Copy Markdown
Member

(ports #187 to master)

(ports astropy#187 to master)

Conflicts:
	astropy_helpers/test_helpers.py
@embray embray added this to the v1.1 milestone Sep 17, 2015
@mdboom

mdboom commented Sep 21, 2015

Copy link
Copy Markdown
Contributor

py.test 2.8.0 just came out on 18-09-2015 with a lot more changes.

https://pytest.org/latest/changelog.html#id1

We may want to move to it while we're at it and make sure there are no more surprises. There are some failures on the astropy suite when moving to it, but I haven't had time to deeply investigate yet.

@embray

embray commented Sep 21, 2015

Copy link
Copy Markdown
Member Author

Perhaps, but I don't want it to hold things up, and I don't see anything in the changes that would help with the specific issue this was trying to address.

@embray embray mentioned this pull request Sep 21, 2015
embray added a commit that referenced this pull request Sep 22, 2015
Adds an option to set the base directory for temporary test files
@embray
embray merged commit 5f0f4a1 into astropy:master Sep 22, 2015
@embray
embray deleted the test-temp-root-master branch September 22, 2015 17:02
embray added a commit to embray/astropy that referenced this pull request Oct 1, 2015
dhomeier pushed a commit to dhomeier/astropy that referenced this pull request Dec 17, 2015
dhomeier pushed a commit to dhomeier/astropy that referenced this pull request Jun 12, 2016
drdavella pushed a commit to astropy/pytest-remotedata that referenced this pull request Sep 20, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants