Merge pull request #103 from IQSS/100-get-dataset-optimize

Optimize GetDataset use case by using includeFiles=false
IQSS · Nov 16, 2023 · af43e45 · af43e45
2 parents aa356a3 + 03005dd
commit af43e45
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/datasets/infra/repositories/DatasetsRepository.ts b/src/datasets/infra/repositories/DatasetsRepository.ts
@@ -36,6 +36,7 @@ export class DatasetsRepository extends ApiRepository implements IDatasetsReposi
       true,
       {
         includeDeaccessioned: includeDeaccessioned,
+        includeFiles: false
       },
     )
       .then((response) => transformVersionResponseToDataset(response))

diff --git a/test/unit/datasets/DatasetsRepository.test.ts b/test/unit/datasets/DatasetsRepository.test.ts
@@ -83,11 +83,11 @@ describe('DatasetsRepository', () => {
   describe('getDataset', () => {
     const testIncludeDeaccessioned = false;
     const expectedRequestConfigApiKey = {
-      params: { includeDeaccessioned: testIncludeDeaccessioned },
+      params: { includeDeaccessioned: testIncludeDeaccessioned, includeFiles: false },
       headers: TestConstants.TEST_EXPECTED_AUTHENTICATED_REQUEST_CONFIG_API_KEY.headers,
     };
     const expectedRequestConfigSessionCookie = {
-      params: { includeDeaccessioned: testIncludeDeaccessioned },
+      params: { includeDeaccessioned: testIncludeDeaccessioned, includeFiles: false },
       withCredentials: TestConstants.TEST_EXPECTED_AUTHENTICATED_REQUEST_CONFIG_SESSION_COOKIE.withCredentials,
       headers: TestConstants.TEST_EXPECTED_AUTHENTICATED_REQUEST_CONFIG_SESSION_COOKIE.headers,
     };
-Original file line number
+Diff line change
@@ Expand Up @@
           true,
           {
             includeDeaccessioned: includeDeaccessioned,
+            includeFiles: false
           },
         )
           .then((response) => transformVersionResponseToDataset(response))
@@ Expand Down @@