x/valence
Code Issues Proposed changes

Merge "Improve response message of flavor deletion"

Browse Source
This commit is contained in:
Jenkins 2017-04-25 04:42:33 +00:00 committed by Gerrit Code Review
commit 4c3fd1d9fd
2 changed files with 10 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
valence/controller/flavors.py
View File

@ -14,6 +14,7 @@
import logging
from valence.common import exception
from valence.db import api as db_api
LOG = logging.getLogger(__name__)
@ -36,7 +37,10 @@ def create_flavor(values):
def delete_flavor(flavorid):
db_api.Connection.delete_flavor(flavorid)
return "Deleted flavor {0}".format(flavorid)
return exception.confirmation(
confirm_code="DELETED",
confirm_detail="This flavor {0} has been deleted successfully"
.format(flavorid))
def update_flavor(flavorid, values):

6
valence/tests/unit/controller/test_flavors.py
View File

@ -14,6 +14,7 @@ from unittest import TestCase
import mock
from valence.common import exception
from valence.controller import flavors
from valence.tests.unit.fakes import flavor_fakes as fakes
@ -34,7 +35,10 @@ class TestFlavors(TestCase):
@mock.patch('valence.db.api.Connection.delete_flavor')
def test_delete_flavor(self, mock_db_delete_flavor):
expected = "Deleted flavor 00000000-0000-0000-0000-000000000000"
expected = exception.confirmation(
confirm_code="DELETED",
confirm_detail="This flavor {0} has been deleted successfully"
.format("00000000-0000-0000-0000-000000000000"))
result = flavors.delete_flavor("00000000-0000-0000-0000-000000000000")
self.assertEqual(expected, result)