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

[RFR] Fix Test: test_infra_report_page_per_item #8003

Merged
merged 2 commits into from Oct 18, 2018

Conversation

valaparthvi
Copy link
Contributor

@valaparthvi valaparthvi commented Oct 17, 2018

This PR introduces changes and fixes to the test test_infra_report_page_per_item. Due to a BZ: 1616201, the test was failing. The BZ has now been fixed on version 5.10.0.20. The new changes include:

  1. Uncollect the test if appliance.version < 5.10.0.20.
  2. Add a new fixture get_report to queue a report as a part of setup and delete it as a part of teardown. The test would fail otherwise.
  3. Navigate to correct page and use correct paginator.
  4. Assert if the value of items_per_page is according to the value set on settings page.

{{pytest: cfme/tests/configure/test_visual_infra.py::test_infra_report_page_per_item --use-template-cache -sqvvv}}

@valaparthvi valaparthvi changed the title Fix Test: test_infra_report_page_per_item [WIPTEST] Fix Test: test_infra_report_page_per_item Oct 17, 2018
@valaparthvi valaparthvi changed the title [WIPTEST] Fix Test: test_infra_report_page_per_item [RFR] Fix Test: test_infra_report_page_per_item Oct 17, 2018
@mshriver mshriver merged commit 80f9a52 into ManageIQ:master Oct 18, 2018
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.

None yet

2 participants