Skip to content

TST: troubleshoot inconsistent xfails #27715

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

Merged
merged 40 commits into from
Aug 5, 2019
Merged

Conversation

jbrockmendel
Copy link
Member

No description provided.

@WillAyd WillAyd added the Testing pandas testing functions or related to the test suite label Aug 2, 2019
@WillAyd
Copy link
Member

WillAyd commented Aug 2, 2019

Thanks for taking a look at these - great to get these working again. Took a quick look at a couple builds and noticed they weren't always the same failures. Could split this up into a few PRs if easier to troubleshoot

@pep8speaks
Copy link

pep8speaks commented Aug 2, 2019

Hello @jbrockmendel! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2019-08-02 23:33:25 UTC

@jbrockmendel
Copy link
Member Author

black and flake8 are OK with this diff, pep8speaks must have its own config or a version mismatch?

@jbrockmendel
Copy link
Member Author

@WillAyd can I get a "booyah"?

@WillAyd
Copy link
Member

WillAyd commented Aug 3, 2019

booyah it is

@jbrockmendel jbrockmendel changed the title WIP: troubleshoot inconsistent xfails TST: troubleshoot inconsistent xfails Aug 4, 2019
Copy link
Contributor

@jreback jreback left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

question

@@ -666,6 +666,7 @@ def test_comparison_tzawareness_compat_scalars(self, op, box_with_array):
# Raising in __eq__ will fallback to NumPy, which warns, fails,
# then re-raises the original exception. So we just need to ignore.
@pytest.mark.filterwarnings("ignore:elementwise comp:DeprecationWarning")
@pytest.mark.filterwarnings("ignore:Converting timezone-aware:FutureWarning")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same question as on another PR, is this hiding something?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AFAICT just noise in the logs

@jreback jreback added this to the 1.0 milestone Aug 4, 2019
@jreback
Copy link
Contributor

jreback commented Aug 4, 2019

any reason we shouldn't do this for 0.25.1?

@jbrockmendel
Copy link
Member Author

any reason we shouldn't do this for 0.25.1?

Not that I'm aware of

@jreback jreback modified the milestones: 1.0, 0.25.1 Aug 5, 2019
@jreback jreback merged commit b9a9b9f into pandas-dev:master Aug 5, 2019
@jreback
Copy link
Contributor

jreback commented Aug 5, 2019

thanks @jbrockmendel

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Testing pandas testing functions or related to the test suite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants