Jean-Luc Makiola
be168c8329
Some checks failed
CI / ci (push) Failing after 25s
Use Bun.serve to create a local test server instead of hitting external URLs. Fixes flaky test in CI environments without network access. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
90 lines
2.6 KiB
TypeScript
90 lines
2.6 KiB
TypeScript
import { afterAll, afterEach, beforeAll, describe, expect, it } from "bun:test";
|
|
import { existsSync, rmSync } from "node:fs";
|
|
import type { Server } from "bun";
|
|
import { fetchImageFromUrl } from "../../src/server/services/image.service.ts";
|
|
|
|
const TEST_UPLOADS_DIR = "test-uploads";
|
|
|
|
// 1x1 transparent PNG (smallest valid PNG)
|
|
const TINY_PNG = new Uint8Array([
|
|
0x89, 0x50, 0x4e, 0x47, 0x0d, 0x0a, 0x1a, 0x0a, 0x00, 0x00, 0x00, 0x0d, 0x49,
|
|
0x48, 0x44, 0x52, 0x00, 0x00, 0x00, 0x01, 0x00, 0x00, 0x00, 0x01, 0x08, 0x06,
|
|
0x00, 0x00, 0x00, 0x1f, 0x15, 0xc4, 0x89, 0x00, 0x00, 0x00, 0x0a, 0x49, 0x44,
|
|
0x41, 0x54, 0x78, 0x9c, 0x62, 0x00, 0x00, 0x00, 0x02, 0x00, 0x01, 0xe2, 0x21,
|
|
0xbc, 0x33, 0x00, 0x00, 0x00, 0x00, 0x49, 0x45, 0x4e, 0x44, 0xae, 0x42, 0x60,
|
|
0x82,
|
|
]);
|
|
|
|
let server: Server;
|
|
let baseUrl: string;
|
|
|
|
beforeAll(() => {
|
|
server = Bun.serve({
|
|
port: 0,
|
|
fetch(req) {
|
|
const url = new URL(req.url);
|
|
if (url.pathname === "/image.png") {
|
|
return new Response(TINY_PNG, {
|
|
headers: { "Content-Type": "image/png" },
|
|
});
|
|
}
|
|
if (url.pathname === "/page.html") {
|
|
return new Response("<html></html>", {
|
|
headers: { "Content-Type": "text/html" },
|
|
});
|
|
}
|
|
return new Response("Not found", { status: 404 });
|
|
},
|
|
});
|
|
baseUrl = `http://localhost:${server.port}`;
|
|
});
|
|
|
|
afterAll(() => {
|
|
server.stop(true);
|
|
});
|
|
|
|
describe("Image Service", () => {
|
|
afterEach(() => {
|
|
if (existsSync(TEST_UPLOADS_DIR)) {
|
|
rmSync(TEST_UPLOADS_DIR, { recursive: true });
|
|
}
|
|
});
|
|
|
|
describe("fetchImageFromUrl", () => {
|
|
it("fetches a valid image URL and saves to disk", async () => {
|
|
const imageUrl = `${baseUrl}/image.png`;
|
|
const result = await fetchImageFromUrl(imageUrl, TEST_UPLOADS_DIR);
|
|
|
|
expect(result.filename).toMatch(/^\d+-[\w-]+\.png$/);
|
|
expect(result.sourceUrl).toBe(imageUrl);
|
|
|
|
const filePath = `${TEST_UPLOADS_DIR}/${result.filename}`;
|
|
expect(existsSync(filePath)).toBe(true);
|
|
});
|
|
|
|
it("rejects non-image content type", async () => {
|
|
await expect(
|
|
fetchImageFromUrl(`${baseUrl}/page.html`, TEST_UPLOADS_DIR),
|
|
).rejects.toThrow("Invalid content type");
|
|
});
|
|
|
|
it("rejects invalid URL format", async () => {
|
|
await expect(
|
|
fetchImageFromUrl("not-a-url", TEST_UPLOADS_DIR),
|
|
).rejects.toThrow("Invalid URL format");
|
|
});
|
|
|
|
it("rejects non-HTTP protocols", async () => {
|
|
await expect(
|
|
fetchImageFromUrl("ftp://example.com/image.png", TEST_UPLOADS_DIR),
|
|
).rejects.toThrow("URL must use HTTP or HTTPS");
|
|
});
|
|
|
|
it("rejects 404 responses", async () => {
|
|
await expect(
|
|
fetchImageFromUrl(`${baseUrl}/missing.jpg`, TEST_UPLOADS_DIR),
|
|
).rejects.toThrow("HTTP 404");
|
|
});
|
|
});
|
|
});
|