Fix issues identified by end user (#138)

sureshcsdp · web-flow · commit 981aee07c208 · 2026-04-11T17:24:46.000+01:00
diff --git a/cleancloud/providers/azure/rules/disk_snapshots_old.py b/cleancloud/providers/azure/rules/disk_snapshots_old.py
diff --git a/cleancloud/providers/azure/scan.py b/cleancloud/providers/azure/scan.py
@@ -22,7 +22,7 @@
 from cleancloud.providers.azure.rules.container_registry_unused import (
     find_unused_container_registries,
 )
-from cleancloud.providers.azure.rules.ebs_snapshots_old import find_old_snapshots
+from cleancloud.providers.azure.rules.disk_snapshots_old import find_old_snapshots
 from cleancloud.providers.azure.rules.lb_no_backends import find_lb_no_backends
 from cleancloud.providers.azure.rules.public_ip_unused import find_unused_public_ips
 from cleancloud.providers.azure.rules.sql_database_idle import find_idle_sql_databases
diff --git a/cleancloud/scan/command.py b/cleancloud/scan/command.py
@@ -843,10 +843,7 @@ def scan(
         click.echo()
         click.echo(f"Authentication failed — {e}")
         click.echo()
-        if provider == "gcp":
-            click.echo("Run `cleancloud doctor --provider gcp` to diagnose.")
-        else:
-            click.echo("Run `cleancloud doctor --provider azure` to diagnose.")
+        click.echo(f"Run `cleancloud doctor --provider {provider}` to diagnose.")
         sys.exit(EXIT_PERMISSION_ERROR)
 
     except botocore.exceptions.NoCredentialsError:
diff --git a/tests/cleancloud/providers/azure/test_azure_disk_snapshots_old.py b/tests/cleancloud/providers/azure/test_azure_disk_snapshots_old.py
@@ -1,7 +1,7 @@
 from datetime import datetime, timedelta, timezone
 from types import SimpleNamespace
 
-from cleancloud.providers.azure.rules.ebs_snapshots_old import find_old_snapshots
+from cleancloud.providers.azure.rules.disk_snapshots_old import find_old_snapshots
 
 
 def test_find_old_snapshots(monkeypatch):
@@ -40,7 +40,7 @@ def __init__(self, credential, subscription_id):
             self.snapshots = MockSnapshots()
 
     monkeypatch.setattr(
-        "cleancloud.providers.azure.rules.ebs_snapshots_old.ComputeManagementClient",
+        "cleancloud.providers.azure.rules.disk_snapshots_old.ComputeManagementClient",
         MockComputeClient,
     )
 
diff --git a/tests/e2e/azure/test_azure_rules_smoke.py b/tests/e2e/azure/test_azure_rules_smoke.py
@@ -9,7 +9,7 @@
 from cleancloud.providers.azure.rules.container_registry_unused import (
     find_unused_container_registries,
 )
-from cleancloud.providers.azure.rules.ebs_snapshots_old import find_old_snapshots
+from cleancloud.providers.azure.rules.disk_snapshots_old import find_old_snapshots
 from cleancloud.providers.azure.rules.lb_no_backends import find_lb_no_backends
 from cleancloud.providers.azure.rules.public_ip_unused import find_unused_public_ips
 from cleancloud.providers.azure.rules.sql_database_idle import find_idle_sql_databases

Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@`
`22`	`22`	`from cleancloud.providers.azure.rules.container_registry_unused import (`
`23`	`23`	`find_unused_container_registries,`
`24`	`24`	`)`
`25`		`-from cleancloud.providers.azure.rules.ebs_snapshots_old import find_old_snapshots`
	`25`	`+from cleancloud.providers.azure.rules.disk_snapshots_old import find_old_snapshots`
`26`	`26`	`from cleancloud.providers.azure.rules.lb_no_backends import find_lb_no_backends`
`27`	`27`	`from cleancloud.providers.azure.rules.public_ip_unused import find_unused_public_ips`
`28`	`28`	`from cleancloud.providers.azure.rules.sql_database_idle import find_idle_sql_databases`
Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@`
`9`	`9`	`from cleancloud.providers.azure.rules.container_registry_unused import (`
`10`	`10`	`find_unused_container_registries,`
`11`	`11`	`)`
`12`		`-from cleancloud.providers.azure.rules.ebs_snapshots_old import find_old_snapshots`
	`12`	`+from cleancloud.providers.azure.rules.disk_snapshots_old import find_old_snapshots`
`13`	`13`	`from cleancloud.providers.azure.rules.lb_no_backends import find_lb_no_backends`
`14`	`14`	`from cleancloud.providers.azure.rules.public_ip_unused import find_unused_public_ips`
`15`	`15`	`from cleancloud.providers.azure.rules.sql_database_idle import find_idle_sql_databases`