added tests for Authorization

src/controllers/AuthControler.ts

@@ -38,8 +38,6 @@ class AuthControler {
         if (err) {
           return res.status(403).json("Invalid token provided");
         }
-
-        console.log("Authorization provided for ", next.name, " to app ", app);
         next();
       });
     } else {

src/index.ts

@@ -14,7 +14,7 @@ app.get("/", (_, res) => {
 });
 
 app.get("/images", imageController.getAllImages);
-app.post("/images", imageController.addImage);
+app.post("/images", authControler.authorize, imageController.addImage);
 app.post("/login", authControler.login)
 
 // Set the default port to 8080, or use the PORT environment variable

tests/app.test.ts

@@ -1,14 +1,20 @@
-import { afterEach, describe, expect, it, mock } from "bun:test";
+import { afterEach, beforeAll, describe, expect, it, mock } from "bun:test";
 import request from "supertest";
 import { app } from "../src";
 import imageService from "../src/services/ImageService";
 
 const imageServiceOriginal = imageService;
+const tok = await request(app)
+  .post("/login")
+  .send({ app: "tester", secret: "test" });
+const token = tok.body.token;
 
 afterEach(() => {
   mock.restore();
-    mock.module("../src/services/ImageService", () => ({ default: imageServiceOriginal }));
-})
+  mock.module("../src/services/ImageService", () => ({
+    default: imageServiceOriginal,
+  }));
+});
 
 describe("GET / shows all of the endpoints", async () => {
   const res = await request(app).get("/");
@@ -19,8 +25,8 @@ describe("GET / shows all of the endpoints", async () => {
 
   it("should be an array", () => {
     expect(Array.isArray(res.body.endpoints)).toBeTrue();
-    })
-})
+  });
+});
 
 describe("GET /images works properly", async () => {
   const res = await request(app).get("/images");
@@ -35,26 +41,59 @@ describe("GET /images works properly", async () => {
 });
 
 describe("POST /images works properly", () => {
+
+    it("should return 401 for unauthenticated requests", async () => {
+        const res = await request(app)
+          .post("/images")
+          .send({
+            url: "https://test.url.com/0",
+            status: "available",
+            tags: ["2girls", "touhou"],
+          });
+        expect(res.status).toBe(401);
+      });
+
+      it("should return 403 for invalid tokens", async () => {
+        const res = await request(app)
+          .post("/images")
+          .set("authorization", `Bearer token`)
+          .send({
+            url: "https://test.url.com/0",
+            status: "available",
+            tags: ["2girls", "touhou"],
+          });
+        expect(res.status).toBe(403);
+      });
+
   it("should return 201 for new image", async () => {
-        const res = await request(app).post("/images").send({
+    const res = await request(app)
+      .post("/images")
+      .set("authorization", `Bearer ${token}`)
+      .send({
         url: "https://test.url.com/1",
         status: "available",
-            tags: ["2girls", "touhou"]
+        tags: ["2girls", "touhou"],
       });
-        expect(res.status).toSatisfy(status => [201].includes(status));
+    expect(res.status).toBe(201);
   });
 
   it("should return 409 for a repeated images", async () => {
-        await request(app).post("/images").send({
+    await request(app)
+      .post("/images")
+      .set("authorization", `Bearer ${token}`)
+      .send({
         url: "https://test.url.com/2",
         status: "available",
-            tags: ["2girls", "touhou"]
+        tags: ["2girls", "touhou"],
       });
 
-        const res = await request(app).post("/images").send({
+    const res = await request(app)
+      .post("/images")
+      .set("authorization", `Bearer ${token}`)
+      .send({
         url: "https://test.url.com/2",
         status: "available",
-            tags: ["2girls", "touhou"]
+        tags: ["2girls", "touhou"],
       });
 
     expect(res.status).toBe(409);
@@ -63,14 +102,19 @@ describe("POST /images works properly", () => {
   it("should return 500 for an error on the service", async () => {
     mock.module("../src/services/ImageService", () => ({
       default: {
-                add: () => { throw new Error("This is an expected testing error"); }
-            }
+        add: () => {
+          throw new Error("This is an expected testing error");
+        },
+      },
     }));
 
-        const res = await request(app).post("/images").send({
+    const res = await request(app)
+      .post("/images")
+      .set("authorization", `Bearer ${token}`)
+      .send({
         url: "https://test.url.com/3",
         status: "available",
-            tags: ["2girls", "touhou"]
+        tags: ["2girls", "touhou"],
       });
 
     expect(res.status).toBe(500);
@@ -78,10 +122,13 @@ describe("POST /images works properly", () => {
 
   it("should return 400 for malformed requests", async () => {
     mock.restore();
-        const res = await request(app).post("/images").send({
+    const res = await request(app)
+      .post("/images")
+      .set("authorization", `Bearer ${token}`)
+      .send({
         url: "https://test.url.com/4",
         status: "wrong",
-            tags: ["2girls", "touhou"]
+        tags: ["2girls", "touhou"],
       });
     expect(res.status).toBe(400);
   });

tests/auth.test.ts

@@ -0,0 +1,22 @@
+import { describe, expect, it, mock } from "bun:test";
+import request from "supertest";
+import { app } from "../src";
+
+describe("/login", async () => {
+  const correctRespose = await request(app).post("/login").send({
+    app: "tester",
+    secret: "test",
+  });
+  it("should return 200 for correct login", () => {
+    expect(correctRespose.status).toBe(200);
+  });
+  it("should contain a token", () => {
+    expect(correctRespose.body).toHaveProperty("token");
+  });
+
+  it("should return 403 for invalid credentials", async () => {
+    const res = await request(app).post("/login").send({});
+    expect(res.status).toBe(403);
+  });
+});
+