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: Ce 1222 load tests #849

Merged
merged 4 commits into from
Jan 6, 2025
Merged

feat: Ce 1222 load tests #849

merged 4 commits into from
Jan 6, 2025

Conversation

mikevespi
Copy link
Contributor

@mikevespi mikevespi commented Jan 3, 2025

Added load testing code and configuration to the repository.

Description

This PR introduces code to perform load tests on the application. More details, including instructions, can be found in performance_tests/README.md. K6 tooling was used to write the tests, which include back- and front-end protocol level tests as well as browser level tests for the front-end which simulates a user logging in and using the app. A sibling PR was created to add more test data prior to running the tests.

Platform Services are to be notified before running tests beyond the smoke test level load.

Fixes # (issue)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Each test has been run against a locally deployed instance of the applications

Checklist

  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

Further comments


Thanks for the PR!

Deployments, as required, will be available below:

Please create PRs in draft mode. Mark as ready to enable:

After merge, new images are deployed in:

@mikevespi mikevespi changed the base branch from main to release/0.6.12 January 3, 2025 18:08
@mikevespi
Copy link
Contributor Author

Quality Gate Failed Quality Gate failed

Failed conditions C Security Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

The new Sonarqube warnings are based on the use of var in the test scripts. While this is seen as bad practice, it was intentional in the test scripts as we're taking a bit of advantage of the scope mechanics that var has.

@afwilcox afwilcox changed the title Ce 1222 load tests feat: Ce 1222 load tests Jan 3, 2025
Copy link

sonarqubecloud bot commented Jan 4, 2025

@afwilcox afwilcox merged commit 4445d65 into release/0.6.12 Jan 6, 2025
15 checks passed
@afwilcox afwilcox deleted the CE-1222-load-tests branch January 6, 2025 15:48
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