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

[1LP][RFR ] - Fix testcase #10341

Merged
merged 1 commit into from Nov 20, 2020
Merged

Conversation

dgaikwad
Copy link
Contributor

@dgaikwad dgaikwad commented Oct 27, 2020

Purpose or Intent

  • Adding test BZ-1879912 blocker for test_vm_migrate test case
  • Fixed test_replication_powertoggle test case. ( testcase was checking OSP instance on the different provider and we were creating instance on diffrent provider.)

PRT Run

{{ pytest: cfme/tests/test_replication.py::test_replication_powertoggle cfme/tests/infrastructure/test_vm_migrate.py::test_vm_migrate -svvvv --long-running --use-template-cache
}}

@dgaikwad dgaikwad changed the title [NOTEST]Add blocker [WIPTEST ] - Fix testcase Oct 28, 2020
@dgaikwad
Copy link
Contributor Author

dgaikwad commented Nov 6, 2020

PRT is running on extra test too due to some unknown reason but my test case is passing successfully.

@dgaikwad dgaikwad changed the title [WIPTEST ] - Fix testcase [RFR ] - Fix testcase Nov 6, 2020
@dgaikwad dgaikwad changed the title [RFR ] - Fix testcase [WIPTEST ] - Fix testcase Nov 6, 2020
@dgaikwad dgaikwad changed the title [WIPTEST ] - Fix testcase [RFR ] - Fix testcase Nov 20, 2020
Copy link
Contributor

@valaparthvi valaparthvi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Optionally you could look into setup_remote_provider fixture, but this looks good for now.

@valaparthvi valaparthvi changed the title [RFR ] - Fix testcase [1LP][RFR ] - Fix testcase Nov 20, 2020
@valaparthvi
Copy link
Contributor

Moved to 1LP since the test in question passed PRT.

@mshriver mshriver merged commit e73976d into ManageIQ:master Nov 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants