From a41ec781fc3b7e7ce3e0aec6a649df47a8408e99 Mon Sep 17 00:00:00 2001 From: Ben Nemec Date: Tue, 18 Sep 2018 21:59:52 +0000 Subject: [PATCH] Shorten UpgradeCheck class names When this library is used, the classes get referred to as something like upgradecheck.UpgradeCheckResult, which is redundant and wordy. Shortening the class names to remove the "UpgradeCheck" part fixes both problems. Change-Id: I621434585ed84256b9fe44d88a516d8c27eb05b3 --- oslo_upgradecheck/__main__.py | 7 +++---- oslo_upgradecheck/tests/test_upgradecheck.py | 22 +++++++------------- oslo_upgradecheck/upgradecheck.py | 18 ++++++++-------- 3 files changed, 20 insertions(+), 27 deletions(-) diff --git a/oslo_upgradecheck/__main__.py b/oslo_upgradecheck/__main__.py index 05a0bd6..9086e66 100644 --- a/oslo_upgradecheck/__main__.py +++ b/oslo_upgradecheck/__main__.py @@ -21,12 +21,11 @@ from oslo_upgradecheck import upgradecheck class Checks(upgradecheck.UpgradeCommands): def success(self): - return upgradecheck.UpgradeCheckResult( - upgradecheck.UpgradeCheckCode.SUCCESS, 'Always succeeds') + return upgradecheck.Result(upgradecheck.Code.SUCCESS, + 'Always succeeds') def failure(self): - return upgradecheck.UpgradeCheckResult( - upgradecheck.UpgradeCheckCode.FAILURE, 'Always fails') + return upgradecheck.Result(upgradecheck.Code.FAILURE, 'Always fails') _upgrade_checks = (('always succeeds', success), ('always fails', failure), diff --git a/oslo_upgradecheck/tests/test_upgradecheck.py b/oslo_upgradecheck/tests/test_upgradecheck.py index 51684fc..5ab70f9 100644 --- a/oslo_upgradecheck/tests/test_upgradecheck.py +++ b/oslo_upgradecheck/tests/test_upgradecheck.py @@ -30,25 +30,21 @@ from oslo_upgradecheck import upgradecheck class TestUpgradeCheckResult(base.BaseTestCase): def test_details(self): - result = upgradecheck.UpgradeCheckResult( - upgradecheck.UpgradeCheckCode.SUCCESS, - 'test details') + result = upgradecheck.Result(upgradecheck.Code.SUCCESS, 'test details') self.assertEqual(0, result.code) self.assertEqual('test details', result.details) class TestCommands(upgradecheck.UpgradeCommands): def success(self): - return upgradecheck.UpgradeCheckResult( - upgradecheck.UpgradeCheckCode.SUCCESS, 'Always succeeds') + return upgradecheck.Result(upgradecheck.Code.SUCCESS, + 'Always succeeds') def warning(self): - return upgradecheck.UpgradeCheckResult( - upgradecheck.UpgradeCheckCode.WARNING, 'Always warns') + return upgradecheck.Result(upgradecheck.Code.WARNING, 'Always warns') def failure(self): - return upgradecheck.UpgradeCheckResult( - upgradecheck.UpgradeCheckCode.FAILURE, 'Always fails') + return upgradecheck.Result(upgradecheck.Code.FAILURE, 'Always fails') _upgrade_checks = (('always succeeds', success), ('always warns', warning), @@ -62,9 +58,7 @@ class SuccessCommands(TestCommands): class TestUpgradeCommands(base.BaseTestCase): def test_get_details(self): - result = upgradecheck.UpgradeCheckResult( - upgradecheck.UpgradeCheckCode.SUCCESS, - '*' * 70) + result = upgradecheck.Result(upgradecheck.Code.SUCCESS, '*' * 70) upgrade_commands = upgradecheck.UpgradeCommands() details = upgrade_commands._get_details(result) wrapped = '*' * 60 + '\n ' + '*' * 10 @@ -73,7 +67,7 @@ class TestUpgradeCommands(base.BaseTestCase): def test_check(self): inst = TestCommands() result = inst.check() - self.assertEqual(upgradecheck.UpgradeCheckCode.FAILURE, result) + self.assertEqual(upgradecheck.Code.FAILURE, result) class TestMain(base.BaseTestCase): @@ -85,7 +79,7 @@ class TestMain(base.BaseTestCase): def test_main(self): inst = TestCommands() - self._run_test(inst.check, upgradecheck.UpgradeCheckCode.FAILURE) + self._run_test(inst.check, upgradecheck.Code.FAILURE) def test_main_exception(self): def raises(): diff --git a/oslo_upgradecheck/upgradecheck.py b/oslo_upgradecheck/upgradecheck.py index 858a254..389802e 100644 --- a/oslo_upgradecheck/upgradecheck.py +++ b/oslo_upgradecheck/upgradecheck.py @@ -25,7 +25,7 @@ import prettytable from oslo_upgradecheck._i18n import _ -class UpgradeCheckCode(enum.IntEnum): +class Code(enum.IntEnum): """Status codes for the upgrade check command""" # All upgrade readiness checks passed successfully and there is @@ -42,23 +42,23 @@ class UpgradeCheckCode(enum.IntEnum): UPGRADE_CHECK_MSG_MAP = { - UpgradeCheckCode.SUCCESS: _('Success'), - UpgradeCheckCode.WARNING: _('Warning'), - UpgradeCheckCode.FAILURE: _('Failure'), + Code.SUCCESS: _('Success'), + Code.WARNING: _('Warning'), + Code.FAILURE: _('Failure'), } -class UpgradeCheckResult(object): +class Result(object): """Class used for 'nova-status upgrade check' results. - The 'code' attribute is an UpgradeCheckCode enum. + The 'code' attribute is a Code enum. The 'details' attribute is a translated message generally only used for checks that result in a warning or failure code. The details should provide information on what issue was discovered along with any remediation. """ def __init__(self, code, details=None): - super(UpgradeCheckResult, self).__init__() + super(Result, self).__init__() self.code = code self.details = details @@ -85,9 +85,9 @@ class UpgradeCommands(object): new code. These checks also require access to potentially all of the Nova databases (nova, nova_api, nova_api_cell0) and external services such as the placement API service. - :returns: UpgradeCheckCode + :returns: Code """ - return_code = UpgradeCheckCode.SUCCESS + return_code = Code.SUCCESS # This is a list if 2-item tuples for the check name and it's results. check_results = [] for name, func in self._upgrade_checks: