test(load): add K6 load test workflow with example tests #1182
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
sethidden
commented
Jun 29, 2022
| '@vue-storefront/eslint-config-jest', | ||
| ], | ||
| globals: { | ||
| "__ENV": "readonly", |
Contributor
Author
There was a problem hiding this comment.
avoid no-undef eslint error
sethidden
force-pushed
the
M2-899-k6-gh-actions
branch
from
9e043ed to
9a7d310
Compare
Contributor
Author
|
@Frodigo given the tests are pretty ugly because of graphql maybe I could push a PR with just a sample test that queries the environment base url? Like a k6 "hello world"? That way the workflow part would be merged at least. |
sethidden
force-pushed
the
M2-899-k6-gh-actions
branch
2 times, most recently
from
b67d6c5 to
f30fb18
Compare
Frodigo
previously approved these changes
Jun 30, 2022
Contributor
Frodigo
left a comment
Frodigo
left a comment
There was a problem hiding this comment.
Looks good, one question, something good to have.
This commit adds a GitHub Actions workflow (you have to *manually* trigger it) that uses the K6 load testing utillity to run tests on a given environment. I had to add the `K6_API_TOKEN` secret to GitHub to allow running the tests in K6's cloud (VSF has a paid plan). You can also run the test on GitHub's agent though. M2-899 fix: use js, not ts file chore: partially revert #1107 This reverts commit 9b8de85. This change was made because K6 recorded tests are gigantic when customQuery sends the GraphQL AST in the request body. After this change, this the request is sent in string form, but is converted into GraphQL AST on the middleware side (because Apollo GraphQL client expects AST (DocumentNode TS type), not string) test(load): improve gql requests refactor: fix newlines
sethidden
force-pushed
the
M2-899-k6-gh-actions
branch
from
f30fb18 to
d73d635
Compare
Frodigo
approved these changes
Jun 30, 2022
sethidden
changed the title
This was referenced Jul 4, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This commit adds a GitHub Actions workflow (you have to manually
trigger it) that uses the K6 load testing utillity to run tests on a
given environment.
I had to add the
K6_API_TOKENsecret to GitHub to allow running thetests in K6's cloud (VSF has a paid plan). You can also run the test on
GitHub's agent though.