From 94a13f4a152adefe25e7acaa2665c59c894c60d3 Mon Sep 17 00:00:00 2001
From: Ramon Wenger <ramon.wenger@iterativ.ch>
Date: Tue, 31 Aug 2021 18:42:22 +0200
Subject: [PATCH] Enable students to add room entries

---
 .../frontend/rooms/room-page.spec.js          | 24 +++++++++++++++++++
 .../rooms/room-entries/NewRoomEntryWizard.vue |  1 +
 client/src/pages/room.vue                     |  8 +++++--
 3 files changed, 31 insertions(+), 2 deletions(-)

diff --git a/client/cypress/integration/frontend/rooms/room-page.spec.js b/client/cypress/integration/frontend/rooms/room-page.spec.js
index 5aba7cf2..18cd123f 100644
--- a/client/cypress/integration/frontend/rooms/room-page.spec.js
+++ b/client/cypress/integration/frontend/rooms/room-page.spec.js
@@ -211,4 +211,28 @@ describe('The Room Page', () => {
     cy.getByDataCy('room-entry-actions').click();
     cy.getByDataCy('edit-room-entry').click();
   });
+
+  it('creates a room entry', () => {
+    const MeQuery = getMinimalMe({isTeacher: false});
+    const room = {
+      id: 'some-room',
+      roomEntries: {
+        edges: []
+      }
+    };
+    const operations = {
+      MeQuery,
+      RoomEntriesQuery: {
+        room
+      },
+    };
+
+    cy.mockGraphqlOps({
+      operations
+    });
+
+    cy.visit(`/room/${slug}`);
+    cy.getByDataCy('add-room-entry').click();
+    cy.getByDataCy('add-room-entry-modal').should('exist');
+  });
 });
diff --git a/client/src/components/rooms/room-entries/NewRoomEntryWizard.vue b/client/src/components/rooms/room-entries/NewRoomEntryWizard.vue
index dfc4ea92..11321e3a 100644
--- a/client/src/components/rooms/room-entries/NewRoomEntryWizard.vue
+++ b/client/src/components/rooms/room-entries/NewRoomEntryWizard.vue
@@ -3,6 +3,7 @@
     :content-block="entry"
     :show-task-selection="false"
     :disable-save="saving"
+    data-cy="add-room-entry-modal"
     block-type="RoomEntry"
     @save="saveEntry"
     @hide="hideModal"
diff --git a/client/src/pages/room.vue b/client/src/pages/room.vue
index 85077128..e467ee73 100644
--- a/client/src/pages/room.vue
+++ b/client/src/pages/room.vue
@@ -30,7 +30,8 @@
     <div class="room__content">
       <add-room-entry-button
         :parent="room"
-        v-if="room.id && canEdit">
+        data-cy="add-room-entry"
+        v-if="room.id && !isReadOnly">
         <!--
         the v-if is there for the case where the room hasn't loaded yet, but there is already an attempt to create
         a new room entry. mainly happens during cypress testing, but could also happen on a very slow connection
@@ -58,8 +59,11 @@
 
     computed: {
       canEdit() {
-        return this.me.isTeacher && !this.me.readOnly && !this.me.selectedClass.readOnly;
+        return this.me.isTeacher && !this.isReadOnly;
       },
+      isReadOnly() {
+        return this.me.readOnly || this.me.selectedClass.readOnly;
+      }
     },
 
     apollo: {