Skip to content

CLN: Work on converting assert_'s to specialized forms, in pandas/tests #6262

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 1 commit into from
Feb 5, 2014
Merged

CLN: Work on converting assert_'s to specialized forms, in pandas/tests #6262

merged 1 commit into from
Feb 5, 2014

Conversation

bwignall
Copy link
Contributor

@bwignall bwignall commented Feb 5, 2014

Work on #6175, in pandas/tests. Tests still pass locally. A work-in-progress, but--with #6261--removes many/most cases of assert_(x == y) in this directory

Replacements:

  • assert_(x == y) ==> assertEqual(x, y)
  • assert_(x == True) ==> assertTrue(x)
  • assert_(x == False) ==> assertFalse(x)

Work on #6175, in pandas/tests. Tests still pass locally. A work-in-progress, but--with #6261--removes many/most cases of assert_(x == y) in this directory
jreback added a commit that referenced this pull request Feb 5, 2014
CLN: Work on converting assert_'s to specialized forms, in pandas/tests
@jreback jreback merged commit 363b214 into pandas-dev:master Feb 5, 2014
@jreback
Copy link
Contributor

jreback commented Feb 5, 2014

thanks on this one too!

@bwignall bwignall deleted the assert_6175_5 branch February 18, 2014 03:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants