refactor: rename --headers in --header

Author: azmeuk

scim2_cli/__init__.py

@@ -44,18 +44,18 @@ def _is_pydantic_model(model: Any) -> TypeGuard[type[BaseModel]]:
 @click.option("--url", help="The SCIM server endpoint.", envvar="SCIM_CLI_URL")
 @click.option(
     "-h",
-    "--headers",
+    "--header",
     multiple=True,
     type=HeaderType(),
-    help="Header to pass in the HTTP requests.",
+    help="Headers to pass in the HTTP requests. Can be passed multiple times.",
     envvar="SCIM_CLI_HEADERS",
 )
 @click.pass_context
-def cli(ctx, url: str, headers: list[str]):
+def cli(ctx, url: str, header: list[str]):
     """SCIM application development CLI."""
     ctx.ensure_object(dict)
     ctx.obj["URL"] = url
-    headers_dict = split_headers(headers)
+    headers_dict = split_headers(header)
     client = Client(base_url=ctx.obj["URL"], headers=headers_dict)
     ctx.obj["client"] = SyncSCIMClient(client, resource_models=(User, Group))
     ctx.obj["client"].register_naive_resource_types()

tests/test_cli.py

@@ -23,7 +23,9 @@ def test_stdin_bad_json(runner, httpserver):
 def test_auth_headers(runner, httpserver, simple_user_payload):
     """Test passing auth bearer headers."""
     httpserver.expect_request(
-        "/Users/foobar", method="GET", headers={"Authorization": "Bearer token"}
+        "/Users/foobar",
+        method="GET",
+        headers={"Authorization": "Bearer token", "foo": "bar"},
     ).respond_with_json(
         simple_user_payload("foobar"),
         status=200,
@@ -47,8 +49,10 @@ def test_auth_headers(runner, httpserver, simple_user_payload):
         [
             "--url",
             httpserver.url_for("/"),
-            "--headers",
+            "--header",
             "Authorization: Bearer token",
+            "--header",
+            "foo: bar",
             "query",
             "user",
             "foobar",
@@ -60,7 +64,9 @@ def test_auth_headers(runner, httpserver, simple_user_payload):
 def test_env_vars(runner, httpserver, simple_user_payload):
     """Test passing host and headers with environment vars."""
     httpserver.expect_request(
-        "/Users/foobar", method="GET", headers={"Authorization": "Bearer token"}
+        "/Users/foobar",
+        method="GET",
+        headers={"Authorization": "Bearer token", "foo": "bar"},
     ).respond_with_json(
         simple_user_payload("foobar"),
         status=200,