aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
Diffstat (limited to 'src')
-rw-r--r--src/routes.test.ts81
-rw-r--r--src/routes.ts2
-rw-r--r--src/util/config.test.ts11
3 files changed, 57 insertions, 37 deletions
diff --git a/src/routes.test.ts b/src/routes.test.ts
index 3a6bd26..c5cc9c0 100644
--- a/src/routes.test.ts
+++ b/src/routes.test.ts
@@ -1,3 +1,8 @@
+// The environment variable HOST_DIR must be defined for the tests to
+// work. Mocking exit gives a more descriptive error.
+const exit = jest
+ .spyOn(process, "exit")
+ .mockImplementation(() => undefined as never);
import _request, { SuperTest, Test } from "supertest";
import express from "express";
import dotenv from "dotenv";
@@ -6,6 +11,18 @@ import path from "path";
dotenv.config();
+test("Environment variable HOST_DIR must be defined", () => {
+ expect(process.env.HOST_DIR).not.toBeUndefined();
+});
+test("HOST_DIR must be readable and writable", () => {
+ expect(() =>
+ fs.accessSync(
+ process.env.HOST_DIR ?? "",
+ fs.constants.W_OK | fs.constants.R_OK
+ )
+ ).not.toThrowError();
+});
+
process.env.UPLOAD_LIMIT = "40000";
import { configOrError, persistTemplate } from "./util/config";
@@ -35,7 +52,9 @@ const mock = (
createRepository: jest.fn()
});
-const request = (mockMeta: MetadataMockType = mock()): SuperTest<Test> => {
+const request = async (
+ mockMeta: MetadataMockType = mock()
+): Promise<SuperTest<Test>> => {
const app = express();
app.use(routes(mockMeta as Metadata, path.join(__dirname, "..", "public")));
@@ -61,9 +80,8 @@ describe("templates", () => {
context: { TARGET_URL }
} as Template);
- const res = await request()
- .get("/bash")
- .expect(200);
+ const res = await (await request()).get("/bash").expect(200);
+ expect(exit).not.toHaveBeenCalled();
expect(res.text).toMatch("curl -X POST");
expect(res.text).toMatch(`url="${TARGET_URL}"`);
});
@@ -83,10 +101,11 @@ describe("templates", () => {
context: { TARGET_URL }
} as Template);
- const res = await request()
+ const res = await (await request())
.get("/")
.expect("Content-Type", /html/)
.expect(200);
+ expect(exit).not.toHaveBeenCalled();
expect(res.text).toMatch(`bash &lt;(curl -s ${TARGET_URL}/bash)`);
});
});
@@ -100,7 +119,7 @@ describe("Static files", () => {
path.join(staticRoot, "favicon.ico")
);
- await request()
+ await (await request())
.get("/favicon.ico")
.expect("Content-Type", /icon/)
.expect(buffer)
@@ -112,9 +131,7 @@ describe("Static files", () => {
path.join(staticRoot, "index.css")
);
- const res = await request()
- .get("/static/index.css")
- .expect(200);
+ const res = await (await request()).get("/static/index.css").expect(200);
expect(res.text).toEqual(buffer.toString("utf-8"));
});
@@ -123,9 +140,7 @@ describe("Static files", () => {
path.join(staticRoot, "404.html")
);
- const res = await request()
- .get("/thisisnotanendpoint")
- .expect(404);
+ const res = await (await request()).get("/thisisnotanendpoint").expect(404);
expect(res.text).toEqual(buffer.toString("utf-8"));
});
});
@@ -162,7 +177,7 @@ describe("Badges and reports", () => {
describe("GET /v1/:org/:repo/:branch/:commit.html", () => {
it("should retrieve the stored report file", async () => {
- const res = await request()
+ const res = await (await request())
.get("/v1/testorg/testrepo/testbranch/testcommit.html")
.expect("Content-Type", /html/)
.expect(200);
@@ -172,7 +187,7 @@ describe("Badges and reports", () => {
});
it("should return 404 if file does not exist", async () => {
- await request()
+ await (await request())
.get("/v1/neorg/nerepo/nebranch/necommit.html")
.expect(404);
});
@@ -181,7 +196,7 @@ describe("Badges and reports", () => {
describe("GET /v1/:org/:repo/:branch.html", () => {
it("should retrieve the stored report file with the associated head commit", async () => {
const mockMeta = mock();
- const res = await request(mockMeta)
+ const res = await (await request(mockMeta))
.get("/v1/testorg/testrepo/testbranch.html")
.expect("Content-Type", /html/)
.expect(200);
@@ -192,23 +207,21 @@ describe("Badges and reports", () => {
});
it("should return 404 if file does not exist", async () => {
- await request()
- .get("/v1/neorg/nerepo/nebranch.html")
- .expect(404);
+ await (await request()).get("/v1/neorg/nerepo/nebranch.html").expect(404);
});
it("should return 404 if head commit not found", async () => {
const head = jest.fn(
() => new Promise(solv => solv(new BranchNotFoundError()))
);
- await request(mock(head))
+ await (await request(mock(head)))
.get("/v1/testorg/testrepo/testbranch.html")
.expect(404);
});
it("should return 500 if promise is rejected", async () => {
const head = jest.fn(() => new Promise((_, rej) => rej("fooey")));
- await request(mock(head))
+ await (await request(mock(head)))
.get("/v1/testorg/testrepo/testbranch.html")
.expect(500);
});
@@ -216,7 +229,7 @@ describe("Badges and reports", () => {
describe("GET /v1/:org/:repo/:branch/:commit.svg", () => {
it("should retrieve the stored report badge", async () => {
- const res = await request()
+ const res = await (await request())
.get("/v1/testorg/testrepo/testbranch/testcommit.svg")
.expect("Content-Type", /svg/)
.expect(200);
@@ -225,7 +238,7 @@ describe("Badges and reports", () => {
});
it("should return 404 if file does not exist", async () => {
- await request()
+ await (await request())
.get("/v1/neorg/nerepo/nebranch/necommit.svg")
.expect(404);
});
@@ -234,7 +247,7 @@ describe("Badges and reports", () => {
describe("GET /v1/:org/:repo/:branch.svg", () => {
it("should retrieve the stored report badge with the associated head commit", async () => {
const mockMeta = mock();
- const res = await request(mockMeta)
+ const res = await (await request(mockMeta))
.get("/v1/testorg/testrepo/testbranch.svg")
.expect("Content-Type", /svg/)
.expect(200);
@@ -244,23 +257,21 @@ describe("Badges and reports", () => {
});
it("should return 404 if file does not exist", async () => {
- await request()
- .get("/v1/neorg/nerepo/nebranch.svg")
- .expect(404);
+ await (await request()).get("/v1/neorg/nerepo/nebranch.svg").expect(404);
});
it("should return 404 if head commit not found", async () => {
const head = jest.fn(
() => new Promise(solv => solv(new BranchNotFoundError()))
);
- await request(mock(head))
+ await (await request(mock(head)))
.get("/v1/testorg/testrepo/testbranch.svg")
.expect(404);
});
it("should return 500 if promise is rejected", async () => {
const head = jest.fn(() => new Promise((_, rej) => rej("fooey")));
- await request(mock(head))
+ await (await request(mock(head)))
.get("/v1/testorg/testrepo/testbranch.svg")
.expect(500);
});
@@ -290,7 +301,7 @@ describe("Uploads", () => {
describe("POST /v1/:org/:repo/:branch/:commit.html", () => {
it("should upload the report and generate a badge", async () => {
const mockMeta = mock();
- await request(mockMeta)
+ await (await request(mockMeta))
.post(
`/v1/testorg/testrepo/newthis/newthat.html?token=${TOKEN}&format=tarpaulin`
)
@@ -315,7 +326,7 @@ describe("Uploads", () => {
});
it("should return 401 when token is not correct", async () => {
- await request()
+ await (await request())
.post(
`/v1/testorg/testrepo/newthis/newthat.html?token=wrong&format=tarpaulin`
)
@@ -324,7 +335,7 @@ describe("Uploads", () => {
});
it("should return 406 with an invalid format", async () => {
- await request()
+ await (await request())
.post(
`/v1/testorg/testrepo/newthis/newthat.html?token=${TOKEN}&format=pepperoni`
)
@@ -333,7 +344,7 @@ describe("Uploads", () => {
});
it("should return 400 when request body is not the appropriate format", async () => {
- await request()
+ await (await request())
.post(
`/v1/testorg/testrepo/newthis/newthat.html?token=${TOKEN}&format=tarpaulin`
)
@@ -344,7 +355,7 @@ describe("Uploads", () => {
it("should return 413 when request body is not the appropriate format", async () => {
const file = await data;
const bigData = Buffer.concat([file, file]);
- await request()
+ await (await request())
.post(
`/v1/testorg/testrepo/newthis/newthat.html?token=${TOKEN}&format=tarpaulin`
)
@@ -354,7 +365,7 @@ describe("Uploads", () => {
it("should return 500 when Metadata does not create branch", async () => {
const update = jest.fn(() => new Promise(solv => solv(false)));
- await request(mock(jest.fn(), update))
+ await (await request(mock(jest.fn(), update)))
.post(
`/v1/testorg/testrepo/newthis/newthat.html?token=${TOKEN}&format=tarpaulin`
)
@@ -364,7 +375,7 @@ describe("Uploads", () => {
it("should return 500 when promise chain is rejected", async () => {
const update = jest.fn(() => new Promise((_, rej) => rej("fooey 2")));
- await request(mock(jest.fn(), update))
+ await (await request(mock(jest.fn(), update)))
.post(
`/v1/testorg/testrepo/newthis/newthat.html?token=${TOKEN}&format=tarpaulin`
)
diff --git a/src/routes.ts b/src/routes.ts
index 98260fa..2b787ba 100644
--- a/src/routes.ts
+++ b/src/routes.ts
@@ -85,7 +85,7 @@ export default (metadata: Metadata, publicPath: string): Router => {
)
.then(
style =>
- new Promise(solv =>
+ new Promise<string>(solv =>
solv(
badgen({
label: "coverage",
diff --git a/src/util/config.test.ts b/src/util/config.test.ts
index a065c47..7506840 100644
--- a/src/util/config.test.ts
+++ b/src/util/config.test.ts
@@ -8,7 +8,7 @@ import {
handleStartup,
handleShutdown
} from "./config";
-import { MongoClient, MongoError } from "mongodb";
+import { MongoClient, MongoError, ReadPreference } from "mongodb";
import { Server } from "http";
import path from "path";
import fs from "fs";
@@ -42,6 +42,14 @@ const MongoMock = (p: Promise<void>): jest.Mock<MongoClient, void[]> =>
jest.fn<MongoClient, void[]>(() => ({
...CommonMocks,
close: jest.fn(() => p),
+ readPreference: {
+ mode: ReadPreference.NEAREST,
+ tags: [],
+ isValid: jest.fn(),
+ slaveOk: jest.fn(),
+ equals: jest.fn()
+ },
+ writeConcern: {},
db: jest.fn()
}));
const ServerMock = (mockErr: Error | undefined): jest.Mock<Server, void[]> =>
@@ -63,6 +71,7 @@ const ServerMock = (mockErr: Error | undefined): jest.Mock<Server, void[]> =>
address: jest.fn(),
getConnections: jest.fn(),
ref: jest.fn(),
+ requestTimeout: 3600,
unref: jest.fn()
}));
generated by cgit (git 2.34.1) at 2026-01-12 12:26:00 +0000