@@ -3135,13 +3135,13 @@ def test_html_formatter_manual_format_html(clean_formatter_state):
31353135
31363136
31373137def test_html_formatter_backward_compatibility_repr_rows (df , clean_formatter_state ):
3138- """Test backward compatibility with custom formatter using deprecated repr_rows attribute .
3138+ """Test backward compatibility with custom formatter using deprecated repr_rows.
31393139
31403140 This test validates that the Rust code correctly handles custom formatter
3141- implementations that only have the deprecated `repr_rows` attribute (no `max_rows`) .
3141+ implementations that only have the deprecated `repr_rows` attribute.
31423142
3143- This is critical for supporting custom formatters created before the `max_rows`
3144- attribute was added. Users should be able to pass their custom formatter objects
3143+ This is critical for supporting custom formatters created before `max_rows`
3144+ was added. Users should be able to pass their custom formatter objects
31453145 without breaking the rendering pipeline.
31463146
31473147 Deprecation Timeline:
@@ -3153,7 +3153,7 @@ def test_html_formatter_backward_compatibility_repr_rows(df, clean_formatter_sta
31533153
31543154 # Create a custom formatter class that ONLY has repr_rows (simulating old code)
31553155 class LegacyCustomFormatter :
3156- """Simulates a custom formatter implementation created before max_rows existed."""
3156+ """Simulates a custom formatter created before max_rows existed."""
31573157
31583158 def __init__ (self ):
31593159 # Only set repr_rows, not max_rows (as old formatters would)
@@ -3166,8 +3166,8 @@ def format_html(self, batches, schema):
31663166 # Just return valid HTML to pass validation
31673167 return "<table><tr><td>test</td></tr></table>"
31683168
3169- # Use the legacy formatter with DataFusion
3170- legacy_formatter = LegacyCustomFormatter ()
3169+ # Use the legacy formatter with DataFusion (currently unused)
3170+ _ = LegacyCustomFormatter ()
31713171
31723172 # This should not raise an error even though max_rows doesn't exist
31733173 # The Rust code should fall back to repr_rows
0 commit comments