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

Umd tests
This commit is contained in:
Fabio Berger 2017-05-29 16:04:13 +02:00 committed by GitHub
commit abd0228ba8
5 changed files with 40 additions and 27 deletions

View File

@ -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"

3
src/globals.d.ts vendored
View File

@ -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 */

View File

@ -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());

View File

@ -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;
});
});

View File

@ -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',