test fixups

Author: jolt-ai-bot

pandas/tests/io/excel/test_adjacent_tables.py

@@ -1272,6 +1272,80 @@ def test_excel_read_tables_with_and_without_blank_row(self, tmp_path):
     def test_excel_read_tables_with_and_without_blank_row(self, engine_and_read_ext, tmp_path):
         """
         GH-61123
+        """
+    def test_excel_read_tables_with_and_without_blank_row(self, engine_and_read_ext, tmp_path):
+        engine, read_ext = engine_and_read_ext
+
+        # Skip incompatible engine/extension combinations
+        if engine == 'xlrd' and read_ext != '.xls':
+            pytest.skip(f"Engine {engine} not compatible with {read_ext}")
+        if engine == 'odf' and read_ext != '.ods':
+            pytest.skip(f"Engine {engine} not compatible with {read_ext}")
+        if engine == 'pyxlsb' and read_ext != '.xlsb':
+            pytest.skip(f"Engine {engine} not compatible with {read_ext}")
+
+        # Map reader engines to appropriate writer engines
+        writer_engine = None
+        if read_ext == '.xlsx' or read_ext == '.xlsm':
+            writer_engine = 'openpyxl'
+        elif read_ext == '.xls':
+            writer_engine = 'xlwt'
+        elif read_ext == '.xlsb':
+            writer_engine = 'xlsxwriter'  # Use xlsxwriter for xlsb files
+        elif read_ext == '.ods':
+            writer_engine = 'odf'
+
+        if writer_engine is None:
+            pytest.skip(f"No writer engine available for {read_ext}")
+
+        try:
+            # Create test files with tables with and without blank rows between them
+            # File 1: Two tables with a blank row between
+            file1 = tmp_path / f"test1{read_ext}"
+            df_upper = pd.DataFrame({"A": [1, 2, 3], "B": [4, 5, 6]})
+            df_lower = pd.DataFrame({"A": [7, 8, 9], "B": [10, 11, 12]})
+
+            with pd.ExcelWriter(file1, engine=writer_engine) as writer:
+                df_upper.to_excel(writer, sheet_name="Sheet1", index=False)
+                # Add blank row by starting lower table at row 5 (0-based index + header)
+                df_lower.to_excel(writer, sheet_name="Sheet1", startrow=5, index=False)
+
+            # File 2: Two tables with no blank row
+            file2 = tmp_path / f"test2{read_ext}"
+            with pd.ExcelWriter(file2, engine=writer_engine) as writer:
+                df_upper.to_excel(writer, sheet_name="Sheet1", index=False)
+                # No blank row, lower table starts right after (row 4 = header of second table)
+                df_lower.to_excel(writer, sheet_name="Sheet1", startrow=4, index=False)
+
+            # Read first 3 rows (header + 3 data rows)
+            # Using nrows=3 to get exactly the upper table without blank rows
+            df1 = pd.read_excel(file1, header=0, nrows=3, engine=engine)
+            df2 = pd.read_excel(file2, header=0, nrows=3, engine=engine)
+
+            # Expected data - just the upper table
+            expected = pd.DataFrame({"A": [1, 2, 3], "B": [4, 5, 6]})
+
+            # Check content
+            tm.assert_frame_equal(df1, expected)
+            tm.assert_frame_equal(df2, expected)
+
+            # Verify we didn't read the header of the next table in df2
+            # If we did, the last row would contain column headers from the second table
+            assert df1.shape == (3, 2), f"Expected (3, 2) but got {df1.shape}"
+            assert df2.shape == (3, 2), f"Expected (3, 2) but got {df2.shape}"
+
+            # Fix the comparison warning by checking specific values instead
+            assert df2.iloc[-1, 0] == 3, f"Expected 3 but got {df2.iloc[-1, 0]}"
+            assert df2.iloc[-1, 1] == 6, f"Expected 6 but got {df2.iloc[-1, 1]}"
+        except ImportError:
+            pytest.skip(f"Required writer engine {writer_engine} not available")
+        except ValueError as e:
+            if "No Excel writer" in str(e):
+                pytest.skip(f"Excel writer {writer_engine} not available")
+            else:
+                raise
+        """
+        GH-61123
         Test that nrows parameter correctly handles adjacent tables with and without blank rows.
         """
         engine, read_ext = engine_and_read_ext