Add one more test case

owncloud · Sep 26, 2019 · fa08e70 · fa08e70
1 parent fc853ec
commit fa08e70
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 9 deletions.
diff --git a/tests/acceptance/features/webUIFiles/copyPrivateLinks.feature b/tests/acceptance/features/webUIFiles/copyPrivateLinks.feature
@@ -8,6 +8,11 @@ Feature: copy current path as a permanent link
     And user "user1" has logged in using the webUI
     And the user has browsed to the files page
 
-  Scenario: Copy permalink to clipboard
-    When  the user copies the permalink of the current folder using the webUI
-    Then the clipboard content should match current folder permalink
+  Scenario Outline: Copy permalink to clipboard
+    When the user browses to the folder <folder_name> on the files page
+    And the user copies the permalink of the current folder using the webUI
+    Then the clipboard content should match permalink of resource <folder_name>
+    Examples:
+      | folder_name     |
+      | "/"             |
+      | "simple-folder" |
diff --git a/tests/acceptance/helpers/webdavHelper.js b/tests/acceptance/helpers/webdavHelper.js
@@ -174,12 +174,13 @@ exports.createFile = function (user, fileName, contents = '') {
  * @param {string} path
  * @param {string} userId
  * @param {array} requestedProps
- * @param {number} folderDepth
 */
-exports.getProperties = function (path, userId, requestedProps, folderDepth = 1) {
+exports.getProperties = function (path, userId, requestedProps) {
   return new Promise((resolve) => {
-    exports.propfind(`/files/${userId}${path}`, userId, userSettings.getPasswordForUser(userId), requestedProps,
-      folderDepth)
+    const trimmedPath = path.replace(/^\/+/, '') // remove a leading slash
+    const relativePath = `/files/${userId}/${trimmedPath}`
+    exports.propfind(relativePath, userId, userSettings.getPasswordForUser(userId), requestedProps,
+      0)
       .then(str => {
         const response = JSON.parse(convert.xml2json(str, { compact: true }))['d:multistatus']['d:response']
         const properties = {}

diff --git a/tests/acceptance/stepDefinitions/filesContext.js b/tests/acceptance/stepDefinitions/filesContext.js
@@ -475,12 +475,18 @@ Given('the user has created the following files', function (entryList) {
   return client
 })
 
+When('the user browses to the folder {string} on the files page', (folderName) => {
+  const targetFolder = folderName === '/' ? '' : folderName
+  return client
+    .page.filesPage()
+    .navigateAndWaitTillLoaded(targetFolder)
+})
 When('the user copies the permalink of the current folder using the webUI', function () {
   return client.page.filesPage().copyPermalinkFromFilesAppBar()
 })
-Then('the clipboard content should match current folder permalink', function () {
+Then('the clipboard content should match permalink of resource {string}', function (folderName) {
   return client.getClipBoardContent(function (value) {
-    webdav.getProperties('/', client.globals.currentUser, ['oc:privatelink'])
+    webdav.getProperties(folderName, client.globals.currentUser, ['oc:privatelink'])
       .then(folderData => {
         assert.strictEqual(folderData['oc:privatelink'], value)
       })