Skip to content

test(auth): enable token refresh integration tests #42

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
kelleyglenn merged 1 commit into from
Feb 24, 2026
Merged

test(auth): enable token refresh integration tests #42

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 3 additions & 9 deletions api/tests/user-service.spec.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -155,9 +155,7 @@
});

test.describe("Token Refresh", () => {
// TODO: Token refresh endpoint not yet implemented
// See: https://github.com/kelleyglenn/AcctAtlas-user-service/issues/22
test.skip("refreshes tokens with valid refresh token", async ({
test("refreshes tokens with valid refresh token", async ({
request,
}) => {
// Register to get initial tokens
Expand Down Expand Up @@ -189,9 +187,7 @@
expect(body.tokens.refreshToken).not.toBe(refreshToken);
});

// TODO: Token refresh endpoint not yet implemented
// See: https://github.com/kelleyglenn/AcctAtlas-user-service/issues/22
test.skip("rejects invalid refresh token", async ({ request }) => {
test("rejects invalid refresh token", async ({ request }) => {
const response = await request.post(`${API_URL}/auth/refresh`, {
data: {
refreshToken: "invalid-refresh-token",
Expand All @@ -201,9 +197,7 @@
expect(response.status()).toBe(401);
});

// TODO: Token refresh endpoint not yet implemented
// See: https://github.com/kelleyglenn/AcctAtlas-user-service/issues/22
test.skip("rejects reused refresh token", async ({ request }) => {
test("rejects reused refresh token", async ({ request }) => {
// Register to get initial tokens
const userData = {
email: `reuse-test-${Date.now()}@example.com`,
Expand Down Expand Up @@ -550,7 +544,7 @@
test.describe("Password Reset", () => {
// TODO: Password reset endpoint not yet implemented
// See: https://github.com/kelleyglenn/AcctAtlas-user-service/issues/25
test.skip("accepts password reset request", async ({ request }) => {

Check warning on line 547 in api/tests/user-service.spec.ts

View workflow job for this annotation

GitHub Actions / integration 

Unexpected use of the `.skip()` annotation
const user = await createTestUser(request);

const response = await request.post(`${API_URL}/auth/password/reset`, {
Expand All @@ -565,7 +559,7 @@

// TODO: Password reset endpoint not yet implemented
// See: https://github.com/kelleyglenn/AcctAtlas-user-service/issues/25
test.skip("accepts reset request for non-existent email", async ({

Check warning on line 562 in api/tests/user-service.spec.ts

View workflow job for this annotation

GitHub Actions / integration 

Unexpected use of the `.skip()` annotation
request,
}) => {
const response = await request.post(`${API_URL}/auth/password/reset`, {
Expand All @@ -580,7 +574,7 @@

// TODO: Password reset endpoint not yet implemented
// See: https://github.com/kelleyglenn/AcctAtlas-user-service/issues/25
test.skip("validates email format", async ({ request }) => {

Check warning on line 577 in api/tests/user-service.spec.ts

View workflow job for this annotation

GitHub Actions / integration 

Unexpected use of the `.skip()` annotation
const response = await request.post(`${API_URL}/auth/password/reset`, {
data: {
email: "not-an-email",
Expand All @@ -592,7 +586,7 @@

// TODO: Password reset confirm endpoint not yet implemented
// See: https://github.com/kelleyglenn/AcctAtlas-user-service/issues/25
test.skip("rejects invalid reset token", async ({ request }) => {

Check warning on line 589 in api/tests/user-service.spec.ts

View workflow job for this annotation

GitHub Actions / integration 

Unexpected use of the `.skip()` annotation
const response = await request.post(
`${API_URL}/auth/password/reset/confirm`,
{
Expand Down
Loading