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 test coverage for SAT-28860 #17153

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Gauravtalreja1
Copy link
Collaborator

Problem Statement

Missing test coverage for new feature SAT-28860

Solution

Add test coverage for new feature SAT-28860

Related Issues

SatelliteQE/airgun#1636
SatelliteQE/nailgun#1254

Signed-off-by: Gaurav Talreja <[email protected]>
@Gauravtalreja1 Gauravtalreja1 added QETestCoverage Issues and PRs relating to a Satellite bug No-CherryPick PR doesnt need CherryPick to previous branches Stream Introduced in or relating directly to Satellite Stream/Master labels Dec 11, 2024
@Gauravtalreja1 Gauravtalreja1 self-assigned this Dec 11, 2024
@Gauravtalreja1 Gauravtalreja1 requested review from a team as code owners December 11, 2024 21:53
@Gauravtalreja1
Copy link
Collaborator Author

trigger: test-robottelo
pytest: tests/foreman/api/test_http_proxy.py::test_positive_end_to_end tests/foreman/api/test_http_proxy.py::test_positive_install_content_with_http_proxy tests/foreman/ui/test_http_proxy.py::test_positive_set_default_http_proxy

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9595
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_http_proxy.py::test_positive_end_to_end tests/foreman/api/test_http_proxy.py::test_positive_install_content_with_http_proxy tests/foreman/ui/test_http_proxy.py::test_positive_set_default_http_proxy --external-logging
Test Result : ============ 1 failed, 60 warnings, 5 errors in 1723.27s (0:28:43) =============

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Dec 11, 2024
@Gauravtalreja1
Copy link
Collaborator Author

trigger: test-robottelo
pytest: tests/foreman/api/test_http_proxy.py::test_positive_end_to_end tests/foreman/api/test_http_proxy.py::test_positive_install_content_with_http_proxy tests/foreman/ui/test_http_proxy.py::test_positive_set_default_http_proxy
nailgun: 1254
airgun: 1636

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9596
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_http_proxy.py::test_positive_end_to_end tests/foreman/api/test_http_proxy.py::test_positive_install_content_with_http_proxy tests/foreman/ui/test_http_proxy.py::test_positive_set_default_http_proxy --external-logging
Test Result : ============ 1 failed, 94 warnings, 5 errors in 1960.40s (0:32:40) =============

@Gauravtalreja1 Gauravtalreja1 marked this pull request as draft December 17, 2024 15:12
@vsedmik
Copy link
Contributor

vsedmik commented Dec 20, 2024

PRT failed in http_proxy = target_sat.api_factory.make_http_proxy()
with

00:40:38  >               setattr(entity, field_name, attrs[field_name])
00:40:38  E               KeyError: 'default_content_http_proxy'

See SatelliteQE/nailgun#1254 (comment)

@Gauravtalreja1
Copy link
Collaborator Author

@vsedmik yes, it'll be fixed in Katello/katello#11266

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No-CherryPick PR doesnt need CherryPick to previous branches PRT-Failed Indicates that latest PRT run is failed for the PR QETestCoverage Issues and PRs relating to a Satellite bug Stream Introduced in or relating directly to Satellite Stream/Master
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants