Test: Bad method has been mocked

[Florent Flament]

Method `api_client.alarms` has been mocked in the following test:
https://github.com/openstack/ceilometer/blob/master/ceilometer/tests/alarm/partition/test_coordination.py#L421

However, the `api_client.alarms.list` method is called from `assigned_alarms`, called by the test:
https://github.com/openstack/ceilometer/blob/master/ceilometer/alarm/partition/coordination.py#L303

The test passes because the `api_client.alarms.list` call raises a `TypeError: "'Mock' object is not iterable"`, which is caught by the `except Exception` catchall. Instead an `Exception: Exception('Boom!',)` should be raised and caught.

In the test:
https://github.com/openstack/ceilometer/blob/master/ceilometer/tests/alarm/partition/test_coordination.py#L421
The method `api_client.alarms.list` should be mocked instead of `api_client.alarms`.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/99890

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/99890
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=3ae3d272d41ef7a5bac6bfeb8bad430b495c2e93
Submitter: Jenkins
Branch: master

commit 3ae3d272d41ef7a5bac6bfeb8bad430b495c2e93
Author: florent <email address hidden>
Date: Fri Jun 13 09:59:09 2014 +0000

    Fix method mocked in a test

    Mock `api_client.alarms.list` instead of `api_client.alarms` in the
    tests/alarm/partition/test_coordination.py:TestCoordinate.test_assigned_alarms
    method.

    Change-Id: Id33f9854de4604b47b2a453c91a58f2dcd15e682
    Closes-Bug: #1329701