Skip to content

fix existing load testing script#2941

Open
thecristen wants to merge 2 commits intomainfrom
cbj/fix-load-test-script
Open

fix existing load testing script#2941
thecristen wants to merge 2 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.

🏋️

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