✅Fixed storage e2e test for S3

linagora · Nov 5, 2024 · 2c5cf0b · 2c5cf0b
1 parent 20ef64f
commit 2c5cf0b
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 4 deletions.
diff --git a/tdrive/backend/node/src/core/platform/services/storage/connectors/S3/s3-service.ts b/tdrive/backend/node/src/core/platform/services/storage/connectors/S3/s3-service.ts
@@ -113,7 +113,7 @@ export default class S3ConnectorService implements StorageConnectorAPI {
           break;
         }
       } catch (e) {
-        logger.error("Error getting information from S3", e);
+        logger.error(e, `Error getting information from S3 for path: ${path}`);
       }
 
       if (i === tries) {

diff --git a/tdrive/backend/node/src/core/platform/services/storage/storage-service.ts b/tdrive/backend/node/src/core/platform/services/storage/storage-service.ts
@@ -69,7 +69,8 @@ export default class StorageService extends TdriveService<StorageAPI> implements
   }
 
   exists(path: string, options?: ReadOptions): Promise<boolean> {
-    return this.getConnector().exists(path, options);
+    //TODO[ASH] check for all the file chunks
+    return this.getConnector().exists(path + "/chunk1", options);
   }
 
   async write(path: string, stream: Stream, options?: WriteOptions): Promise<WriteMetadata> {

diff --git a/tdrive/backend/node/src/services/files/services/index.ts b/tdrive/backend/node/src/services/files/services/index.ts
@@ -431,7 +431,7 @@ export class FileServiceImpl {
         id: externalId,
         company_id: "00000000-0000-4000-0000-000000000000",
       });
-      const exist = await gr.platformServices.storage.exists(getFilePath(file) + "/chunk1");
+      const exist = await gr.platformServices.storage.exists(getFilePath(file));
       if (exist) {
         return { exist: true, file };
       } else {

diff --git a/tdrive/docker-compose.tests.yml b/tdrive/docker-compose.tests.yml
@@ -1,4 +1,4 @@
-version: "2"
+version: "3"
 
 services:
   # rabbitmq: