test: update

Author: danielroe

test/unit/server/utils/gravatar.spec.ts

@@ -1,4 +1,3 @@
-import { Buffer } from 'node:buffer'
 import { createHash } from 'node:crypto'
 import { beforeEach, describe, expect, it, vi } from 'vitest'
 
@@ -9,66 +8,36 @@ vi.mock('#server/utils/npm', () => ({
 const { getGravatarFromUsername } = await import('../../../../server/utils/gravatar')
 const { fetchUserEmail } = await import('#server/utils/npm')
 
-const mockFetch = vi.fn()
-
 describe('gravatar utils', () => {
   beforeEach(() => {
     vi.clearAllMocks()
-    mockFetch.mockReset()
-    vi.stubGlobal('fetch', mockFetch)
   })
 
   it('returns null when username is empty', async () => {
-    const url = await getGravatarFromUsername('')
+    const hash = await getGravatarFromUsername('')
 
-    expect(url).toBeNull()
+    expect(hash).toBeNull()
     expect(fetchUserEmail).not.toHaveBeenCalled()
   })
 
   it('returns null when email is not available', async () => {
     vi.mocked(fetchUserEmail).mockResolvedValue(null)
 
-    const url = await getGravatarFromUsername('user')
+    const hash = await getGravatarFromUsername('user')
 
-    expect(url).toBeNull()
+    expect(hash).toBeNull()
     expect(fetchUserEmail).toHaveBeenCalledOnce()
   })
 
-  it('builds a gravatar data URL with a trimmed, lowercased email hash', async () => {
+  it('returns md5 hash of trimmed, lowercased email', async () => {
     const email = ' Test@Example.com '
     const normalized = 'test@example.com'
-    const hash = createHash('md5').update(normalized).digest('hex')
-    const imageBytes = new Uint8Array([1, 2, 3])
-    const base64 = Buffer.from(imageBytes).toString('base64')
-    vi.mocked(fetchUserEmail).mockResolvedValue(email)
-    mockFetch.mockResolvedValue({
-      ok: true,
-      headers: { get: () => 'image/png' },
-      arrayBuffer: vi.fn().mockResolvedValue(imageBytes.buffer),
-    })
-
-    const url = await getGravatarFromUsername('user')
-
-    expect(url).toBe(`data:image/png;base64,${base64}`)
-    expect(mockFetch).toHaveBeenCalledWith(`https://www.gravatar.com/avatar/${hash}?s=80&d=404`)
-  })
-
-  it('supports custom size', async () => {
-    const email = 'user@example.com'
-    const hash = createHash('md5').update(email).digest('hex')
-    const imageBytes = new Uint8Array([4, 5, 6])
-    const base64 = Buffer.from(imageBytes).toString('base64')
+    const expectedHash = createHash('md5').update(normalized).digest('hex')
     vi.mocked(fetchUserEmail).mockResolvedValue(email)
-    mockFetch.mockResolvedValue({
-      ok: true,
-      headers: { get: () => 'image/png' },
-      arrayBuffer: vi.fn().mockResolvedValue(imageBytes.buffer),
-    })
 
-    const url = await getGravatarFromUsername('user', 128)
+    const hash = await getGravatarFromUsername('user')
 
-    expect(url).toBe(`data:image/png;base64,${base64}`)
-    expect(mockFetch).toHaveBeenCalledWith(`https://www.gravatar.com/avatar/${hash}?s=128&d=404`)
+    expect(hash).toBe(expectedHash)
   })
 
   it('trims the username before lookup', async () => {