Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MWPW-144108: Project PEP unit tests #2136

Conversation

robert-bogos
Copy link
Contributor

@robert-bogos robert-bogos commented Apr 11, 2024

Description

This PR adds unit tests for project PEP

Related Issue

Resolves: MWPW-144108

Testing instructions

  1. change web-test-runner.config.mjs by adding files: ['./test/features/webapp-prompt/webapp-prompt.test.js'], to the default exported json (this will help target the webapp-prompt.test.js for tests)
  2. run npm run test:file:watch

Screenshots:

Screenshot 2024-04-11 at 15 40 39

Test URLs

Milo:

@robert-bogos robert-bogos self-assigned this Apr 11, 2024
@robert-bogos robert-bogos requested a review from a team as a code owner April 11, 2024 12:54
@robert-bogos robert-bogos added zero-impact trivial PR doesn't require E2E testing by a reviewer labels Apr 11, 2024
@mokimo
Copy link
Contributor

mokimo commented Apr 11, 2024

Add some dummy links that just go to the milo homepage to make the PSI check pass

Copy link
Contributor

aem-code-sync bot commented Apr 11, 2024

@SilviuLCF
Copy link

No QA needed , PEP E2E testing will be done on https://jira.corp.adobe.com/browse/MWPW-142861

@SilviuLCF SilviuLCF self-requested a review April 12, 2024 07:36
Copy link
Contributor

@mokimo mokimo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, clean and tests a lot of the different use-cases & interactions.
Great job!

test/features/webapp-prompt/webapp-prompt.test.js Outdated Show resolved Hide resolved
test/features/webapp-prompt/webapp-prompt.test.js Outdated Show resolved Hide resolved
Copy link
Contributor

Reminder to set the Ready for Stage label - to queue this to get merged to stage & production.

@SilviuLCF SilviuLCF added verified PR has been E2E tested by a reviewer Ready for Stage labels Apr 15, 2024
@robert-bogos
Copy link
Contributor Author

Since this is open against a feature branch, has approvals, all tests are passing; I will merge for now.

@robert-bogos robert-bogos merged commit 2578179 into adobecom:project-pep Apr 15, 2024
10 checks passed
overmyheadandbody pushed a commit that referenced this pull request Apr 29, 2024
* unit tests for pep

* fixes for gnav unit tests

* optimizing pep unit tests
@mokimo mokimo mentioned this pull request May 3, 2024
Blainegunn pushed a commit that referenced this pull request May 3, 2024
* [MWPW-139990] PEP prompt

* MWPW-142617: Microsoft Edge & Chrome Win - progress bar not completely loaded (#1882)

* hotfix

* refactor: changed loader animation logic

* update

* revert update

* more gpu acceleration for pep loader

* hotfix

* hotfix

* [MWPW-141023] PEP authoring (#1889)

* MWPW-141021: Project PEP Accessibility Requirements (#1853)

* [MWPW-139990] PEP prompt

* pep accessibility requirements

* hotfix

* hotfixes

* initial focus on close icon

* implementing feedback

* hotfix

* [MWPW-139990] PEP prompt

* MWPW-142617: Microsoft Edge & Chrome Win - progress bar not completely loaded (#1882)

* hotfix

* refactor: changed loader animation logic

* update

* revert update

* more gpu acceleration for pep loader

* hotfix

* hotfix

* [MWPW-141023] PEP authoring (#1889)

* implementing feedback

* css variables

* hotfix: moved css variables

* hotfix

---------

Co-authored-by: Rares Munteanu <overmyheadandbody@gmail.com>

* MWPW-142622: [Project PEP] Close button focus  (#1897)

* [MWPW-139990] PEP prompt

* MWPW-142617: Microsoft Edge & Chrome Win - progress bar not completely loaded (#1882)

* hotfix

* refactor: changed loader animation logic

* update

* revert update

* more gpu acceleration for pep loader

* hotfix

* hotfix

* [MWPW-141023] PEP authoring (#1889)

* hotfix: blue outline on close button when focused

* hotfix

---------

Co-authored-by: Rares Munteanu <overmyheadandbody@gmail.com>

* MWPW-141020: [Project PEP] Implement Analytics Enablement (#1951)

* pep analytics

* hotfix

* hotfix

* hotfix

* removed capitalization from analytics label

* [MWPW-141050] Fetch profile for PEP

* [MWPW-141025] Integrate XLG with PEP

* [MWPW-142625] PEP relative to App Switcher

* MWPW-144108: Project PEP unit tests (#2136)

* unit tests for pep

* fixes for gnav unit tests

* optimizing pep unit tests

* MWPW-146582: PEP fixes (#2144)

* added aria-label attribute to pep close button

* force load imslib.min.js

* revert force load imslib.min.js

* changed aria label for close button

* pep final touches

* [MWPW-147145] Allow SVG in PEP Prompt; change redirect method

---------

Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: Robert Bogos <robert.adobe.github@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
trivial PR doesn't require E2E testing by a reviewer verified PR has been E2E tested by a reviewer zero-impact
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants