no unit tests for SQL catalog backend

Bug #958950 reported by yong sheng gong
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Fix Released
High
Dolph Mathews

Bug Description

There are test cases in test_backend_kvs.py to test kvs catalog backend, however test_backend_sql.py misses cases to test sql catalog backend.

We can see also on https://jenkins.openstack.org/view/Keystone/job/keystone-coverage/420/cobertura/keystone_catalog_backends/sql_py/ this backend is not covered.

affects: glance → keystone
Hua Zhang (zhhuabj)
Changed in keystone:
assignee: nobody → Zhang Hua (zhhuabj)
Joseph Heck (heckj)
Changed in keystone:
status: New → Confirmed
importance: Undecided → High
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to keystone (master)

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

Changed in keystone:
status: Confirmed → In Progress
Joseph Heck (heckj)
tags: removed: keystone
Revision history for this message
Dolph Mathews (dolph) wrote :

Picking this up, as the above patch was abandoned.

Changed in keystone:
assignee: Hua Zhang (zhhuabj) → Dolph Mathews (dolph)
milestone: none → folsom-3
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to keystone (master)

Reviewed: https://review.openstack.org/10682
Committed: http://github.com/openstack/keystone/commit/f82c7c22a83ac1b8cd6483ed3e9b4708c90a89ba
Submitter: Jenkins
Branch: master

commit f82c7c22a83ac1b8cd6483ed3e9b4708c90a89ba
Author: Dolph Mathews <email address hidden>
Date: Wed Aug 1 14:02:22 2012 -0500

    Enabling SQL Catalog tests (bug 958950)

    Change-Id: I9d33d95ffa357b88f099a5a37aa4a139d93fd82f

Changed in keystone:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in keystone:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in keystone:
milestone: folsom-3 → 2012.2
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.