updated with some fixes

Author: jaysomani

src/VCS/Adapter/Git/Gitea.php

@@ -13,7 +13,7 @@ class Gitea extends Git
     public const CONTENTS_FILE = 'file';
 
     public const CONTENTS_DIRECTORY = 'dir';
-    
+
     protected string $endpoint = 'http://gitea:3000/api/v1';
 
     protected string $accessToken;
@@ -152,7 +152,7 @@ public function getRepositoryName(string $repositoryId): string
 
     public function getRepositoryTree(string $owner, string $repositoryName, string $branch, bool $recursive = false): array
     {
-        $url = "/repos/{$owner}/{$repositoryName}/git/trees/{$branch}" . ($recursive ? '?recursive=1' : '');
+        $url = "/repos/{$owner}/{$repositoryName}/git/trees/" . urlencode($branch) . ($recursive ? '?recursive=1' : '');
 
         $response = $this->call(self::METHOD_GET, $url, ['Authorization' => "token $this->accessToken"]);
 
@@ -180,7 +180,7 @@ public function getRepositoryContent(string $owner, string $repositoryName, stri
     {
         $url = "/repos/{$owner}/{$repositoryName}/contents/{$path}";
         if (!empty($ref)) {
-            $url .= "?ref={$ref}";
+            $url .= "?ref=" . urlencode($ref);
         }
 
         $response = $this->call(self::METHOD_GET, $url, ['Authorization' => "token $this->accessToken"]);
@@ -212,7 +212,7 @@ public function listRepositoryContents(string $owner, string $repositoryName, st
             $url .= "/{$path}";
         }
         if (!empty($ref)) {
-            $url .= "?ref={$ref}";
+            $url .= "?ref=" . urlencode($ref);
         }
 
         $response = $this->call(self::METHOD_GET, $url, ['Authorization' => "token $this->accessToken"]);

tests/VCS/Adapter/GiteaTest.php

@@ -63,7 +63,7 @@ private function createFile(string $owner, string $repo, string $filepath, strin
     {
         $giteaUrl = System::getEnv('TESTS_GITEA_URL', 'http://gitea:3000') ?? '';
         $url = "{$giteaUrl}/api/v1/repos/{$owner}/{$repo}/contents/{$filepath}";
-        
+
         $ch = curl_init($url);
         curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);
         curl_setopt($ch, CURLOPT_POST, true);
@@ -75,9 +75,14 @@ private function createFile(string $owner, string $repo, string $filepath, strin
             'content' => base64_encode($content),
             'message' => $message
         ]));
-        
-        curl_exec($ch);
+
+        $response = curl_exec($ch);
+        $httpCode = curl_getinfo($ch, CURLINFO_HTTP_CODE);
         curl_close($ch);
+
+        if ($httpCode >= 400) {
+            throw new \Exception("Failed to create file {$filepath}: HTTP {$httpCode}");
+        }
     }
 
     public function testCreateRepository(): void
@@ -128,6 +133,42 @@ public function testGetComment(): void
         $this->markTestSkipped('Will be implemented in follow-up PR');
     }
 
+    public function testGetRepositoryTreeWithSlashInBranchName(): void
+    {
+        $repositoryName = 'test-branch-with-slash-' . \uniqid();
+        $this->vcsAdapter->createRepository(self::$owner, $repositoryName, false);
+
+        // Create a file on main branch first
+        $this->createFile(self::$owner, $repositoryName, 'README.md', '# Test');
+
+        // Create a branch with a slash in the name using curl
+        $giteaUrl = System::getEnv('TESTS_GITEA_URL', 'http://gitea:3000') ?? '';
+        $url = "{$giteaUrl}/api/v1/repos/" . self::$owner . "/{$repositoryName}/branches";
+
+        $ch = curl_init($url);
+        curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);
+        curl_setopt($ch, CURLOPT_POST, true);
+        curl_setopt($ch, CURLOPT_HTTPHEADER, [
+            'Authorization: token ' . self::$accessToken,
+            'Content-Type: application/json'
+        ]);
+        curl_setopt($ch, CURLOPT_POSTFIELDS, json_encode([
+            'new_branch_name' => 'feature/test-branch',
+            'old_branch_name' => 'main'
+        ]));
+        curl_exec($ch);
+        curl_close($ch);
+
+        // Now try to get tree from the branch with slash
+        $tree = $this->vcsAdapter->getRepositoryTree(self::$owner, $repositoryName, 'feature/test-branch');
+
+        $this->assertIsArray($tree);
+        $this->assertNotEmpty($tree); // Should have README.md
+        $this->assertContains('README.md', $tree);
+
+        $this->vcsAdapter->deleteRepository(self::$owner, $repositoryName);
+    }
+
     public function testGetRepository(): void
     {
         $repositoryName = 'test-get-repository-' . \uniqid();
@@ -280,7 +321,6 @@ public function testGetRepositoryContentFileNotFound(): void
         $this->expectException(\Utopia\VCS\Exception\FileNotFound::class);
         $this->vcsAdapter->getRepositoryContent(self::$owner, $repositoryName, 'non-existing.txt');
 
-        $this->vcsAdapter->deleteRepository(self::$owner, $repositoryName);
     }
 
     public function testListRepositoryContents(): void
@@ -446,6 +486,8 @@ public function testListRepositoryLanguages(): void
         $this->createFile(self::$owner, $repositoryName, 'script.js', 'console.log("test");');
         $this->createFile(self::$owner, $repositoryName, 'style.css', 'body { margin: 0; }');
 
+        sleep(2);
+
         $languages = $this->vcsAdapter->listRepositoryLanguages(self::$owner, $repositoryName);
 
         $this->assertIsArray($languages);