Ruff autofix

build/lib/cli/build/role_include.py

@@ -104,7 +104,6 @@ def topological_sort(graph, in_degree, roles=None):
     Perform topological sort on the dependency graph.
     If a cycle is detected, raise an Exception with detailed debug info.
     """
-    from collections import deque
 
     queue = deque([r for r, d in in_degree.items() if d == 0])
     sorted_roles = []

build/lib/cli/create/role.py

@@ -2,11 +2,11 @@
 import argparse
 import shutil
 import ipaddress
-import difflib
 from jinja2 import Environment, FileSystemLoader
 from ruamel.yaml import YAML
 
-import sys, os
+import sys
+import os
 sys.path.insert(0, os.path.abspath(os.path.join(os.path.dirname(__file__), '..')))
 from module_utils.entity_name_utils import get_entity_name
 

build/lib/cli/encrypt/inventory.py

@@ -1,9 +1,7 @@
 import argparse
-import subprocess
-import sys
 from pathlib import Path
 import yaml
-from typing import Dict, Any
+from typing import Any
 from module_utils.handler.vault import VaultHandler, VaultScalar
 from module_utils.handler.yaml import YamlHandler
 from yaml.dumper import SafeDumper

build/lib/cli/fix/tabs.py

@@ -1,6 +1,5 @@
 #!/usr/bin/env python3
 
-import os
 import argparse
 from pathlib import Path
 

build/lib/cli/setup/applications.py

@@ -2,7 +2,6 @@
 import argparse
 import yaml
 import sys
-import time
 from pathlib import Path
 
 # Ensure project root on PYTHONPATH so module_utils is importable

build/lib/filter_plugins/active_docker.py

@@ -13,7 +13,7 @@ Returns an integer. If ensure_min_one=True, returns at least 1.
 """
 
 import re
-from typing import Any, Dict, Mapping, Iterable
+from typing import Any, Mapping, Iterable
 
 
 def _is_mapping(x: Any) -> bool:

build/lib/filter_plugins/domain_redirect_mappings.py

@@ -1,5 +1,6 @@
 from ansible.errors import AnsibleFilterError
-import sys, os
+import sys
+import os
 sys.path.insert(0, os.path.abspath(os.path.join(os.path.dirname(__file__), '..')))
 from module_utils.entity_name_utils import get_entity_name
 

build/lib/filter_plugins/get_app_conf.py

@@ -1,6 +1,7 @@
-import sys, os
+import sys
+import os
 sys.path.insert(0, os.path.abspath(os.path.join(os.path.dirname(__file__), '..')))
-from module_utils.config_utils import get_app_conf, AppConfigKeyError,ConfigEntryNotSetError
+from module_utils.config_utils import get_app_conf
 
 class FilterModule(object):
     ''' Infinito.Nexus application config extraction filters '''

build/lib/filter_plugins/get_docker_paths.py

@@ -1,4 +1,5 @@
-import sys, os
+import sys
+import os
 sys.path.insert(0, os.path.abspath(os.path.join(os.path.dirname(__file__), '..')))
 from module_utils.entity_name_utils import get_entity_name
 

build/lib/filter_plugins/get_entity_name.py

@@ -1,4 +1,5 @@
-import sys, os
+import sys
+import os
 sys.path.insert(0, os.path.abspath(os.path.join(os.path.dirname(__file__), '..')))
 from module_utils.entity_name_utils import get_entity_name
 

build/lib/filter_plugins/get_url.py

@@ -1,5 +1,6 @@
 #!/usr/bin/python
-import sys, os
+import sys
+import os
 sys.path.insert(0, os.path.abspath(os.path.join(os.path.dirname(__file__), '..')))
 from module_utils.get_url import get_url
 

build/lib/filter_plugins/memory_filters.py

@@ -1,6 +1,8 @@
 from __future__ import annotations
 
-import sys, os, re
+import sys
+import os
+import re
 sys.path.insert(0, os.path.abspath(os.path.join(os.path.dirname(__file__), '..')))
 
 from ansible.errors import AnsibleFilterError

build/lib/filter_plugins/resource_filter.py

@@ -1,7 +1,8 @@
 # filter_plugins/resource_filter.py
 from __future__ import annotations
 
-import sys, os
+import sys
+import os
 sys.path.insert(0, os.path.abspath(os.path.join(os.path.dirname(__file__), '..')))
 
 from module_utils.config_utils import get_app_conf, AppConfigKeyError, ConfigEntryNotSetError  # noqa: F401

build/lib/module_utils/cert_utils.py

@@ -5,7 +5,6 @@ __metaclass__ = type
 
 import os
 import subprocess
-import time
 from datetime import datetime
 
 class CertUtils:

build/lib/module_utils/get_url.py

@@ -1,5 +1,6 @@
 from ansible.errors import AnsibleFilterError
-import sys, os
+import sys
+import os
 
 def get_url(domains, application_id, protocol):
     plugin_dir   = os.path.dirname(__file__)

build/lib/module_utils/valid_deploy_id.py

@@ -4,8 +4,6 @@ Utility for validating deployment application IDs against defined roles and inve
 """
 import os
 import yaml
-import glob
-import configparser
 
 from filter_plugins.get_all_application_ids import get_all_application_ids
 

cli/build/role_include.py

@@ -104,7 +104,6 @@ def topological_sort(graph, in_degree, roles=None):
     Perform topological sort on the dependency graph.
     If a cycle is detected, raise an Exception with detailed debug info.
     """
-    from collections import deque
 
     queue = deque([r for r, d in in_degree.items() if d == 0])
     sorted_roles = []

cli/create/role.py

@@ -2,11 +2,11 @@
 import argparse
 import shutil
 import ipaddress
-import difflib
 from jinja2 import Environment, FileSystemLoader
 from ruamel.yaml import YAML
 
-import sys, os
+import sys
+import os
 sys.path.insert(0, os.path.abspath(os.path.join(os.path.dirname(__file__), '..')))
 from module_utils.entity_name_utils import get_entity_name
 

cli/encrypt/inventory.py

@@ -1,9 +1,7 @@
 import argparse
-import subprocess
-import sys
 from pathlib import Path
 import yaml
-from typing import Dict, Any
+from typing import Any
 from module_utils.handler.vault import VaultHandler, VaultScalar
 from module_utils.handler.yaml import YamlHandler
 from yaml.dumper import SafeDumper

cli/fix/tabs.py

@@ -1,6 +1,5 @@
 #!/usr/bin/env python3
 
-import os
 import argparse
 from pathlib import Path
 

cli/setup/applications.py

@@ -2,7 +2,6 @@
 import argparse
 import yaml
 import sys
-import time
 from pathlib import Path
 
 # Ensure project root on PYTHONPATH so module_utils is importable

filter_plugins/active_docker.py

@@ -13,7 +13,7 @@ Returns an integer. If ensure_min_one=True, returns at least 1.
 """
 
 import re
-from typing import Any, Dict, Mapping, Iterable
+from typing import Any, Mapping, Iterable
 
 
 def _is_mapping(x: Any) -> bool:

filter_plugins/domain_redirect_mappings.py

@@ -1,5 +1,6 @@
 from ansible.errors import AnsibleFilterError
-import sys, os
+import sys
+import os
 sys.path.insert(0, os.path.abspath(os.path.join(os.path.dirname(__file__), '..')))
 from module_utils.entity_name_utils import get_entity_name
 

filter_plugins/get_app_conf.py

@@ -1,6 +1,7 @@
-import sys, os
+import sys
+import os
 sys.path.insert(0, os.path.abspath(os.path.join(os.path.dirname(__file__), '..')))
-from module_utils.config_utils import get_app_conf, AppConfigKeyError,ConfigEntryNotSetError
+from module_utils.config_utils import get_app_conf
 
 class FilterModule(object):
     ''' Infinito.Nexus application config extraction filters '''

filter_plugins/get_docker_paths.py

@@ -1,4 +1,5 @@
-import sys, os
+import sys
+import os
 sys.path.insert(0, os.path.abspath(os.path.join(os.path.dirname(__file__), '..')))
 from module_utils.entity_name_utils import get_entity_name
 

filter_plugins/get_entity_name.py

@@ -1,4 +1,5 @@
-import sys, os
+import sys
+import os
 sys.path.insert(0, os.path.abspath(os.path.join(os.path.dirname(__file__), '..')))
 from module_utils.entity_name_utils import get_entity_name
 

filter_plugins/get_url.py

@@ -1,5 +1,6 @@
 #!/usr/bin/python
-import sys, os
+import sys
+import os
 sys.path.insert(0, os.path.abspath(os.path.join(os.path.dirname(__file__), '..')))
 from module_utils.get_url import get_url
 

filter_plugins/memory_filters.py

@@ -1,6 +1,8 @@
 from __future__ import annotations
 
-import sys, os, re
+import sys
+import os
+import re
 sys.path.insert(0, os.path.abspath(os.path.join(os.path.dirname(__file__), '..')))
 
 from ansible.errors import AnsibleFilterError

filter_plugins/resource_filter.py

@@ -1,7 +1,8 @@
 # filter_plugins/resource_filter.py
 from __future__ import annotations
 
-import sys, os
+import sys
+import os
 sys.path.insert(0, os.path.abspath(os.path.join(os.path.dirname(__file__), '..')))
 
 from module_utils.config_utils import get_app_conf, AppConfigKeyError, ConfigEntryNotSetError  # noqa: F401

main.py

@@ -5,7 +5,6 @@ import subprocess
 import sys
 import textwrap
 import threading
-import signal
 from datetime import datetime
 import pty
 from module_utils.sounds import Sound
@@ -236,7 +235,6 @@ def play_start_intro():
 
 
 from multiprocessing import Process, get_start_method, set_start_method
-import time
 
 def _call_sound(method_name: str):
    # Re-import inside child to (re)init audio backend cleanly under 'spawn'

module_utils/cert_utils.py

@@ -5,7 +5,6 @@ __metaclass__ = type
 
 import os
 import subprocess
-import time
 from datetime import datetime
 
 class CertUtils:

module_utils/get_url.py

@@ -1,5 +1,6 @@
 from ansible.errors import AnsibleFilterError
-import sys, os
+import sys
+import os
 
 def get_url(domains, application_id, protocol):
     plugin_dir   = os.path.dirname(__file__)

module_utils/valid_deploy_id.py

@@ -4,8 +4,6 @@ Utility for validating deployment application IDs against defined roles and inve
 """
 import os
 import yaml
-import glob
-import configparser
 
 from filter_plugins.get_all_application_ids import get_all_application_ids
 

roles/svc-bkp-loc-2-usb/files/script.py

@@ -2,7 +2,6 @@
 
 import sys
 import subprocess
-import shutil
 import os
 import glob
 import datetime

roles/svc-opt-ssd-hdd/files/script.py

@@ -1,7 +1,6 @@
 import subprocess
 import os
 import time
-import sys
 import shutil
 import argparse
 

roles/sys-lock/files/sys-lock.py

@@ -1,7 +1,6 @@
 import argparse
 import subprocess
 import time
-import os
 from datetime import datetime
 
 # Global variable definition

roles/sys-svc-cdn/filter_plugins/cdn_paths.py

@@ -1,4 +1,3 @@
-import datetime
 import os
 
 def cdn_paths(cdn_root, application_id, version):

roles/web-app-keycloak/filter_plugins/redirect_uris.py

@@ -1,6 +1,7 @@
 # roles/web-app-keycloak/filter_plugins/redirect_uris.py
 from __future__ import annotations
-import os, sys
+import os
+import sys
 from typing import Iterable, Sequence
 from ansible.errors import AnsibleFilterError
 

roles/web-app-xwiki/filter_plugins/xwiki_filters.py

@@ -2,7 +2,7 @@
 from __future__ import annotations
 
 import re
-from typing import Any, Dict, Optional, Iterable
+from typing import Any, Iterable
 try:
     # Ansible provides this; don't hard-depend at import time for unit tests
     from ansible.errors import AnsibleFilterError

roles/web-svc-logout/filter_plugins/domain_filters.py

@@ -1,7 +1,8 @@
 # roles/web-svc-logout/filter_plugins/domain_filters.py
 
 from ansible.errors import AnsibleFilterError
-import sys, os
+import sys
+import os
 sys.path.insert(0, os.path.abspath(os.path.join(os.path.dirname(__file__), '..', '..', '..')))
 from module_utils.config_utils import get_app_conf
 

tests/integration/test_application_id.py

@@ -3,7 +3,7 @@ import yaml
 import os
 import glob
 
-from filter_plugins.invokable_paths import get_invokable_paths, get_non_invokable_paths
+from filter_plugins.invokable_paths import get_invokable_paths
 
 class TestSysRolesApplicationId(unittest.TestCase):
     """

tests/integration/test_application_id_consistency.py

@@ -1,4 +1,3 @@
-import os
 import yaml
 import unittest
 from pathlib import Path

tests/integration/test_docker_images_configuration.py

@@ -1,7 +1,6 @@
 import unittest
 import yaml
 from pathlib import Path
-import re
 
 class TestDockerRoleServicesConfiguration(unittest.TestCase):
     def test_services_keys_and_templates(self):

tests/integration/test_domain_application_ids.py

@@ -1,4 +1,3 @@
-import glob
 import os
 import re
 import unittest

tests/integration/test_domain_uniqueness.py

@@ -2,7 +2,7 @@ import unittest
 import yaml
 import subprocess
 from pathlib import Path
-from collections import Counter, defaultdict
+from collections import defaultdict
 
 class TestDomainUniqueness(unittest.TestCase):
     def test_no_duplicate_domains(self):

tests/integration/test_domains_structure.py

@@ -1,4 +1,3 @@
-import os
 import yaml
 import unittest
 from pathlib import Path

tests/integration/test_filters_are_defined.py

@@ -247,7 +247,7 @@ class TestAllUsedFiltersAreDefined(unittest.TestCase):
                 "(and are not in BUILTIN_FILTERS):"
             ]
             for f in unknown:
-                lines.append(f"- " + f)
+                lines.append("- " + f)
             self.fail("\n".join(lines))
 
 

tests/integration/test_meta_roles_integration.py

@@ -1,7 +1,6 @@
 import unittest
 from pathlib import Path
 import re
-import os
 import sys
 
 # Ensure your project root is on PYTHONPATH so filter_plugins can be imported

tests/integration/test_oauth2_acl_mutual_exclusive.py

@@ -1,4 +1,3 @@
-import os
 import yaml
 import unittest
 from pathlib import Path

tests/integration/test_run_after_application_ids.py

@@ -1,5 +1,4 @@
 #!/usr/bin/env python3
-import os
 import unittest
 import yaml
 from pathlib import Path

tests/integration/test_run_once_tags.py

@@ -16,7 +16,6 @@ def is_under_root_tasks(fpath):
 
 
 import os
-import re
 
 def find_role_includes(roles_dir):
     """

tests/integration/test_unnecessary_role_dependencies.py

@@ -4,7 +4,7 @@ import re
 import glob
 import yaml
 import unittest
-from typing import Dict, Set, List, Tuple, Optional
+from typing import Dict, Set, List, Optional
 
 # ---------------- Utilities ----------------
 

tests/integration/test_when_condition_duplication.py

@@ -1,5 +1,4 @@
 # tests/integration/test_when_condition_duplication.py
-import os
 import unittest
 from pathlib import Path
 from typing import Any, Dict, Iterable, List, Tuple

tests/integration/test_yaml_syntax.py

@@ -1,5 +1,4 @@
 import os
-import sys
 import unittest
 import yaml
 

tests/unit/cli/build/inventory/test_full.py

@@ -1,5 +1,4 @@
 import unittest
-import sys
 import os
 import importlib.util
 PROJECT_ROOT = os.path.abspath(os.path.join(os.path.dirname(__file__), '../../../../../'))

tests/unit/cli/create/test_credentials.py

@@ -11,7 +11,7 @@ sys.path.insert(0, dir_path)
 
 # Import functions and classes to test
 from cli.create.credentials import ask_for_confirmation, main
-from module_utils.handler.vault import VaultHandler, VaultScalar
+from module_utils.handler.vault import VaultHandler
 import subprocess
 import tempfile
 import yaml

tests/unit/cli/create/test_inventory.py

@@ -17,7 +17,6 @@ from cli.create.inventory import (  # type: ignore
     ensure_become_password,
     parse_roles_list,
     filter_inventory_by_include,
-    filter_inventory_by_ignore,
     get_path_administrator_home_from_group_vars,
     ensure_administrator_authorized_keys,
     apply_vars_overrides,

tests/unit/cli/meta/test_tree.py

@@ -2,7 +2,6 @@ import unittest
 import tempfile
 import shutil
 import os
-import json
 from cli.build import tree
 
 

tests/unit/cli/setup/test_applications_and_users.py

@@ -1,4 +1,3 @@
-import os
 import unittest
 import tempfile
 import shutil

tests/unit/filter_plugins/test_active_docker.py

@@ -1,5 +1,4 @@
 # tests/unit/filter_plugins/test_active_docker.py
-import os
 import sys
 import unittest
 from pathlib import Path

tests/unit/filter_plugins/test_get_category_entries.py

@@ -1,9 +1,7 @@
 # Unit tests for the get_category_entries Ansible filter plugin (unittest version).
 
-import os
 import unittest
 import tempfile
-import shutil
 from pathlib import Path
 
 from filter_plugins.get_category_entries import get_category_entries

tests/unit/filter_plugins/test_get_domain.py

@@ -1,6 +1,4 @@
 import unittest
-import sys
-import os
 from filter_plugins.get_domain import FilterModule
 from ansible.errors import AnsibleFilterError
 

tests/unit/roles/svc-bkp-rmt-2-loc/files/test_pull_specific_host.py

@@ -1,10 +1,8 @@
 import unittest
-import sys
 import types
 from pathlib import Path
-from unittest.mock import patch, MagicMock
+from unittest.mock import patch
 import subprocess
-import time
 import os
 
 

tests/unit/roles/svc-db-openldap/test_build_ldap_role_entries.py

@@ -1,5 +1,4 @@
 import unittest
-import sys
 import os
 import importlib.util
 

tests/unit/roles/sys-ctl-hlth-disc-space/files/script.py

@@ -1,6 +1,5 @@
 #!/usr/bin/env python3
 import io
-import os
 import sys
 import unittest
 import pathlib

tests/unit/roles/sys-ctl-rpr-docker-soft/files/test_script.py

@@ -1,6 +1,5 @@
 import unittest
 import types
-import sys
 from pathlib import Path
 from importlib.util import spec_from_file_location, module_from_spec
 

tests/unit/roles/sys-front-inj-all/filter_plugins/test_inj_snippets.py

@@ -9,7 +9,6 @@ Unit tests for roles/sys-front-inj-all/filter_plugins/inj_snippets.py
 """
 
 import os
-import sys
 import unittest
 import tempfile
 import importlib.util