Added acceptance test for enforced expiration date on a user share

owncloud · Mar 10, 2020 · 4430f33 · 4430f33
1 parent b65eda1
commit 4430f33
Show file tree

Hide file tree

Showing 2 changed files with 28 additions and 4 deletions.
diff --git a/tests/acceptance/features/webUISharingInternalUsers/shareWithUsers.feature b/tests/acceptance/features/webUISharingInternalUsers/shareWithUsers.feature
@@ -605,3 +605,20 @@ Feature: Sharing files and folders with internal users
       | uid_owner  | user1      |
       | share_with | user2      |
       | expiration | +7         |
+
+  Scenario Outline: enforced expiration date on collaborators share
+    Given the setting "shareapi_default_expire_date" of app "core" has been set to "yes"
+    And the setting "shareapi_expire_after_n_days" of app "core" has been set to "42"
+    And user "user1" has logged in using the webUI
+    When the user shares file "<shared-resource>" with user "User Two" using the webUI
+    Then user "user1" should have a share with these details:
+      | field      | value              |
+      | path       | /<shared-resource> |
+      | share_type | user               |
+      | uid_owner  | user1              |
+      | share_with | user2              |
+      | expiration | null               |
+    Examples:
+      | shared-resource |
+      | lorem.txt       |
+      | simple-folder   |
diff --git a/tests/acceptance/helpers/sharingHelper.js b/tests/acceptance/helpers/sharingHelper.js
@@ -107,10 +107,17 @@ module.exports = {
             } else if (expectedDetail.field === 'expiration') {
               expectedDetail.value = sharingHelper.calculateDate(expectedDetail.value)
             }
-
-            if (!(expectedDetail.field in share) || share[expectedDetail.field].toString() !== expectedDetail.value) {
-              found = false
-              break
+            if (share[expectedDetail.field] === null) {
+              // assertion for null values
+              if (!(expectedDetail.field in share) || expectedDetail.value !== 'null') {
+                found = false
+                break
+              }
+            } else {
+              if (!(expectedDetail.field in share) || share[expectedDetail.field].toString() !== expectedDetail.value) {
+                found = false
+                break
+              }
             }
           }
           if (found) {