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

added the missing directory path for ui_session #13212

Merged
merged 1 commit into from
Nov 29, 2023

Conversation

omkarkhatavkar
Copy link

Problem Statement

Some of the directories were missing for adding the record_property to test folders. This causes failure in the tests.

Solution

Adding the directory paths fixes this issue now.

@omkarkhatavkar omkarkhatavkar requested a review from a team as a code owner November 29, 2023 11:59
@omkarkhatavkar omkarkhatavkar added the CherryPick PR needs CherryPick to previous branches label Nov 29, 2023
@omkarkhatavkar omkarkhatavkar added 6.12.z Introduced in or relating directly to Satellite 6.12 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 Nov 29, 2023
@omkarkhatavkar
Copy link
Author

/Users/okhatavk/satellite/robottelo/env/bin/python /Applications/PyCharm CE.app/Contents/plugins/python-ce/helpers/pycharm/_jb_pytest_runner.py --path /Users/okhatavk/satellite/robottelo/tests/foreman/sanity/test_bvt.py -- -k test_all_interfaces_are_accessible --junitxml=report.xml 
Testing started at 5:09 pm ...
Launching pytest with arguments -k test_all_interfaces_are_accessible --junitxml=report.xml /Users/okhatavk/satellite/robottelo/tests/foreman/sanity/test_bvt.py --no-header --no-summary -q in /Users/okhatavk/satellite/robottelo

============================= test session starts ==============================
collecting ... collected 2 items / 1 deselected / 1 selected

tests/foreman/sanity/test_bvt.py::test_all_interfaces_are_accessible 

=========== 1 passed, 1 deselected, 11 warnings in 176.08s (0:02:56) ===========

@jyejare
Copy link
Member

jyejare commented Nov 29, 2023

Merging this on priority with T2 ACK!

@jyejare jyejare merged commit 2fa446e into SatelliteQE:master Nov 29, 2023
10 checks passed
github-actions bot pushed a commit that referenced this pull request Nov 29, 2023
github-actions bot pushed a commit that referenced this pull request Nov 29, 2023
github-actions bot pushed a commit that referenced this pull request Nov 29, 2023
omkarkhatavkar added a commit to omkarkhatavkar/robottelo that referenced this pull request Nov 29, 2023
omkarkhatavkar added a commit to omkarkhatavkar/robottelo that referenced this pull request Nov 29, 2023
@omkarkhatavkar omkarkhatavkar changed the title added the missing direcotry path for ui_session added the missing directory path for ui_session Nov 29, 2023
omkarkhatavkar added a commit to omkarkhatavkar/robottelo that referenced this pull request Nov 29, 2023
Gauravtalreja1 pushed a commit that referenced this pull request Nov 29, 2023
added the missing direcotry path for ui_session (#13212)

(cherry picked from commit 2fa446e)

Co-authored-by: Omkar Khatavkar <[email protected]>
Gauravtalreja1 pushed a commit that referenced this pull request Nov 29, 2023
added the missing direcotry path for ui_session (#13212)

(cherry picked from commit 2fa446e)

Co-authored-by: Omkar Khatavkar <[email protected]>
Gauravtalreja1 pushed a commit that referenced this pull request Nov 29, 2023
added the missing direcotry path for ui_session (#13212)

(cherry picked from commit 2fa446e)

Co-authored-by: Omkar Khatavkar <[email protected]>
shweta83 pushed a commit to shweta83/robottelo that referenced this pull request Apr 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.12.z Introduced in or relating directly to Satellite 6.12 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 CherryPick PR needs CherryPick to previous branches
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants