Merge pull request #62 from reynaldichernando/test-error-module

Add test error module

Author: bitsnaps

src/modules/ErrorModule.js

@@ -1,11 +1,11 @@
-const ERROR_BUFFER_LIMIT = 20;
+export const ERROR_BUFFER_LIMIT = 20;
 
-const errors = [];
+export const errors = [];
 
 export const report = (error) => {
   errors.push(error);
   if (errors.length > ERROR_BUFFER_LIMIT) {
-    errors = errors.slice(errors.length - ERROR_BUFFER_LIMIT);
+    errors.splice(0, errors.length - ERROR_BUFFER_LIMIT)
   }
 }
 export const showLast = () => {

tests/ErrorModule.test.js

@@ -0,0 +1,42 @@
+import { it, describe, expect, vi, beforeEach } from "vitest";
+
+vi.spyOn(console, "log").mockImplementation(() => { });
+vi.spyOn(console, "error").mockImplementation(() => { });
+
+let errors, report, ERROR_BUFFER_LIMIT, showLast;
+
+beforeEach(async () => {
+  vi.resetModules();
+  const module = await import("../src/modules/ErrorModule");
+  errors = module.errors;
+  report = module.report;
+  ERROR_BUFFER_LIMIT = module.ERROR_BUFFER_LIMIT;
+  showLast = module.showLast;
+});
+
+describe("report", () => {
+  it("should be able to report error", () => {
+    report("hehe")
+    expect(errors).toHaveLength(1)
+  })
+
+  it("should not exceed error buffer limit", () => {
+    for (let i = 0; i < 100; i++) {
+      report(`error ${i}`)
+    }
+    expect(errors.length).lessThanOrEqual(ERROR_BUFFER_LIMIT);
+  })
+})
+
+describe("showLast", () => {
+  it("should not log error if no error exists", () => {
+    showLast();
+    expect(console.log).toHaveBeenCalledWith(expect.stringContaining("No errors to report"));
+  })
+
+  it("should log error if error exists", () => {
+    report("hehe")
+    showLast();
+    expect(console.error).toHaveBeenCalledWith(expect.stringContaining("hehe"));
+  })
+})