Merge pull request #16 from 0xProject/umd-tests

Umd tests

5 files changed, 40 insertions, 27 deletions

diff --git a/package.json b/package.json
index 5c352bc88..3b29795a8 100644
--- a/package.json
+++ b/package.json
@@ -10,18 +10,27 @@
     "exchange"
   ],
   "scripts": {
-    "clean": "shx rm -rf _bundles lib",
-    "build:bundle": "webpack",
-    "build:commonjs": "tsc; copyfiles -u 2 ./src/artifacts/*.json ../0x.js/lib/src/artifacts;",
-    "build": "npm run clean && run-p build:*",
+    "build": "npm run clean && run-p build:*:prod",
     "lint": "tslint src/**/*.ts",
-    "test": "run-s clean build:commonjs && mocha lib/test/**/*_test.js",
+    "test": "run-s test:commonjs test:umd",
     "test:coverage": "nyc npm run test --all",
+    "update_contracts": "for i in ${npm_package_config_artifacts}; do copyfiles -u 4 ../contracts/build/contracts/$i.json ../0x.js/src/artifacts; done;",
+    "testrpc": "testrpc -p 8545 --networkId 50",
     "docs:json": "typedoc --json docs/index.json .",
     "docs:generate": "typedoc --out docs .",
     "docs:open": "opn docs/index.html",
-    "update_contracts": "for i in ${npm_package_config_artifacts}; do copyfiles -u 4 ../contracts/build/contracts/$i.json ../0x.js/src/artifacts; done;",
-    "testrpc": "testrpc -p 8545 --networkId 50"
+
+    "clean": "shx rm -rf _bundles lib test_temp",
+    "build:dev": "npm run clean && run-p build:*:dev",
+    "build:umd:dev": "webpack",
+    "build:umd:prod": "webpack -p",
+    "build:commonjs:dev": "tsc; copyfiles -u 2 ./src/artifacts/*.json ../0x.js/lib/src/artifacts;",
+    "run_mocha": "mocha test_temp/test/**/*_test.js",
+    "test:commonjs": "run-s build:commonjs:dev setup_commonjs run_mocha tear_down",
+    "test:umd": "run-s build:*:dev setup_umd run_mocha tear_down",
+    "setup_commonjs": "shx cp -r lib test_temp",
+    "setup_umd": "shx mkdir -p test_temp/src && shx cp _bundles/* test_temp/src && shx cp -r lib/test test_temp/test",
+    "tear_down": "shx rm -rf test_temp"
   },
   "config": {
     "artifacts": "Proxy Exchange TokenRegistry Token Mintable EtherToken"
diff --git a/src/globals.d.ts b/src/globals.d.ts
index dee957f2f..0062a05cb 100644
--- a/src/globals.d.ts
+++ b/src/globals.d.ts
@@ -15,7 +15,8 @@ declare interface Schema {
 declare namespace Chai {
     interface Assertion {
         bignumber: Assertion;
-        eventually: Assertion;
+        // HACK: In order to comply with chai-as-promised we make eventually a `PromisedAssertion` not an `Assertion`
+        eventually: PromisedAssertion;
     }
 }
 /* tslint:enable */
diff --git a/test/0x.js_test.ts b/test/0x.js_test.ts
index b8eece5d4..289c823af 100644
--- a/test/0x.js_test.ts
+++ b/test/0x.js_test.ts
@@ -4,7 +4,7 @@ import 'mocha';
 import * as BigNumber from 'bignumber.js';
 import ChaiBigNumber = require('chai-bignumber');
 import {ZeroEx} from '../src/0x.js';
-import {constants} from '../src/utils/constants';
+import {constants} from './utils/constants';
 
 // Use BigNumber chai add-on
 chai.use(ChaiBigNumber());
diff --git a/test/contract_wrapper_test.ts b/test/contract_wrapper_test.ts
index 4ff56ea3e..4bfc49374 100644
--- a/test/contract_wrapper_test.ts
+++ b/test/contract_wrapper_test.ts
@@ -3,21 +3,18 @@ import * as chai from 'chai';
 import chaiAsPromised = require('chai-as-promised');
 import * as Web3 from 'web3';
 import {web3Factory} from './utils/web3_factory';
-import {ExchangeWrapper} from '../src/contract_wrappers/exchange_wrapper';
+import {ZeroEx} from '../src/0x.js';
 import {BlockchainLifecycle} from './utils/blockchain_lifecycle';
-import {Web3Wrapper} from './../src/web3_wrapper';
 
 const expect = chai.expect;
 chai.use(chaiAsPromised);
 const blockchainLifecycle = new BlockchainLifecycle();
 
 describe('ExchangeWrapper', () => {
-    let web3Wrapper: Web3Wrapper;
-    let exchangeWrapper: ExchangeWrapper;
+    let zeroEx: ZeroEx;
     before(async () => {
         const web3 = web3Factory.create();
-        web3Wrapper = new Web3Wrapper(web3);
-        exchangeWrapper = new ExchangeWrapper(web3Wrapper);
+        zeroEx = new ZeroEx(web3);
     });
     beforeEach(async () => {
         await blockchainLifecycle.startAsync();
@@ -42,7 +39,7 @@ describe('ExchangeWrapper', () => {
                     r: signature.r,
                     s: signature.s,
                 };
-                expect(exchangeWrapper.isValidSignatureAsync(dataHex, malformedSignature, address))
+                expect(zeroEx.exchange.isValidSignatureAsync(dataHex, malformedSignature, address))
                     .to.be.rejected;
             });
             it('r lacks 0x prefix', () => {
@@ -52,7 +49,7 @@ describe('ExchangeWrapper', () => {
                     r: malformedR,
                     s: signature.s,
                 };
-                expect(exchangeWrapper.isValidSignatureAsync(dataHex, malformedSignature, address))
+                expect(zeroEx.exchange.isValidSignatureAsync(dataHex, malformedSignature, address))
                     .to.be.rejected;
             });
             it('r is too short', () => {
@@ -62,7 +59,7 @@ describe('ExchangeWrapper', () => {
                     r: malformedR,
                     s: signature.s.replace('0', 'z'),
                 };
-                expect(exchangeWrapper.isValidSignatureAsync(dataHex, malformedSignature, address))
+                expect(zeroEx.exchange.isValidSignatureAsync(dataHex, malformedSignature, address))
                     .to.be.rejected;
             });
             it('s is not hex', () => {
@@ -72,26 +69,26 @@ describe('ExchangeWrapper', () => {
                     r: signature.r,
                     s: malformedS,
                 };
-                expect(exchangeWrapper.isValidSignatureAsync(dataHex, malformedSignature, address))
+                expect(zeroEx.exchange.isValidSignatureAsync(dataHex, malformedSignature, address))
                     .to.be.rejected;
             });
         });
         it('should return false if the data doesn\'t pertain to the signature & address', async () => {
-            const isValid = await exchangeWrapper.isValidSignatureAsync('0x0', signature, address);
+            const isValid = await zeroEx.exchange.isValidSignatureAsync('0x0', signature, address);
             expect(isValid).to.be.false;
         });
         it('should return false if the address doesn\'t pertain to the signature & dataHex', async () => {
             const validUnrelatedAddress = '0x8b0292B11a196601eD2ce54B665CaFEca0347D42';
-            const isValid = await exchangeWrapper.isValidSignatureAsync(dataHex, signature, validUnrelatedAddress);
+            const isValid = await zeroEx.exchange.isValidSignatureAsync(dataHex, signature, validUnrelatedAddress);
             expect(isValid).to.be.false;
         });
         it('should return false if the signature doesn\'t pertain to the dataHex & address', async () => {
             const wrongSignature = Object.assign({}, signature, {v: 28});
-            const isValid = await exchangeWrapper.isValidSignatureAsync(dataHex, wrongSignature, address);
+            const isValid = await zeroEx.exchange.isValidSignatureAsync(dataHex, wrongSignature, address);
             expect(isValid).to.be.false;
         });
         it('should return true if the signature does pertain to the dataHex & address', async () => {
-            const isValid = await exchangeWrapper.isValidSignatureAsync(dataHex, signature, address);
+            const isValid = await zeroEx.exchange.isValidSignatureAsync(dataHex, signature, address);
             expect(isValid).to.be.true;
         });
     });
diff --git a/webpack.config.js b/webpack.config.js
index 67af51f7a..03a7c6cac 100644
--- a/webpack.config.js
+++ b/webpack.config.js
@@ -1,14 +1,20 @@
 /**
  * This is to generate the umd bundle only
  */
+const lodash = require('lodash');
 const webpack = require('webpack');
 const path = require('path');
+const production = process.env.NODE_ENV === 'production';
+
+let entry = {
+    '0x': './src/0x.js.ts',
+};
+if (production) {
+    entry = _.assign({}, entry, {'0x.min': './src/0x.js.ts'});
+}
 
 module.exports = {
-    entry: {
-        '0x': './src/ts/0x.js.ts',
-        '0x.min': './src/ts/0x.js.ts'
-    },
+    entry,
     output: {
         path: path.resolve(__dirname, '_bundles'),
         filename: '[name].js',