From 0c2f9bb09cf625cfa5eaf1d784511d3defd8f5a0 Mon Sep 17 00:00:00 2001
From: Milan Rother <milan.rother@gmx.de>
Date: Thu, 5 Nov 2026 13:37:25 +0100
Subject: [PATCH 1/2] Add block icons

---
 src/pathsim_flight/icons/CAStoEAS.svg      |  1 +
 src/pathsim_flight/icons/CAStoMach.svg     |  1 +
 src/pathsim_flight/icons/CAStoTAS.svg      |  1 +
 src/pathsim_flight/icons/EAStoTAS.svg      |  1 +
 src/pathsim_flight/icons/ISAtmosphere.svg  |  1 +
 src/pathsim_flight/icons/JSBSimWrapper.svg |  1 +
 src/pathsim_flight/icons/MachtoCAS.svg     |  1 +
 src/pathsim_flight/icons/TAStoCAS.svg      |  1 +
 tests/test_icons.py                        | 29 ++++++++++++++++++++++
 9 files changed, 37 insertions(+)
 create mode 100644 src/pathsim_flight/icons/CAStoEAS.svg
 create mode 100644 src/pathsim_flight/icons/CAStoMach.svg
 create mode 100644 src/pathsim_flight/icons/CAStoTAS.svg
 create mode 100644 src/pathsim_flight/icons/EAStoTAS.svg
 create mode 100644 src/pathsim_flight/icons/ISAtmosphere.svg
 create mode 100644 src/pathsim_flight/icons/JSBSimWrapper.svg
 create mode 100644 src/pathsim_flight/icons/MachtoCAS.svg
 create mode 100644 src/pathsim_flight/icons/TAStoCAS.svg
 create mode 100644 tests/test_icons.py

diff --git a/src/pathsim_flight/icons/CAStoEAS.svg b/src/pathsim_flight/icons/CAStoEAS.svg
new file mode 100644
index 0000000..e6469ed
--- /dev/null
+++ b/src/pathsim_flight/icons/CAStoEAS.svg
@@ -0,0 +1 @@
+<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 96 64" fill="none" stroke="currentColor" stroke-width="1.6" stroke-linecap="round" stroke-linejoin="round"><path d="M 29.0 42.2 A 20 20 0 1 1 67.0 42.2"/><line x1="32.8" y1="40.9" x2="29.0" y2="42.2"/><line x1="32.8" y1="31.1" x2="29.0" y2="29.8"/><line x1="38.6" y1="23.1" x2="36.2" y2="19.8"/><line x1="48.0" y1="20.0" x2="48.0" y2="16.0"/><line x1="57.4" y1="23.1" x2="59.8" y2="19.8"/><line x1="63.2" y1="31.1" x2="67.0" y2="29.8"/><line x1="63.2" y1="40.9" x2="67.0" y2="42.2"/><line x1="48" y1="36" x2="54.4" y2="23.5"/><circle cx="48" cy="36" r="2.5" fill="currentColor" stroke="none"/></svg>
diff --git a/src/pathsim_flight/icons/CAStoMach.svg b/src/pathsim_flight/icons/CAStoMach.svg
new file mode 100644
index 0000000..e6469ed
--- /dev/null
+++ b/src/pathsim_flight/icons/CAStoMach.svg
@@ -0,0 +1 @@
+<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 96 64" fill="none" stroke="currentColor" stroke-width="1.6" stroke-linecap="round" stroke-linejoin="round"><path d="M 29.0 42.2 A 20 20 0 1 1 67.0 42.2"/><line x1="32.8" y1="40.9" x2="29.0" y2="42.2"/><line x1="32.8" y1="31.1" x2="29.0" y2="29.8"/><line x1="38.6" y1="23.1" x2="36.2" y2="19.8"/><line x1="48.0" y1="20.0" x2="48.0" y2="16.0"/><line x1="57.4" y1="23.1" x2="59.8" y2="19.8"/><line x1="63.2" y1="31.1" x2="67.0" y2="29.8"/><line x1="63.2" y1="40.9" x2="67.0" y2="42.2"/><line x1="48" y1="36" x2="54.4" y2="23.5"/><circle cx="48" cy="36" r="2.5" fill="currentColor" stroke="none"/></svg>
diff --git a/src/pathsim_flight/icons/CAStoTAS.svg b/src/pathsim_flight/icons/CAStoTAS.svg
new file mode 100644
index 0000000..e6469ed
--- /dev/null
+++ b/src/pathsim_flight/icons/CAStoTAS.svg
@@ -0,0 +1 @@
+<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 96 64" fill="none" stroke="currentColor" stroke-width="1.6" stroke-linecap="round" stroke-linejoin="round"><path d="M 29.0 42.2 A 20 20 0 1 1 67.0 42.2"/><line x1="32.8" y1="40.9" x2="29.0" y2="42.2"/><line x1="32.8" y1="31.1" x2="29.0" y2="29.8"/><line x1="38.6" y1="23.1" x2="36.2" y2="19.8"/><line x1="48.0" y1="20.0" x2="48.0" y2="16.0"/><line x1="57.4" y1="23.1" x2="59.8" y2="19.8"/><line x1="63.2" y1="31.1" x2="67.0" y2="29.8"/><line x1="63.2" y1="40.9" x2="67.0" y2="42.2"/><line x1="48" y1="36" x2="54.4" y2="23.5"/><circle cx="48" cy="36" r="2.5" fill="currentColor" stroke="none"/></svg>
diff --git a/src/pathsim_flight/icons/EAStoTAS.svg b/src/pathsim_flight/icons/EAStoTAS.svg
new file mode 100644
index 0000000..e6469ed
--- /dev/null
+++ b/src/pathsim_flight/icons/EAStoTAS.svg
@@ -0,0 +1 @@
+<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 96 64" fill="none" stroke="currentColor" stroke-width="1.6" stroke-linecap="round" stroke-linejoin="round"><path d="M 29.0 42.2 A 20 20 0 1 1 67.0 42.2"/><line x1="32.8" y1="40.9" x2="29.0" y2="42.2"/><line x1="32.8" y1="31.1" x2="29.0" y2="29.8"/><line x1="38.6" y1="23.1" x2="36.2" y2="19.8"/><line x1="48.0" y1="20.0" x2="48.0" y2="16.0"/><line x1="57.4" y1="23.1" x2="59.8" y2="19.8"/><line x1="63.2" y1="31.1" x2="67.0" y2="29.8"/><line x1="63.2" y1="40.9" x2="67.0" y2="42.2"/><line x1="48" y1="36" x2="54.4" y2="23.5"/><circle cx="48" cy="36" r="2.5" fill="currentColor" stroke="none"/></svg>
diff --git a/src/pathsim_flight/icons/ISAtmosphere.svg b/src/pathsim_flight/icons/ISAtmosphere.svg
new file mode 100644
index 0000000..bdac52a
--- /dev/null
+++ b/src/pathsim_flight/icons/ISAtmosphere.svg
@@ -0,0 +1 @@
+<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 96 64" fill="none" stroke="currentColor" stroke-width="1.6" stroke-linecap="round" stroke-linejoin="round"><line x1="10" y1="50" x2="86" y2="50"/><line x1="10" y1="10" x2="10" y2="54"/><path d="M 62 50 L 34 30 V 20 L 46 10"/></svg>
diff --git a/src/pathsim_flight/icons/JSBSimWrapper.svg b/src/pathsim_flight/icons/JSBSimWrapper.svg
new file mode 100644
index 0000000..4c56caf
--- /dev/null
+++ b/src/pathsim_flight/icons/JSBSimWrapper.svg
@@ -0,0 +1 @@
+<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 96 64" fill="none" stroke="currentColor" stroke-width="1.6" stroke-linecap="round" stroke-linejoin="round"><path d="M 48 10 C 51 10 51 16 51 20 L 51 27 L 76 36 L 76 40 L 51 35 L 51 46 L 58 51 L 58 54 L 48 51 L 38 54 L 38 51 L 45 46 L 45 35 L 20 40 L 20 36 L 45 27 L 45 20 C 45 16 45 10 48 10 Z"/></svg>
diff --git a/src/pathsim_flight/icons/MachtoCAS.svg b/src/pathsim_flight/icons/MachtoCAS.svg
new file mode 100644
index 0000000..e6469ed
--- /dev/null
+++ b/src/pathsim_flight/icons/MachtoCAS.svg
@@ -0,0 +1 @@
+<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 96 64" fill="none" stroke="currentColor" stroke-width="1.6" stroke-linecap="round" stroke-linejoin="round"><path d="M 29.0 42.2 A 20 20 0 1 1 67.0 42.2"/><line x1="32.8" y1="40.9" x2="29.0" y2="42.2"/><line x1="32.8" y1="31.1" x2="29.0" y2="29.8"/><line x1="38.6" y1="23.1" x2="36.2" y2="19.8"/><line x1="48.0" y1="20.0" x2="48.0" y2="16.0"/><line x1="57.4" y1="23.1" x2="59.8" y2="19.8"/><line x1="63.2" y1="31.1" x2="67.0" y2="29.8"/><line x1="63.2" y1="40.9" x2="67.0" y2="42.2"/><line x1="48" y1="36" x2="54.4" y2="23.5"/><circle cx="48" cy="36" r="2.5" fill="currentColor" stroke="none"/></svg>
diff --git a/src/pathsim_flight/icons/TAStoCAS.svg b/src/pathsim_flight/icons/TAStoCAS.svg
new file mode 100644
index 0000000..e6469ed
--- /dev/null
+++ b/src/pathsim_flight/icons/TAStoCAS.svg
@@ -0,0 +1 @@
+<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 96 64" fill="none" stroke="currentColor" stroke-width="1.6" stroke-linecap="round" stroke-linejoin="round"><path d="M 29.0 42.2 A 20 20 0 1 1 67.0 42.2"/><line x1="32.8" y1="40.9" x2="29.0" y2="42.2"/><line x1="32.8" y1="31.1" x2="29.0" y2="29.8"/><line x1="38.6" y1="23.1" x2="36.2" y2="19.8"/><line x1="48.0" y1="20.0" x2="48.0" y2="16.0"/><line x1="57.4" y1="23.1" x2="59.8" y2="19.8"/><line x1="63.2" y1="31.1" x2="67.0" y2="29.8"/><line x1="63.2" y1="40.9" x2="67.0" y2="42.2"/><line x1="48" y1="36" x2="54.4" y2="23.5"/><circle cx="48" cy="36" r="2.5" fill="currentColor" stroke="none"/></svg>
diff --git a/tests/test_icons.py b/tests/test_icons.py
new file mode 100644
index 0000000..dcaf3cc
--- /dev/null
+++ b/tests/test_icons.py
@@ -0,0 +1,29 @@
+import unittest
+import xml.etree.ElementTree as ET
+
+from pathsim.blocks import Block
+
+import pathsim_flight
+
+
+class TestIcons(unittest.TestCase):
+    """Every block of the toolbox has an icon, read by Block.icon()."""
+
+    def test_blocks_have_icons(self):
+        blocks = [
+            obj for obj in vars(pathsim_flight).values()
+            if isinstance(obj, type) and issubclass(obj, Block)
+            and obj.__module__.startswith("pathsim_flight")
+        ]
+        self.assertGreater(len(blocks), 0)
+
+        for cls in blocks:
+            with self.subTest(block=cls.__name__):
+                icon = cls.icon()
+                self.assertIsNotNone(icon)
+                root = ET.fromstring(icon)
+                self.assertEqual(root.get("viewBox"), "0 0 96 64")
+
+
+if __name__ == "__main__":
+    unittest.main()

From 4c53ce6d983de7d3ac5e19f8a7b2bfc623b74e98 Mon Sep 17 00:00:00 2001
From: Milan Rother <milan.rother@gmx.de>
Date: Thu, 5 Nov 2026 13:52:14 +0100
Subject: [PATCH 2/2] Format the icon test

---
 tests/test_icons.py | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/tests/test_icons.py b/tests/test_icons.py
index dcaf3cc..6fab108 100644
--- a/tests/test_icons.py
+++ b/tests/test_icons.py
@@ -11,8 +11,10 @@ class TestIcons(unittest.TestCase):
 
     def test_blocks_have_icons(self):
         blocks = [
-            obj for obj in vars(pathsim_flight).values()
-            if isinstance(obj, type) and issubclass(obj, Block)
+            obj
+            for obj in vars(pathsim_flight).values()
+            if isinstance(obj, type)
+            and issubclass(obj, Block)
             and obj.__module__.startswith("pathsim_flight")
         ]
         self.assertGreater(len(blocks), 0)