Fix lint violations in date utils tests

Author: martijnrusschen

src/test/date_utils_test.test.ts

@@ -926,6 +926,41 @@ describe("date_utils", () => {
     });
   });
 
+  describe("isTimeInDisabledRange edge cases", () => {
+    it("throws when either minTime or maxTime is missing", () => {
+      expect(() =>
+        isTimeInDisabledRange(newDate(), { minTime: newDate() }),
+      ).toThrow("Both minTime and maxTime props required");
+    });
+
+    it("returns false when isWithinInterval throws", async () => {
+      jest.doMock("date-fns", () => {
+        const actual = jest.requireActual("date-fns");
+        return {
+          ...actual,
+          isWithinInterval: () => {
+            throw new Error("boom");
+          },
+        };
+      });
+
+      try {
+        const { isTimeInDisabledRange: mockedIsTimeInDisabledRange } =
+          await import("../date_utils");
+
+        expect(
+          mockedIsTimeInDisabledRange(new Date(), {
+            minTime: new Date(),
+            maxTime: new Date(),
+          }),
+        ).toBe(false);
+      } finally {
+        jest.resetModules();
+        jest.dontMock("date-fns");
+      }
+    });
+  });
+
   describe("isDayInRange", () => {
     it("should tell if day is in range", () => {
       const day = newDate("2016-02-15 09:40");
@@ -1157,42 +1192,6 @@ describe("date_utils", () => {
     });
   });
 
-  describe("isTimeInDisabledRange", () => {
-    it("throws when either minTime or maxTime is missing", () => {
-      expect(() =>
-        isTimeInDisabledRange(newDate(), { minTime: newDate() }),
-      ).toThrow("Both minTime and maxTime props required");
-    });
-
-    it("returns false when isWithinInterval throws", () => {
-      jest.isolateModules(() => {
-        jest.doMock("date-fns", () => {
-          const actual = jest.requireActual("date-fns");
-          return {
-            ...actual,
-            isWithinInterval: () => {
-              throw new Error("boom");
-            },
-          };
-        });
-
-        // eslint-disable-next-line @typescript-eslint/no-var-requires -- isolated mock import
-        const {
-          isTimeInDisabledRange: mockedIsTimeInDisabledRange,
-        } = require("../date_utils");
-        expect(
-          mockedIsTimeInDisabledRange(new Date(), {
-            minTime: new Date(),
-            maxTime: new Date(),
-          }),
-        ).toBe(false);
-
-        jest.resetModules();
-        jest.dontMock("date-fns");
-      });
-    });
-  });
-
   describe("isYearInRange", () => {
     it("should return true if the year passed is in range", () => {
       const startDate = newDate("2000-01-01");
@@ -1632,31 +1631,33 @@ describe("date_utils", () => {
   });
 
   describe("isDayInRange error handling", () => {
-    it("returns false when isWithinInterval throws", () => {
-      jest.isolateModules(() => {
-        jest.doMock("date-fns", () => {
-          const actual = jest.requireActual("date-fns");
-          return {
-            ...actual,
-            isWithinInterval: () => {
-              throw new Error("boom");
-            },
-          };
-        });
-
-        // eslint-disable-next-line @typescript-eslint/no-var-requires -- isolated mock import
-        const { isDayInRange: mockedIsDayInRange } = require("../date_utils");
+    it("returns false when isWithinInterval throws", async () => {
+      jest.doMock("date-fns", () => {
+        const actual = jest.requireActual("date-fns");
+        return {
+          ...actual,
+          isWithinInterval: () => {
+            throw new Error("boom");
+          },
+        };
+      });
+
+      try {
+        const { isDayInRange: mockedIsDayInRange } = await import(
+          "../date_utils"
+        );
+
         expect(
           mockedIsDayInRange(
             new Date("2024-01-15"),
             new Date("2024-01-10"),
             new Date("2024-01-20"),
           ),
         ).toBe(false);
-
+      } finally {
         jest.resetModules();
         jest.dontMock("date-fns");
-      });
+      }
     });
 
     it("returns true for dates inside a valid range", () => {