Skip to content

Commit 9029cab

Browse files
committed
fix tests
1 parent a69e963 commit 9029cab

File tree

3 files changed

+30
-20
lines changed

3 files changed

+30
-20
lines changed

app/routes/resources+/delete-image.test.tsx

Lines changed: 25 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
import { faker } from '@faker-js/faker'
55
import fs from 'fs'
66
import { createPassword, createUser } from 'tests/db-utils.ts'
7-
import { BASE_URL, getUserSetCookieHeader } from 'tests/vitest-utils.ts'
7+
import { BASE_URL, getSessionSetCookieHeader } from 'tests/vitest-utils.ts'
88
import invariant from 'tiny-invariant'
99
import { expect, test } from 'vitest'
1010
import { prisma } from '~/utils/db.server.ts'
@@ -14,31 +14,42 @@ const RESOURCE_URL = `${BASE_URL}${ROUTE_PATH}`
1414

1515
async function setupUser() {
1616
const userData = createUser()
17-
const user = await prisma.user.create({
17+
const session = await prisma.session.create({
1818
data: {
19-
...userData,
20-
password: {
21-
create: createPassword(userData.username),
22-
},
23-
image: {
19+
expirationDate: new Date(Date.now() + 1000 * 60 * 60 * 24 * 30),
20+
user: {
2421
create: {
25-
contentType: 'image/jpeg',
26-
file: {
22+
...userData,
23+
password: {
24+
create: createPassword(userData.username),
25+
},
26+
image: {
2727
create: {
28-
blob: await fs.promises.readFile(
29-
'./tests/fixtures/test-profile.jpg',
30-
),
28+
contentType: 'image/jpeg',
29+
file: {
30+
create: {
31+
blob: await fs.promises.readFile(
32+
'./tests/fixtures/test-profile.jpg',
33+
),
34+
},
35+
},
3136
},
3237
},
3338
},
3439
},
3540
},
36-
select: { id: true, imageId: true },
41+
select: {
42+
id: true,
43+
user: {
44+
select: { id: true, imageId: true },
45+
},
46+
},
3747
})
48+
const { user } = session
3849
invariant(user.imageId, 'User should have an image')
3950
return {
4051
user: { ...user, imageId: user.imageId },
41-
cookie: await getUserSetCookieHeader(user),
52+
cookie: await getSessionSetCookieHeader(session),
4253
}
4354
}
4455

app/utils/auth.server.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,6 @@ export async function requireUserId(
6868
if (!session) {
6969
throw redirect(failureRedirect)
7070
}
71-
console.log(session.expirationDate, session.expirationDate > new Date())
7271
return session.userId
7372
}
7473

tests/vitest-utils.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3,12 +3,12 @@ import { commitSession, getSession } from '~/utils/session.server.ts'
33

44
export const BASE_URL = 'https://epicstack.dev'
55

6-
export async function getUserSetCookieHeader(
7-
user: { id: string },
6+
export async function getSessionSetCookieHeader(
7+
session: { id: string },
88
existingCookie?: string,
99
) {
10-
const session = await getSession(existingCookie)
11-
session.set(authenticator.sessionKey, user.id)
12-
const setCookieHeader = await commitSession(session)
10+
const cookieSession = await getSession(existingCookie)
11+
cookieSession.set(authenticator.sessionKey, session.id)
12+
const setCookieHeader = await commitSession(cookieSession)
1313
return setCookieHeader
1414
}

0 commit comments

Comments
 (0)