Skip to content

Conversation

@mroeschke mroeschke added the Testing pandas testing functions or related to the test suite label May 28, 2021
@mroeschke mroeschke added this to the 1.3 milestone May 28, 2021
Copy link
Member

@simonjayhawkins simonjayhawkins left a comment

Choose a reason for hiding this comment

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

Thanks @mroeschke lgtm. one suggestion.

}
)
df = df.set_index(["A", "B"])
with pytest.raises(KeyError, match="1"):
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
with pytest.raises(KeyError, match="1"):
with pytest.raises(KeyError, match="^1$"):

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.

sure, ex @simonjayhawkins comment

@jreback jreback merged commit a811c96 into pandas-dev:master May 28, 2021
@jreback
Copy link
Contributor

jreback commented May 28, 2021

thanks @mroeschke

@mroeschke mroeschke deleted the tests/old_issues branch May 28, 2021 19:14
TLouf pushed a commit to TLouf/pandas that referenced this pull request Jun 1, 2021
JulianWgs pushed a commit to JulianWgs/pandas that referenced this pull request Jul 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment