ledgered 0.7.2.dev6__tar.gz → 0.7.2.dev8__tar.gz

ledgered-0.7.2.dev8/.github/workflows/fast-checks.yml

@@ -0,0 +1,41 @@
+name: Fast checks
+
+on:
+  workflow_dispatch:
+  push:
+    branches:
+      - develop
+      - master
+  pull_request:
+
+jobs:
+  ruff:
+    name: Checks & format
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v4
+    - name: Ruff - lint check
+      uses: astral-sh/ruff-action@v3
+    - name: Ruff - format check
+      run: ruff format --check
+
+  mypy:
+    name: Type checking
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v4
+    - run: pip install mypy types-toml
+    - name: Mypy type checking
+      run: mypy src
+
+  misspell:
+    name: Check misspellings
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v4
+    - name: Check misspellings
+      uses: codespell-project/actions-codespell@v1
+      with:
+        builtin: clear,rare
+        check_filenames: true
+        ignore_words_list: assertIn,crate,

{ledgered-0.7.2.dev6 → ledgered-0.7.2.dev8}/.gitignore

@@ -6,4 +6,3 @@ coverage.xml
 build/
 dist/
 __version__.py
-

ledgered-0.7.2.dev8/.pre-commit-config.yaml

@@ -0,0 +1,28 @@
+# To install hooks, run:
+# pre-commit install --hook-type pre-commit
+# pre-commit install --hook-type commit-msg
+
+repos:
+  - repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v5.0.0
+    hooks:
+      - id: trailing-whitespace
+      - id: end-of-file-fixer
+      - id: check-yaml
+      - id: check-toml
+      - id: check-added-large-files
+
+  - repo: https://github.com/astral-sh/ruff-pre-commit
+    rev: v0.9.3
+    hooks:
+      - id: ruff-format
+        args: ["--check"]
+      - id: ruff
+
+  - repo: https://github.com/pre-commit/mirrors-mypy
+    rev: v1.14.1
+    hooks:
+      - id: mypy
+        additional_dependencies:
+          - types-toml
+          - types-requests

{ledgered-0.7.2.dev6/src/ledgered.egg-info → ledgered-0.7.2.dev8}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.2
 Name: ledgered
-Version: 0.7.2.dev6
+Version: 0.7.2.dev8
 Summary: Python tools, utils, libraries, to be used with Ledger cryptodevices
 Author-email: Ledger <hello@ledger.fr>
 License: MIT License

{ledgered-0.7.2.dev6 → ledgered-0.7.2.dev8}/pyproject.toml

@@ -55,10 +55,6 @@ local_scheme = "no-local-version"
 [tool.mypy]
 ignore_missing_imports = true
 
-[tool.yapf]
-based_on_style = "pep8"
-column_limit = 100
-
 [tool.coverage.report]
 show_missing = true
 exclude_lines = [
@@ -66,8 +62,5 @@ exclude_lines = [
     "pragma: no cover"
 ]
 
-[tool.bandit]
-skips = ["B101"]
-
-[tool.flake8]
-max-line-length = 100
+[tool.ruff]
+line-length = 100

{ledgered-0.7.2.dev6 → ledgered-0.7.2.dev8}/src/ledgered/__version__.py

@@ -12,5 +12,5 @@ __version__: str
 __version_tuple__: VERSION_TUPLE
 version_tuple: VERSION_TUPLE
 
-__version__ = version = '0.7.2.dev6'
-__version_tuple__ = version_tuple = (0, 7, 2, 'dev6')
+__version__ = version = '0.7.2.dev8'
+__version_tuple__ = version_tuple = (0, 7, 2, 'dev8')

{ledgered-0.7.2.dev6 → ledgered-0.7.2.dev8}/src/ledgered/binary.py

@@ -32,7 +32,6 @@ class Sections(Jsonable):
 
 
 class LedgerBinaryApp:
-
     def __init__(self, binary_path: Union[str, Path]):
         if isinstance(binary_path, str):
             binary_path = Path(binary_path)
@@ -41,7 +40,8 @@ class LedgerBinaryApp:
         with self._path.open("rb") as filee:
             sections = {
                 s.name.replace(LEDGER_PREFIX, ""): s.data().decode().strip()
-                for s in ELFFile(filee).iter_sections() if LEDGER_PREFIX in s.name
+                for s in ELFFile(filee).iter_sections()
+                if LEDGER_PREFIX in s.name
             }
         self._sections = Sections(**sections)
 
@@ -51,16 +51,15 @@ class LedgerBinaryApp:
 
 
 def set_parser() -> ArgumentParser:
-    parser = ArgumentParser(prog="ledger-binary",
-                            description="Utilitary to parse Ledger embedded application ELF file "
-                            "and output metadatas")
-    parser.add_argument('-v', '--verbose', action='count', default=0)
+    parser = ArgumentParser(
+        prog="ledger-binary",
+        description="Utilitary to parse Ledger embedded application ELF file and output metadatas",
+    )
+    parser.add_argument("-v", "--verbose", action="count", default=0)
     parser.add_argument("binary", type=Path, help="The ledger embedded application ELF file")
-    parser.add_argument("-j",
-                        "--json",
-                        required=False,
-                        action="store_true",
-                        help="outputs as JSON rather than text")
+    parser.add_argument(
+        "-j", "--json", required=False, action="store_true", help="outputs as JSON rather than text"
+    )
     return parser
 
 

{ledgered-0.7.2.dev6 → ledgered-0.7.2.dev8}/src/ledgered/github.py

@@ -17,14 +17,14 @@ class Condition(IntEnum):
 
 
 class NoManifestException(FileNotFoundError):
-
     def __init__(self, repository: "AppRepository"):
-        super().__init__(f"`ledger_app.toml` manifest not found in repository '{repository.url}', "
-                         f"branch '{repository.current_branch}'.")
+        super().__init__(
+            f"`ledger_app.toml` manifest not found in repository '{repository.url}', "
+            f"branch '{repository.current_branch}'."
+        )
 
 
 class AppRepository(PyRepository.Repository):
-
     def __init__(self, *args, **kwargs) -> None:
         super().__init__(*args, **kwargs)
         self._manifest: Optional[Manifest] = None
@@ -60,8 +60,9 @@ class AppRepository(PyRepository.Repository):
     def makefile(self) -> str:
         if self._makefile is None:
             # paths on Windows contain "\" which are not compatible with GitHub remote paths
-            makefile = self.get_contents(str(self.makefile_path).replace("\\", "/"),
-                                         ref=self.current_branch)
+            makefile = self.get_contents(
+                str(self.makefile_path).replace("\\", "/"), ref=self.current_branch
+            )
             # `get_contents` can return a list, but here there can only be one Makefile / Cargo.toml
             assert isinstance(makefile, PyContentFile.ContentFile)
             self._makefile = makefile.decoded_content.decode()
@@ -72,9 +73,9 @@ class AppRepository(PyRepository.Repository):
         variants = []
         for line in self.makefile.splitlines():
             if "VARIANTS" in line:
-                variants.extend(line.split(' ')[3:])
+                variants.extend(line.split(" ")[3:])
             elif "VARIANT_VALUES = " in line:
-                variants.extend(line.split(" = ")[1].split(' '))
+                variants.extend(line.split(" = ")[1].split(" "))
         return variants
 
     @property
@@ -90,14 +91,15 @@ class AppRepository(PyRepository.Repository):
 
 
 class GitHubApps(list):
-
     def __init__(self, apps: List[AppRepository]):
         super().__init__([r for r in apps if r.name.startswith("app-")])
 
-    def filter(self,
-               name: Optional[str] = None,
-               archived: Condition = Condition.WITH,
-               private: Condition = Condition.WITH) -> "GitHubApps":
+    def filter(
+        self,
+        name: Optional[str] = None,
+        archived: Condition = Condition.WITH,
+        private: Condition = Condition.WITH,
+    ) -> "GitHubApps":
         new_list = [i for i in self]
         # archived filtering
         if archived == Condition.WITHOUT:
@@ -120,7 +122,6 @@ class GitHubApps(list):
 
 
 class GitHubLedgerHQ(PyGithub):
-
     def __init__(self, *args, **kwargs) -> None:
         super().__init__(*args, **kwargs)
         self._org = self.get_organization(LEDGER_ORG_NAME)

{ledgered-0.7.2.dev6 → ledgered-0.7.2.dev8}/src/ledgered/manifest/cli.py

@@ -37,81 +37,99 @@ def text_output(content: Dict, indent: int = 0) -> None:
 
 
 def set_parser() -> ArgumentParser:
-    parser = ArgumentParser(prog="ledger-manifest",
-                            description="Utilitary to parse and check an application "
-                            "'ledger_app.toml' manifest")
+    parser = ArgumentParser(
+        prog="ledger-manifest",
+        description="Utilitary to parse and check an application 'ledger_app.toml' manifest",
+    )
 
     # generic options
     parser.add_argument("-v", "--verbose", action="count", default=0)
-    parser.add_argument("-c",
-                        "--check",
-                        required=False,
-                        type=Path,
-                        default=None,
-                        help="Check the manifest content against the provided directory.")
+    parser.add_argument(
+        "-c",
+        "--check",
+        required=False,
+        type=Path,
+        default=None,
+        help="Check the manifest content against the provided directory.",
+    )
 
     # display options
-    parser.add_argument("manifest",
-                        type=Path,
-                        help=f"The manifest file (generally '{MANIFEST_FILE_NAME}' at the root of "
-                        "the application's repository), or the name of the app if the `--url` "
-                        "option is activated")
-    parser.add_argument("-u",
-                        "--url",
-                        action="store_true",
-                        default=False,
-                        help="Tells if the `manifest` is a repository HTTP URL rather than a file")
-    parser.add_argument("-os",
-                        "--output-sdk",
-                        required=False,
-                        action="store_true",
-                        default=False,
-                        help="outputs the SDK type")
-    parser.add_argument("-ob",
-                        "--output-build-directory",
-                        required=False,
-                        action="store_true",
-                        default=False,
-                        help="outputs the build directory (where the Makefile in C app, or the "
-                        "Cargo.toml in Rust app is expected to be)")
-    parser.add_argument("-od",
-                        "--output-devices",
-                        required=False,
-                        action="store_true",
-                        default=False,
-                        help="outputs the list of devices supported by the application")
-    parser.add_argument("-otu",
-                        "--output-tests-unit-directory",
-                        required=False,
-                        action="store_true",
-                        default=False,
-                        help="outputs the directory of the unit tests. Fails if none")
-    parser.add_argument("-otp",
-                        "--output-tests-pytest-directory",
-                        required=False,
-                        action="store_true",
-                        default=False,
-                        help="outputs the directory of the pytest (functional) tests. Fails if "
-                        "none")
-    parser.add_argument("-otd",
-                        "--output-tests-dependencies",
-                        required=False,
-                        action='store',
-                        default=None,
-                        nargs='*',
-                        help="outputs the given use cases. Fails if none")
-    parser.add_argument("-ouc",
-                        "--output-use-cases",
-                        required=False,
-                        default=None,
-                        action='store',
-                        nargs='*',
-                        help="outputs the given use cases. Fails if none")
-    parser.add_argument("-j",
-                        "--json",
-                        required=False,
-                        action="store_true",
-                        help="outputs as JSON rather than text")
+    parser.add_argument(
+        "manifest",
+        type=Path,
+        help=f"The manifest file (generally '{MANIFEST_FILE_NAME}' at the root of "
+        "the application's repository), or the name of the app if the `--url` "
+        "option is activated",
+    )
+    parser.add_argument(
+        "-u",
+        "--url",
+        action="store_true",
+        default=False,
+        help="Tells if the `manifest` is a repository HTTP URL rather than a file",
+    )
+    parser.add_argument(
+        "-os",
+        "--output-sdk",
+        required=False,
+        action="store_true",
+        default=False,
+        help="outputs the SDK type",
+    )
+    parser.add_argument(
+        "-ob",
+        "--output-build-directory",
+        required=False,
+        action="store_true",
+        default=False,
+        help="outputs the build directory (where the Makefile in C app, or the "
+        "Cargo.toml in Rust app is expected to be)",
+    )
+    parser.add_argument(
+        "-od",
+        "--output-devices",
+        required=False,
+        action="store_true",
+        default=False,
+        help="outputs the list of devices supported by the application",
+    )
+    parser.add_argument(
+        "-otu",
+        "--output-tests-unit-directory",
+        required=False,
+        action="store_true",
+        default=False,
+        help="outputs the directory of the unit tests. Fails if none",
+    )
+    parser.add_argument(
+        "-otp",
+        "--output-tests-pytest-directory",
+        required=False,
+        action="store_true",
+        default=False,
+        help="outputs the directory of the pytest (functional) tests. Fails if none",
+    )
+    parser.add_argument(
+        "-otd",
+        "--output-tests-dependencies",
+        required=False,
+        action="store",
+        default=None,
+        nargs="*",
+        help="outputs the given use cases. Fails if none",
+    )
+    parser.add_argument(
+        "-ouc",
+        "--output-use-cases",
+        required=False,
+        default=None,
+        action="store",
+        nargs="*",
+        help="outputs the given use cases. Fails if none",
+    )
+    parser.add_argument(
+        "-j", "--json", required=False, action="store_true", help="outputs as JSON rather than text"
+    )
     return parser
 
 
@@ -170,8 +188,7 @@ def main() -> None:  # pragma: no cover
         non_empty = len(dependencies) > 0
         if len(args.output_tests_dependencies) != 0:
             dependencies = {
-                k: v
-                for (k, v) in dependencies.items() if k in args.output_tests_dependencies
+                k: v for (k, v) in dependencies.items() if k in args.output_tests_dependencies
             }
         if not len(dependencies) and non_empty:
             logger.error("No use case match these ones: '%s'", args.output_tests_dependencies)

{ledgered-0.7.2.dev6 → ledgered-0.7.2.dev8}/src/ledgered/manifest/manifest.py

@@ -17,10 +17,9 @@ class Manifest(Jsonable):
     tests: Optional[TestsConfig]
     use_cases: Optional[UseCasesConfig]
 
-    def __init__(self,
-                 app: Dict,
-                 tests: Optional[Dict] = None,
-                 use_cases: Optional[Dict] = None) -> None:
+    def __init__(
+        self, app: Dict, tests: Optional[Dict] = None, use_cases: Optional[Dict] = None
+    ) -> None:
         self.app = AppConfig(**app)
         self.tests = None if tests is None else TestsConfig(**tests)
         self.use_cases = None if use_cases is None else UseCasesConfig(**use_cases)
@@ -43,8 +42,13 @@ class Manifest(Jsonable):
     def check(self, base_directory: Union[str, Path]) -> None:
         base_directory = Path(base_directory)
         assert base_directory.is_dir(), f"Given '{base_directory}' must be a directory"
-        build_file = base_directory / self.app.build_directory / \
-            ("Cargo.toml" if self.app.is_rust else "Makefile")
+        build_file = (
+            base_directory
+            / self.app.build_directory
+            / ("Cargo.toml" if self.app.is_rust else "Makefile")
+        )
         logging.info("Checking existence of file %s", build_file)
-        assert build_file.is_file(), f"No file '{build_file}' (from the given base directory " \
+        assert build_file.is_file(), (
+            f"No file '{build_file}' (from the given base directory "
             f"'{base_directory}' + the manifest path '{self.app.build_directory}') was found"
+        )

{ledgered-0.7.2.dev6 → ledgered-0.7.2.dev8}/src/ledgered/manifest/tests.py

@@ -33,8 +33,9 @@ class TestsDependencyConfig(Jsonable):
 
     @property
     def dir(self) -> Path:
-        return (self._base_dir / APPLICATION_DIRECTORY_NAME /
-                f"{self._name}-{self.ref}-{self.use_case}")
+        return (
+            self._base_dir / APPLICATION_DIRECTORY_NAME / f"{self._name}-{self.ref}-{self.use_case}"
+        )
 
     def __eq__(self, other: object) -> bool:
         if not isinstance(other, TestsDependencyConfig):
@@ -81,10 +82,12 @@ class TestsConfig(Jsonable):
     pytest_directory: Optional[Path]
     dependencies: Optional[JsonDict]
 
-    def __init__(self,
-                 pytest_directory: Optional[Union[str, Path]] = None,
-                 unit_directory: Optional[Union[str, Path]] = None,
-                 dependencies: Optional[Dict[str, List]] = None) -> None:
+    def __init__(
+        self,
+        pytest_directory: Optional[Union[str, Path]] = None,
+        unit_directory: Optional[Union[str, Path]] = None,
+        dependencies: Optional[Dict[str, List]] = None,
+    ) -> None:
         logger = getLogger()
         logger.debug("Parsing test dependencies")
         self.unit_directory = None if unit_directory is None else Path(unit_directory)

{ledgered-0.7.2.dev6 → ledgered-0.7.2.dev8}/src/ledgered/manifest/use_cases.py

@@ -12,8 +12,10 @@ class UseCasesConfig(Jsonable):
     def __init__(self, **cases: Optional[Dict]) -> None:
         for key, value in cases.items():
             if not isinstance(value, str):
-                raise ValueError(f"Use case '{key} = {value}' should have '{value}' as a string, "
-                                 f"not a {type(value)}")
+                raise ValueError(
+                    f"Use case '{key} = {value}' should have '{value}' as a string, "
+                    f"not a {type(value)}"
+                )
             if key == DEFAULT_USE_CASE:
                 raise ValueError(f"'{key}' use case is reserved and cannot be overridden")
         self.cases = JsonDict(cases)

{ledgered-0.7.2.dev6 → ledgered-0.7.2.dev8}/src/ledgered/manifest/utils.py

@@ -1,7 +1,7 @@
 import logging
 
 __logger__ = None
-DEFAULT_FORMAT = '[%(asctime)s][%(levelname)s] %(name)s - %(message)s'
+DEFAULT_FORMAT = "[%(asctime)s][%(levelname)s] %(name)s - %(message)s"
 
 
 def getLogger() -> logging.Logger:

{ledgered-0.7.2.dev6 → ledgered-0.7.2.dev8}/src/ledgered/serializers.py

@@ -6,12 +6,11 @@ def to_str_int(value: Any) -> Union[int, str]:
 
 
 class Jsonable:
-
     @property
     def json(self) -> Union[Dict, List]:
         output: Dict[Union[str, int], Any] = dict()
         for key, value in self.__dict__.items():
-            if key.startswith('_'):
+            if key.startswith("_"):
                 # 'hidden' properties are not to be included into the output
                 continue
             if isinstance(value, Jsonable):
@@ -22,7 +21,6 @@ class Jsonable:
 
 
 class JsonList(list, Jsonable):
-
     @property
     def json(self) -> List:
         output: List[Any] = list()
@@ -35,7 +33,6 @@ class JsonList(list, Jsonable):
 
 
 class JsonSet(set, Jsonable):
-
     @property
     def json(self) -> List:
         output: List[Any] = list()
@@ -48,7 +45,6 @@ class JsonSet(set, Jsonable):
 
 
 class JsonDict(dict, Jsonable):
-
     @property
     def json(self) -> Dict:
         output: Dict[Union[str, int], Any] = dict()

{ledgered-0.7.2.dev6 → ledgered-0.7.2.dev8/src/ledgered.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.2
 Name: ledgered
-Version: 0.7.2.dev6
+Version: 0.7.2.dev8
 Summary: Python tools, utils, libraries, to be used with Ledger cryptodevices
 Author-email: Ledger <hello@ledger.fr>
 License: MIT License

{ledgered-0.7.2.dev6 → ledgered-0.7.2.dev8}/src/ledgered.egg-info/SOURCES.txt

@@ -1,4 +1,5 @@
 .gitignore
+.pre-commit-config.yaml
 CHANGELOG.md
 LICENSE
 README.md

{ledgered-0.7.2.dev6 → ledgered-0.7.2.dev8}/tests/_data/minimal.toml

@@ -1,4 +1,4 @@
 [app]
 build_directory = "./"
 sdk = "C"
-devices = ["nanos", "nanox"]
+devices = ["nanos", "nanox"]

{ledgered-0.7.2.dev6 → ledgered-0.7.2.dev8}/tests/functional/conftest.py

@@ -6,9 +6,13 @@ from ledgered.github import GitHubLedgerHQ
 
 
 def pytest_addoption(parser):
-    parser.addoption("--token", required=False, default=None,
-                     help="Provide a GitHub token so that functional test won't trigger API "
-                     "restrictions too fast")
+    parser.addoption(
+        "--token",
+        required=False,
+        default=None,
+        help="Provide a GitHub token so that functional test won't trigger API "
+        "restrictions too fast",
+    )
 
 
 @pytest.fixture(scope="session")

{ledgered-0.7.2.dev6 → ledgered-0.7.2.dev8}/tests/functional/manifest/test_cli.py

@@ -4,13 +4,11 @@ from unittest import TestCase
 from unittest.mock import MagicMock, patch
 
 from ledgered.manifest.cli import main, set_parser
-from ledgered.manifest.tests import APPLICATION_DIRECTORY_KEY, APPLICATION_DIRECTORY_NAME
 
 from .. import TEST_MANIFEST_DIRECTORY
 
 
 class PrintMock(MagicMock):
-
     def get(self):
         result = list()
         for call_args in self.call_args_list:
@@ -18,6 +16,7 @@ class PrintMock(MagicMock):
         self.reset_mock()
         return "\n".join(result)
 
+
 FULL_EXPECTED_TEXT = """build_directory: .
 sdk: c
 devices:
@@ -33,14 +32,8 @@ FULL_EXPECTED_JSON = {
     "build_directory": ".",
     "sdk": "c",
     "devices": ["nanox"],
-    "use_cases": {
-        "debug": "DEBUG=1",
-        "test": "DEBUG=1"
-    },
-    "tests": {
-        "unit_directory": "tests/unit",
-        "pytest_directory": "tests/functional"
-    }
+    "use_cases": {"debug": "DEBUG=1", "test": "DEBUG=1"},
+    "tests": {"unit_directory": "tests/unit", "pytest_directory": "tests/functional"},
 }
 
 UC_D_EXPECTED_TEXT_CHUNKS = [
@@ -50,7 +43,7 @@ UC_D_EXPECTED_TEXT_CHUNKS = [
 tests:
   dependencies:
     testing_develop:""",
-    f"""
+    """
       url: https://github.com/<owner>/<app-repository>
       ref: develop
       use_case: debug
@@ -59,12 +52,12 @@ tests:
       url: https://github.com/<owner>/<other-app-repository>
       ref: develop
       use_case: default
-      application_directory: tests/functional/.dependencies/<other-app-repository>-develop-default"""
+      application_directory: tests/functional/.dependencies/<other-app-repository>-develop-default""",
 ]
 
 UC_D_EXPECTED_JSON = {
-    "use_cases": { "debug": "DEBUG=1", "test": "DEBUG=1" },
-    "tests": { "dependencies": {} }
+    "use_cases": {"debug": "DEBUG=1", "test": "DEBUG=1"},
+    "tests": {"dependencies": {}},
 }
 
 EXPECTED_DEPENDENCIES_JSON = [
@@ -72,14 +65,14 @@ EXPECTED_DEPENDENCIES_JSON = [
         "url": "https://github.com/<owner>/<app-repository>",
         "ref": "develop",
         "use_case": "debug",
-        "application_directory": "tests/functional/.dependencies/<app-repository>-develop-debug"
-     },
+        "application_directory": "tests/functional/.dependencies/<app-repository>-develop-debug",
+    },
     {
         "url": "https://github.com/<owner>/<other-app-repository>",
         "ref": "develop",
         "use_case": "default",
-        "application_directory": "tests/functional/.dependencies/<other-app-repository>-develop-default"
-    }
+        "application_directory": "tests/functional/.dependencies/<other-app-repository>-develop-default",
+    },
 ]
 
 
@@ -132,7 +125,7 @@ class TestCLIMain(TestCase):
     def test_cases_and_dependencies_json(self):
         self.args.output_use_cases = list()
         self.args.output_tests_dependencies = list()
-        self.args.json=True
+        self.args.json = True
         self.assertIsNone(main())
         # like before, the output needs to be divided and the lists compared
         # separately as they may have been re-ordered
@@ -180,12 +173,16 @@ class TestCLIMain(TestCase):
   ref: develop
   use_case: debug
   application_directory: some/dir/.dependencies/<app-repository>-develop-debug"""
-        expected_json = {"testing_develop": [{
-            "url": "https://github.com/<owner>/<app-repository>",
-            "ref": "develop",
-            "use_case": "debug",
-            "application_directory": "some/dir/.dependencies/<app-repository>-develop-debug"
-        }]}
+        expected_json = {
+            "testing_develop": [
+                {
+                    "url": "https://github.com/<owner>/<app-repository>",
+                    "ref": "develop",
+                    "use_case": "debug",
+                    "application_directory": "some/dir/.dependencies/<app-repository>-develop-debug",
+                }
+            ]
+        }
         self.assertIsNone(main())
         self.assertEqual(self.text, expected_text)
 
@@ -224,7 +221,6 @@ class TestCLIMain(TestCase):
 
 
 class TestCLIset_parser(TestCase):
-
     diffMax = None
 
     def test_set_parser(self):

{ledgered-0.7.2.dev6 → ledgered-0.7.2.dev8}/tests/functional/test_github.py

@@ -1,10 +1,9 @@
 import pytest
 import requests
 from pathlib import Path
-from unittest import TestCase
 
 from github.GithubException import GithubException
-from ledgered.github import AppRepository, Condition, GitHubApps, GitHubLedgerHQ
+from ledgered.github import AppRepository, GitHubApps
 
 
 def test_apps(gh):
@@ -29,7 +28,9 @@ def test_exchange_makefile_path(exchange):
 
 
 def test_exchange_makefile(exchange):
-    makefile = requests.get("https://raw.githubusercontent.com/LedgerHQ/app-exchange/develop/Makefile").content.decode()
+    makefile = requests.get(
+        "https://raw.githubusercontent.com/LedgerHQ/app-exchange/develop/Makefile"
+    ).content.decode()
     assert exchange.makefile == makefile
 
 

{ledgered-0.7.2.dev6 → ledgered-0.7.2.dev8}/tests/unit/manifest/test_app.py

@@ -5,7 +5,6 @@ from ledgered.manifest.app import AppConfig
 
 
 class TestAppConfig(TestCase):
-
     def test___init___ok_complete(self):
         sdk = "Rust"
         bd = Path("some path")

{ledgered-0.7.2.dev6 → ledgered-0.7.2.dev8}/tests/unit/manifest/test_manifest.py

@@ -1,13 +1,12 @@
 from pathlib import Path
 from unittest import TestCase
 
-from ledgered.manifest.manifest import Manifest, MANIFEST_FILE_NAME
+from ledgered.manifest.manifest import Manifest, MANIFEST_FILE_NAME, TestsConfig
 
 from .. import TEST_MANIFEST_DIRECTORY
 
 
 class TestManifest(TestCase):
-
     def check_ledger_app_toml(self, manifest: Manifest) -> None:
         self.assertEqual(manifest.app.sdk, "rust")
         self.assertEqual(manifest.app.devices, {"nanos", "stax", "flex"})
@@ -15,6 +14,7 @@ class TestManifest(TestCase):
         self.assertTrue(manifest.app.is_rust)
         self.assertFalse(manifest.app.is_c)
 
+        self.assertIsInstance(manifest.tests, TestsConfig)
         self.assertEqual(manifest.tests.unit_directory, Path("unit"))
         self.assertEqual(manifest.tests.pytest_directory, Path("pytest"))
 

{ledgered-0.7.2.dev6 → ledgered-0.7.2.dev8}/tests/unit/manifest/test_tests.py

@@ -3,12 +3,17 @@ from unittest import TestCase
 
 from ledgered.manifest.constants import DEFAULT_USE_CASE
 from ledgered.manifest.errors import MissingField
-from ledgered.manifest.tests import DuplicateDependencyError, TestsConfig, TestsDependencyConfig, \
-    TestsDependenciesConfig, APPLICATION_DIRECTORY_KEY, APPLICATION_DIRECTORY_NAME
+from ledgered.manifest.tests import (
+    DuplicateDependencyError,
+    TestsConfig,
+    TestsDependencyConfig,
+    TestsDependenciesConfig,
+    APPLICATION_DIRECTORY_KEY,
+    APPLICATION_DIRECTORY_NAME,
+)
 
 
 class TestTestsDependencyConfig(TestCase):
-
     def setUp(self):
         self.n, self.r, self.bd, self.uc = "name", "ref", Path("base_dir"), "use_case"
         self.url = "/".join(["some url", self.n])
@@ -43,26 +48,38 @@ class TestTestsDependencyConfig(TestCase):
 
 
 class TestTestsDependenciesConfig(TestCase):
-
     def setUp(self):
         self.bd = Path("test")
         self.u1, self.r1, self.uc1 = "url1", "ref1", "use_case1"
         self.u2, self.r2, self.uc2 = "url2", "ref2", "use_case2"
-        self.input = [{"url": self.u1, "ref": self.r1, "use_case": self.uc1},
-                      {"url": self.u2, "ref": self.r2, "use_case": self.uc2}]
+        self.input = [
+            {"url": self.u1, "ref": self.r1, "use_case": self.uc1},
+            {"url": self.u2, "ref": self.r2, "use_case": self.uc2},
+        ]
         self.tdc = TestsDependenciesConfig(self.input, self.bd)
 
     def test___init__ok(self):
-        self.assertSetEqual(self.tdc.dependencies, {TestsDependencyConfig(self.u1, self.r1, self.bd, self.uc1),
-                                                    TestsDependencyConfig(self.u2, self.r2, self.bd, self.uc2)})
+        self.assertSetEqual(
+            self.tdc.dependencies,
+            {
+                TestsDependencyConfig(self.u1, self.r1, self.bd, self.uc1),
+                TestsDependencyConfig(self.u2, self.r2, self.bd, self.uc2),
+            },
+        )
 
     def test___init__nok(self):
         with self.assertRaises(DuplicateDependencyError):
-            TestsDependenciesConfig([{"url": self.u1, "ref": self.r1, "use_case": self.uc1}]*2, base_dir="something")
+            TestsDependenciesConfig(
+                [{"url": self.u1, "ref": self.r1, "use_case": self.uc1}] * 2, base_dir="something"
+            )
 
     def test_json(self):
-        self.input[0][APPLICATION_DIRECTORY_KEY] = str(self.bd / APPLICATION_DIRECTORY_NAME / f"{self.u1}-{self.r1}-{self.uc1}")
-        self.input[1][APPLICATION_DIRECTORY_KEY] = str(self.bd / APPLICATION_DIRECTORY_NAME / f"{self.u2}-{self.r2}-{self.uc2}")
+        self.input[0][APPLICATION_DIRECTORY_KEY] = str(
+            self.bd / APPLICATION_DIRECTORY_NAME / f"{self.u1}-{self.r1}-{self.uc1}"
+        )
+        self.input[1][APPLICATION_DIRECTORY_KEY] = str(
+            self.bd / APPLICATION_DIRECTORY_NAME / f"{self.u2}-{self.r2}-{self.uc2}"
+        )
         # CountEqual rather than ListEqual, as the list is managed as a set and the content can
         # be reordered when serialized back to list
         self.assertCountEqual(self.tdc.json, self.input)
@@ -74,9 +91,13 @@ class TestTestsConfig(TestCase):
     def test____init___ok_complete(self):
         ud = Path("")
         pd = Path("something")
-        deps = {"first": [{"url": "url", "ref": "ref"}],
-                "second": [{"url": "u1", "ref": "r1", "use_case": "uc1"},
-                           {"url": "u2", "ref": "r2", "use_case": "uc2"}]}
+        deps = {
+            "first": [{"url": "url", "ref": "ref"}],
+            "second": [
+                {"url": "u1", "ref": "r1", "use_case": "uc1"},
+                {"url": "u2", "ref": "r2", "use_case": "uc2"},
+            ],
+        }
         config = TestsConfig(unit_directory=str(ud), pytest_directory=str(pd), dependencies=deps)
         self.assertIsNotNone(config.unit_directory)
         self.assertEqual(config.unit_directory, ud)
@@ -90,9 +111,15 @@ class TestTestsConfig(TestCase):
         result_json = config.json
         result_deps = result_json.pop("dependencies")
         self.assertEqual(len(result_deps), len(deps))
-        deps["first"][0][APPLICATION_DIRECTORY_KEY] = str(pd / APPLICATION_DIRECTORY_NAME / f"url-ref-default")
-        deps["second"][0][APPLICATION_DIRECTORY_KEY] = str(pd / APPLICATION_DIRECTORY_NAME / f"u1-r1-uc1")
-        deps["second"][1][APPLICATION_DIRECTORY_KEY] = str(pd / APPLICATION_DIRECTORY_NAME / f"u2-r2-uc2")
+        deps["first"][0][APPLICATION_DIRECTORY_KEY] = str(
+            pd / APPLICATION_DIRECTORY_NAME / "url-ref-default"
+        )
+        deps["second"][0][APPLICATION_DIRECTORY_KEY] = str(
+            pd / APPLICATION_DIRECTORY_NAME / "u1-r1-uc1"
+        )
+        deps["second"][1][APPLICATION_DIRECTORY_KEY] = str(
+            pd / APPLICATION_DIRECTORY_NAME / "u2-r2-uc2"
+        )
         for k, v in result_deps.items():
             self.assertIn(k, deps)
             self.assertCountEqual(v, deps[k])

{ledgered-0.7.2.dev6 → ledgered-0.7.2.dev8}/tests/unit/manifest/test_use_cases.py

@@ -1,4 +1,3 @@
-from pathlib import Path
 from unittest import TestCase
 
 from ledgered.manifest.constants import DEFAULT_USE_CASE
@@ -6,11 +5,9 @@ from ledgered.manifest.use_cases import UseCasesConfig
 
 
 class TestUseCaseConfig(TestCase):
-
     def setUp(self):
         self.key = "first"
-        self.cases = {self.key: "something",
-                   "other": "something else"}
+        self.cases = {self.key: "something", "other": "something else"}
         self.uc = UseCasesConfig(**self.cases)
 
     def test___init__empty(self):

{ledgered-0.7.2.dev6 → ledgered-0.7.2.dev8}/tests/unit/test_binary.py

@@ -1,6 +1,6 @@
 from dataclasses import dataclass
 from unittest import TestCase
-from unittest.mock import MagicMock, patch
+from unittest.mock import patch
 from pathlib import Path
 from typing import Any
 
@@ -8,7 +8,6 @@ from ledgered import binary as B
 
 
 class TestSections(TestCase):
-
     def setUp(self):
         self.inputs = {
             "api_level": "api_level",
@@ -44,26 +43,26 @@ class TestSections(TestCase):
 
     def test___str__(self):
         sections = B.Sections(**self.inputs)
-        self.assertEqual("\n".join(f"{k} {v}"
-                                   for k,v in sorted(self.inputs.items())),
-                         str(sections))
+        self.assertEqual(
+            "\n".join(f"{k} {v}" for k, v in sorted(self.inputs.items())), str(sections)
+        )
 
     def test_json(self):
         sections = B.Sections(**self.inputs)
         # explicit `str(v)` as None values needs to be converted to 'None'
-        self.assertDictEqual({k: str(v) for k,v in self.inputs.items()}, sections.json)
+        self.assertDictEqual({k: str(v) for k, v in self.inputs.items()}, sections.json)
 
 
 @dataclass
 class Section:
     name: str
     _data: Any
+
     def data(self) -> Any:
         return self._data
 
 
 class TestLedgerBinaryApp(TestCase):
-
     def test___init__(self):
         path = Path("/dev/urandom")
         api_level, sdk_hash = "something", "some hash"
@@ -73,12 +72,12 @@ class TestLedgerBinaryApp(TestCase):
                 Section("unused", 1),
                 Section("ledger.api_level", api_level.encode()),
                 Section("ledger.sdk_hash", sdk_hash.encode()),
-                Section("still not used", b"some data")
+                Section("still not used", b"some data"),
             ]
             bin = B.LedgerBinaryApp(path)
         self.assertEqual(bin.sections, expected)
 
     def test___init__from_str(self):
         path = "/dev/urandom"
-        with patch("ledgered.binary.ELFFile") as elfmock:
+        with patch("ledgered.binary.ELFFile"):
             B.LedgerBinaryApp(path)

{ledgered-0.7.2.dev6 → ledgered-0.7.2.dev8}/tests/unit/test_github.py

@@ -1,8 +1,7 @@
 from dataclasses import dataclass
 from unittest import TestCase
-from unittest.mock import patch, MagicMock
 
-from ledgered.github import AppRepository, Condition, GitHubApps, GitHubLedgerHQ
+from ledgered.github import Condition, GitHubApps, GitHubLedgerHQ
 
 
 @dataclass
@@ -13,7 +12,6 @@ class AppRepositoryMock:
 
 
 class TestGitHubApps(TestCase):
-
     def setUp(self):
         self.app1 = AppRepositoryMock("app-1")
         self.app2 = AppRepositoryMock("not-app")
@@ -39,7 +37,6 @@ class TestGitHubApps(TestCase):
 
 
 class TestGitHubLedgerHQ(TestCase):
-
     def setUp(self):
         self.g = GitHubLedgerHQ()
 

{ledgered-0.7.2.dev6 → ledgered-0.7.2.dev8}/tests/unit/test_serializers.py

@@ -11,6 +11,7 @@ class JsonableTest1(Jsonable):
     def __hash__(self):
         return 1
 
+
 @dataclass
 class JsonableTest2(Jsonable):
     one: str
@@ -18,7 +19,6 @@ class JsonableTest2(Jsonable):
 
 
 class TestTypesModule(TestCase):
-
     def setUp(self):
         self.jt1 = JsonableTest1("base")
 
@@ -30,30 +30,30 @@ class TestTypesModule(TestCase):
     def test_JsonList_json(self):
         integer = 4
         string = "str"
-        l = JsonList()
-        l.append(integer)
-        l.append(string)
-        l.append(self.jt1)
+        j_list = JsonList()
+        j_list.append(integer)
+        j_list.append(string)
+        j_list.append(self.jt1)
         expected = [integer, string, {"base": "base"}]
-        self.assertEqual(l.json, expected)
+        self.assertEqual(j_list.json, expected)
 
     def test_JsonSet_json(self):
         integer = 4
         string = "str"
-        l = JsonSet()
-        l.add(integer)
-        l.add(string)
-        l.add(self.jt1)
+        j_set = JsonSet()
+        j_set.add(integer)
+        j_set.add(string)
+        j_set.add(self.jt1)
         expected = [integer, string, {"base": "base"}]
-        result = l.json
+        result = j_set.json
         # the JsonSet.json returns a list, but the set may have broken the order so we can't
         # directly compare the two lists.
         self.assertIsInstance(result, list)
         self.assertCountEqual(result, expected)
 
     def test_JsonDict_json(self):
-        l = JsonDict()
-        l[4] = 5
-        l["base"] = self.jt1
+        j_dict = JsonDict()
+        j_dict[4] = 5
+        j_dict["base"] = self.jt1
         expected = {4: 5, "base": {"base": "base"}}
-        self.assertEqual(l.json, expected)
+        self.assertEqual(j_dict.json, expected)

ledgered-0.7.2.dev6/.github/workflows/fast-checks.yml

@@ -1,64 +0,0 @@
-name: Fast checks
-
-on:
-  workflow_dispatch:
-  push:
-    branches:
-      - develop
-      - master
-  pull_request:
-
-jobs:
-  lint:
-    name: Linting
-    runs-on: ubuntu-latest
-    steps:
-    - name: Clone
-      uses: actions/checkout@v4
-    - run: pip install flake8 flake8-pyproject
-    - name: Flake8 lint Python code
-      run: flake8 src/
-
-  yapf:
-    name: Formatting
-    runs-on: ubuntu-latest
-    steps:
-    - name: Clone
-      uses: actions/checkout@v4
-    - run: pip install yapf toml
-    - name: Yapf source formatting
-      run: |
-        yapf src/ --recursive -d
-
-  mypy:
-    name: Type checking
-    runs-on: ubuntu-latest
-    steps:
-    - name: Clone
-      uses: actions/checkout@v4
-    - run: pip install mypy types-toml
-    - name: Mypy type checking
-      run: mypy src
-
-  bandit:
-    name: Security checking
-    runs-on: ubuntu-latest
-    steps:
-    - name: Clone
-      uses: actions/checkout@v4
-    - run: pip install bandit
-    - name: Bandit security checking
-      run: bandit -r src -ll
-
-  misspell:
-    name: Check misspellings
-    runs-on: ubuntu-latest
-    steps:
-    - name: Clone
-      uses: actions/checkout@v4
-    - name: Check misspellings
-      uses: codespell-project/actions-codespell@v1
-      with:
-        builtin: clear,rare
-        check_filenames: true
-        ignore_words_list: assertIn,crate,