Skip to content

BUG: in error message raised when invalid axis parameter #25553

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

Merged
merged 5 commits into from
Mar 6, 2019
Merged
Show file tree
Hide file tree
Changes from 4 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion doc/source/whatsnew/v0.25.0.rst
Original file line number Diff line number Diff line change
Expand Up @@ -254,7 +254,7 @@ Sparse
Other
^^^^^

-
- Bug in error message raised when invalid ``axis`` parameter passed (:issue:`25555`)
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 reword a bit, I am not sure what this means here.

-
-

Expand Down
4 changes: 2 additions & 2 deletions pandas/core/generic.py
Original file line number Diff line number Diff line change
Expand Up @@ -358,7 +358,7 @@ def _get_axis_number(cls, axis):
except KeyError:
pass
raise ValueError('No axis named {0} for object type {1}'
.format(axis, type(cls)))
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this should just be cls as it gives the fully qualified path

.format(axis, cls))

@classmethod
def _get_axis_name(cls, axis):
Expand All @@ -372,7 +372,7 @@ def _get_axis_name(cls, axis):
except KeyError:
pass
raise ValueError('No axis named {0} for object type {1}'
.format(axis, type(cls)))
.format(axis, cls))

def _get_axis(self, axis):
name = self._get_axis_name(axis)
Expand Down
6 changes: 4 additions & 2 deletions pandas/tests/frame/test_analytics.py
Original file line number Diff line number Diff line change
Expand Up @@ -1385,7 +1385,8 @@ def test_idxmin(self, float_frame, int_frame):
skipna=skipna)
tm.assert_series_equal(result, expected)

msg = "No axis named 2 for object type <class 'type'>"
msg = ("No axis named 2 for object type"
" <class 'pandas.core.frame.DataFrame'>")
with pytest.raises(ValueError, match=msg):
frame.idxmin(axis=2)

Expand All @@ -1402,7 +1403,8 @@ def test_idxmax(self, float_frame, int_frame):
skipna=skipna)
tm.assert_series_equal(result, expected)

msg = "No axis named 2 for object type <class 'type'>"
msg = ("No axis named 2 for object type"
" <class 'pandas.core.frame.DataFrame'>")
with pytest.raises(ValueError, match=msg):
frame.idxmax(axis=2)

Expand Down
3 changes: 2 additions & 1 deletion pandas/tests/frame/test_api.py
Original file line number Diff line number Diff line change
Expand Up @@ -366,7 +366,8 @@ def test_swapaxes(self):
self._assert_frame_equal(df.T, df.swapaxes(0, 1))
self._assert_frame_equal(df.T, df.swapaxes(1, 0))
self._assert_frame_equal(df, df.swapaxes(0, 0))
msg = "No axis named 2 for object type <class 'type'>"
msg = ("No axis named 2 for object type"
r" <class 'pandas.core(.sparse)?.frame.(Sparse)?DataFrame'>")
with pytest.raises(ValueError, match=msg):
df.swapaxes(2, 5)

Expand Down
3 changes: 2 additions & 1 deletion pandas/tests/frame/test_axis_select_reindex.py
Original file line number Diff line number Diff line change
Expand Up @@ -1067,7 +1067,8 @@ def test_reindex_axis(self):
reindexed2 = self.intframe.reindex(index=rows)
assert_frame_equal(reindexed1, reindexed2)

msg = "No axis named 2 for object type <class 'type'>"
msg = ("No axis named 2 for object type"
" <class 'pandas.core.frame.DataFrame'>")
with pytest.raises(ValueError, match=msg):
self.intframe.reindex_axis(rows, axis=2)

Expand Down
3 changes: 2 additions & 1 deletion pandas/tests/frame/test_missing.py
Original file line number Diff line number Diff line change
Expand Up @@ -140,7 +140,8 @@ def test_dropna(self):
assert_frame_equal(dropped, expected)

# bad input
msg = "No axis named 3 for object type <class 'type'>"
msg = ("No axis named 3 for object type"
" <class 'pandas.core.frame.DataFrame'>")
with pytest.raises(ValueError, match=msg):
df.dropna(axis=3)

Expand Down
6 changes: 4 additions & 2 deletions pandas/tests/frame/test_quantile.py
Original file line number Diff line number Diff line change
Expand Up @@ -95,10 +95,12 @@ def test_quantile_axis_parameter(self):
result = df.quantile(.5, axis="columns")
assert_series_equal(result, expected)

msg = "No axis named -1 for object type <class 'type'>"
msg = ("No axis named -1 for object type"
" <class 'pandas.core.frame.DataFrame'>")
with pytest.raises(ValueError, match=msg):
df.quantile(0.1, axis=-1)
msg = "No axis named column for object type <class 'type'>"
msg = ("No axis named column for object type"
" <class 'pandas.core.frame.DataFrame'>")
with pytest.raises(ValueError, match=msg):
df.quantile(0.1, axis="column")

Expand Down
3 changes: 2 additions & 1 deletion pandas/tests/frame/test_sorting.py
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,8 @@ def test_sort_values(self):
sorted_df = frame.sort_values(by=['B', 'A'], ascending=[True, False])
assert_frame_equal(sorted_df, expected)

msg = "No axis named 2 for object type <class 'type'>"
msg = ("No axis named 2 for object type"
" <class 'pandas.core.frame.DataFrame'>")
with pytest.raises(ValueError, match=msg):
frame.sort_values(by=['A', 'B'], axis=2, inplace=True)

Expand Down
3 changes: 2 additions & 1 deletion pandas/tests/frame/test_timeseries.py
Original file line number Diff line number Diff line change
Expand Up @@ -862,7 +862,8 @@ def test_frame_to_period(self):
pts = df.to_period('M', axis=1)
tm.assert_index_equal(pts.columns, exp.columns.asfreq('M'))

msg = "No axis named 2 for object type <class 'type'>"
msg = ("No axis named 2 for object type"
" <class 'pandas.core.frame.DataFrame'>")
with pytest.raises(ValueError, match=msg):
df.to_period(axis=2)

Expand Down
4 changes: 3 additions & 1 deletion pandas/tests/series/test_analytics.py
Original file line number Diff line number Diff line change
Expand Up @@ -771,6 +771,7 @@ def test_isin_empty(self, empty):
result = s.isin(empty)
tm.assert_series_equal(expected, result)

@pytest.mark.skipif(PY2, reason="pytest.raises match regex fails")
def test_ptp(self):
# GH21614
N = 1000
Expand All @@ -796,7 +797,8 @@ def test_ptp(self):
with tm.assert_produces_warning(FutureWarning, check_stacklevel=False):
tm.assert_series_equal(s.ptp(level=0, skipna=False), expected)

msg = r"No axis named 1 for object type <(class|type) 'type'>"
msg = ("No axis named 1 for object type"
" <class 'pandas.core.series.Series'>")
with pytest.raises(ValueError, match=msg):
with tm.assert_produces_warning(FutureWarning,
check_stacklevel=False):
Expand Down
6 changes: 4 additions & 2 deletions pandas/tests/series/test_missing.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
import pytz

from pandas._libs.tslib import iNaT
from pandas.compat import range
from pandas.compat import PY2, range
from pandas.errors import PerformanceWarning
import pandas.util._test_decorators as td

Expand Down Expand Up @@ -654,14 +654,16 @@ def test_timedelta64_nan(self):
# expected = (datetime_series >= -0.5) & (datetime_series <= 0.5)
# assert_series_equal(selector, expected)

@pytest.mark.skipif(PY2, reason="pytest.raises match regex fails")
def test_dropna_empty(self):
s = Series([])
assert len(s.dropna()) == 0
s.dropna(inplace=True)
assert len(s) == 0

# invalid axis
msg = r"No axis named 1 for object type <(class|type) 'type'>"
msg = ("No axis named 1 for object type"
" <class 'pandas.core.series.Series'>")
with pytest.raises(ValueError, match=msg):
s.dropna(axis=1)

Expand Down
6 changes: 4 additions & 2 deletions pandas/tests/series/test_rank.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
from pandas._libs.algos import Infinity, NegInfinity
from pandas._libs.tslib import iNaT
import pandas.compat as compat
from pandas.compat import product
from pandas.compat import PY2, product
import pandas.util._test_decorators as td

from pandas import NaT, Series, Timestamp, date_range
Expand Down Expand Up @@ -203,10 +203,12 @@ def test_rank_categorical(self):
assert_series_equal(na_ser.rank(na_option='bottom', pct=True), exp_bot)
assert_series_equal(na_ser.rank(na_option='keep', pct=True), exp_keep)

@pytest.mark.skipif(PY2, reason="pytest.raises match regex fails")
def test_rank_signature(self):
s = Series([0, 1])
s.rank(method='average')
msg = r"No axis named average for object type <(class|type) 'type'>"
msg = ("No axis named average for object type"
" <class 'pandas.core.series.Series'>")
with pytest.raises(ValueError, match=msg):
s.rank('average')

Expand Down
6 changes: 5 additions & 1 deletion pandas/tests/series/test_sorting.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,8 @@
import numpy as np
import pytest

from pandas.compat import PY2

from pandas import Categorical, DataFrame, IntervalIndex, MultiIndex, Series
import pandas.util.testing as tm
from pandas.util.testing import assert_almost_equal, assert_series_equal
Expand Down Expand Up @@ -88,6 +90,7 @@ def test_sort_values(self):
with pytest.raises(ValueError, match=msg):
s.sort_values(inplace=True)

@pytest.mark.skipif(PY2, reason="pytest.raises match regex fails")
def test_sort_index(self):
rindex = list(self.ts.index)
random.shuffle(rindex)
Expand All @@ -109,7 +112,8 @@ def test_sort_index(self):
sorted_series = random_order.sort_index(axis=0)
assert_series_equal(sorted_series, self.ts)

msg = r"No axis named 1 for object type <(class|type) 'type'>"
msg = ("No axis named 1 for object type"
" <class 'pandas.core.series.Series'>")
with pytest.raises(ValueError, match=msg):
random_order.sort_values(axis=1)

Expand Down
6 changes: 4 additions & 2 deletions pandas/tests/series/test_timeseries.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@

from pandas._libs.tslib import iNaT
from pandas._libs.tslibs.np_datetime import OutOfBoundsDatetime
from pandas.compat import StringIO, lrange, product
from pandas.compat import PY2, StringIO, lrange, product
from pandas.errors import NullFrequencyError
import pandas.util._test_decorators as td

Expand Down Expand Up @@ -867,6 +867,7 @@ def test_between_time_formats(self):
for time_string in strings:
assert len(ts.between_time(*time_string)) == expected_length

@pytest.mark.skipif(PY2, reason="pytest.raises match regex fails")
def test_between_time_axis(self):
# issue 8839
rng = date_range('1/1/2000', periods=100, freq='10min')
Expand All @@ -876,7 +877,8 @@ def test_between_time_axis(self):

assert len(ts.between_time(stime, etime)) == expected_length
assert len(ts.between_time(stime, etime, axis=0)) == expected_length
msg = r"No axis named 1 for object type <(class|type) 'type'>"
msg = ("No axis named 1 for object type"
" <class 'pandas.core.series.Series'>")
with pytest.raises(ValueError, match=msg):
ts.between_time(stime, etime, axis=1)

Expand Down