Skip to content

Conversation

@justine202429
Copy link

@justine202429 justine202429 commented Oct 4, 2025

@justine202429 justine202429 marked this pull request as ready for review October 13, 2025 09:26
Copy link
Member

@Alvaro-Kothe Alvaro-Kothe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It needs tests and an entry in whatsnew.

@Alvaro-Kothe Alvaro-Kothe added the IO Excel read_excel, to_excel label Oct 14, 2025
Comment on lines +1597 to +1600
with ExcelWriter("test.xlsx", engine="openpyxl") as writer:
df.to_excel(writer, sheet_name="Sheet1", merge_cells=merge_cells)

reader = ExcelFile("test.xlsx")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't read and write to CWD, either use a temporary file, or use an in-memory buffer.

DummyClass.assert_called_and_reset()

@td.skip_if_no("openpyxl")
def test_to_excel_multiindex_nan_in_columns(self, merge_cells):
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test all excel engines.

Comment on lines +1601 to +1605
result = pd.read_excel(reader, index_col=0, header=[0, 1])

original_values = df.to_numpy()
result_values = result.to_numpy()
tm.assert_numpy_array_equal(original_values, result_values)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Create an expected DataFrame and use tm.assert_frame_equal

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

IO Excel read_excel, to_excel

3 participants