Skip to content

docs: remove manual undo/redo test case - #341

Merged
dcalhoun merged 1 commit into
trunkfrom
docs/remove-manual-undo-redo-test-case
Feb 27, 2026
Merged

dcalhoun merged 1 commit into
trunkfrom
docs/remove-manual-undo-redo-test-case

Conversation

@dcalhoun

@dcalhoun dcalhoun commented Feb 27, 2026 •

Copy link
Copy Markdown
Member

What?

Removes the manual undo/redo test case from the smoke test cases documentation and renumbers the remaining test cases.

Why?

The undo/redo functionality is now covered by automated E2E tests, making the manual test case redundant. Removing it keeps the manual test cases focused on scenarios that require human verification.

How?

  • Removed the "S.1. Undo/Redo Actions" section from docs/test-cases.md
  • Renumbered the remaining smoke test cases (S.2 → S.1, S.3 → S.2, S.4 → S.3)

Testing Instructions

  1. Review the changes in docs/test-cases.md to verify the removed test case and correct renumbering.

@dcalhoun dcalhoun added the [Type] Developer Documentation Documentation for developers label Feb 27, 2026
@dcalhoun
dcalhoun marked this pull request as ready for review February 27, 2026 15:34
@dcalhoun
dcalhoun requested a review from nbradbury February 27, 2026 15:34
@dcalhoun dcalhoun changed the title docs: Remove manual undo/redo test case docs: remove manual undo/redo test case Feb 27, 2026
This test case is now covered with automated E2E testing:

- 348ad35
- 0e2c493
@dcalhoun
dcalhoun force-pushed the docs/remove-manual-undo-redo-test-case branch from e971abc to 608fd5c Compare February 27, 2026 15:35
@dcalhoun
dcalhoun enabled auto-merge (squash) February 27, 2026 15:36
Comment thread docs/test-cases.md
- **Expected Outcome:** Image uploads and displays in the block. An activity indicator is shown while the image is uploading.

### S.3. Upload an video
### S.2. Upload an video

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor nit unrelated to this PR, but this should be "a video" instead of "an video."

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch. It is removed entirely in #329. I'll defer to that resolution.

@nbradbury nbradbury left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! :shipit:

@dcalhoun
dcalhoun merged commit 6177c9c into trunk Feb 27, 2026
14 checks passed
@dcalhoun
dcalhoun deleted the docs/remove-manual-undo-redo-test-case branch February 27, 2026 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Type] Developer Documentation Documentation for developers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants