nextcloud / server

apps/files_reminders/lib/Controller/ApiController.php

Indentation +100 added lines, -100 removed lines

@@ -25,111 +25,111 @@
 use Psr\Log\LoggerInterface;
 
 class ApiController extends OCSController {
-	public function __construct(
-		string $appName,
-		IRequest $request,
-		protected ReminderService $reminderService,
-		protected IUserSession $userSession,
-		protected LoggerInterface $logger,
-	) {
-		parent::__construct($appName, $request);
-	}
+    public function __construct(
+        string $appName,
+        IRequest $request,
+        protected ReminderService $reminderService,
+        protected IUserSession $userSession,
+        protected LoggerInterface $logger,
+    ) {
+        parent::__construct($appName, $request);
+    }
 
-	/**
-	 * Get a reminder
-	 *
-	 * @param int $fileId ID of the file
-	 * @return DataResponse<Http::STATUS_OK, array{dueDate: ?string}, array{}>|DataResponse<Http::STATUS_UNAUTHORIZED, list<empty>, array{}>
-	 *
-	 * 200: Reminder returned
-	 * 401: Account not found
-	 */
-	#[NoAdminRequired]
-	public function get(int $fileId): DataResponse {
-		$user = $this->userSession->getUser();
-		if ($user === null) {
-			return new DataResponse([], Http::STATUS_UNAUTHORIZED);
-		}
+    /**
+     * Get a reminder
+     *
+     * @param int $fileId ID of the file
+     * @return DataResponse<Http::STATUS_OK, array{dueDate: ?string}, array{}>|DataResponse<Http::STATUS_UNAUTHORIZED, list<empty>, array{}>
+     *
+     * 200: Reminder returned
+     * 401: Account not found
+     */
+    #[NoAdminRequired]
+    public function get(int $fileId): DataResponse {
+        $user = $this->userSession->getUser();
+        if ($user === null) {
+            return new DataResponse([], Http::STATUS_UNAUTHORIZED);
+        }
 
-		try {
-			$reminder = $this->reminderService->getDueForUser($user, $fileId);
-			if ($reminder === null) {
-				return new DataResponse(['dueDate' => null], Http::STATUS_OK);
-			}
-			return new DataResponse([
-				'dueDate' => $reminder->getDueDate()->format(DateTimeInterface::ATOM), // ISO 8601
-			], Http::STATUS_OK);
-		} catch (NodeNotFoundException $e) {
-			return new DataResponse(['dueDate' => null], Http::STATUS_OK);
-		}
-	}
+        try {
+            $reminder = $this->reminderService->getDueForUser($user, $fileId);
+            if ($reminder === null) {
+                return new DataResponse(['dueDate' => null], Http::STATUS_OK);
+            }
+            return new DataResponse([
+                'dueDate' => $reminder->getDueDate()->format(DateTimeInterface::ATOM), // ISO 8601
+            ], Http::STATUS_OK);
+        } catch (NodeNotFoundException $e) {
+            return new DataResponse(['dueDate' => null], Http::STATUS_OK);
+        }
+    }
 
-	/**
-	 * Set a reminder
-	 *
-	 * @param int $fileId ID of the file
-	 * @param string $dueDate ISO 8601 formatted date time string
-	 *
-	 * @return DataResponse<Http::STATUS_OK|Http::STATUS_CREATED|Http::STATUS_BAD_REQUEST|Http::STATUS_UNAUTHORIZED|Http::STATUS_NOT_FOUND, list<empty>, array{}>
-	 *
-	 * 200: Reminder updated
-	 * 201: Reminder created successfully
-	 * 400: Creating reminder is not possible
-	 * 401: Account not found
-	 * 404: File not found
-	 */
-	#[NoAdminRequired]
-	public function set(int $fileId, string $dueDate): DataResponse {
-		try {
-			$dueDate = (new DateTime($dueDate))->setTimezone(new DateTimeZone('UTC'));
-			$nowDate = (new DateTime('now'))->setTimezone(new DateTimeZone('UTC'));
-			if ($dueDate <= $nowDate) {
-				return new DataResponse([], Http::STATUS_BAD_REQUEST);
-			}
-		} catch (Exception $e) {
-			$this->logger->error($e->getMessage(), ['exception' => $e]);
-			return new DataResponse([], Http::STATUS_BAD_REQUEST);
-		}
+    /**
+     * Set a reminder
+     *
+     * @param int $fileId ID of the file
+     * @param string $dueDate ISO 8601 formatted date time string
+     *
+     * @return DataResponse<Http::STATUS_OK|Http::STATUS_CREATED|Http::STATUS_BAD_REQUEST|Http::STATUS_UNAUTHORIZED|Http::STATUS_NOT_FOUND, list<empty>, array{}>
+     *
+     * 200: Reminder updated
+     * 201: Reminder created successfully
+     * 400: Creating reminder is not possible
+     * 401: Account not found
+     * 404: File not found
+     */
+    #[NoAdminRequired]
+    public function set(int $fileId, string $dueDate): DataResponse {
+        try {
+            $dueDate = (new DateTime($dueDate))->setTimezone(new DateTimeZone('UTC'));
+            $nowDate = (new DateTime('now'))->setTimezone(new DateTimeZone('UTC'));
+            if ($dueDate <= $nowDate) {
+                return new DataResponse([], Http::STATUS_BAD_REQUEST);
+            }
+        } catch (Exception $e) {
+            $this->logger->error($e->getMessage(), ['exception' => $e]);
+            return new DataResponse([], Http::STATUS_BAD_REQUEST);
+        }
 
-		$user = $this->userSession->getUser();
-		if ($user === null) {
-			return new DataResponse([], Http::STATUS_UNAUTHORIZED);
-		}
+        $user = $this->userSession->getUser();
+        if ($user === null) {
+            return new DataResponse([], Http::STATUS_UNAUTHORIZED);
+        }
 
-		try {
-			$created = $this->reminderService->createOrUpdate($user, $fileId, $dueDate);
-			if ($created) {
-				return new DataResponse([], Http::STATUS_CREATED);
-			}
-			return new DataResponse([], Http::STATUS_OK);
-		} catch (NodeNotFoundException $e) {
-			return new DataResponse([], Http::STATUS_NOT_FOUND);
-		}
-	}
+        try {
+            $created = $this->reminderService->createOrUpdate($user, $fileId, $dueDate);
+            if ($created) {
+                return new DataResponse([], Http::STATUS_CREATED);
+            }
+            return new DataResponse([], Http::STATUS_OK);
+        } catch (NodeNotFoundException $e) {
+            return new DataResponse([], Http::STATUS_NOT_FOUND);
+        }
+    }
 
-	/**
-	 * Remove a reminder
-	 *
-	 * @param int $fileId ID of the file
-	 *
-	 * @return DataResponse<Http::STATUS_OK|Http::STATUS_UNAUTHORIZED|Http::STATUS_NOT_FOUND, list<empty>, array{}>
-	 *
-	 * 200: Reminder deleted successfully
-	 * 401: Account not found
-	 * 404: Reminder not found
-	 */
-	#[NoAdminRequired]
-	public function remove(int $fileId): DataResponse {
-		$user = $this->userSession->getUser();
-		if ($user === null) {
-			return new DataResponse([], Http::STATUS_UNAUTHORIZED);
-		}
+    /**
+     * Remove a reminder
+     *
+     * @param int $fileId ID of the file
+     *
+     * @return DataResponse<Http::STATUS_OK|Http::STATUS_UNAUTHORIZED|Http::STATUS_NOT_FOUND, list<empty>, array{}>
+     *
+     * 200: Reminder deleted successfully
+     * 401: Account not found
+     * 404: Reminder not found
+     */
+    #[NoAdminRequired]
+    public function remove(int $fileId): DataResponse {
+        $user = $this->userSession->getUser();
+        if ($user === null) {
+            return new DataResponse([], Http::STATUS_UNAUTHORIZED);
+        }
 
-		try {
-			$this->reminderService->remove($user, $fileId);
-			return new DataResponse([], Http::STATUS_OK);
-		} catch (NodeNotFoundException|ReminderNotFoundException $e) {
-			return new DataResponse([], Http::STATUS_NOT_FOUND);
-		}
-	}
+        try {
+            $this->reminderService->remove($user, $fileId);
+            return new DataResponse([], Http::STATUS_OK);
+        } catch (NodeNotFoundException|ReminderNotFoundException $e) {
+            return new DataResponse([], Http::STATUS_NOT_FOUND);
+        }
+    }
 }

build/integration/features/bootstrap/BasicStructure.php

Indentation +552 added lines, -552 removed lines

@@ -16,556 +16,556 @@
 require __DIR__ . '/../../vendor/autoload.php';
 
 trait BasicStructure {
-	use Auth;
-	use Avatar;
-	use Download;
-	use Mail;
-	use Theming;
-
-	/** @var string */
-	private $currentUser = '';
-
-	/** @var string */
-	private $currentServer = '';
-
-	/** @var string */
-	private $baseUrl = '';
-
-	/** @var int */
-	private $apiVersion = 1;
-
-	/** @var ResponseInterface */
-	private $response = null;
-
-	/** @var CookieJar */
-	private $cookieJar;
-
-	/** @var string */
-	private $requestToken;
-
-	protected $adminUser;
-	protected $regularUser;
-	protected $localBaseUrl;
-	protected $remoteBaseUrl;
-
-	public function __construct($baseUrl, $admin, $regular_user_password) {
-		// Initialize your context here
-		$this->baseUrl = $baseUrl;
-		$this->adminUser = $admin;
-		$this->regularUser = $regular_user_password;
-		$this->localBaseUrl = $this->baseUrl;
-		$this->remoteBaseUrl = $this->baseUrl;
-		$this->currentServer = 'LOCAL';
-		$this->cookieJar = new CookieJar();
-
-		// in case of ci deployment we take the server url from the environment
-		$testServerUrl = getenv('TEST_SERVER_URL');
-		if ($testServerUrl !== false) {
-			$this->baseUrl = $testServerUrl;
-			$this->localBaseUrl = $testServerUrl;
-		}
-
-		// federated server url from the environment
-		$testRemoteServerUrl = getenv('TEST_SERVER_FED_URL');
-		if ($testRemoteServerUrl !== false) {
-			$this->remoteBaseUrl = $testRemoteServerUrl;
-		}
-	}
-
-	/**
-	 * @Given /^using api version "(\d+)"$/
-	 * @param string $version
-	 */
-	public function usingApiVersion($version) {
-		$this->apiVersion = (int)$version;
-	}
-
-	/**
-	 * @Given /^As an "([^"]*)"$/
-	 * @param string $user
-	 */
-	public function asAn($user) {
-		$this->currentUser = $user;
-	}
-
-	/**
-	 * @Given /^Using server "(LOCAL|REMOTE)"$/
-	 * @param string $server
-	 * @return string Previous used server
-	 */
-	public function usingServer($server) {
-		$previousServer = $this->currentServer;
-		if ($server === 'LOCAL') {
-			$this->baseUrl = $this->localBaseUrl;
-			$this->currentServer = 'LOCAL';
-			return $previousServer;
-		} else {
-			$this->baseUrl = $this->remoteBaseUrl;
-			$this->currentServer = 'REMOTE';
-			return $previousServer;
-		}
-	}
-
-	/**
-	 * @When /^sending "([^"]*)" to "([^"]*)"$/
-	 * @param string $verb
-	 * @param string $url
-	 */
-	public function sendingTo($verb, $url) {
-		$this->sendingToWith($verb, $url, null);
-	}
-
-	/**
-	 * Parses the xml or json answer to get ocs response which doesn't match with
-	 * http one in v1 of the api.
-	 *
-	 * @param ResponseInterface $response
-	 * @return string
-	 */
-	public function getOCSResponseCode($response): int {
-		if ($response === null) {
-			throw new \RuntimeException('No response available');
-		}
-
-		$body = (string)$response->getBody();
-		if (str_starts_with($body, '<')) {
-			$body = simplexml_load_string($body);
-			if ($body === false) {
-				throw new \RuntimeException('Could not parse OCS response, body is not valid XML');
-			}
-			return (int)$body->meta[0]->statuscode;
-		}
-
-		return json_decode($body, true)['ocs']['meta']['statuscode'];
-	}
-
-	/**
-	 * This function is needed to use a vertical fashion in the gherkin tables.
-	 *
-	 * @param array $arrayOfArrays
-	 * @return array
-	 */
-	public function simplifyArray($arrayOfArrays) {
-		$a = array_map(function ($subArray) {
-			return $subArray[0];
-		}, $arrayOfArrays);
-		return $a;
-	}
-
-	/**
-	 * @When /^sending "([^"]*)" to "([^"]*)" with$/
-	 * @param string $verb
-	 * @param string $url
-	 * @param TableNode $body
-	 */
-	public function sendingToWith($verb, $url, $body) {
-		$fullUrl = $this->baseUrl . "v{$this->apiVersion}.php" . $url;
-		$client = new Client();
-		$options = [];
-		if ($this->currentUser === 'admin') {
-			$options['auth'] = $this->adminUser;
-		} elseif (strpos($this->currentUser, 'anonymous') !== 0) {
-			$options['auth'] = [$this->currentUser, $this->regularUser];
-		}
-		$options['headers'] = [
-			'OCS-APIRequest' => 'true'
-		];
-		if ($body instanceof TableNode) {
-			$fd = $body->getRowsHash();
-			$options['form_params'] = $fd;
-		}
-
-		// TODO: Fix this hack!
-		if ($verb === 'PUT' && $body === null) {
-			$options['form_params'] = [
-				'foo' => 'bar',
-			];
-		}
-
-		try {
-			$this->response = $client->request($verb, $fullUrl, $options);
-		} catch (ClientException $ex) {
-			$this->response = $ex->getResponse();
-		} catch (ServerException $ex) {
-			$this->response = $ex->getResponse();
-		}
-	}
-
-	/**
-	 * @param string $verb
-	 * @param string $url
-	 * @param TableNode|array|null $body
-	 * @param array $headers
-	 */
-	protected function sendRequestForJSON(string $verb, string $url, $body = null, array $headers = []): void {
-		$fullUrl = $this->baseUrl . "v{$this->apiVersion}.php" . $url;
-		$client = new Client();
-		$options = [];
-		if ($this->currentUser === 'admin') {
-			$options['auth'] = ['admin', 'admin'];
-		} elseif (strpos($this->currentUser, 'anonymous') !== 0) {
-			$options['auth'] = [$this->currentUser, $this->regularUser];
-		}
-		if ($body instanceof TableNode) {
-			$fd = $body->getRowsHash();
-			$options['form_params'] = $fd;
-		} elseif (is_array($body)) {
-			$options['form_params'] = $body;
-		}
-
-		$options['headers'] = array_merge($headers, [
-			'OCS-ApiRequest' => 'true',
-			'Accept' => 'application/json',
-		]);
-
-		try {
-			$this->response = $client->{$verb}($fullUrl, $options);
-		} catch (ClientException $ex) {
-			$this->response = $ex->getResponse();
-		}
-	}
-
-	/**
-	 * @When /^sending "([^"]*)" with exact url to "([^"]*)"$/
-	 * @param string $verb
-	 * @param string $url
-	 */
-	public function sendingToDirectUrl($verb, $url) {
-		$this->sendingToWithDirectUrl($verb, $url, null);
-	}
-
-	public function sendingToWithDirectUrl($verb, $url, $body) {
-		$fullUrl = substr($this->baseUrl, 0, -5) . $url;
-		$client = new Client();
-		$options = [];
-		if ($this->currentUser === 'admin') {
-			$options['auth'] = $this->adminUser;
-		} elseif (strpos($this->currentUser, 'anonymous') !== 0) {
-			$options['auth'] = [$this->currentUser, $this->regularUser];
-		}
-		if ($body instanceof TableNode) {
-			$fd = $body->getRowsHash();
-			$options['form_params'] = $fd;
-		}
-
-		try {
-			$this->response = $client->request($verb, $fullUrl, $options);
-		} catch (ClientException $ex) {
-			$this->response = $ex->getResponse();
-		}
-	}
-
-	public function isExpectedUrl($possibleUrl, $finalPart) {
-		$baseUrlChopped = substr($this->baseUrl, 0, -4);
-		$endCharacter = strlen($baseUrlChopped) + strlen($finalPart);
-		return (substr($possibleUrl, 0, $endCharacter) == "$baseUrlChopped" . "$finalPart");
-	}
-
-	/**
-	 * @Then /^the OCS status code should be "([^"]*)"$/
-	 * @param int $statusCode
-	 */
-	public function theOCSStatusCodeShouldBe($statusCode) {
-		Assert::assertEquals($statusCode, $this->getOCSResponseCode($this->response));
-	}
-
-	/**
-	 * @Then /^the HTTP status code should be "([^"]*)"$/
-	 * @param int $statusCode
-	 */
-	public function theHTTPStatusCodeShouldBe($statusCode) {
-		Assert::assertEquals($statusCode, $this->response->getStatusCode());
-	}
-
-	/**
-	 * @Then /^the Content-Type should be "([^"]*)"$/
-	 * @param string $contentType
-	 */
-	public function theContentTypeShouldbe($contentType) {
-		Assert::assertEquals($contentType, $this->response->getHeader('Content-Type')[0]);
-	}
-
-	/**
-	 * @param ResponseInterface $response
-	 */
-	private function extracRequestTokenFromResponse(ResponseInterface $response) {
-		$this->requestToken = substr(preg_replace('/(.*)data-requesttoken="(.*)">(.*)/sm', '\2', $response->getBody()->getContents()), 0, 89);
-	}
-
-	/**
-	 * @Given Logging in using web as :user
-	 * @param string $user
-	 */
-	public function loggingInUsingWebAs($user) {
-		$baseUrl = substr($this->baseUrl, 0, -5);
-		$loginUrl = $baseUrl . '/index.php/login';
-		// Request a new session and extract CSRF token
-		$client = new Client();
-		$response = $client->get(
-			$loginUrl,
-			[
-				'cookies' => $this->cookieJar,
-			]
-		);
-		$this->extracRequestTokenFromResponse($response);
-
-		// Login and extract new token
-		$password = ($user === 'admin') ? 'admin' : '123456';
-		$client = new Client();
-		$response = $client->post(
-			$loginUrl,
-			[
-				'form_params' => [
-					'user' => $user,
-					'password' => $password,
-					'requesttoken' => $this->requestToken,
-				],
-				'cookies' => $this->cookieJar,
-				'headers' => [
-					'Origin' => $baseUrl,
-				],
-			]
-		);
-		$this->extracRequestTokenFromResponse($response);
-	}
-
-	/**
-	 * @When Sending a :method to :url with requesttoken
-	 * @param string $method
-	 * @param string $url
-	 * @param TableNode|array|null $body
-	 */
-	public function sendingAToWithRequesttoken($method, $url, $body = null) {
-		$baseUrl = substr($this->baseUrl, 0, -5);
-
-		$options = [
-			'cookies' => $this->cookieJar,
-			'headers' => [
-				'requesttoken' => $this->requestToken
-			],
-		];
-
-		if ($body instanceof TableNode) {
-			$fd = $body->getRowsHash();
-			$options['form_params'] = $fd;
-		} elseif ($body) {
-			$options = array_merge_recursive($options, $body);
-		}
-
-		$client = new Client();
-		try {
-			$this->response = $client->request(
-				$method,
-				$baseUrl . $url,
-				$options
-			);
-		} catch (ClientException $e) {
-			$this->response = $e->getResponse();
-		}
-	}
-
-	/**
-	 * @When Sending a :method to :url without requesttoken
-	 * @param string $method
-	 * @param string $url
-	 */
-	public function sendingAToWithoutRequesttoken($method, $url) {
-		$baseUrl = substr($this->baseUrl, 0, -5);
-
-		$client = new Client();
-		try {
-			$this->response = $client->request(
-				$method,
-				$baseUrl . $url,
-				[
-					'cookies' => $this->cookieJar
-				]
-			);
-		} catch (ClientException $e) {
-			$this->response = $e->getResponse();
-		}
-	}
-
-	public static function removeFile($path, $filename) {
-		if (file_exists("$path" . "$filename")) {
-			unlink("$path" . "$filename");
-		}
-	}
-
-	/**
-	 * @Given User :user modifies text of :filename with text :text
-	 * @param string $user
-	 * @param string $filename
-	 * @param string $text
-	 */
-	public function modifyTextOfFile($user, $filename, $text) {
-		self::removeFile($this->getDataDirectory() . "/$user/files", "$filename");
-		file_put_contents($this->getDataDirectory() . "/$user/files" . "$filename", "$text");
-	}
-
-	private function getDataDirectory() {
-		// Based on "runOcc" from CommandLine trait
-		$args = ['config:system:get', 'datadirectory'];
-		$args = array_map(function ($arg) {
-			return escapeshellarg($arg);
-		}, $args);
-		$args[] = '--no-ansi --no-warnings';
-		$args = implode(' ', $args);
-
-		$descriptor = [
-			0 => ['pipe', 'r'],
-			1 => ['pipe', 'w'],
-			2 => ['pipe', 'w'],
-		];
-		$process = proc_open('php console.php ' . $args, $descriptor, $pipes, $ocPath = '../..');
-		$lastStdOut = stream_get_contents($pipes[1]);
-		proc_close($process);
-
-		return trim($lastStdOut);
-	}
-
-	/**
-	 * @Given file :filename is created :times times in :user user data
-	 * @param string $filename
-	 * @param string $times
-	 * @param string $user
-	 */
-	public function fileIsCreatedTimesInUserData($filename, $times, $user) {
-		for ($i = 0; $i < $times; $i++) {
-			file_put_contents($this->getDataDirectory() . "/$user/files" . "$filename-$i", "content-$i");
-		}
-	}
-
-	public function createFileSpecificSize($name, $size) {
-		$file = fopen('work/' . "$name", 'w');
-		fseek($file, $size - 1, SEEK_CUR);
-		fwrite($file, 'a'); // write a dummy char at SIZE position
-		fclose($file);
-	}
-
-	public function createFileWithText($name, $text) {
-		$file = fopen('work/' . "$name", 'w');
-		fwrite($file, $text);
-		fclose($file);
-	}
-
-	/**
-	 * @Given file :filename of size :size is created in local storage
-	 * @param string $filename
-	 * @param string $size
-	 */
-	public function fileIsCreatedInLocalStorageWithSize($filename, $size) {
-		$this->createFileSpecificSize("local_storage/$filename", $size);
-	}
-
-	/**
-	 * @Given file :filename with text :text is created in local storage
-	 * @param string $filename
-	 * @param string $text
-	 */
-	public function fileIsCreatedInLocalStorageWithText($filename, $text) {
-		$this->createFileWithText("local_storage/$filename", $text);
-	}
-
-	/**
-	 * @When Sleep for :seconds seconds
-	 * @param int $seconds
-	 */
-	public function sleepForSeconds($seconds) {
-		sleep((int)$seconds);
-	}
-
-	/**
-	 * @BeforeSuite
-	 */
-	public static function addFilesToSkeleton() {
-		for ($i = 0; $i < 5; $i++) {
-			file_put_contents('../../core/skeleton/' . 'textfile' . "$i" . '.txt', "Nextcloud test text file\n");
-		}
-		if (!file_exists('../../core/skeleton/FOLDER')) {
-			mkdir('../../core/skeleton/FOLDER', 0777, true);
-		}
-		if (!file_exists('../../core/skeleton/PARENT')) {
-			mkdir('../../core/skeleton/PARENT', 0777, true);
-		}
-		file_put_contents('../../core/skeleton/PARENT/' . 'parent.txt', "Nextcloud test text file\n");
-		if (!file_exists('../../core/skeleton/PARENT/CHILD')) {
-			mkdir('../../core/skeleton/PARENT/CHILD', 0777, true);
-		}
-		file_put_contents('../../core/skeleton/PARENT/CHILD/' . 'child.txt', "Nextcloud test text file\n");
-	}
-
-	/**
-	 * @AfterSuite
-	 */
-	public static function removeFilesFromSkeleton() {
-		for ($i = 0; $i < 5; $i++) {
-			self::removeFile('../../core/skeleton/', 'textfile' . "$i" . '.txt');
-		}
-		if (is_dir('../../core/skeleton/FOLDER')) {
-			rmdir('../../core/skeleton/FOLDER');
-		}
-		self::removeFile('../../core/skeleton/PARENT/CHILD/', 'child.txt');
-		if (is_dir('../../core/skeleton/PARENT/CHILD')) {
-			rmdir('../../core/skeleton/PARENT/CHILD');
-		}
-		self::removeFile('../../core/skeleton/PARENT/', 'parent.txt');
-		if (is_dir('../../core/skeleton/PARENT')) {
-			rmdir('../../core/skeleton/PARENT');
-		}
-	}
-
-	/**
-	 * @BeforeScenario @local_storage
-	 */
-	public static function removeFilesFromLocalStorageBefore() {
-		$dir = './work/local_storage/';
-		$di = new RecursiveDirectoryIterator($dir, FilesystemIterator::SKIP_DOTS);
-		$ri = new RecursiveIteratorIterator($di, RecursiveIteratorIterator::CHILD_FIRST);
-		foreach ($ri as $file) {
-			$file->isDir() ? rmdir($file) : unlink($file);
-		}
-	}
-
-	/**
-	 * @AfterScenario @local_storage
-	 */
-	public static function removeFilesFromLocalStorageAfter() {
-		$dir = './work/local_storage/';
-		$di = new RecursiveDirectoryIterator($dir, FilesystemIterator::SKIP_DOTS);
-		$ri = new RecursiveIteratorIterator($di, RecursiveIteratorIterator::CHILD_FIRST);
-		foreach ($ri as $file) {
-			$file->isDir() ? rmdir($file) : unlink($file);
-		}
-	}
-
-	/**
-	 * @Given /^cookies are reset$/
-	 */
-	public function cookiesAreReset() {
-		$this->cookieJar = new CookieJar();
-	}
-
-	/**
-	 * @Then The following headers should be set
-	 * @param TableNode $table
-	 * @throws \Exception
-	 */
-	public function theFollowingHeadersShouldBeSet(TableNode $table) {
-		foreach ($table->getTable() as $header) {
-			$headerName = $header[0];
-			$expectedHeaderValue = $header[1];
-			$returnedHeader = $this->response->getHeader($headerName)[0];
-			if ($returnedHeader !== $expectedHeaderValue) {
-				throw new \Exception(
-					sprintf(
-						"Expected value '%s' for header '%s', got '%s'",
-						$expectedHeaderValue,
-						$headerName,
-						$returnedHeader
-					)
-				);
-			}
-		}
-	}
+    use Auth;
+    use Avatar;
+    use Download;
+    use Mail;
+    use Theming;
+
+    /** @var string */
+    private $currentUser = '';
+
+    /** @var string */
+    private $currentServer = '';
+
+    /** @var string */
+    private $baseUrl = '';
+
+    /** @var int */
+    private $apiVersion = 1;
+
+    /** @var ResponseInterface */
+    private $response = null;
+
+    /** @var CookieJar */
+    private $cookieJar;
+
+    /** @var string */
+    private $requestToken;
+
+    protected $adminUser;
+    protected $regularUser;
+    protected $localBaseUrl;
+    protected $remoteBaseUrl;
+
+    public function __construct($baseUrl, $admin, $regular_user_password) {
+        // Initialize your context here
+        $this->baseUrl = $baseUrl;
+        $this->adminUser = $admin;
+        $this->regularUser = $regular_user_password;
+        $this->localBaseUrl = $this->baseUrl;
+        $this->remoteBaseUrl = $this->baseUrl;
+        $this->currentServer = 'LOCAL';
+        $this->cookieJar = new CookieJar();
+
+        // in case of ci deployment we take the server url from the environment
+        $testServerUrl = getenv('TEST_SERVER_URL');
+        if ($testServerUrl !== false) {
+            $this->baseUrl = $testServerUrl;
+            $this->localBaseUrl = $testServerUrl;
+        }
+
+        // federated server url from the environment
+        $testRemoteServerUrl = getenv('TEST_SERVER_FED_URL');
+        if ($testRemoteServerUrl !== false) {
+            $this->remoteBaseUrl = $testRemoteServerUrl;
+        }
+    }
+
+    /**
+     * @Given /^using api version "(\d+)"$/
+     * @param string $version
+     */
+    public function usingApiVersion($version) {
+        $this->apiVersion = (int)$version;
+    }
+
+    /**
+     * @Given /^As an "([^"]*)"$/
+     * @param string $user
+     */
+    public function asAn($user) {
+        $this->currentUser = $user;
+    }
+
+    /**
+     * @Given /^Using server "(LOCAL|REMOTE)"$/
+     * @param string $server
+     * @return string Previous used server
+     */
+    public function usingServer($server) {
+        $previousServer = $this->currentServer;
+        if ($server === 'LOCAL') {
+            $this->baseUrl = $this->localBaseUrl;
+            $this->currentServer = 'LOCAL';
+            return $previousServer;
+        } else {
+            $this->baseUrl = $this->remoteBaseUrl;
+            $this->currentServer = 'REMOTE';
+            return $previousServer;
+        }
+    }
+
+    /**
+     * @When /^sending "([^"]*)" to "([^"]*)"$/
+     * @param string $verb
+     * @param string $url
+     */
+    public function sendingTo($verb, $url) {
+        $this->sendingToWith($verb, $url, null);
+    }
+
+    /**
+     * Parses the xml or json answer to get ocs response which doesn't match with
+     * http one in v1 of the api.
+     *
+     * @param ResponseInterface $response
+     * @return string
+     */
+    public function getOCSResponseCode($response): int {
+        if ($response === null) {
+            throw new \RuntimeException('No response available');
+        }
+
+        $body = (string)$response->getBody();
+        if (str_starts_with($body, '<')) {
+            $body = simplexml_load_string($body);
+            if ($body === false) {
+                throw new \RuntimeException('Could not parse OCS response, body is not valid XML');
+            }
+            return (int)$body->meta[0]->statuscode;
+        }
+
+        return json_decode($body, true)['ocs']['meta']['statuscode'];
+    }
+
+    /**
+     * This function is needed to use a vertical fashion in the gherkin tables.
+     *
+     * @param array $arrayOfArrays
+     * @return array
+     */
+    public function simplifyArray($arrayOfArrays) {
+        $a = array_map(function ($subArray) {
+            return $subArray[0];
+        }, $arrayOfArrays);
+        return $a;
+    }
+
+    /**
+     * @When /^sending "([^"]*)" to "([^"]*)" with$/
+     * @param string $verb
+     * @param string $url
+     * @param TableNode $body
+     */
+    public function sendingToWith($verb, $url, $body) {
+        $fullUrl = $this->baseUrl . "v{$this->apiVersion}.php" . $url;
+        $client = new Client();
+        $options = [];
+        if ($this->currentUser === 'admin') {
+            $options['auth'] = $this->adminUser;
+        } elseif (strpos($this->currentUser, 'anonymous') !== 0) {
+            $options['auth'] = [$this->currentUser, $this->regularUser];
+        }
+        $options['headers'] = [
+            'OCS-APIRequest' => 'true'
+        ];
+        if ($body instanceof TableNode) {
+            $fd = $body->getRowsHash();
+            $options['form_params'] = $fd;
+        }
+
+        // TODO: Fix this hack!
+        if ($verb === 'PUT' && $body === null) {
+            $options['form_params'] = [
+                'foo' => 'bar',
+            ];
+        }
+
+        try {
+            $this->response = $client->request($verb, $fullUrl, $options);
+        } catch (ClientException $ex) {
+            $this->response = $ex->getResponse();
+        } catch (ServerException $ex) {
+            $this->response = $ex->getResponse();
+        }
+    }
+
+    /**
+     * @param string $verb
+     * @param string $url
+     * @param TableNode|array|null $body
+     * @param array $headers
+     */
+    protected function sendRequestForJSON(string $verb, string $url, $body = null, array $headers = []): void {
+        $fullUrl = $this->baseUrl . "v{$this->apiVersion}.php" . $url;
+        $client = new Client();
+        $options = [];
+        if ($this->currentUser === 'admin') {
+            $options['auth'] = ['admin', 'admin'];
+        } elseif (strpos($this->currentUser, 'anonymous') !== 0) {
+            $options['auth'] = [$this->currentUser, $this->regularUser];
+        }
+        if ($body instanceof TableNode) {
+            $fd = $body->getRowsHash();
+            $options['form_params'] = $fd;
+        } elseif (is_array($body)) {
+            $options['form_params'] = $body;
+        }
+
+        $options['headers'] = array_merge($headers, [
+            'OCS-ApiRequest' => 'true',
+            'Accept' => 'application/json',
+        ]);
+
+        try {
+            $this->response = $client->{$verb}($fullUrl, $options);
+        } catch (ClientException $ex) {
+            $this->response = $ex->getResponse();
+        }
+    }
+
+    /**
+     * @When /^sending "([^"]*)" with exact url to "([^"]*)"$/
+     * @param string $verb
+     * @param string $url
+     */
+    public function sendingToDirectUrl($verb, $url) {
+        $this->sendingToWithDirectUrl($verb, $url, null);
+    }
+
+    public function sendingToWithDirectUrl($verb, $url, $body) {
+        $fullUrl = substr($this->baseUrl, 0, -5) . $url;
+        $client = new Client();
+        $options = [];
+        if ($this->currentUser === 'admin') {
+            $options['auth'] = $this->adminUser;
+        } elseif (strpos($this->currentUser, 'anonymous') !== 0) {
+            $options['auth'] = [$this->currentUser, $this->regularUser];
+        }
+        if ($body instanceof TableNode) {
+            $fd = $body->getRowsHash();
+            $options['form_params'] = $fd;
+        }
+
+        try {
+            $this->response = $client->request($verb, $fullUrl, $options);
+        } catch (ClientException $ex) {
+            $this->response = $ex->getResponse();
+        }
+    }
+
+    public function isExpectedUrl($possibleUrl, $finalPart) {
+        $baseUrlChopped = substr($this->baseUrl, 0, -4);
+        $endCharacter = strlen($baseUrlChopped) + strlen($finalPart);
+        return (substr($possibleUrl, 0, $endCharacter) == "$baseUrlChopped" . "$finalPart");
+    }
+
+    /**
+     * @Then /^the OCS status code should be "([^"]*)"$/
+     * @param int $statusCode
+     */
+    public function theOCSStatusCodeShouldBe($statusCode) {
+        Assert::assertEquals($statusCode, $this->getOCSResponseCode($this->response));
+    }
+
+    /**
+     * @Then /^the HTTP status code should be "([^"]*)"$/
+     * @param int $statusCode
+     */
+    public function theHTTPStatusCodeShouldBe($statusCode) {
+        Assert::assertEquals($statusCode, $this->response->getStatusCode());
+    }
+
+    /**
+     * @Then /^the Content-Type should be "([^"]*)"$/
+     * @param string $contentType
+     */
+    public function theContentTypeShouldbe($contentType) {
+        Assert::assertEquals($contentType, $this->response->getHeader('Content-Type')[0]);
+    }
+
+    /**
+     * @param ResponseInterface $response
+     */
+    private function extracRequestTokenFromResponse(ResponseInterface $response) {
+        $this->requestToken = substr(preg_replace('/(.*)data-requesttoken="(.*)">(.*)/sm', '\2', $response->getBody()->getContents()), 0, 89);
+    }
+
+    /**
+     * @Given Logging in using web as :user
+     * @param string $user
+     */
+    public function loggingInUsingWebAs($user) {
+        $baseUrl = substr($this->baseUrl, 0, -5);
+        $loginUrl = $baseUrl . '/index.php/login';
+        // Request a new session and extract CSRF token
+        $client = new Client();
+        $response = $client->get(
+            $loginUrl,
+            [
+                'cookies' => $this->cookieJar,
+            ]
+        );
+        $this->extracRequestTokenFromResponse($response);
+
+        // Login and extract new token
+        $password = ($user === 'admin') ? 'admin' : '123456';
+        $client = new Client();
+        $response = $client->post(
+            $loginUrl,
+            [
+                'form_params' => [
+                    'user' => $user,
+                    'password' => $password,
+                    'requesttoken' => $this->requestToken,
+                ],
+                'cookies' => $this->cookieJar,
+                'headers' => [
+                    'Origin' => $baseUrl,
+                ],
+            ]
+        );
+        $this->extracRequestTokenFromResponse($response);
+    }
+
+    /**
+     * @When Sending a :method to :url with requesttoken
+     * @param string $method
+     * @param string $url
+     * @param TableNode|array|null $body
+     */
+    public function sendingAToWithRequesttoken($method, $url, $body = null) {
+        $baseUrl = substr($this->baseUrl, 0, -5);
+
+        $options = [
+            'cookies' => $this->cookieJar,
+            'headers' => [
+                'requesttoken' => $this->requestToken
+            ],
+        ];
+
+        if ($body instanceof TableNode) {
+            $fd = $body->getRowsHash();
+            $options['form_params'] = $fd;
+        } elseif ($body) {
+            $options = array_merge_recursive($options, $body);
+        }
+
+        $client = new Client();
+        try {
+            $this->response = $client->request(
+                $method,
+                $baseUrl . $url,
+                $options
+            );
+        } catch (ClientException $e) {
+            $this->response = $e->getResponse();
+        }
+    }
+
+    /**
+     * @When Sending a :method to :url without requesttoken
+     * @param string $method
+     * @param string $url
+     */
+    public function sendingAToWithoutRequesttoken($method, $url) {
+        $baseUrl = substr($this->baseUrl, 0, -5);
+
+        $client = new Client();
+        try {
+            $this->response = $client->request(
+                $method,
+                $baseUrl . $url,
+                [
+                    'cookies' => $this->cookieJar
+                ]
+            );
+        } catch (ClientException $e) {
+            $this->response = $e->getResponse();
+        }
+    }
+
+    public static function removeFile($path, $filename) {
+        if (file_exists("$path" . "$filename")) {
+            unlink("$path" . "$filename");
+        }
+    }
+
+    /**
+     * @Given User :user modifies text of :filename with text :text
+     * @param string $user
+     * @param string $filename
+     * @param string $text
+     */
+    public function modifyTextOfFile($user, $filename, $text) {
+        self::removeFile($this->getDataDirectory() . "/$user/files", "$filename");
+        file_put_contents($this->getDataDirectory() . "/$user/files" . "$filename", "$text");
+    }
+
+    private function getDataDirectory() {
+        // Based on "runOcc" from CommandLine trait
+        $args = ['config:system:get', 'datadirectory'];
+        $args = array_map(function ($arg) {
+            return escapeshellarg($arg);
+        }, $args);
+        $args[] = '--no-ansi --no-warnings';
+        $args = implode(' ', $args);
+
+        $descriptor = [
+            0 => ['pipe', 'r'],
+            1 => ['pipe', 'w'],
+            2 => ['pipe', 'w'],
+        ];
+        $process = proc_open('php console.php ' . $args, $descriptor, $pipes, $ocPath = '../..');
+        $lastStdOut = stream_get_contents($pipes[1]);
+        proc_close($process);
+
+        return trim($lastStdOut);
+    }
+
+    /**
+     * @Given file :filename is created :times times in :user user data
+     * @param string $filename
+     * @param string $times
+     * @param string $user
+     */
+    public function fileIsCreatedTimesInUserData($filename, $times, $user) {
+        for ($i = 0; $i < $times; $i++) {
+            file_put_contents($this->getDataDirectory() . "/$user/files" . "$filename-$i", "content-$i");
+        }
+    }
+
+    public function createFileSpecificSize($name, $size) {
+        $file = fopen('work/' . "$name", 'w');
+        fseek($file, $size - 1, SEEK_CUR);
+        fwrite($file, 'a'); // write a dummy char at SIZE position
+        fclose($file);
+    }
+
+    public function createFileWithText($name, $text) {
+        $file = fopen('work/' . "$name", 'w');
+        fwrite($file, $text);
+        fclose($file);
+    }
+
+    /**
+     * @Given file :filename of size :size is created in local storage
+     * @param string $filename
+     * @param string $size
+     */
+    public function fileIsCreatedInLocalStorageWithSize($filename, $size) {
+        $this->createFileSpecificSize("local_storage/$filename", $size);
+    }
+
+    /**
+     * @Given file :filename with text :text is created in local storage
+     * @param string $filename
+     * @param string $text
+     */
+    public function fileIsCreatedInLocalStorageWithText($filename, $text) {
+        $this->createFileWithText("local_storage/$filename", $text);
+    }
+
+    /**
+     * @When Sleep for :seconds seconds
+     * @param int $seconds
+     */
+    public function sleepForSeconds($seconds) {
+        sleep((int)$seconds);
+    }
+
+    /**
+     * @BeforeSuite
+     */
+    public static function addFilesToSkeleton() {
+        for ($i = 0; $i < 5; $i++) {
+            file_put_contents('../../core/skeleton/' . 'textfile' . "$i" . '.txt', "Nextcloud test text file\n");
+        }
+        if (!file_exists('../../core/skeleton/FOLDER')) {
+            mkdir('../../core/skeleton/FOLDER', 0777, true);
+        }
+        if (!file_exists('../../core/skeleton/PARENT')) {
+            mkdir('../../core/skeleton/PARENT', 0777, true);
+        }
+        file_put_contents('../../core/skeleton/PARENT/' . 'parent.txt', "Nextcloud test text file\n");
+        if (!file_exists('../../core/skeleton/PARENT/CHILD')) {
+            mkdir('../../core/skeleton/PARENT/CHILD', 0777, true);
+        }
+        file_put_contents('../../core/skeleton/PARENT/CHILD/' . 'child.txt', "Nextcloud test text file\n");
+    }
+
+    /**
+     * @AfterSuite
+     */
+    public static function removeFilesFromSkeleton() {
+        for ($i = 0; $i < 5; $i++) {
+            self::removeFile('../../core/skeleton/', 'textfile' . "$i" . '.txt');
+        }
+        if (is_dir('../../core/skeleton/FOLDER')) {
+            rmdir('../../core/skeleton/FOLDER');
+        }
+        self::removeFile('../../core/skeleton/PARENT/CHILD/', 'child.txt');
+        if (is_dir('../../core/skeleton/PARENT/CHILD')) {
+            rmdir('../../core/skeleton/PARENT/CHILD');
+        }
+        self::removeFile('../../core/skeleton/PARENT/', 'parent.txt');
+        if (is_dir('../../core/skeleton/PARENT')) {
+            rmdir('../../core/skeleton/PARENT');
+        }
+    }
+
+    /**
+     * @BeforeScenario @local_storage
+     */
+    public static function removeFilesFromLocalStorageBefore() {
+        $dir = './work/local_storage/';
+        $di = new RecursiveDirectoryIterator($dir, FilesystemIterator::SKIP_DOTS);
+        $ri = new RecursiveIteratorIterator($di, RecursiveIteratorIterator::CHILD_FIRST);
+        foreach ($ri as $file) {
+            $file->isDir() ? rmdir($file) : unlink($file);
+        }
+    }
+
+    /**
+     * @AfterScenario @local_storage
+     */
+    public static function removeFilesFromLocalStorageAfter() {
+        $dir = './work/local_storage/';
+        $di = new RecursiveDirectoryIterator($dir, FilesystemIterator::SKIP_DOTS);
+        $ri = new RecursiveIteratorIterator($di, RecursiveIteratorIterator::CHILD_FIRST);
+        foreach ($ri as $file) {
+            $file->isDir() ? rmdir($file) : unlink($file);
+        }
+    }
+
+    /**
+     * @Given /^cookies are reset$/
+     */
+    public function cookiesAreReset() {
+        $this->cookieJar = new CookieJar();
+    }
+
+    /**
+     * @Then The following headers should be set
+     * @param TableNode $table
+     * @throws \Exception
+     */
+    public function theFollowingHeadersShouldBeSet(TableNode $table) {
+        foreach ($table->getTable() as $header) {
+            $headerName = $header[0];
+            $expectedHeaderValue = $header[1];
+            $returnedHeader = $this->response->getHeader($headerName)[0];
+            if ($returnedHeader !== $expectedHeaderValue) {
+                throw new \Exception(
+                    sprintf(
+                        "Expected value '%s' for header '%s', got '%s'",
+                        $expectedHeaderValue,
+                        $headerName,
+                        $returnedHeader
+                    )
+                );
+            }
+        }
+    }
 }

Spacing +26 added lines, -26 removed lines

@@ -13,7 +13,7 @@ 
 use PHPUnit\Framework\Assert;
 use Psr\Http\Message\ResponseInterface;
 
-require __DIR__ . '/../../vendor/autoload.php';
+require __DIR__.'/../../vendor/autoload.php';
 
 trait BasicStructure {
 	use Auth;
@@ -77,7 +77,7 @@ 
 	 * @param string $version
 	 */
 	public function usingApiVersion($version) {
-		$this->apiVersion = (int)$version;
+		$this->apiVersion = (int) $version;
 	}
 
 	/**
@@ -127,13 +127,13 @@ 
 			throw new \RuntimeException('No response available');
 		}
 
-		$body = (string)$response->getBody();
+		$body = (string) $response->getBody();
 		if (str_starts_with($body, '<')) {
 			$body = simplexml_load_string($body);
 			if ($body === false) {
 				throw new \RuntimeException('Could not parse OCS response, body is not valid XML');
 			}
-			return (int)$body->meta[0]->statuscode;
+			return (int) $body->meta[0]->statuscode;
 		}
 
 		return json_decode($body, true)['ocs']['meta']['statuscode'];
@@ -146,7 +146,7 @@ 
 	 * @return array
 	 */
 	public function simplifyArray($arrayOfArrays) {
-		$a = array_map(function ($subArray) {
+		$a = array_map(function($subArray) {
 			return $subArray[0];
 		}, $arrayOfArrays);
 		return $a;
@@ -159,7 +159,7 @@ 
 	 * @param TableNode $body
 	 */
 	public function sendingToWith($verb, $url, $body) {
-		$fullUrl = $this->baseUrl . "v{$this->apiVersion}.php" . $url;
+		$fullUrl = $this->baseUrl."v{$this->apiVersion}.php".$url;
 		$client = new Client();
 		$options = [];
 		if ($this->currentUser === 'admin') {
@@ -198,7 +198,7 @@ 
 	 * @param array $headers
 	 */
 	protected function sendRequestForJSON(string $verb, string $url, $body = null, array $headers = []): void {
-		$fullUrl = $this->baseUrl . "v{$this->apiVersion}.php" . $url;
+		$fullUrl = $this->baseUrl."v{$this->apiVersion}.php".$url;
 		$client = new Client();
 		$options = [];
 		if ($this->currentUser === 'admin') {
@@ -235,7 +235,7 @@ 
 	}
 
 	public function sendingToWithDirectUrl($verb, $url, $body) {
-		$fullUrl = substr($this->baseUrl, 0, -5) . $url;
+		$fullUrl = substr($this->baseUrl, 0, -5).$url;
 		$client = new Client();
 		$options = [];
 		if ($this->currentUser === 'admin') {
@@ -258,7 +258,7 @@ 
 	public function isExpectedUrl($possibleUrl, $finalPart) {
 		$baseUrlChopped = substr($this->baseUrl, 0, -4);
 		$endCharacter = strlen($baseUrlChopped) + strlen($finalPart);
-		return (substr($possibleUrl, 0, $endCharacter) == "$baseUrlChopped" . "$finalPart");
+		return (substr($possibleUrl, 0, $endCharacter) == "$baseUrlChopped"."$finalPart");
 	}
 
 	/**
@@ -298,7 +298,7 @@ 
 	 */
 	public function loggingInUsingWebAs($user) {
 		$baseUrl = substr($this->baseUrl, 0, -5);
-		$loginUrl = $baseUrl . '/index.php/login';
+		$loginUrl = $baseUrl.'/index.php/login';
 		// Request a new session and extract CSRF token
 		$client = new Client();
 		$response = $client->get(
@@ -356,7 +356,7 @@ 
 		try {
 			$this->response = $client->request(
 				$method,
-				$baseUrl . $url,
+				$baseUrl.$url,
 				$options
 			);
 		} catch (ClientException $e) {
@@ -376,7 +376,7 @@ 
 		try {
 			$this->response = $client->request(
 				$method,
-				$baseUrl . $url,
+				$baseUrl.$url,
 				[
 					'cookies' => $this->cookieJar
 				]
@@ -387,8 +387,8 @@ 
 	}
 
 	public static function removeFile($path, $filename) {
-		if (file_exists("$path" . "$filename")) {
-			unlink("$path" . "$filename");
+		if (file_exists("$path"."$filename")) {
+			unlink("$path"."$filename");
 		}
 	}
 
@@ -399,14 +399,14 @@ 
 	 * @param string $text
 	 */
 	public function modifyTextOfFile($user, $filename, $text) {
-		self::removeFile($this->getDataDirectory() . "/$user/files", "$filename");
-		file_put_contents($this->getDataDirectory() . "/$user/files" . "$filename", "$text");
+		self::removeFile($this->getDataDirectory()."/$user/files", "$filename");
+		file_put_contents($this->getDataDirectory()."/$user/files"."$filename", "$text");
 	}
 
 	private function getDataDirectory() {
 		// Based on "runOcc" from CommandLine trait
 		$args = ['config:system:get', 'datadirectory'];
-		$args = array_map(function ($arg) {
+		$args = array_map(function($arg) {
 			return escapeshellarg($arg);
 		}, $args);
 		$args[] = '--no-ansi --no-warnings';
@@ -417,7 +417,7 @@ 
 			1 => ['pipe', 'w'],
 			2 => ['pipe', 'w'],
 		];
-		$process = proc_open('php console.php ' . $args, $descriptor, $pipes, $ocPath = '../..');
+		$process = proc_open('php console.php '.$args, $descriptor, $pipes, $ocPath = '../..');
 		$lastStdOut = stream_get_contents($pipes[1]);
 		proc_close($process);
 
@@ -432,19 +432,19 @@ 
 	 */
 	public function fileIsCreatedTimesInUserData($filename, $times, $user) {
 		for ($i = 0; $i < $times; $i++) {
-			file_put_contents($this->getDataDirectory() . "/$user/files" . "$filename-$i", "content-$i");
+			file_put_contents($this->getDataDirectory()."/$user/files"."$filename-$i", "content-$i");
 		}
 	}
 
 	public function createFileSpecificSize($name, $size) {
-		$file = fopen('work/' . "$name", 'w');
+		$file = fopen('work/'."$name", 'w');
 		fseek($file, $size - 1, SEEK_CUR);
 		fwrite($file, 'a'); // write a dummy char at SIZE position
 		fclose($file);
 	}
 
 	public function createFileWithText($name, $text) {
-		$file = fopen('work/' . "$name", 'w');
+		$file = fopen('work/'."$name", 'w');
 		fwrite($file, $text);
 		fclose($file);
 	}
@@ -472,7 +472,7 @@ 
 	 * @param int $seconds
 	 */
 	public function sleepForSeconds($seconds) {
-		sleep((int)$seconds);
+		sleep((int) $seconds);
 	}
 
 	/**
@@ -480,7 +480,7 @@ 
 	 */
 	public static function addFilesToSkeleton() {
 		for ($i = 0; $i < 5; $i++) {
-			file_put_contents('../../core/skeleton/' . 'textfile' . "$i" . '.txt', "Nextcloud test text file\n");
+			file_put_contents('../../core/skeleton/'.'textfile'."$i".'.txt', "Nextcloud test text file\n");
 		}
 		if (!file_exists('../../core/skeleton/FOLDER')) {
 			mkdir('../../core/skeleton/FOLDER', 0777, true);
@@ -488,11 +488,11 @@ 
 		if (!file_exists('../../core/skeleton/PARENT')) {
 			mkdir('../../core/skeleton/PARENT', 0777, true);
 		}
-		file_put_contents('../../core/skeleton/PARENT/' . 'parent.txt', "Nextcloud test text file\n");
+		file_put_contents('../../core/skeleton/PARENT/'.'parent.txt', "Nextcloud test text file\n");
 		if (!file_exists('../../core/skeleton/PARENT/CHILD')) {
 			mkdir('../../core/skeleton/PARENT/CHILD', 0777, true);
 		}
-		file_put_contents('../../core/skeleton/PARENT/CHILD/' . 'child.txt', "Nextcloud test text file\n");
+		file_put_contents('../../core/skeleton/PARENT/CHILD/'.'child.txt', "Nextcloud test text file\n");
 	}
 
 	/**
@@ -500,7 +500,7 @@ 
 	 */
 	public static function removeFilesFromSkeleton() {
 		for ($i = 0; $i < 5; $i++) {
-			self::removeFile('../../core/skeleton/', 'textfile' . "$i" . '.txt');
+			self::removeFile('../../core/skeleton/', 'textfile'."$i".'.txt');
 		}
 		if (is_dir('../../core/skeleton/FOLDER')) {
 			rmdir('../../core/skeleton/FOLDER');

build/integration/features/bootstrap/FilesRemindersContext.php

Indentation +72 added lines, -72 removed lines

@@ -14,83 +14,83 @@
  * Files reminders context.
  */
 class FilesRemindersContext implements Context {
-	use BasicStructure;
-	use WebDav;
+    use BasicStructure;
+    use WebDav;
 
-	/**
-	 * @When the user sets a reminder for :path with due date :dueDate
-	 */
-	public function settingAReminderForFileWithDueDate($path, $dueDate) {
-		$fileId = $this->getFileIdForPath($this->currentUser, $path);
-		$this->sendRequestForJSON(
-			'PUT',
-			'/apps/files_reminders/api/v1/' . $fileId,
-			['dueDate' => $dueDate],
-			['OCS-APIREQUEST' => 'true']
-		);
-	}
+    /**
+     * @When the user sets a reminder for :path with due date :dueDate
+     */
+    public function settingAReminderForFileWithDueDate($path, $dueDate) {
+        $fileId = $this->getFileIdForPath($this->currentUser, $path);
+        $this->sendRequestForJSON(
+            'PUT',
+            '/apps/files_reminders/api/v1/' . $fileId,
+            ['dueDate' => $dueDate],
+            ['OCS-APIREQUEST' => 'true']
+        );
+    }
 
-	/**
-	 * @Then the user sees the reminder for :path is set to :dueDate
-	 */
-	public function retrievingTheReminderForFile($path, $dueDate) {
-		$fileId = $this->getFileIdForPath($this->currentUser, $path);
-		$this->sendRequestForJSON(
-			'GET',
-			'/apps/files_reminders/api/v1/' . $fileId,
-			null,
-			['OCS-APIREQUEST' => 'true']
-		);
-		$response = $this->getDueDateFromOCSResponse();
-		Assert::assertEquals($dueDate, $response);
-	}
+    /**
+     * @Then the user sees the reminder for :path is set to :dueDate
+     */
+    public function retrievingTheReminderForFile($path, $dueDate) {
+        $fileId = $this->getFileIdForPath($this->currentUser, $path);
+        $this->sendRequestForJSON(
+            'GET',
+            '/apps/files_reminders/api/v1/' . $fileId,
+            null,
+            ['OCS-APIREQUEST' => 'true']
+        );
+        $response = $this->getDueDateFromOCSResponse();
+        Assert::assertEquals($dueDate, $response);
+    }
 
-	/**
-	 * @Then the user sees the reminder for :path is not set
-	 */
-	public function retrievingTheReminderForFileIsNotSet($path) {
-		$fileId = $this->getFileIdForPath($this->currentUser, $path);
-		$this->sendRequestForJSON(
-			'GET',
-			'/apps/files_reminders/api/v1/' . $fileId,
-			null,
-			['OCS-APIREQUEST' => 'true']
-		);
-		$response = $this->getDueDateFromOCSResponse();
-		Assert::assertNull($response);
-	}
+    /**
+     * @Then the user sees the reminder for :path is not set
+     */
+    public function retrievingTheReminderForFileIsNotSet($path) {
+        $fileId = $this->getFileIdForPath($this->currentUser, $path);
+        $this->sendRequestForJSON(
+            'GET',
+            '/apps/files_reminders/api/v1/' . $fileId,
+            null,
+            ['OCS-APIREQUEST' => 'true']
+        );
+        $response = $this->getDueDateFromOCSResponse();
+        Assert::assertNull($response);
+    }
 
-	/**
-	 * @When the user removes the reminder for :path
-	 */
-	public function removingTheReminderForFile($path) {
-		$fileId = $this->getFileIdForPath($this->currentUser, $path);
-		$this->sendRequestForJSON(
-			'DELETE',
-			'/apps/files_reminders/api/v1/' . $fileId,
-			null,
-			['OCS-APIREQUEST' => 'true']
-		);
-	}
+    /**
+     * @When the user removes the reminder for :path
+     */
+    public function removingTheReminderForFile($path) {
+        $fileId = $this->getFileIdForPath($this->currentUser, $path);
+        $this->sendRequestForJSON(
+            'DELETE',
+            '/apps/files_reminders/api/v1/' . $fileId,
+            null,
+            ['OCS-APIREQUEST' => 'true']
+        );
+    }
 
-	/**
-	 * Check the due date from OCS response
-	 */
-	private function getDueDateFromOCSResponse(): ?string {
-		if ($this->response === null) {
-			throw new \RuntimeException('No response available');
-		}
+    /**
+     * Check the due date from OCS response
+     */
+    private function getDueDateFromOCSResponse(): ?string {
+        if ($this->response === null) {
+            throw new \RuntimeException('No response available');
+        }
 
-		$body = (string)$this->response->getBody();
-		if (str_starts_with($body, '<')) {
-			$body = simplexml_load_string($body);
-			if ($body === false) {
-				throw new \RuntimeException('Could not parse OCS response, body is not valid XML');
-			}
-			return $body->data->dueDate;
-		}
+        $body = (string)$this->response->getBody();
+        if (str_starts_with($body, '<')) {
+            $body = simplexml_load_string($body);
+            if ($body === false) {
+                throw new \RuntimeException('Could not parse OCS response, body is not valid XML');
+            }
+            return $body->data->dueDate;
+        }
 
-		$body = json_decode($body, true);
-		return $body['ocs']['data']['dueDate'] ?? null;
-	}
+        $body = json_decode($body, true);
+        return $body['ocs']['data']['dueDate'] ?? null;
+    }
 }

Spacing +6 added lines, -6 removed lines

@@ -7,7 +7,7 @@ 
 use Behat\Behat\Context\Context;
 use PHPUnit\Framework\Assert;
 
-require __DIR__ . '/../../vendor/autoload.php';
+require __DIR__.'/../../vendor/autoload.php';
 
 
 /**
@@ -24,7 +24,7 @@ 
 		$fileId = $this->getFileIdForPath($this->currentUser, $path);
 		$this->sendRequestForJSON(
 			'PUT',
-			'/apps/files_reminders/api/v1/' . $fileId,
+			'/apps/files_reminders/api/v1/'.$fileId,
 			['dueDate' => $dueDate],
 			['OCS-APIREQUEST' => 'true']
 		);
@@ -37,7 +37,7 @@ 
 		$fileId = $this->getFileIdForPath($this->currentUser, $path);
 		$this->sendRequestForJSON(
 			'GET',
-			'/apps/files_reminders/api/v1/' . $fileId,
+			'/apps/files_reminders/api/v1/'.$fileId,
 			null,
 			['OCS-APIREQUEST' => 'true']
 		);
@@ -52,7 +52,7 @@ 
 		$fileId = $this->getFileIdForPath($this->currentUser, $path);
 		$this->sendRequestForJSON(
 			'GET',
-			'/apps/files_reminders/api/v1/' . $fileId,
+			'/apps/files_reminders/api/v1/'.$fileId,
 			null,
 			['OCS-APIREQUEST' => 'true']
 		);
@@ -67,7 +67,7 @@ 
 		$fileId = $this->getFileIdForPath($this->currentUser, $path);
 		$this->sendRequestForJSON(
 			'DELETE',
-			'/apps/files_reminders/api/v1/' . $fileId,
+			'/apps/files_reminders/api/v1/'.$fileId,
 			null,
 			['OCS-APIREQUEST' => 'true']
 		);
@@ -81,7 +81,7 @@ 
 			throw new \RuntimeException('No response available');
 		}
 
-		$body = (string)$this->response->getBody();
+		$body = (string) $this->response->getBody();
 		if (str_starts_with($body, '<')) {
 			$body = simplexml_load_string($body);
 			if ($body === false) {