Skip to content

Commit d870c2e

Browse files
fix: use validation error
1 parent ed47ca9 commit d870c2e

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

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

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ import {
55
SCONE_TAG,
66
} from '../../config/config.js';
77
import {
8+
ValidationError,
89
WorkflowError,
910
handleIfProtocolError,
1011
prepareBulkRequestErrorMessage,
@@ -50,10 +51,6 @@ export const prepareBulkRequest = async ({
5051
onStatusUpdate = () => {},
5152
}: IExecConsumer &
5253
PrepareBulkRequestParams): Promise<PrepareBulkRequestResponse> => {
53-
const vBulkAccesses = bulkAccesses;
54-
if (!vBulkAccesses || vBulkAccesses.length === 0) {
55-
throw new Error('bulkAccesses is required and must not be empty');
56-
}
5754
const vApp = addressOrEnsSchema().required().label('app').validateSync(app);
5855
const vMaxProtectedDataPerTask = positiveNumberSchema()
5956
.label('maxProtectedDataPerTask')
@@ -91,8 +88,12 @@ export const prepareBulkRequest = async ({
9188
);
9289
}
9390

91+
if (!bulkAccesses || bulkAccesses.length === 0) {
92+
throw new ValidationError('bulkAccesses is required and must not be empty');
93+
}
9494
// TODO validate bulkOrders?
9595
// price, volume, app, workerpool, requester, signature (including whitelists)
96+
const vBulkAccesses = bulkAccesses;
9697

9798
try {
9899
vOnStatusUpdate({

0 commit comments

Comments
 (0)