Magnum

Tech Debt: Add unit test to local cert manager

Bug #1494579 reported by yuanying
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Magnum
Fix Released
Wishlist
yuanying
Magnum mitaka-1

Bug Description

some of unit test seems not cover negative cases, like raise StorageCertificatieException cases and so on.

* https://review.openstack.org/#/c/212395/7/magnum/tests/unit/common/cert_manager/test_local.py

Tags: tech-debt
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to magnum (master)

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

Changed in magnum:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to magnum (master)

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

commit 22b83b09497afcebfa5f63daf487531f64804a37
Author: OTSUKA, Yuanying <email address hidden>
Date: Fri Sep 11 14:24:58 2015 +0900

    Add test to local_cert_manager

    Some of unit test seems not cover negative cases, like
    raise StorageCertificatieException cases and so on.
    This patch adds unit test for this.

    Change-Id: I29f8b4bc36fce91169b492125b760231fb5faea8
    Closes-bug: #1494579

Changed in magnum:
status: In Progress → 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.
Subscribing...

Other bug subscribers

Remote bug watches

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