Skip to content

fix: validate blob list limits#231

Merged
krowvin merged 1 commit into
mainfrom
fix/blob-limit-validation-193
Jul 11, 2026
Merged

fix: validate blob list limits#231
krowvin merged 1 commit into
mainfrom
fix/blob-limit-validation-193

Conversation

@krowvin

@krowvin krowvin commented Jul 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • require positive values for cwms-cli blob list --limit and --page-size
  • apply the same validation to cwms-cli clob list for the matching options
  • add CLI regression coverage for zero and negative values

Resolves #193

Validation

  • poetry run pytest tests/cli/test_blob_list_validation.py -q
  • poetry run pytest -q
  • poetry check

@krowvin krowvin marked this pull request as ready for review July 11, 2026 20:33
@krowvin krowvin requested a review from Enovotny as a code owner July 11, 2026 20:33
@krowvin krowvin merged commit 0917ce9 into main Jul 11, 2026
9 checks passed
@krowvin krowvin deleted the fix/blob-limit-validation-193 branch July 11, 2026 20:55
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.

No input validation for --limit

1 participant