Skip to content

Handle invalid list sort parameters#223

Open
Rohithmatham12 wants to merge 1 commit into
cortexapps:mainfrom
Rohithmatham12:fix-invalid-list-sort-parameter
Open

Handle invalid list sort parameters#223
Rohithmatham12 wants to merge 1 commit into
cortexapps:mainfrom
Rohithmatham12:fix-invalid-list-sort-parameter

Conversation

@Rohithmatham12

Copy link
Copy Markdown

Summary

  • validate list sort paths before sorting table/CSV output
  • return a Typer BadParameter when the requested path does not match returned rows
  • add regression coverage for invalid sort-path casing

Fixes #137.

Testing

  • POETRY_CACHE_DIR=/private/tmp/cortexapps-cli-poetry-cache poetry run pytest tests/test_utils.py -q

Signed-off-by: Rohithmatham12 <rohithmatham@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Need graceful exit when invalid list parameters are provided

1 participant