Skip to content

Add Excel/document preprocessing coverage and UI error-contract tests#43

Merged
rad1092 merged 1 commit into
mainfrom
codex/update-tests-and-readme-for-csv/excel-validation
Feb 15, 2026
Merged

Add Excel/document preprocessing coverage and UI error-contract tests#43
rad1092 merged 1 commit into
mainfrom
codex/update-tests-and-readme-for-csv/excel-validation

Conversation

@rad1092
Copy link
Copy Markdown
Owner

@rad1092 rad1092 commented Feb 15, 2026

Motivation

  • Validate end-to-end handling of CSV/Excel/document inputs (single/multi-sheet, empty sheets, header validation) and ensure document extraction fallback and API/UI error contract are consistent.
  • Make supported input scope, constraints and recommended file preparation explicit in the README for users uploading CSV/Excel/document files.

Description

  • Add comprehensive web-layer tests in tests/test_web.py including an XLSX generator, a local HTTP server helper, and tests for Excel single-sheet parsing, multi-sheet selection, empty-sheet validation, header empty/duplicate checks, document extract success/failure payloads, and /api/analyze fallback error contract handling.
  • Add tests/test_ui_contract.py to statically verify UI-side error detail priority (error_detail first) and presence of key failure status messages used by the frontend.
  • Update README.md with an "업로드 입력 지원 범위 / 제약 / 준비 권장사항" section describing supported formats, limitations (e.g. only .xlsx), error behaviors, and recommended file preparation steps.

Testing

  • Ran pytest -q tests/test_web.py tests/test_document_extract.py tests/test_ui_contract.py and all tests passed: 16 passed in 1.21s.
  • Changes are tests and documentation only; no production runtime behavior was modified by this PR.

Codex Task

@rad1092 rad1092 merged commit ef2aca1 into main Feb 15, 2026
4 checks passed
@rad1092 rad1092 deleted the codex/update-tests-and-readme-for-csv/excel-validation branch February 15, 2026 04:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant