refactor: use bulkOnly filter on fetchDatasetOrderbook

Author: PierreJeanjacquot

packages/sdk/src/lib/dataProtectorCore/getGrantedAccess.ts

@@ -1,4 +1,3 @@
-import { utils } from 'iexec';
 import { WorkflowError, handleIfProtocolError } from '../../utils/errors.js';
 import { formatGrantedAccess } from '../../utils/formatGrantedAccess.js';
 import {
@@ -51,18 +50,11 @@ export const getGrantedAccess = async ({
       isAppStrict: true,
       page: vPage,
       pageSize: vPageSize,
+      bulkOnly: vBulkOnly,
     });
-    let grantedAccess = orders?.map((order) =>
+    const grantedAccess = orders?.map((order) =>
       formatGrantedAccess(order.order, order.remaining)
     );
-
-    // Filter for bulk orders if bulkOnly is true
-    if (vBulkOnly && grantedAccess) {
-      grantedAccess = grantedAccess.filter(
-        (access) => access.volume === utils.DATASET_INFINITE_VOLUME.toString()
-      );
-    }
-
     return { count, grantedAccess };
   } catch (e) {
     handleIfProtocolError(e);

packages/sdk/tests/unit/dataProtectorCore/getGrantedAccess.test.ts

@@ -125,6 +125,7 @@ describe('getGrantedAccess', () => {
           pageSize: 10,
           isRequesterStrict: true,
           isAppStrict: true,
+          bulkOnly: false,
         });
       });
     });
@@ -242,6 +243,7 @@ describe('getGrantedAccess', () => {
         pageSize: undefined,
         isRequesterStrict: false,
         isAppStrict: true,
+        bulkOnly: false,
       });
 
       expect(grantedAccessResult.count).toBe(1);