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

[1LP][RFR] Fixing the type error #9635

Merged
merged 1 commit into from
Nov 18, 2019

Conversation

mnadeem92
Copy link
Contributor

get_vm method do not have any param name template
Fixing below error
TypeError: get_vm() got an unexpected keyword argument 'template'

Signed-off-by: mnadeem92 [email protected]

get_vm method do not have any param name template
Fixing below error
TypeError: get_vm() got an unexpected keyword argument 'template'

Signed-off-by: mnadeem92 <[email protected]>
@dajoRH dajoRH added the lint-ok label Nov 18, 2019
@dajoRH
Copy link
Contributor

dajoRH commented Nov 18, 2019

I detected some fixture changes in commit bf5777a

The global fixture mapping_data_vm_obj_mini is used in the following files:

  • cfme/tests/v2v/test_post_migrations.py
    • test_migration_post_attribute
  • cfme/tests/v2v/test_v2v_cancel_migrations.py
    • cancel_migration_plan
  • cfme/tests/v2v/test_v2v_migrations_single_vcenter.py
  • cfme/tests/v2v/test_v2v_migrations_ui.py
    • test_duplicate_plan_name
    • test_duplicate_mapping_name
  • cfme/tests/v2v/test_v2v_schedule_migrations.py
    • test_schedule_migration

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

@valaparthvi valaparthvi changed the title [RFR] Fixing the type error [1LP][RFR] Fixing the type error Nov 18, 2019
@izapolsk izapolsk merged commit a2e5d6a into ManageIQ:master Nov 18, 2019
spusateri pushed a commit to spusateri/integration_tests that referenced this pull request Jan 27, 2020
get_vm method do not have any param name template
Fixing below error
TypeError: get_vm() got an unexpected keyword argument 'template'

Signed-off-by: mnadeem92 <[email protected]>
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