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 mimetype confusion during tests #116

Merged
merged 1 commit into from Apr 5, 2019
Merged

Fix mimetype confusion during tests #116

merged 1 commit into from Apr 5, 2019

Conversation

cbanek
Copy link
Contributor

@cbanek cbanek commented Apr 4, 2019

It seems like it's easy for python to get confused when you name
a file the same as a dependent module. This happened for mimetypes,
and seems to call the wrong mimetypes when trying to add_type the
second time.

@coveralls
Copy link

coveralls commented Apr 4, 2019

Coverage Status

Coverage remained the same at 70.571% when pulling 12cc686 on cbanek:mimetype-fix into f854676 on pyvirtobs:master.

It seems like it's easy for python to get confused when you name
a file the same as a dependent module.  This happened for mimetypes,
and seems to call the wrong mimetypes when trying to call add_type.
@cbanek
Copy link
Contributor Author

cbanek commented Apr 4, 2019

I think if you also take PR #112 we can get back to passing? The one test that fails (and isn't style related, that is fixed by the other PR), passes on this branch with this fix.

@funbaker funbaker merged commit 2b74127 into astropy:master Apr 5, 2019
@funbaker
Copy link
Contributor

funbaker commented Apr 5, 2019

Merged this and the referenced one. Lets see if this unbreaks the tests.

Thank you!

@cbanek
Copy link
Contributor Author

cbanek commented Apr 5, 2019

Well looks like we're closer to unbreaking the tests. The other PR will get the style ones. I'm still vexed as to why this didn't fix that last failing test. It didn't fail in PR #112 's travis branch run!

@cbanek cbanek deleted the mimetype-fix branch April 5, 2019 16:37
@funbaker
Copy link
Contributor

funbaker commented Apr 6, 2019

I think it is related to https://github.com/astropy/ccdproc/pull/650/files

@bsipocz bsipocz added this to the v0.9.3 milestone Mar 7, 2023
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.

None yet

4 participants