Skip to content

fix existing load testing script#2941

Merged
thecristen merged 3 commits intomainfrom
cbj/fix-load-test-script
Feb 14, 2026
Merged

fix existing load testing script#2941
thecristen merged 3 commits intomainfrom
cbj/fix-load-test-script

Conversation

@thecristen
Copy link
Collaborator

Scope

No ticket, I noticed the current load testing script added in #1891 didn't actually work 🫠

Implementation

  • First was a mismatch between our Playwright version and the version associated with Artillery, our load testing library. Updating them to these versions worked.
    • Also updated axios because security 🔒
  • The references to the named functions were formatted incorrectly, so I updated them to reflect the correct values
    • One test was for a feature which no longer exists, so I removed that one

How to test

  • Install/update the dependencies (npm install --ignore-scripts from the root folder)
  • Run the load tests and note it runs to completion (can use below or substitute target of choice):
npx artillery run ./integration/load_tests/all-scenarios.yml --target https://dev-blue.mbtace.com

@thecristen thecristen requested a review from a team as a code owner February 11, 2026 18:38
Copy link
Contributor

@joshlarson joshlarson left a comment

Choose a reason for hiding this comment

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

🏋️

@thecristen thecristen enabled auto-merge (squash) February 14, 2026 02:26
@thecristen thecristen merged commit bd343e5 into main Feb 14, 2026
17 checks passed
@thecristen thecristen deleted the cbj/fix-load-test-script branch February 14, 2026 02:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants