-
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
Migrate test_general_setting documentation to qa-docs #2387
Conversation
|
||
references: | ||
- https://documentation.wazuh.com/current/user-manual/capabilities/vulnerability-detection/index.html | ||
- https://documentation.wazuh.com/current/user-manual/reference/ossec-conf/vuln-detector.html#ignore-time |
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.
- https://documentation.wazuh.com/current/user-manual/reference/ossec-conf/vuln-detector.html#ignore-time | |
- https://documentation.wazuh.com/current/user-manual/reference/ossec-conf/vuln-detector.html#interval |
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.
Done in this link
the wazuh_interval.yaml file. | ||
|
||
expected_output: | ||
- 'Missing sleep between scans' |
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.
- 'Missing sleep between scans' | |
- r'.* Sleeping for (.*)...' |
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.
Done in this link
- 'Alert did not appear at the start of the test' | ||
- 'Alert appeared before ignore_time was finished' | ||
- 'Alert did not appear at the end of the test' |
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.
- 'Alert did not appear at the start of the test' | |
- 'Alert appeared before ignore_time was finished' | |
- 'Alert did not appear at the end of the test' | |
- r''.* is vulnerable to .*' |
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.
Done in this link
- 'Could not find vulnerability starting scan log' | ||
- 'Found starting scan log when run on start is disabled' |
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.
- 'Could not find vulnerability starting scan log' | |
- 'Found starting scan log when run on start is disabled' | |
- 'Starting vulnerability scan' | |
- 'Vulnerability scan finished' |
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.
Done in this link
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.
Some callbacks were missing, but good job!
real_nvd_feed.json is used to check for vulnerabilities. | ||
|
||
expected_output: | ||
- r''.* is vulnerable to .*' |
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.
I would add time travel callback
- r''.* is vulnerable to .*' | |
- r''.* is vulnerable to .*' | |
- r'.*Sending FIM event: (.+)$' |
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.
Done in this link
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.
👍
Description
As part of epic #1796, this PR adds the missing documentation and migrates the current documentation to the new format used by qa-docs.
The schema used is the one defined in issue #1694
Generated documentation
test_general_settings_enabled.json
test_general_settings_ignore_time.json
test_general_settings_interval.json
test_general_settings_run_on_start.json
Tests
pycodestyle --max-line-length=120 --show-source --show-pep8 file.py
.qa-docs
tool does not raise any error.