add some integration tests

Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>

Author: schiessle

build/integration/features/bootstrap/FederationContext.php

@@ -35,6 +35,7 @@
 class FederationContext implements Context, SnippetAcceptingContext {
 
 	use WebDav;
+	use AppConfiguration;
 
 	/**
 	 * @Given /^User "([^"]*)" from server "(LOCAL|REMOTE)" shares "([^"]*)" with user "([^"]*)" from server "(LOCAL|REMOTE)"$/
@@ -56,6 +57,27 @@ public function federateSharing($sharerUser, $sharerServer, $sharerPath, $sharee
 		$this->usingServer($previous);
 	}
 
+
+	/**
+	 * @Given /^User "([^"]*)" from server "(LOCAL|REMOTE)" shares "([^"]*)" with group "([^"]*)" from server "(LOCAL|REMOTE)"$/
+	 *
+	 * @param string $sharerUser
+	 * @param string $sharerServer "LOCAL" or "REMOTE"
+	 * @param string $sharerPath
+	 * @param string $shareeUser
+	 * @param string $shareeServer "LOCAL" or "REMOTE"
+	 */
+	public function federateGroupSharing($sharerUser, $sharerServer, $sharerPath, $shareeGroup, $shareeServer){
+		if ($shareeServer == "REMOTE"){
+			$shareWith = "$shareeGroup@" . substr($this->remoteBaseUrl, 0, -4);
+		} else {
+			$shareWith = "$shareeGroup@" . substr($this->localBaseUrl, 0, -4);
+		}
+		$previous = $this->usingServer($sharerServer);
+		$this->createShare($sharerUser, $sharerPath, 9, $shareWith, null, null, null);
+		$this->usingServer($previous);
+	}
+
 	/**
 	 * @When /^User "([^"]*)" from server "(LOCAL|REMOTE)" accepts last pending share$/
 	 * @param string $user
@@ -73,4 +95,9 @@ public function acceptLastPendingShare($user, $server){
 		$this->theOCSStatusCodeShouldBe('100');
 		$this->usingServer($previous);
 	}
+
+	protected function resetAppConfigs() {
+		$this->modifyServerConfig('files_sharing', 'incoming_server2server_group_share_enabled', 'no');
+		$this->modifyServerConfig('files_sharing', 'outgoing_server2server_group_share_enabled', 'no');
+	}
 }

build/integration/federation_features/federated.feature

@@ -28,6 +28,39 @@ Feature: federated
 			| share_with | user1@REMOTE |
 			| share_with_displayname | user1@REMOTE |
 
+	Scenario: Federated group share a file with another server
+		Given Using server "REMOTE"
+		And parameter "incoming_server2server_group_share_enabled" of app "files_sharing" is set to "yes"
+		And parameter "outgoing_server2server_group_share_enabled" of app "files_sharing" is set to "yes"
+		And user "gs-user1" exists
+		And user "gs-user2" exists
+		And group "group1" exists
+		And check that user "gs-user1" belongs to group "group1"
+		And check that user "gs-user2" belongs to group "group1"
+		And Using server "LOCAL"
+		And user "gs-user0" exists
+		When User "gs-user0" from server "LOCAL" shares "/textfile0.txt" with group "group1" from server "REMOTE"
+		Then the OCS status code should be "100"
+		And the HTTP status code should be "200"
+		And Share fields of last share match with
+			| id | A_NUMBER |
+			| item_type | file |
+			| item_source | A_NUMBER |
+			| share_type | 9 |
+			| file_source | A_NUMBER |
+			| path | /textfile0.txt |
+			| permissions | 19 |
+			| stime | A_NUMBER |
+			| storage | A_NUMBER |
+			| mail_send | 0 |
+			| uid_owner | gs-user0 |
+			| storage_id | home::gs-user0 |
+			| file_parent | A_NUMBER |
+			| displayname_owner | gs-user0 |
+			| share_with | group1@REMOTE |
+			| share_with_displayname | group1@REMOTE |
+
+
 	Scenario: Federate share a file with local server
 		Given Using server "LOCAL"
 		And user "user0" exists