CLN: test_intersect_str_dates appears to have been mistakenly placed in tests/indexes/test_range.py

[jschendel]

Code Sample, a copy-pastable example if possible

pandas/pandas/tests/indexes/test_range.py

Lines 642 to 649 in 36dadd7

	def test_intersect_str_dates(self):
	dt_dates = [datetime(2012, 2, 9), datetime(2012, 2, 22)]
	i1 = Index(dt_dates, dtype=object)
	i2 = Index(['aa'], dtype=object)
	res = i2.intersection(i1)
	assert len(res) == 0

Problem description

The test_intersect_str_dates test within tests/indexes/test_range.py doesn't appear to have anything to do with RangeIndex. It seems like this should be moved to tests/indexes/test_base.py? Happy to submit a PR if this is the case.

[gfyoung]

That seems like a pretty good home for it. BTW, for reference, you probably could just have submitted a PR directly, as this is relatively trivial.

[jschendel]

Okay, will omit creating an issue for trivial stuff like this in the future. Need to get my brain out of "every PR needs a corresponding issue" mode 😆

[gfyoung]

No worries! I'm just saying this to make your life a little easier (or rather, to streamline the process for you to contribute). We won't get mad if you follow the standard issue --> PR format. 😄