From 23f3f01b6d8c2c1fa73a830349b82affd7cb05d7 Mon Sep 17 00:00:00 2001
From: davelopez <46503462+davelopez@users.noreply.github.com>
Date: Thu, 30 Nov 2023 16:28:19 +0100
Subject: [PATCH 1/2] Disable notifications management conditionally

---
 .../Notifications/NotificationsManagement.vue | 43 +++++++++++++------
 1 file changed, 30 insertions(+), 13 deletions(-)

diff --git a/client/src/components/admin/Notifications/NotificationsManagement.vue b/client/src/components/admin/Notifications/NotificationsManagement.vue
index 1ffb7d7a557d..ef06eba45f4d 100644
--- a/client/src/components/admin/Notifications/NotificationsManagement.vue
+++ b/client/src/components/admin/Notifications/NotificationsManagement.vue
@@ -1,12 +1,15 @@
 <script setup lang="ts">
 import { FontAwesomeIcon } from "@fortawesome/vue-fontawesome";
-import { BButton } from "bootstrap-vue";
+import { BAlert, BButton } from "bootstrap-vue";
 import { useRouter } from "vue-router/composables";
 
+import { useConfig } from "@/composables/config";
+
 import BroadcastsList from "@/components/admin/Notifications/BroadcastsList.vue";
 import Heading from "@/components/Common/Heading.vue";
 
 const router = useRouter();
+const { config, isConfigLoaded } = useConfig();
 
 function goToCreateNewNotification() {
     router.push("/admin/notifications/create_new_notification");
@@ -28,18 +31,32 @@ function goToCreateNewBroadcast() {
             <i>broadcast notifications</i> to all users (even anonymous users).
         </p>
 
-        <div>
-            <BButton class="mb-2" variant="outline-primary" @click="goToCreateNewNotification">
-                <FontAwesomeIcon icon="plus" />
-                Send new notification
-            </BButton>
-
-            <BButton class="mb-2" variant="outline-primary" @click="goToCreateNewBroadcast">
-                <FontAwesomeIcon icon="plus" />
-                Create new broadcast
-            </BButton>
+        <div v-if="isConfigLoaded && config.enable_notification_system">
+            <div>
+                <BButton
+                    id="send-notification-button"
+                    class="mb-2"
+                    variant="outline-primary"
+                    @click="goToCreateNewNotification">
+                    <FontAwesomeIcon icon="plus" />
+                    Send new notification
+                </BButton>
+
+                <BButton
+                    id="create-broadcast-button"
+                    class="mb-2"
+                    variant="outline-primary"
+                    @click="goToCreateNewBroadcast">
+                    <FontAwesomeIcon icon="plus" />
+                    Create new broadcast
+                </BButton>
+            </div>
+
+            <BroadcastsList class="mt-2" />
         </div>
-
-        <BroadcastsList class="mt-2" />
+        <BAlert v-else variant="warning" show>
+            The notification system is disabled. To enable it, set the
+            <code>enable_notification_system</code> option to <code>true</code> in the Galaxy configuration file.
+        </BAlert>
     </div>
 </template>

From ac408cd1bbe013afeb87ece4073be562b46a6060 Mon Sep 17 00:00:00 2001
From: davelopez <46503462+davelopez@users.noreply.github.com>
Date: Thu, 30 Nov 2023 16:28:40 +0100
Subject: [PATCH 2/2] Add NotificationsManagement component tests

---
 .../NotificationsManagement.test.ts           | 55 +++++++++++++++++++
 1 file changed, 55 insertions(+)
 create mode 100644 client/src/components/admin/Notifications/NotificationsManagement.test.ts

diff --git a/client/src/components/admin/Notifications/NotificationsManagement.test.ts b/client/src/components/admin/Notifications/NotificationsManagement.test.ts
new file mode 100644
index 000000000000..41474db51618
--- /dev/null
+++ b/client/src/components/admin/Notifications/NotificationsManagement.test.ts
@@ -0,0 +1,55 @@
+import { createTestingPinia } from "@pinia/testing";
+import { getLocalVue } from "@tests/jest/helpers";
+import { shallowMount } from "@vue/test-utils";
+import flushPromises from "flush-promises";
+import { setActivePinia } from "pinia";
+
+import { mockFetcher } from "@/api/schema/__mocks__";
+
+import NotificationsManagement from "./NotificationsManagement.vue";
+
+jest.mock("@/api/schema");
+
+const localVue = getLocalVue(true);
+
+const selectors = {
+    sendNotificationButton: "#send-notification-button",
+    createBroadcastButton: "#create-broadcast-button",
+} as const;
+
+async function mountNotificationsManagement(config: any = {}) {
+    const pinia = createTestingPinia();
+    setActivePinia(pinia);
+
+    mockFetcher.path("/api/configuration").method("get").mock({ data: config });
+
+    const wrapper = shallowMount(NotificationsManagement as object, {
+        localVue,
+        pinia,
+        stubs: {
+            FontAwesomeIcon: true,
+        },
+    });
+
+    await flushPromises();
+
+    return wrapper;
+}
+
+describe("NotificationsManagement.vue", () => {
+    it("should render the create notification buttons if the notification system is enabled", async () => {
+        const config = { enable_notification_system: true };
+        const wrapper = await mountNotificationsManagement(config);
+
+        expect(wrapper.find(selectors.sendNotificationButton).exists()).toBe(true);
+        expect(wrapper.find(selectors.createBroadcastButton).exists()).toBe(true);
+    });
+
+    it("should not render the create notification buttons if the notification system is disabled", async () => {
+        const config = { enable_notification_system: false };
+        const wrapper = await mountNotificationsManagement(config);
+
+        expect(wrapper.find(selectors.sendNotificationButton).exists()).toBe(false);
+        expect(wrapper.find(selectors.createBroadcastButton).exists()).toBe(false);
+    });
+});