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

[1LP][RFR] Quota test fixes #10085

Merged
merged 1 commit into from
May 6, 2020
Merged

[1LP][RFR] Quota test fixes #10085

merged 1 commit into from
May 6, 2020

Conversation

tpapaioa
Copy link
Contributor

@tpapaioa tpapaioa commented Apr 29, 2020

This PR makes some changes to quota tests, including:

1.) Renamed fixtures for readability and clarity:

roottenant -> root_tenant
set_roottenant_quota -> set_root_tenant_quota
check_hosts -> migration_destination_host

2.) Use create_vm_modscope fixture when provisioning VMs.
3.) Use f-strings instead of format().

One of the tests is failing in PRT, but it appears to be due to an intermittent bug in CFME that I haven't been able to reproduce reliably.

{{ pytest: -v --long-running cfme/tests/infrastructure/test_tenant_quota.py }}

@dajoRH
Copy link
Contributor

dajoRH commented Apr 29, 2020

I detected some fixture changes in commit 1eaf033

The local fixture set_root_tenant_quota is used in the following files:

  • cfme/tests/infrastructure/test_tenant_quota.py
    • test_tenant_quota_enforce_via_lifecycle_infra
    • test_tenant_quota_enforce_via_service_infra
    • test_tenant_quota_vm_reconfigure

The local fixture migration_destination_host is used in the following files:

  • cfme/tests/infrastructure/test_tenant_quota.py

The local fixture quota_limit is used in the following files:

  • cfme/tests/infrastructure/test_tenant_quota.py
    • test_quota_with_reconfigure_resize_disks

The local fixture configure_mail is used in the following files:

  • cfme/tests/infrastructure/test_tenant_quota.py
    • test_quota_exceed_mail_with_more_info_link

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

@tpapaioa tpapaioa changed the title [WIPTEST] Quota test fixes [RFR] Quota test fixes Apr 30, 2020
@john-dupuy john-dupuy added fix-test test-cleanup Test removal, collection changes, re-organization labels May 5, 2020
@john-dupuy john-dupuy changed the title [RFR] Quota test fixes [1LP][RFR] Quota test fixes May 5, 2020
@john-dupuy
Copy link
Contributor

PRT failures do not appear to be related to this PR.

@mshriver mshriver merged commit d3f70f6 into ManageIQ:master May 6, 2020
@tpapaioa tpapaioa deleted the fix_quota_tests branch May 6, 2020 20:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
fix-test lint-ok test-cleanup Test removal, collection changes, re-organization
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants