Skip to content

Conversation

@mroeschke mroeschke added the Testing pandas testing functions or related to the test suite label Jan 20, 2020
@mroeschke mroeschke added this to the 1.1 milestone Jan 20, 2020
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.

lgtm, a couple of comments

# GH 12163
expected = pd.Series(dtype=object), MyTz()

with tm.ensure_clean() as path:
Copy link
Contributor

Choose a reason for hiding this comment

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

can you use tm.round_trip_pickle

@@ -1413,3 +1413,12 @@ def test_constructor_tz_mixed_data(self):
result = Series(dt_list)
expected = Series(dt_list, dtype=object)
tm.assert_series_equal(result, expected)

def test_contructor_dict_tuple_indexer(self):
Copy link
Contributor

Choose a reason for hiding this comment

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

can you move near the other contructor_dict tests

@jreback
Copy link
Contributor

jreback commented Jan 20, 2020

lgtm. merge on green.

@mroeschke mroeschke merged commit bbcda98 into pandas-dev:master Jan 21, 2020
@mroeschke mroeschke deleted the regression_testing branch January 21, 2020 01:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment