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.

Sign up for GitHub

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

Jump to bottom

[Test Fix] test_negative_generate_hostpkgcompare_nonexistent_host #14068

Merged
merged 1 commit into from
Feb 14, 2024
Merged

[Test Fix] test_negative_generate_hostpkgcompare_nonexistent_host #14068

merged 1 commit into from
Feb 14, 2024

Conversation

ColeHiggins2
Copy link
Member

Problem Statement

test_negative_generate_hostpkgcompare_nonexistent_host is failing due to our of date attributes in assertion

Solution

Change cm.exeption.stderr to cm.value.stderr

@ColeHiggins2 ColeHiggins2 added Easy Fix :) Easiest Fix to review and quick merge request. CherryPick PR needs CherryPick to previous branches 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 labels Feb 13, 2024
@ColeHiggins2 ColeHiggins2 self-assigned this Feb 13, 2024
@ColeHiggins2 ColeHiggins2 requested a review from a team as a code owner February 13, 2024 20:50
@ColeHiggins2
Copy link
Member Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_reporttemplates.py::test_negative_generate_hostpkgcompare_nonexistent_host

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 5707
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/cli/test_reporttemplates.py::test_negative_generate_hostpkgcompare_nonexistent_host
Test Result : ================== 1 passed, 11 warnings in 733.00s (0:12:12) ==================

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Feb 13, 2024
@synkd synkd merged commit 51ad778 into SatelliteQE:master Feb 14, 2024
12 checks passed
github-actions bot pushed a commit that referenced this pull request Feb 14, 2024
@ColeHiggins2 @web-flow
[Test Fix] test_negative_generate_hostpkgcompare_nonexistent_host (#1…
fee2d2a 
…4068)

Updating Assertion for test fix

(cherry picked from commit 51ad778)
@Satellite-QE Satellite-QE mentioned this pull request Feb 14, 2024
Merged
github-actions bot pushed a commit that referenced this pull request Feb 14, 2024
@ColeHiggins2 @web-flow
[Test Fix] test_negative_generate_hostpkgcompare_nonexistent_host (#1…
53bf98b 
…4068)

Updating Assertion for test fix

(cherry picked from commit 51ad778)
@Satellite-QE Satellite-QE mentioned this pull request Feb 14, 2024
Merged
Gauravtalreja1 pushed a commit that referenced this pull request Feb 14, 2024
@Satellite-QE @ColeHiggins2
[6.15.z] [Test Fix] test_negative_generate_hostpkgcompare_nonexistent…
4de97c7 
…_host (#14079)

[Test Fix] test_negative_generate_hostpkgcompare_nonexistent_host (#14068)

Updating Assertion for test fix

(cherry picked from commit 51ad778)

Co-authored-by: Cole Higgins <[email protected]>
Gauravtalreja1 pushed a commit that referenced this pull request Feb 14, 2024
@Satellite-QE @ColeHiggins2
[6.14.z] [Test Fix] test_negative_generate_hostpkgcompare_nonexistent…
a928ca2 
…_host (#14078)

[Test Fix] test_negative_generate_hostpkgcompare_nonexistent_host (#14068)

Updating Assertion for test fix

(cherry picked from commit 51ad778)

Co-authored-by: Cole Higgins <[email protected]>
shweta83 pushed a commit to shweta83/robottelo that referenced this pull request Apr 10, 2024
@ColeHiggins2 @shweta83
[Test Fix] test_negative_generate_hostpkgcompare_nonexistent_host (Sa…
380d1b7 
…telliteQE#14068)

Updating Assertion for test fix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@synkd synkd synkd approved these changes

@lhellebr lhellebr lhellebr approved these changes

Assignees

@ColeHiggins2 ColeHiggins2

Labels
6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 CherryPick PR needs CherryPick to previous branches Easy Fix :) Easiest Fix to review and quick merge request. PRT-Passed Indicates that latest PRT run is passed for the PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@ColeHiggins2 @Satellite-QE @lhellebr @synkd