show tracing link

.woodpecker.star

@@ -434,6 +434,7 @@ MINIO_MC_ENV = {
         "from_secret": "cache_s3_secret_key",
     },
     "PUBLIC_BUCKET": "public",
+    "MC_PUBLIC_HOST": "https://s3.ci.opencloud.eu",
 }
 
 CI_HTTP_PROXY_ENV = {
@@ -1758,7 +1759,7 @@ def uploadTracingResult(ctx):
             "mc cp -a %s/reports/e2e/playwright/tracing/* s3/$PUBLIC_BUCKET/web/tracing/$CI_REPO_NAME/$CI_PIPELINE_NUMBER/" % dirs["web"],
             "cd %s/reports/e2e/playwright/tracing/" % dirs["web"],
             'echo "To see the trace, please open the following link in the console"',
-            'for f in *.zip; do echo "npx playwright show-trace $MC_HOST/$PUBLIC_BUCKET/web/tracing/$CI_REPO_NAME/$CI_PIPELINE_NUMBER/$f \n"; done',
+            'for f in *.zip; do echo "npx playwright show-trace $MC_PUBLIC_HOST/$PUBLIC_BUCKET/web/tracing/$CI_REPO_NAME/$CI_PIPELINE_NUMBER/$f \n"; done',
         ],
         "when": {
             "status": status,

tests/acceptance/TestHelpers/HttpRequestHelper.php

@@ -100,7 +100,7 @@ class HttpRequestHelper {
 			$parsedUrl = parse_url($url);
 			$baseUrl = $parsedUrl['scheme'] . '://' . $parsedUrl['host'];
 			$baseUrl .= isset($parsedUrl['port']) ? ':' . $parsedUrl['port'] : '';
-			$testUrl = $baseUrl . "/graph/v1.0/me";
+			$testUrl = $baseUrl . "/graph/v1.0/user/$user";
 			if (OcHelper::isTestingOnReva()) {
 				$url = $baseUrl . "/ocs/v2.php/cloud/users/$user";
 			}

tests/acceptance/TestHelpers/TokenHelper.php

@@ -23,7 +23,7 @@ namespace TestHelpers;
 use GuzzleHttp\Client;
 use GuzzleHttp\Cookie\CookieJar;
 use GuzzleHttp\Exception\GuzzleException;
-use PHPUnit\Framework\Assert;
+use Exception;
 
 /**
  * Helper for obtaining bearer tokens for users
@@ -143,15 +143,21 @@ class TokenHelper {
 			]
 		);
 
-		Assert::assertEquals(
-			200,
-			$response->getStatusCode(),
-			'Token refresh failed: Expected status code 200 but received ' . $response->getStatusCode()
-		);
+		if ($response->getStatusCode() !== 200) {
+			throw new Exception(
+				\sprintf(
+					'Token refresh failed: Expected status code 200 but received %d. Message: %s',
+					$response->getStatusCode(),
+					$response->getReasonPhrase()
+				)
+			);
+		}
+
 		$data = json_decode($response->getBody()->getContents(), true);
 
-		Assert::assertArrayHasKey('access_token', $data, 'Missing access_token in refresh response');
-		Assert::assertArrayHasKey('refresh_token', $data, 'Missing refresh_token in refresh response');
+		if (!isset($data['access_token']) || !isset($data['refresh_token'])) {
+			throw new Exception('Missing tokens in refresh response');
+		}
 
 		return [
 			'access_token' => $data['access_token'],
@@ -247,24 +253,21 @@ class TokenHelper {
 	): string {
 		$response = self::makeLoginRequest($username, $password, $baseUrl, $cookieJar);
 
-		Assert::assertEquals(
-			200,
-			$response->getStatusCode(),
-			'Logon failed: Expected status code 200 but received: ' . $response->getStatusCode()
-		);
+		if ($response->getStatusCode() !== 200) {
+			throw new Exception(
+				\sprintf(
+					'Logon failed: Expected status code 200 but received %d. Message: %s',
+					$response->getStatusCode(),
+					$response->getReasonPhrase()
+				)
+			);
+		}
+
 		$data = json_decode($response->getBody()->getContents(), true);
 
-		Assert::assertArrayHasKey(
-			'hello',
-			$data,
-			'Logon response does not contain "hello" object'
-		);
-
-		Assert::assertArrayHasKey(
-			'continue_uri',
-			$data['hello'],
-			'Missing continue_uri in logon response'
-		);
+		if (!isset($data['hello']['continue_uri'])) {
+			throw new Exception('Missing continue_uri in logon response');
+		}
 
 		return $data['hello']['continue_uri'];
 	}
@@ -306,17 +309,42 @@ class TokenHelper {
 			]
 		);
 
-		Assert::assertEquals(
-			302,
-			$response->getStatusCode(),
-			'Authorization request failed: Expected status code 302 but received: ' . $response->getStatusCode()
-		);
+		if ($response->getStatusCode() !== 302) {
+			// Add debugging to understand what is happening
+			$body = $response->getBody()->getContents();
+			throw new Exception(
+				\sprintf(
+					'Authorization failed: Expected status code 302 but received %d. Message: %s. Body: %s',
+					$response->getStatusCode(),
+					$response->getReasonPhrase(),
+					$body
+				)
+			);
+		}
 
 		$location = $response->getHeader('Location')[0] ?? '';
-		Assert::assertNotEmpty($location, 'Missing Location header in authorization response');
+
+		if (empty($location)) {
+			throw new Exception('Missing Location header in authorization response');
+		}
 
 		parse_str(parse_url($location, PHP_URL_QUERY), $queryParams);
-		Assert::assertArrayHasKey('code', $queryParams, 'Missing code parameter in redirect URL');
+
+		// Check for errors
+		if (isset($queryParams['error'])) {
+			throw new Exception(
+				\sprintf(
+					'Authorization error: %s - %s',
+					$queryParams['error'],
+					urldecode($queryParams['error_description'] ?? 'No description')
+				)
+			);
+		}
+
+		if (!isset($queryParams['code'])) {
+			throw new Exception('Missing auth code in redirect URL. Location: ' . $location);
+		}
+
 		return $queryParams['code'];
 	}
 
@@ -355,15 +383,21 @@ class TokenHelper {
 			]
 		);
 
-		Assert::assertEquals(
-			200,
-			$response->getStatusCode(),
-			'Token request failed: Expected status code 200 but received: ' . $response->getStatusCode()
-		);
+		if ($response->getStatusCode() !== 200) {
+			throw new Exception(
+				\sprintf(
+					'Token request failed: Expected status code 200 but received %d. Message: %s',
+					$response->getStatusCode(),
+					$response->getReasonPhrase()
+				)
+			);
+		}
 
 		$data = json_decode($response->getBody()->getContents(), true);
-		Assert::assertArrayHasKey('access_token', $data, 'Missing access_token in token response');
-		Assert::assertArrayHasKey('refresh_token', $data, 'Missing refresh_token in token response');
+
+		if (!isset($data['access_token']) || !isset($data['refresh_token'])) {
+			throw new Exception('Missing tokens in response');
+		}
 
 		return [
 			'access_token' => $data['access_token'],