From 93660cf421416e19f6f89c6083c3509e85152711 Mon Sep 17 00:00:00 2001
From: Ramon Wenger <ramon.wenger@iterativ.ch>
Date: Tue, 12 Nov 2024 16:02:50 +0100
Subject: [PATCH] Hide documents link conditionally

---
 .../cockpitPage/CockpitExpertParentPage.vue   | 23 ++++++++++++++++---
 cypress/e2e/settings.cy.js                    |  8 ++++++-
 2 files changed, 27 insertions(+), 4 deletions(-)

diff --git a/client/src/pages/cockpit/cockpitPage/CockpitExpertParentPage.vue b/client/src/pages/cockpit/cockpitPage/CockpitExpertParentPage.vue
index a5fca484..3e861216 100644
--- a/client/src/pages/cockpit/cockpitPage/CockpitExpertParentPage.vue
+++ b/client/src/pages/cockpit/cockpitPage/CockpitExpertParentPage.vue
@@ -1,9 +1,11 @@
 <script setup lang="ts">
 import SubNavigation from "@/components/header/SubNavigation.vue";
+import { useCurrentCourseSession } from "@/composables";
 import { useExpertCockpitPageData } from "@/pages/cockpit/cockpitPage/composables";
 import { ATTENDANCE_ROUTE, COCKPIT_ROUTE, DOCUMENTS_ROUTE } from "@/router/names";
 import { useTranslation } from "i18next-vue";
 import log from "loglevel";
+import { computed } from "vue";
 
 const props = defineProps<{
   courseSlug: string;
@@ -25,10 +27,25 @@ const documentsRoute = {
 
 const { t } = useTranslation();
 
-const items = [
+const courseSession = useCurrentCourseSession();
+
+const enableDocuments = computed(() => {
+  return !!courseSession.value.course.configuration.enable_circle_documents;
+});
+
+const items = computed(() => [
   { id: 1, name: t("a.Übersicht"), route: defaultRoute },
   { id: 2, name: t("a.Teilnehmer"), route: attendanceRoute },
-  { id: 3, name: t("a.Unterlagen"), route: documentsRoute },
+  ...(enableDocuments.value
+    ? [
+        {
+          id: 3,
+          name: t("a.Unterlagen"),
+          route: documentsRoute,
+          dataCy: "circle-documents",
+        },
+      ]
+    : []),
 
   {
     id: 4,
@@ -40,7 +57,7 @@ const items = [
     name: "MS Teams",
     route: "https://vbvbern.sharepoint.com/sites/myVBV-AFA_K-CI",
   },
-];
+]);
 </script>
 
 <template>
diff --git a/cypress/e2e/settings.cy.js b/cypress/e2e/settings.cy.js
index 94e52c7a..70a986df 100644
--- a/cypress/e2e/settings.cy.js
+++ b/cypress/e2e/settings.cy.js
@@ -1,20 +1,24 @@
-import {EXPERT_COCKPIT_URL, login} from "./helpers";
+import { EXPERT_COCKPIT_URL, login } from "./helpers";
 
 describe("settings.cy.js", () => {
   beforeEach(() => {
     cy.manageCommand("cypress_reset");
+    cy.intercept("/server/graphql").as("graphql");
   });
 
   describe("with circle documents enabled", () => {
     it("student can see circle documents", () => {
       login("test-student1@example.com", "test");
       cy.visit("/course/test-lehrgang/learn/fahrzeug");
+      cy.wait(["@graphql", "@graphql"]);
       cy.get('[data-cy="circle-document-section"]').should("exist");
     });
 
     it("trainer can see circle documents", () => {
       login("test-trainer1@example.com", "test");
+
       cy.visit(EXPERT_COCKPIT_URL);
+      cy.wait(["@graphql", "@graphql"]);
       cy.get('[data-cy="circle-documents"]').should("exist");
     });
   });
@@ -27,6 +31,7 @@ describe("settings.cy.js", () => {
     it("student cannot see circle documents", () => {
       login("test-student1@example.com", "test");
       cy.visit("/course/test-lehrgang/learn/fahrzeug");
+      cy.wait(["@graphql", "@graphql"]);
       cy.get('[data-cy="circle-title"]').should("contain", "Fahrzeug");
       cy.get('[data-cy="circle-document-section"]').should("not.exist");
     });
@@ -34,6 +39,7 @@ describe("settings.cy.js", () => {
     it("trainer cannot see circle documents", () => {
       login("test-trainer1@example.com", "test");
       cy.visit(EXPERT_COCKPIT_URL);
+      cy.wait(["@graphql", "@graphql"]);
       cy.get('[data-cy="circle-documents"]').should("not.exist");
     });
   });