From 07c14c6e6435df69bca73242fe8fd2082b5b2968 Mon Sep 17 00:00:00 2001 From: Darren Burns Date: Wed, 4 May 2022 12:11:12 +0100 Subject: [PATCH] Fix broken overflow, add regression test --- src/textual/css/_styles_builder.py | 2 +- tests/css/test_parse.py | 12 ++++++++++++ 2 files changed, 13 insertions(+), 1 deletion(-) diff --git a/src/textual/css/_styles_builder.py b/src/textual/css/_styles_builder.py index 2c72e13d3..b3c2f243d 100644 --- a/src/textual/css/_styles_builder.py +++ b/src/textual/css/_styles_builder.py @@ -127,7 +127,7 @@ class StylesBuilder: """ return [attr[8:] for attr in dir(self) if attr.startswith("process_")] - def _get_process_enum_multiple( + def _process_enum_multiple( self, name: str, tokens: list[Token], valid_values: set[str], count: int ) -> tuple[str, ...]: """Generic code to process a declaration with two enumerations, like overflow: auto auto""" diff --git a/tests/css/test_parse.py b/tests/css/test_parse.py index ccf7a75f4..dd1e73809 100644 --- a/tests/css/test_parse.py +++ b/tests/css/test_parse.py @@ -983,6 +983,18 @@ class TestParseOffset: assert styles.offset.y == parsed_y +class TestParseOverflow: + def test_multiple_enum(self): + css = "#some-widget { overflow: hidden auto; }" + stylesheet = Stylesheet() + stylesheet.add_source(css) + + styles = stylesheet.rules[0].styles + + assert len(stylesheet.rules) == 1 + assert styles.overflow_x == "hidden" + assert styles.overflow_y == "auto" + class TestParseTransition: @pytest.mark.parametrize( "duration, parsed_duration",