refactor: move the CLI in its own file

Author: azmeuk

scim2_tester/__init__.py

@@ -1,10 +1,10 @@
-from .checker import check_server
-from .discovery import get_all_available_tags
-from .discovery import get_standard_resource_types
-from .utils import CheckConfig
-from .utils import CheckResult
-from .utils import SCIMTesterError
-from .utils import Status
+from scim2_tester.checker import check_server
+from scim2_tester.discovery import get_all_available_tags
+from scim2_tester.discovery import get_standard_resource_types
+from scim2_tester.utils import CheckConfig
+from scim2_tester.utils import CheckResult
+from scim2_tester.utils import SCIMTesterError
+from scim2_tester.utils import Status
 
 __all__ = [
     "check_server",
@@ -15,3 +15,8 @@
     "get_all_available_tags",
     "get_standard_resource_types",
 ]
+
+if __name__ == "__main__":  # pragma: no cover
+    from scim2_tester.cli import cli
+
+    cli()

scim2_tester/checker.py

@@ -1,5 +1,3 @@
-import argparse
-
 from scim2_client.engines.httpx import SyncSCIMClient
 
 from scim2_tester.checkers import random_url
@@ -117,61 +115,3 @@ def check_server(
         results.extend(resource_results)
 
     return results
-
-
-if __name__ == "__main__":
-    from httpx import Client
-    from scim2_client.engines.httpx import SyncSCIMClient
-
-    parser = argparse.ArgumentParser(description="SCIM server compliance checker.")
-    parser.add_argument("host")
-    parser.add_argument("--token", required=False)
-    parser.add_argument("--verbose", required=False, action="store_true")
-    parser.add_argument(
-        "--include-tags",
-        nargs="+",
-        help="Run only checks with these tags",
-        required=False,
-    )
-    parser.add_argument(
-        "--exclude-tags",
-        nargs="+",
-        help="Skip checks with these tags",
-        required=False,
-    )
-    parser.add_argument(
-        "--resource-types",
-        nargs="+",
-        help="Filter by resource type names",
-        required=False,
-    )
-    args = parser.parse_args()
-
-    client = Client(
-        base_url=args.host,
-        headers={"Authorization": f"Bearer {args.token}"} if args.token else None,
-    )
-    scim = SyncSCIMClient(client)
-    scim.discover()  # type: ignore[no-untyped-call]
-
-    include_tags: set[str] | None = (
-        set(args.include_tags) if args.include_tags else None
-    )
-    exclude_tags: set[str] | None = (
-        set(args.exclude_tags) if args.exclude_tags else None
-    )
-
-    results = check_server(
-        scim,
-        include_tags=include_tags,
-        exclude_tags=exclude_tags,
-        resource_types=args.resource_types,
-    )
-
-    for result in results:
-        resource_info = f" [{result.resource_type}]" if result.resource_type else ""
-        print(f"{result.status.name} {result.title}{resource_info}")
-        if result.reason:
-            print("  ", result.reason)
-            if args.verbose and result.data:
-                print("  ", result.data)

scim2_tester/cli.py

@@ -0,0 +1,61 @@
+import argparse
+
+from httpx import Client
+from scim2_client.engines.httpx import SyncSCIMClient
+
+from scim2_tester.checker import check_server
+
+
+def cli():
+    parser = argparse.ArgumentParser(description="SCIM server compliance checker.")
+    parser.add_argument("host")
+    parser.add_argument("--token", required=False)
+    parser.add_argument("--verbose", required=False, action="store_true")
+    parser.add_argument(
+        "--include-tags",
+        nargs="+",
+        help="Run only checks with these tags",
+        required=False,
+    )
+    parser.add_argument(
+        "--exclude-tags",
+        nargs="+",
+        help="Skip checks with these tags",
+        required=False,
+    )
+    parser.add_argument(
+        "--resource-types",
+        nargs="+",
+        help="Filter by resource type names",
+        required=False,
+    )
+    args = parser.parse_args()
+
+    client = Client(
+        base_url=args.host,
+        headers={"Authorization": f"Bearer {args.token}"} if args.token else None,
+    )
+    scim = SyncSCIMClient(client)
+    scim.discover()  # type: ignore[no-untyped-call]
+
+    include_tags: set[str] | None = (
+        set(args.include_tags) if args.include_tags else None
+    )
+    exclude_tags: set[str] | None = (
+        set(args.exclude_tags) if args.exclude_tags else None
+    )
+
+    results = check_server(
+        scim,
+        include_tags=include_tags,
+        exclude_tags=exclude_tags,
+        resource_types=args.resource_types,
+    )
+
+    for result in results:
+        resource_info = f" [{result.resource_type}]" if result.resource_type else ""
+        print(f"{result.status.name} {result.title}{resource_info}")
+        if result.reason:
+            print("  ", result.reason)
+            if args.verbose and result.data:
+                print("  ", result.data)