fix(cli): add -u short form to lsp add --update #1262

Merged
freemo merged 1 commits from fix/lsp-add-update-short-form into master 2026-04-02 16:58:33 +00:00
3 changed files with 16 additions and 0 deletions
+8
View File
@@ -39,6 +39,14 @@ Feature: LSP CLI commands coverage
Then the lsp CLI command should succeed
And the lsp CLI output should contain "LSP Server Registered"
@lsp_cli_add
Scenario: Add LSP server with -u short form overwrites existing
Given a valid LSP server YAML config file
And the LSP server has been registered via CLI
When I run lsp CLI add with --config and -u short form
Then the lsp CLI command should succeed
And the lsp CLI output should contain "LSP Server Registered"
@lsp_cli_add
Scenario: Add LSP server with --format json
Given a valid LSP server YAML config file
@@ -218,6 +218,13 @@ def step_run_lsp_add_update(context: Context) -> None:
)
@when("I run lsp CLI add with --config and -u short form")
def step_run_lsp_add_u_short(context: Context) -> None:
context.lsp_cli_result = _runner.invoke(
app, ["add", "--config", context.lsp_yaml_path, "-u"]
)
@when("I run lsp CLI add with --config and --format json")
def step_run_lsp_add_json(context: Context) -> None:
context.lsp_cli_result = _runner.invoke(
+1
View File
@@ -122,6 +122,7 @@ def add(
bool,
typer.Option(
"--update",
"-u",
help="Allow overwriting an existing LSP server registration",
),
] = False,