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.

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

TST: split out melt-tests in pandas/tests/reshape/test_reshape.py #18367

Closed
jreback opened this issue Nov 19, 2017 · 2 comments · Fixed by #18428
Closed

TST: split out melt-tests in pandas/tests/reshape/test_reshape.py #18367

jreback opened this issue Nov 19, 2017 · 2 comments · Fixed by #18428
Labels
Reshaping Concat, Merge/Join, Stack/Unstack, Explode Testing pandas testing functions or related to the test suite
Milestone
0.23.0

Comments

@jreback
Copy link
Contributor

jreback commented Nov 19, 2017

since melt and friends were moved to pandas/core/reshape/melt.py want to follow suit with the tests.

if you want to go once further and split out the melt, wide_to_long and lreshape tests to a new sub-module, e.g. pandas.tests.reshape.melt.test_melt, test_lreshape, test_wide_to_long would not object.
@jreback jreback added Reshaping Concat, Merge/Join, Stack/Unstack, Explode Testing pandas testing functions or related to the test suite labels Nov 19, 2017
@jreback jreback added this to the 0.22.0 milestone Nov 19, 2017
@jreback
Copy link
Contributor Author

jreback commented Nov 19, 2017

cc @tdpetrou

@tdpetrou
Copy link
Contributor

Sure I can do this. I'll just keep it to one file test_melt for now.

@tdpetrou tdpetrou mentioned this issue Nov 22, 2017
Merged
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Reshaping Concat, Merge/Join, Stack/Unstack, Explode 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: move melt tests to separate file
2 participants
@jreback @tdpetrou