-
Notifications
You must be signed in to change notification settings - Fork 30
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
Vulnerability detector test module refactor: test_duplicate_feeds
#2575
Vulnerability detector test module refactor: test_duplicate_feeds
#2575
Conversation
deps/wazuh_testing/wazuh_testing/modules/vulnerability_detector/event_monitor.py
Outdated
Show resolved
Hide resolved
deps/wazuh_testing/wazuh_testing/modules/vulnerability_detector/event_monitor.py
Outdated
Show resolved
Hide resolved
tests/integration/test_vulnerability_detector/test_providers/test_enabled.py
Outdated
Show resolved
Hide resolved
tests/integration/test_vulnerability_detector/test_providers/test_missing_os.py
Outdated
Show resolved
Hide resolved
tests/integration/test_vulnerability_detector/test_providers/test_multiple_provider_feeds.py
Outdated
Show resolved
Hide resolved
tests/integration/test_vulnerability_detector/test_scan_results/test_duplicate_feeds.py
Outdated
Show resolved
Hide resolved
number_feeds_second = check_update_finish_logs(wazuh_log_monitor=log_monitor, | ||
provider_name=metadata['provider_name'], | ||
provider_json_name=metadata['provider_json_name'], | ||
timeout=vd.VULN_DETECTOR_MEDIUM_TIMEOUT) |
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.
Why you use here this custom timeout, and not in the other cases?
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.
Because the second time I check for the Finish...
message It fails for the timeout exception. This situation did not happen in the first call of the function.
Description
This PR adds changes and enhancements to the testing framework for vulnerability detector.
The changes made are as follows:
Added
test_duplicate_feeds
and data.Improvements
Tests
test_alert_vulnerability_removal
Outputs
All vulnerability detector tests
test_alert_vulnerability_removal