Test for list_users in test_backend is not safe

[Henry Nash]

The test for list_users in test_backend is written as:

    def test_list_users(self):
        users = self.identity_api.list_users()
        for test_user in default_fixtures.USERS:
            self.assertTrue(x for x in users if x['id'] == test_user['id'])

This is not safe. You can show this by, for instance, setting users = [] instead of the correct result from self.identity_api.list_users() and no asset is raised

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/45591
Committed: http://github.com/openstack/keystone/commit/2d9cdceadaca5ce44d7ae14c5371e2326128df64
Submitter: Jenkins
Branch: master

commit 2d9cdceadaca5ce44d7ae14c5371e2326128df64
Author: Henry Nash <email address hidden>
Date: Mon Sep 9 00:11:35 2013 +0100

    Fix incorrect test for list_users

    The current test would not fail if incorrect data was returned by
    the driver. The list_roles test also has the same issue.

    Fixes bug 1222574

    Change-Id: Icc5e76e9453436ac208c5f262d0dd4082b700d02