Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix travis-ci test failure #123

Merged
merged 4 commits into from Jun 6, 2014
Merged

Fix travis-ci test failure #123

merged 4 commits into from Jun 6, 2014

Conversation

cdeil
Copy link
Contributor

@cdeil cdeil commented Jun 5, 2014

Currently the master branch is broken on travis-ci.
In this PR I try to understand and fix the test errors.

@cdeil
Copy link
Contributor Author

cdeil commented Jun 5, 2014

@astrofrog @mdboom I think this might be an Astropy or pytest issue:
https://travis-ci.org/gammapy/gammapy/jobs/26885100#L808

The TestGammaSpectralCube.test_flux_array test should be skipped because of

@pytest.mark.skipif('not HAS_SCIPY')
class TestGammaSpectralCube(object):

(see here) and because HAS_SCIPY = False (see here).

I can't reproduce this locally, so I think it must be a problem with a certain Astropy or pytest version.

Do you understand what is going on or can you reproduce?

@cdeil cdeil added the bug label Jun 5, 2014
@cdeil cdeil added this to the 0.1 milestone Jun 5, 2014
@cdeil cdeil self-assigned this Jun 5, 2014
cdeil added a commit that referenced this pull request Jun 6, 2014
@cdeil cdeil merged commit 70165cb into gammapy:master Jun 6, 2014
@cdeil cdeil deleted the travis-42 branch June 6, 2014 05:17
@cdeil cdeil changed the title Test travis-ci issues Fix travis-ci test failure Apr 8, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant