split some tests that were testing behviour of string and datetime series simultaniously

Author: SaturnFromTitan

pandas/tests/series/test_api.py

@@ -108,12 +108,14 @@ def test_getitem_preserve_name(self, datetime_series):
         result = datetime_series[5:10]
         assert result.name == datetime_series.name
 
-    def test_pickle(self, datetime_series, string_series):
-        unp_series = self._pickle_roundtrip(string_series)
+    def test_pickle_datetimes(self, datetime_series):
         unp_ts = self._pickle_roundtrip(datetime_series)
-        assert_series_equal(unp_series, string_series)
         assert_series_equal(unp_ts, datetime_series)
 
+    def test_pickle_strings(self, string_series):
+        unp_series = self._pickle_roundtrip(string_series)
+        assert_series_equal(unp_series, string_series)
+
     def _pickle_roundtrip(self, obj):
 
         with ensure_clean() as path:
@@ -308,13 +310,14 @@ def test_not_hashable(self):
     def test_contains(self, datetime_series):
         tm.assert_contains_all(datetime_series.index, datetime_series)
 
-    def test_iter(self, datetime_series, string_series):
-        for i, val in enumerate(string_series):
-            assert val == string_series[i]
-
+    def test_iter_datetimes(self, datetime_series):
         for i, val in enumerate(datetime_series):
             assert val == datetime_series[i]
 
+    def test_iter_strings(self, string_series):
+        for i, val in enumerate(string_series):
+            assert val == string_series[i]
+
     def test_keys(self, datetime_series):
         # HACK: By doing this in two stages, we avoid 2to3 wrapping the call
         # to .keys() in a list()
@@ -326,23 +329,25 @@ def test_values(self, datetime_series):
             datetime_series.values, datetime_series, check_dtype=False
         )
 
-    def test_iteritems(self, datetime_series, string_series):
-        for idx, val in string_series.iteritems():
-            assert val == string_series[idx]
-
+    def test_iteritems_datetimes(self, datetime_series):
         for idx, val in datetime_series.iteritems():
             assert val == datetime_series[idx]
 
+    def test_iteritems_strings(self, string_series):
+        for idx, val in string_series.iteritems():
+            assert val == string_series[idx]
+
         # assert is lazy (genrators don't define reverse, lists do)
         assert not hasattr(string_series.iteritems(), "reverse")
 
-    def test_items(self, datetime_series, string_series):
-        for idx, val in string_series.items():
-            assert val == string_series[idx]
-
+    def test_items_datetimes(self, datetime_series):
         for idx, val in datetime_series.items():
             assert val == datetime_series[idx]
 
+    def test_items_strings(self, string_series):
+        for idx, val in string_series.items():
+            assert val == string_series[idx]
+
         # assert is lazy (genrators don't define reverse, lists do)
         assert not hasattr(string_series.items(), "reverse")