Skip to content

test: add edge-case coverage for validation helper functions#1297

Closed
jyotish6699 wants to merge 1 commit into
steam-bell-92:mainfrom
jyotish6699:test-validation-edge-cases
Closed

test: add edge-case coverage for validation helper functions#1297
jyotish6699 wants to merge 1 commit into
steam-bell-92:mainfrom
jyotish6699:test-validation-edge-cases

Conversation

@jyotish6699

Copy link
Copy Markdown
Contributor

Summary

This PR adds additional edge-case coverage for validation helper functions in utils/validation.py.

Added Test Cases

  • Verify custom error_invalid handling in get_choice()
  • Verify empty input handling in get_yes_no()
  • Verify empty input handling in get_int_list()
  • Verify empty input handling in get_float_list()
  • Verify whitespace-only input handling in get_non_empty_string()

Motivation

These scenarios exercise existing validation branches that were previously not covered by automated tests. Improving coverage helps prevent regressions and ensures consistent user input handling across mini-projects that depend on these helper utilities.

Verification

  • Added new edge-case tests in tests/test_validation.py
  • All existing tests pass successfully
  • Full test suite passes locally (236 passed)

Closes #1296

@vercel

vercel Bot commented Jun 21, 2026

Copy link
Copy Markdown

@jyotish6699 is attempting to deploy a commit to the Anuj's projects Team on Vercel.

A member of the Team first needs to authorize it.

@steam-bell-92

Copy link
Copy Markdown
Owner

@jyotish6699 i've deleted all test files so this one also not merging

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.

[Enhancement] Add edge-case test coverage for validation helper functions

2 participants