From e490b58e962c1731558741c6f58457f0a68d6318 Mon Sep 17 00:00:00 2001
From: Ramon Wenger <ramon.wenger@iterativ.ch>
Date: Wed, 11 Aug 2021 17:10:46 +0200
Subject: [PATCH] Update cypress test

---
 .../integration/frontend/school-class.spec.js | 72 ++++++-------------
 1 file changed, 20 insertions(+), 52 deletions(-)

diff --git a/client/cypress/integration/frontend/school-class.spec.js b/client/cypress/integration/frontend/school-class.spec.js
index f1ae6cf9..4f4ce5f6 100644
--- a/client/cypress/integration/frontend/school-class.spec.js
+++ b/client/cypress/integration/frontend/school-class.spec.js
@@ -21,16 +21,7 @@ const selectedClass = {
 
 describe('Class Management', () => {
   beforeEach(() => {
-    cy.server();
-
-    cy.task('getSchema').then(schema => {
-      cy.mockGraphql({
-        schema,
-      });
-    });
-
-    cy.viewport('macbook-15');
-    cy.fakeLogin('rachel.green', 'test');
+    cy.setup();
   });
 
   // fixme: cache misbehaves with mequery, but only for test
@@ -114,43 +105,30 @@ describe('Class Management', () => {
   //   //
   // });
 
-  it('should not be able to leave class', () => {
-    cy.mockGraphqlOps({
-      operations: {
-        MeQuery: me,
-        MySchoolClassQuery: selectedClass,
-      },
-    });
-
-    cy.visit('/me/my-class');
-    cy.get('[data-cy=group-list-member]').should('have.length', 2);
-    cy.get('[data-cy=remove-from-class]').should('have.length', 0);
-    cy.get('[data-cy=add-to-class]').should('have.length', 0);
-  });
-
   it('should leave and re-join class', () => {
     const {me: teacher} = getMinimalMe({});
-    const teacherSelectedClass = {
-      me: {
-        ...selectedClass.me,
-        isTeacher: true,
-      },
-    };
     cy.mockGraphqlOps({
       operations: {
-        MeQuery: teacher,
+        MeQuery: {
+          me: teacher
+        },
         AddRemoveMember: {
           addRemoveMember: {
             success: true,
           },
         },
-        MySchoolClassQuery: teacherSelectedClass,
+        MySchoolClassQuery: {
+          me: {
+            ...teacher,
+            selectedClass
+          }
+        },
       },
     });
 
     cy.visit('/me/my-class');
     cy.get('[data-cy=active-class-members-list]').within(() => {
-      cy.get('[data-cy=school-class-member]').should('have.length', 2);
+      cy.get('[data-cy=group-list-member]').should('have.length', 2);
     });
     cy.get('[data-cy=inactive-class-members-list]').should('not.exist');
 
@@ -159,16 +137,16 @@ describe('Class Management', () => {
     cy.get('[data-cy=modal-save-button]').click();
 
     cy.get('[data-cy=active-class-members-list]').within(() => {
-      cy.get('[data-cy=school-class-member]').should('have.length', 1);
+      cy.get('[data-cy=group-list-member]').should('have.length', 1);
     });
     cy.get('[data-cy=inactive-class-members-list]').within(() => {
-      cy.get('[data-cy=school-class-member]').should('have.length', 1);
+      cy.get('[data-cy=group-list-member]').should('have.length', 1);
     });
 
     cy.get('[data-cy=add-to-class]').first().click();
 
     cy.get('[data-cy=active-class-members-list]').within(() => {
-      cy.get('[data-cy=school-class-member]').should('have.length', 2);
+      cy.get('[data-cy=group-list-member]').should('have.length', 2);
     });
     cy.get('[data-cy=inactive-class-members-list]').should('not.exist');
   });
@@ -197,26 +175,16 @@ describe('Class Management', () => {
 
 describe('Teacher Class Management', () => {
   beforeEach(() => {
-    cy.server();
-
-    cy.task('getSchema').then(schema => {
-      cy.mockGraphql({
-        schema,
-      });
-    });
-
-    cy.viewport('macbook-15');
-    cy.fakeLogin('ross.geller', 'test');
+    cy.setup();
   });
 
   it('changes class name', () => {
     let className = 'Gotta have class';
 
+    const {me: teacher} = getMinimalMe({});
+
     let localMe = {
-      me: {
-        ...me.me,
-        isTeacher: true,
-      },
+      me: teacher
     };
 
     cy.mockGraphqlOps({
@@ -224,8 +192,8 @@ describe('Teacher Class Management', () => {
         MeQuery: localMe,
         MySchoolClassQuery: {
           me: {
-            ...selectedClass.me,
-            isTeacher: true,
+            ...teacher,
+            selectedClass,
           },
         },
         UpdateSchoolClass: {