Fixing failing tests

src/components/HomePage/SearchResultsArea/SearchResultsWidget/SearchURLWidget.spec.js

@@ -23,7 +23,7 @@ describe("Search URL Widget", () => {
 
             const testString = "test-url";
 
-            global.window = Object.create(window);
+            // global.window = Object.create(window);
             Object.defineProperty(window, "location", {
                 value: {
                     href: testString,
@@ -49,7 +49,7 @@ describe("Search URL Widget", () => {
 
             const testString = "test-string";
 
-            global.window = Object.create(window);
+            //global.window = Object.create(window);
             Object.defineProperty(window, "location", {
                 value: {
                     href: testString,

src/components/Review/Applications/ApplicationsReviewWidget.spec.js

@@ -49,7 +49,7 @@ const generateApplications = (n, forceState) => {
     for (let i = 0; i < n; i++) {
         applications.push(generateApplication(
             i,
-            forceState || STATES[i % 3]
+            forceState || STATES[(i) % 4]
         ));
     }
     return applications;
@@ -805,6 +805,8 @@ describe("Application Review Widget", () => {
 
     it("Should reset filters", async () => {
         const applications = generateApplications(5);
+        console.log("Hello");
+        console.log(applications);
 
         fetch.mockResponse(JSON.stringify({ applications }));
 
@@ -820,13 +822,13 @@ describe("Application Review Widget", () => {
 
         fireEvent.click(screen.getByLabelText("Filter list"));
 
-        fireEvent.change(screen.getByLabelText("Company Name"), { target: { value: "0" } });
+        fireEvent.change(screen.getByLabelText("Company Name"), { target: { value: "1" } });
 
         fireEvent.change(screen.getByLabelText("Date From..."), { target: { value:
-            format(new Date(applications[0].submittedAt), "yyyy-MM-dd"),
+            format(new Date(applications[1].submittedAt), "yyyy-MM-dd"),
         } });
         fireEvent.change(screen.getByLabelText("Date To..."), { target: { value:
-            format(new Date(applications[0].submittedAt), "yyyy-MM-dd"),
+            format(new Date(applications[1].submittedAt), "yyyy-MM-dd"),
         } });
 
         await userEvent.click(screen.getByLabelText("State"));
@@ -837,7 +839,7 @@ describe("Application Review Widget", () => {
 
         expect(screen.getAllByTestId("application-row")
             .map((el) => el.querySelector("td:nth-child(2)").textContent)
-        ).toStrictEqual([applications[0].companyName]);
+        ).toStrictEqual([applications[1].companyName]);
 
         fireEvent.click(screen.getByLabelText("Filter list"));
 

src/utils/UndoableActionsHandlerProvider.js

@@ -86,16 +86,25 @@ const ActionNotification = connect(mapStateToProps, mapDispatchToProps)(BaseActi
 const UndoableActionsHandlerProvider = ({ children }) => {
     const [actions, setActions] = useState({});
     const [closeLock, setCloseLock] = useState(false);
-    const initialBeforeUnloadEventListener = useRef();
+    const initialBeforeUnloadEventListener = useRef(window.onbeforeunload);
 
     useEffect(() => {
         if (Object.keys(actions).length && !closeLock) {
             setCloseLock(true);
             initialBeforeUnloadEventListener.current = window.onbeforeunload;
-            window.onbeforeunload =  () => "Some changes might have not taken effect yet...";
+
+            Object.defineProperty(window, "onbeforeunload", {
+                value: () => "Some changes might have not taken effect yet...",
+                writable: true,
+            });
+
         } else if (!Object.keys(actions).length && closeLock) {
             setCloseLock(false);
-            window.onbeforeunload = initialBeforeUnloadEventListener.current;
+
+            Object.defineProperty(window, "onbeforeunload", {
+                value: initialBeforeUnloadEventListener.current,
+                writable: true,
+            });
         }
 
     }, [actions, closeLock]);