fix: allow CLI group help without config

Author: Saithej2k

app/connectors_service/connectors/connectors_cli.py

@@ -45,6 +45,10 @@ def load_config(ctx, config):
         raise FileNotFoundError(msg)
 
 
+def is_help_request(ctx):
+    return any(arg in ctx.help_option_names for arg in ctx.args)
+
+
 # Main group
 @click.group(
     invoke_without_command=True,
@@ -58,13 +62,15 @@ def cli(ctx, config):
     if ctx.invoked_subcommand is None:
         click.echo(ctx.get_help())
         return
+    if is_help_request(ctx):
+        return
 
     ctx.ensure_object(dict)
     try:
         ctx.obj["config"] = load_config(ctx, config)
     except FileNotFoundError as e:
         click.echo(
-            f"{e} Make sure that the config is either present at the default location ({CONFIG_FILE_PATH}) or it's passed via the '-c' or '--config' option."
+            f"{e} Run `connectors login` first, or pass a config via the '-c' or '--config' option."
         )
         ctx.exit(1)
 

app/connectors_service/tests/test_connectors_cli.py

@@ -128,6 +128,20 @@ def test_connector_help_page():
     assert "Commands:" in result.output
 
 
+@pytest.mark.parametrize(
+    "commands", [["connector", "--help"], ["index", "--help"], ["job", "--help"]]
+)
+def test_group_help_page_does_not_require_config(commands, mock_cli_config):
+    mock_cli_config.side_effect = FileNotFoundError(CONFIG_FILE_PATH)
+
+    runner = CliRunner()
+    result = runner.invoke(cli, commands)
+
+    assert result.exit_code == 0
+    assert "Usage:" in result.output
+    mock_cli_config.assert_not_called()
+
+
 @patch("connectors.cli.connector.Connector.list_connectors", AsyncMock(return_value=[]))
 def test_connector_list_no_connectors():
     runner = CliRunner()