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

[6.13.z] Fix get_downloads_list which is causing save_downloaded_file to fail #1102

Conversation

Satellite-QE
Copy link
Contributor

Cherrypick of PR: #1098

Fix get_downloads_list which is causing save_downloaded_file to fail. Fixed issue and removed some unnecessary code.

tests/foreman/ui/test_rhcloud_inventory.py:209: in test_rh_cloud_inventory_settings
    report_path = session.cloudinventory.download_report(org.name)
../../lib64/python3.11/site-packages/airgun/entities/cloud_inventory.py:43: in download_report
    return self.browser.save_downloaded_file()
../../lib64/python3.11/site-packages/airgun/browser.py:430: in save_downloaded_file
    file_uri = files[0]
E   IndexError: list index out of range

@Satellite-QE Satellite-QE added 6.13.z Auto_Cherry_Picked GHA has automatically cherrypicked this PR No-CherryPick PR doesnt need CherryPick to previous branches labels Jan 3, 2024
@Satellite-QE
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_rhcloud_inventory.py -k test_rh_cloud_inventory_settings

@Satellite-QE Satellite-QE added the AutoMerge_Cherry_Picked Automatically merge the PR is PRT and all checks are passing label Jan 3, 2024
@omkarkhatavkar omkarkhatavkar merged commit b6f5a89 into 6.13.z Jan 4, 2024
13 of 14 checks passed
@ogajduse ogajduse deleted the cherry-pick-6.13.z-b72a45d59096212cadb4bea298f3a001af1f0731 branch January 16, 2024 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.13.z Auto_Cherry_Picked GHA has automatically cherrypicked this PR AutoMerge_Cherry_Picked Automatically merge the PR is PRT and all checks are passing No-CherryPick PR doesnt need CherryPick to previous branches
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants