Skip to content

Commit

Permalink
chore: lint
Browse files Browse the repository at this point in the history
  • Loading branch information
DanielRivers committed Nov 19, 2024
1 parent 6012806 commit e900979
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 9 deletions.
16 changes: 10 additions & 6 deletions lib/utils/refreshTimer.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -5,27 +5,31 @@ describe("refreshTimer", () => {
beforeAll(() => {
vi.spyOn(window, "setTimeout");
vi.spyOn(window, "clearTimeout");
vi.spyOn(RefreshTimer, 'clearRefreshTimer');
})
vi.spyOn(RefreshTimer, "clearRefreshTimer");
});

it("set timer and not call callback instantly", () => {
const callback = vi.fn();
RefreshTimer.setRefreshTimer(10, callback);
expect(callback).not.toHaveBeenCalled();
})
});

it("error when timeout is negative", () => {
const callback = vi.fn();
expect(()=>RefreshTimer.setRefreshTimer(-10, callback)).toThrowError("Timer duration must be positive");
})
expect(() => RefreshTimer.setRefreshTimer(-10, callback)).toThrowError(
"Timer duration must be positive",
);
});

it("should throw error when window is undefined", () => {
const originalWindow = global.window;
// Temporarily delete the window object
delete global.window;

Check failure on line 27 in lib/utils/refreshTimer.test.ts

View workflow job for this annotation

GitHub Actions / main (20.x)

The operand of a 'delete' operator must be optional.

const callback = vi.fn();
expect(() => RefreshTimer.setRefreshTimer(10, callback)).toThrowError("setRefreshTimer requires a browser environment");
expect(() => RefreshTimer.setRefreshTimer(10, callback)).toThrowError(
"setRefreshTimer requires a browser environment",
);

// Restore the window object
global.window = originalWindow;
Expand Down
6 changes: 3 additions & 3 deletions lib/utils/refreshTimer.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,11 @@ let refreshTimer: number | undefined;

export function setRefreshTimer(timer: number, callback: () => void) {
clearRefreshTimer();
if (typeof window === 'undefined') {
throw new Error('setRefreshTimer requires a browser environment');
if (typeof window === "undefined") {
throw new Error("setRefreshTimer requires a browser environment");
}
if (timer <= 0) {
throw new Error('Timer duration must be positive');
throw new Error("Timer duration must be positive");
}
refreshTimer = window.setTimeout(callback, timer * 1000 - 10000);
}
Expand Down

0 comments on commit e900979

Please sign in to comment.