Skip to content

Commit a5c3c44

Browse files
committed
update
1 parent 377e3d1 commit a5c3c44

2 files changed

Lines changed: 1 addition & 23 deletions

File tree

packages/react-core/src/modules/core/hooks/useAuthPostMessageListener.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,7 @@ export function useAuthPostMessageListener(
8585
}
8686

8787
// Ignore messages when this integration hasn't opened a popup
88-
if (!popupWindowRef.current || popupWindowRef.current.closed) {
88+
if (!popupWindowRef.current) {
8989
return;
9090
}
9191

packages/react-core/test/core/hooks/useAuthPostMessageListener.test.ts

Lines changed: 0 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -154,28 +154,6 @@ describe("useAuthPostMessageListener", () => {
154154
expect(onAuthenticationComplete).not.toHaveBeenCalled();
155155
});
156156

157-
it("should ignore messages when popup is closed", () => {
158-
mockPopup.closed = true;
159-
160-
renderHook(() =>
161-
useAuthPostMessageListener({
162-
knockHost,
163-
popupWindowRef,
164-
setConnectionStatus,
165-
onAuthenticationComplete,
166-
}),
167-
);
168-
169-
const event = new MessageEvent("message", {
170-
data: "authComplete",
171-
origin: knockHost,
172-
});
173-
window.dispatchEvent(event);
174-
175-
expect(setConnectionStatus).not.toHaveBeenCalled();
176-
expect(onAuthenticationComplete).not.toHaveBeenCalled();
177-
});
178-
179157
it("should ignore messages from different origins", () => {
180158
renderHook(() =>
181159
useAuthPostMessageListener({

0 commit comments

Comments
 (0)