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.

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

Full Yellow Local - Fix Vulnerability Detector tests in 4.2 #1755

Merged
merged 39 commits into from
Aug 23, 2021

Conversation

damarisg
Copy link
Member

@damarisg damarisg commented Aug 23, 2021

Related issue
Closes #1531

Settings

Package Details

Jenkins branch QA branch Version Revision
4.2 1516-4.2.0-full-green 4.2 1.1493
Type Format Architecture Version Revision Tag File name
Server rpm x86_64 4.2.0 40207 v4.2.0-rc8 wazuh-manager-4.2.0-1.1493.x86_64.rpm

Setup Local_internal_opcion.conf

wazuh_modules.debug=2
monitord.rotate_log=0

pytest_args

python3 -m pytest -ssv --html=report.html --self-contained-html test_vulnerability_detector/

Description

After research #1493, we saw that Vulnerability Detector tests were inconsistent.
Issue #1531 was created In order to fix these tests cases.

Linux

Linux tests are working properly with no errors ahead.

Test Results

Test Executions Date By Status
test_vulnerability_detector_local_r1.log 19-08-21 Miguel 🟢
test_vulnerability_detector_local_r2.log 19-08-21 Miguel 🟢
test_vulnerability_detector_local_r3.log 19-08-21 Miguel 🟢
Results - VulnDetect.log 19-08-21 Seyla 🟢
Results - VulnDetect.log 19-08-21 Seyla 🟢

pereyra-m and others added 30 commits July 14, 2021 11:31
Set 2020 as the minimum update year for redhat, nvd and msu feeds.

Closes: #1542
Delete duplicate lines that stop and start the wazuh-db service again.

Closes: #1542
Change year of feed update in `test_download_feeds.py`
Clear log file before run each use case in `test_invalid_syntax_redhat_feed.py`
Clear log file before run each use case in `test_invalid_values_redhat_feed.py`
Clear log file before run each use case in `test_missing_fields_redhat_feed.py`
fix: Clear log file before run each use case

Closes: #1644
…-time

Enable V.D. at startup to create the CVE database in `test_general_settings_ignore_time.py`
Apply correct formatting to `wazuh_providers_multiple_providers.yaml`
fix: Reduce the update interval of the NVD feed
refac: Remove unnecessary modulesd service restarting
refac: Reduce log-monitor timeout

Closes: #1565
The reason is that for the log required by the test to be written,
this table must not be empty.

Closes: #1565
Fix and optimize: `test_cpe_indexing.py`
Timeout Extended to Vulnerability Detector Tests
@damarisg damarisg changed the title 1531 full yellow vuln det Full Yellow Local - Fix Vulnerability Detector tests in 4.2 Aug 23, 2021
@snaow
Copy link
Contributor

snaow commented Aug 23, 2021

LGTM.
Note: Many skipped tests cause all the QA and product issues already discussed in #1531.
It is mandatory for us to rework and reenable those tests in next iterations.

Status Reports
report_vdt_scan_results 🟢 report_vdt_scan_results-r1.html.txt
report_vdt_feeds 🟢 report_vdt_feeds-r1.html.txt
report_vdt_windows 🟢 report_vdt_windows-r1.html.txt
report_vdt_providers 🟢 report_vdt_providers-r1.html.txt
report_vdt_general_settings 🟢 report_vdt_general_settings-r1.html.txt

@snaow snaow merged commit 43a6d78 into 1516-4.2.0-full-green Aug 23, 2021
@snaow snaow deleted the 1531-full-yellow-vuln-det branch August 23, 2021 17:41
TomasTurina pushed a commit that referenced this pull request Sep 1, 2021
Full Yellow Local - Fix Vulnerability Detector tests in 4.2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants