From 282219a70723351a681ac3c20156eeef14a758c9 Mon Sep 17 00:00:00 2001
From: Leonid Logvinov <logvinov.leon@gmail.com>
Date: Wed, 7 Jun 2017 13:27:48 +0200
Subject: Add assertion for empty batch

---
 src/contract_wrappers/exchange_wrapper.ts | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/contract_wrappers/exchange_wrapper.ts b/src/contract_wrappers/exchange_wrapper.ts
index 11cfd134f..1ae5b260b 100644
--- a/src/contract_wrappers/exchange_wrapper.ts
+++ b/src/contract_wrappers/exchange_wrapper.ts
@@ -142,6 +142,7 @@ export class ExchangeWrapper extends ContractWrapper {
     public async batchFillOrderAsync(signedOrders: SignedOrder[], fillTakerAmounts: BigNumber.BigNumber[],
                                      shouldCheckTransfer: boolean, takerAddress: string): Promise<void> {
         assert.isSameLength('signedOrders', signedOrders, 'fillTakerAmounts', fillTakerAmounts);
+        assert.assert(!_.isEmpty(signedOrders), 'Can not cancel an empty batch');
         assert.isBoolean('shouldCheckTransfer', shouldCheckTransfer);
         await assert.isSenderAddressAsync('takerAddress', takerAddress, this.web3Wrapper);
         // _.zip doesn't type check if values have different types :'(
-- 
cgit v1.2.3