feat(gatling): improve Gatling integration#32657
Open
CharlesWong wants to merge 1 commit intojhipster:mainfrom
Open
feat(gatling): improve Gatling integration#32657CharlesWong wants to merge 1 commit intojhipster:mainfrom
CharlesWong wants to merge 1 commit intojhipster:mainfrom
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
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.
Improvements to Gatling integration
Addresses the concerns raised in #13390.
Changes
1. Documentation improvements (README.md template)
-DbaseURL=...-Dusers=N -Dramp=S2. Microservices support fixes (simulation template)
baseURLnow points to the gateway (http://localhost:${gatewayServerPort}) instead of the microservice port. This is the root cause of the authentication failure in Improve Gatling integration #13390 — microservices cannot issue OAuth2 tokens, only the gateway canapiPrefixvariable (/services/<baseName>for microservices,""for monoliths/gateways) to replace verbose inline conditionals throughout the scenario3. Simulation quality improvements
headerRegex("Location", "(.*)")toheader("Location")for capturing the created entity URL (simpler and more correct).check(status().is(204))on DELETE requests to properly assert successful deletion@codeand@preblocks showing runtime configuration optionsTesting
These changes are template-level fixes. The existing snapshot tests cover the default (non-microservice) config and are not affected. Microservice scenarios with OAuth2 will now correctly target the gateway for authentication.
Closes #13390