Merge ~mthaddon/charm-k8s-mattermost/+git/charm-k8s-mattermost:license_secret_test into charm-k8s-mattermost:master

[Tom Haddon]

Add a test for getting the licence secret name

Add a test for getting the licence secret name

[🤖 Canonical IS Merge Bot (canonical-is-mergebot)]

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

[Paul Collins (pjdc)]

review: Approve

[Paul Collins (pjdc)]

I assume your commit email is not intentional -- leaving for you to top-approve once updated.

[🤖 Canonical IS Merge Bot (canonical-is-mergebot)]

Change successfully merged at revision 87dc53420879d4401d94acf8be54e64a85fee1e5

diff --git a/tests/unit/test_charm.py b/tests/unit/test_charm.py
index 7e8c809..13b47f1 100644
--- a/tests/unit/test_charm.py
+++ b/tests/unit/test_charm.py
@@ -41,6 +41,8 @@ CONFIG_IMAGE_NO_PASSWORD = {
     'sso': False,
 }
 
+CONFIG_LICENCE_SECRET = {"licence": "RANDOMSTRING"}
+
 CONFIG_NO_S3_SETTINGS_S3_ENABLED = {
     'clustering': False,
     'mattermost_image_path': 'example.com/mattermost:latest',
@@ -119,3 +121,8 @@ class TestMattermostK8sCharm(unittest.TestCase):
         """Any CIDRs that has host bits set must be rejected, even if others are OK."""
         expected = 'range_mixed: invalid network(s): 10.242.0.0/8'
         self.assertEqual(check_ranges(RANGE_MIXED, 'range_mixed'), expected)
+
+    def test_get_licence_secret_name(self):
+        """Test the licence secret name is correctly constructed"""
+        self.harness.charm.model.config = copy.deepcopy(CONFIG_LICENCE_SECRET)
+        self.assertEqual(self.harness.charm._get_licence_secret_name(), "mattermost-licence-b5bbb1bf")