From dcfd5b405dd457246ff1fb644de52838c277f1c5 Mon Sep 17 00:00:00 2001 From: Gabriel Cocenza Date: Wed, 24 Apr 2024 10:25:16 -0300 Subject: [PATCH] - apply rewording suggestion --- cou/apps/auxiliary.py | 2 +- cou/apps/base.py | 3 +-- tests/unit/apps/test_auxiliary.py | 16 ++++++++-------- tests/unit/apps/test_auxiliary_subordinate.py | 4 ++-- tests/unit/apps/test_base.py | 4 ++-- tests/unit/apps/test_subordinate.py | 4 ++-- 6 files changed, 16 insertions(+), 17 deletions(-) diff --git a/cou/apps/auxiliary.py b/cou/apps/auxiliary.py index c582207f..dbc07373 100644 --- a/cou/apps/auxiliary.py +++ b/cou/apps/auxiliary.py @@ -132,7 +132,7 @@ def channel_codename(self) -> OpenStackRelease: if not compatible_os_releases: raise ApplicationError( f"Channel: {self.channel} for charm '{self.charm}' on series '{self.series}' is " - f"currently not supported in this tool. Please take a look at the documentation: " + f"not supported by COU. Please take a look at the documentation: " "https://docs.openstack.org/charm-guide/latest/project/charm-delivery.html to see " "if you are using the right track." ) diff --git a/cou/apps/base.py b/cou/apps/base.py index ddb0da63..1f6cb73a 100644 --- a/cou/apps/base.py +++ b/cou/apps/base.py @@ -789,8 +789,7 @@ def _check_channel(self) -> None: raise ApplicationError( f"Channel: {self.channel} for charm '{self.charm}' on series " - f"'{self.series}' is currently not supported in this tool. Please take a look at the " - "documentation: " + f"'{self.series}' is not supported by COU. Please take a look at the documentation: " "https://docs.openstack.org/charm-guide/latest/project/charm-delivery.html to see if " "you are using the right track." ) diff --git a/tests/unit/apps/test_auxiliary.py b/tests/unit/apps/test_auxiliary.py index 99dbfe98..e612c3d7 100644 --- a/tests/unit/apps/test_auxiliary.py +++ b/tests/unit/apps/test_auxiliary.py @@ -335,8 +335,8 @@ def test_auxiliary_upgrade_plan_unknown_track(model): """Test auxiliary upgrade plan with unknown track.""" channel = "2.0/stable" exp_msg = ( - f"Channel: {channel} for charm 'rabbitmq-server' on series 'focal' is currently " - "not supported in this tool. Please take a look at the documentation: " + f"Channel: {channel} for charm 'rabbitmq-server' on series 'focal' is not supported by " + "COU. Please take a look at the documentation: " "https://docs.openstack.org/charm-guide/latest/project/charm-delivery.html " "to see if you are using the right track." ) @@ -398,8 +398,8 @@ def test_auxiliary_raise_error_unknown_series(model): series = "foo" channel = "3.8/stable" exp_msg = ( - f"Channel: {channel} for charm 'rabbitmq-server' on series '{series}' is currently " - "not supported in this tool. Please take a look at the documentation: " + f"Channel: {channel} for charm 'rabbitmq-server' on series '{series}' is not supported by " + "COU. Please take a look at the documentation: " "https://docs.openstack.org/charm-guide/latest/project/charm-delivery.html " "to see if you are using the right track." ) @@ -436,8 +436,8 @@ def test_auxiliary_raise_error_os_not_on_lookup(current_os_release, model): """ current_os_release.return_value = OpenStackRelease("diablo") exp_error_msg = ( - "Channel: 3.8/stable for charm 'rabbitmq-server' on series 'focal' is currently not " - "supported in this tool. Please take a look at the documentation: " + "Channel: 3.8/stable for charm 'rabbitmq-server' on series 'focal' is not supported by " + "COU. Please take a look at the documentation: " "https://docs.openstack.org/charm-guide/latest/project/charm-delivery.html to see if you " "are using the right track." ) @@ -895,8 +895,8 @@ def test_ovn_no_compatible_os_release(channel, model): charm = "ovn-central" machines = {"0": MagicMock(spec_set=Machine)} exp_msg = ( - f"Channel: {channel} for charm '{charm}' on series 'focal' is currently " - "not supported in this tool. Please take a look at the documentation: " + f"Channel: {channel} for charm '{charm}' on series 'focal' is not supported by COU. " + "Please take a look at the documentation: " "https://docs.openstack.org/charm-guide/latest/project/charm-delivery.html " "to see if you are using the right track." ) diff --git a/tests/unit/apps/test_auxiliary_subordinate.py b/tests/unit/apps/test_auxiliary_subordinate.py index d5072c64..5df4ebbf 100644 --- a/tests/unit/apps/test_auxiliary_subordinate.py +++ b/tests/unit/apps/test_auxiliary_subordinate.py @@ -363,8 +363,8 @@ def test_auxiliary_subordinate_channel_codename_raise(model): ) exp_msg = ( - "Channel: luminous/stable for charm 'ceph-dashboard' on series 'focal' is currently " - "not supported in this tool. Please take a look at the documentation: " + "Channel: luminous/stable for charm 'ceph-dashboard' on series 'focal' is not supported " + "by COU. Please take a look at the documentation: " "https://docs.openstack.org/charm-guide/latest/project/charm-delivery.html " "to see if you are using the right track." ) diff --git a/tests/unit/apps/test_base.py b/tests/unit/apps/test_base.py index e1f3da78..8302a637 100644 --- a/tests/unit/apps/test_base.py +++ b/tests/unit/apps/test_base.py @@ -315,8 +315,8 @@ def test_check_channel_error(_): channel = "stable" series = "focal" exp_error_msg = ( - f"Channel: {channel} for charm '{name}' on series '{series}' is currently not supported " - "in this tool. Please take a look at the documentation: " + f"Channel: {channel} for charm '{name}' on series '{series}' is not supported by COU. " + "Please take a look at the documentation: " "https://docs.openstack.org/charm-guide/latest/project/charm-delivery.html to see if you " "are using the right track." ) diff --git a/tests/unit/apps/test_subordinate.py b/tests/unit/apps/test_subordinate.py index eac6f66e..7cdabdb1 100644 --- a/tests/unit/apps/test_subordinate.py +++ b/tests/unit/apps/test_subordinate.py @@ -129,8 +129,8 @@ def test_channel_setter_invalid(model, channel): """Test unsuccessful validation of channel upgrade plan for SubordinateApplication.""" machines = {"0": MagicMock(spec_set=Machine)} exp_error_msg = ( - f"Channel: {channel} for charm 'keystone-ldap' on series 'focal' is currently not " - "supported in this tool. Please take a look at the documentation: " + f"Channel: {channel} for charm 'keystone-ldap' on series 'focal' is not supported by COU. " + "Please take a look at the documentation: " "https://docs.openstack.org/charm-guide/latest/project/charm-delivery.html to see if you " "are using the right track." )