Skip to content

TST: organize pandas/tests/groupby/test_aggregate.py #18490

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.

Sign up for GitHub

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

Jump to bottom
Closed
jreback opened this issue Nov 25, 2017 · 1 comment · Fixed by #18931
Closed

TST: organize pandas/tests/groupby/test_aggregate.py #18490

jreback opened this issue Nov 25, 2017 · 1 comment · Fixed by #18931
Labels
good first issue Groupby Testing pandas testing functions or related to the test suite
Milestone
0.23.0

Comments

@jreback
Copy link
Contributor

jreback commented Nov 25, 2017

might be nice to group these tests into separate classes within this file for organization purposes. The setup that is done (setup_method) is only done for a small subset of tests, the others don't need it.
@jreback jreback added Difficulty Novice Groupby Testing pandas testing functions or related to the test suite labels Nov 25, 2017
@jreback jreback added this to the 0.22.0 milestone Nov 25, 2017
@alysivji
Copy link
Contributor

I've started working on this on my morning commute. Should have it done in the next week or so.

@alysivji alysivji mentioned this issue Dec 24, 2017
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Groupby Testing pandas testing functions or related to the test suite
Projects
None yet
Milestone
0.23.0
Development

Successfully merging a pull request may close this issue.

TST: organize and cleanup pandas/tests/groupby/test_aggregate.py
2 participants
@jreback @alysivji