Rephrase Patrole skip message for better sorting
This is a trivial fix that is meant to address a quality of life
issue around sorting test results.
This addresses essentially the following concern:
We are starting to run Patrole and we are finding that it has a
lot different messages. We should reverse the message so they sort
better.
Change-Id: I3b5e2d9b1a9d1425982260e910a861d822625b06
diff --git a/patrole_tempest_plugin/rbac_utils.py b/patrole_tempest_plugin/rbac_utils.py
index 1ada69b..b3cb271 100644
--- a/patrole_tempest_plugin/rbac_utils.py
+++ b/patrole_tempest_plugin/rbac_utils.py
@@ -216,7 +216,7 @@
def skip_rbac_checks(cls):
if not CONF.patrole.enable_rbac:
raise cls.skipException(
- '%s skipped as Patrole testing not enabled.' % cls.__name__)
+ 'Patrole testing not enabled so skipping %s.' % cls.__name__)
@classmethod
def setup_rbac_utils(cls):
diff --git a/patrole_tempest_plugin/tests/unit/test_rbac_utils.py b/patrole_tempest_plugin/tests/unit/test_rbac_utils.py
index 55db501..5e730d3 100644
--- a/patrole_tempest_plugin/tests/unit/test_rbac_utils.py
+++ b/patrole_tempest_plugin/tests/unit/test_rbac_utils.py
@@ -254,6 +254,6 @@
with testtools.ExpectedException(
testtools.TestCase.skipException,
- value_re=('%s skipped as Patrole testing not enabled.'
+ value_re=('Patrole testing not enabled so skipping %s.'
% ChildRbacTest.__name__)):
child_test.setUpClass()