Add more unit tests
This gets the coverage of the upgradecheck module to 98%.
This commit is contained in:
parent
1f5d31a272
commit
8d76949735
@ -19,6 +19,9 @@ test_upgradecheck
|
|||||||
Tests for `upgradecheck` module.
|
Tests for `upgradecheck` module.
|
||||||
"""
|
"""
|
||||||
|
|
||||||
|
import sys
|
||||||
|
|
||||||
|
import mock
|
||||||
from oslotest import base
|
from oslotest import base
|
||||||
|
|
||||||
from oslo_upgradecheck import upgradecheck
|
from oslo_upgradecheck import upgradecheck
|
||||||
@ -34,6 +37,25 @@ class TestUpgradeCheckResult(base.BaseTestCase):
|
|||||||
self.assertEqual('test details', result.details)
|
self.assertEqual('test details', result.details)
|
||||||
|
|
||||||
|
|
||||||
|
class TestCommands(upgradecheck.UpgradeCommands):
|
||||||
|
def success(self):
|
||||||
|
return upgradecheck.UpgradeCheckResult(
|
||||||
|
upgradecheck.UpgradeCheckCode.SUCCESS, 'Always succeeds')
|
||||||
|
|
||||||
|
def warning(self):
|
||||||
|
return upgradecheck.UpgradeCheckResult(
|
||||||
|
upgradecheck.UpgradeCheckCode.WARNING, 'Always warns')
|
||||||
|
|
||||||
|
def failure(self):
|
||||||
|
return upgradecheck.UpgradeCheckResult(
|
||||||
|
upgradecheck.UpgradeCheckCode.FAILURE, 'Always fails')
|
||||||
|
|
||||||
|
_upgrade_checks = (('always succeeds', success),
|
||||||
|
('always warns', warning),
|
||||||
|
('always fails', failure),
|
||||||
|
)
|
||||||
|
|
||||||
|
|
||||||
class TestUpgradeCommands(base.BaseTestCase):
|
class TestUpgradeCommands(base.BaseTestCase):
|
||||||
def test_get_details(self):
|
def test_get_details(self):
|
||||||
result = upgradecheck.UpgradeCheckResult(
|
result = upgradecheck.UpgradeCheckResult(
|
||||||
@ -43,3 +65,16 @@ class TestUpgradeCommands(base.BaseTestCase):
|
|||||||
details = upgrade_commands._get_details(result)
|
details = upgrade_commands._get_details(result)
|
||||||
wrapped = '*' * 60 + '\n ' + '*' * 10
|
wrapped = '*' * 60 + '\n ' + '*' * 10
|
||||||
self.assertEqual(wrapped, details)
|
self.assertEqual(wrapped, details)
|
||||||
|
|
||||||
|
def test_check(self):
|
||||||
|
inst = TestCommands()
|
||||||
|
result = inst.check()
|
||||||
|
self.assertEqual(upgradecheck.UpgradeCheckCode.FAILURE, result)
|
||||||
|
|
||||||
|
|
||||||
|
class TestMain(base.BaseTestCase):
|
||||||
|
def test_main(self):
|
||||||
|
mock_argv = ['test-status', 'upgrade', 'check']
|
||||||
|
with mock.patch.object(sys, 'argv', mock_argv, create=True):
|
||||||
|
inst = TestCommands()
|
||||||
|
upgradecheck.main(inst.check)
|
||||||
|
Loading…
Reference in New Issue
Block a user