Skip to content
This repository has been archived by the owner on Apr 7, 2022. It is now read-only.

[RFR] Fix test failure #9731

Merged
merged 2 commits into from
Dec 12, 2019
Merged

Conversation

valaparthvi
Copy link
Contributor

@valaparthvi valaparthvi commented Dec 5, 2019

Purpose or Intent

  • Fixing
    1. test_create_picture_with_role[individual
    2. webui/test_rest.py

PRT Run

{{ pytest: cfme/tests -k "test_create_picture_with_role or test_delete_advanced_search_filter_from_detail or test_delete_advanced_search_filter_from_collection" -vvv }}

@valaparthvi valaparthvi changed the title [WIPTEST] Fix config_management_rest tests [WIPTEST] Fix test failure Dec 5, 2019
@valaparthvi valaparthvi force-pushed the fix_config_test branch 5 times, most recently from ff65232 to 18893f5 Compare December 10, 2019 09:04
@dajoRH
Copy link
Contributor

dajoRH commented Dec 10, 2019

I detected some fixture changes in commit 18893f5e3ae8c21a05a2e12099065a5186ac02be

The local fixture config_manager_rest is used in the following files:

  • cfme/tests/infrastructure/test_config_management_rest.py
    • test_config_manager_create_rest
    • test_config_manager_edit_rest
    • test_config_manager_delete_rest

Please, consider creating a PRT run to make sure your fixture changes do not break existing usage 😃

@valaparthvi valaparthvi changed the title [WIPTEST] Fix test failure [RFR] Fix test failure Dec 10, 2019
@john-dupuy
Copy link
Contributor

@valaparthvi changes LGTM but these will also be fixed by #9317

@valaparthvi
Copy link
Contributor Author

Hey @john-dupuy I see this PR will bring some major changes. But, since the PR is big, I'm assuming it would take some time to get merged, so can we carry on with these fixes for now?

@dajoRH dajoRH changed the title [RFR] Fix test failure [WIP] Fix test failure Dec 10, 2019
@valaparthvi valaparthvi changed the title [WIP] Fix test failure [WIPTEST] Fix test failure Dec 11, 2019
@dajoRH dajoRH added WIP-testing and removed WIP labels Dec 11, 2019
@valaparthvi valaparthvi changed the title [WIPTEST] Fix test failure [RFR] Fix test failure Dec 11, 2019
@valaparthvi
Copy link
Contributor Author

Hey @john-dupuy I removed the changes related to config_manager, can we take this PR further now?

@izapolsk izapolsk merged commit 07359fb into ManageIQ:master Dec 12, 2019
spusateri pushed a commit to spusateri/integration_tests that referenced this pull request Jan 27, 2020
* Fix webui/test_rest.py

* Fix test test_create_picture_with_role
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants