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.

Sign up for GitHub

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

Jump to bottom

Add errors grouping for dynamic endpoint #993

Merged
merged 1 commit into from
Oct 30, 2019
Merged

Add errors grouping for dynamic endpoint #993

merged 1 commit into from
Oct 30, 2019

Conversation

dduleba
Copy link
Contributor

@dduleba dduleba commented Apr 6, 2019

  • currentlly there is option to group urls name for dynamic endpoint
    • adding name parameter to request
  • this commit group error messages for dynamic endpoints
    • if name parameter was provided

Add errors grouping for dynamic endpoint
d1ca550 
* currentlly there is option to group urls name for dynamic endpoint
  * adding name parameter to request
* this commit group error messages for dynamic endpoints
  * if name parameter was provided
@dduleba
Copy link
Contributor Author

dduleba commented Apr 9, 2019

During preparation for presentation on local QA meetup I noticed issue with multiple errors for endpoint with provided name

  • failures for this endpoint was not grouped by name provided in request parameters because of dynamic url name in each error

To show more details you can preview the attached
multiple errors for dynamic endpoint gif

with this fix url will be replaced with url name as it is done for grouping requests name on statistics tab

@cyberw cyberw merged commit 5c84591 into locustio:master Oct 30, 2019
@cyberw
Copy link
Collaborator

cyberw commented Oct 30, 2019

LGTM! Sorry for the slow response.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dduleba @cyberw