From b9c18b5526e27ea1c8cd4a62f00c5963843c6840 Mon Sep 17 00:00:00 2001 From: Cesar Gonzalez Date: Tue, 25 Jun 2024 12:06:30 -0500 Subject: [PATCH] [PM-5189] Working through jest tests for OverlayBackground and refining repositioning delays --- .../src/autofill/background/overlay.background.spec.ts | 2 +- apps/browser/src/autofill/background/overlay.background.ts | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/apps/browser/src/autofill/background/overlay.background.spec.ts b/apps/browser/src/autofill/background/overlay.background.spec.ts index c592c492bdc..4c843b47749 100644 --- a/apps/browser/src/autofill/background/overlay.background.spec.ts +++ b/apps/browser/src/autofill/background/overlay.background.spec.ts @@ -515,7 +515,7 @@ describe("OverlayBackground", () => { async function flushUpdateInlineMenuPromises() { await flushOverlayRepositionPromises(); await flushPromises(); - jest.advanceTimersByTime(150); + jest.advanceTimersByTime(250); await flushPromises(); } diff --git a/apps/browser/src/autofill/background/overlay.background.ts b/apps/browser/src/autofill/background/overlay.background.ts index 5fca147d339..18fe6bc5243 100644 --- a/apps/browser/src/autofill/background/overlay.background.ts +++ b/apps/browser/src/autofill/background/overlay.background.ts @@ -163,7 +163,7 @@ export class OverlayBackground implements OverlayBackgroundInterface { private initOverlayEventObservables() { this.repositionInlineMenuSubject .pipe( - debounceTime(800), + debounceTime(950), switchMap((sender) => this.repositionInlineMenu(sender)), ) .subscribe(); @@ -176,7 +176,7 @@ export class OverlayBackground implements OverlayBackgroundInterface { // Debounce used to update inline menu position merge( - this.startUpdateInlineMenuPositionSubject.pipe(debounceTime(150)), + this.startUpdateInlineMenuPositionSubject.pipe(debounceTime(250)), this.cancelUpdateInlineMenuPositionSubject, ) .pipe(switchMap((sender) => this.updateInlineMenuPositionAfterRepositionEvent(sender)))