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

Feat(playwright performance): Pw perf tests #959

Merged
merged 3 commits into from
Jan 9, 2025
Merged

Conversation

zerogravit1
Copy link
Collaborator

🎫 Linked Ticket

none

💬 Description / Notes

Testing for Time to First Byte
Time to Largest Contentful Paint
Time to First Contentful Paint

🛠 Changes

Add performance tests

Copy link
Contributor

github-actions bot commented Dec 29, 2024

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 80.64% 7139 / 8852
🔵 Statements 79.64% 7460 / 9367
🔵 Functions 71.89% 2077 / 2889
🔵 Branches 48.43% 1334 / 2754
File CoverageNo changed files found.
Generated in workflow #1422 for commit c2b4132 by the Vitest Coverage Report Action

Copy link
Collaborator

@asharonbaltazar asharonbaltazar left a comment

Choose a reason for hiding this comment

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

Looks good. Left some questions

test/e2e/tests/perf/index.spec.ts Outdated Show resolved Hide resolved
test/e2e/tests/perf/index.spec.ts Show resolved Hide resolved
test/e2e/tests/perf/index.spec.ts Show resolved Hide resolved
test/e2e/tests/perf/index.spec.ts Outdated Show resolved Hide resolved
@zerogravit1 zerogravit1 merged commit b92334d into main Jan 9, 2025
15 checks passed
@zerogravit1 zerogravit1 deleted the pw-perf-tests branch January 9, 2025 18:43
RanyeM pushed a commit that referenced this pull request Jan 9, 2025
* feat: initial perf tests for routes

fix: refactor

* Fix: updated per review comments

---------

Co-authored-by: asharonbaltazar <[email protected]>
RanyeM added a commit that referenced this pull request Jan 9, 2025
* feature: adding new FAQ to FAQ page

* feature: question should open corresponding section

* fix: corrected code back to faq page direct link

* fix: removed id

* fix: #5 and #6 to expanded section

* fix: test coverage

* revert package json to what is on main

* fix(email): Fixes an issue with Outlook not correctly displaying the OneMAC logo and header (#972)

* fixed index test

* fix(test): Create mock for calling lambdas (#973)

* Added msw endpoints for Lambda, Step Functions, Secure Token Services, and other related services
* Added/updated test data to support new endpoints
* Updated tests to use the new endpoints instead of mocking individually

* fix(email): switch email logo back to using url

* fix(schema): remove logic for determining next business day for `submissionDate` (#980)

* fix: remove logic for determining next business day for `submissionDate`

* fix: update `submissionDate` value

* feat(ui): Remove status date from package details pages for all users (#965)

* fix(initial verbiage fix): Email verbiage fixes from qa (#974)

* correct verbage for med and chip spa

* fix issues with verbiage need confirmation

* remove comment

* add date and attachment logic

* update snapshots

* Update ChipSpaState.tsx

* test(email): proces email test (#979)

* feat(test)proces email test

* Removed console.log

* linter correction

* removed a ine

* remove brackets

* add region

* Address change reqeust

* feat(test)-submit lambda (#987)

* feat(test)-submit lambda

* Update withdrawpackage to withdrawrai

* linter corrections

* Fix some test

* Test for malformed objects

* Remove import vs code added in

* remove console.log

* Updatign to use get requestcontext

* feat(test) fix email test (#991)

* feat(test) fix email test
* Update processEmails.test.ts

* fix(os) helpers for getting domain and index for opensearch (#992)

* created helpers for getting domain and index for opensearch
* fixed import

* feat(test) updates to s3 calls on backend (#985)

* updates to s3 calls on backend

* feat(test)-unauthorized submission test (#996)

* feat(amend): Add waiver amendment emails (#993)

* Feat(playwright performance): Pw perf tests (#959)

* feat: initial perf tests for routes

fix: refactor

* Fix: updated per review comments

---------

Co-authored-by: asharonbaltazar <[email protected]>

---------

Co-authored-by: 13bfrancis <[email protected]>
Co-authored-by: Ty Bolt <[email protected]>
Co-authored-by: Tiffany Forkner <[email protected]>
Co-authored-by: asharonbaltazar <[email protected]>
Co-authored-by: James Dinh <[email protected]>
Co-authored-by: Thomas Walker <[email protected]>
Co-authored-by: Benjamin Paige <[email protected]>
Co-authored-by: Jon Schaffer <[email protected]>
Copy link
Contributor

github-actions bot commented Jan 9, 2025

🎉 This PR is included in version 1.5.0-val.103 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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.

2 participants