-
Notifications
You must be signed in to change notification settings - Fork 32
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
Update API performance tests #1519
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
a56c454
a18437b
to
31c1199
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
GJ, but some changes are required
31c1199
to
1edc50b
Compare
1edc50b
to
7787ebc
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
Hi team,
This PR closes #1490. In this PR we have updated the API performance tests based on the latest changes made in the main repo.
Also, we have added the
xfailed
flag to all tests that should fail until the following issues are fixed:queue/alerts/ar
socket wazuh#9125queue/sockets/auth
socket wazuh#9127agent-groups
files in theDELETE /groups
endpoint wazuh#9141Test results
Tests
pycodestyle --max-line-length=120 --show-source --show-pep8 file.py
.provision_documentation.sh
generate the docs without errors.