statable 3.2.0__tar.gz → 3.2.2__tar.gz

{statable-3.2.0 → statable-3.2.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: statable
-Version: 3.2.0
+Version: 3.2.2
 Summary: MISRA C:2012-aware state machine design and C code generation for embedded systems
 Author: Hideki Akashi
 License-Expression: Apache-2.0

{statable-3.2.0 → statable-3.2.2}/codegen/__init__.py

@@ -8,7 +8,7 @@ StaTable code generation layer
     so it is not re-exported at top level (to reduce startup time)
 """
 
-__version__ = "3.2.0"
+__version__ = "3.2.2"
 
 __all__ = [
     'CCodeGenerator',

{statable-3.2.0 → statable-3.2.2}/pyproject.toml

@@ -6,7 +6,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "statable"
-version = "3.2.0"
+version = "3.2.2"
 description = "MISRA C:2012-aware state machine design and C code generation for embedded systems"
 readme = "README.md"
 requires-python = ">=3.10"

{statable-3.2.0 → statable-3.2.2}/statable/__init__.py

@@ -40,7 +40,7 @@ from .global_defs import (
     StructMemberDef,
 )
 
-__version__ = "3.2.0"
+__version__ = "3.2.2"
 
 __all__ = [
     # model

{statable-3.2.0 → statable-3.2.2}/statable/cli.py

@@ -27,7 +27,7 @@ import json
 import sys
 from pathlib import Path
 
-CLI_VERSION = "3.2.0"
+CLI_VERSION = "3.2.2"
 
 
 def _err(msg: str) -> None:

{statable-3.2.0 → statable-3.2.2}/statable.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: statable
-Version: 3.2.0
+Version: 3.2.2
 Summary: MISRA C:2012-aware state machine design and C code generation for embedded systems
 Author: Hideki Akashi
 License-Expression: Apache-2.0

{statable-3.2.0 → statable-3.2.2}/statable.egg-info/SOURCES.txt

@@ -160,4 +160,5 @@ tests/test_v3_0_s1_packaging.py
 tests/test_v3_0_s2_public_api.py
 tests/test_v3_0_s3_cli.py
 tests/test_v3_0_s4_docs.py
-tests/test_v3_2_s1_wizard.py
+tests/test_v3_2_s1_wizard.py
+tests/test_v3_2_s2_signals.py

{statable-3.2.0 → statable-3.2.2}/statable_gui/main_window.py

@@ -211,6 +211,8 @@ class MainWindow(QMainWindow):
 
         self.tab_widget = QTabWidget()
         self.tab_widget.setTabsClosable(True)
+        # [v3.2.1] Enable drag-and-drop tab reordering
+        self.tab_widget.setMovable(True)
         self.tab_widget.tabCloseRequested.connect(
             self.close_tab)
         self.setCentralWidget(self.tab_widget)
@@ -332,7 +334,8 @@ class MainWindow(QMainWindow):
 
         open_btn = QAction(self.tr("Open"), self)
         open_btn.setToolTip(self.tr("Open project"))
-        open_btn.triggered.connect(self.open_project)
+        # [v3.2.2 fix] lambda: avoid passing triggered(bool) as filepath
+        open_btn.triggered.connect(lambda: self.open_project())
         toolbar.addAction(open_btn)
 
         save_btn = QAction(self.tr("Save"), self)
@@ -384,7 +387,8 @@ class MainWindow(QMainWindow):
         file_menu.addSeparator()
 
         open_action = QAction(self.tr("Open Project..."), self)
-        open_action.triggered.connect(self.open_project)
+        # [v3.2.1 fix] lambda: avoid passing triggered(bool) as filepath
+        open_action.triggered.connect(lambda: self.open_project())
         file_menu.addAction(open_action)
 
         save_action = QAction(self.tr("Save Project..."), self)
@@ -812,6 +816,10 @@ class MainWindow(QMainWindow):
 
     def open_project(self, filepath: str = None):
         """Load the whole project"""
+        # [v3.2.1 fix] Qt triggered signal passes bool; normalize to None
+        if isinstance(filepath, bool):
+            filepath = None
+
         # [v2.3] Prompt to save before loading
         if not self._maybe_save():
             return

{statable-3.2.0 → statable-3.2.2}/tests/test_v3_0_s1_packaging.py

@@ -3,11 +3,11 @@
 Phase S-1 (v3.0) tests: packaging foundation.
 
 Verifies:
-  - code/pyproject.toml exists and declares statable/3.2.0
+  - code/pyproject.toml exists and declares statable/3.2.2
   - statable-cli entry point is defined
   - extras: gui / dev
-  - statable.__version__ == "3.2.0"
-  - codegen.__version__ == "3.2.0"
+  - statable.__version__ == "3.2.2"
+  - codegen.__version__ == "3.2.2"
 """
 from __future__ import annotations
 
@@ -64,8 +64,8 @@ def main() -> int:
     check("project.name == 'statable'",
           project.get("name") == "statable",
           f"got: {project.get('name')!r}")
-    check("project.version == '3.2.0'",
-          project.get("version") == "3.2.0",
+    check("project.version == '3.2.2'",
+          project.get("version") == "3.2.2",
           f"got: {project.get('version')!r}")
 
     # --- T4: requires-python ---------------------------------------
@@ -101,16 +101,16 @@ def main() -> int:
     # --- T8: runtime __version__ -----------------------------------
     try:
         import statable
-        check("statable.__version__ == '3.2.0'",
-              getattr(statable, "__version__", None) == "3.2.0",
+        check("statable.__version__ == '3.2.2'",
+              getattr(statable, "__version__", None) == "3.2.2",
               f"got: {getattr(statable, '__version__', None)!r}")
     except Exception as e:
         check("statable importable", False, repr(e))
 
     try:
         import codegen
-        check("codegen.__version__ == '3.2.0'",
-              getattr(codegen, "__version__", None) == "3.2.0",
+        check("codegen.__version__ == '3.2.2'",
+              getattr(codegen, "__version__", None) == "3.2.2",
               f"got: {getattr(codegen, '__version__', None)!r}")
     except Exception as e:
         check("codegen importable", False, repr(e))

{statable-3.2.0 → statable-3.2.2}/tests/test_v3_0_s2_public_api.py

@@ -47,8 +47,8 @@ def main() -> int:
         _summary()
         return 1
 
-    check("statable has __version__ == '3.2.0'",
-          getattr(statable, "__version__", None) == "3.2.0",
+    check("statable has __version__ == '3.2.2'",
+          getattr(statable, "__version__", None) == "3.2.2",
           f"got: {getattr(statable, '__version__', None)!r}")
     check("'__version__' in statable.__all__",
           "__version__" in statable.__all__,
@@ -69,8 +69,8 @@ def main() -> int:
         _summary()
         return 1
 
-    check("codegen has __version__ == '3.2.0'",
-          getattr(codegen, "__version__", None) == "3.2.0",
+    check("codegen has __version__ == '3.2.2'",
+          getattr(codegen, "__version__", None) == "3.2.2",
           f"got: {getattr(codegen, '__version__', None)!r}")
     check("'validate' in codegen.__all__",
           "validate" in codegen.__all__,

{statable-3.2.0 → statable-3.2.2}/tests/test_v3_0_s3_cli.py

@@ -60,8 +60,8 @@ def main() -> int:
     try:
         from statable import cli
         check("statable.cli importable", True)
-        check("CLI_VERSION == '3.2.0'",
-              cli.CLI_VERSION == "3.2.0",
+        check("CLI_VERSION == '3.2.2'",
+              cli.CLI_VERSION == "3.2.2",
               f"got: {cli.CLI_VERSION!r}")
     except Exception as e:
         check("statable.cli importable", False, repr(e))
@@ -91,8 +91,8 @@ def main() -> int:
     # --- T3: version --------------------------------------------------
     rc, out, _ = run_cli(["version"])
     check("version: exit 0", rc == 0, f"rc={rc}")
-    check("version: prints 'statable-cli 3.2.0'",
-          "statable-cli 3.2.0" in out,
+    check("version: prints 'statable-cli 3.2.2'",
+          "statable-cli 3.2.2" in out,
           f"out={out!r}")
 
     # --- T4: validate on real XML (json) ------------------------------

statable-3.2.2/tests/test_v3_2_s2_signals.py

@@ -0,0 +1,144 @@
+# tests/test_v3_2_s2_signals.py
+"""Test Qt signal-slot safety (v3.2.2).
+
+Regression test for the v3.2.0 bug where `open_project(filepath=None)`
+received Qt's `triggered(bool)` argument, resulting in
+`project_from_xml(False)` and a hang on `open(False, "rb")`.
+
+Key insight: PySide6 auto-truncates args for slots with fixed signatures,
+so most `triggered.connect(self.slot)` are safe. The bug only occurs
+when the slot has an OPTIONAL parameter that receives the bool.
+"""
+from __future__ import annotations
+
+import os
+import re
+import sys
+from pathlib import Path
+
+os.environ.setdefault("QT_QPA_PLATFORM", "offscreen")
+sys.path.insert(0, str(Path(__file__).parent.parent))
+
+
+def main():
+    passed = 0
+    failed = 0
+
+    def check(name, cond):
+        nonlocal passed, failed
+        if cond:
+            passed += 1
+            print(f"[PASS] {name}")
+        else:
+            failed += 1
+            print(f"[FAIL] {name}")
+
+    # ================================================================
+    # T1: open_project normalizes bool arguments
+    # ================================================================
+    print("=" * 70)
+    print("  T1: open_project(bool) normalization")
+    print("=" * 70)
+
+    try:
+        from PySide6.QtWidgets import QApplication
+        app = QApplication.instance() or QApplication(sys.argv)
+
+        from statable_gui.main_window import MainWindow
+
+        win = MainWindow()
+
+        import statable_gui.main_window as mw_mod
+        original_pfx = mw_mod.project_from_xml
+        called_with = []
+
+        def fake_pfx(path):
+            called_with.append(path)
+            raise RuntimeError("TEST_STOP")
+
+        mw_mod.project_from_xml = fake_pfx
+
+        from PySide6.QtWidgets import QFileDialog
+        original_dialog = QFileDialog.getOpenFileName
+        QFileDialog.getOpenFileName = staticmethod(lambda *a, **k: ("", ""))
+
+        try:
+            win.open_project(False)
+            check("open_project(False) does not hang", True)
+        except Exception as e:
+            check(f"open_project(False) raised: {e}", False)
+
+        check("project_from_xml not called with False",
+              False not in called_with)
+
+        called_with.clear()
+        try:
+            win.open_project(True)
+            check("open_project(True) does not hang", True)
+        except Exception as e:
+            check(f"open_project(True) raised: {e}", False)
+        check("project_from_xml not called with True",
+              True not in called_with)
+
+        mw_mod.project_from_xml = original_pfx
+        QFileDialog.getOpenFileName = original_dialog
+
+    except Exception as e:
+        import traceback
+        print(traceback.format_exc())
+        check(f"T1 setup failed: {e}", False)
+
+    # ================================================================
+    # T2: open_project is connected via lambda (the actual bug)
+    # ================================================================
+    print()
+    print("=" * 70)
+    print("  T2: open_project is lambda-wrapped")
+    print("=" * 70)
+
+    mw_path = Path(__file__).parent.parent / "statable_gui" / "main_window.py"
+    src = mw_path.read_text(encoding="utf-8")
+
+    # The bug: `triggered.connect(self.open_project)` without lambda
+    # would pass triggered(bool) as filepath.
+    # FIX: must use `triggered.connect(lambda: self.open_project())`
+    bug_pattern = re.compile(
+        r'triggered\.connect\(\s*self\.open_project\s*\)'
+    )
+    check("open_project not connected without lambda",
+          not bug_pattern.search(src))
+
+    fix_pattern = re.compile(
+        r'triggered\.connect\(\s*lambda\s*:\s*self\.open_project\(\)\s*\)'
+    )
+    check("open_project connected via lambda",
+          bool(fix_pattern.search(src)))
+
+    # ================================================================
+    # T3: open_project has bool guard in body
+    # ================================================================
+    print()
+    print("=" * 70)
+    print("  T3: open_project has bool normalization guard")
+    print("=" * 70)
+
+    check("open_project has isinstance(filepath, bool) guard",
+          "isinstance(filepath, bool)" in src)
+
+    # Verify that the signature accepts Optional[str]
+    check("open_project signature has filepath parameter",
+          re.search(r'def open_project\(self,\s*filepath', src) is not None)
+
+    # ================================================================
+    # Summary
+    # ================================================================
+    print()
+    print("=" * 70)
+    print(f"  TOTAL: {passed + failed}   PASSED: {passed}   FAILED: {failed}")
+    print("=" * 70)
+
+    return 0 if failed == 0 else 1
+
+
+if __name__ == "__main__":
+    sys.exit(main())