Skip to content

test(e2e): add follow-user E2E test suite (16 tests)#87

Merged
aquie00t merged 2 commits intomainfrom
test/e2e-follow-user-coverage
Apr 17, 2026
Merged

test(e2e): add follow-user E2E test suite (16 tests)#87
aquie00t merged 2 commits intomainfrom
test/e2e-follow-user-coverage

Conversation

@aquie00t
Copy link
Copy Markdown
Collaborator

  • follow.test.ts: follow 200, idempotent, self-follow 400, 401 (4 tests)
  • unfollow.test.ts: unfollow 200, idempotent, self-unfollow 400, 404, 401 (5 tests)
  • get-follows.test.ts: followers shape, public isMe/isFollowing, pagination, empty + following shape, pagination, empty (7 tests)

fix(routes): add optionalAuthenticate hook to followers/following routes so isMe and isFollowing computed fields are populated when a JWT is present

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

- follow.test.ts: follow 200, idempotent, self-follow 400, 401 (4 tests)
- unfollow.test.ts: unfollow 200, idempotent, self-unfollow 400, 404, 401 (5 tests)
- get-follows.test.ts: followers shape, public isMe/isFollowing, pagination,
  empty + following shape, pagination, empty (7 tests)

fix(routes): add optionalAuthenticate hook to followers/following routes
so isMe and isFollowing computed fields are populated when a JWT is present
Comment thread tests/e2e/follow-user/get-follows.test.ts Fixed
…tion or class'

Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
@aquie00t aquie00t merged commit 8ee32c1 into main Apr 17, 2026
6 checks passed
@aquie00t aquie00t deleted the test/e2e-follow-user-coverage branch April 17, 2026 12:56
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