search

thoth-station / report-processing

Library for processing all outputs/reports produced by Thoth components.
GNU General Public License v3.0
4 stars 8 forks source link

Fix tests and update data #151

Closed pacospace closed 2 years ago

pacospace commented 3 years ago

Signed-off-by: Francesco Murdaca fmurdaca@redhat.com

Related Issues and Dependencies

Fixes: https://github.com/thoth-station/report-processing/issues/150

pacospace commented 3 years ago

@pacospace: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command aicoe-ci/prow/pytest 55444f3 link true /test thoth-pytest-py38 Full PR test history. Your PR dashboard. Please help us and open an issue when you hit one in your PR.

Running pipenv run python3 setup.py test all tests passes, not sure why prow fails. It reports an error that is not there.

pacospace commented 3 years ago

/retest

pacospace commented 2 years ago

ok this is weird...

from prow:

============================= test session starts ==============================
platform linux -- Python 3.8.3, pytest-6.2.5, py-1.11.0, pluggy-1.0.0 -- /opt/app-root/bin/python3
cachedir: .pytest_cache
hypothesis profile 'default' -> database=DirectoryBasedExampleDatabase('/home/prow/go/src/github.com/thoth-station/report-processing/.hypothesis/examples')
rootdir: /home/prow/go/src/github.com/thoth-station/report-processing
plugins: hypothesis-6.27.1, cov-3.0.0, mypy-0.8.1, timeout-2.0.1
timeout: 45.0s
timeout method: signal
timeout func_only: False
collecting ... collected 27 items

tests/__init__.py::mypy PASSED
tests/__init__.py::mypy-status PASSED
tests/adviser_test.py::mypy PASSED
tests/adviser_test.py::TestAdviser::test_get_adviser_files PASSED
tests/adviser_test.py::TestAdviser::test_create_adviser_dataframe_with_results PASSED
tests/adviser_test.py::TestAdviser::test_create_adviser_dataframe_histogram PASSED
tests/adviser_test.py::TestAdviser::test_create_adviser_dataframe_heatmap PASSED
tests/base_test.py::mypy PASSED
tests/inspection_test.py::mypy PASSED
tests/inspection_test.py::TestInpsection::test_get_inspection_runs PASSED
tests/inspection_test.py::TestInpsection::test_create_inspection_summary PASSED
tests/inspection_test.py::TestInpsection::test_final_dataframe FAILED
tests/inspection_test.py::TestInpsection::test_filter_final_dataframe FAILED
tests/si_aggregator_test.py::mypy PASSED
tests/si_aggregator_test.py::TestSecurityReportsBandit::test_create_si_aggregated_dataframe PASSED
tests/si_aggregator_test.py::TestSecurityReportsBandit::test_create_si_aggregated_json PASSED
tests/si_bandit_test.py::mypy PASSED
tests/si_bandit_test.py::TestSecurityReportsBandit::test_get_security_indicator_bandit_report PASSED
tests/si_bandit_test.py::TestSecurityReportsBandit::test_get_metadata_df_from_bandit_report PASSED
tests/si_bandit_test.py::TestSecurityReportsBandit::test_get_severity_confidence_info_from_bandit_report PASSED
tests/si_bandit_test.py::TestSecurityReportsBandit::test_get_severity_confidence_info_df_from_bandit_report PASSED
tests/si_cloc_test.py::mypy PASSED
tests/si_cloc_test.py::TestSecurityReportsCloc::test_get_security_indicator_cloc_report PASSED
tests/si_cloc_test.py::TestSecurityReportsCloc::test_get_metadata_df_from_cloc_report PASSED
tests/solver_test.py::mypy PASSED
tests/solver_test.py::TestSolver::test_get_solver_reports PASSED
tests/solver_test.py::TestSolver::test_get_metadata_df_from_solver_report PASSED

from local machine:

platform linux -- Python 3.8.12, pytest-6.2.5, py-1.11.0, pluggy-1.0.0 -- /home/fmurdaca/.local/share/virtualenvs/report-processing-HiOqFbsr/bin/python3
cachedir: .pytest_cache
hypothesis profile 'default' -> database=DirectoryBasedExampleDatabase('/home/fmurdaca/work/aicoe/report-processing/.hypothesis/examples')
rootdir: /home/fmurdaca/work/aicoe/report-processing
plugins: timeout-2.0.1, hypothesis-6.27.1, mypy-0.8.1, cov-3.0.0
timeout: 45.0s
timeout method: signal
timeout func_only: False
collected 27 items                                                                                                                                               

tests/__init__.py::mypy PASSED
tests/__init__.py::mypy-status PASSED
tests/adviser_test.py::mypy PASSED
tests/adviser_test.py::TestAdviser::test_get_adviser_files PASSED
tests/adviser_test.py::TestAdviser::test_create_adviser_dataframe_with_results PASSED
tests/adviser_test.py::TestAdviser::test_create_adviser_dataframe_histogram PASSED
tests/adviser_test.py::TestAdviser::test_create_adviser_dataframe_heatmap PASSED
tests/base_test.py::mypy PASSED
tests/inspection_test.py::mypy PASSED
tests/inspection_test.py::TestInpsection::test_get_inspection_runs PASSED
tests/inspection_test.py::TestInpsection::test_create_inspection_summary PASSED
tests/inspection_test.py::TestInpsection::test_final_dataframe PASSED
tests/inspection_test.py::TestInpsection::test_filter_final_dataframe PASSED
tests/si_aggregator_test.py::mypy PASSED
tests/si_aggregator_test.py::TestSecurityReportsBandit::test_create_si_aggregated_dataframe PASSED
tests/si_aggregator_test.py::TestSecurityReportsBandit::test_create_si_aggregated_json PASSED
tests/si_bandit_test.py::mypy PASSED
tests/si_bandit_test.py::TestSecurityReportsBandit::test_get_security_indicator_bandit_report PASSED
tests/si_bandit_test.py::TestSecurityReportsBandit::test_get_metadata_df_from_bandit_report PASSED
tests/si_bandit_test.py::TestSecurityReportsBandit::test_get_severity_confidence_info_from_bandit_report PASSED
tests/si_bandit_test.py::TestSecurityReportsBandit::test_get_severity_confidence_info_df_from_bandit_report PASSED
tests/si_cloc_test.py::mypy PASSED
tests/si_cloc_test.py::TestSecurityReportsCloc::test_get_security_indicator_cloc_report PASSED
tests/si_cloc_test.py::TestSecurityReportsCloc::test_get_metadata_df_from_cloc_report PASSED
tests/solver_test.py::mypy PASSED
tests/solver_test.py::TestSolver::test_get_solver_reports PASSED
tests/solver_test.py::TestSolver::test_get_metadata_df_from_solver_report PASSED

cc @fridex, @harshad16

mayaCostantini commented 2 years ago

/approve

sesheta commented 2 years ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mayaCostantini

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/thoth-station/report-processing/blob/master/OWNERS)~~ [mayaCostantini] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment