diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 909bb93f683..a4aa94a2ed0 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -84,12 +84,18 @@ jobs: reporter: jest-junit fail-on-error: true - - name: Upload to codecov.io + - name: Upload coverage to codecov.io uses: codecov/codecov-action@54bcd8715eee62d40e33596ef5e8f0f48dbbccab # v4.1.0 if: ${{ needs.check-test-secrets.outputs.available == 'true' }} env: CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }} + - name: Upload results to codecov.io + uses: codecov/test-results-action@1b5b448b98e58ba90d1a1a1d9fcb72ca2263be46 # v1.0.0 + if: ${{ needs.check-test-secrets.outputs.available == 'true' }} + env: + CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }} + rust: name: Run Rust tests on ${{ matrix.os }} runs-on: ${{ matrix.os || 'ubuntu-latest' }} diff --git a/apps/browser/src/autofill/content/auto-submit-login.spec.ts b/apps/browser/src/autofill/content/auto-submit-login.spec.ts index c5bc9dbccbd..d8a192dbcab 100644 --- a/apps/browser/src/autofill/content/auto-submit-login.spec.ts +++ b/apps/browser/src/autofill/content/auto-submit-login.spec.ts @@ -61,10 +61,13 @@ describe("AutoSubmitLogin content script", () => { await initAutoSubmitWorkflow(); - expect(chrome.runtime.sendMessage).toHaveBeenCalledWith({ - command: "updateIsFieldCurrentlyFilling", - isFieldCurrentlyFilling: false, - }); + expect(chrome.runtime.sendMessage).toHaveBeenCalledWith( + { + command: "updateIsFieldCurrentlyFilling", + isFieldCurrentlyFilling: false, + }, + expect.any(Function), + ); }); describe("when the page contains form fields", () => { @@ -78,10 +81,13 @@ describe("AutoSubmitLogin content script", () => { }); await flushPromises(); - expect(chrome.runtime.sendMessage).toHaveBeenCalledWith({ - command: "updateIsFieldCurrentlyFilling", - isFieldCurrentlyFilling: false, - }); + expect(chrome.runtime.sendMessage).toHaveBeenCalledWith( + { + command: "updateIsFieldCurrentlyFilling", + isFieldCurrentlyFilling: false, + }, + expect.any(Function), + ); }); describe("triggering auto-submit on formless fields", () => { diff --git a/apps/browser/src/autofill/overlay/inline-menu/pages/list/autofill-inline-menu-list.ts b/apps/browser/src/autofill/overlay/inline-menu/pages/list/autofill-inline-menu-list.ts index 6ec0bc83991..b5dc76f2226 100644 --- a/apps/browser/src/autofill/overlay/inline-menu/pages/list/autofill-inline-menu-list.ts +++ b/apps/browser/src/autofill/overlay/inline-menu/pages/list/autofill-inline-menu-list.ts @@ -177,6 +177,7 @@ export class AutofillInlineMenuList extends AutofillInlineMenuPageElement { this.loadPageOfCiphers(); this.inlineMenuListContainer.appendChild(this.ciphersList); + this.toggleScrollClass(); if (!this.showInlineMenuAccountCreation) { return; @@ -973,13 +974,19 @@ export class AutofillInlineMenuList extends AutofillInlineMenuPageElement { * * @param height - The height of the inline menu list actions container. */ - private toggleScrollClass = (height: number) => { + private toggleScrollClass = (height?: number) => { if (!this.ciphersList) { return; } const scrollbarClass = "inline-menu-list-actions--scrollbar"; - if (height >= 170) { + let containerHeight = height; + if (!containerHeight) { + const inlineMenuListContainerRects = this.inlineMenuListContainer.getBoundingClientRect(); + containerHeight = inlineMenuListContainerRects.height; + } + + if (containerHeight >= 170) { this.ciphersList.classList.add(scrollbarClass); return; }