-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
CLN: Testing and unused #49754
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
CLN: Testing and unused #49754
Changes from all commits
9c66ab9
d694fcb
6ff40b1
274b7ad
51051d6
7beb962
b24e5df
96ba740
3ec730f
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
@@ -441,7 +441,6 @@ def test_drop_duplicates_null_in_object_column(nulls_fixture): | |||||||||||||||
tm.assert_frame_equal(result, df) | ||||||||||||||||
|
||||||||||||||||
|
||||||||||||||||
@pytest.mark.parametrize("keep", ["first", "last", False]) | ||||||||||||||||
def test_drop_duplicates_series_vs_dataframe(keep): | ||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. cause this is the same as the fixture? 🤔 might be an idea for an automated test There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yup same as this one Lines 304 to 310 in cca262d
Yeah it would be great to have a check to answer "Does a @pytest.mark.parametrize return the same params as an existing fixture" |
||||||||||||||||
# GH#14192 | ||||||||||||||||
df = DataFrame( | ||||||||||||||||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is the idea here just that
any_numpy_dtype
is a superset ofany_numpy_dtype_reduced
, and that one might as well test over the larger set?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yup, exactly