diff --git a/build/lib/module_utils/sounds.py b/build/lib/module_utils/sounds.py index 9dd0eaec..592db9fe 100644 --- a/build/lib/module_utils/sounds.py +++ b/build/lib/module_utils/sounds.py @@ -175,7 +175,6 @@ try: def play_infinito_intro_sound(cls) -> None: build_time = 10.0 celebr_time = 12.0 - breakdown_time = 10.0 overlap = 3.0 bass_seg = 0.125 diff --git a/build/lib/module_utils/valid_deploy_id.py b/build/lib/module_utils/valid_deploy_id.py index 29ea8daf..a7ce24da 100644 --- a/build/lib/module_utils/valid_deploy_id.py +++ b/build/lib/module_utils/valid_deploy_id.py @@ -45,7 +45,6 @@ class ValidDeployId: Manually parse INI inventory for sections and host lists. Returns True if app_id matches a section name or a host in a section. """ - present = False with open(inventory_path, 'r', encoding='utf-8') as f: current_section = None for raw in f: diff --git a/module_utils/sounds.py b/module_utils/sounds.py index 9dd0eaec..592db9fe 100644 --- a/module_utils/sounds.py +++ b/module_utils/sounds.py @@ -175,7 +175,6 @@ try: def play_infinito_intro_sound(cls) -> None: build_time = 10.0 celebr_time = 12.0 - breakdown_time = 10.0 overlap = 3.0 bass_seg = 0.125 diff --git a/module_utils/valid_deploy_id.py b/module_utils/valid_deploy_id.py index 29ea8daf..a7ce24da 100644 --- a/module_utils/valid_deploy_id.py +++ b/module_utils/valid_deploy_id.py @@ -45,7 +45,6 @@ class ValidDeployId: Manually parse INI inventory for sections and host lists. Returns True if app_id matches a section name or a host in a section. """ - present = False with open(inventory_path, 'r', encoding='utf-8') as f: current_section = None for raw in f: diff --git a/tests/integration/test_docker_images_configuration.py b/tests/integration/test_docker_images_configuration.py index 3649f491..d53a932d 100644 --- a/tests/integration/test_docker_images_configuration.py +++ b/tests/integration/test_docker_images_configuration.py @@ -24,7 +24,7 @@ class TestDockerRoleServicesConfiguration(unittest.TestCase): try: config = yaml.safe_load(cfg_file.read_text("utf-8")) or {} main_file = role_path / "vars" / "main.yml" - main = yaml.safe_load(main_file.read_text("utf-8")) or {} + yaml.safe_load(main_file.read_text("utf-8")) or {} except yaml.YAMLError as e: errors.append(f"{role_path.name}: YAML parse error: {e}") continue diff --git a/tests/integration/test_unnecessary_role_dependencies.py b/tests/integration/test_unnecessary_role_dependencies.py index 5882e3d6..e4cc167e 100644 --- a/tests/integration/test_unnecessary_role_dependencies.py +++ b/tests/integration/test_unnecessary_role_dependencies.py @@ -247,16 +247,12 @@ class TestUnnecessaryRoleDependencies(unittest.TestCase): # --- 2) Task-level analysis any_usage = False any_bad_order = False - any_include_somewhere = False for path, text in task_texts: if not text: continue include_off = first_include_offset_for_role(text, producer) - if include_off is not None: - any_include_somewhere = True - var_use_off = first_var_use_offset_in_text(text, provided_vars) notify_offs = find_notify_offsets_for_handlers(text, provider_handlers) @@ -277,14 +273,12 @@ class TestUnnecessaryRoleDependencies(unittest.TestCase): if not any_bad_order: if not any_usage: reason = "no variable/handler usage detected in consumer" - suggest = "remove from meta/dependencies; include only where needed (optionally guarded)." warnings.append( f"[{consumer}] meta dependency on '{producer}' appears unnecessary: {reason}." ) else: # Usage exists but guarded by include in each file reason = "all usages occur after include/import in the same task file" - suggest = "remove from meta/dependencies; keep the explicit include/import (guard with run_once_* if global)." warnings.append( f"[{consumer}] meta dependency on '{producer}' appears unnecessary: {reason}." ) diff --git a/tests/unit/filter_plugins/test_domain_mappings.py b/tests/unit/filter_plugins/test_domain_mappings.py index 3e023e84..0b0970f2 100644 --- a/tests/unit/filter_plugins/test_domain_mappings.py +++ b/tests/unit/filter_plugins/test_domain_mappings.py @@ -20,7 +20,6 @@ class TestDomainMappings(unittest.TestCase): def test_empty_domains_cfg(self): apps = {'web-app-desktop': {'domains': {}}} - default = 'desktop.example.com' expected = [] result = self.filter.domain_mappings(apps, self.primary, True) self.assertEqual(result, expected)