Skip to content

docs: remove manual undo/redo test case#341

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

docs: remove manual undo/redo test case#341
dcalhoun merged 1 commit intotrunkfrom
docs/remove-manual-undo-redo-test-case

Conversation

@dcalhoun
Copy link
Member

@dcalhoun dcalhoun commented Feb 27, 2026

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
- **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
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
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.

Copy link
Contributor

@nbradbury nbradbury left a comment

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