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

Fix CLI import puppet class test #15650

Merged
merged 1 commit into from
Jul 12, 2024
Merged

Fix CLI import puppet class test #15650

merged 1 commit into from
Jul 12, 2024

Conversation

Griffin-Sullivan
Copy link
Contributor

Problem Statement

This test is failing but the API version which is very similar isn't.

Solution

Add missing fixture setup.

Related Issues

trigger: test-robottelo
pytest: tests/foreman/cli/test_capsule.py::test_positive_import_puppet_classes

@Griffin-Sullivan Griffin-Sullivan added CherryPick PR needs CherryPick to previous branches AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing 6.13.z Introduced in or relating directly to Satellite 6.13 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 Jul 12, 2024
@Griffin-Sullivan Griffin-Sullivan requested a review from a team as a code owner July 12, 2024 18:24
@Griffin-Sullivan
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_capsule.py::test_positive_import_puppet_classes

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7735
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/cli/test_capsule.py::test_positive_import_puppet_classes --external-logging
Test Result : ================= 1 passed, 13 warnings in 1780.70s (0:29:40) ==================

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Jul 12, 2024
@Gauravtalreja1 Gauravtalreja1 added the Easy Fix :) Easiest Fix to review and quick merge request. label Jul 12, 2024
@Gauravtalreja1 Gauravtalreja1 merged commit c51873d into SatelliteQE:master Jul 12, 2024
15 checks passed
github-actions bot pushed a commit that referenced this pull request Jul 12, 2024
@Griffin-Sullivan @web-flow
Fix CLI import puppet class test (#15650)
1168c77 
(cherry picked from commit c51873d)
@Satellite-QE Satellite-QE mentioned this pull request Jul 12, 2024
Merged
github-actions bot pushed a commit that referenced this pull request Jul 12, 2024
@Griffin-Sullivan @web-flow
Fix CLI import puppet class test (#15650)
95118f5 
(cherry picked from commit c51873d)
@Satellite-QE Satellite-QE mentioned this pull request Jul 12, 2024
Merged
github-actions bot pushed a commit that referenced this pull request Jul 12, 2024
@Griffin-Sullivan @web-flow
Fix CLI import puppet class test (#15650)
7c96954 
(cherry picked from commit c51873d)
@Satellite-QE Satellite-QE mentioned this pull request Jul 12, 2024
Merged
jyejare pushed a commit to jyejare/robottelo that referenced this pull request Oct 19, 2024
@Griffin-Sullivan @jyejare
Fix CLI import puppet class test (SatelliteQE#15650)
ea165b7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Gauravtalreja1 Gauravtalreja1 Gauravtalreja1 approved these changes

Assignees

@Griffin-Sullivan Griffin-Sullivan

Labels
6.13.z Introduced in or relating directly to Satellite 6.13 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing 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.
3 participants
@Griffin-Sullivan @Satellite-QE @Gauravtalreja1