enh(api): permissions and page mode

Add the following functions to the js api: * `updateCollectiveEditPermissions` * `updateCollectiveSharePermissions` * `updateCollectiveEditMode` These wrap the corresponding API endpoints. Use them from the store and the cypress tests. Also extract `collective-edit-mode.spec.js` from `pages.spec.js`. It requires a different beforeEach function to seed the edit mode before visiting the collective. Signed-off-by: Max <[email protected]>
nextcloud · Nov 27, 2023 · 9aca9df · 9aca9df
1 parent c475252
commit 9aca9df
Show file tree

Hide file tree

Showing 6 changed files with 84 additions and 39 deletions.
diff --git a/cypress/e2e/collective-page-mode.spec.js b/cypress/e2e/collective-page-mode.spec.js
@@ -0,0 +1,62 @@
+/**
+ * @copyright Copyright (c) 2021 Azul <[email protected]>
+ *
+ * @author Azul <[email protected]>
+ *
+ * @license AGPL-3.0-or-later
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
+ *
+ */
+
+describe('Collective page mode', function() {
+
+	before(function() {
+		cy.loginAs('bob')
+		cy.deleteAndSeedCollective('Our Garden')
+			.seedPage('Day 1', '', 'Readme.md')
+			.seedPage('Day 2', '', 'Readme.md')
+	})
+
+	beforeEach(function() {
+		cy.loginAs('bob')
+	})
+
+	describe('Changing page mode', function() {
+		it('Opens edit mode per default', function() {
+			cy.seedCollectivePageMode('Our Garden', 1)
+			cy.visit('/apps/collectives/Our Garden')
+			// make sure the page list loaded properly
+			cy.contains('.app-content-list-item a', 'Day 1')
+			cy.openPage('Day 2')
+			cy.getEditor()
+				.should('be.visible')
+			cy.getReadOnlyEditor()
+				.should('not.be.visible')
+		})
+
+		it('Opens view mode per default', function() {
+			cy.seedCollectivePageMode('Our Garden', 0)
+			cy.visit('/apps/collectives/Our Garden')
+			// make sure the page list loaded properly
+			cy.contains('.app-content-list-item a', 'Day 1')
+			cy.openPage('Day 2')
+			cy.getReadOnlyEditor()
+				.should('be.visible')
+			cy.getEditor()
+				.should('not.be.visible')
+		})
+	})
+})
+
diff --git a/cypress/e2e/collective-readonly.spec.js b/cypress/e2e/collective-readonly.spec.js
@@ -28,7 +28,6 @@ describe('Read-only collective', function() {
 			.seedPage('SecondPage')
 		cy.circleFind('PermissionCollective')
 			.circleAddMember('bob')
-		cy.visit('apps/collectives/PermissionCollective')
 		cy.seedCollectivePermissions('PermissionCollective', 'edit', 4)
 	})
 

diff --git a/cypress/e2e/pages.spec.js b/cypress/e2e/pages.spec.js
@@ -25,6 +25,7 @@
  */
 
 describe('Page', function() {
+
 	before(function() {
 		cy.loginAs('bob')
 		cy.deleteAndSeedCollective('Our Garden').as('garden')
@@ -264,26 +265,6 @@ describe('Page', function() {
 		})
 	}
 
-	describe('Changing page mode', function() {
-		it('Opens edit mode per default', function() {
-			cy.seedCollectivePageMode('Our Garden', 1)
-			cy.openPage('Day 2')
-			cy.getEditor()
-				.should('be.visible')
-			cy.getReadOnlyEditor()
-				.should('not.be.visible')
-		})
-
-		it('Opens view mode per default', function() {
-			cy.seedCollectivePageMode('Our Garden', 0)
-			cy.openPage('Day 2')
-			cy.getReadOnlyEditor()
-				.should('be.visible')
-			cy.getEditor()
-				.should('not.be.visible')
-		})
-	})
-
 	describe('Full width view', function() {
 		it('Allows to toggle persistent full-width view', function() {
 			cy.openPage('Day 2')

diff --git a/cypress/support/commands.js b/cypress/support/commands.js
@@ -254,21 +254,21 @@ Cypress.Commands.add('deleteCollectiveFromTrash', (name) => {
 Cypress.Commands.add('seedCollectivePermissions', (name, type, level) => {
 	Cypress.log()
 	const action = (type === 'edit')
-		? UPDATE_COLLECTIVE_EDIT_PERMISSIONS
-		: UPDATE_COLLECTIVE_SHARE_PERMISSIONS
-	cy.store('state.collectives.collectives')
+		? api.updateCollectiveEditPermissions
+		: api.updateCollectiveSharePermissions
+	cy.getCollectives()
 		.findBy({ name })
-		.dispatch(action, { level })
+		.then((found) => action(found.id, level))
 })
 
 /**
  * Change default page mode for a collective
  */
 Cypress.Commands.add('seedCollectivePageMode', (name, mode) => {
 	Cypress.log()
-	cy.store('state.collectives.collectives')
+	cy.getCollectives()
 		.findBy({ name })
-		.dispatch(UPDATE_COLLECTIVE_PAGE_MODE, { mode })
+		.then((found) => api.updateCollectivePageMode(found.id, mode))
 })
 
 function collectiveContext(collective) {

diff --git a/src/api.js b/src/api.js
@@ -35,6 +35,18 @@ export function deleteCollective(id, circle) {
 	return axios.delete(url('/trash/' + id + doCircle))
 }
 
+export function updateCollectiveEditPermissions(id, level) {
+	return axios.put(url(`/${id}/editLevel`), { level })
+}
+
+export function updateCollectiveSharePermissions(id, level) {
+	return axios.put(url(`/${id}/shareLevel`), { level })
+}
+
+export function updateCollectivePageMode(id, mode) {
+	return axios.put(url(`/${id}/pageMode`), { mode })
+}
+
 export function getPages(context) {
 	return axios.get(pagesUrl(context))
 }

diff --git a/src/store/collectives.js b/src/store/collectives.js
@@ -394,10 +394,7 @@ export default {
 		 * @param {number} data.level new minimum level for sharing
 		 */
 		async [UPDATE_COLLECTIVE_EDIT_PERMISSIONS]({ commit }, { id, level }) {
-			const response = await axios.put(
-				generateUrl('/apps/collectives/_api/' + id + '/editLevel'),
-				{ level },
-			)
+			const response = await api.updateCollectiveEditPermissions(id, level)
 			commit(ADD_OR_UPDATE_COLLECTIVE, response.data.data)
 		},
 
@@ -409,10 +406,7 @@ export default {
 		 * @param {number} data.level new minimum level for sharing
 		 */
 		async [UPDATE_COLLECTIVE_SHARE_PERMISSIONS]({ commit }, { id, level }) {
-			const response = await axios.put(
-				generateUrl('/apps/collectives/_api/' + id + '/shareLevel'),
-				{ level },
-			)
+			const response = await api.updateCollectiveSharePermissions(id, level)
 			commit(ADD_OR_UPDATE_COLLECTIVE, response.data.data)
 		},
 
@@ -424,10 +418,7 @@ export default {
 		 * @param {number} data.mode page mode
 		 */
 		async [UPDATE_COLLECTIVE_PAGE_MODE]({ commit }, { id, mode }) {
-			const response = await axios.put(
-				generateUrl('/apps/collectives/_api/' + id + '/pageMode'),
-				{ mode },
-			)
+			const response = await api.updateCollectivePageMode(id, mode)
 			commit(ADD_OR_UPDATE_COLLECTIVE, response.data.data)
 		},