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

Add more verbose output from pytest suite #694

Merged
merged 3 commits into from
Jul 4, 2024
Merged

Conversation

phracek
Copy link
Member

@phracek phracek commented Jun 26, 2024

This pull request only adds more verbose output from pytest module.

We would like to see what's happened during the tests.

Signed-off-by: Petr "Stone" Hracek <phracek@redhat.com>
@phracek
Copy link
Member Author

phracek commented Jun 26, 2024

[test-openshift-pytest]

@phracek
Copy link
Member Author

phracek commented Jun 27, 2024

The output looks great. See https://artifacts.osci.redhat.com/testing-farm/8a939e48-ed7d-4491-83c2-ae79c8620f39/. We can see progress of the tests and 'short test summary info' as well.

=========================== short test summary info ============================
PASSED test_deploy_templates.py::TestDeployTemplate::()::test_python_template_inside_cluster[django-postgresql.json]
PASSED test_deploy_templates.py::TestDeployTemplate::()::test_python_template_inside_cluster[django-postgresql-persistent.json]
PASSED test_imagestreams_quickstart.py::TestImagestreamsQuickstart::()::test_python_template_inside_cluster[django-postgresql.json]
PASSED test_imagestreams_quickstart.py::TestImagestreamsQuickstart::()::test_python_template_inside_cluster[django-postgresql-persistent.json]
PASSED test_python_ex_standalone.py::TestPythonExTemplate::()::test_python_ex_template_inside_cluster
PASSED test_python_ex_template.py::TestPythonExTemplate::()::test_python_ex_template_inside_cluster

@phracek
Copy link
Member Author

phracek commented Jun 27, 2024

In case there is something, that I have missing from PyTest arguments, improvements are welcome, as commented in this pull request.

Signed-off-by: Petr "Stone" Hracek <phracek@redhat.com>
@phracek
Copy link
Member Author

phracek commented Jun 28, 2024

Le't run all tests. For sure.

[test-all]

@phracek
Copy link
Member Author

phracek commented Jul 2, 2024

@frenzymadness It is interesting issue: Testing Farm - RHEL8 - PyTest - OpenShift 4 - 3.11 and in logs https://artifacts.osci.redhat.com/testing-farm/b786a35c-ed0a-457e-a35a-6e622cf2a4da/ at the end is no 'summary info'
But in Testing Farm - RHEL9 - PyTest - OpenShift 4 - 3.11 and in logs https://artifacts.osci.redhat.com/testing-farm/866f4180-ce8f-47b2-a65d-b6fc245897dc/ at the end is summary:

=========================== short test summary info ============================
PASSED test_deploy_templates.py::TestDeployTemplate::test_python_template_inside_cluster[django-postgresql.json]
PASSED test_deploy_templates.py::TestDeployTemplate::test_python_template_inside_cluster[django-postgresql-persistent.json]
PASSED test_imagestreams_quickstart.py::TestImagestreamsQuickstart::test_python_template_inside_cluster[django-postgresql.json]
PASSED test_imagestreams_quickstart.py::TestImagestreamsQuickstart::test_python_template_inside_cluster[django-postgresql-persistent.json]
PASSED test_python_ex_standalone.py::TestPythonExTemplate::test_python_ex_template_inside_cluster
PASSED test_python_ex_template.py::TestPythonExTemplate::test_python_ex_template_inside_cluster
================== 6 passed, 4 warnings in 651.28s (0:10:51) ===================

@frenzymadness
Copy link
Member

If you use main Python (3.6 in RHEL 8, 3.9 in RHEL 9) it might also lead to different pytest versions being installed due to newest pytest not supporting Python 3.6.

@pkubatrh
Copy link
Member

pkubatrh commented Jul 3, 2024

Let's move to latest Python and use rpm-based pytest if possible on all RHEL versions.

@phracek
Copy link
Member Author

phracek commented Jul 3, 2024

Signed-off-by: Petr "Stone" Hracek <phracek@redhat.com>
@phracek
Copy link
Member Author

phracek commented Jul 3, 2024

@phracek
Copy link
Member Author

phracek commented Jul 4, 2024

@phracek
Copy link
Member Author

phracek commented Jul 4, 2024

[test-openshift-pytest]

@phracek phracek merged commit f250582 into master Jul 4, 2024
5 checks passed
@phracek phracek deleted the add_verbose_output_pytest branch July 4, 2024 10:35
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants