refactors

owncloud · Mar 6, 2020 · 804ef17 · 804ef17
1 parent a1139ac
commit 804ef17
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/tests/acceptance/pageObjects/FilesPageElement/sharingDialog.js b/tests/acceptance/pageObjects/FilesPageElement/sharingDialog.js
@@ -190,7 +190,7 @@ module.exports = {
         .waitForOutstandingAjaxCalls()
         .waitForElementNotPresent('@addShareSaveButton')
     },
-    saveCollaboratorEditForm: function () {
+    saveChanges: function () {
       return this.waitForElementVisible('@saveShareButton')
         .initAjaxCounters()
         .click('@saveShareButton')
@@ -268,7 +268,7 @@ module.exports = {
         }
       }
       if (changed) {
-        await this.saveCollaboratorEditForm()
+        await this.saveChanges()
       } else {
         await this.clickCancel()
       }
@@ -298,7 +298,7 @@ module.exports = {
       for (const permission of enabledPermissions) {
         await this.toggleSinglePermission(permission)
       }
-      await this.saveCollaboratorEditForm()
+      await this.saveChanges()
     },
     /**
      *
@@ -370,7 +370,7 @@ module.exports = {
     changeCollaboratorRole: async function (collaborator, newRole) {
       await collaboratorDialog.clickEditShare(collaborator)
       await this.changeCollaboratorRoleInDropdown(newRole)
-      return this.saveCollaboratorEditForm()
+      return this.saveChanges()
     },
     /**
      * @params {string} newRole
@@ -498,7 +498,7 @@ module.exports = {
         .FilesPageElement
         .publicLinksDialog()
         .setPublicLinkExpiryDate(value)
-      return this.saveCollaboratorEditForm()
+      return this.saveChanges()
     }
   },
   elements: {

diff --git a/tests/acceptance/stepDefinitions/sharingContext.js b/tests/acceptance/stepDefinitions/sharingContext.js
@@ -141,8 +141,8 @@ Given('user {string} has created a new share with following settings',
   function (sharer, dataTable) {
     const settings = dataTable.rowsHash()
     let expireDate = settings.expireDate
-    if (typeof settings.expireDate !== 'undefined') {
-      expireDate = sharingHelper.calculateDate(settings.expireDate)
+    if (typeof expireDate !== 'undefined') {
+      expireDate = sharingHelper.calculateDate(expireDate)
     }
     return shareFileFolder(
       settings.path,