From ce0b5564b16369e7fc057db07ddaf46cff019702 Mon Sep 17 00:00:00 2001
From: Josef Friedrich <josef@friedrich.rocks>
Date: Fri, 19 Jan 2024 15:52:58 +0100
Subject: [PATCH] Test --clean option

---
 mscxyz/cli.py       |  8 +++-----
 tests/test_style.py | 36 ++++++++++++++++++++++++++++++++----
 2 files changed, 35 insertions(+), 9 deletions(-)

diff --git a/mscxyz/cli.py b/mscxyz/cli.py
index 200c08a..0659f97 100644
--- a/mscxyz/cli.py
+++ b/mscxyz/cli.py
@@ -714,6 +714,9 @@ def list_styles(version: int) -> None:
 
         # style
 
+        if args.style_clean:
+            score.style.clean()
+
         for style_name, value in args.style_value:
             score.style.set(style_name, value)
 
@@ -750,11 +753,6 @@ def list_styles(version: int) -> None:
 
         #     print("\n" + utils.color(file, "red"))
 
-        #     if args.subcommand == "clean":
-        #         score = Score(file)
-        #         print(score.filename)
-        #         score.clean()
-
         #     elif args.subcommand == "lyrics":
         #         score = Score(file)
         if args.lyrics_remap:
diff --git a/tests/test_style.py b/tests/test_style.py
index 646e9ab..870e70b 100644
--- a/tests/test_style.py
+++ b/tests/test_style.py
@@ -317,7 +317,8 @@ def test_method_set_all(styles: str) -> None:
 
 
 class TestClean:
-    def test_clean(self) -> None:
+    @pytest.mark.legacy
+    def test_clean_legacy(self) -> None:
         c = Cli("clean", legacy=True).append_score("formats.mscz").execute()
 
         uncleaned: str = c.pre.read_as_text()
@@ -336,14 +337,41 @@ def test_clean(self) -> None:
         assert "<LayoutBreak>" not in cleaned
         assert "<StemDirection>" not in cleaned
 
-    def test_clean_add_style(self) -> None:
+    def test_clean(self) -> None:
+        c = Cli("--clean").append_score("formats.mscz").execute()
+
+        uncleaned: str = c.pre.read_as_text()
+        assert "<font" in uncleaned
+        assert "<b>" in uncleaned
+        assert "<i>" in uncleaned
+        # assert "<pos" in uncleaned
+        assert "<LayoutBreak>" in uncleaned
+        assert "<StemDirection>" in uncleaned
+
+        cleaned: str = c.post.read_as_text()
+        assert "<font" not in cleaned
+        assert "<b>" not in cleaned
+        assert "<i>" not in cleaned
+        assert "<pos" not in cleaned
+        assert "<LayoutBreak>" not in cleaned
+        assert "<StemDirection>" not in cleaned
+
+    @pytest.mark.legacy
+    def test_clean_add_style_legacy(self) -> None:
         score = (
             Cli("clean", "--style", helper.get_score("style.mss", 2), legacy=True)
             .append_score("simple.mscx", 2)
             .score()
         )
-        style = score.read_as_text()
-        assert "<staffUpperBorder>77</staffUpperBorder>" in style
+        assert "<staffUpperBorder>77</staffUpperBorder>" in score.read_as_text()
+
+    def test_clean_add_style(self) -> None:
+        score = (
+            Cli("--clean", "--style-file", helper.get_file("Jazz.mss", 4))
+            .append_score("score.mscz")
+            .score()
+        )
+        assert "<musicalSymbolFont>MuseJazz</musicalSymbolFont>" in score.read_as_text()
 
 
 def test_load_style_file() -> None: