From adf716433309c0cf9c9748ad270e76cddc7832c7 Mon Sep 17 00:00:00 2001
From: Ferdinand Thiessen <opensource@fthiessen.de>
Date: Tue, 30 Jul 2024 20:10:33 +0200
Subject: [PATCH] test(files_sharing): Add test for "note to recipient" feature

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
---
 .../e2e/files_sharing/FilesSharingUtils.ts    |  8 +-
 .../e2e/files_sharing/note-to-recipient.cy.ts | 73 +++++++++++++++++++
 2 files changed, 79 insertions(+), 2 deletions(-)
 create mode 100644 cypress/e2e/files_sharing/note-to-recipient.cy.ts

diff --git a/cypress/e2e/files_sharing/FilesSharingUtils.ts b/cypress/e2e/files_sharing/FilesSharingUtils.ts
index 77f5d38ba8aa0..ef8cf462a06c9 100644
--- a/cypress/e2e/files_sharing/FilesSharingUtils.ts
+++ b/cypress/e2e/files_sharing/FilesSharingUtils.ts
@@ -5,14 +5,13 @@
 /* eslint-disable jsdoc/require-jsdoc */
 import { triggerActionForFile } from '../files/FilesUtils'
 
-import { EntryId as FileRequestEntryID } from '../../../apps/files_sharing/src/new/newFileRequest'
-
 export interface ShareSetting {
 	read: boolean
 	update: boolean
 	delete: boolean
 	share: boolean
 	download: boolean
+	note: string
 }
 
 export function createShare(fileName: string, username: string, shareSettings: Partial<ShareSetting> = {}) {
@@ -85,6 +84,11 @@ export function updateShare(fileName: string, index: number, shareSettings: Part
 			}
 		}
 
+		if (shareSettings.note !== undefined) {
+			cy.findByRole('checkbox', { name: /note to recipient/i }).check({ force: true, scrollBehavior: 'nearest' })
+			cy.findByRole('textbox', { name: /note to recipient/i }).type(shareSettings.note)
+		}
+
 		cy.get('[data-cy-files-sharing-share-editor-action="save"]').click({ scrollBehavior: 'nearest' })
 
 		cy.wait('@updateShare')
diff --git a/cypress/e2e/files_sharing/note-to-recipient.cy.ts b/cypress/e2e/files_sharing/note-to-recipient.cy.ts
new file mode 100644
index 0000000000000..d1153f2c995b4
--- /dev/null
+++ b/cypress/e2e/files_sharing/note-to-recipient.cy.ts
@@ -0,0 +1,73 @@
+/*!
+ * SPDX-FileCopyrightText: 2024 Nextcloud GmbH and Nextcloud contributors
+ * SPDX-License-Identifier: AGPL-3.0-or-later
+ */
+import type { User } from '@nextcloud/cypress'
+import { createShare, openSharingPanel } from './FilesSharingUtils.ts'
+import { getRowForFile, navigateToFolder } from '../files/FilesUtils.ts'
+
+describe('files_sharing: Note to recipient', { testIsolation: true }, () => {
+	let user: User
+	let sharee: User
+
+	beforeEach(() => {
+		cy.createRandomUser().then(($user) => {
+			user = $user
+		})
+		cy.createRandomUser().then(($user) => {
+			sharee = $user
+		})
+	})
+
+	it('displays the note to the sharee', () => {
+		cy.mkdir(user, '/folder')
+		cy.uploadContent(user, new Blob([]), 'text/plain', '/folder/file')
+		cy.login(user)
+		cy.visit('/apps/files')
+
+		// share the folder
+		createShare('folder', sharee.userId, { read: true, download: true, note: 'Hello, this is the note.' })
+
+		cy.logout()
+		// Now for the sharee
+		cy.login(sharee)
+
+		// visit shared files view
+		cy.visit('/apps/files')
+		navigateToFolder('folder')
+		cy.get('.note-to-recipient')
+			.should('be.visible')
+			.and('contain.text', 'Hello, this is the note.')
+	})
+
+	/**
+	 * Regression test for https://github.com/nextcloud/server/issues/46188
+	 */
+	it('shows an existing note when editing a share', () => {
+		cy.mkdir(user, '/folder')
+		cy.login(user)
+		cy.visit('/apps/files')
+
+		// share the folder
+		createShare('folder', sharee.userId, { read: true, download: true, note: 'Hello, this is the note.' })
+
+		// reload just to be sure
+		cy.reload()
+
+		// open the sharing tab
+		openSharingPanel('folder')
+
+		cy.get('[data-cy-sidebar]').within(() => {
+			// Open the share
+			cy.get('[data-cy-files-sharing-share-actions]').first().click()
+			// Open the custom settings
+			cy.get('[data-cy-files-sharing-share-permissions-bundle="custom"]').click()
+
+			cy.findByRole('checkbox', { name: /note to recipient/i })
+				.and('be.checked')
+			cy.findByRole('textbox', { name: /note to recipient/i })
+				.should('be.visible')
+				.and('have.value', 'Hello, this is the note.')
+		})
+	})
+})