Update test cases for test_bay.py and test_baymodel.py

Bug #1481765 reported by Guangya Liu (Jay Lau)
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Magnum
Fix Released
Undecided
Guangya Liu (Jay Lau)

Bug Description

This is a follow up for the following gerrit review:
https://review.openstack.org/#/c/208137/1/magnum/tests/unit/api/controllers/v1/test_bay.py
https://review.openstack.org/#/c/208137/1/magnum/tests/unit/api/controllers/v1/test_baymodel.py

Planning to use following code style to make the code looks more cleaner.

- self.assertIn('name', response)
- self.assertIn('baymodel_id', response)
- self.assertIn('node_count', response)
- self.assertIn('status', response)
- self.assertIn('discovery_url', response)
- self.assertIn('api_address', response)
- self.assertIn('node_addresses', response)
+ for key in ("name", "baymodel_id", "node_count", "status",
+ "api_address", "discovery_url", "node_addresses"):
+ self.assertIn(key, response)

Changed in magnum:
assignee: nobody → Jay Lau (jay-lau-513)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to magnum (master)

Reviewed: https://review.openstack.org/209078
Committed: https://git.openstack.org/cgit/openstack/magnum/commit/?id=d3a6091519164cca4350d9ab082331ca05421dcf
Submitter: Jenkins
Branch: master

commit d3a6091519164cca4350d9ab082331ca05421dcf
Author: Jay Lau (Guangya Liu) <email address hidden>
Date: Tue Aug 4 09:56:55 2015 -0400

    Update test cases for test_bay.py and test_baymodel.py

    This is a follow up for the following gerrit review:
    https://review.openstack.org/#/c/208137/1/magnum/tests/unit/api/controllers/v1/test_bay.py
    https://review.openstack.org/#/c/208137/1/magnum/tests/unit/api/controllers/v1/test_baymodel.py

    Closes-Bug: #1481765

    Change-Id: Ifa134b6da6fadf40e388aee2c6168d777e112cab

Changed in magnum:
status: New → Fix Committed
Adrian Otto (aotto)
Changed in magnum:
milestone: none → mitaka-1
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.