TST/CLN: salvage or remove FIXME-commented-out tests (#65476)

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: jbrockmendel

pandas/tests/frame/test_block_internals.py

@@ -196,23 +196,29 @@ def test_construction_with_conversions(self):
             {
                 "dt1": Timestamp("20130101").as_unit("s"),
                 "dt2": date_range("20130101", periods=3).astype("M8[s]"),
-                # 'dt3' : date_range('20130101 00:00:01',periods=3,freq='s'),
-                # FIXME: don't leave commented-out
+                "dt3": date_range("20130101 00:00:01", periods=3, freq="s").astype(
+                    "M8[s]"
+                ),
             },
             index=range(3),
         )
         assert expected.dtypes["dt1"] == "M8[s]"
         assert expected.dtypes["dt2"] == "M8[s]"
+        assert expected.dtypes["dt3"] == "M8[s]"
 
         dt1 = np.datetime64("2013-01-01")
         dt2 = np.array(
             ["2013-01-01", "2013-01-02", "2013-01-03"], dtype="datetime64[D]"
         )
-        df = DataFrame({"dt1": dt1, "dt2": dt2})
-
-        # df['dt3'] = np.array(['2013-01-01 00:00:01','2013-01-01
-        # 00:00:02','2013-01-01 00:00:03'],dtype='datetime64[s]')
-        # FIXME: don't leave commented-out
+        dt3 = np.array(
+            [
+                "2013-01-01 00:00:01",
+                "2013-01-01 00:00:02",
+                "2013-01-01 00:00:03",
+            ],
+            dtype="datetime64[s]",
+        )
+        df = DataFrame({"dt1": dt1, "dt2": dt2, "dt3": dt3})
 
         tm.assert_frame_equal(df, expected)
 

pandas/tests/io/formats/test_format.py

@@ -1286,10 +1286,6 @@ def test_info_repr_max_cols(self):
         ):
             assert not has_non_verbose_info_repr(df)
 
-        # FIXME: don't leave commented-out
-        # test verbose overrides
-        # set_option('display.max_info_columns', 4)  # exceeded
-
     def test_pprint_pathological_object(self):
         """
         If the test fails, it at least won't hang.

pandas/tests/io/formats/test_to_string.py

@@ -857,10 +857,7 @@ def test_to_string(self):
         tm.assert_series_equal(recons["B"], biggie["B"])
         assert recons["A"].count() == biggie["A"].count()
         assert (np.abs(recons["A"].dropna() - biggie["A"].dropna()) < 0.1).all()
-
-        # FIXME: don't leave commented-out
-        # expected = ['B', 'A']
-        # assert header == expected
+        assert header == ["B", "A"]
 
         result = biggie.to_string(columns=["A"], col_space=17)
         header = result.split("\n")[0].strip().split()

pandas/tests/io/test_stata.py

@@ -211,16 +211,7 @@ def test_read_dta2(self, datapath):
             parsed_115 = self.read_dta(path2)
         with tm.assert_produces_warning(UserWarning, match=msg):
             parsed_117 = self.read_dta(path3)
-            # FIXME: don't leave commented-out
-            # 113 is buggy due to limits of date format support in Stata
-            # parsed_113 = self.read_dta(
-            # datapath("io", "data", "stata", "stata2_113.dta")
-            # )
-
-        # FIXME: don't leave commented-out
-        # buggy test because of the NaT comparison on certain platforms
-        # Format 113 test fails since it does not support tc and tC formats
-        # tm.assert_frame_equal(parsed_113, expected)
+
         tm.assert_frame_equal(parsed_114, expected)
         tm.assert_frame_equal(parsed_115, expected)
         tm.assert_frame_equal(parsed_117, expected)

pandas/tests/resample/test_period_index.py

@@ -459,10 +459,6 @@ def test_resample_to_quarterly_start_end(self, simple_period_range_series, how):
         expected = ts.asfreq("Q-MAR", how=how)
         expected = expected.reindex(result.index, method="ffill")
 
-        # FIXME: don't leave commented-out
-        # .to_timestamp('D')
-        # expected = expected.resample('Q-MAR').ffill()
-
         tm.assert_series_equal(result, expected)
 
     def test_resample_fill_missing(self):