use-page-errors.tests PR comment updates

Author: myty

src/hooks/use-page-errors.test.tsx

@@ -1,10 +1,17 @@
 import { renderHook, act } from "@testing-library/react-hooks";
 import { usePageErrors } from "./use-page-errors";
-import { ResultRecord } from "andculturecode-javascript-core";
+import {
+    ResultRecord,
+    ResultErrorRecord,
+} from "andculturecode-javascript-core";
 
 describe("usePageErrors", () => {
-    describe("initialized", () => {
-        test("returns empty array", async () => {
+    // -----------------------------------------------------------------------------------------
+    // #region initialization
+    // -----------------------------------------------------------------------------------------
+
+    describe("initialization", () => {
+        test("returns empty array", () => {
             // Arrange & Act
             const { result } = renderHook(() => usePageErrors());
 
@@ -13,8 +20,14 @@ describe("usePageErrors", () => {
         });
     });
 
+    // #endregion
+
+    // -----------------------------------------------------------------------------------------
+    // #region setPageErrors
+    // -----------------------------------------------------------------------------------------
+
     describe("setPageErrors", () => {
-        test("when set with string array, pageErrors returns array", async () => {
+        test("when set with string array, pageErrors returns array", () => {
             // Arrange
             const errorMessage = "Error Message";
             const { result } = renderHook(() => usePageErrors());
@@ -25,13 +38,19 @@ describe("usePageErrors", () => {
             });
 
             // Assert
-            expect(result.current.pageErrors.length).toBe(1);
+            expect(result.current.pageErrors).toHaveLength(1);
             expect(result.current.pageErrors[0]).toBe(errorMessage);
         });
     });
 
+    // #endregion setPageErrors
+
+    // -----------------------------------------------------------------------------------------
+    // #region handlePageLoadError
+    // -----------------------------------------------------------------------------------------
+
     describe("handlePageLoadError", () => {
-        test("when error is string, pageErrors returns array", async () => {
+        test("when error is string, pageErrors returns array", () => {
             // Arrange
             const errorMessage = "Error Message";
             const { result } = renderHook(() => usePageErrors());
@@ -42,32 +61,41 @@ describe("usePageErrors", () => {
             });
 
             // Assert
-            expect(result.current.pageErrors.length).toBe(1);
+            expect(result.current.pageErrors).toHaveLength(1);
             expect(result.current.pageErrors[0]).toBe(errorMessage);
         });
 
-        test("when error is ResultRecord, pageErrors returns array", async () => {
+        test("when error is ResultRecord, pageErrors returns array", () => {
             // Arrange
             const errorMessage = "Error Message";
+            const erroredResultRecord = new ResultRecord<any>().with({
+                errors: [
+                    new ResultErrorRecord().with({
+                        message: errorMessage,
+                    }),
+                ],
+            });
             const { result } = renderHook(() => usePageErrors());
 
             // Act
             act(() => {
-                const error = {
-                    listErrorMessages: () => [errorMessage],
-                } as ResultRecord<any>;
-
-                result.current.handlePageLoadError(error);
+                result.current.handlePageLoadError(erroredResultRecord);
             });
 
             // Assert
-            expect(result.current.pageErrors.length).toBe(1);
+            expect(result.current.pageErrors).toHaveLength(1);
             expect(result.current.pageErrors[0]).toBe(errorMessage);
         });
     });
 
+    // #endregion handlePageLoadError
+
+    // -----------------------------------------------------------------------------------------
+    // #region resetPageErrors
+    // -----------------------------------------------------------------------------------------
+
     describe("resetPageErrors", () => {
-        test("when executed, pageErrors returns empty array", async () => {
+        test("when executed, pageErrors returns empty array", () => {
             // Arrange
             const { result } = renderHook(() => usePageErrors());
             act(() => {
@@ -83,4 +111,6 @@ describe("usePageErrors", () => {
             expect(result.current.pageErrors).toBeEmpty();
         });
     });
+
+    // #endregion resetPageErrors
 });