Skip to content

[AUTOMATED-PR] Update imslib.min.js dependency#1

Open
github-actions[bot] wants to merge 1 commit intostagefrom
update-imslib
Open

[AUTOMATED-PR] Update imslib.min.js dependency#1
github-actions[bot] wants to merge 1 commit intostagefrom
update-imslib

Conversation

@github-actions
Copy link

@aem-code-sync
Copy link

aem-code-sync bot commented Sep 12, 2024

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
In case there are problems, just click a checkbox below to rerun the respective action.

  • Re-run PSI checks
  • Re-sync branch
Commits

@aem-code-sync
Copy link

aem-code-sync bot commented Sep 12, 2024

rohitsahu pushed a commit that referenced this pull request May 14, 2025
…obecom#3549)

* fixed the flaky test for logWebVitalsUtils.test.js (#1)

* fixed the flaky test

* move performance mock to beforeEach/afterEach for isolation, simplify lcpElType check

* removed unused code

* fixed the same issue in logWebVitals.test.js and reverted unnecessary changes

* Update fstab.yaml

* Update fstab.yaml

* Update fstab.yaml

* Update fstab.yaml

* Update fstab.yaml

* removed dependency on gist.github.com by mocking script request

* replaced setTimeout to prevent flakiness

* Replaced Promise.resolve().then() with setTimeout()

* earlier return for if statement

* Improve callback matching in script mock

* improve event loop handling

* Replaced IIFE with async
rohitsahu pushed a commit that referenced this pull request May 14, 2025
* fixed the flaky test for logWebVitalsUtils.test.js (#1)

* fixed the flaky test

* move performance mock to beforeEach/afterEach for isolation, simplify lcpElType check

* removed unused code

* fixed the same issue in logWebVitals.test.js and reverted unnecessary changes

* Update fstab.yaml

* Update fstab.yaml

* Update fstab.yaml

* Update fstab.yaml

* Update fstab.yaml

* enhanced tool tips Unit tests

* added assertions and isolated DOM state

* changed unnecessary async to sync

* added loadIcons for duplicate test

* tooltip should be visible on hover

* added a test case for visibility of tooltip

* testcase for tooltip visibility

* Changes visibility test case & gave appropriate name to tests

* Update test/features/icons/icons.test.js

Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>

---------

Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant