forked from HAL9000/cleveragents-core
freemo
2d07cd5ef8
The spec (docs/reference/actor_cli.md) defines the synopsis for `agents actor add` as: agents actor add <NAME> --config <FILE> [--update] [--unsafe] [--set-default] [--option key=value] [--format FORMAT] The implementation was missing the required <NAME> positional argument, silently reading the actor name from the config file's `name` field instead. This deviates from the spec and breaks the expected CLI UX. Changes: - Add `name` as a required positional Argument to the `add` command - Update docstring to match spec synopsis exactly - The positional NAME takes precedence over any `name` field in config - Remove the now-redundant config-file name validation (name comes from CLI) - Add Behave BDD feature + steps for the NAME positional argument (TDD) - Update all existing Behave step invocations to pass NAME positional arg - Update Robot Framework helpers to pass NAME positional arg ISSUES CLOSED: #2905
23 lines
1.1 KiB
Gherkin
23 lines
1.1 KiB
Gherkin
Feature: agents actor add NAME positional argument
|
|
As a user of the CleverAgents CLI
|
|
I want to pass the actor name as a positional argument to `agents actor add`
|
|
So that the CLI matches the spec synopsis: agents actor add <NAME> --config <FILE>
|
|
|
|
Scenario: actor add accepts NAME as positional argument
|
|
Given an actor CLI runner
|
|
And I have an actor JSON config file without a name field
|
|
When I run actor add with NAME positional argument and config
|
|
Then the actor add should succeed with the positional name
|
|
|
|
Scenario: actor add NAME positional argument takes precedence over config name
|
|
Given an actor CLI runner
|
|
And I have an actor JSON config file with a different name
|
|
When I run actor add with NAME positional argument overriding config name
|
|
Then the actor add should use the positional NAME not the config name
|
|
|
|
Scenario: actor add without NAME positional argument fails
|
|
Given an actor CLI runner
|
|
And I have an actor JSON config file without a name field
|
|
When I run actor add with config but no NAME positional argument
|
|
Then the actor command should fail with missing argument error
|