Extend office e2e coverage to .vsdx, all-island uploads, delete, and .txt/.csv smoke #148

New issue

Open

opened 2026-04-26 07:47:40 +00:00 by zaelgohary · 0 comments

zaelgohary commented

2026-04-26 07:47:40 +00:00

Member

tests/e2e/office_editor_real.spec.ts covers xlsx/docx/pptx/pdf; office_upload.spec.ts only exercises xlsx via Sheets. Several declared paths are untested.

Acceptance

Add a Hello.vsdx seed for the Diagrams island (DocType::Diagram); extend the editor-loads spec to cover it. All five doc types in DocType::ALL should round-trip through editor open.
Extend the upload spec to PUT through Documents / Presentations / PDF / Diagrams Upload buttons, not just Sheets.
Add a delete spec that uses the FileRow Delete button and asserts the row disappears (service::delete_file ships untested).
Add a smoke test that opens a fresh test.txt and asserts the OnlyOffice encoding dialog appears (= editor functioning); guards against the cache-poisoning regression class.
Add one large-file case (>16 MB) flowing through the office Upload button so the streaming path through hero_router is exercised end-to-end via the UI, not just via curl.

`tests/e2e/office_editor_real.spec.ts` covers xlsx/docx/pptx/pdf; `office_upload.spec.ts` only exercises xlsx via Sheets. Several declared paths are untested. ## Acceptance - Add a `Hello.vsdx` seed for the Diagrams island (`DocType::Diagram`); extend the editor-loads spec to cover it. All five doc types in `DocType::ALL` should round-trip through editor open. - Extend the upload spec to PUT through Documents / Presentations / PDF / Diagrams Upload buttons, not just Sheets. - Add a delete spec that uses the FileRow Delete button and asserts the row disappears (`service::delete_file` ships untested). - Add a smoke test that opens a fresh `test.txt` and asserts the OnlyOffice encoding dialog appears (= editor functioning); guards against the cache-poisoning regression class. - Add one large-file case (>16 MB) flowing through the office Upload button so the streaming path through hero_router is exercised end-to-end via the UI, not just via curl.