fix: tests

Add mock `logger` instance to all tests that use context.
lidofinance · Jan 19, 2025 · 0bb160b · 0bb160b
1 parent af8411e
commit 0bb160b
Show file tree

Hide file tree

Showing 2 changed files with 41 additions and 3 deletions.
diff --git a/__tests__/handlers_alternative.test.js b/__tests__/handlers_alternative.test.js
@@ -17,6 +17,11 @@ test("hook works", async () => {
   // mock
   const ctx = {
     routes: { test: "/dev/null" },
+    logger: {
+      info: jest.fn(),
+      warn: jest.fn(),
+      error: jest.fn(),
+    },
     params: { slug: "test" },
     request: {
       body: {
@@ -34,7 +39,7 @@ test("hook works", async () => {
   expect(axios.post.mock.calls).toMatchSnapshot();
 });
 
-test("balval features is correct", async () => {
+test("balval features are correct", async () => {
   const getAlert = (i) => {
     return {
       status: i % 2 === 0 ? "resolved" : "fired", // resolved should be first
@@ -53,6 +58,11 @@ test("balval features is correct", async () => {
   // mock
   const ctx = {
     routes: { test: "/dev/null" },
+    logger: {
+      info: jest.fn(),
+      warn: jest.fn(),
+      error: jest.fn(),
+    },
     params: { slug: "test" },
     request: {
       body: {
@@ -87,6 +97,11 @@ test("mention works", async () => {
   // mock
   const ctx = {
     routes: { test: "/dev/null" },
+    logger: {
+      info: jest.fn(),
+      warn: jest.fn(),
+      error: jest.fn(),
+    },
     params: { slug: "test" },
     request: {
       body: {

diff --git a/__tests__/handlers_default.test.js b/__tests__/handlers_default.test.js
@@ -18,6 +18,11 @@ test("hook works (no mentions)", async () => {
   // mock
   const ctx = {
     routes: { test: "/dev/null" },
+    logger: {
+      info: jest.fn(),
+      warn: jest.fn(),
+      error: jest.fn(),
+    },
     params: { slug: "test" },
     query: { q: "42" },
 
@@ -41,6 +46,11 @@ test("hook works (missing annotations subfield)", async () => {
   // mock
   const ctx = {
     routes: { test: "/dev/null" },
+    logger: {
+      info: jest.fn(),
+      warn: jest.fn(),
+      error: jest.fn(),
+    },
     params: { slug: "test" },
     query: {},
 
@@ -75,11 +85,15 @@ test("hook works (missing annotations subfield)", async () => {
   expect(axios.post.mock.calls).toMatchSnapshot();
 });
 
-
 test("hook works (mentions)", async () => {
   // mock
   const ctx = {
     routes: { test: "/dev/null" },
+    logger: {
+      info: jest.fn(),
+      warn: jest.fn(),
+      error: jest.fn(),
+    },
     params: { slug: "test" },
     query: {},
 
@@ -107,6 +121,11 @@ test("hook works (fields)", async () => {
   // mock
   const ctx = {
     routes: { test: "/dev/null" },
+    logger: {
+      info: jest.fn(),
+      warn: jest.fn(),
+      error: jest.fn(),
+    },
     params: { slug: "test" },
     query: {},
 
@@ -134,11 +153,15 @@ test("hook works (fields)", async () => {
   expect(axios.post.mock.calls).toMatchSnapshot();
 });
 
-
 test("healthcheck works", async () => {
   // mock
   const ctx = {
     routes: { test: "/dev/null" },
+    logger: {
+      info: jest.fn(),
+      warn: jest.fn(),
+      error: jest.fn(),
+    },
     params: { slug: "test" },
     query: {},
   };