Skip to content

test(e2e): add comment E2E test suite (27 tests)#86

Merged
aquie00t merged 1 commit intomainfrom
test/e2e-comment-coverage
Apr 17, 2026
Merged

test(e2e): add comment E2E test suite (27 tests)#86
aquie00t merged 1 commit intomainfrom
test/e2e-comment-coverage

Conversation

@aquie00t
Copy link
Copy Markdown
Collaborator

  • create.test.ts: top-level comment, nested reply, 404 (post/parent), 400 (cross-post parentId), 401, validation error (7 tests)
  • delete.test.ts: owner delete 204, 404, non-owner 403, 401 (4 tests)
  • interact.test.ts: like/unlike with idempotency, 404, 401 (8 tests)
  • get-comments.test.ts: list with pagination, single comment shape, replies with pagination, 404 cases (8 tests)

What does this PR do?

A brief description of the changes introduced.

Closes #


Type of Change

  • Bug fix
  • New feature
  • Refactor
  • Documentation
  • Chore

Checklist

  • My branch follows the naming convention (feature/, fix/, chore/, docs/)
  • My commits follow Conventional Commits
  • I have tested my changes locally
  • I have not introduced any breaking changes
  • I have updated relevant documentation if needed

- create.test.ts: top-level comment, nested reply, 404 (post/parent),
  400 (cross-post parentId), 401, validation error (7 tests)
- delete.test.ts: owner delete 204, 404, non-owner 403, 401 (4 tests)
- interact.test.ts: like/unlike with idempotency, 404, 401 (8 tests)
- get-comments.test.ts: list with pagination, single comment shape,
  replies with pagination, 404 cases (8 tests)
@aquie00t aquie00t merged commit e01eade into main Apr 17, 2026
6 checks passed
@aquie00t aquie00t deleted the test/e2e-comment-coverage branch April 17, 2026 11:03
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.

1 participant