yahoo-eng-team team mailing list archive

[Steve Martinelli <stevemar@xxxxxxxxxx>]

i think the testing here is sufficient
https://github.com/openstack/keystone/blob/5de11025f45ca10d2b02b70234c1305fae91766c/keystone/tests/unit/test_backend.py#L1101-L1148
and covers a lot of what was in the abandoned patch

** Changed in: keystone
       Status: In Progress => Invalid

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Identity (keystone).
https://bugs.launchpad.net/bugs/1367480

Title:
  Add test for grant CRUD on test_backend

Status in OpenStack Identity (keystone):
  Invalid

Bug description:
  The fact of not having tests for this may cause some bugs.
  For example, in KVS backend we call a non existent method and it would be avoided if we had such tests. [1]

  [1]
  https://github.com/openstack/keystone/blame/master/keystone/assignment/backends/kvs.py#L512

To manage notifications about this bug go to:
https://bugs.launchpad.net/keystone/+bug/1367480/+subscriptions