objutils 0.10.20__tar.gz → 0.10.21__tar.gz

{objutils-0.10.20 → objutils-0.10.21}/.github/workflows/pythonapp.yml

@@ -22,12 +22,12 @@ jobs:
           - os: windows-latest
             cibw_archs_windows: AMD64
     steps:
-      - uses: actions/checkout@v6
-      - uses: pypa/cibuildwheel@v3.3
+      - uses: actions/checkout@v7
+      - uses: pypa/cibuildwheel@v4.1.0
         env:
           CIBW_ARCHS_WINDOWS: ${{ matrix.cibw_archs_windows || '' }}
           CIBW_ARCHS_MACOS: arm64
-      - uses: actions/upload-artifact@v6
+      - uses: actions/upload-artifact@v7
         with:
           name: cibw-wheels-${{ matrix.os }}-${{ strategy.job-index }}
           path: ./wheelhouse/*.whl
@@ -36,12 +36,12 @@ jobs:
     name: Build source distribution
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v6
+      - uses: actions/checkout@v7
       - name: Build sdist
         run: |
           pip install -U build
           python -m build --sdist
-      - uses: actions/upload-artifact@v6
+      - uses: actions/upload-artifact@v7
         with:
           path: dist/*.tar.gz
           include-hidden-files: true
@@ -53,11 +53,11 @@ jobs:
       id-token: write
       contents: read
     steps:
-      - uses: actions/download-artifact@v7
+      - uses: actions/download-artifact@v8
         with:
           path: dist
           merge-multiple: true
       - name: Publish package to PyPI
-        uses: pypa/gh-action-pypi-publish@v1.13.0
+        uses: pypa/gh-action-pypi-publish@v1.14.0
         with:
           attestations: true

{objutils-0.10.20 → objutils-0.10.21}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.2
 Name: objutils
-Version: 0.10.20
+Version: 0.10.21
 Summary: Objectfile library for Python
 Keywords: hex files,intel hex,s19,srec,srecords,object files,map files,embedded,microcontroller,ECU,shf,rfc4194
 Author-Email: Christoph Schueler <cpu12.gems@googlemail.com>

{objutils-0.10.20 → objutils-0.10.21}/objutils/__init__.py

@@ -13,7 +13,7 @@ Registers CODECS and implements an interface to them.
 The first parameter is always the codec name.
 """
 
-__version__ = "0.10.20"
+__version__ = "0.10.21"
 
 __all__ = [
     "Image",

{objutils-0.10.20 → objutils-0.10.21}/objutils/tests/test_elf.py

@@ -10,12 +10,14 @@ Based on the actual ElfParser API which uses:
 """
 
 import collections
+import gc
 import tempfile
+import weakref
 from pathlib import Path
 
 import pytest
 
-from objutils.elf import ElfParser
+from objutils.elf import ElfParser, import_dwarf_to_db, model
 
 # Test data directory
 EXAMPLES_DIR = Path(__file__).parent.parent.parent / "examples"
@@ -341,6 +343,57 @@ class TestElfInMemoryDatabase:
             parser_disk.close()
             parser_mem.close()
 
+    def test_in_memory_context_manager_closes_parser(self, sample_elf_path):
+        """Context-manager usage must release in-memory parser resources."""
+        with ElfParser(str(sample_elf_path), in_memory=True) as parser:
+            assert parser.session is not None
+            assert parser.e_machine == 83
+
+        assert parser._closed is True
+        assert parser.db is None
+        assert parser.session is None
+        assert parser.fp is None
+
+    def test_in_memory_close_is_idempotent(self, sample_elf_path):
+        """Calling close() multiple times should be safe."""
+        parser = ElfParser(str(sample_elf_path), in_memory=True)
+        parser.close()
+        parser.close()
+        assert parser._closed is True
+
+    def test_in_memory_multiple_parsers_sequential_cleanup(self, sample_elf_path):
+        """Opening multiple in-memory parsers sequentially must not raise cleanup errors."""
+        for _ in range(25):
+            with ElfParser(str(sample_elf_path), in_memory=True) as parser:
+                assert isinstance(parser.e_entry, int)
+
+    def test_in_memory_stress_multiple_elf_files_cleanup(self):
+        """Stress-test in-memory cleanup across multiple ELF files and many open/close cycles."""
+        available_elfs = [EXAMPLES_DIR / name for name in SAMPLE_ELF_FILES if (EXAMPLES_DIR / name).exists()]
+        if not available_elfs:
+            pytest.skip("No sample ELF files found for stress test")
+
+        for cycle in range(15):
+            for elf_path in available_elfs:
+                with ElfParser(str(elf_path), in_memory=True) as parser:
+                    # Touch common ELF and optional debug/DWARF query paths.
+                    assert isinstance(parser.e_machine, int)
+                    _ = parser.debug_sections()
+                    ref = weakref.ref(parser)
+
+                assert parser._closed is True
+                assert parser.db is None
+                assert parser.session is None
+                assert parser.fp is None
+
+                del parser
+                gc.collect()
+                assert ref() is None
+
+            # Force periodic collection between larger batches.
+            if cycle % 5 == 4:
+                gc.collect()
+
 
 class TestElfImageCreation:
     """Test image creation from ELF."""
@@ -378,5 +431,34 @@ class TestElfDebugSections:
         assert isinstance(debug_secs, (collections.OrderedDict, type(None)))
 
 
+class TestDwarfImportRegression:
+    """Regression tests for DWARF importer into .prgdb databases."""
+
+    def test_import_dwarf_populates_database(self):
+        """Importing DWARF should succeed and persist DIE/attribute rows."""
+        candidates = [
+            EXAMPLES_DIR / "lang-generics.elf",
+            EXAMPLES_DIR / "hello_xcp.ino.elf",
+            EXAMPLES_DIR / "avr.elf",
+        ]
+        elf_path = next((p for p in candidates if p.exists()), None)
+        if elf_path is None:
+            pytest.skip("No suitable ELF sample found for DWARF import regression test")
+
+        with tempfile.TemporaryDirectory(ignore_cleanup_errors=True) as tmpdir:
+            out_db = Path(tmpdir) / "imported.prgdb"
+            rc = import_dwarf_to_db(str(elf_path), str(out_db), quiet=True, verbose=False, force=True)
+            assert rc == 0
+            assert out_db.exists()
+
+            db = model.Model(str(out_db))
+            try:
+                session = db.session
+                assert session.query(model.DebugInformationEntry).count() > 0
+                assert session.query(model.DIEAttribute).count() > 0
+            finally:
+                db.close()
+
+
 if __name__ == "__main__":
     pytest.main([__file__, "-v"])

{objutils-0.10.20 → objutils-0.10.21}/objutils/version.py

@@ -1,3 +1,3 @@
 """objutils version module"""
 
-__version__ = "0.10.20"
+__version__ = "0.10.21"

{objutils-0.10.20 → objutils-0.10.21}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "scikit_build_core.build"
 
 [project]
 name = "objutils"
-version = "0.10.20"
+version = "0.10.21"
 description = "Objectfile library for Python"
 authors = [
     {name = "Christoph Schueler", email = "cpu12.gems@googlemail.com"}
@@ -127,7 +127,7 @@ skip = ["*_i686", "*-musllinux*"]
 build-frontend = "build"
 
 [tool.bumpver]
-current_version = "0.10.20"
+current_version = "0.10.21"
 version_pattern = "MAJOR.MINOR.PATCH"
 commit_message = "bump version {old_version} -> {new_version}"
 commit = true