Skip to content

Add additional load test options#436

Merged
ericbuckley merged 9 commits into
mainfrom
feature/load-test-options
Jun 17, 2025
Merged

Add additional load test options#436
ericbuckley merged 9 commits into
mainfrom
feature/load-test-options

Conversation

@ericbuckley

@ericbuckley ericbuckley commented Jun 14, 2025 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Description

Adding more options to run load tests with, including the ability to toggle between /link and /match.

Additional Notes

  • adding missing load testing dependencies to pyproject.toml
  • adding additional arguments to scripts/gen_seed_test_data.py to allow for data scrambling at creation time
  • adding a load testing README.md

<--------------------- REMOVE THE LINES BELOW BEFORE MERGING --------------------->

Checklist

Please review and complete the following checklist before submitting your pull request:

  • I have ensured that the pull request is of a manageable size, allowing it to be reviewed within a single session.
  • I have reviewed my changes to ensure they are clear, concise, and well-documented.
  • I have updated the documentation, if applicable.
  • I have added or updated test cases to cover my changes, if applicable.
  • I have minimized the number of reviewers to include only those essential for the review.

Checklist for Reviewers

Please review and complete the following checklist during the review process:

  • The code follows best practices and conventions.
  • The changes implement the desired functionality or fix the reported issue.
  • The tests cover the new changes and pass successfully.
  • Any potential edge cases or error scenarios have been considered.

@ericbuckley ericbuckley self-assigned this Jun 14, 2025
@ericbuckley ericbuckley added the qa Technical improvements to increase code quality label Jun 14, 2025
@codecov

codecov Bot commented Jun 14, 2025 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.54%. Comparing base (cc89832) to head (840f602).
Report is 2 commits behind head on main.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main     #436      +/-   ##
==========================================
+ Coverage   98.51%   98.54%   +0.02%     
==========================================
  Files          33       33              
  Lines        1951     1987      +36     
==========================================
+ Hits         1922     1958      +36     
  Misses         29       29

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ericbuckley ericbuckley marked this pull request as ready for review June 14, 2025 17:02
m-goggins
m-goggins approved these changes Jun 16, 2025
View reviewed changes

@m-goggins m-goggins left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

🚀

m-goggins
m-goggins reviewed Jun 16, 2025
View reviewed changes
Comment thread tests/load/locustfile.py
Comment thread tests/load/locustfile.py
@ericbuckley ericbuckley merged commit c20c4e7 into main Jun 17, 2025
15 checks passed
@ericbuckley ericbuckley deleted the feature/load-test-options branch June 17, 2025 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@m-goggins m-goggins m-goggins approved these changes

@bamader bamader Awaiting requested review from bamader bamader is a code owner

Assignees

@ericbuckley ericbuckley

Labels

qa Technical improvements to increase code quality

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ericbuckley @m-goggins