Skip to content

Commit 5b66616

Browse files
gfyoungpcluo
authored andcommitted
MAINT: Remove assertIsNone from testing (pandas-dev#16102)
1 parent af3c45b commit 5b66616

37 files changed

+134
-139
lines changed

pandas/tests/computation/test_eval.py

+3-3
Original file line numberDiff line numberDiff line change
@@ -1336,15 +1336,15 @@ def test_multi_line_expression(self):
13361336
c = a + b
13371337
d = c + b""", inplace=True)
13381338
assert_frame_equal(expected, df)
1339-
self.assertIsNone(ans)
1339+
assert ans is None
13401340

13411341
expected['a'] = expected['a'] - 1
13421342
expected['e'] = expected['a'] + 2
13431343
ans = df.eval("""
13441344
a = a - 1
13451345
e = a + 2""", inplace=True)
13461346
assert_frame_equal(expected, df)
1347-
self.assertIsNone(ans)
1347+
assert ans is None
13481348

13491349
# multi-line not valid if not all assignments
13501350
with pytest.raises(ValueError):
@@ -1384,7 +1384,7 @@ def test_multi_line_expression_local_variable(self):
13841384
d = c + @local_var
13851385
""", inplace=True)
13861386
assert_frame_equal(expected, df)
1387-
self.assertIsNone(ans)
1387+
assert ans is None
13881388

13891389
def test_assignment_in_query(self):
13901390
# GH 8664

pandas/tests/frame/test_api.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ def test_copy_index_name_checking(self):
3838
ind.name = None
3939
cp = self.frame.copy()
4040
getattr(cp, attr).name = 'foo'
41-
self.assertIsNone(getattr(self.frame, attr).name)
41+
assert getattr(self.frame, attr).name is None
4242

4343
def test_getitem_pop_assign_name(self):
4444
s = self.frame['A']
@@ -116,7 +116,7 @@ def test_new_empty_index(self):
116116
df1 = DataFrame(randn(0, 3))
117117
df2 = DataFrame(randn(0, 3))
118118
df1.index.name = 'foo'
119-
self.assertIsNone(df2.index.name)
119+
assert df2.index.name is None
120120

121121
def test_array_interface(self):
122122
with np.errstate(all='ignore'):

pandas/tests/frame/test_constructors.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ def test_constructor_dtype_nocast_view(self):
9191
def test_constructor_dtype_list_data(self):
9292
df = DataFrame([[1, '2'],
9393
[None, 'a']], dtype=object)
94-
self.assertIsNone(df.loc[1, 0])
94+
assert df.loc[1, 0] is None
9595
self.assertEqual(df.loc[0, 1], '2')
9696

9797
def test_constructor_list_frames(self):

pandas/tests/frame/test_indexing.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -81,15 +81,15 @@ def test_get(self):
8181
b = self.frame.get('B')
8282
assert_series_equal(b, self.frame['B'])
8383

84-
self.assertIsNone(self.frame.get('foo'))
84+
assert self.frame.get('foo') is None
8585
assert_series_equal(self.frame.get('foo', self.frame['B']),
8686
self.frame['B'])
8787
# None
8888
# GH 5652
8989
for df in [DataFrame(), DataFrame(columns=list('AB')),
9090
DataFrame(columns=list('AB'), index=range(3))]:
9191
result = df.get(None)
92-
self.assertIsNone(result)
92+
assert result is None
9393

9494
def test_getitem_iterator(self):
9595
idx = iter(['A', 'B', 'C'])

pandas/tests/frame/test_repr_info.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -189,7 +189,7 @@ def test_latex_repr(self):
189189
self.assertEqual(result, df._repr_latex_())
190190

191191
# GH 12182
192-
self.assertIsNone(df._repr_latex_())
192+
assert df._repr_latex_() is None
193193

194194
@tm.capture_stdout
195195
def test_info(self):

pandas/tests/frame/test_timeseries.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -415,8 +415,8 @@ def test_first_last_valid(self):
415415

416416
# GH12800
417417
empty = DataFrame()
418-
self.assertIsNone(empty.last_valid_index())
419-
self.assertIsNone(empty.first_valid_index())
418+
assert empty.last_valid_index() is None
419+
assert empty.first_valid_index() is None
420420

421421
def test_at_time_frame(self):
422422
rng = date_range('1/1/2000', '1/5/2000', freq='5min')

pandas/tests/groupby/test_groupby.py

+4-4
Original file line numberDiff line numberDiff line change
@@ -1972,11 +1972,11 @@ def test_apply_frame_yield_constant(self):
19721972
# GH13568
19731973
result = self.df.groupby(['A', 'B']).apply(len)
19741974
self.assertTrue(isinstance(result, Series))
1975-
self.assertIsNone(result.name)
1975+
assert result.name is None
19761976

19771977
result = self.df.groupby(['A', 'B'])[['C', 'D']].apply(len)
19781978
self.assertTrue(isinstance(result, Series))
1979-
self.assertIsNone(result.name)
1979+
assert result.name is None
19801980

19811981
def test_apply_frame_to_series(self):
19821982
grouped = self.df.groupby(['A', 'B'])
@@ -2954,7 +2954,7 @@ def test_no_nonsense_name(self):
29542954
s.name = None
29552955

29562956
result = s.groupby(self.frame['A']).agg(np.sum)
2957-
self.assertIsNone(result.name)
2957+
assert result.name is None
29582958

29592959
def test_multifunc_sum_bug(self):
29602960
# GH #1065
@@ -3060,7 +3060,7 @@ def test_no_dummy_key_names(self):
30603060
# GH #1291
30613061

30623062
result = self.df.groupby(self.df['A'].values).sum()
3063-
self.assertIsNone(result.index.name)
3063+
assert result.index.name is None
30643064

30653065
result = self.df.groupby([self.df['A'].values, self.df['B'].values
30663066
]).sum()

pandas/tests/indexes/common.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -195,7 +195,7 @@ def test_set_name_methods(self):
195195
res = ind.rename(new_name, inplace=True)
196196

197197
# should return None
198-
self.assertIsNone(res)
198+
assert res is None
199199
self.assertEqual(ind.name, new_name)
200200
self.assertEqual(ind.names, [new_name])
201201
# with assertRaisesRegexp(TypeError, "list-like"):

pandas/tests/indexes/datetimes/test_datetime.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -480,7 +480,7 @@ def test_take(self):
480480
for taken in [taken1, taken2]:
481481
tm.assert_index_equal(taken, expected)
482482
assert isinstance(taken, DatetimeIndex)
483-
self.assertIsNone(taken.freq)
483+
assert taken.freq is None
484484
self.assertEqual(taken.tz, expected.tz)
485485
self.assertEqual(taken.name, expected.name)
486486

pandas/tests/indexes/datetimes/test_ops.py

+15-15
Original file line numberDiff line numberDiff line change
@@ -197,7 +197,7 @@ def test_repeat_range(self):
197197
rng = date_range('1/1/2000', '1/1/2001')
198198

199199
result = rng.repeat(5)
200-
self.assertIsNone(result.freq)
200+
assert result.freq is None
201201
self.assertEqual(len(result), 5 * len(rng))
202202

203203
for tz in self.tz:
@@ -206,14 +206,14 @@ def test_repeat_range(self):
206206
'2001-01-02', '2001-01-02'], tz=tz)
207207
for res in [index.repeat(2), np.repeat(index, 2)]:
208208
tm.assert_index_equal(res, exp)
209-
self.assertIsNone(res.freq)
209+
assert res.freq is None
210210

211211
index = pd.date_range('2001-01-01', periods=2, freq='2D', tz=tz)
212212
exp = pd.DatetimeIndex(['2001-01-01', '2001-01-01',
213213
'2001-01-03', '2001-01-03'], tz=tz)
214214
for res in [index.repeat(2), np.repeat(index, 2)]:
215215
tm.assert_index_equal(res, exp)
216-
self.assertIsNone(res.freq)
216+
assert res.freq is None
217217

218218
index = pd.DatetimeIndex(['2001-01-01', 'NaT', '2003-01-01'],
219219
tz=tz)
@@ -223,7 +223,7 @@ def test_repeat_range(self):
223223
tz=tz)
224224
for res in [index.repeat(3), np.repeat(index, 3)]:
225225
tm.assert_index_equal(res, exp)
226-
self.assertIsNone(res.freq)
226+
assert res.freq is None
227227

228228
def test_repeat(self):
229229
reps = 2
@@ -242,7 +242,7 @@ def test_repeat(self):
242242

243243
res = rng.repeat(reps)
244244
tm.assert_index_equal(res, expected_rng)
245-
self.assertIsNone(res.freq)
245+
assert res.freq is None
246246

247247
tm.assert_index_equal(np.repeat(rng, reps), expected_rng)
248248
tm.assertRaisesRegexp(ValueError, msg, np.repeat,
@@ -694,26 +694,26 @@ def test_order(self):
694694
for idx, expected in [(idx1, exp1), (idx2, exp2), (idx3, exp3)]:
695695
ordered = idx.sort_values()
696696
tm.assert_index_equal(ordered, expected)
697-
self.assertIsNone(ordered.freq)
697+
assert ordered.freq is None
698698

699699
ordered = idx.sort_values(ascending=False)
700700
tm.assert_index_equal(ordered, expected[::-1])
701-
self.assertIsNone(ordered.freq)
701+
assert ordered.freq is None
702702

703703
ordered, indexer = idx.sort_values(return_indexer=True)
704704
tm.assert_index_equal(ordered, expected)
705705

706706
exp = np.array([0, 4, 3, 1, 2])
707707
tm.assert_numpy_array_equal(indexer, exp, check_dtype=False)
708-
self.assertIsNone(ordered.freq)
708+
assert ordered.freq is None
709709

710710
ordered, indexer = idx.sort_values(return_indexer=True,
711711
ascending=False)
712712
tm.assert_index_equal(ordered, expected[::-1])
713713

714714
exp = np.array([2, 1, 3, 4, 0])
715715
tm.assert_numpy_array_equal(indexer, exp, check_dtype=False)
716-
self.assertIsNone(ordered.freq)
716+
assert ordered.freq is None
717717

718718
def test_getitem(self):
719719
idx1 = pd.date_range('2011-01-01', '2011-01-31', freq='D', name='idx')
@@ -757,10 +757,10 @@ def test_drop_duplicates_metadata(self):
757757
self.assertEqual(idx.freq, result.freq)
758758

759759
idx_dup = idx.append(idx)
760-
self.assertIsNone(idx_dup.freq) # freq is reset
760+
assert idx_dup.freq is None # freq is reset
761761
result = idx_dup.drop_duplicates()
762762
tm.assert_index_equal(idx, result)
763-
self.assertIsNone(result.freq)
763+
assert result.freq is None
764764

765765
def test_drop_duplicates(self):
766766
# to check Index/Series compat
@@ -816,14 +816,14 @@ def test_take(self):
816816
'2011-01-06'],
817817
freq=None, tz=idx.tz, name='idx')
818818
tm.assert_index_equal(result, expected)
819-
self.assertIsNone(result.freq)
819+
assert result.freq is None
820820

821821
result = idx.take([-3, 2, 5])
822822
expected = DatetimeIndex(['2011-01-29', '2011-01-03',
823823
'2011-01-06'],
824824
freq=None, tz=idx.tz, name='idx')
825825
tm.assert_index_equal(result, expected)
826-
self.assertIsNone(result.freq)
826+
assert result.freq is None
827827

828828
def test_take_invalid_kwargs(self):
829829
idx = pd.date_range('2011-01-01', '2011-01-31', freq='D', name='idx')
@@ -1145,7 +1145,7 @@ def test_getitem(self):
11451145
fancy_indexed = self.rng[[4, 3, 2, 1, 0]]
11461146
self.assertEqual(len(fancy_indexed), 5)
11471147
assert isinstance(fancy_indexed, DatetimeIndex)
1148-
self.assertIsNone(fancy_indexed.freq)
1148+
assert fancy_indexed.freq is None
11491149

11501150
# 32-bit vs. 64-bit platforms
11511151
self.assertEqual(self.rng[4], self.rng[np.int_(4)])
@@ -1240,7 +1240,7 @@ def test_getitem(self):
12401240
fancy_indexed = self.rng[[4, 3, 2, 1, 0]]
12411241
self.assertEqual(len(fancy_indexed), 5)
12421242
assert isinstance(fancy_indexed, DatetimeIndex)
1243-
self.assertIsNone(fancy_indexed.freq)
1243+
assert fancy_indexed.freq is None
12441244

12451245
# 32-bit vs. 64-bit platforms
12461246
self.assertEqual(self.rng[4], self.rng[np.int_(4)])

pandas/tests/indexes/datetimes/test_setops.py

+6-6
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ def test_union_freq_both_none(self):
6565

6666
result = expected.union(expected)
6767
tm.assert_index_equal(result, expected)
68-
self.assertIsNone(result.freq)
68+
assert result.freq is None
6969

7070
def test_union_dataframe_index(self):
7171
rng1 = date_range('1/1/1999', '1/1/2012', freq='MS')
@@ -137,7 +137,7 @@ def test_intersection(self):
137137
result = base.intersection(rng)
138138
tm.assert_index_equal(result, expected)
139139
self.assertEqual(result.name, expected.name)
140-
self.assertIsNone(result.freq)
140+
assert result.freq is None
141141
self.assertEqual(result.tz, expected.tz)
142142

143143
# empty same freq GH2129
@@ -251,7 +251,7 @@ def test_outer_join(self):
251251

252252
the_join = left.join(right, how='outer')
253253
assert isinstance(the_join, DatetimeIndex)
254-
self.assertIsNone(the_join.freq)
254+
assert the_join.freq is None
255255

256256
# non-overlapping, no gap
257257
left = self.rng[:5]
@@ -265,7 +265,7 @@ def test_outer_join(self):
265265

266266
the_join = self.rng.join(rng, how='outer')
267267
assert isinstance(the_join, DatetimeIndex)
268-
self.assertIsNone(the_join.freq)
268+
assert the_join.freq is None
269269

270270
def test_union_not_cacheable(self):
271271
rng = date_range('1/1/2000', periods=50, freq=Minute())
@@ -395,7 +395,7 @@ def test_outer_join(self):
395395

396396
the_join = left.join(right, how='outer')
397397
assert isinstance(the_join, DatetimeIndex)
398-
self.assertIsNone(the_join.freq)
398+
assert the_join.freq is None
399399

400400
# non-overlapping, no gap
401401
left = self.rng[:5]
@@ -409,7 +409,7 @@ def test_outer_join(self):
409409

410410
the_join = self.rng.join(rng, how='outer')
411411
assert isinstance(the_join, DatetimeIndex)
412-
self.assertIsNone(the_join.freq)
412+
assert the_join.freq is None
413413

414414
def test_intersection_bug(self):
415415
# GH #771

pandas/tests/indexes/test_base.py

+6-6
Original file line numberDiff line numberDiff line change
@@ -659,12 +659,12 @@ def test_intersection(self):
659659

660660
second.name = 'B'
661661
intersect = first.intersection(second)
662-
self.assertIsNone(intersect.name)
662+
assert intersect.name is None
663663

664664
first.name = None
665665
second.name = 'B'
666666
intersect = first.intersection(second)
667-
self.assertIsNone(intersect.name)
667+
assert intersect.name is None
668668

669669
def test_union(self):
670670
first = self.strIndex[5:20]
@@ -844,7 +844,7 @@ def test_append_empty_preserve_name(self):
844844
right = Index([1, 2, 3], name='bar')
845845

846846
result = left.append(right)
847-
self.assertIsNone(result.name)
847+
assert result.name is None
848848

849849
def test_add_string(self):
850850
# from bug report
@@ -896,12 +896,12 @@ def test_symmetric_difference(self):
896896
result = idx1.symmetric_difference(idx2)
897897
expected = Index([1, 5])
898898
self.assertTrue(tm.equalContents(result, expected))
899-
self.assertIsNone(result.name)
899+
assert result.name is None
900900

901901
# __xor__ syntax
902902
expected = idx1 ^ idx2
903903
self.assertTrue(tm.equalContents(result, expected))
904-
self.assertIsNone(result.name)
904+
assert result.name is None
905905

906906
# multiIndex
907907
idx1 = MultiIndex.from_tuples(self.tuples)
@@ -1015,7 +1015,7 @@ def test_format_none(self):
10151015

10161016
idx = Index(values)
10171017
idx.format()
1018-
self.assertIsNone(idx[3])
1018+
assert idx[3] is None
10191019

10201020
def test_logical_compat(self):
10211021
idx = self.create_index()

0 commit comments

Comments
 (0)