Fix Integratord tests #3362
Closed
Fix Integratord tests #3362
Conversation
This file contains 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
After a change made in a message, we had to change a callback from ERROR to WARNING. Also, we have to change the test cases IDs for us to be capable of running tests selecting them by theirs IDs. Finally, we made changes to the current documentation. Refs: wazuh/wazuh#14431
damarisg
requested changes
Sep 26, 2022
tests/integration/test_integratord/data/test_cases/cases_integratord_change_inode_alert.yaml
Outdated
Show resolved
Hide resolved
tests/integration/test_integratord/test_integratord_change_inode_alert.py
Outdated
Show resolved
Hide resolved
| # Read Response in ossec.log | ||
| check_integratord_event(file_monitor=wazuh_monitor, timeout=global_parameters.default_timeout, | ||
| check_integratord_event(file_monitor=wazuh_monitor, timeout=global_parameters.default_timeout + 2, |
There was a problem hiding this comment.
@damarisg Because after multiple executions that was the time in which Wazuh managed to process the alert and display the expected message in the log.
Detect when analysisd has finished its initialization. Set analysisd in debug mode to detect its initialization. Restart fixture changed to restart a list of daemons and stop them in order.
Update 27/09/2022The tests are failing: 🔴 After modifying the test so that only the necessary daemons are restarted, the test started to run faster. This message appeared in the I see 2 possible solutions:
Update: The second alternative will be applied. |
Some timeouts were replaced with the default timeout. Somo minor changes in the test documentation were applied.
This was referenced Oct 21, 2022
|
Duplicated #3499 |
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.
Description
After a change was made in a message, we had to change a callback from ERROR to WARNING.
Also, we have to change the test case IDs to be capable of running tests and selecting them by their IDs.
Finally, we made changes to the current documentation.
Refs: wazuh/wazuh#14431
Updated
test_integratord_change_json_inode: The sleep time was reduced to 1 second becausewazuh-integratordhas a one-second delaydeps/wazuh_testing/wazuh_testing/modules/integratord/__init__.py: A callback was changed.Testing performed
test_integratord_change_inode_alert.pyis unstable, and it cause is being researched