Skip to content

Commit b9cb3fe

Browse files
gfyoungjreback
authored andcommitted
MAINT: Remove tm.assertNotIsInstance (#16061)
1 parent 97215b3 commit b9cb3fe

16 files changed

+151
-153
lines changed

pandas/tests/indexes/common.py

+12-10
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
# -*- coding: utf-8 -*-
22

3+
import pytest
4+
35
from pandas import compat
46
from pandas.compat import PY3
57

@@ -793,41 +795,41 @@ def test_numpy_ufuncs(self):
793795
if isinstance(idx, DatetimeIndexOpsMixin):
794796
# raise TypeError or ValueError (PeriodIndex)
795797
# PeriodIndex behavior should be changed in future version
796-
with tm.assertRaises(Exception):
798+
with pytest.raises(Exception):
797799
with np.errstate(all='ignore'):
798800
func(idx)
799801
elif isinstance(idx, (Float64Index, Int64Index, UInt64Index)):
800802
# coerces to float (e.g. np.sin)
801803
with np.errstate(all='ignore'):
802804
result = func(idx)
803805
exp = Index(func(idx.values), name=idx.name)
804-
self.assert_index_equal(result, exp)
805-
self.assertIsInstance(result, pd.Float64Index)
806+
807+
tm.assert_index_equal(result, exp)
808+
assert isinstance(result, pd.Float64Index)
806809
else:
807810
# raise AttributeError or TypeError
808811
if len(idx) == 0:
809812
continue
810813
else:
811-
with tm.assertRaises(Exception):
814+
with pytest.raises(Exception):
812815
with np.errstate(all='ignore'):
813816
func(idx)
814817

815818
for func in [np.isfinite, np.isinf, np.isnan, np.signbit]:
816819
if isinstance(idx, DatetimeIndexOpsMixin):
817820
# raise TypeError or ValueError (PeriodIndex)
818-
with tm.assertRaises(Exception):
821+
with pytest.raises(Exception):
819822
func(idx)
820823
elif isinstance(idx, (Float64Index, Int64Index, UInt64Index)):
821-
# results in bool array
824+
# Results in bool array
822825
result = func(idx)
823-
exp = func(idx.values)
824-
self.assertIsInstance(result, np.ndarray)
825-
tm.assertNotIsInstance(result, Index)
826+
assert isinstance(result, np.ndarray)
827+
assert not isinstance(result, Index)
826828
else:
827829
if len(idx) == 0:
828830
continue
829831
else:
830-
with tm.assertRaises(Exception):
832+
with pytest.raises(Exception):
831833
func(idx)
832834

833835
def test_hasnans_isnans(self):

pandas/tests/indexes/datetimes/test_indexing.py

+23-23
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
1-
import numpy as np
1+
import pytest
22

3+
import numpy as np
34
import pandas as pd
45
import pandas.util.testing as tm
56
import pandas.compat as compat
@@ -54,9 +55,9 @@ def test_insert(self):
5455
expected = Index([datetime(2000, 1, 4), 'inserted',
5556
datetime(2000, 1, 1),
5657
datetime(2000, 1, 2)], name='idx')
57-
self.assertNotIsInstance(result, DatetimeIndex)
58+
assert not isinstance(result, DatetimeIndex)
5859
tm.assert_index_equal(result, expected)
59-
self.assertEqual(result.name, expected.name)
60+
assert result.name == expected.name
6061

6162
idx = date_range('1/1/2000', periods=3, freq='M', name='idx')
6263

@@ -85,33 +86,32 @@ def test_insert(self):
8586
for n, d, expected in cases:
8687
result = idx.insert(n, d)
8788
tm.assert_index_equal(result, expected)
88-
self.assertEqual(result.name, expected.name)
89-
self.assertEqual(result.freq, expected.freq)
89+
assert result.name == expected.name
90+
assert result.freq == expected.freq
9091

9192
# reset freq to None
9293
result = idx.insert(3, datetime(2000, 1, 2))
9394
expected = DatetimeIndex(['2000-01-31', '2000-02-29', '2000-03-31',
9495
'2000-01-02'], name='idx', freq=None)
9596
tm.assert_index_equal(result, expected)
96-
self.assertEqual(result.name, expected.name)
97-
self.assertTrue(result.freq is None)
97+
assert result.name == expected.name
98+
assert result.freq is None
9899

99100
# GH 7299
100101
tm._skip_if_no_pytz()
101102
import pytz
102103

103104
idx = date_range('1/1/2000', periods=3, freq='D', tz='Asia/Tokyo',
104105
name='idx')
105-
with tm.assertRaises(ValueError):
106-
result = idx.insert(3, pd.Timestamp('2000-01-04'))
107-
with tm.assertRaises(ValueError):
108-
result = idx.insert(3, datetime(2000, 1, 4))
109-
with tm.assertRaises(ValueError):
110-
result = idx.insert(3, pd.Timestamp('2000-01-04', tz='US/Eastern'))
111-
with tm.assertRaises(ValueError):
112-
result = idx.insert(3,
113-
datetime(2000, 1, 4,
114-
tzinfo=pytz.timezone('US/Eastern')))
106+
with pytest.raises(ValueError):
107+
idx.insert(3, pd.Timestamp('2000-01-04'))
108+
with pytest.raises(ValueError):
109+
idx.insert(3, datetime(2000, 1, 4))
110+
with pytest.raises(ValueError):
111+
idx.insert(3, pd.Timestamp('2000-01-04', tz='US/Eastern'))
112+
with pytest.raises(ValueError):
113+
idx.insert(3, datetime(2000, 1, 4,
114+
tzinfo=pytz.timezone('US/Eastern')))
115115

116116
for tz in ['US/Pacific', 'Asia/Singapore']:
117117
idx = date_range('1/1/2000 09:00', periods=6, freq='H', tz=tz,
@@ -124,9 +124,9 @@ def test_insert(self):
124124

125125
result = idx.insert(6, d)
126126
tm.assert_index_equal(result, expected)
127-
self.assertEqual(result.name, expected.name)
128-
self.assertEqual(result.freq, expected.freq)
129-
self.assertEqual(result.tz, expected.tz)
127+
assert result.name == expected.name
128+
assert result.freq == expected.freq
129+
assert result.tz == expected.tz
130130

131131
expected = DatetimeIndex(['2000-01-01 09:00', '2000-01-01 10:00',
132132
'2000-01-01 11:00',
@@ -139,9 +139,9 @@ def test_insert(self):
139139
pytz.timezone(tz).localize(datetime(2000, 1, 1, 10))]:
140140
result = idx.insert(6, d)
141141
tm.assert_index_equal(result, expected)
142-
self.assertEqual(result.name, expected.name)
143-
self.assertTrue(result.freq is None)
144-
self.assertEqual(result.tz, expected.tz)
142+
assert result.name == expected.name
143+
assert result.tz == expected.tz
144+
assert result.freq is None
145145

146146
def test_delete(self):
147147
idx = date_range(start='2000-01-01', periods=5, freq='M', name='idx')

pandas/tests/indexes/test_base.py

+15-15
Original file line numberDiff line numberDiff line change
@@ -283,27 +283,27 @@ def test_constructor_dtypes(self):
283283
for idx in [Index(np.array([1, 2, 3], dtype=int)),
284284
Index(np.array([1, 2, 3], dtype=int), dtype=int),
285285
Index([1, 2, 3], dtype=int)]:
286-
self.assertIsInstance(idx, Int64Index)
286+
assert isinstance(idx, Int64Index)
287287

288-
# these should coerce
288+
# These should coerce
289289
for idx in [Index(np.array([1., 2., 3.], dtype=float), dtype=int),
290290
Index([1., 2., 3.], dtype=int)]:
291-
self.assertIsInstance(idx, Int64Index)
291+
assert isinstance(idx, Int64Index)
292292

293293
for idx in [Index(np.array([1., 2., 3.], dtype=float)),
294294
Index(np.array([1, 2, 3], dtype=int), dtype=float),
295295
Index(np.array([1., 2., 3.], dtype=float), dtype=float),
296296
Index([1, 2, 3], dtype=float),
297297
Index([1., 2., 3.], dtype=float)]:
298-
self.assertIsInstance(idx, Float64Index)
298+
assert isinstance(idx, Float64Index)
299299

300300
for idx in [Index(np.array([True, False, True], dtype=bool)),
301301
Index([True, False, True]),
302302
Index(np.array([True, False, True], dtype=bool),
303303
dtype=bool),
304304
Index([True, False, True], dtype=bool)]:
305-
self.assertIsInstance(idx, Index)
306-
self.assertEqual(idx.dtype, object)
305+
assert isinstance(idx, Index)
306+
assert idx.dtype == object
307307

308308
for idx in [Index(np.array([1, 2, 3], dtype=int), dtype='category'),
309309
Index([1, 2, 3], dtype='category'),
@@ -312,32 +312,32 @@ def test_constructor_dtypes(self):
312312
dtype='category'),
313313
Index([datetime(2011, 1, 1), datetime(2011, 1, 2)],
314314
dtype='category')]:
315-
self.assertIsInstance(idx, CategoricalIndex)
315+
assert isinstance(idx, CategoricalIndex)
316316

317317
for idx in [Index(np.array([np_datetime64_compat('2011-01-01'),
318318
np_datetime64_compat('2011-01-02')])),
319319
Index([datetime(2011, 1, 1), datetime(2011, 1, 2)])]:
320-
self.assertIsInstance(idx, DatetimeIndex)
320+
assert isinstance(idx, DatetimeIndex)
321321

322322
for idx in [Index(np.array([np_datetime64_compat('2011-01-01'),
323323
np_datetime64_compat('2011-01-02')]),
324324
dtype=object),
325325
Index([datetime(2011, 1, 1),
326326
datetime(2011, 1, 2)], dtype=object)]:
327-
self.assertNotIsInstance(idx, DatetimeIndex)
328-
self.assertIsInstance(idx, Index)
329-
self.assertEqual(idx.dtype, object)
327+
assert not isinstance(idx, DatetimeIndex)
328+
assert isinstance(idx, Index)
329+
assert idx.dtype == object
330330

331331
for idx in [Index(np.array([np.timedelta64(1, 'D'), np.timedelta64(
332332
1, 'D')])), Index([timedelta(1), timedelta(1)])]:
333-
self.assertIsInstance(idx, TimedeltaIndex)
333+
assert isinstance(idx, TimedeltaIndex)
334334

335335
for idx in [Index(np.array([np.timedelta64(1, 'D'),
336336
np.timedelta64(1, 'D')]), dtype=object),
337337
Index([timedelta(1), timedelta(1)], dtype=object)]:
338-
self.assertNotIsInstance(idx, TimedeltaIndex)
339-
self.assertIsInstance(idx, Index)
340-
self.assertEqual(idx.dtype, object)
338+
assert not isinstance(idx, TimedeltaIndex)
339+
assert isinstance(idx, Index)
340+
assert idx.dtype == object
341341

342342
def test_constructor_dtypes_datetime(self):
343343

pandas/tests/indexes/test_category.py

+38-32
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
# -*- coding: utf-8 -*-
22

3+
import pytest
4+
35
import pandas.util.testing as tm
46
from pandas.core.indexes.api import Index, CategoricalIndex
57
from .common import Base
@@ -37,62 +39,66 @@ def test_construction(self):
3739

3840
result = Index(ci)
3941
tm.assert_index_equal(result, ci, exact=True)
40-
self.assertFalse(result.ordered)
42+
assert not result.ordered
4143

4244
result = Index(ci.values)
4345
tm.assert_index_equal(result, ci, exact=True)
44-
self.assertFalse(result.ordered)
46+
assert not result.ordered
4547

4648
# empty
4749
result = CategoricalIndex(categories=categories)
48-
self.assert_index_equal(result.categories, Index(categories))
50+
tm.assert_index_equal(result.categories, Index(categories))
4951
tm.assert_numpy_array_equal(result.codes, np.array([], dtype='int8'))
50-
self.assertFalse(result.ordered)
52+
assert not result.ordered
5153

5254
# passing categories
5355
result = CategoricalIndex(list('aabbca'), categories=categories)
54-
self.assert_index_equal(result.categories, Index(categories))
56+
tm.assert_index_equal(result.categories, Index(categories))
5557
tm.assert_numpy_array_equal(result.codes,
56-
np.array([0, 0, 1, 1, 2, 0], dtype='int8'))
58+
np.array([0, 0, 1,
59+
1, 2, 0], dtype='int8'))
5760

5861
c = pd.Categorical(list('aabbca'))
5962
result = CategoricalIndex(c)
60-
self.assert_index_equal(result.categories, Index(list('abc')))
63+
tm.assert_index_equal(result.categories, Index(list('abc')))
6164
tm.assert_numpy_array_equal(result.codes,
62-
np.array([0, 0, 1, 1, 2, 0], dtype='int8'))
63-
self.assertFalse(result.ordered)
65+
np.array([0, 0, 1,
66+
1, 2, 0], dtype='int8'))
67+
assert not result.ordered
6468

6569
result = CategoricalIndex(c, categories=categories)
66-
self.assert_index_equal(result.categories, Index(categories))
70+
tm.assert_index_equal(result.categories, Index(categories))
6771
tm.assert_numpy_array_equal(result.codes,
68-
np.array([0, 0, 1, 1, 2, 0], dtype='int8'))
69-
self.assertFalse(result.ordered)
72+
np.array([0, 0, 1,
73+
1, 2, 0], dtype='int8'))
74+
assert not result.ordered
7075

7176
ci = CategoricalIndex(c, categories=list('abcd'))
7277
result = CategoricalIndex(ci)
73-
self.assert_index_equal(result.categories, Index(categories))
78+
tm.assert_index_equal(result.categories, Index(categories))
7479
tm.assert_numpy_array_equal(result.codes,
75-
np.array([0, 0, 1, 1, 2, 0], dtype='int8'))
76-
self.assertFalse(result.ordered)
80+
np.array([0, 0, 1,
81+
1, 2, 0], dtype='int8'))
82+
assert not result.ordered
7783

7884
result = CategoricalIndex(ci, categories=list('ab'))
79-
self.assert_index_equal(result.categories, Index(list('ab')))
85+
tm.assert_index_equal(result.categories, Index(list('ab')))
8086
tm.assert_numpy_array_equal(result.codes,
81-
np.array([0, 0, 1, 1, -1, 0],
82-
dtype='int8'))
83-
self.assertFalse(result.ordered)
87+
np.array([0, 0, 1,
88+
1, -1, 0], dtype='int8'))
89+
assert not result.ordered
8490

8591
result = CategoricalIndex(ci, categories=list('ab'), ordered=True)
86-
self.assert_index_equal(result.categories, Index(list('ab')))
92+
tm.assert_index_equal(result.categories, Index(list('ab')))
8793
tm.assert_numpy_array_equal(result.codes,
88-
np.array([0, 0, 1, 1, -1, 0],
89-
dtype='int8'))
90-
self.assertTrue(result.ordered)
94+
np.array([0, 0, 1,
95+
1, -1, 0], dtype='int8'))
96+
assert result.ordered
9197

9298
# turn me to an Index
9399
result = Index(np.array(ci))
94-
self.assertIsInstance(result, Index)
95-
self.assertNotIsInstance(result, CategoricalIndex)
100+
assert isinstance(result, Index)
101+
assert not isinstance(result, CategoricalIndex)
96102

97103
def test_construction_with_dtype(self):
98104

@@ -325,9 +331,9 @@ def test_delete(self):
325331
expected = CategoricalIndex(list('aabbc'), categories=categories)
326332
tm.assert_index_equal(result, expected, exact=True)
327333

328-
with tm.assertRaises((IndexError, ValueError)):
329-
# either depeidnig on numpy version
330-
result = ci.delete(10)
334+
with pytest.raises((IndexError, ValueError)):
335+
# Either depending on NumPy version
336+
ci.delete(10)
331337

332338
def test_astype(self):
333339

@@ -336,12 +342,12 @@ def test_astype(self):
336342
tm.assert_index_equal(result, ci, exact=True)
337343

338344
result = ci.astype(object)
339-
self.assert_index_equal(result, Index(np.array(ci)))
345+
tm.assert_index_equal(result, Index(np.array(ci)))
340346

341347
# this IS equal, but not the same class
342-
self.assertTrue(result.equals(ci))
343-
self.assertIsInstance(result, Index)
344-
self.assertNotIsInstance(result, CategoricalIndex)
348+
assert result.equals(ci)
349+
assert isinstance(result, Index)
350+
assert not isinstance(result, CategoricalIndex)
345351

346352
# interval
347353
ii = IntervalIndex.from_arrays(left=[-0.001, 2.0],

pandas/tests/indexes/test_multi.py

+6-6
Original file line numberDiff line numberDiff line change
@@ -539,17 +539,17 @@ def test_astype(self):
539539
def test_constructor_single_level(self):
540540
single_level = MultiIndex(levels=[['foo', 'bar', 'baz', 'qux']],
541541
labels=[[0, 1, 2, 3]], names=['first'])
542-
tm.assertIsInstance(single_level, Index)
543-
self.assertNotIsInstance(single_level, MultiIndex)
544-
self.assertEqual(single_level.name, 'first')
542+
assert isinstance(single_level, Index)
543+
assert not isinstance(single_level, MultiIndex)
544+
assert single_level.name == 'first'
545545

546546
single_level = MultiIndex(levels=[['foo', 'bar', 'baz', 'qux']],
547547
labels=[[0, 1, 2, 3]])
548-
self.assertIsNone(single_level.name)
548+
assert single_level.name is None
549549

550550
def test_constructor_no_levels(self):
551-
assertRaisesRegexp(ValueError, "non-zero number of levels/labels",
552-
MultiIndex, levels=[], labels=[])
551+
tm.assertRaisesRegexp(ValueError, "non-zero number of levels/labels",
552+
MultiIndex, levels=[], labels=[])
553553
both_re = re.compile('Must pass both levels and labels')
554554
with tm.assertRaisesRegexp(TypeError, both_re):
555555
MultiIndex(levels=[])

0 commit comments

Comments
 (0)