Publish

- @0x/contracts-asset-proxy@2.2.6
 - @0x/contracts-coordinator@2.0.11
 - @0x/contracts-dev-utils@0.0.8
 - @0x/contracts-erc1155@1.1.13
 - @0x/contracts-erc20@2.2.12
 - @0x/contracts-erc721@2.1.13
 - @0x/contracts-exchange-forwarder@3.0.10
 - @0x/contracts-exchange-libs@3.0.6
 - @0x/contracts-exchange@2.1.12
 - @0x/contracts-extensions@4.0.6
 - @0x/contracts-multisig@3.1.12
 - @0x/contracts-test-utils@3.1.14
 - @0x/contracts-utils@3.2.2
 - 0x.js@7.0.0
 - @0x/abi-gen-templates@2.4.2
 - @0x/abi-gen-wrappers@5.3.0
 - @0x/abi-gen@4.1.1
 - @0x/assert@2.1.4
 - @0x/asset-buyer@6.1.12
 - @0x/asset-swapper@1.0.2
 - @0x/base-contract@5.3.2
 - @0x/connect@5.0.17
 - @0x/contract-addresses@3.1.0
 - @0x/contract-artifacts@2.2.0
 - @0x/contract-wrappers@11.1.0
 - @0x/dev-utils@2.3.1
 - @0x/fill-scenarios@3.0.17
 - @0x/instant@1.0.29
 - @0x/json-schemas@4.0.0
 - @0x/migrations@4.3.0
 - @0x/monorepo-scripts@1.0.35
 - @0x/order-utils@8.3.0
 - @0x/python-contract-wrappers@1.0.4
 - @0x/sol-compiler@3.1.13
 - @0x/sol-coverage@3.0.10
 - @0x/sol-doc@2.0.17
 - @0x/sol-profiler@3.1.12
 - @0x/sol-trace@2.0.18
 - @0x/sol-tracing-utils@6.0.17
 - @0x/sra-spec@2.0.15
 - @0x/subproviders@5.0.2
 - @0x/testnet-faucets@1.0.85
 - @0x/web3-wrapper@6.0.11
 - @0x/website@0.0.88

.circleci/config.yml

@@ -23,21 +23,37 @@ jobs:
             #       command: npm set prefix=/home/circleci/npm && echo 'export PATH=$HOME/circleci/npm/bin:$PATH' >> /home/circleci/.bashrc
             - run:
                   name: install-yarn
-                  command: npm install --global yarn@1.17.0
+                  command: npm install --global yarn@1.9.4
             - run:
                   name: yarn
                   command: yarn --frozen-lockfile --ignore-engines install || yarn --frozen-lockfile --ignore-engines install
             - setup_remote_docker
-            - run: yarn build:ci
+            - run: yarn build:ci:no_website
             - run: yarn build:ts
             - save_cache:
                   key: repo-{{ .Environment.CIRCLE_SHA1 }}
                   paths:
                       - ~/repo
+            - save_cache:
+                  key: python-contract-wrappers-{{ .Environment.CIRCLE_SHA1 }}
+                  paths:
+                      - ~/repo/packages/python-contract-wrappers/generated
+            - store_artifacts:
+                  path: ~/repo/packages/python-contract-wrappers/generated
             - store_artifacts:
                   path: ~/repo/packages/abi-gen/test-cli/output
             - store_artifacts:
                   path: ~/repo/packages/abi-gen-wrappers/generated_docs
+    build-website:
+        resource_class: medium+
+        docker:
+            - image: nikolaik/python-nodejs:python3.7-nodejs8
+        working_directory: ~/repo
+        steps:
+            - restore_cache:
+                  keys:
+                      - repo-{{ .Environment.CIRCLE_SHA1 }}
+            - run: cd packages/website && yarn build:prod
     test-contracts-ganache:
         resource_class: medium+
         docker:
@@ -47,31 +63,19 @@ jobs:
             - restore_cache:
                   keys:
                       - repo-{{ .Environment.CIRCLE_SHA1 }}
-            - run: yarn wsrun test:circleci @0x/contracts-multisig @0x/contracts-utils @0x/contracts-exchange-libs @0x/contracts-erc20 @0x/contracts-erc721 @0x/contracts-erc1155 @0x/contracts-extensions @0x/contracts-asset-proxy @0x/contracts-exchange @0x/contracts-exchange-forwarder @0x/contracts-coordinator @0x/contracts-dev-utils @0x/contracts-staking
-    test-exchange-ganache-3.0:
-        resource_class: medium+
+            - run: yarn wsrun test:circleci @0x/contracts-multisig @0x/contracts-utils @0x/contracts-exchange-libs @0x/contracts-erc20 @0x/contracts-erc721 @0x/contracts-erc1155 @0x/contracts-extensions @0x/contracts-asset-proxy @0x/contracts-exchange @0x/contracts-exchange-forwarder @0x/contracts-coordinator @0x/contracts-dev-utils
+    test-contracts-geth:
         docker:
             - image: nikolaik/python-nodejs:python3.7-nodejs8
+            - image: 0xorg/devnet
         working_directory: ~/repo
         steps:
             - restore_cache:
                   keys:
                       - repo-{{ .Environment.CIRCLE_SHA1 }}
-            - run: yarn wsrun test:circleci @0x/contracts-exchange
-    test-contracts-rest-ganache-3.0:
-        resource_class: medium+
-        docker:
-            - image: nikolaik/python-nodejs:python3.7-nodejs8
-        working_directory: ~/repo
-        steps:
-            - restore_cache:
-                  keys:
-                      - repo-{{ .Environment.CIRCLE_SHA1 }}
-            - run: yarn wsrun test:circleci @0x/contracts-multisig @0x/contracts-utils @0x/contracts-exchange-libs @0x/contracts-erc20 @0x/contracts-erc721 @0x/contracts-erc1155 @0x/contracts-asset-proxy @0x/contracts-exchange-forwarder @0x/contracts-dev-utils @0x/contracts-staking
-            # TODO(dorothy-zbornak): Re-enable after updating this package for 3.0.
-            # - run: yarn wsrun test:circleci @0x/contracts-extensions
-            # TODO(abandeali): Re-enable after this package is complete.
-            # - run: yarn wsrun test:circleci @0x/contracts-coordinator
+            # HACK(albrow): we need to sleep 10 seconds to ensure the devnet is
+            # initialized
+            - run: sleep 10 && TEST_PROVIDER=geth yarn wsrun test:circleci @0x/contracts-multisig @0x/contracts-utils @0x/contracts-exchange-libs @0x/contracts-erc20 @0x/contracts-erc721 @0x/contracts-erc1155 @0x/contracts-extensions @0x/contracts-asset-proxy @0x/contracts-exchange @0x/contracts-exchange-forwarder @0x/contracts-coordinator @0x/contracts-dev-utils
     test-publish:
         resource_class: medium+
         docker:
@@ -82,9 +86,7 @@ jobs:
             - restore_cache:
                   keys:
                       - repo-{{ .Environment.CIRCLE_SHA1 }}
-            - run: 
-                command: yarn test:publish:circleci
-                no_output_timeout: 1800
+            - run: yarn test:publish:circleci
     test-doc-generation:
         docker:
             - image: nikolaik/python-nodejs:python3.7-nodejs8
@@ -93,9 +95,7 @@ jobs:
             - restore_cache:
                   keys:
                       - repo-{{ .Environment.CIRCLE_SHA1 }}
-            - run:
-                command: yarn test:generate_docs:circleci
-                no_output_timeout: 1200
+            - run: yarn test:generate_docs:circleci
     test-rest:
         docker:
             - image: nikolaik/python-nodejs:python3.7-nodejs8
@@ -106,13 +106,11 @@ jobs:
                       - repo-{{ .Environment.CIRCLE_SHA1 }}
             - run: yarn wsrun test:circleci @0x/contracts-test-utils
             - run: yarn wsrun test:circleci @0x/abi-gen
-            # TODO (xianny): Needs to be updated for 3.0
-            # - run: yarn wsrun test:circleci @0x/asset-buyer
+            - run: yarn wsrun test:circleci @0x/asset-buyer
             - run: yarn wsrun test:circleci @0x/contract-artifacts
             - run: yarn wsrun test:circleci @0x/assert
             - run: yarn wsrun test:circleci @0x/base-contract
-            # TODO (xianny): Needs to be updated for 3.0
-            # - run: yarn wsrun test:circleci @0x/connect
+            - run: yarn wsrun test:circleci @0x/connect
             - run: yarn wsrun test:circleci @0x/contract-wrappers
             - run: yarn wsrun test:circleci @0x/dev-utils
             - run: yarn wsrun test:circleci @0x/json-schemas
@@ -197,8 +195,7 @@ jobs:
             - restore_cache:
                   key: installed-py-{{ .Branch }}-{{ .Environment.CIRCLE_SHA1 }}
             - restore_cache:
-                  keys:
-                      - repo-{{ .Environment.CIRCLE_SHA1 }}
+                  key: python-contract-wrappers-{{ .Environment.CIRCLE_SHA1 }}
             - run:
                   command: |
                       cd python-packages
@@ -240,7 +237,9 @@ jobs:
                   paths:
                       - ~/repo/python-packages/sra_client/.coverage
             - store_artifacts:
-                  path: ~/repo/python-packages/contract_wrappers/src/zero_ex/contract_wrappers/*/__init__.py
+                  path: ~/repo/python-packages/contract_wrappers/src/zero_ex/contract_wrappers/erc20_token/__init__.py
+            - store_artifacts:
+                  path: ~/repo/python-packages/contract_wrappers/src/zero_ex/contract_wrappers/exchange/__init__.py
             - store_artifacts:
                   path: ~/repo/python-packages/contract_addresses/build
             - store_artifacts:
@@ -290,8 +289,7 @@ jobs:
             - restore_cache:
                   key: installed-py-{{ .Branch }}-{{ .Environment.CIRCLE_SHA1 }}
             - restore_cache:
-                  keys:
-                      - repo-{{ .Environment.CIRCLE_SHA1 }}
+                  key: python-contract-wrappers-{{ .Environment.CIRCLE_SHA1 }}
             - run:
                   command: |
                       python -m ensurepip
@@ -300,7 +298,6 @@ jobs:
                       ./install
                       ./lint
     static-tests:
-        resource_class: large
         working_directory: ~/repo
         docker:
             - image: nikolaik/python-nodejs:python3.7-nodejs8
@@ -311,7 +308,6 @@ jobs:
             - run: yarn lerna run lint
             - run: yarn prettier:ci
             - run: yarn deps_versions:ci
-            - run: yarn diff_md_docs:ci
             - run: cd packages/0x.js && yarn build:umd:prod
             - run: yarn bundlewatch
     submit-coverage:
@@ -391,12 +387,17 @@ workflows:
     main:
         jobs:
             - build
-            - test-exchange-ganache-3.0:
+            - build-website:
                   requires:
                       - build
-            - test-contracts-rest-ganache-3.0:
+            - test-contracts-ganache:
                   requires:
                       - build
+            # TODO(albrow): Tests always fail on Geth right now because our fork
+            # is outdated. Uncomment once we have updated our Geth fork.
+            # - test-contracts-geth:
+            #       requires:
+            #           - build
             - test-rest:
                   requires:
                       - build
@@ -411,16 +412,13 @@ workflows:
                       - build
             - submit-coverage:
                   requires:
-                      - test-contracts-rest-ganache-3.0
-                      - test-exchange-ganache-3.0
                       - test-rest
-                      - static-tests
-            # - test-python:
-            #       requires:
-            #           - build
-            #           - test-rest
-            # - static-tests-python:
-            #       requires:
-            #           - test-python
+                      - test-python
+            - static-tests-python:
+                  requires:
+                      - test-python
+            - test-python:
+                  requires:
+                      - build
             # skip python tox run for now, as we don't yet have multiple test environments to support.
-            # - test-rest-python
+            #- test-rest-python

.github/autolabeler.yml

@@ -13,6 +13,7 @@ contracts: ['contracts']
 @0x/instant: ['packages/instant']
 @0x/abi-gen-templates: ['packages/abi-gen-templates']
 @0x/abi-gen: ['packages/abi-gen']
+@0x/website: ['packages/website']
 @0x/sol-coverage: ['packages/sol-coverage']
 @0x/sol-profiler: ['packages/sol-profiler']
 @0x/sol-trace: ['packages/sol-trace']
@@ -32,5 +33,7 @@ contracts: ['contracts']
 @0x/json-schemas: ['packages/json-schemas']
 @0x/ethereum-types: ['ethereum-types']
 @0x/connect: ['packages/connect']
+@0x/fill-scenarios: ['packages/fill-scenarios']
+@0x/dev-tools-pages: ['packages/dev-tools-pages']
 @0x/testnet-faucets: ['packages/testnet-faucets']
 @0x/monorepo-scripts: ['packages/monorepo-scripts']

.gitignore

@@ -40,12 +40,9 @@ build/Release
 node_modules/
 jspm_packages/
 
-# TypeScript v1 declaration files
+# Typescript v1 declaration files
 typings/
 
-# NVM config
-.nvmrc
-
 # Optional npm cache directory
 .npm
 .npmrc
@@ -78,11 +75,13 @@ TODO.md
 # VSCode file
 .vscode
 
+packages/website/public/bundle*
+packages/dev-tools-pages/public/bundle*
+
 # server cli
 packages/testnet-faucets/server/
 
 # generated contract artifacts/
-contracts/staking/generated-artifacts/
 contracts/coordinator/generated-artifacts/
 contracts/exchange/generated-artifacts/
 contracts/asset-proxy/generated-artifacts/
@@ -98,24 +97,9 @@ contracts/dev-utils/generated-artifacts/
 packages/sol-tracing-utils/test/fixtures/artifacts/
 python-packages/contract_artifacts/src/zero_ex/contract_artifacts/artifacts/
 
-# generated truffle contract artifacts/
-contracts/staking/build/
-contracts/coordinator/build/
-contracts/exchange/build/
-contracts/asset-proxy/build/
-contracts/multisig/build/
-contracts/utils/build/
-contracts/exchange-libs/build/
-contracts/erc20/build/
-contracts/erc721/build/
-contracts/erc1155/build/
-contracts/extensions/build/
-contracts/exchange-forwarder/build/
-contracts/dev-utils/build/
-
 # generated contract wrappers
+packages/abi-gen-wrappers/src/generated-wrappers/
 packages/python-contract-wrappers/generated/
-contracts/staking/generated-wrappers/
 contracts/coordinator/generated-wrappers/
 contracts/exchange/generated-wrappers/
 contracts/asset-proxy/generated-wrappers/
@@ -152,6 +136,9 @@ python-packages/contract_wrappers/src/zero_ex/contract_wrappers/zrx_token/__init
 # solc-bin in sol-compiler
 packages/sol-compiler/solc_bin/
 
+# Monorepo scripts
+packages/*/scripts/
+
 # python stuff
 .eggs
 .mypy_cache
@@ -164,8 +151,3 @@ python-packages/*/.coverage
 
 # python keeps package-local copies of json schemas
 python-packages/json_schemas/src/zero_ex/json_schemas/schemas
-
-# Doc README copy
-packages/*/docs/README.md
-
-.DS_Store

.prettierignore

@@ -1,7 +1,5 @@
 lib
 .nyc_output
-/contracts/staking/generated-wrappers
-/contracts/staking/generated-artifacts
 /contracts/coordinator/generated-wrappers
 /contracts/coordinator/generated-artifacts
 /contracts/exchange/generated-wrappers
@@ -26,23 +24,11 @@ lib
 /contracts/exchange-forwarder/generated-artifacts
 /contracts/dev-utils/generated-wrappers
 /contracts/dev-utils/generated-artifacts
-/contracts/staking/build/
-/contracts/coordinator/build/
-/contracts/exchange/build/
-/contracts/asset-proxy/build/
-/contracts/multisig/build/
-/contracts/utils/build/
-/contracts/exchange-libs/build/
-/contracts/erc20/build/
-/contracts/erc721/build/
-/contracts/erc1155/build/
-/contracts/extensions/build/
-/contracts/exchange-forwarder/build/
-/contracts/dev-utils/build/
 /packages/abi-gen/test-cli/output
 /packages/json-schemas/schemas
 /python-packages/json_schemas/src/zero_ex/json_schemas/schemas
 /packages/sra-spec/public/
+/packages/dev-tools-pages/ts/**/data.json
 package.json
 scripts/postpublish_utils.js
 packages/sol-coverage/test/fixtures/artifacts
@@ -52,4 +38,3 @@ packages/sol-coverage/test/fixtures/artifacts
 packages/abi-gen/test-cli/fixtures/artifacts/AbiGenDummy.json
 packages/abi-gen/test-cli/fixtures/artifacts/LibDummy.json
 packages/abi-gen/test-cli/fixtures/artifacts/TestLibDummy.json
-packages/*/docs

CODEOWNERS

@@ -7,6 +7,7 @@
 # Website
 packages/asset-buyer/  @BMillman19 @fragosti @steveklebanoff
 packages/instant/  @BMillman19 @fragosti @steveklebanoff
+packages/website/  @BMillman19 @fragosti @fabioberger @steveklebanoff
 
 # Dev tools & setup
 .circleci/ @LogvinovLeon

CONTRIBUTING.md

@@ -29,9 +29,9 @@ ALL PRs should be opened against `development`.
 
 Branch names should be prefixed with `fix`, `feature` or `refactor`.
 
--   e.g `fix/missing-import`
+-   e.g `fix/broken-wiki-link`
 -   If the PR only edits a single package, add it's name too
-    -   e.g `fix/subproviders/missing-import`
+    -   e.g `fix/website/broken-wiki-link`
 
 ### CHANGELOGs
 
@@ -55,7 +55,7 @@ If an entry without a `timestamp` already exists, this means other changes have
 
 ### Development Tooling
 
-We strongly recommend you use the [VSCode](https://code.visualstudio.com/) text editor since most of our code is written in TypeScript and it offers amazing support for the language.
+We strongly recommend you use the [VSCode](https://code.visualstudio.com/) text editor since most of our code is written in Typescript and it offers amazing support for the language.
 
 #### Linter
 
@@ -89,7 +89,7 @@ A few of our coding conventions are not yet enforced by the linter/auto-formatte
 1.  Do not import from a project's `index.ts` (e.g import { Token } from '../src';). Always import from the source file itself.
 1.  Generic error variables should be named `err` instead of `e` or `error`.
 1.  If you _must_ cast a variable to any - try to type it back as fast as possible. (e.g., `const cw = ((zeroEx as any)._contractWrappers as ContractWrappers);`). This ensures subsequent code is type-safe.
-1.  Our enum conventions coincide with the recommended TypeScript conventions, using capitalized keys, and all-caps snake-case values. Eg `GetStats = 'GET_STATS'`
+1.  Our enum conventions coincide with the recommended Typescript conventions, using capitalized keys, and all-caps snake-case values. Eg `GetStats = 'GET_STATS'`
 1.  All public, exported methods/functions/classes must have associated Javadoc-style comments.
 
 ### Fix `submit-coverage` CI failure

README.md

@@ -6,16 +6,21 @@
 
 This repository is a monorepo including the 0x protocol smart contracts and numerous developer tools. Each public sub-package is independently published to NPM.
 
-[website-url]: https://0x.org
+If you're developing on 0x now or are interested in using 0x infrastructure in the future, please join our [developer mailing list][dev-mailing-list-url] for updates.
+
+[website-url]: https://0xproject.com
+[whitepaper-url]: https://0xproject.com/pdfs/0x_white_paper.pdf
+[dev-mailing-list-url]: http://eepurl.com/dx4cPf
 
 [![CircleCI](https://circleci.com/gh/0xProject/0x-monorepo.svg?style=svg&circle-token=61bf7cd8c9b4e11b132089dfcffdd1be277d1e0c)](https://circleci.com/gh/0xProject/0x-monorepo)
 [![Coverage Status](https://coveralls.io/repos/github/0xProject/0x-monorepo/badge.svg?branch=development)](https://coveralls.io/github/0xProject/0x-monorepo?branch=development)
-[![Discord](https://img.shields.io/badge/chat-discord.chat-yellow.svg?style=flat)](https://discordapp.com/invite/d3FTX3M)
+[![Discord](https://img.shields.io/badge/chat-rocket.chat-yellow.svg?style=flat)](https://chat.0xproject.com)
+[![Join the chat at https://gitter.im/0xProject/Lobby](https://badges.gitter.im/0xProject/Lobby.svg)](https://gitter.im/0xProject/Lobby?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge)
 [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0)
 
 ## Packages
 
-Visit our [developer portal](https://0x.org/docs/tools/order-utils) for a comprehensive list of core & community maintained packages. All packages maintained with this monorepo are listed below.
+Visit our [developer portal](https://0xproject.com/docs/order-utils) for a comprehensive list of core & community maintained packages. All packages maintained with this monorepo are listed below.
 
 ### Python Packages
 
@@ -43,13 +48,12 @@ These packages are all under development. See [/contracts/README.md](/contracts/
 | [`@0x/contracts-exchange-libs`](/contracts/exchange-libs)           | [![npm](https://img.shields.io/npm/v/@0x/contracts-exchange-libs.svg)](https://www.npmjs.com/package/@0x/contracts-exchange-libs)           | Protocol specific libraries used within the [`Exchange`](https://github.com/0xProject/0x-protocol-specification/blob/master/v2/v2-specification.md#exchange) contract                                                                                 |
 | [`@0x/contracts-extensions`](/contracts/extensions)                 | [![npm](https://img.shields.io/npm/v/@0x/contracts-extensions.svg)](https://www.npmjs.com/package/@0x/contracts-extensions)                 | Contracts that interact with and extend the functionality of the core protocol                                                                                                                                                                        |
 | [`@0x/contracts-multisig`](/contracts/multisig)                     | [![npm](https://img.shields.io/npm/v/@0x/contracts-multisig.svg)](https://www.npmjs.com/package/@0x/contracts-multisig)                     | Various implementations of multisignature wallets, including the [`AssetProxyOwner`](https://github.com/0xProject/0x-protocol-specification/blob/master/v2/v2-specification.md#assetproxyowner) contract that has permissions to upgrade the protocol |
-| [`@0x/contracts-test-utils`](/contracts/test-utils)                 | [![npm](https://img.shields.io/npm/v/@0x/contracts-test-utils.svg)](https://www.npmjs.com/package/@0x/contracts-test-utils)                 | TypeScript/Javascript shared utilities used for testing contracts                                                                                                                                                                                     |
+| [`@0x/contracts-test-utils`](/contracts/test-utils)                 | [![npm](https://img.shields.io/npm/v/@0x/contracts-test-utils.svg)](https://www.npmjs.com/package/@0x/contracts-test-utils)                 | Typescript/Javascript shared utilities used for testing contracts                                                                                                                                                                                     |
 | [`@0x/contracts-utils`](/contracts/utils)                           | [![npm](https://img.shields.io/npm/v/@0x/contracts-utils.svg)](https://www.npmjs.com/package/@0x/contracts-utils)                           | Generic libraries and utilities used throughout all of the contracts                                                                                                                                                                                  |
 | [`@0x/contracts-coordinator`](/contracts/coordinator)               | [![npm](https://img.shields.io/npm/v/@0x/contracts-coordinator.svg)](https://www.npmjs.com/package/@0x/contracts-coordinator)               | A contract that allows users to execute 0x transactions with permission from a Coordinator                                                                                                                                                            |
 | [`@0x/contracts-dev-utils`](/contracts/dev-utils)                   | [![npm](https://img.shields.io/npm/v/@0x/contracts-dev-utils.svg)](https://www.npmjs.com/package/@0x/contracts-dev-utils)                   | A contract contains utility functions for developers (such as validating many orders using a single eth_call)                                                                                                                                         |
-| [`@0x/contracts-staking`](/contracts/staking)                       | [![npm](https://img.shields.io/npm/v/@0x/contracts-staking.svg)](https://www.npmjs.com/package/@0x/contracts-staking)                       | Implements the stake-based liquidity incentives defined by [`ZEIP-31`](https://github.com/0xProject/ZEIPs/issues/31)                                                                                                                                  |
 
-### TypeScript/Javascript Packages
+### Typescript/Javascript Packages
 
 #### 0x-specific packages
 
@@ -93,6 +97,7 @@ These packages are all under development. See [/contracts/README.md](/contracts/
 | [`@0x/assert`](/packages/assert)                         | [![npm](https://img.shields.io/npm/v/@0x/assert.svg)](https://www.npmjs.com/package/@0x/assert)                         | Type and schema assertions used by our packages                 |
 | [`@0x/base-contract`](/packages/base-contract)           | [![npm](https://img.shields.io/npm/v/@0x/base-contract.svg)](https://www.npmjs.com/package/@0x/base-contract)           | BaseContract used by auto-generated `abi-gen` wrapper contracts |
 | [`@0x/dev-utils`](/packages/dev-utils)                   | [![npm](https://img.shields.io/npm/v/@0x/dev-utils.svg)](https://www.npmjs.com/package/@0x/dev-utils)                   | Dev utils to be shared across 0x packages                       |
+| [`@0x/fill-scenarios`](/packages/fill-scenarios)         | [![npm](https://img.shields.io/npm/v/@0x/fill-scenarios.svg)](https://www.npmjs.com/package/@0x/fill-scenarios)         | 0x order fill scenario generator                                |
 
 #### Private Packages
 
@@ -100,6 +105,7 @@ These packages are all under development. See [/contracts/README.md](/contracts/
 | -------------------------------------------------- | -------------------------------------------------------------------------------- |
 | [`@0x/instant`](/packages/instant)                 | A free and flexible way to offer simple crypto purchasing in any app or website. |
 | [`@0x/testnet-faucets`](/packages/testnet-faucets) | A faucet micro-service that dispenses test ERC20 tokens or Ether                 |
+| [`@0x/website`](/packages/website)                 | 0x website                                                                       |
 
 ## Usage
 

contracts/asset-proxy/.solhintignore

@@ -1,2 +0,0 @@
-# solhint can't parse `abi.decode` syntax.
-contracts/src/ERC1155Proxy.sol

contracts/asset-proxy/CHANGELOG.json

@@ -1,52 +1,4 @@
 [
-    {
-        "version": "2.3.0-beta.0",
-        "changes": [
-            {
-                "note": "Disallow the zero address from being made an authorized address in MixinAuthorizable, and created an archive directory that includes an old version of Ownable",
-                "pr": 2019
-            },
-            {
-                "note": "Remove `LibAssetProxyIds` contract",
-                "pr": 2055
-            },
-            {
-                "note": "Compile and export all contracts, artifacts, and wrappers by default",
-                "pr": 2055
-            },
-            {
-                "note": "Remove unused dependency on IAuthorizable in IAssetProxy",
-                "pr": 1910
-            },
-            {
-                "note": "Add `ERC20BridgeProxy`",
-                "pr": 2220
-            },
-            {
-                "note": "Add `Eth2DaiBridge`",
-                "pr": 2221
-            }
-        ],
-        "timestamp": 1570135330
-    },
-    {
-        "timestamp": 1568744790,
-        "version": "2.2.8",
-        "changes": [
-            {
-                "note": "Dependencies updated"
-            }
-        ]
-    },
-    {
-        "timestamp": 1567521715,
-        "version": "2.2.7",
-        "changes": [
-            {
-                "note": "Dependencies updated"
-            }
-        ]
-    },
     {
         "timestamp": 1566446343,
         "version": "2.2.6",
@@ -153,18 +105,6 @@
             {
                 "note": "Update tests to use contract-built-in `awaitTransactionSuccessAsync`",
                 "pr": 1797
-            },
-            {
-                "note": "Make `ERC721Wrapper.setApprovalForAll()` take an owner address instead of a token ID",
-                "pr": 1819
-            },
-            {
-                "note": "Automatically set unlimited proxy allowances in `ERC721.setBalancesAndAllowancesAsync()`",
-                "pr": 1819
-            },
-            {
-                "note": "Add `setProxyAllowanceForAllAsync()` to `ERC1155ProxyWrapper`.",
-                "pr": 1819
             }
         ],
         "timestamp": 1557507213

contracts/asset-proxy/CHANGELOG.md

@@ -5,23 +5,6 @@ Edit the package's CHANGELOG.json file only.
 
 CHANGELOG
 
-## v2.3.0-beta.0 - _October 3, 2019_
-
-    * Disallow the zero address from being made an authorized address in MixinAuthorizable, and created an archive directory that includes an old version of Ownable (#2019)
-    * Remove `LibAssetProxyIds` contract (#2055)
-    * Compile and export all contracts, artifacts, and wrappers by default (#2055)
-    * Remove unused dependency on IAuthorizable in IAssetProxy (#1910)
-    * Add `ERC20BridgeProxy` (#2220)
-    * Add `Eth2DaiBridge` (#2221)
-
-## v2.2.8 - _September 17, 2019_
-
-    * Dependencies updated
-
-## v2.2.7 - _September 3, 2019_
-
-    * Dependencies updated
-
 ## v2.2.6 - _August 22, 2019_
 
     * Dependencies updated
@@ -67,9 +50,6 @@ CHANGELOG
 ## v2.1.2 - _May 10, 2019_
 
     * Update tests to use contract-built-in `awaitTransactionSuccessAsync` (#1797)
-    * Make `ERC721Wrapper.setApprovalForAll()` take an owner address instead of a token ID (#1819)
-    * Automatically set unlimited proxy allowances in `ERC721.setBalancesAndAllowancesAsync()` (#1819)
-    * Add `setProxyAllowanceForAllAsync()` to `ERC1155ProxyWrapper`. (#1819)
 
 ## v2.1.1 - _April 11, 2019_
 

contracts/asset-proxy/README.md

@@ -1,6 +1,6 @@
 ## AssetProxy
 
-This package contains the implementations of all of the [`AssetProxy`](https://github.com/0xProject/0x-protocol-specification/blob/master/v2/v2-specification.md#assetproxy) contracts available within the 0x protocol. These contracts are responsible for decoding the `assetData` sent to them and performing the actual transfer of assets. Addresses of the deployed contracts can be found in this 0x [guide](https://0x.org/docs/guides/0x-cheat-sheet) or the [DEPLOYS](./DEPLOYS.json) file within this package.
+This package contains the implementations of all of the [`AssetProxy`](https://github.com/0xProject/0x-protocol-specification/blob/master/v2/v2-specification.md#assetproxy) contracts available within the 0x protocol. These contracts are responsible for decoding the `assetData` sent to them and performing the actual transfer of assets. Addresses of the deployed contracts can be found in the 0x [wiki](https://0xproject.com/wiki#Deployed-Addresses) or the [DEPLOYS](./DEPLOYS.json) file within this package.
 
 ## Installation
 
@@ -12,7 +12,7 @@ npm install @0x/contracts-asset-proxy --save
 
 ## Bug bounty
 
-A bug bounty for the 2.0.0 contracts is ongoing! Instructions can be found [here](https://0x.org/docs/guides/bug-bounty-program).
+A bug bounty for the 2.0.0 contracts is ongoing! Instructions can be found [here](https://0xproject.com/wiki#Bug-Bounty).
 
 ## Contributing
 

contracts/asset-proxy/compiler.json

@@ -22,5 +22,17 @@
                 ]
             }
         }
-    }
+    },
+    "contracts": [
+        "src/ERC1155Proxy.sol",
+        "src/ERC20Proxy.sol",
+        "src/ERC721Proxy.sol",
+        "src/MixinAuthorizable.sol",
+        "src/MultiAssetProxy.sol",
+        "src/StaticCallProxy.sol",
+        "src/interfaces/IAssetData.sol",
+        "src/interfaces/IAssetProxy.sol",
+        "src/interfaces/IAuthorizable.sol",
+        "test/TestStaticCallTarget.sol"
+    ]
 }

contracts/asset-proxy/contracts/archive/Ownable.sol

@@ -1,33 +0,0 @@
-pragma solidity ^0.5.9;
-
-import "@0x/contracts-utils/contracts/src/interfaces/IOwnable.sol";
-
-
-contract Ownable is
-    IOwnable
-{
-    address public owner;
-
-    constructor ()
-        public
-    {
-        owner = msg.sender;
-    }
-
-    modifier onlyOwner() {
-        require(
-            msg.sender == owner,
-            "ONLY_CONTRACT_OWNER"
-        );
-        _;
-    }
-
-    function transferOwnership(address newOwner)
-        public
-        onlyOwner
-    {
-        if (newOwner != address(0)) {
-            owner = newOwner;
-        }
-    }
-}

contracts/asset-proxy/contracts/src/ERC1155Proxy.sol

@@ -1,6 +1,6 @@
 /*
 
-  Copyright 2019 ZeroEx Intl.
+  Copyright 2018 ZeroEx Intl.
 
   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
@@ -21,7 +21,7 @@ pragma solidity ^0.5.9;
 import "@0x/contracts-utils/contracts/src/LibBytes.sol";
 import "@0x/contracts-utils/contracts/src/SafeMath.sol";
 import "@0x/contracts-erc1155/contracts/src/interfaces/IERC1155.sol";
-import "../archive/MixinAuthorizable.sol";
+import "./MixinAuthorizable.sol";
 import "./interfaces/IAssetProxy.sol";
 
 
@@ -69,9 +69,9 @@ contract ERC1155Proxy is
         for (uint256 i = 0; i != length; i++) {
             // We write the scaled values to an unused location in memory in order
             // to avoid copying over `ids` or `data`. This is possible if they are
-            // identical to `values` and the offsets for each are pointing to the
+            // identical to `values` and the offsets for each are pointing to the 
             // same location in the ABI encoded calldata.
-            scaledValues[i] = _safeMul(values[i], amount);
+            scaledValues[i] = safeMul(values[i], amount);
         }
 
         // Execute `safeBatchTransferFrom` call

contracts/asset-proxy/contracts/src/ERC20BridgeProxy.sol

@@ -1,126 +0,0 @@
-/*
-
-  Copyright 2019 ZeroEx Intl.
-
-  Licensed under the Apache License, Version 2.0 (the "License");
-  you may not use this file except in compliance with the License.
-  You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-  Unless required by applicable law or agreed to in writing, software
-  distributed under the License is distributed on an "AS IS" BASIS,
-  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-  See the License for the specific language governing permissions and
-  limitations under the License.
-
-*/
-
-pragma solidity ^0.5.9;
-pragma experimental ABIEncoderV2;
-
-import "@0x/contracts-utils/contracts/src/LibBytes.sol";
-import "@0x/contracts-utils/contracts/src/LibSafeMath.sol";
-import "@0x/contracts-utils/contracts/src/Authorizable.sol";
-import "@0x/contracts-erc20/contracts/src/interfaces/IERC20Token.sol";
-import "./interfaces/IAssetProxy.sol";
-import "./interfaces/IERC20Bridge.sol";
-
-
-contract ERC20BridgeProxy is
-    IAssetProxy,
-    Authorizable
-{
-    using LibBytes for bytes;
-    using LibSafeMath for uint256;
-
-    // @dev Id of this proxy. Also the result of a successful bridge call.
-    //      bytes4(keccak256("ERC20Bridge(address,address,bytes)"))
-    bytes4 constant private PROXY_ID = 0xdc1600f3;
-
-    /// @dev Calls a bridge contract to transfer `amount` of ERC20 from `from`
-    ///      to `to`. Asserts that the balance of `to` has increased by `amount`.
-    /// @param assetData Abi-encoded data for this asset proxy encoded as:
-    ///          abi.encodeWithSelector(
-    ///             bytes4 PROXY_ID,
-    ///             address tokenAddress,
-    ///             address bridgeAddress,
-    ///             bytes bridgeData
-    ///          )
-    /// @param from Address to transfer asset from.
-    /// @param to Address to transfer asset to.
-    /// @param amount Amount of asset to transfer.
-    function transferFrom(
-        bytes calldata assetData,
-        address from,
-        address to,
-        uint256 amount
-    )
-        external
-        onlyAuthorized
-    {
-        // Extract asset data fields.
-        (
-            address tokenAddress,
-            address bridgeAddress,
-            bytes memory bridgeData
-        ) = abi.decode(
-            assetData.sliceDestructive(4, assetData.length),
-            (address, address, bytes)
-        );
-
-        // Remember the balance of `to` before calling the bridge.
-        uint256 balanceBefore = balanceOf(tokenAddress, to);
-        // Call the bridge, who should transfer `amount` of `tokenAddress` to
-        // `to`.
-        bytes4 success = IERC20Bridge(bridgeAddress).withdrawTo(
-            tokenAddress,
-            from,
-            to,
-            amount,
-            bridgeData
-        );
-        // Bridge must return the proxy ID to indicate success.
-        require(success == PROXY_ID, "BRIDGE_FAILED");
-        // Ensure that the balance of `to` has increased by at least `amount`.
-        require(
-            balanceBefore.safeAdd(amount) <= balanceOf(tokenAddress, to),
-            "BRIDGE_UNDERPAY"
-        );
-    }
-
-    /// @dev Gets the proxy id associated with this asset proxy.
-    /// @return proxyId The proxy id.
-    function getProxyId()
-        external
-        pure
-        returns (bytes4 proxyId)
-    {
-        return PROXY_ID;
-    }
-
-    /// @dev Retrieves the balance of `owner` for this asset.
-    /// @return balance The balance of the ERC20 token being transferred by this
-    ///         asset proxy.
-    function balanceOf(bytes calldata assetData, address owner)
-        external
-        view
-        returns (uint256 balance)
-    {
-        (address tokenAddress) = abi.decode(
-            assetData.sliceDestructive(4, assetData.length),
-            (address)
-        );
-        return balanceOf(tokenAddress, owner);
-    }
-
-    /// @dev Retrieves the balance of `owner` given an ERC20 address.
-    /// @return balance The balance of the ERC20 token for `owner`.
-    function balanceOf(address tokenAddress, address owner)
-        private
-        view
-        returns (uint256 balance)
-    {
-        return IERC20Token(tokenAddress).balanceOf(owner);
-    }
-}

contracts/asset-proxy/contracts/src/ERC20Proxy.sol

@@ -1,6 +1,6 @@
 /*
 
-  Copyright 2019 ZeroEx Intl.
+  Copyright 2018 ZeroEx Intl.
 
   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
@@ -16,9 +16,9 @@
 
 */
 
-pragma solidity ^0.5.9;
+pragma solidity ^0.5.5;
 
-import "../archive/MixinAuthorizable.sol";
+import "./MixinAuthorizable.sol";
 
 
 contract ERC20Proxy is
@@ -26,9 +26,9 @@ contract ERC20Proxy is
 {
     // Id of this proxy.
     bytes4 constant internal PROXY_ID = bytes4(keccak256("ERC20Token(address)"));
-
+    
     // solhint-disable-next-line payable-fallback
-    function ()
+    function () 
         external
     {
         assembly {
@@ -117,13 +117,13 @@ contract ERC20Proxy is
                 // * The "token address" is offset 32+4=36 bytes into "assetData" (tables 1 & 2).
                 //   [tokenOffset = assetDataOffsetFromHeader + 36 = calldataload(4) + 4 + 36]
                 let token := calldataload(add(calldataload(4), 40))
-
+                
                 /////// Setup Header Area ///////
                 // This area holds the 4-byte `transferFrom` selector.
                 // Any trailing data in transferFromSelector will be
                 // overwritten in the next `mstore` call.
                 mstore(0, 0x23b872dd00000000000000000000000000000000000000000000000000000000)
-
+                
                 /////// Setup Params Area ///////
                 // We copy the fields `from`, `to` and `amount` in bulk
                 // from our own calldata to the new calldata.
@@ -147,7 +147,7 @@ contract ERC20Proxy is
                 // If the token does return data, we require that it is a single
                 // nonzero 32 bytes value.
                 // So the transfer succeeded if the call succeeded and either
-                // returned nothing, or returned a non-zero 32 byte value.
+                // returned nothing, or returned a non-zero 32 byte value. 
                 success := and(success, or(
                     iszero(returndatasize),
                     and(
@@ -158,7 +158,7 @@ contract ERC20Proxy is
                 if success {
                     return(0, 0)
                 }
-
+                
                 // Revert with `Error("TRANSFER_FAILED")`
                 mstore(0, 0x08c379a000000000000000000000000000000000000000000000000000000000)
                 mstore(32, 0x0000002000000000000000000000000000000000000000000000000000000000)

contracts/asset-proxy/contracts/src/ERC721Proxy.sol

@@ -1,6 +1,6 @@
 /*
 
-  Copyright 2019 ZeroEx Intl.
+  Copyright 2018 ZeroEx Intl.
 
   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
@@ -16,9 +16,9 @@
 
 */
 
-pragma solidity ^0.5.9;
+pragma solidity ^0.5.5;
 
-import "../archive/MixinAuthorizable.sol";
+import "./MixinAuthorizable.sol";
 
 
 contract ERC721Proxy is
@@ -28,7 +28,7 @@ contract ERC721Proxy is
     bytes4 constant internal PROXY_ID = bytes4(keccak256("ERC721Token(address,uint256)"));
 
     // solhint-disable-next-line payable-fallback
-    function ()
+    function () 
         external
     {
         assembly {
@@ -93,10 +93,10 @@ contract ERC721Proxy is
                 // | Params   |        | 2 * 32  | function parameters:                |
                 // |          | 4      | 12 + 20 |   1. token address                  |
                 // |          | 36     |         |   2. tokenId                        |
-
+                
                 // We construct calldata for the `token.transferFrom` ABI.
                 // The layout of this calldata is in the table below.
-                //
+                // 
                 // | Area     | Offset | Length  | Contents                            |
                 // |----------|--------|---------|-------------------------------------|
                 // | Header   | 0      | 4       | function selector                   |
@@ -121,7 +121,7 @@ contract ERC721Proxy is
                 // Any trailing data in transferFromSelector will be
                 // overwritten in the next `mstore` call.
                 mstore(0, 0x23b872dd00000000000000000000000000000000000000000000000000000000)
-
+                
                 /////// Setup Params Area ///////
                 // We copy the fields `from` and `to` in bulk
                 // from our own calldata to the new calldata.
@@ -145,7 +145,7 @@ contract ERC721Proxy is
                 if success {
                     return(0, 0)
                 }
-
+                
                 // Revert with `Error("TRANSFER_FAILED")`
                 mstore(0, 0x08c379a000000000000000000000000000000000000000000000000000000000)
                 mstore(32, 0x0000002000000000000000000000000000000000000000000000000000000000)

contracts/asset-proxy/contracts/src/MixinAssetProxyDispatcher.sol

@@ -1,6 +1,6 @@
 /*
 
-  Copyright 2019 ZeroEx Intl.
+  Copyright 2018 ZeroEx Intl.
 
   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
@@ -16,16 +16,16 @@
 
 */
 
-pragma solidity ^0.5.9;
+pragma solidity ^0.5.5;
 
-import "../archive/Ownable.sol";
-import "../src/interfaces/IAssetProxy.sol";
-import "../src/interfaces/IAssetProxyDispatcher.sol";
+import "@0x/contracts-utils/contracts/src/Ownable.sol";
+import "./mixins/MAssetProxyDispatcher.sol";
+import "./interfaces/IAssetProxy.sol";
 
 
 contract MixinAssetProxyDispatcher is
     Ownable,
-    IAssetProxyDispatcher
+    MAssetProxyDispatcher
 {
     // Mapping from Asset Proxy Id's to their respective Asset Proxy
     mapping (bytes4 => address) public assetProxies;
@@ -69,7 +69,7 @@ contract MixinAssetProxyDispatcher is
     /// @param from Address to transfer token from.
     /// @param to Address to transfer token to.
     /// @param amount Amount of token to transfer.
-    function _dispatchTransferFrom(
+    function dispatchTransferFrom(
         bytes memory assetData,
         address from,
         address to,
@@ -84,7 +84,7 @@ contract MixinAssetProxyDispatcher is
                 assetData.length > 3,
                 "LENGTH_GREATER_THAN_3_REQUIRED"
             );
-
+            
             // Lookup assetProxy. We do not use `LibBytes.readBytes4` for gas efficiency reasons.
             bytes4 assetProxyId;
             assembly {
@@ -100,10 +100,10 @@ contract MixinAssetProxyDispatcher is
                 assetProxy != address(0),
                 "ASSET_PROXY_DOES_NOT_EXIST"
             );
-
+            
             // We construct calldata for the `assetProxy.transferFrom` ABI.
             // The layout of this calldata is in the table below.
-            //
+            // 
             // | Area     | Offset | Length  | Contents                                    |
             // | -------- |--------|---------|-------------------------------------------- |
             // | Header   | 0      | 4       | function selector                           |
@@ -127,12 +127,12 @@ contract MixinAssetProxyDispatcher is
                 // `cdEnd` is the end of the calldata for `assetProxy.transferFrom`.
                 let cdEnd := add(cdStart, add(132, dataAreaLength))
 
-
+                
                 /////// Setup Header Area ///////
                 // This area holds the 4-byte `transferFromSelector`.
                 // bytes4(keccak256("transferFrom(bytes,address,address,uint256)")) = 0xa85e59e4
                 mstore(cdStart, 0xa85e59e400000000000000000000000000000000000000000000000000000000)
-
+                
                 /////// Setup Params Area ///////
                 // Each parameter is padded to 32-bytes. The entire Params Area is 128 bytes.
                 // Notes:
@@ -142,7 +142,7 @@ contract MixinAssetProxyDispatcher is
                 mstore(add(cdStart, 36), and(from, 0xffffffffffffffffffffffffffffffffffffffff))
                 mstore(add(cdStart, 68), and(to, 0xffffffffffffffffffffffffffffffffffffffff))
                 mstore(add(cdStart, 100), amount)
-
+                
                 /////// Setup Data Area ///////
                 // This area holds `assetData`.
                 let dataArea := add(cdStart, 132)
@@ -159,7 +159,7 @@ contract MixinAssetProxyDispatcher is
                     assetProxy,             // call address of asset proxy
                     0,                      // don't send any ETH
                     cdStart,                // pointer to start of input
-                    sub(cdEnd, cdStart),    // length of input
+                    sub(cdEnd, cdStart),    // length of input  
                     cdStart,                // write output over input
                     512                     // reserve 512 bytes for output
                 )

contracts/asset-proxy/contracts/src/MixinAuthorizable.sol

@@ -1,6 +1,6 @@
 /*
 
-  Copyright 2019 ZeroEx Intl.
+  Copyright 2018 ZeroEx Intl.
 
   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
@@ -16,15 +16,15 @@
 
 */
 
-pragma solidity ^0.5.9;
+pragma solidity ^0.5.5;
 
-import "../archive/Ownable.sol";
-import "../src/interfaces/IAuthorizable.sol";
+import "@0x/contracts-utils/contracts/src/Ownable.sol";
+import "./mixins/MAuthorizable.sol";
 
 
 contract MixinAuthorizable is
     Ownable,
-    IAuthorizable
+    MAuthorizable
 {
     /// @dev Only authorized addresses can invoke functions with this modifier.
     modifier onlyAuthorized {

contracts/asset-proxy/contracts/src/MultiAssetProxy.sol

@@ -1,6 +1,6 @@
 /*
 
-  Copyright 2019 ZeroEx Intl.
+  Copyright 2018 ZeroEx Intl.
 
   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
@@ -16,10 +16,10 @@
 
 */
 
-pragma solidity ^0.5.9;
+pragma solidity ^0.5.5;
 
-import "../archive/MixinAssetProxyDispatcher.sol";
-import "../archive/MixinAuthorizable.sol";
+import "./MixinAssetProxyDispatcher.sol";
+import "./MixinAuthorizable.sol";
 
 
 contract MultiAssetProxy is
@@ -105,7 +105,7 @@ contract MultiAssetProxy is
                 // |          | 36          |         |   2. offset to nestedAssetData (*)  |
                 // | Data     |             |         | amounts:                            |
                 // |          | 68          | 32      | amounts Length                      |
-                // |          | 100         | a       | amounts Contents                    |
+                // |          | 100         | a       | amounts Contents                    | 
                 // |          |             |         | nestedAssetData:                    |
                 // |          | 100 + a     | 32      | nestedAssetData Length              |
                 // |          | 132 + a     | b       | nestedAssetData Contents (offsets)  |
@@ -149,8 +149,8 @@ contract MultiAssetProxy is
                 // + 32 (amounts offset)
                 let nestedAssetDataOffset := calldataload(add(assetDataOffset, 68))
 
-                // In order to find the start of the `amounts` contents, we must add:
-                // assetDataOffset
+                // In order to find the start of the `amounts` contents, we must add: 
+                // assetDataOffset 
                 // + 32 (assetData len)
                 // + 4 (assetProxyId)
                 // + amountsOffset
@@ -160,8 +160,8 @@ contract MultiAssetProxy is
                 // Load number of elements in `amounts`
                 let amountsLen := calldataload(sub(amountsContentsStart, 32))
 
-                // In order to find the start of the `nestedAssetData` contents, we must add:
-                // assetDataOffset
+                // In order to find the start of the `nestedAssetData` contents, we must add: 
+                // assetDataOffset 
                 // + 32 (assetData len)
                 // + 4 (assetProxyId)
                 // + nestedAssetDataOffset
@@ -190,10 +190,10 @@ contract MultiAssetProxy is
 
                 // Overwrite existing offset to `assetData` with our own
                 mstore(4, 128)
-
+                
                 // Load `amount`
                 let amount := calldataload(100)
-
+        
                 // Calculate number of bytes in `amounts` contents
                 let amountsByteLen := mul(amountsLen, 32)
 
@@ -208,7 +208,7 @@ contract MultiAssetProxy is
                     let amountsElement := calldataload(add(amountsContentsStart, i))
                     let totalAmount := mul(amountsElement, amount)
 
-                    // Revert if `amount` != 0 and multiplication resulted in an overflow
+                    // Revert if `amount` != 0 and multiplication resulted in an overflow 
                     if iszero(or(
                         iszero(amount),
                         eq(div(totalAmount, amount), amountsElement)
@@ -228,7 +228,7 @@ contract MultiAssetProxy is
                     let nestedAssetDataElementOffset := calldataload(add(nestedAssetDataContentsStart, i))
 
                     // In order to find the start of the `nestedAssetData[i]` contents, we must add:
-                    // assetDataOffset
+                    // assetDataOffset 
                     // + 32 (assetData len)
                     // + 4 (assetProxyId)
                     // + nestedAssetDataOffset
@@ -274,7 +274,7 @@ contract MultiAssetProxy is
                         mstore(164, assetProxies_slot)
                         assetProxy := sload(keccak256(132, 64))
                     }
-
+                    
                     // Revert if AssetProxy with given id does not exist
                     if iszero(assetProxy) {
                         // Revert with `Error("ASSET_PROXY_DOES_NOT_EXIST")`
@@ -284,7 +284,7 @@ contract MultiAssetProxy is
                         mstore(96, 0)
                         revert(0, 100)
                     }
-
+    
                     // Copy `nestedAssetData[i]` from calldata to memory
                     calldatacopy(
                         132,                                // memory slot after `amounts[i]`
@@ -298,7 +298,7 @@ contract MultiAssetProxy is
                         assetProxy,                             // call address of asset proxy
                         0,                                      // don't send any ETH
                         0,                                      // pointer to start of input
-                        add(164, nestedAssetDataElementLen),    // length of input
+                        add(164, nestedAssetDataElementLen),    // length of input  
                         0,                                      // write output over memory that won't be reused
                         0                                       // don't copy output to memory
                     )

contracts/asset-proxy/contracts/src/StaticCallProxy.sol

@@ -1,6 +1,6 @@
 /*
 
-  Copyright 2019 ZeroEx Intl.
+  Copyright 2018 ZeroEx Intl.
 
   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.

contracts/asset-proxy/contracts/src/bridges/Eth2DaiBridge.sol

@@ -1,142 +0,0 @@
-/*
-
-  Copyright 2019 ZeroEx Intl.
-
-  Licensed under the Apache License, Version 2.0 (the "License");
-  you may not use this file except in compliance with the License.
-  You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-  Unless required by applicable law or agreed to in writing, software
-  distributed under the License is distributed on an "AS IS" BASIS,
-  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-  See the License for the specific language governing permissions and
-  limitations under the License.
-
-*/
-
-pragma solidity ^0.5.9;
-pragma experimental ABIEncoderV2;
-
-import "@0x/contracts-erc20/contracts/src/interfaces/IERC20Token.sol";
-import "../interfaces/IERC20Bridge.sol";
-import "../interfaces/IEth2Dai.sol";
-import "../interfaces/IWallet.sol";
-
-
-// solhint-disable space-after-comma
-contract Eth2DaiBridge is
-    IERC20Bridge,
-    IWallet
-{
-    /* Mainnet addresses */
-    address constant public ETH2DAI_ADDRESS = 0x39755357759cE0d7f32dC8dC45414CCa409AE24e;
-
-    /// @dev Callback for `IERC20Bridge`. Tries to buy `amount` of
-    ///      `toTokenAddress` tokens by selling the entirety of the opposing asset
-    ///      (DAI or WETH) to the Eth2Dai contract, then transfers the bought
-    ///      tokens to `to`.
-    /// @param toTokenAddress The token to give to `to` (either DAI or WETH).
-    /// @param to The recipient of the bought tokens.
-    /// @param amount Minimum amount of `toTokenAddress` tokens to buy.
-    /// @param bridgeData The abi-encoeded "from" token address.
-    /// @return success The magic bytes if successful.
-    function withdrawTo(
-        address toTokenAddress,
-        address /* from */,
-        address to,
-        uint256 amount,
-        bytes calldata bridgeData
-    )
-        external
-        returns (bytes4 success)
-    {
-        // Decode the bridge data to get the `fromTokenAddress`.
-        (address fromTokenAddress) = abi.decode(bridgeData, (address));
-
-        IEth2Dai exchange = _getEth2DaiContract();
-        // Grant an allowance to the exchange to spend `fromTokenAddress` token.
-        IERC20Token(fromTokenAddress).approve(address(exchange), uint256(-1));
-
-        // Try to sell all of this contract's `fromTokenAddress` token balance.
-        uint256 boughtAmount = _getEth2DaiContract().sellAllAmount(
-            address(fromTokenAddress),
-            IERC20Token(fromTokenAddress).balanceOf(address(this)),
-            toTokenAddress,
-            amount
-        );
-        // Transfer the converted `toToken`s to `to`.
-        _transferERC20Token(toTokenAddress, to, boughtAmount);
-        return BRIDGE_SUCCESS;
-    }
-
-    /// @dev `SignatureType.Wallet` callback, so that this bridge can be the maker
-    ///      and sign for itself in orders. Always succeeds.
-    /// @return magicValue Magic success bytes, always.
-    function isValidSignature(
-        bytes32,
-        bytes calldata
-    )
-        external
-        view
-        returns (bytes4 magicValue)
-    {
-        return LEGACY_WALLET_MAGIC_VALUE;
-    }
-
-    /// @dev Overridable way to get the eth2dai contract.
-    /// @return exchange The Eth2Dai exchange contract.
-    function _getEth2DaiContract()
-        internal
-        view
-        returns (IEth2Dai exchange)
-    {
-        return IEth2Dai(ETH2DAI_ADDRESS);
-    }
-
-    /// @dev Permissively transfers an ERC20 token that may not adhere to
-    ///      specs.
-    /// @param tokenAddress The token contract address.
-    /// @param to The token recipient.
-    /// @param amount The amount of tokens to transfer.
-    function _transferERC20Token(
-        address tokenAddress,
-        address to,
-        uint256 amount
-    )
-        private
-    {
-        // Transfer tokens.
-        // We do a raw call so we can check the success separate
-        // from the return data.
-        (bool didSucceed, bytes memory returnData) = tokenAddress.call(
-            abi.encodeWithSelector(
-                IERC20Token(0).transfer.selector,
-                to,
-                amount
-            )
-        );
-        if (!didSucceed) {
-            assembly { revert(add(returnData, 0x20), mload(returnData)) }
-        }
-
-        // Check return data.
-        // If there is no return data, we assume the token incorrectly
-        // does not return a bool. In this case we expect it to revert
-        // on failure, which was handled above.
-        // If the token does return data, we require that it is a single
-        // value that evaluates to true.
-        assembly {
-            if returndatasize {
-                didSucceed := 0
-                if eq(returndatasize, 32) {
-                    // First 64 bytes of memory are reserved scratch space
-                    returndatacopy(0, 0, 32)
-                    didSucceed := mload(0)
-                }
-            }
-        }
-        require(didSucceed, "ERC20_TRANSFER_FAILED");
-    }
-}

contracts/asset-proxy/contracts/src/interfaces/IAssetData.sol

@@ -1,6 +1,6 @@
 /*
 
-  Copyright 2019 ZeroEx Intl.
+  Copyright 2018 ZeroEx Intl.
 
   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
@@ -17,7 +17,7 @@
 */
 
 // solhint-disable
-pragma solidity ^0.5.9;
+pragma solidity ^0.5.5;
 pragma experimental ABIEncoderV2;
 
 
@@ -26,63 +26,33 @@ pragma experimental ABIEncoderV2;
 // This argument is ABI encoded as one of the methods of this interface.
 interface IAssetData {
 
-    /// @dev Function signature for encoding ERC20 assetData.
-    /// @param tokenAddress Address of ERC20Token contract.
     function ERC20Token(address tokenAddress)
         external;
-
-    /// @dev Function signature for encoding ERC721 assetData.
-    /// @param tokenAddress Address of ERC721 token contract.
-    /// @param tokenId Id of ERC721 token to be transferred.
+    
     function ERC721Token(
         address tokenAddress,
         uint256 tokenId
     )
         external;
 
-    /// @dev Function signature for encoding ERC1155 assetData.
-    /// @param tokenAddress Address of ERC1155 token contract.
-    /// @param tokenIds Array of ids of tokens to be transferred.
-    /// @param values Array of values that correspond to each token id to be transferred.
-    ///        Note that each value will be multiplied by the amount being filled in the order before transferring.
-    /// @param callbackData Extra data to be passed to receiver's `onERC1155Received` callback function.
     function ERC1155Assets(
         address tokenAddress,
         uint256[] calldata tokenIds,
-        uint256[] calldata values,
+        uint256[] calldata tokenValues,
         bytes calldata callbackData
     )
         external;
 
-    /// @dev Function signature for encoding MultiAsset assetData.
-    /// @param values Array of amounts that correspond to each asset to be transferred.
-    ///        Note that each value will be multiplied by the amount being filled in the order before transferring.
-    /// @param nestedAssetData Array of assetData fields that will be be dispatched to their correspnding AssetProxy contract.
     function MultiAsset(
-        uint256[] calldata values,
+        uint256[] calldata amounts,
         bytes[] calldata nestedAssetData
     )
         external;
 
-    /// @dev Function signature for encoding StaticCall assetData.
-    /// @param staticCallTargetAddress Address that will execute the staticcall.
-    /// @param staticCallData Data that will be executed via staticcall on the staticCallTargetAddress.
-    /// @param expectedReturnDataHash Keccak-256 hash of the expected staticcall return data.
     function StaticCall(
-        address staticCallTargetAddress,
+        address callTarget,
         bytes calldata staticCallData,
-        bytes32 expectedReturnDataHash
-    )
-        external;
-
-    /// @dev Function signature for encoding ERC20Bridge assetData.
-    /// @param tokenAddress Address of token to transfer.
-    /// @param bridgeAddress Address of the bridge contract.
-    /// @param bridgeData Arbitrary data to be passed to the bridge contract.
-    function ERC20Bridge(
-        address tokenAddress,
-        address bridgeAddress,
-        bytes calldata bridgeData
+        bytes32 callResultHash
     )
         external;
 }

contracts/asset-proxy/contracts/src/interfaces/IAssetProxy.sol

@@ -1,6 +1,6 @@
 /*
 
-  Copyright 2019 ZeroEx Intl.
+  Copyright 2018 ZeroEx Intl.
 
   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
@@ -16,7 +16,9 @@
 
 */
 
-pragma solidity ^0.5.9;
+pragma solidity ^0.5.5;
+
+import "./IAuthorizable.sol";
 
 
 contract IAssetProxy {

contracts/asset-proxy/contracts/src/interfaces/IAssetProxyDispatcher.sol

@@ -1,6 +1,6 @@
 /*
 
-  Copyright 2019 ZeroEx Intl.
+  Copyright 2018 ZeroEx Intl.
 
   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
@@ -16,17 +16,11 @@
 
 */
 
-pragma solidity ^0.5.9;
+pragma solidity ^0.5.5;
 
 
 contract IAssetProxyDispatcher {
 
-    // Logs registration of new asset proxy
-    event AssetProxyRegistered(
-        bytes4 id,              // Id of new registered AssetProxy.
-        address assetProxy      // Address of new registered AssetProxy.
-    );
-
     /// @dev Registers an asset proxy to its asset proxy id.
     ///      Once an asset proxy is registered, it cannot be unregistered.
     /// @param assetProxy Address of new asset proxy to register.

contracts/asset-proxy/contracts/src/interfaces/IAuthorizable.sol

@@ -1,6 +1,6 @@
 /*
 
-  Copyright 2019 ZeroEx Intl.
+  Copyright 2018 ZeroEx Intl.
 
   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
@@ -16,7 +16,7 @@
 
 */
 
-pragma solidity ^0.5.9;
+pragma solidity ^0.5.5;
 
 import "@0x/contracts-utils/contracts/src/interfaces/IOwnable.sol";
 
@@ -24,18 +24,6 @@ import "@0x/contracts-utils/contracts/src/interfaces/IOwnable.sol";
 contract IAuthorizable is
     IOwnable
 {
-    // Event logged when a new address is authorized.
-    event AuthorizedAddressAdded(
-        address indexed target,
-        address indexed caller
-    );
-
-    // Event logged when a currently authorized address is unauthorized.
-    event AuthorizedAddressRemoved(
-        address indexed target,
-        address indexed caller
-    );
-
     /// @dev Authorizes an address.
     /// @param target Address to authorize.
     function addAuthorizedAddress(address target)
@@ -54,7 +42,7 @@ contract IAuthorizable is
         uint256 index
     )
         external;
-
+    
     /// @dev Gets all authorized addresses.
     /// @return Array of authorized addresses.
     function getAuthorizedAddresses()

contracts/asset-proxy/contracts/src/interfaces/IERC20Bridge.sol

@@ -1,43 +0,0 @@
-/*
-
-  Copyright 2019 ZeroEx Intl.
-
-  Licensed under the Apache License, Version 2.0 (the "License");
-  you may not use this file except in compliance with the License.
-  You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-  Unless required by applicable law or agreed to in writing, software
-  distributed under the License is distributed on an "AS IS" BASIS,
-  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-  See the License for the specific language governing permissions and
-  limitations under the License.
-
-*/
-
-pragma solidity ^0.5.9;
-
-
-contract IERC20Bridge {
-
-    // @dev Result of a successful bridge call.
-    bytes4 constant internal BRIDGE_SUCCESS = 0xdc1600f3;
-
-    /// @dev Transfers `amount` of the ERC20 `tokenAddress` from `from` to `to`.
-    /// @param tokenAddress The address of the ERC20 token to transfer.
-    /// @param from Address to transfer asset from.
-    /// @param to Address to transfer asset to.
-    /// @param amount Amount of asset to transfer.
-    /// @param bridgeData Arbitrary asset data needed by the bridge contract.
-    /// @return success The magic bytes `0x37708e9b` if successful.
-    function withdrawTo(
-        address tokenAddress,
-        address from,
-        address to,
-        uint256 amount,
-        bytes calldata bridgeData
-    )
-        external
-        returns (bytes4 success);
-}

contracts/asset-proxy/contracts/src/interfaces/IEth2Dai.sol

@@ -1,38 +0,0 @@
-/*
-
-  Copyright 2019 ZeroEx Intl.
-
-  Licensed under the Apache License, Version 2.0 (the "License");
-  you may not use this file except in compliance with the License.
-  You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-  Unless required by applicable law or agreed to in writing, software
-  distributed under the License is distributed on an "AS IS" BASIS,
-  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-  See the License for the specific language governing permissions and
-  limitations under the License.
-
-*/
-
-pragma solidity ^0.5.9;
-
-
-interface IEth2Dai {
-
-    /// @dev Sell `sellAmount` of `fromToken` token and receive `toToken` token.
-    /// @param fromToken The token being sold.
-    /// @param sellAmount The amount of `fromToken` token being sold.
-    /// @param toToken The token being bought.
-    /// @param minFillAmount Minimum amount of `toToken` token to buy.
-    /// @return fillAmount Amount of `toToken` bought.
-    function sellAllAmount(
-        address fromToken,
-        uint256 sellAmount,
-        address toToken,
-        uint256 minFillAmount
-    )
-        external
-        returns (uint256 fillAmount);
-}

contracts/asset-proxy/contracts/src/libs/LibAssetProxyIds.sol

@@ -0,0 +1,40 @@
+/*
+
+  Copyright 2019 ZeroEx Intl.
+
+  Licensed under the Apache License, Version 2.0 (the "License");
+  you may not use this file except in compliance with the License.
+  You may obtain a copy of the License at
+
+    http://www.apache.org/licenses/LICENSE-2.0
+
+  Unless required by applicable law or agreed to in writing, software
+  distributed under the License is distributed on an "AS IS" BASIS,
+  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+  See the License for the specific language governing permissions and
+  limitations under the License.
+
+*/
+
+pragma solidity ^0.5.5;
+
+
+contract LibAssetProxyIds {
+
+    // AssetProxy Ids are equiavalent the first 4 bytes of the keccak256 hash of the function signature assigned to each AssetProxy.
+
+    // ERC20Token(address)
+    bytes4 constant public ERC20_PROXY_ID = 0xf47261b0;
+
+    // ERC721Token(address,uint256)
+    bytes4 constant public ERC721_PROXY_ID = 0x02571792;
+
+    // ERC1155Assets(address,uint256[],uint256[],bytes)
+    bytes4 constant public ERC1155_PROXY_ID = 0xa7cb5fb7;
+
+    // MultiAsset(uint256[],bytes[])
+    bytes4 constant public MULTI_ASSET_PROXY_ID = 0x94cfcdd7;
+
+    // StaticCall(address,bytes,bytes32)
+    bytes4 constant public STATIC_CALL_PROXY_ID = 0xc339d10a;
+}

contracts/asset-proxy/contracts/src/mixins/MAssetProxyDispatcher.sol

@@ -0,0 +1,45 @@
+/*
+
+  Copyright 2018 ZeroEx Intl.
+
+  Licensed under the Apache License, Version 2.0 (the "License");
+  you may not use this file except in compliance with the License.
+  You may obtain a copy of the License at
+
+    http://www.apache.org/licenses/LICENSE-2.0
+
+  Unless required by applicable law or agreed to in writing, software
+  distributed under the License is distributed on an "AS IS" BASIS,
+  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+  See the License for the specific language governing permissions and
+  limitations under the License.
+
+*/
+
+pragma solidity ^0.5.5;
+
+import "../interfaces/IAssetProxyDispatcher.sol";
+
+
+contract MAssetProxyDispatcher is
+    IAssetProxyDispatcher
+{
+    // Logs registration of new asset proxy
+    event AssetProxyRegistered(
+        bytes4 id,              // Id of new registered AssetProxy.
+        address assetProxy      // Address of new registered AssetProxy.
+    );
+
+    /// @dev Forwards arguments to assetProxy and calls `transferFrom`. Either succeeds or throws.
+    /// @param assetData Byte array encoded for the asset.
+    /// @param from Address to transfer token from.
+    /// @param to Address to transfer token to.
+    /// @param amount Amount of token to transfer.
+    function dispatchTransferFrom(
+        bytes memory assetData,
+        address from,
+        address to,
+        uint256 amount
+    )
+        internal;
+}

contracts/asset-proxy/contracts/src/mixins/MAuthorizable.sol

@@ -0,0 +1,41 @@
+/*
+
+  Copyright 2018 ZeroEx Intl.
+
+  Licensed under the Apache License, Version 2.0 (the "License");
+  you may not use this file except in compliance with the License.
+  You may obtain a copy of the License at
+
+    http://www.apache.org/licenses/LICENSE-2.0
+
+  Unless required by applicable law or agreed to in writing, software
+  distributed under the License is distributed on an "AS IS" BASIS,
+  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+  See the License for the specific language governing permissions and
+  limitations under the License.
+
+*/
+
+pragma solidity ^0.5.5;
+
+import "../interfaces/IAuthorizable.sol";
+
+
+contract MAuthorizable is
+    IAuthorizable
+{
+    // Event logged when a new address is authorized.
+    event AuthorizedAddressAdded(
+        address indexed target,
+        address indexed caller
+    );
+
+    // Event logged when a currently authorized address is unauthorized.
+    event AuthorizedAddressRemoved(
+        address indexed target,
+        address indexed caller
+    );
+
+    /// @dev Only authorized addresses can invoke functions with this modifier.
+    modifier onlyAuthorized { revert(); _; }
+}

contracts/asset-proxy/contracts/test/TestERC20Bridge.sol

@@ -1,108 +0,0 @@
-/*
-
-  Copyright 2019 ZeroEx Intl.
-
-  Licensed under the Apache License, Version 2.0 (the "License");
-  you may not use this file except in compliance with the License.
-  You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-  Unless required by applicable law or agreed to in writing, software
-  distributed under the License is distributed on an "AS IS" BASIS,
-  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-  See the License for the specific language governing permissions and
-  limitations under the License.
-
-*/
-
-pragma solidity ^0.5.9;
-pragma experimental ABIEncoderV2;
-
-import "../src/interfaces/IERC20Bridge.sol";
-
-
-/// @dev Test bridge token
-contract TestERC20BridgeToken {
-    mapping (address => uint256) private _balances;
-
-    function addBalance(address owner, int256 amount)
-        external
-    {
-        setBalance(owner, uint256(int256(balanceOf(owner)) + amount));
-    }
-
-    function setBalance(address owner, uint256 balance)
-        public
-    {
-        _balances[owner] = balance;
-    }
-
-    function balanceOf(address owner)
-        public
-        view
-        returns (uint256)
-    {
-        return _balances[owner];
-    }
-}
-
-
-/// @dev Test bridge contract.
-contract TestERC20Bridge is
-    IERC20Bridge
-{
-    TestERC20BridgeToken public testToken;
-
-    event BridgeWithdrawTo(
-        address tokenAddress,
-        address from,
-        address to,
-        uint256 amount,
-        bytes bridgeData
-    );
-
-    constructor() public {
-        testToken = new TestERC20BridgeToken();
-    }
-
-    function setTestTokenBalance(address owner, uint256 balance)
-        external
-    {
-        testToken.setBalance(owner, balance);
-    }
-
-    function withdrawTo(
-        address tokenAddress,
-        address from,
-        address to,
-        uint256 amount,
-        bytes calldata bridgeData
-    )
-        external
-        returns (bytes4)
-    {
-        emit BridgeWithdrawTo(
-            tokenAddress,
-            from,
-            to,
-            amount,
-            bridgeData
-        );
-        // Unpack the bridgeData.
-        (
-            int256 transferAmount,
-            bytes memory revertData,
-            bytes memory returnData
-        ) = abi.decode(bridgeData, (int256, bytes, bytes));
-
-        // If `revertData` is set, revert.
-        if (revertData.length != 0) {
-            assembly { revert(add(revertData, 0x20), mload(revertData)) }
-        }
-        // Increase `to`'s balance by `transferAmount`.
-        TestERC20BridgeToken(tokenAddress).addBalance(to, transferAmount);
-        // Return `returnData`.
-        assembly { return(add(returnData, 0x20), mload(returnData)) }
-    }
-}

contracts/asset-proxy/contracts/test/TestEth2DaiBridge.sol

@@ -1,202 +0,0 @@
-/*
-
-  Copyright 2019 ZeroEx Intl.
-
-  Licensed under the Apache License, Version 2.0 (the "License");
-  you may not use this file except in compliance with the License.
-  You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-  Unless required by applicable law or agreed to in writing, software
-  distributed under the License is distributed on an "AS IS" BASIS,
-  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-  See the License for the specific language governing permissions and
-  limitations under the License.
-
-*/
-
-pragma solidity ^0.5.9;
-pragma experimental ABIEncoderV2;
-
-import "@0x/contracts-erc20/contracts/src/interfaces/IERC20Token.sol";
-import "../src/bridges/Eth2DaiBridge.sol";
-import "../src/interfaces/IEth2Dai.sol";
-
-
-// solhint-disable no-simple-event-func-name
-contract TestEvents {
-
-    event TokenTransfer(
-        address token,
-        address from,
-        address to,
-        uint256 amount
-    );
-
-    event TokenApprove(
-        address token,
-        address spender,
-        uint256 allowance
-    );
-
-    function raiseTokenTransfer(
-        address from,
-        address to,
-        uint256 amount
-    )
-        external
-    {
-        emit TokenTransfer(
-            msg.sender,
-            from,
-            to,
-            amount
-        );
-    }
-
-    function raiseTokenApprove(address spender, uint256 allowance)
-        external
-    {
-        emit TokenApprove(msg.sender, spender, allowance);
-    }
-}
-
-
-/// @dev A minimalist ERC20 token.
-contract TestToken {
-
-    mapping (address => uint256) public balances;
-    string private _nextTransferRevertReason;
-    bytes private _nextTransferReturnData;
-
-    /// @dev Just calls `raiseTokenTransfer()` on the caller.
-    function transfer(address to, uint256 amount)
-        external
-        returns (bool)
-    {
-        TestEvents(msg.sender).raiseTokenTransfer(msg.sender, to, amount);
-        if (bytes(_nextTransferRevertReason).length != 0) {
-            revert(_nextTransferRevertReason);
-        }
-        bytes memory returnData = _nextTransferReturnData;
-        assembly { return(add(returnData, 0x20), mload(returnData)) }
-    }
-
-    /// @dev Set the balance for `owner`.
-    function setBalance(address owner, uint256 balance)
-        external
-    {
-        balances[owner] = balance;
-    }
-
-    /// @dev Set the behavior of the `transfer()` call.
-    function setTransferBehavior(
-        string calldata revertReason,
-        bytes calldata returnData
-    )
-        external
-    {
-        _nextTransferRevertReason = revertReason;
-        _nextTransferReturnData = returnData;
-    }
-
-    /// @dev Just calls `raiseTokenApprove()` on the caller.
-    function approve(address spender, uint256 allowance)
-        external
-        returns (bool)
-    {
-        TestEvents(msg.sender).raiseTokenApprove(spender, allowance);
-        return true;
-    }
-
-    /// @dev Retrieve the balance for `owner`.
-    function balanceOf(address owner)
-        external
-        view
-        returns (uint256)
-    {
-        return balances[owner];
-    }
-}
-
-
-/// @dev Eth2DaiBridge overridden to mock tokens and
-///      implement IEth2Dai.
-contract TestEth2DaiBridge is
-    TestEvents,
-    IEth2Dai,
-    Eth2DaiBridge
-{
-    event SellAllAmount(
-        address sellToken,
-        uint256 sellTokenAmount,
-        address buyToken,
-        uint256 minimumFillAmount
-    );
-
-    mapping (address => TestToken)  public testTokens;
-    string private _nextRevertReason;
-    uint256 private _nextFillAmount;
-
-    /// @dev Create a token and set this contract's balance.
-    function createToken(uint256 balance)
-        external
-        returns (address tokenAddress)
-    {
-        TestToken token = new TestToken();
-        testTokens[address(token)] = token;
-        token.setBalance(address(this), balance);
-        return address(token);
-    }
-
-    /// @dev Set the behavior for `IEth2Dai.sellAllAmount()`.
-    function setFillBehavior(string calldata revertReason, uint256 fillAmount)
-        external
-    {
-        _nextRevertReason = revertReason;
-        _nextFillAmount = fillAmount;
-    }
-
-    /// @dev Set the behavior of a token's `transfer()`.
-    function setTransferBehavior(
-        address tokenAddress,
-        string calldata revertReason,
-        bytes calldata returnData
-    )
-        external
-    {
-        testTokens[tokenAddress].setTransferBehavior(revertReason, returnData);
-    }
-
-    /// @dev Implementation of `IEth2Dai.sellAllAmount()`
-    function sellAllAmount(
-        address sellTokenAddress,
-        uint256 sellTokenAmount,
-        address buyTokenAddress,
-        uint256 minimumFillAmount
-    )
-        external
-        returns (uint256 fillAmount)
-    {
-        emit SellAllAmount(
-            sellTokenAddress,
-            sellTokenAmount,
-            buyTokenAddress,
-            minimumFillAmount
-        );
-        if (bytes(_nextRevertReason).length != 0) {
-            revert(_nextRevertReason);
-        }
-        return _nextFillAmount;
-    }
-
-    // @dev This contract will double as the Eth2Dai contract.
-    function _getEth2DaiContract()
-        internal
-        view
-        returns (IEth2Dai)
-    {
-        return IEth2Dai(address(this));
-    }
-}

contracts/asset-proxy/contracts/test/TestStaticCallTarget.sol

@@ -1,6 +1,6 @@
 /*
 
-  Copyright 2019 ZeroEx Intl.
+  Copyright 2018 ZeroEx Intl.
 
   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.

contracts/asset-proxy/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@0x/contracts-asset-proxy",
-    "version": "2.3.0-beta.0",
+    "version": "2.2.6",
     "engines": {
         "node": ">=6.12"
     },
@@ -12,7 +12,7 @@
     "scripts": {
         "build": "yarn pre_build && tsc -b",
         "build:ci": "yarn build",
-        "pre_build": "run-s compile contracts:gen generate_contract_wrappers",
+        "pre_build": "run-s compile generate_contract_wrappers",
         "test": "yarn run_mocha",
         "rebuild_and_test": "run-s build test",
         "test:coverage": "SOLIDITY_COVERAGE=true run-s build run_mocha coverage:report:text coverage:report:lcov",
@@ -22,7 +22,7 @@
         "compile": "sol-compiler",
         "watch": "sol-compiler -w",
         "clean": "shx rm -rf lib generated-artifacts generated-wrappers",
-        "generate_contract_wrappers": "abi-gen --abis  ${npm_package_config_abis} --output generated-wrappers --backend ethers",
+        "generate_contract_wrappers": "abi-gen --abis  ${npm_package_config_abis} --template ../../node_modules/@0x/abi-gen-templates/contract.handlebars --partials '../../node_modules/@0x/abi-gen-templates/partials/**/*.handlebars' --output generated-wrappers --backend ethers",
         "lint": "tslint --format stylish --project . --exclude ./generated-wrappers/**/* --exclude ./generated-artifacts/**/* --exclude **/lib/**/* && yarn lint-contracts",
         "fix": "tslint --fix --format stylish --project . --exclude ./generated-wrappers/**/* --exclude ./generated-artifacts/**/* --exclude **/lib/**/* && yarn lint-contracts",
         "coverage:report:text": "istanbul report text",
@@ -31,11 +31,10 @@
         "coverage:report:lcov": "istanbul report lcov",
         "test:circleci": "yarn test",
         "contracts:gen": "contracts-gen",
-        "lint-contracts": "solhint -c ../.solhint.json contracts/**/**/**/**/*.sol",
-        "compile:truffle": "truffle compile"
+        "lint-contracts": "solhint -c ../.solhint.json contracts/**/**/**/**/*.sol"
     },
     "config": {
-        "abis": "./generated-artifacts/@(ERC1155Proxy|ERC20BridgeProxy|ERC20Proxy|ERC721Proxy|Eth2DaiBridge|IAssetData|IAssetProxy|IAssetProxyDispatcher|IAuthorizable|IERC20Bridge|IEth2Dai|IWallet|MixinAssetProxyDispatcher|MixinAuthorizable|MultiAssetProxy|Ownable|StaticCallProxy|TestERC20Bridge|TestEth2DaiBridge|TestStaticCallTarget).json",
+        "abis": "./generated-artifacts/@(ERC1155Proxy|ERC20Proxy|ERC721Proxy|IAssetData|IAssetProxy|IAuthorizable|MixinAuthorizable|MultiAssetProxy|StaticCallProxy|TestStaticCallTarget).json",
         "abis:comment": "This list is auto-generated by contracts-gen. Don't edit manually."
     },
     "repository": {
@@ -48,11 +47,11 @@
     },
     "homepage": "https://github.com/0xProject/0x-monorepo/contracts/protocol/README.md",
     "devDependencies": {
-        "@0x/abi-gen": "^4.3.0-beta.0",
-        "@0x/contracts-gen": "^1.1.0-beta.0",
-        "@0x/contracts-test-utils": "^3.2.0-beta.0",
-        "@0x/dev-utils": "^2.4.0-beta.0",
-        "@0x/sol-compiler": "^3.2.0-beta.0",
+        "@0x/abi-gen": "^4.1.1",
+        "@0x/contracts-gen": "^1.0.13",
+        "@0x/contracts-test-utils": "^3.1.14",
+        "@0x/dev-utils": "^2.3.1",
+        "@0x/sol-compiler": "^3.1.13",
         "@0x/tslint-config": "^3.0.1",
         "@types/lodash": "4.14.104",
         "@types/mocha": "^5.2.7",
@@ -66,22 +65,21 @@
         "npm-run-all": "^4.1.2",
         "shx": "^0.2.2",
         "solhint": "^1.4.1",
-        "truffle": "^5.0.32",
         "tslint": "5.11.0",
         "typescript": "3.0.1"
     },
     "dependencies": {
-        "@0x/base-contract": "^5.5.0-beta.0",
-        "@0x/contracts-erc1155": "^1.2.0-beta.0",
-        "@0x/contracts-erc20": "^2.3.0-beta.0",
-        "@0x/contracts-erc721": "^2.2.0-beta.0",
-        "@0x/contracts-utils": "^3.3.0-beta.0",
-        "@0x/order-utils": "^8.5.0-beta.0",
-        "@0x/types": "^2.5.0-beta.0",
-        "@0x/typescript-typings": "^4.4.0-beta.0",
-        "@0x/utils": "^4.6.0-beta.0",
-        "@0x/web3-wrapper": "^6.1.0-beta.0",
-        "ethereum-types": "^2.2.0-beta.0",
+        "@0x/base-contract": "^5.3.2",
+        "@0x/contracts-erc1155": "^1.1.13",
+        "@0x/contracts-erc20": "^2.2.12",
+        "@0x/contracts-erc721": "^2.1.13",
+        "@0x/contracts-utils": "^3.2.2",
+        "@0x/order-utils": "^8.3.0",
+        "@0x/types": "^2.4.1",
+        "@0x/typescript-typings": "^4.2.4",
+        "@0x/utils": "^4.5.0",
+        "@0x/web3-wrapper": "^6.0.11",
+        "ethereum-types": "^2.1.4",
         "ethereumjs-util": "^5.1.1",
         "lodash": "^4.17.11"
     },

contracts/asset-proxy/src/artifacts.ts

@@ -6,44 +6,24 @@
 import { ContractArtifact } from 'ethereum-types';
 
 import * as ERC1155Proxy from '../generated-artifacts/ERC1155Proxy.json';
-import * as ERC20BridgeProxy from '../generated-artifacts/ERC20BridgeProxy.json';
 import * as ERC20Proxy from '../generated-artifacts/ERC20Proxy.json';
 import * as ERC721Proxy from '../generated-artifacts/ERC721Proxy.json';
-import * as Eth2DaiBridge from '../generated-artifacts/Eth2DaiBridge.json';
 import * as IAssetData from '../generated-artifacts/IAssetData.json';
 import * as IAssetProxy from '../generated-artifacts/IAssetProxy.json';
-import * as IAssetProxyDispatcher from '../generated-artifacts/IAssetProxyDispatcher.json';
 import * as IAuthorizable from '../generated-artifacts/IAuthorizable.json';
-import * as IERC20Bridge from '../generated-artifacts/IERC20Bridge.json';
-import * as IEth2Dai from '../generated-artifacts/IEth2Dai.json';
-import * as IWallet from '../generated-artifacts/IWallet.json';
-import * as MixinAssetProxyDispatcher from '../generated-artifacts/MixinAssetProxyDispatcher.json';
 import * as MixinAuthorizable from '../generated-artifacts/MixinAuthorizable.json';
 import * as MultiAssetProxy from '../generated-artifacts/MultiAssetProxy.json';
-import * as Ownable from '../generated-artifacts/Ownable.json';
 import * as StaticCallProxy from '../generated-artifacts/StaticCallProxy.json';
-import * as TestERC20Bridge from '../generated-artifacts/TestERC20Bridge.json';
-import * as TestEth2DaiBridge from '../generated-artifacts/TestEth2DaiBridge.json';
 import * as TestStaticCallTarget from '../generated-artifacts/TestStaticCallTarget.json';
 export const artifacts = {
-    MixinAssetProxyDispatcher: MixinAssetProxyDispatcher as ContractArtifact,
-    MixinAuthorizable: MixinAuthorizable as ContractArtifact,
-    Ownable: Ownable as ContractArtifact,
     ERC1155Proxy: ERC1155Proxy as ContractArtifact,
-    ERC20BridgeProxy: ERC20BridgeProxy as ContractArtifact,
     ERC20Proxy: ERC20Proxy as ContractArtifact,
     ERC721Proxy: ERC721Proxy as ContractArtifact,
+    MixinAuthorizable: MixinAuthorizable as ContractArtifact,
     MultiAssetProxy: MultiAssetProxy as ContractArtifact,
     StaticCallProxy: StaticCallProxy as ContractArtifact,
-    Eth2DaiBridge: Eth2DaiBridge as ContractArtifact,
     IAssetData: IAssetData as ContractArtifact,
     IAssetProxy: IAssetProxy as ContractArtifact,
-    IAssetProxyDispatcher: IAssetProxyDispatcher as ContractArtifact,
     IAuthorizable: IAuthorizable as ContractArtifact,
-    IERC20Bridge: IERC20Bridge as ContractArtifact,
-    IEth2Dai: IEth2Dai as ContractArtifact,
-    IWallet: IWallet as ContractArtifact,
-    TestERC20Bridge: TestERC20Bridge as ContractArtifact,
-    TestEth2DaiBridge: TestEth2DaiBridge as ContractArtifact,
     TestStaticCallTarget: TestStaticCallTarget as ContractArtifact,
 };

contracts/asset-proxy/src/wrappers.ts

@@ -4,22 +4,12 @@
  * -----------------------------------------------------------------------------
  */
 export * from '../generated-wrappers/erc1155_proxy';
-export * from '../generated-wrappers/erc20_bridge_proxy';
 export * from '../generated-wrappers/erc20_proxy';
 export * from '../generated-wrappers/erc721_proxy';
-export * from '../generated-wrappers/eth2_dai_bridge';
 export * from '../generated-wrappers/i_asset_data';
 export * from '../generated-wrappers/i_asset_proxy';
-export * from '../generated-wrappers/i_asset_proxy_dispatcher';
 export * from '../generated-wrappers/i_authorizable';
-export * from '../generated-wrappers/i_erc20_bridge';
-export * from '../generated-wrappers/i_eth2_dai';
-export * from '../generated-wrappers/i_wallet';
-export * from '../generated-wrappers/mixin_asset_proxy_dispatcher';
 export * from '../generated-wrappers/mixin_authorizable';
 export * from '../generated-wrappers/multi_asset_proxy';
-export * from '../generated-wrappers/ownable';
 export * from '../generated-wrappers/static_call_proxy';
-export * from '../generated-wrappers/test_erc20_bridge';
-export * from '../generated-wrappers/test_eth2_dai_bridge';
 export * from '../generated-wrappers/test_static_call_target';

contracts/asset-proxy/test/authorizable.ts

@@ -27,11 +27,9 @@ describe('Authorizable', () => {
     before(async () => {
         await blockchainLifecycle.startAsync();
     });
-
     after(async () => {
         await blockchainLifecycle.revertAsync();
     });
-
     before(async () => {
         const accounts = await web3Wrapper.getAvailableAddressesAsync();
         [owner, address, notOwner] = _.slice(accounts, 0, 3);
@@ -42,23 +40,19 @@ describe('Authorizable', () => {
             artifacts,
         );
     });
-
     beforeEach(async () => {
         await blockchainLifecycle.startAsync();
     });
-
     afterEach(async () => {
         await blockchainLifecycle.revertAsync();
     });
-
     describe('addAuthorizedAddress', () => {
-        it('should revert if not called by owner', async () => {
-            await expectTransactionFailedAsync(
+        it('should throw if not called by owner', async () => {
+            return expectTransactionFailedAsync(
                 authorizable.addAuthorizedAddress.sendTransactionAsync(notOwner, { from: notOwner }),
                 RevertReason.OnlyContractOwner,
             );
         });
-
         it('should allow owner to add an authorized address', async () => {
             await authorizable.addAuthorizedAddress.awaitTransactionSuccessAsync(
                 address,
@@ -68,8 +62,7 @@ describe('Authorizable', () => {
             const isAuthorized = await authorizable.authorized.callAsync(address);
             expect(isAuthorized).to.be.true();
         });
-
-        it('should revert if owner attempts to authorize a duplicate address', async () => {
+        it('should throw if owner attempts to authorize a duplicate address', async () => {
             await authorizable.addAuthorizedAddress.awaitTransactionSuccessAsync(
                 address,
                 { from: owner },
@@ -83,14 +76,16 @@ describe('Authorizable', () => {
     });
 
     describe('removeAuthorizedAddress', () => {
-        it('should revert if not called by owner', async () => {
+        it('should throw if not called by owner', async () => {
             await authorizable.addAuthorizedAddress.awaitTransactionSuccessAsync(
                 address,
                 { from: owner },
                 constants.AWAIT_TRANSACTION_MINED_MS,
             );
-            await expectTransactionFailedAsync(
-                authorizable.removeAuthorizedAddress.sendTransactionAsync(address, { from: notOwner }),
+            return expectTransactionFailedAsync(
+                authorizable.removeAuthorizedAddress.sendTransactionAsync(address, {
+                    from: notOwner,
+                }),
                 RevertReason.OnlyContractOwner,
             );
         });
@@ -110,7 +105,7 @@ describe('Authorizable', () => {
             expect(isAuthorized).to.be.false();
         });
 
-        it('should revert if owner attempts to remove an address that is not authorized', async () => {
+        it('should throw if owner attempts to remove an address that is not authorized', async () => {
             return expectTransactionFailedAsync(
                 authorizable.removeAuthorizedAddress.sendTransactionAsync(address, {
                     from: owner,
@@ -121,22 +116,21 @@ describe('Authorizable', () => {
     });
 
     describe('removeAuthorizedAddressAtIndex', () => {
-        it('should revert if not called by owner', async () => {
+        it('should throw if not called by owner', async () => {
             await authorizable.addAuthorizedAddress.awaitTransactionSuccessAsync(
                 address,
                 { from: owner },
                 constants.AWAIT_TRANSACTION_MINED_MS,
             );
             const index = new BigNumber(0);
-            await expectTransactionFailedAsync(
+            return expectTransactionFailedAsync(
                 authorizable.removeAuthorizedAddressAtIndex.sendTransactionAsync(address, index, {
                     from: notOwner,
                 }),
                 RevertReason.OnlyContractOwner,
             );
         });
-
-        it('should revert if index is >= authorities.length', async () => {
+        it('should throw if index is >= authorities.length', async () => {
             await authorizable.addAuthorizedAddress.awaitTransactionSuccessAsync(
                 address,
                 { from: owner },
@@ -150,8 +144,7 @@ describe('Authorizable', () => {
                 RevertReason.IndexOutOfBounds,
             );
         });
-
-        it('should revert if owner attempts to remove an address that is not authorized', async () => {
+        it('should throw if owner attempts to remove an address that is not authorized', async () => {
             const index = new BigNumber(0);
             return expectTransactionFailedAsync(
                 authorizable.removeAuthorizedAddressAtIndex.sendTransactionAsync(address, index, {
@@ -160,8 +153,7 @@ describe('Authorizable', () => {
                 RevertReason.TargetNotAuthorized,
             );
         });
-
-        it('should revert if address at index does not match target', async () => {
+        it('should throw if address at index does not match target', async () => {
             const address1 = address;
             const address2 = notOwner;
             await authorizable.addAuthorizedAddress.awaitTransactionSuccessAsync(
@@ -182,7 +174,6 @@ describe('Authorizable', () => {
                 RevertReason.AuthorizedAddressMismatch,
             );
         });
-
         it('should allow owner to remove an authorized address', async () => {
             await authorizable.addAuthorizedAddress.awaitTransactionSuccessAsync(
                 address,

contracts/asset-proxy/test/erc1155_proxy.ts

@@ -17,7 +17,7 @@ import {
 import { BlockchainLifecycle } from '@0x/dev-utils';
 import { assetDataUtils } from '@0x/order-utils';
 import { AssetProxyId, RevertReason } from '@0x/types';
-import { BigNumber, SafeMathRevertErrors } from '@0x/utils';
+import { BigNumber } from '@0x/utils';
 import * as chai from 'chai';
 import { LogWithDecodedArgs } from 'ethereum-types';
 import * as ethUtil from 'ethereumjs-util';
@@ -1748,14 +1748,9 @@ describe('ERC1155Proxy', () => {
                 nftNotOwnerBalance,
             ];
             await erc1155Wrapper.assertBalancesAsync(tokenHolders, tokensToTransfer, expectedInitialBalances);
-            const expectedError = new SafeMathRevertErrors.Uint256BinOpError(
-                SafeMathRevertErrors.BinOpErrorCodes.MultiplicationOverflow,
-                maxUintValue,
-                valueMultiplier,
-            );
             // execute transfer
             // note - this will overflow because we are trying to transfer `maxUintValue * 2` of the 2nd token
-            await expect(
+            await expectTransactionFailedAsync(
                 erc1155ProxyWrapper.transferFromAsync(
                     spender,
                     receiver,
@@ -1766,7 +1761,8 @@ describe('ERC1155Proxy', () => {
                     receiverCallbackData,
                     authorized,
                 ),
-            ).to.revertWith(expectedError);
+                RevertReason.Uint256Overflow,
+            );
         });
         it('should revert if transferring > 1 instances of a non-fungible token (valueMultiplier field >1)', async () => {
             // setup test parameters
@@ -1836,23 +1832,20 @@ describe('ERC1155Proxy', () => {
             // check balances before transfer
             const expectedInitialBalances = [spenderInitialFungibleBalance, receiverInitialFungibleBalance];
             await erc1155Wrapper.assertBalancesAsync(tokenHolders, tokensToTransfer, expectedInitialBalances);
-            const expectedError = new SafeMathRevertErrors.Uint256BinOpError(
-                SafeMathRevertErrors.BinOpErrorCodes.SubtractionUnderflow,
-                spenderInitialFungibleBalance,
-                valuesToTransfer[0].times(valueMultiplier),
-            );
             // execute transfer
-            const tx = erc1155ProxyWrapper.transferFromAsync(
-                spender,
-                receiver,
-                erc1155Contract.address,
-                tokensToTransfer,
-                valuesToTransfer,
-                valueMultiplier,
-                receiverCallbackData,
-                authorized,
+            await expectTransactionFailedAsync(
+                erc1155ProxyWrapper.transferFromAsync(
+                    spender,
+                    receiver,
+                    erc1155Contract.address,
+                    tokensToTransfer,
+                    valuesToTransfer,
+                    valueMultiplier,
+                    receiverCallbackData,
+                    authorized,
+                ),
+                RevertReason.Uint256Underflow,
             );
-            return expect(tx).to.revertWith(expectedError);
         });
         it('should revert if sender allowance is insufficient', async () => {
             // dremove allowance for ERC1155 proxy

contracts/asset-proxy/test/erc20bridge_proxy.ts

@@ -1,299 +0,0 @@
-import {
-    blockchainTests,
-    constants,
-    expect,
-    getRandomInteger,
-    hexLeftPad,
-    hexRightPad,
-    hexSlice,
-    Numberish,
-    randomAddress,
-} from '@0x/contracts-test-utils';
-import { AssetProxyId } from '@0x/types';
-import { AbiEncoder, AuthorizableRevertErrors, BigNumber, StringRevertError } from '@0x/utils';
-import { DecodedLogs } from 'ethereum-types';
-import * as _ from 'lodash';
-
-import {
-    artifacts,
-    ERC20BridgeProxyContract,
-    TestERC20BridgeBridgeWithdrawToEventArgs,
-    TestERC20BridgeContract,
-} from '../src';
-
-blockchainTests.resets('ERC20BridgeProxy unit tests', env => {
-    const PROXY_ID = AssetProxyId.ERC20Bridge;
-    const BRIDGE_SUCCESS_RETURN_DATA = hexRightPad(PROXY_ID);
-    let owner: string;
-    let badCaller: string;
-    let assetProxy: ERC20BridgeProxyContract;
-    let bridgeContract: TestERC20BridgeContract;
-    let testTokenAddress: string;
-
-    before(async () => {
-        [owner, badCaller] = await env.getAccountAddressesAsync();
-        assetProxy = await ERC20BridgeProxyContract.deployFrom0xArtifactAsync(
-            artifacts.ERC20BridgeProxy,
-            env.provider,
-            env.txDefaults,
-            artifacts,
-        );
-        bridgeContract = await TestERC20BridgeContract.deployFrom0xArtifactAsync(
-            artifacts.TestERC20Bridge,
-            env.provider,
-            env.txDefaults,
-            artifacts,
-        );
-        testTokenAddress = await bridgeContract.testToken.callAsync();
-        await assetProxy.addAuthorizedAddress.awaitTransactionSuccessAsync(owner);
-    });
-
-    interface AssetDataOpts {
-        tokenAddress: string;
-        bridgeAddress: string;
-        bridgeData: BridgeDataOpts;
-    }
-
-    interface BridgeDataOpts {
-        transferAmount: Numberish;
-        revertError?: string;
-        returnData: string;
-    }
-
-    function createAssetData(opts?: Partial<AssetDataOpts>): AssetDataOpts {
-        return _.merge(
-            {
-                tokenAddress: testTokenAddress,
-                bridgeAddress: bridgeContract.address,
-                bridgeData: createBridgeData(),
-            },
-            opts,
-        );
-    }
-
-    function createBridgeData(opts?: Partial<BridgeDataOpts>): BridgeDataOpts {
-        return _.merge(
-            {
-                transferAmount: constants.ZERO_AMOUNT,
-                returnData: BRIDGE_SUCCESS_RETURN_DATA,
-            },
-            opts,
-        );
-    }
-
-    function encodeAssetData(opts: AssetDataOpts): string {
-        const encoder = AbiEncoder.createMethod('ERC20BridgeProxy', [
-            { name: 'tokenAddress', type: 'address' },
-            { name: 'bridgeAddress', type: 'address' },
-            { name: 'bridgeData', type: 'bytes' },
-        ]);
-        return encoder.encode([opts.tokenAddress, opts.bridgeAddress, encodeBridgeData(opts.bridgeData)]);
-    }
-
-    function encodeBridgeData(opts: BridgeDataOpts): string {
-        const encoder = AbiEncoder.create([
-            { name: 'transferAmount', type: 'int256' },
-            { name: 'revertData', type: 'bytes' },
-            { name: 'returnData', type: 'bytes' },
-        ]);
-        const revertErrorBytes =
-            opts.revertError !== undefined ? new StringRevertError(opts.revertError).encode() : '0x';
-        return encoder.encode([new BigNumber(opts.transferAmount), revertErrorBytes, opts.returnData]);
-    }
-
-    async function setTestTokenBalanceAsync(_owner: string, balance: Numberish): Promise<void> {
-        await bridgeContract.setTestTokenBalance.awaitTransactionSuccessAsync(_owner, new BigNumber(balance));
-    }
-
-    describe('transferFrom()', () => {
-        interface TransferFromOpts {
-            assetData: AssetDataOpts;
-            from: string;
-            to: string;
-            amount: Numberish;
-        }
-
-        function createTransferFromOpts(opts?: Partial<TransferFromOpts>): TransferFromOpts {
-            const transferAmount = _.get(opts, ['amount'], getRandomInteger(1, 100e18)) as BigNumber;
-            return _.merge(
-                {
-                    assetData: createAssetData({
-                        bridgeData: createBridgeData({
-                            transferAmount,
-                        }),
-                    }),
-                    from: randomAddress(),
-                    to: randomAddress(),
-                    amount: transferAmount,
-                },
-                opts,
-            );
-        }
-
-        async function transferFromAsync(opts?: Partial<TransferFromOpts>, caller?: string): Promise<DecodedLogs> {
-            const _opts = createTransferFromOpts(opts);
-            const { logs } = await assetProxy.transferFrom.awaitTransactionSuccessAsync(
-                encodeAssetData(_opts.assetData),
-                _opts.from,
-                _opts.to,
-                new BigNumber(_opts.amount),
-                { from: caller },
-            );
-            return (logs as any) as DecodedLogs;
-        }
-
-        it('succeeds if the bridge succeeds and balance increases by `amount`', async () => {
-            const tx = transferFromAsync();
-            return expect(tx).to.be.fulfilled('');
-        });
-
-        it('succeeds if balance increases more than `amount`', async () => {
-            const amount = getRandomInteger(1, 100e18);
-            const tx = transferFromAsync({
-                amount,
-                assetData: createAssetData({
-                    bridgeData: createBridgeData({
-                        transferAmount: amount.plus(1),
-                    }),
-                }),
-            });
-            return expect(tx).to.be.fulfilled('');
-        });
-
-        it('passes the correct arguments to the bridge contract', async () => {
-            const opts = createTransferFromOpts();
-            const logs = await transferFromAsync(opts);
-            expect(logs.length).to.eq(1);
-            const args = logs[0].args as TestERC20BridgeBridgeWithdrawToEventArgs;
-            expect(args.tokenAddress).to.eq(opts.assetData.tokenAddress);
-            expect(args.from).to.eq(opts.from);
-            expect(args.to).to.eq(opts.to);
-            expect(args.amount).to.bignumber.eq(opts.amount);
-            expect(args.bridgeData).to.eq(encodeBridgeData(opts.assetData.bridgeData));
-        });
-
-        it('fails if not called by an authorized address', async () => {
-            const tx = transferFromAsync({}, badCaller);
-            return expect(tx).to.revertWith(new AuthorizableRevertErrors.SenderNotAuthorizedError(badCaller));
-        });
-
-        it('fails if asset data is truncated', async () => {
-            const opts = createTransferFromOpts();
-            const truncatedAssetData = hexSlice(encodeAssetData(opts.assetData), 0, -1);
-            const tx = assetProxy.transferFrom.awaitTransactionSuccessAsync(
-                truncatedAssetData,
-                opts.from,
-                opts.to,
-                new BigNumber(opts.amount),
-            );
-            return expect(tx).to.be.rejected();
-        });
-
-        it('fails if bridge returns nothing', async () => {
-            const tx = transferFromAsync({
-                assetData: createAssetData({
-                    bridgeData: createBridgeData({
-                        returnData: '0x',
-                    }),
-                }),
-            });
-            // This will actually revert when the AP tries to decode the return
-            // value.
-            return expect(tx).to.be.rejected();
-        });
-
-        it('fails if bridge returns true', async () => {
-            const tx = transferFromAsync({
-                assetData: createAssetData({
-                    bridgeData: createBridgeData({
-                        returnData: hexLeftPad('0x1'),
-                    }),
-                }),
-            });
-            // This will actually revert when the AP tries to decode the return
-            // value.
-            return expect(tx).to.be.rejected();
-        });
-
-        it('fails if bridge returns 0x1', async () => {
-            const tx = transferFromAsync({
-                assetData: createAssetData({
-                    bridgeData: createBridgeData({
-                        returnData: hexRightPad('0x1'),
-                    }),
-                }),
-            });
-            return expect(tx).to.revertWith('BRIDGE_FAILED');
-        });
-
-        it('fails if bridge is an EOA', async () => {
-            const tx = transferFromAsync({
-                assetData: createAssetData({
-                    bridgeAddress: randomAddress(),
-                }),
-            });
-            // This will actually revert when the AP tries to decode the return
-            // value.
-            return expect(tx).to.be.rejected();
-        });
-
-        it('fails if bridge reverts', async () => {
-            const revertError = 'FOOBAR';
-            const tx = transferFromAsync({
-                assetData: createAssetData({
-                    bridgeData: createBridgeData({
-                        revertError,
-                    }),
-                }),
-            });
-            return expect(tx).to.revertWith(revertError);
-        });
-
-        it('fails if balance of `to` increases by less than `amount`', async () => {
-            const amount = getRandomInteger(1, 100e18);
-            const tx = transferFromAsync({
-                amount,
-                assetData: createAssetData({
-                    bridgeData: createBridgeData({
-                        transferAmount: amount.minus(1),
-                    }),
-                }),
-            });
-            return expect(tx).to.revertWith('BRIDGE_UNDERPAY');
-        });
-
-        it('fails if balance of `to` decreases', async () => {
-            const toAddress = randomAddress();
-            await setTestTokenBalanceAsync(toAddress, 1e18);
-            const tx = transferFromAsync({
-                to: toAddress,
-                assetData: createAssetData({
-                    bridgeData: createBridgeData({
-                        transferAmount: -1,
-                    }),
-                }),
-            });
-            return expect(tx).to.revertWith('BRIDGE_UNDERPAY');
-        });
-    });
-
-    describe('balanceOf()', () => {
-        it('retrieves the balance of the encoded token', async () => {
-            const _owner = randomAddress();
-            const balance = getRandomInteger(1, 100e18);
-            await bridgeContract.setTestTokenBalance.awaitTransactionSuccessAsync(_owner, balance);
-            const assetData = createAssetData({
-                tokenAddress: testTokenAddress,
-            });
-            const actualBalance = await assetProxy.balanceOf.callAsync(encodeAssetData(assetData), _owner);
-            expect(actualBalance).to.bignumber.eq(balance);
-        });
-    });
-
-    describe('getProxyId()', () => {
-        it('returns the correct proxy ID', async () => {
-            const proxyId = await assetProxy.getProxyId.callAsync();
-            expect(proxyId).to.eq(PROXY_ID);
-        });
-    });
-});

contracts/asset-proxy/test/eth2dai_bridge.ts

@@ -1,192 +0,0 @@
-import {
-    blockchainTests,
-    constants,
-    expect,
-    filterLogsToArguments,
-    getRandomInteger,
-    hexLeftPad,
-    hexRandom,
-    Numberish,
-    randomAddress,
-    TransactionHelper,
-} from '@0x/contracts-test-utils';
-import { AssetProxyId } from '@0x/types';
-import { BigNumber } from '@0x/utils';
-import { DecodedLogs } from 'ethereum-types';
-import * as _ from 'lodash';
-
-import {
-    artifacts,
-    TestEth2DaiBridgeContract,
-    TestEth2DaiBridgeEvents,
-    TestEth2DaiBridgeSellAllAmountEventArgs,
-    TestEth2DaiBridgeTokenApproveEventArgs,
-    TestEth2DaiBridgeTokenTransferEventArgs,
-} from '../src';
-
-blockchainTests.resets('Eth2DaiBridge unit tests', env => {
-    const txHelper = new TransactionHelper(env.web3Wrapper, artifacts);
-    let testContract: TestEth2DaiBridgeContract;
-
-    before(async () => {
-        testContract = await TestEth2DaiBridgeContract.deployFrom0xArtifactAsync(
-            artifacts.TestEth2DaiBridge,
-            env.provider,
-            env.txDefaults,
-            artifacts,
-        );
-    });
-
-    describe('isValidSignature()', () => {
-        it('returns success bytes', async () => {
-            const LEGACY_WALLET_MAGIC_VALUE = '0xb0671381';
-            const result = await testContract.isValidSignature.callAsync(hexRandom(), hexRandom(_.random(0, 32)));
-            expect(result).to.eq(LEGACY_WALLET_MAGIC_VALUE);
-        });
-    });
-
-    describe('withdrawTo()', () => {
-        interface WithdrawToOpts {
-            toTokenAddress?: string;
-            fromTokenAddress?: string;
-            toAddress: string;
-            amount: Numberish;
-            fromTokenBalance: Numberish;
-            revertReason: string;
-            fillAmount: Numberish;
-            toTokentransferRevertReason: string;
-            toTokenTransferReturnData: string;
-        }
-
-        interface WithdrawToResult {
-            opts: WithdrawToOpts;
-            result: string;
-            logs: DecodedLogs;
-        }
-
-        function createWithdrawToOpts(opts?: Partial<WithdrawToOpts>): WithdrawToOpts {
-            return {
-                toAddress: randomAddress(),
-                amount: getRandomInteger(1, 100e18),
-                revertReason: '',
-                fillAmount: getRandomInteger(1, 100e18),
-                fromTokenBalance: getRandomInteger(1, 100e18),
-                toTokentransferRevertReason: '',
-                toTokenTransferReturnData: hexLeftPad(1),
-                ...opts,
-            };
-        }
-
-        async function withdrawToAsync(opts?: Partial<WithdrawToOpts>): Promise<WithdrawToResult> {
-            const _opts = createWithdrawToOpts(opts);
-            // Set the fill behavior.
-            await testContract.setFillBehavior.awaitTransactionSuccessAsync(
-                _opts.revertReason,
-                new BigNumber(_opts.fillAmount),
-            );
-            // Create tokens and balances.
-            if (_opts.fromTokenAddress === undefined) {
-                [_opts.fromTokenAddress] = await txHelper.getResultAndReceiptAsync(
-                    testContract.createToken,
-                    new BigNumber(_opts.fromTokenBalance),
-                );
-            }
-            if (_opts.toTokenAddress === undefined) {
-                [_opts.toTokenAddress] = await txHelper.getResultAndReceiptAsync(
-                    testContract.createToken,
-                    constants.ZERO_AMOUNT,
-                );
-            }
-            // Set the transfer behavior of `toTokenAddress`.
-            await testContract.setTransferBehavior.awaitTransactionSuccessAsync(
-                _opts.toTokenAddress,
-                _opts.toTokentransferRevertReason,
-                _opts.toTokenTransferReturnData,
-            );
-            // Call withdrawTo().
-            const [result, { logs }] = await txHelper.getResultAndReceiptAsync(
-                testContract.withdrawTo,
-                // "to" token address
-                _opts.toTokenAddress,
-                // Random from address.
-                randomAddress(),
-                // To address.
-                _opts.toAddress,
-                new BigNumber(_opts.amount),
-                // ABI-encode the "from" token address as the bridge data.
-                hexLeftPad(_opts.fromTokenAddress as string),
-            );
-            return {
-                opts: _opts,
-                result,
-                logs: (logs as any) as DecodedLogs,
-            };
-        }
-
-        it('returns magic bytes on success', async () => {
-            const BRIDGE_SUCCESS_RETURN_DATA = AssetProxyId.ERC20Bridge;
-            const { result } = await withdrawToAsync();
-            expect(result).to.eq(BRIDGE_SUCCESS_RETURN_DATA);
-        });
-
-        it('calls `Eth2Dai.sellAllAmount()`', async () => {
-            const { opts, logs } = await withdrawToAsync();
-            const transfers = filterLogsToArguments<TestEth2DaiBridgeSellAllAmountEventArgs>(
-                logs,
-                TestEth2DaiBridgeEvents.SellAllAmount,
-            );
-            expect(transfers.length).to.eq(1);
-            expect(transfers[0].sellToken).to.eq(opts.fromTokenAddress);
-            expect(transfers[0].buyToken).to.eq(opts.toTokenAddress);
-            expect(transfers[0].sellTokenAmount).to.bignumber.eq(opts.fromTokenBalance);
-            expect(transfers[0].minimumFillAmount).to.bignumber.eq(opts.amount);
-        });
-
-        it('sets an unlimited allowance on the `fromTokenAddress` token', async () => {
-            const { opts, logs } = await withdrawToAsync();
-            const approvals = filterLogsToArguments<TestEth2DaiBridgeTokenApproveEventArgs>(
-                logs,
-                TestEth2DaiBridgeEvents.TokenApprove,
-            );
-            expect(approvals.length).to.eq(1);
-            expect(approvals[0].token).to.eq(opts.fromTokenAddress);
-            expect(approvals[0].spender).to.eq(testContract.address);
-            expect(approvals[0].allowance).to.bignumber.eq(constants.MAX_UINT256);
-        });
-
-        it('transfers filled amount to `to`', async () => {
-            const { opts, logs } = await withdrawToAsync();
-            const transfers = filterLogsToArguments<TestEth2DaiBridgeTokenTransferEventArgs>(
-                logs,
-                TestEth2DaiBridgeEvents.TokenTransfer,
-            );
-            expect(transfers.length).to.eq(1);
-            expect(transfers[0].token).to.eq(opts.toTokenAddress);
-            expect(transfers[0].from).to.eq(testContract.address);
-            expect(transfers[0].to).to.eq(opts.toAddress);
-            expect(transfers[0].amount).to.bignumber.eq(opts.fillAmount);
-        });
-
-        it('fails if `Eth2Dai.sellAllAmount()` reverts', async () => {
-            const opts = createWithdrawToOpts({ revertReason: 'FOOBAR' });
-            const tx = withdrawToAsync(opts);
-            return expect(tx).to.revertWith(opts.revertReason);
-        });
-
-        it('fails if `toTokenAddress.transfer()` reverts', async () => {
-            const opts = createWithdrawToOpts({ toTokentransferRevertReason: 'FOOBAR' });
-            const tx = withdrawToAsync(opts);
-            return expect(tx).to.revertWith(opts.toTokentransferRevertReason);
-        });
-
-        it('fails if `toTokenAddress.transfer()` returns falsey', async () => {
-            const opts = createWithdrawToOpts({ toTokenTransferReturnData: hexLeftPad(0) });
-            const tx = withdrawToAsync(opts);
-            return expect(tx).to.revertWith('ERC20_TRANSFER_FAILED');
-        });
-
-        it('succeeds if `toTokenAddress.transfer()` returns truthy', async () => {
-            await withdrawToAsync({ toTokenTransferReturnData: hexLeftPad(100) });
-        });
-    });
-});

contracts/asset-proxy/test/proxies.ts

@@ -679,14 +679,7 @@ describe('Asset Transfer Proxies', () => {
                 // Verify pre-condition
                 const ownerFromAsset = await erc721TokenA.ownerOf.callAsync(erc721AFromTokenId);
                 expect(ownerFromAsset).to.be.equal(fromAddress);
-                // Remove blanket transfer approval for fromAddress.
-                await erc721TokenA.setApprovalForAll.awaitTransactionSuccessAsync(
-                    erc721Proxy.address,
-                    false,
-                    { from: fromAddress },
-                    constants.AWAIT_TRANSACTION_MINED_MS,
-                );
-                // Remove token transfer approval for fromAddress.
+                // Remove transfer approval for fromAddress.
                 await erc721TokenA.approve.awaitTransactionSuccessAsync(
                     constants.NULL_ADDRESS,
                     erc721AFromTokenId,

contracts/asset-proxy/test/static_call_proxy.ts

@@ -43,13 +43,7 @@ describe('StaticCallProxy', () => {
             txDefaults,
             artifacts,
         );
-        staticCallProxy = new IAssetProxyContract(
-            staticCallProxyWithoutTransferFrom.address,
-            provider,
-            txDefaults,
-            {},
-            StaticCallProxyContract.deployedBytecode,
-        );
+        staticCallProxy = new IAssetProxyContract(staticCallProxyWithoutTransferFrom.address, provider, txDefaults);
         staticCallTarget = await TestStaticCallTargetContract.deployFrom0xArtifactAsync(
             artifacts.TestStaticCallTarget,
             provider,

contracts/asset-proxy/test/utils/erc1155_proxy_wrapper.ts

@@ -336,22 +336,6 @@ export class ERC1155ProxyWrapper {
         };
         return holdingsByOwner;
     }
-    /**
-     * @dev Set the approval for the proxy on behalf of `userAddress` .
-     * @param userAddress owner of ERC1155 tokens.
-     * @param contractAddress address of ERC1155 contract.
-     * @param isApproved Whether to approve the proxy for all or not.
-     */
-    public async setProxyAllowanceForAllAsync(
-        userAddress: string,
-        contractAddress: string,
-        isApproved: boolean,
-    ): Promise<void> {
-        this._validateProxyContractExistsOrThrow();
-        const tokenWrapper = this.getContractWrapper(contractAddress);
-        const operator = (this._proxyContract as ERC1155ProxyContract).address;
-        await tokenWrapper.setApprovalForAllAsync(userAddress, operator, isApproved);
-    }
     /**
      * @dev Checks if proxy is approved to transfer tokens on behalf of `userAddress`.
      * @param userAddress owner of ERC1155 tokens.

contracts/asset-proxy/test/utils/erc721_wrapper.ts

@@ -71,7 +71,7 @@ export class ERC721Wrapper {
                     }
                     this._initialTokenIdsByOwner[tokenOwnerAddress][dummyTokenContract.address].push(tokenId);
 
-                    await this.approveProxyForAllAsync(dummyTokenContract.address, tokenOwnerAddress, true);
+                    await this.approveProxyAsync(dummyTokenContract.address, tokenId);
                 }
             }
         }
@@ -86,17 +86,14 @@ export class ERC721Wrapper {
         const proxyAddress = (this._proxyContract as ERC721ProxyContract).address;
         await this.approveAsync(proxyAddress, tokenAddress, tokenId);
     }
-    public async approveProxyForAllAsync(
-        tokenAddress: string,
-        ownerAddress: string,
-        isApproved: boolean,
-    ): Promise<void> {
+    public async approveProxyForAllAsync(tokenAddress: string, tokenId: BigNumber, isApproved: boolean): Promise<void> {
         const tokenContract = this._getTokenContractFromAssetData(tokenAddress);
+        const tokenOwner = await this.ownerOfAsync(tokenAddress, tokenId);
         const proxyAddress = (this._proxyContract as ERC721ProxyContract).address;
         await tokenContract.setApprovalForAll.awaitTransactionSuccessAsync(
             proxyAddress,
             isApproved,
-            { from: ownerAddress },
+            { from: tokenOwner },
             constants.AWAIT_TRANSACTION_MINED_MS,
         );
     }

contracts/asset-proxy/truffle-config.js

@@ -1,96 +0,0 @@
-/**
- * Use this file to configure your truffle project. It's seeded with some
- * common settings for different networks and features like migrations,
- * compilation and testing. Uncomment the ones you need or modify
- * them to suit your project as necessary.
- *
- * More information about configuration can be found at:
- *
- * truffleframework.com/docs/advanced/configuration
- *
- * To deploy via Infura you'll need a wallet provider (like truffle-hdwallet-provider)
- * to sign your transactions before they're sent to a remote public node. Infura accounts
- * are available for free at: infura.io/register.
- *
- * You'll also need a mnemonic - the twelve word phrase the wallet uses to generate
- * public/private key pairs. If you're publishing your code to GitHub make sure you load this
- * phrase from a file you've .gitignored so it doesn't accidentally become public.
- *
- */
-
-// const HDWalletProvider = require('truffle-hdwallet-provider');
-// const infuraKey = "fj4jll3k.....";
-//
-// const fs = require('fs');
-// const mnemonic = fs.readFileSync(".secret").toString().trim();
-
-module.exports = {
-    /**
-     * Networks define how you connect to your ethereum client and let you set the
-     * defaults web3 uses to send transactions. If you don't specify one truffle
-     * will spin up a development blockchain for you on port 9545 when you
-     * run `develop` or `test`. You can ask a truffle command to use a specific
-     * network from the command line, e.g
-     *
-     * $ truffle test --network <network-name>
-     */
-
-    networks: {
-        // Useful for testing. The `development` name is special - truffle uses it by default
-        // if it's defined here and no other network is specified at the command line.
-        // You should run a client (like ganache-cli, geth or parity) in a separate terminal
-        // tab if you use this network and you must also set the `host`, `port` and `network_id`
-        // options below to some value.
-        //
-        // development: {
-        //  host: "127.0.0.1",     // Localhost (default: none)
-        //  port: 8545,            // Standard Ethereum port (default: none)
-        //  network_id: "*",       // Any network (default: none)
-        // },
-        // Another network with more advanced options...
-        // advanced: {
-        // port: 8777,             // Custom port
-        // network_id: 1342,       // Custom network
-        // gas: 8500000,           // Gas sent with each transaction (default: ~6700000)
-        // gasPrice: 20000000000,  // 20 gwei (in wei) (default: 100 gwei)
-        // from: <address>,        // Account to send txs from (default: accounts[0])
-        // websockets: true        // Enable EventEmitter interface for web3 (default: false)
-        // },
-        // Useful for deploying to a public network.
-        // NB: It's important to wrap the provider as a function.
-        // ropsten: {
-        // provider: () => new HDWalletProvider(mnemonic, `https://ropsten.infura.io/v3/YOUR-PROJECT-ID`),
-        // network_id: 3,       // Ropsten's id
-        // gas: 5500000,        // Ropsten has a lower block limit than mainnet
-        // confirmations: 2,    // # of confs to wait between deployments. (default: 0)
-        // timeoutBlocks: 200,  // # of blocks before a deployment times out  (minimum/default: 50)
-        // skipDryRun: true     // Skip dry run before migrations? (default: false for public nets )
-        // },
-        // Useful for private networks
-        // private: {
-        // provider: () => new HDWalletProvider(mnemonic, `https://network.io`),
-        // network_id: 2111,   // This network is yours, in the cloud.
-        // production: true    // Treats this network as if it was a public net. (default: false)
-        // }
-    },
-
-    // Set default mocha options here, use special reporters etc.
-    mocha: {
-        // timeout: 100000
-    },
-
-    // Configure your compilers
-    compilers: {
-        solc: {
-            version: '0.5.9',
-            settings: {
-                evmVersion: 'constantinople',
-                optimizer: {
-                    enabled: true,
-                    runs: 1000000,
-                    details: { yul: true, deduplicate: true, cse: true, constantOptimizer: true },
-                },
-            },
-        },
-    },
-};

contracts/asset-proxy/tsconfig.json

@@ -4,24 +4,14 @@
     "include": ["./src/**/*", "./test/**/*", "./generated-wrappers/**/*"],
     "files": [
         "generated-artifacts/ERC1155Proxy.json",
-        "generated-artifacts/ERC20BridgeProxy.json",
         "generated-artifacts/ERC20Proxy.json",
         "generated-artifacts/ERC721Proxy.json",
-        "generated-artifacts/Eth2DaiBridge.json",
         "generated-artifacts/IAssetData.json",
         "generated-artifacts/IAssetProxy.json",
-        "generated-artifacts/IAssetProxyDispatcher.json",
         "generated-artifacts/IAuthorizable.json",
-        "generated-artifacts/IERC20Bridge.json",
-        "generated-artifacts/IEth2Dai.json",
-        "generated-artifacts/IWallet.json",
-        "generated-artifacts/MixinAssetProxyDispatcher.json",
         "generated-artifacts/MixinAuthorizable.json",
         "generated-artifacts/MultiAssetProxy.json",
-        "generated-artifacts/Ownable.json",
         "generated-artifacts/StaticCallProxy.json",
-        "generated-artifacts/TestERC20Bridge.json",
-        "generated-artifacts/TestEth2DaiBridge.json",
         "generated-artifacts/TestStaticCallTarget.json"
     ],
     "exclude": ["./deploy/solc/solc_bin"]

contracts/coordinator/.solhintignore

@@ -1,2 +0,0 @@
-# solhint can't parse `abi.decode` syntax.
-contracts/src/MixinCoordinatorApprovalVerifier.sol

contracts/coordinator/CHANGELOG.json

@@ -1,68 +1,4 @@
 [
-    {
-        "version": "2.1.0-beta.0",
-        "changes": [
-            {
-                "note": "Add chainId to domain separator",
-                "pr": 1742
-            },
-            {
-                "note": "Inherit Exchange domain constants from `exchange-libs` to reduce code duplication",
-                "pr": 1742
-            },
-            {
-                "note": "Update domain separator",
-                "pr": 1742
-            },
-            {
-                "note": "Refactor contract to use new ITransactions interface",
-                "pr": 1753
-            },
-            {
-                "note": "Add verifyingContractIfExists arg to LibEIP712CoordinatorDomain constructor",
-                "pr": 1753
-            },
-            {
-                "note": "Remove LibZeroExTransaction contract",
-                "pr": 1753
-            },
-            {
-                "note": "Update tests for arbitrary fee tokens (ZEIP-28).",
-                "pr": 1819
-            },
-            {
-                "note": "Update for new `marketXOrders` consolidation.",
-                "pr": 2042
-            },
-            {
-                "note": "Use built in selectors instead of hard coded constants",
-                "pr": 2055
-            },
-            {
-                "note": "Compile and export all contracts, artifacts, and wrappers by default",
-                "pr": 2055
-            }
-        ],
-        "timestamp": 1570135330
-    },
-    {
-        "timestamp": 1568744790,
-        "version": "2.0.13",
-        "changes": [
-            {
-                "note": "Dependencies updated"
-            }
-        ]
-    },
-    {
-        "timestamp": 1567521715,
-        "version": "2.0.12",
-        "changes": [
-            {
-                "note": "Dependencies updated"
-            }
-        ]
-    },
     {
         "timestamp": 1566446343,
         "version": "2.0.11",

contracts/coordinator/CHANGELOG.md

@@ -5,27 +5,6 @@ Edit the package's CHANGELOG.json file only.
 
 CHANGELOG
 
-## v2.1.0-beta.0 - _October 3, 2019_
-
-    * Add chainId to domain separator (#1742)
-    * Inherit Exchange domain constants from `exchange-libs` to reduce code duplication (#1742)
-    * Update domain separator (#1742)
-    * Refactor contract to use new ITransactions interface (#1753)
-    * Add verifyingContractIfExists arg to LibEIP712CoordinatorDomain constructor (#1753)
-    * Remove LibZeroExTransaction contract (#1753)
-    * Update tests for arbitrary fee tokens (ZEIP-28). (#1819)
-    * Update for new `marketXOrders` consolidation. (#2042)
-    * Use built in selectors instead of hard coded constants (#2055)
-    * Compile and export all contracts, artifacts, and wrappers by default (#2055)
-
-## v2.0.13 - _September 17, 2019_
-
-    * Dependencies updated
-
-## v2.0.12 - _September 3, 2019_
-
-    * Dependencies updated
-
 ## v2.0.11 - _August 22, 2019_
 
     * Dependencies updated

contracts/coordinator/README.md

@@ -1,6 +1,6 @@
 ## Coordinator
 
-This package contains a contract that allows users to call arbitrary functions on the Exchange contract with permission from one or more Coordinators. Addresses of the deployed contracts can be found in this 0x [guide](https://0x.org/docs/guides/0x-cheat-sheet) or the [DEPLOYS](./DEPLOYS.json) file within this package.
+This package contains a contract that allows users to call arbitrary functions on the Exchange contract with permission from one or more Coordinators. Addresses of the deployed contracts can be found in the 0x [wiki](https://0xproject.com/wiki#Deployed-Addresses) or the [DEPLOYS](./DEPLOYS.json) file within this package.
 
 ## Installation
 
@@ -12,7 +12,7 @@ npm install @0x/contracts-coordinator --save
 
 ## Bug bounty
 
-A bug bounty for the 2.0.0 contracts is ongoing! Instructions can be found [here](https://0x.org/docs/guides/bug-bounty-program).
+A bug bounty for the 2.0.0 contracts is ongoing! Instructions can be found [here](https://0xproject.com/wiki#Bug-Bounty).
 
 ## Contributing
 

contracts/coordinator/compiler.json

@@ -21,5 +21,6 @@
                 ]
             }
         }
-    }
+    },
+    "contracts": ["src/Coordinator.sol", "src/registry/CoordinatorRegistry.sol"]
 }

contracts/coordinator/contracts/src/Coordinator.sol

@@ -1,6 +1,6 @@
 /*
 
-  Copyright 2019 ZeroEx Intl.
+  Copyright 2018 ZeroEx Intl.
 
   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
@@ -16,12 +16,10 @@
 
 */
 
-pragma solidity ^0.5.9;
-pragma experimental ABIEncoderV2;
+pragma solidity ^0.5.5;
+pragma experimental "ABIEncoderV2";
 
-import "@0x/contracts-exchange-libs/contracts/src/LibEIP712ExchangeDomain.sol";
 import "./libs/LibConstants.sol";
-import "./libs/LibEIP712CoordinatorDomain.sol";
 import "./MixinSignatureValidator.sol";
 import "./MixinCoordinatorApprovalVerifier.sol";
 import "./MixinCoordinatorCore.sol";
@@ -34,12 +32,8 @@ contract Coordinator is
     MixinCoordinatorApprovalVerifier,
     MixinCoordinatorCore
 {
-    /// @param exchange Address of the 0x Exchange contract.
-    /// @param chainId Chain ID of the network this contract is deployed on.
-    constructor (address exchange, uint256 chainId)
+    constructor (address _exchange)
         public
-        LibConstants(exchange)
-        LibEIP712CoordinatorDomain(chainId, address(0))
-        LibEIP712ExchangeDomain(chainId, exchange)
+        LibConstants(_exchange)
     {}
 }

contracts/coordinator/contracts/src/MixinCoordinatorApprovalVerifier.sol

@@ -1,6 +1,6 @@
 /*
 
-  Copyright 2019 ZeroEx Intl.
+  Copyright 2018 ZeroEx Intl.
 
   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
@@ -16,25 +16,26 @@
 
 */
 
-pragma solidity ^0.5.9;
-pragma experimental ABIEncoderV2;
+pragma solidity ^0.5.5;
+pragma experimental "ABIEncoderV2";
 
+import "@0x/contracts-exchange-libs/contracts/src/LibExchangeSelectors.sol";
 import "@0x/contracts-exchange-libs/contracts/src/LibOrder.sol";
-import "@0x/contracts-exchange-libs/contracts/src/LibZeroExTransaction.sol";
 import "@0x/contracts-utils/contracts/src/LibBytes.sol";
 import "@0x/contracts-utils/contracts/src/LibAddressArray.sol";
-import "@0x/contracts-exchange/contracts/src/interfaces/IExchange.sol";
 import "./libs/LibCoordinatorApproval.sol";
-import "./interfaces/ICoordinatorSignatureValidator.sol";
-import "./interfaces/ICoordinatorApprovalVerifier.sol";
+import "./libs/LibZeroExTransaction.sol";
+import "./mixins/MSignatureValidator.sol";
+import "./mixins/MCoordinatorApprovalVerifier.sol";
 
 
 // solhint-disable avoid-tx-origin
 contract MixinCoordinatorApprovalVerifier is
+    LibExchangeSelectors,
     LibCoordinatorApproval,
     LibZeroExTransaction,
-    ICoordinatorSignatureValidator,
-    ICoordinatorApprovalVerifier
+    MSignatureValidator,
+    MCoordinatorApprovalVerifier
 {
     using LibBytes for bytes;
     using LibAddressArray for address[];
@@ -62,7 +63,7 @@ contract MixinCoordinatorApprovalVerifier is
         // No approval is required for non-fill methods
         if (orders.length > 0) {
             // Revert if approval is invalid for transaction orders
-            _assertValidTransactionOrdersApproval(
+            assertValidTransactionOrdersApproval(
                 transaction,
                 orders,
                 txOrigin,
@@ -83,9 +84,9 @@ contract MixinCoordinatorApprovalVerifier is
     {
         bytes4 selector = data.readBytes4(0);
         if (
-            selector == IExchange(address(0)).fillOrder.selector ||
-            selector == IExchange(address(0)).fillOrderNoThrow.selector ||
-            selector == IExchange(address(0)).fillOrKillOrder.selector
+            selector == FILL_ORDER_SELECTOR ||
+            selector == FILL_ORDER_NO_THROW_SELECTOR ||
+            selector == FILL_OR_KILL_ORDER_SELECTOR
         ) {
             // Decode single order
             (LibOrder.Order memory order) = abi.decode(
@@ -95,11 +96,13 @@ contract MixinCoordinatorApprovalVerifier is
             orders = new LibOrder.Order[](1);
             orders[0] = order;
         } else if (
-            selector == IExchange(address(0)).batchFillOrders.selector ||
-            selector == IExchange(address(0)).batchFillOrdersNoThrow.selector ||
-            selector == IExchange(address(0)).batchFillOrKillOrders.selector ||
-            selector == IExchange(address(0)).marketBuyOrders.selector ||
-            selector == IExchange(address(0)).marketSellOrders.selector
+            selector == BATCH_FILL_ORDERS_SELECTOR ||
+            selector == BATCH_FILL_ORDERS_NO_THROW_SELECTOR ||
+            selector == BATCH_FILL_OR_KILL_ORDERS_SELECTOR ||
+            selector == MARKET_BUY_ORDERS_SELECTOR ||
+            selector == MARKET_BUY_ORDERS_NO_THROW_SELECTOR ||
+            selector == MARKET_SELL_ORDERS_SELECTOR ||
+            selector == MARKET_SELL_ORDERS_NO_THROW_SELECTOR
         ) {
             // Decode all orders
             // solhint-disable indent
@@ -107,7 +110,7 @@ contract MixinCoordinatorApprovalVerifier is
                 data.slice(4, data.length),
                 (LibOrder.Order[])
             );
-        } else if (selector == IExchange(address(0)).matchOrders.selector) {
+        } else if (selector == MATCH_ORDERS_SELECTOR) {
             // Decode left and right orders
             (LibOrder.Order memory leftOrder, LibOrder.Order memory rightOrder) = abi.decode(
                 data.slice(4, data.length),
@@ -129,7 +132,7 @@ contract MixinCoordinatorApprovalVerifier is
     /// @param transactionSignature Proof that the transaction has been signed by the signer.
     /// @param approvalExpirationTimeSeconds Array of expiration times in seconds for which each corresponding approval signature expires.
     /// @param approvalSignatures Array of signatures that correspond to the feeRecipients of each order.
-    function _assertValidTransactionOrdersApproval(
+    function assertValidTransactionOrdersApproval(
         LibZeroExTransaction.ZeroExTransaction memory transaction,
         LibOrder.Order[] memory orders,
         address txOrigin,

contracts/coordinator/contracts/src/MixinCoordinatorCore.sol

@@ -1,6 +1,6 @@
 /*
 
-  Copyright 2019 ZeroEx Intl.
+  Copyright 2018 ZeroEx Intl.
 
   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
@@ -16,18 +16,18 @@
 
 */
 
-pragma solidity ^0.5.9;
-pragma experimental ABIEncoderV2;
+pragma solidity ^0.5.5;
+pragma experimental "ABIEncoderV2";
 
-import "@0x/contracts-exchange-libs/contracts/src/LibZeroExTransaction.sol";
+import "./libs/LibZeroExTransaction.sol";
 import "./libs/LibConstants.sol";
+import "./mixins/MCoordinatorApprovalVerifier.sol";
 import "./interfaces/ICoordinatorCore.sol";
-import "./interfaces/ICoordinatorApprovalVerifier.sol";
 
 
 contract MixinCoordinatorCore is
     LibConstants,
-    ICoordinatorApprovalVerifier,
+    MCoordinatorApprovalVerifier,
     ICoordinatorCore
 {
     /// @dev Executes a 0x transaction that has been signed by the feeRecipients that correspond to each order in the transaction's Exchange calldata.
@@ -55,6 +55,11 @@ contract MixinCoordinatorCore is
         );
 
         // Execute the transaction
-        EXCHANGE.executeTransaction(transaction, transactionSignature);
+        EXCHANGE.executeTransaction(
+            transaction.salt,
+            transaction.signerAddress,
+            transaction.data,
+            transactionSignature
+        );
     }
 }

contracts/coordinator/contracts/src/MixinSignatureValidator.sol

@@ -1,6 +1,6 @@
 /*
 
-  Copyright 2019 ZeroEx Intl.
+  Copyright 2018 ZeroEx Intl.
 
   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
@@ -16,14 +16,14 @@
 
 */
 
-pragma solidity ^0.5.9;
+pragma solidity ^0.5.5;
 
 import "@0x/contracts-utils/contracts/src/LibBytes.sol";
-import "./interfaces/ICoordinatorSignatureValidator.sol";
+import "./mixins/MSignatureValidator.sol";
 
 
 contract MixinSignatureValidator is
-    ICoordinatorSignatureValidator
+    MSignatureValidator
 {
     using LibBytes for bytes;
 

contracts/coordinator/contracts/src/interfaces/ICoordinatorApprovalVerifier.sol

@@ -1,6 +1,6 @@
 /*
 
-  Copyright 2019 ZeroEx Intl.
+  Copyright 2018 ZeroEx Intl.
 
   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
@@ -16,11 +16,11 @@
 
 */
 
-pragma solidity ^0.5.9;
-pragma experimental ABIEncoderV2;
+pragma solidity ^0.5.5;
+pragma experimental "ABIEncoderV2";
 
 import "@0x/contracts-exchange-libs/contracts/src/LibOrder.sol";
-import "@0x/contracts-exchange-libs/contracts/src/LibZeroExTransaction.sol";
+import "../libs/LibZeroExTransaction.sol";
 
 
 contract ICoordinatorApprovalVerifier {

contracts/coordinator/contracts/src/interfaces/ICoordinatorCore.sol

@@ -1,6 +1,6 @@
 /*
 
-  Copyright 2019 ZeroEx Intl.
+  Copyright 2018 ZeroEx Intl.
 
   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
@@ -16,10 +16,10 @@
 
 */
 
-pragma solidity ^0.5.9;
-pragma experimental ABIEncoderV2;
+pragma solidity ^0.5.5;
+pragma experimental "ABIEncoderV2";
 
-import "@0x/contracts-exchange-libs/contracts/src/LibZeroExTransaction.sol";
+import "../libs/LibZeroExTransaction.sol";
 
 
 contract ICoordinatorCore {

contracts/coordinator/contracts/src/interfaces/ICoordinatorSignatureValidator.sol

@@ -1,45 +0,0 @@
-/*
-
-  Copyright 2019 ZeroEx Intl.
-
-  Licensed under the Apache License, Version 2.0 (the "License");
-  you may not use this file except in compliance with the License.
-  You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-  Unless required by applicable law or agreed to in writing, software
-  distributed under the License is distributed on an "AS IS" BASIS,
-  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-  See the License for the specific language governing permissions and
-  limitations under the License.
-
-*/
-
-pragma solidity ^0.5.9;
-
-
-contract ICoordinatorSignatureValidator {
-
-   // Allowed signature types.
-    enum SignatureType {
-        Illegal,                // 0x00, default value
-        Invalid,                // 0x01
-        EIP712,                 // 0x02
-        EthSign,                // 0x03
-        Wallet,                 // 0x04
-        Validator,              // 0x05
-        PreSigned,              // 0x06
-        OrderValidator,         // 0x07
-        WalletOrderValidator,   // 0x08
-        NSignatureTypes         // 0x09, number of signature types. Always leave at end.
-    }
-
-    /// @dev Recovers the address of a signer given a hash and signature.
-    /// @param hash Any 32 byte hash.
-    /// @param signature Proof that the hash has been signed by signer.
-    function getSignerAddress(bytes32 hash, bytes memory signature)
-        public
-        pure
-        returns (address signerAddress);
-}

contracts/coordinator/contracts/src/interfaces/ISignatureValidator.sol

@@ -1,6 +1,6 @@
 /*
 
-  Copyright 2019 ZeroEx Intl.
+  Copyright 2018 ZeroEx Intl.
 
   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
@@ -18,10 +18,14 @@
 
 pragma solidity ^0.5.5;
 
-import "../src/libs/LibExchangeRichErrorDecoder.sol";
 
+contract ISignatureValidator {
 
-// solhint-disable no-empty-blocks
-contract TestLibExchangeRichErrorDecoder is
-    LibExchangeRichErrorDecoder
-{}
+    /// @dev Recovers the address of a signer given a hash and signature.
+    /// @param hash Any 32 byte hash.
+    /// @param signature Proof that the hash has been signed by signer.
+    function getSignerAddress(bytes32 hash, bytes memory signature)
+        public
+        pure
+        returns (address signerAddress);
+}

contracts/coordinator/contracts/src/interfaces/ITransactions.sol

@@ -1,6 +1,6 @@
 /*
 
-  Copyright 2019 ZeroEx Intl.
+  Copyright 2018 ZeroEx Intl.
 
   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
@@ -15,24 +15,21 @@
   limitations under the License.
 
 */
-
-pragma solidity ^0.5.9;
-
-import "@0x/contracts-utils/contracts/src/LibEIP1271.sol";
+pragma solidity ^0.5.5;
 
 
-contract IEIP1271Wallet is
-    LibEIP1271
-{
-    /// @dev Verifies that a signature is valid.
-    /// @param data Arbitrary signed data.
-    /// @param signature Proof that data has been signed.
-    /// @return magicValue bytes4(0x20c13b0b) if the signature check succeeds.
-    function isValidSignature(
+contract ITransactions {
+
+    /// @dev Executes an exchange method call in the context of signer.
+    /// @param salt Arbitrary number to ensure uniqueness of transaction hash.
+    /// @param signerAddress Address of transaction signer.
+    /// @param data AbiV2 encoded calldata.
+    /// @param signature Proof of signer transaction by signer.
+    function executeTransaction(
+        uint256 salt,
+        address signerAddress,
         bytes calldata data,
         bytes calldata signature
     )
-        external
-        view
-        returns (bytes4 magicValue);
+        external;
 }

contracts/coordinator/contracts/src/libs/LibConstants.sol

@@ -1,6 +1,6 @@
 /*
 
-  Copyright 2019 ZeroEx Intl.
+  Copyright 2018 ZeroEx Intl.
 
   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
@@ -16,21 +16,19 @@
 
 */
 
-pragma solidity ^0.5.9;
+pragma solidity ^0.5.5;
 
-import "@0x/contracts-exchange/contracts/src/interfaces/ITransactions.sol";
+import "../interfaces/ITransactions.sol";
 
 
-// solhint-disable var-name-mixedcase
 contract LibConstants {
 
-    // The 0x Exchange contract.
+     // solhint-disable-next-line var-name-mixedcase
     ITransactions internal EXCHANGE;
 
-    /// @param exchange Address of the 0x Exchange contract.
-    constructor (address exchange)
+    constructor (address _exchange)
         public
     {
-        EXCHANGE = ITransactions(exchange);
+        EXCHANGE = ITransactions(_exchange);
     }
 }

contracts/coordinator/contracts/src/libs/LibCoordinatorApproval.sol

@@ -1,6 +1,6 @@
 /*
 
-  Copyright 2019 ZeroEx Intl.
+  Copyright 2018 ZeroEx Intl.
 
   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
@@ -16,14 +16,14 @@
 
 */
 
-pragma solidity ^0.5.9;
-pragma experimental ABIEncoderV2;
+pragma solidity ^0.5.5;
+pragma experimental "ABIEncoderV2";
 
-import "./LibEIP712CoordinatorDomain.sol";
+import "./LibEIP712Domain.sol";
 
 
 contract LibCoordinatorApproval is
-    LibEIP712CoordinatorDomain
+    LibEIP712Domain
 {
     // Hash for the EIP712 Coordinator approval message
     // keccak256(abi.encodePacked(
@@ -34,7 +34,7 @@ contract LibCoordinatorApproval is
     //     "uint256 approvalExpirationTimeSeconds",
     //     ")"
     // ));
-    bytes32 constant public EIP712_COORDINATOR_APPROVAL_SCHEMA_HASH = 0x2fbcdbaa76bc7589916958ae919dfbef04d23f6bbf26de6ff317b32c6cc01e05;
+    bytes32 constant internal EIP712_COORDINATOR_APPROVAL_SCHEMA_HASH = 0x2fbcdbaa76bc7589916958ae919dfbef04d23f6bbf26de6ff317b32c6cc01e05;
 
     struct CoordinatorApproval {
         address txOrigin;                       // Required signer of Ethereum transaction that is submitting approval.
@@ -51,14 +51,14 @@ contract LibCoordinatorApproval is
         view
         returns (bytes32 approvalHash)
     {
-        approvalHash = _hashEIP712CoordinatorMessage(_hashCoordinatorApproval(approval));
+        approvalHash = hashEIP712CoordinatorMessage(hashCoordinatorApproval(approval));
         return approvalHash;
     }
 
     /// @dev Calculated the EIP712 hash of the Coordinator approval mesasage with no domain separator.
     /// @param approval Coordinator approval message containing the transaction hash, transaction signature, and expiration of the approval.
     /// @return EIP712 hash of the Coordinator approval message with no domain separator.
-    function _hashCoordinatorApproval(CoordinatorApproval memory approval)
+    function hashCoordinatorApproval(CoordinatorApproval memory approval)
         internal
         pure
         returns (bytes32 result)

contracts/coordinator/contracts/src/libs/LibEIP712CoordinatorDomain.sol

@@ -1,66 +0,0 @@
-/*
-
-  Copyright 2019 ZeroEx Intl.
-
-  Licensed under the Apache License, Version 2.0 (the "License");
-  you may not use this file except in compliance with the License.
-  You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-  Unless required by applicable law or agreed to in writing, software
-  distributed under the License is distributed on an "AS IS" BASIS,
-  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-  See the License for the specific language governing permissions and
-  limitations under the License.
-
-*/
-
-pragma solidity ^0.5.9;
-
-import "@0x/contracts-utils/contracts/src/LibEIP712.sol";
-
-
-contract LibEIP712CoordinatorDomain is
-    LibEIP712
-{
-
-    // EIP712 Domain Name value for the Coordinator
-    string constant public EIP712_COORDINATOR_DOMAIN_NAME = "0x Protocol Coordinator";
-
-    // EIP712 Domain Version value for the Coordinator
-    string constant public EIP712_COORDINATOR_DOMAIN_VERSION = "2.0.0";
-
-    // Hash of the EIP712 Domain Separator data for the Coordinator
-    // solhint-disable-next-line var-name-mixedcase
-    bytes32 public EIP712_COORDINATOR_DOMAIN_HASH;
-
-    /// @param chainId Chain ID of the network this contract is deployed on.
-    /// @param verifyingContractAddressIfExists Address of the verifying contract (null if the address of this contract)
-    constructor (
-        uint256 chainId,
-        address verifyingContractAddressIfExists
-    )
-        public
-    {
-        address verifyingContractAddress = verifyingContractAddressIfExists == address(0) ? address(this) : verifyingContractAddressIfExists;
-        EIP712_COORDINATOR_DOMAIN_HASH = LibEIP712.hashEIP712Domain(
-            EIP712_COORDINATOR_DOMAIN_NAME,
-            EIP712_COORDINATOR_DOMAIN_VERSION,
-            chainId,
-            verifyingContractAddress
-        );
-    }
-
-    /// @dev Calculates EIP712 encoding for a hash struct in the EIP712 domain
-    ///      of this contract.
-    /// @param hashStruct The EIP712 hash struct.
-    /// @return EIP712 hash applied to this EIP712 Domain.
-    function _hashEIP712CoordinatorMessage(bytes32 hashStruct)
-        internal
-        view
-        returns (bytes32 result)
-    {
-        return LibEIP712.hashEIP712Message(EIP712_COORDINATOR_DOMAIN_HASH, hashStruct);
-    }
-}

contracts/coordinator/contracts/src/libs/LibEIP712Domain.sol

@@ -0,0 +1,131 @@
+/*
+
+  Copyright 2018 ZeroEx Intl.
+
+  Licensed under the Apache License, Version 2.0 (the "License");
+  you may not use this file except in compliance with the License.
+  You may obtain a copy of the License at
+
+    http://www.apache.org/licenses/LICENSE-2.0
+
+  Unless required by applicable law or agreed to in writing, software
+  distributed under the License is distributed on an "AS IS" BASIS,
+  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+  See the License for the specific language governing permissions and
+  limitations under the License.
+
+*/
+
+pragma solidity ^0.5.5;
+
+import "./LibConstants.sol";
+
+
+contract LibEIP712Domain is
+    LibConstants
+{
+
+    // EIP191 header for EIP712 prefix
+    string constant internal EIP191_HEADER = "\x19\x01";
+
+    // EIP712 Domain Name value for the Coordinator
+    string constant internal EIP712_COORDINATOR_DOMAIN_NAME = "0x Protocol Coordinator";
+
+    // EIP712 Domain Version value for the Coordinator
+    string constant internal EIP712_COORDINATOR_DOMAIN_VERSION = "1.0.0";
+
+    // EIP712 Domain Name value for the Exchange
+    string constant internal EIP712_EXCHANGE_DOMAIN_NAME = "0x Protocol";
+
+    // EIP712 Domain Version value for the Exchange
+    string constant internal EIP712_EXCHANGE_DOMAIN_VERSION = "2";
+
+    // Hash of the EIP712 Domain Separator Schema
+    bytes32 constant internal EIP712_DOMAIN_SEPARATOR_SCHEMA_HASH = keccak256(abi.encodePacked(
+        "EIP712Domain(",
+        "string name,",
+        "string version,",
+        "address verifyingContract",
+        ")"
+    ));
+
+    // Hash of the EIP712 Domain Separator data for the Coordinator
+    // solhint-disable-next-line var-name-mixedcase
+    bytes32 public EIP712_COORDINATOR_DOMAIN_HASH;
+
+    // Hash of the EIP712 Domain Separator data for the Exchange
+    // solhint-disable-next-line var-name-mixedcase
+    bytes32 public EIP712_EXCHANGE_DOMAIN_HASH;
+
+    constructor ()
+        public
+    {
+        EIP712_COORDINATOR_DOMAIN_HASH = keccak256(abi.encodePacked(
+            EIP712_DOMAIN_SEPARATOR_SCHEMA_HASH,
+            keccak256(bytes(EIP712_COORDINATOR_DOMAIN_NAME)),
+            keccak256(bytes(EIP712_COORDINATOR_DOMAIN_VERSION)),
+            uint256(address(this))
+        ));
+
+        EIP712_EXCHANGE_DOMAIN_HASH = keccak256(abi.encodePacked(
+            EIP712_DOMAIN_SEPARATOR_SCHEMA_HASH,
+            keccak256(bytes(EIP712_EXCHANGE_DOMAIN_NAME)),
+            keccak256(bytes(EIP712_EXCHANGE_DOMAIN_VERSION)),
+            uint256(address(EXCHANGE))
+        ));
+    }
+
+    /// @dev Calculates EIP712 encoding for a hash struct in the EIP712 domain
+    ///      of this contract.
+    /// @param hashStruct The EIP712 hash struct.
+    /// @return EIP712 hash applied to this EIP712 Domain.
+    function hashEIP712CoordinatorMessage(bytes32 hashStruct)
+        internal
+        view
+        returns (bytes32 result)
+    {
+        return hashEIP712Message(EIP712_COORDINATOR_DOMAIN_HASH, hashStruct);
+    }
+
+    /// @dev Calculates EIP712 encoding for a hash struct in the EIP712 domain
+    ///      of the Exchange contract.
+    /// @param hashStruct The EIP712 hash struct.
+    /// @return EIP712 hash applied to the Exchange EIP712 Domain.
+    function hashEIP712ExchangeMessage(bytes32 hashStruct)
+        internal
+        view
+        returns (bytes32 result)
+    {
+        return hashEIP712Message(EIP712_EXCHANGE_DOMAIN_HASH, hashStruct);
+    }
+
+    /// @dev Calculates EIP712 encoding for a hash struct with a given domain hash.
+    /// @param eip712DomainHash Hash of the domain domain separator data.
+    /// @param hashStruct The EIP712 hash struct.
+    /// @return EIP712 hash applied to the Exchange EIP712 Domain.
+    function hashEIP712Message(bytes32 eip712DomainHash, bytes32 hashStruct)
+        internal
+        pure
+        returns (bytes32 result)
+    {
+        // Assembly for more efficient computing:
+        // keccak256(abi.encodePacked(
+        //     EIP191_HEADER,
+        //     EIP712_DOMAIN_HASH,
+        //     hashStruct
+        // ));
+
+        assembly {
+            // Load free memory pointer
+            let memPtr := mload(64)
+
+            mstore(memPtr, 0x1901000000000000000000000000000000000000000000000000000000000000)  // EIP191 header
+            mstore(add(memPtr, 2), eip712DomainHash)                                            // EIP712 domain hash
+            mstore(add(memPtr, 34), hashStruct)                                                 // Hash of struct
+
+            // Compute hash
+            result := keccak256(memPtr, 66)
+        }
+        return result;
+    }
+}

contracts/coordinator/contracts/src/libs/LibZeroExTransaction.sol

@@ -0,0 +1,95 @@
+/*
+
+  Copyright 2018 ZeroEx Intl.
+
+  Licensed under the Apache License, Version 2.0 (the "License");
+  you may not use this file except in compliance with the License.
+  You may obtain a copy of the License at
+
+    http://www.apache.org/licenses/LICENSE-2.0
+
+  Unless required by applicable law or agreed to in writing, software
+  distributed under the License is distributed on an "AS IS" BASIS,
+  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+  See the License for the specific language governing permissions and
+  limitations under the License.
+
+*/
+
+pragma solidity ^0.5.5;
+pragma experimental "ABIEncoderV2";
+
+import "./LibEIP712Domain.sol";
+
+
+contract LibZeroExTransaction is
+    LibEIP712Domain
+{
+    // Hash for the EIP712 0x transaction schema
+    // keccak256(abi.encodePacked(
+    //    "ZeroExTransaction(",
+    //    "uint256 salt,",
+    //    "address signerAddress,",
+    //    "bytes data",
+    //    ")"
+    // ));
+    bytes32 constant internal EIP712_ZEROEX_TRANSACTION_SCHEMA_HASH = 0x213c6f636f3ea94e701c0adf9b2624aa45a6c694f9a292c094f9a81c24b5df4c;
+
+    struct ZeroExTransaction {
+        uint256 salt;           // Arbitrary number to ensure uniqueness of transaction hash.
+        address signerAddress;  // Address of transaction signer.
+        bytes data;             // AbiV2 encoded calldata.
+    }
+
+    /// @dev Calculates the EIP712 hash of a 0x transaction using the domain separator of the Exchange contract.
+    /// @param transaction 0x transaction containing salt, signerAddress, and data.
+    /// @return EIP712 hash of the transaction with the domain separator of this contract.
+    function getTransactionHash(ZeroExTransaction memory transaction)
+        public
+        view
+        returns (bytes32 transactionHash)
+    {
+        // Hash the transaction with the domain separator of the Exchange contract.
+        transactionHash = hashEIP712ExchangeMessage(hashZeroExTransaction(transaction));
+        return transactionHash;
+    }
+
+    /// @dev Calculates EIP712 hash of the 0x transaction with no domain separator.
+    /// @param transaction 0x transaction containing salt, signerAddress, and data.
+    /// @return EIP712 hash of the transaction with no domain separator.
+    function hashZeroExTransaction(ZeroExTransaction memory transaction)
+        internal
+        pure
+        returns (bytes32 result)
+    {
+        bytes32 schemaHash = EIP712_ZEROEX_TRANSACTION_SCHEMA_HASH;
+        bytes memory data = transaction.data;
+        uint256 salt = transaction.salt;
+        address signerAddress = transaction.signerAddress;
+
+        // Assembly for more efficiently computing:
+        // keccak256(abi.encodePacked(
+        //     EIP712_ZEROEX_TRANSACTION_SCHEMA_HASH,
+        //     transaction.salt,
+        //     uint256(transaction.signerAddress),
+        //     keccak256(transaction.data)
+        // ));
+
+        assembly {
+            // Compute hash of data
+            let dataHash := keccak256(add(data, 32), mload(data))
+
+            // Load free memory pointer
+            let memPtr := mload(64)
+
+            mstore(memPtr, schemaHash)                                                               // hash of schema
+            mstore(add(memPtr, 32), salt)                                                            // salt
+            mstore(add(memPtr, 64), and(signerAddress, 0xffffffffffffffffffffffffffffffffffffffff))  // signerAddress
+            mstore(add(memPtr, 96), dataHash)                                                        // hash of data
+
+            // Compute hash
+            result := keccak256(memPtr, 128)
+        }
+        return result;
+    }
+}

contracts/coordinator/contracts/src/mixins/MCoordinatorApprovalVerifier.sol

@@ -0,0 +1,46 @@
+/*
+
+  Copyright 2018 ZeroEx Intl.
+
+  Licensed under the Apache License, Version 2.0 (the "License");
+  you may not use this file except in compliance with the License.
+  You may obtain a copy of the License at
+
+    http://www.apache.org/licenses/LICENSE-2.0
+
+  Unless required by applicable law or agreed to in writing, software
+  distributed under the License is distributed on an "AS IS" BASIS,
+  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+  See the License for the specific language governing permissions and
+  limitations under the License.
+
+*/
+
+pragma solidity ^0.5.5;
+pragma experimental "ABIEncoderV2";
+
+import "@0x/contracts-exchange-libs/contracts/src/LibOrder.sol";
+import "../interfaces/ICoordinatorApprovalVerifier.sol";
+
+
+contract MCoordinatorApprovalVerifier is
+    ICoordinatorApprovalVerifier
+{
+    /// @dev Validates that the feeRecipients of a batch of order have approved a 0x transaction.
+    /// @param transaction 0x transaction containing salt, signerAddress, and data.
+    /// @param orders Array of order structs containing order specifications.
+    /// @param txOrigin Required signer of Ethereum transaction calling this function.
+    /// @param transactionSignature Proof that the transaction has been signed by the signer.
+    /// @param approvalExpirationTimeSeconds Array of expiration times in seconds for which each corresponding approval signature expires.
+    /// @param approvalSignatures Array of signatures that correspond to the feeRecipients of each order.
+    function assertValidTransactionOrdersApproval(
+        LibZeroExTransaction.ZeroExTransaction memory transaction,
+        LibOrder.Order[] memory orders,
+        address txOrigin,
+        bytes memory transactionSignature,
+        uint256[] memory approvalExpirationTimeSeconds,
+        bytes[] memory approvalSignatures
+    )
+        internal
+        view;
+}

contracts/coordinator/contracts/src/mixins/MSignatureValidator.sol

@@ -1,6 +1,6 @@
 /*
 
-  Copyright 2019 ZeroEx Intl.
+  Copyright 2018 ZeroEx Intl.
 
   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
@@ -16,27 +16,20 @@
 
 */
 
-pragma solidity ^0.5.9;
-pragma experimental ABIEncoderV2;
+pragma solidity ^0.5.5;
 
-import "../src/Staking.sol";
+import "../interfaces/ISignatureValidator.sol";
 
 
-contract TestExchangeManager is
-    Staking
+contract MSignatureValidator is
+    ISignatureValidator
 {
-    function setValidExchange(address exchange)
-        external
-    {
-        validExchanges[exchange] = true;
-    }
-
-    function onlyExchangeFunction()
-        external
-        view
-        onlyExchange
-        returns (bool)
-    {
-        return true;
+    // Allowed signature types.
+    enum SignatureType {
+        Illegal,         // 0x00, default value
+        Invalid,         // 0x01
+        EIP712,          // 0x02
+        EthSign,         // 0x03
+        NSignatureTypes  // 0x04, number of signature types. Always leave at end.
     }
 }

contracts/coordinator/contracts/src/registry/CoordinatorRegistry.sol

@@ -1,6 +1,6 @@
 /*
 
-  Copyright 2019 ZeroEx Intl.
+  Copyright 2018 ZeroEx Intl.
 
   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
@@ -16,7 +16,7 @@
 
 */
 
-pragma solidity ^0.5.9;
+pragma solidity ^0.5.5;
 
 import "./MixinCoordinatorRegistryCore.sol";
 

contracts/coordinator/contracts/src/registry/MixinCoordinatorRegistryCore.sol

@@ -1,6 +1,6 @@
 /*
 
-  Copyright 2019 ZeroEx Intl.
+  Copyright 2018 ZeroEx Intl.
 
   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
@@ -16,7 +16,7 @@
 
 */
 
-pragma solidity ^0.5.9;
+pragma solidity ^0.5.5;
 
 import "./interfaces/ICoordinatorRegistryCore.sol";
 

contracts/coordinator/contracts/src/registry/interfaces/ICoordinatorRegistryCore.sol

@@ -1,6 +1,6 @@
 /*
 
-  Copyright 2019 ZeroEx Intl.
+  Copyright 2018 ZeroEx Intl.
 
   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
@@ -16,7 +16,7 @@
 
 */
 
-pragma solidity ^0.5.9;
+pragma solidity ^0.5.5;
 
 
 // solhint-disable no-empty-blocks

contracts/coordinator/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@0x/contracts-coordinator",
-    "version": "2.1.0-beta.0",
+    "version": "2.0.11",
     "engines": {
         "node": ">=6.12"
     },
@@ -12,7 +12,7 @@
     "scripts": {
         "build": "yarn pre_build && tsc -b",
         "build:ci": "yarn build",
-        "pre_build": "run-s compile contracts:gen generate_contract_wrappers",
+        "pre_build": "run-s compile generate_contract_wrappers",
         "test": "yarn run_mocha",
         "rebuild_and_test": "run-s build test",
         "test:coverage": "SOLIDITY_COVERAGE=true run-s build run_mocha coverage:report:text coverage:report:lcov",
@@ -22,7 +22,7 @@
         "compile": "sol-compiler",
         "watch": "sol-compiler -w",
         "clean": "shx rm -rf lib generated-artifacts generated-wrappers",
-        "generate_contract_wrappers": "abi-gen --abis  ${npm_package_config_abis} --output generated-wrappers --backend ethers",
+        "generate_contract_wrappers": "abi-gen --abis  ${npm_package_config_abis} --template ../../node_modules/@0x/abi-gen-templates/contract.handlebars --partials '../../node_modules/@0x/abi-gen-templates/partials/**/*.handlebars' --output generated-wrappers --backend ethers",
         "lint": "tslint --format stylish --project . --exclude ./generated-wrappers/**/* --exclude ./generated-artifacts/**/* --exclude **/lib/**/* && yarn lint-contracts",
         "fix": "tslint --fix --format stylish --project . --exclude ./generated-wrappers/**/* --exclude ./generated-artifacts/**/* --exclude **/lib/**/* && yarn lint-contracts",
         "coverage:report:text": "istanbul report text",
@@ -31,8 +31,7 @@
         "coverage:report:lcov": "istanbul report lcov",
         "test:circleci": "yarn test",
         "contracts:gen": "contracts-gen",
-        "lint-contracts": "solhint -c ../.solhint.json contracts/**/**/**/**/*.sol",
-        "compile:truffle": "truffle compile"
+        "lint-contracts": "solhint -c ../.solhint.json contracts/**/**/**/**/*.sol"
     },
     "config": {
         "abis": "./generated-artifacts/@(Coordinator|CoordinatorRegistry).json",
@@ -48,11 +47,11 @@
     },
     "homepage": "https://github.com/0xProject/0x-monorepo/contracts/extensions/README.md",
     "devDependencies": {
-        "@0x/abi-gen": "^4.3.0-beta.0",
-        "@0x/contracts-gen": "^1.1.0-beta.0",
-        "@0x/contracts-test-utils": "^3.2.0-beta.0",
-        "@0x/dev-utils": "^2.4.0-beta.0",
-        "@0x/sol-compiler": "^3.2.0-beta.0",
+        "@0x/abi-gen": "^4.1.1",
+        "@0x/contracts-gen": "^1.0.13",
+        "@0x/contracts-test-utils": "^3.1.14",
+        "@0x/dev-utils": "^2.3.1",
+        "@0x/sol-compiler": "^3.1.13",
         "@0x/tslint-config": "^3.0.1",
         "@types/lodash": "4.14.104",
         "@types/mocha": "^5.2.7",
@@ -66,23 +65,22 @@
         "npm-run-all": "^4.1.2",
         "shx": "^0.2.2",
         "solhint": "^1.4.1",
-        "truffle": "^5.0.32",
         "tslint": "5.11.0",
         "typescript": "3.0.1"
     },
     "dependencies": {
-        "@0x/base-contract": "^5.5.0-beta.0",
-        "@0x/contracts-asset-proxy": "^2.3.0-beta.0",
-        "@0x/contracts-erc20": "^2.3.0-beta.0",
-        "@0x/contracts-exchange": "^2.2.0-beta.0",
-        "@0x/contracts-exchange-libs": "^3.1.0-beta.0",
-        "@0x/contracts-utils": "^3.3.0-beta.0",
-        "@0x/order-utils": "^8.5.0-beta.0",
-        "@0x/types": "^2.5.0-beta.0",
-        "@0x/typescript-typings": "^4.4.0-beta.0",
-        "@0x/utils": "^4.6.0-beta.0",
-        "@0x/web3-wrapper": "^6.1.0-beta.0",
-        "ethereum-types": "^2.2.0-beta.0",
+        "@0x/base-contract": "^5.3.2",
+        "@0x/contracts-asset-proxy": "^2.2.6",
+        "@0x/contracts-erc20": "^2.2.12",
+        "@0x/contracts-exchange": "^2.1.12",
+        "@0x/contracts-exchange-libs": "^3.0.6",
+        "@0x/contracts-utils": "^3.2.2",
+        "@0x/order-utils": "^8.3.0",
+        "@0x/types": "^2.4.1",
+        "@0x/typescript-typings": "^4.2.4",
+        "@0x/utils": "^4.5.0",
+        "@0x/web3-wrapper": "^6.0.11",
+        "ethereum-types": "^2.1.4",
         "ethereumjs-util": "^5.1.1",
         "lodash": "^4.17.11"
     },

contracts/coordinator/test/coordinator.ts

@@ -2,17 +2,14 @@ import { ERC20ProxyContract, ERC20Wrapper } from '@0x/contracts-asset-proxy';
 import { DummyERC20TokenContract } from '@0x/contracts-erc20';
 import {
     artifacts as exchangeArtifacts,
-    constants as exchangeConstants,
     ExchangeCancelEventArgs,
     ExchangeCancelUpToEventArgs,
     ExchangeContract,
-    exchangeDataEncoder,
     ExchangeFillEventArgs,
-    ExchangeFunctionName,
 } from '@0x/contracts-exchange';
 import {
     chaiSetup,
-    constants,
+    constants as devConstants,
     expectTransactionFailedAsync,
     getLatestBlockTimestampAsync,
     OrderFactory,
@@ -23,12 +20,12 @@ import {
 } from '@0x/contracts-test-utils';
 import { BlockchainLifecycle } from '@0x/dev-utils';
 import { assetDataUtils, orderHashUtils } from '@0x/order-utils';
-import { RevertReason } from '@0x/types';
-import { BigNumber, providerUtils } from '@0x/utils';
+import { RevertReason, SignedOrder } from '@0x/types';
+import { BigNumber } from '@0x/utils';
 import * as chai from 'chai';
 import { LogWithDecodedArgs } from 'ethereum-types';
 
-import { ApprovalFactory, artifacts, CoordinatorContract } from '../src';
+import { ApprovalFactory, artifacts, constants, CoordinatorContract, exchangeDataEncoder } from '../src';
 
 chaiSetup.configure();
 const expect = chai.expect;
@@ -36,7 +33,6 @@ const blockchainLifecycle = new BlockchainLifecycle(web3Wrapper);
 web3Wrapper.abiDecoder.addABI(exchangeArtifacts.Exchange.compilerOutput.abi);
 // tslint:disable:no-unnecessary-type-assertion
 describe('Coordinator tests', () => {
-    let chainId: number;
     let makerAddress: string;
     let owner: string;
     let takerAddress: string;
@@ -45,7 +41,7 @@ describe('Coordinator tests', () => {
     let erc20Proxy: ERC20ProxyContract;
     let erc20TokenA: DummyERC20TokenContract;
     let erc20TokenB: DummyERC20TokenContract;
-    let makerFeeToken: DummyERC20TokenContract;
+    let zrxToken: DummyERC20TokenContract;
     let coordinatorContract: CoordinatorContract;
     let exchange: ExchangeContract;
 
@@ -62,16 +58,15 @@ describe('Coordinator tests', () => {
         await blockchainLifecycle.revertAsync();
     });
     before(async () => {
-        chainId = await providerUtils.getChainIdAsync(provider);
         const accounts = await web3Wrapper.getAvailableAddressesAsync();
         const usedAddresses = ([owner, makerAddress, takerAddress, feeRecipientAddress] = accounts.slice(0, 4));
 
         erc20Wrapper = new ERC20Wrapper(provider, usedAddresses, owner);
         erc20Proxy = await erc20Wrapper.deployProxyAsync();
         const numDummyErc20ToDeploy = 3;
-        [erc20TokenA, erc20TokenB, makerFeeToken] = await erc20Wrapper.deployDummyTokensAsync(
+        [erc20TokenA, erc20TokenB, zrxToken] = await erc20Wrapper.deployDummyTokensAsync(
             numDummyErc20ToDeploy,
-            constants.DUMMY_TOKEN_DECIMALS,
+            devConstants.DUMMY_TOKEN_DECIMALS,
         );
         await erc20Wrapper.setBalancesAndAllowancesAsync();
 
@@ -79,17 +74,18 @@ describe('Coordinator tests', () => {
             exchangeArtifacts.Exchange,
             provider,
             txDefaults,
-            new BigNumber(chainId),
+            artifacts,
+            assetDataUtils.encodeERC20AssetData(zrxToken.address),
         );
 
         await web3Wrapper.awaitTransactionSuccessAsync(
             await erc20Proxy.addAuthorizedAddress.sendTransactionAsync(exchange.address, { from: owner }),
-            constants.AWAIT_TRANSACTION_MINED_MS,
+            devConstants.AWAIT_TRANSACTION_MINED_MS,
         );
 
         await web3Wrapper.awaitTransactionSuccessAsync(
             await exchange.registerAssetProxy.sendTransactionAsync(erc20Proxy.address, { from: owner }),
-            constants.AWAIT_TRANSACTION_MINED_MS,
+            devConstants.AWAIT_TRANSACTION_MINED_MS,
         );
 
         coordinatorContract = await CoordinatorContract.deployFrom0xArtifactAsync(
@@ -98,28 +94,24 @@ describe('Coordinator tests', () => {
             txDefaults,
             artifacts,
             exchange.address,
-            new BigNumber(chainId),
         );
 
         // Configure order defaults
         const defaultOrderParams = {
-            ...constants.STATIC_ORDER_PARAMS,
+            ...devConstants.STATIC_ORDER_PARAMS,
+            exchangeAddress: exchange.address,
             senderAddress: coordinatorContract.address,
             makerAddress,
             feeRecipientAddress,
             makerAssetData: assetDataUtils.encodeERC20AssetData(erc20TokenA.address),
             takerAssetData: assetDataUtils.encodeERC20AssetData(erc20TokenB.address),
-            makerFeeAssetData: assetDataUtils.encodeERC20AssetData(makerFeeToken.address),
-            takerFeeAssetData: assetDataUtils.encodeERC20AssetData(makerFeeToken.address),
-            exchangeAddress: exchange.address,
-            chainId,
         };
-        const makerPrivateKey = constants.TESTRPC_PRIVATE_KEYS[accounts.indexOf(makerAddress)];
-        const takerPrivateKey = constants.TESTRPC_PRIVATE_KEYS[accounts.indexOf(takerAddress)];
-        const feeRecipientPrivateKey = constants.TESTRPC_PRIVATE_KEYS[accounts.indexOf(feeRecipientAddress)];
+        const makerPrivateKey = devConstants.TESTRPC_PRIVATE_KEYS[accounts.indexOf(makerAddress)];
+        const takerPrivateKey = devConstants.TESTRPC_PRIVATE_KEYS[accounts.indexOf(takerAddress)];
+        const feeRecipientPrivateKey = devConstants.TESTRPC_PRIVATE_KEYS[accounts.indexOf(feeRecipientAddress)];
         orderFactory = new OrderFactory(makerPrivateKey, defaultOrderParams);
-        makerTransactionFactory = new TransactionFactory(makerPrivateKey, exchange.address, chainId);
-        takerTransactionFactory = new TransactionFactory(takerPrivateKey, exchange.address, chainId);
+        makerTransactionFactory = new TransactionFactory(makerPrivateKey, exchange.address);
+        takerTransactionFactory = new TransactionFactory(takerPrivateKey, exchange.address);
         approvalFactory = new ApprovalFactory(feeRecipientPrivateKey, coordinatorContract.address);
     });
     beforeEach(async () => {
@@ -130,11 +122,11 @@ describe('Coordinator tests', () => {
     });
 
     describe('single order fills', () => {
-        for (const fnName of exchangeConstants.SINGLE_FILL_FN_NAMES) {
+        for (const fnName of constants.SINGLE_FILL_FN_NAMES) {
             it(`${fnName} should fill the order with a signed approval`, async () => {
                 const orders = [await orderFactory.newSignedOrderAsync()];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await takerTransactionFactory.newSignedTransactionAsync({ data });
+                const transaction = takerTransactionFactory.newSignedTransaction(data);
                 const currentTimestamp = await getLatestBlockTimestampAsync();
                 const approvalExpirationTimeSeconds = new BigNumber(currentTimestamp).plus(constants.TIME_BUFFER);
                 const approval = approvalFactory.newSignedApproval(
@@ -151,7 +143,7 @@ describe('Coordinator tests', () => {
                         [approval.signature],
                         { from: takerAddress },
                     ),
-                    constants.AWAIT_TRANSACTION_MINED_MS,
+                    devConstants.AWAIT_TRANSACTION_MINED_MS,
                 );
                 const fillLogs = transactionReceipt.logs.filter(
                     log => (log as LogWithDecodedArgs<ExchangeFillEventArgs>).event === 'Fill',
@@ -173,7 +165,7 @@ describe('Coordinator tests', () => {
             it(`${fnName} should fill the order if called by approver`, async () => {
                 const orders = [await orderFactory.newSignedOrderAsync()];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await takerTransactionFactory.newSignedTransactionAsync({ data });
+                const transaction = takerTransactionFactory.newSignedTransaction(data);
                 const transactionReceipt = await web3Wrapper.awaitTransactionSuccessAsync(
                     await coordinatorContract.executeTransaction.sendTransactionAsync(
                         transaction,
@@ -183,7 +175,7 @@ describe('Coordinator tests', () => {
                         [],
                         { from: feeRecipientAddress },
                     ),
-                    constants.AWAIT_TRANSACTION_MINED_MS,
+                    devConstants.AWAIT_TRANSACTION_MINED_MS,
                 );
                 const fillLogs = transactionReceipt.logs.filter(
                     log => (log as LogWithDecodedArgs<ExchangeFillEventArgs>).event === 'Fill',
@@ -205,7 +197,7 @@ describe('Coordinator tests', () => {
             it(`${fnName} should revert with no approval signature`, async () => {
                 const orders = [await orderFactory.newSignedOrderAsync()];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await takerTransactionFactory.newSignedTransactionAsync({ data });
+                const transaction = takerTransactionFactory.newSignedTransaction(data);
                 await expectTransactionFailedAsync(
                     coordinatorContract.executeTransaction.sendTransactionAsync(
                         transaction,
@@ -215,7 +207,7 @@ describe('Coordinator tests', () => {
                         [],
                         {
                             from: takerAddress,
-                            gas: constants.MAX_EXECUTE_TRANSACTION_GAS,
+                            gas: devConstants.MAX_EXECUTE_TRANSACTION_GAS,
                         },
                     ),
                     RevertReason.InvalidApprovalSignature,
@@ -224,7 +216,7 @@ describe('Coordinator tests', () => {
             it(`${fnName} should revert with an invalid approval signature`, async () => {
                 const orders = [await orderFactory.newSignedOrderAsync()];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await takerTransactionFactory.newSignedTransactionAsync({ data });
+                const transaction = takerTransactionFactory.newSignedTransaction(data);
                 const currentTimestamp = await getLatestBlockTimestampAsync();
                 const approvalExpirationTimeSeconds = new BigNumber(currentTimestamp).plus(constants.TIME_BUFFER);
                 const approval = approvalFactory.newSignedApproval(
@@ -248,7 +240,7 @@ describe('Coordinator tests', () => {
             it(`${fnName} should revert with an expired approval`, async () => {
                 const orders = [await orderFactory.newSignedOrderAsync()];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await takerTransactionFactory.newSignedTransactionAsync({ data });
+                const transaction = takerTransactionFactory.newSignedTransaction(data);
                 const currentTimestamp = await getLatestBlockTimestampAsync();
                 const approvalExpirationTimeSeconds = new BigNumber(currentTimestamp).minus(constants.TIME_BUFFER);
                 const approval = approvalFactory.newSignedApproval(
@@ -271,7 +263,7 @@ describe('Coordinator tests', () => {
             it(`${fnName} should revert if not called by tx signer or approver`, async () => {
                 const orders = [await orderFactory.newSignedOrderAsync()];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await takerTransactionFactory.newSignedTransactionAsync({ data });
+                const transaction = takerTransactionFactory.newSignedTransaction(data);
                 const currentTimestamp = await getLatestBlockTimestampAsync();
                 const approvalExpirationTimeSeconds = new BigNumber(currentTimestamp).plus(constants.TIME_BUFFER);
                 const approval = approvalFactory.newSignedApproval(
@@ -294,11 +286,11 @@ describe('Coordinator tests', () => {
         }
     });
     describe('batch order fills', () => {
-        for (const fnName of [...exchangeConstants.MARKET_FILL_FN_NAMES, ...exchangeConstants.BATCH_FILL_FN_NAMES]) {
+        for (const fnName of [...constants.MARKET_FILL_FN_NAMES, ...constants.BATCH_FILL_FN_NAMES]) {
             it(`${fnName} should fill the orders with a signed approval`, async () => {
                 const orders = [await orderFactory.newSignedOrderAsync(), await orderFactory.newSignedOrderAsync()];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await takerTransactionFactory.newSignedTransactionAsync({ data });
+                const transaction = takerTransactionFactory.newSignedTransaction(data);
                 const currentTimestamp = await getLatestBlockTimestampAsync();
                 const approvalExpirationTimeSeconds = new BigNumber(currentTimestamp).plus(constants.TIME_BUFFER);
                 const approval = approvalFactory.newSignedApproval(
@@ -313,9 +305,9 @@ describe('Coordinator tests', () => {
                         transaction.signature,
                         [approvalExpirationTimeSeconds],
                         [approval.signature],
-                        { from: takerAddress, gas: constants.MAX_EXECUTE_TRANSACTION_GAS },
+                        { from: takerAddress, gas: devConstants.MAX_EXECUTE_TRANSACTION_GAS },
                     ),
-                    constants.AWAIT_TRANSACTION_MINED_MS,
+                    devConstants.AWAIT_TRANSACTION_MINED_MS,
                 );
                 const fillLogs = transactionReceipt.logs.filter(
                     log => (log as LogWithDecodedArgs<ExchangeFillEventArgs>).event === 'Fill',
@@ -339,7 +331,7 @@ describe('Coordinator tests', () => {
             it(`${fnName} should fill the orders if called by approver`, async () => {
                 const orders = [await orderFactory.newSignedOrderAsync(), await orderFactory.newSignedOrderAsync()];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await takerTransactionFactory.newSignedTransactionAsync({ data });
+                const transaction = takerTransactionFactory.newSignedTransaction(data);
                 const transactionReceipt = await web3Wrapper.awaitTransactionSuccessAsync(
                     await coordinatorContract.executeTransaction.sendTransactionAsync(
                         transaction,
@@ -347,9 +339,9 @@ describe('Coordinator tests', () => {
                         transaction.signature,
                         [],
                         [],
-                        { from: feeRecipientAddress, gas: constants.MAX_EXECUTE_TRANSACTION_GAS },
+                        { from: feeRecipientAddress, gas: devConstants.MAX_EXECUTE_TRANSACTION_GAS },
                     ),
-                    constants.AWAIT_TRANSACTION_MINED_MS,
+                    devConstants.AWAIT_TRANSACTION_MINED_MS,
                 );
                 const fillLogs = transactionReceipt.logs.filter(
                     log => (log as LogWithDecodedArgs<ExchangeFillEventArgs>).event === 'Fill',
@@ -373,7 +365,7 @@ describe('Coordinator tests', () => {
             it(`${fnName} should revert with an invalid approval signature`, async () => {
                 const orders = [await orderFactory.newSignedOrderAsync(), await orderFactory.newSignedOrderAsync()];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await takerTransactionFactory.newSignedTransactionAsync({ data });
+                const transaction = takerTransactionFactory.newSignedTransaction(data);
                 const currentTimestamp = await getLatestBlockTimestampAsync();
                 const approvalExpirationTimeSeconds = new BigNumber(currentTimestamp).plus(constants.TIME_BUFFER);
                 const approval = approvalFactory.newSignedApproval(
@@ -397,7 +389,7 @@ describe('Coordinator tests', () => {
             it(`${fnName} should revert with an expired approval`, async () => {
                 const orders = [await orderFactory.newSignedOrderAsync(), await orderFactory.newSignedOrderAsync()];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await takerTransactionFactory.newSignedTransactionAsync({ data });
+                const transaction = takerTransactionFactory.newSignedTransaction(data);
                 const currentTimestamp = await getLatestBlockTimestampAsync();
                 const approvalExpirationTimeSeconds = new BigNumber(currentTimestamp).minus(constants.TIME_BUFFER);
                 const approval = approvalFactory.newSignedApproval(
@@ -420,7 +412,7 @@ describe('Coordinator tests', () => {
             it(`${fnName} should revert if not called by tx signer or approver`, async () => {
                 const orders = [await orderFactory.newSignedOrderAsync(), await orderFactory.newSignedOrderAsync()];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await takerTransactionFactory.newSignedTransactionAsync({ data });
+                const transaction = takerTransactionFactory.newSignedTransaction(data);
                 const currentTimestamp = await getLatestBlockTimestampAsync();
                 const approvalExpirationTimeSeconds = new BigNumber(currentTimestamp).plus(constants.TIME_BUFFER);
                 const approval = approvalFactory.newSignedApproval(
@@ -445,8 +437,8 @@ describe('Coordinator tests', () => {
     describe('cancels', () => {
         it('cancelOrder call should be successful without an approval', async () => {
             const orders = [await orderFactory.newSignedOrderAsync()];
-            const data = exchangeDataEncoder.encodeOrdersToExchangeData(ExchangeFunctionName.CancelOrder, orders);
-            const transaction = await makerTransactionFactory.newSignedTransactionAsync({ data });
+            const data = exchangeDataEncoder.encodeOrdersToExchangeData(constants.CANCEL_ORDER, orders);
+            const transaction = makerTransactionFactory.newSignedTransaction(data);
             const transactionReceipt = await web3Wrapper.awaitTransactionSuccessAsync(
                 await coordinatorContract.executeTransaction.sendTransactionAsync(
                     transaction,
@@ -473,8 +465,8 @@ describe('Coordinator tests', () => {
         });
         it('batchCancelOrders call should be successful without an approval', async () => {
             const orders = [await orderFactory.newSignedOrderAsync(), await orderFactory.newSignedOrderAsync()];
-            const data = exchangeDataEncoder.encodeOrdersToExchangeData(ExchangeFunctionName.BatchCancelOrders, orders);
-            const transaction = await makerTransactionFactory.newSignedTransactionAsync({ data });
+            const data = exchangeDataEncoder.encodeOrdersToExchangeData(constants.BATCH_CANCEL_ORDERS, orders);
+            const transaction = makerTransactionFactory.newSignedTransaction(data);
             const transactionReceipt = await web3Wrapper.awaitTransactionSuccessAsync(
                 await coordinatorContract.executeTransaction.sendTransactionAsync(
                     transaction,
@@ -502,9 +494,9 @@ describe('Coordinator tests', () => {
             });
         });
         it('cancelOrdersUpTo call should be successful without an approval', async () => {
-            const targetEpoch = constants.ZERO_AMOUNT;
-            const data = exchange.cancelOrdersUpTo.getABIEncodedTransactionData(targetEpoch);
-            const transaction = await makerTransactionFactory.newSignedTransactionAsync({ data });
+            const orders: SignedOrder[] = [];
+            const data = exchangeDataEncoder.encodeOrdersToExchangeData(constants.CANCEL_ORDERS_UP_TO, orders);
+            const transaction = makerTransactionFactory.newSignedTransaction(data);
             const transactionReceipt = await web3Wrapper.awaitTransactionSuccessAsync(
                 await coordinatorContract.executeTransaction.sendTransactionAsync(
                     transaction,
@@ -523,8 +515,8 @@ describe('Coordinator tests', () => {
             expect(cancelLogs.length).to.eq(1);
             const cancelLogArgs = (cancelLogs[0] as LogWithDecodedArgs<ExchangeCancelUpToEventArgs>).args;
             expect(cancelLogArgs.makerAddress).to.eq(makerAddress);
-            expect(cancelLogArgs.orderSenderAddress).to.eq(coordinatorContract.address);
-            expect(cancelLogArgs.orderEpoch).to.bignumber.eq(targetEpoch.plus(1));
+            expect(cancelLogArgs.senderAddress).to.eq(coordinatorContract.address);
+            expect(cancelLogArgs.orderEpoch).to.bignumber.eq(new BigNumber(1));
         });
     });
 });

contracts/coordinator/test/libs.ts

@@ -1,7 +1,7 @@
-import { chaiSetup, constants, provider, randomAddress, txDefaults, web3Wrapper } from '@0x/contracts-test-utils';
+import { addressUtils, chaiSetup, constants, provider, txDefaults, web3Wrapper } from '@0x/contracts-test-utils';
 import { BlockchainLifecycle } from '@0x/dev-utils';
 import { transactionHashUtils } from '@0x/order-utils';
-import { BigNumber, providerUtils } from '@0x/utils';
+import { BigNumber } from '@0x/utils';
 import * as chai from 'chai';
 
 import { artifacts, CoordinatorContract, hashUtils } from '../src';
@@ -12,8 +12,7 @@ const blockchainLifecycle = new BlockchainLifecycle(web3Wrapper);
 
 describe('Libs tests', () => {
     let coordinatorContract: CoordinatorContract;
-    let chainId: number;
-    const exchangeAddress = randomAddress();
+    const exchangeAddress = addressUtils.generatePseudoRandomAddress();
 
     before(async () => {
         await blockchainLifecycle.startAsync();
@@ -22,14 +21,12 @@ describe('Libs tests', () => {
         await blockchainLifecycle.revertAsync();
     });
     before(async () => {
-        chainId = await providerUtils.getChainIdAsync(provider);
         coordinatorContract = await CoordinatorContract.deployFrom0xArtifactAsync(
             artifacts.Coordinator,
             provider,
             txDefaults,
             artifacts,
             exchangeAddress,
-            new BigNumber(chainId),
         );
     });
     beforeEach(async () => {
@@ -42,14 +39,10 @@ describe('Libs tests', () => {
     describe('getTransactionHash', () => {
         it('should return the correct transaction hash', async () => {
             const tx = {
+                verifyingContractAddress: exchangeAddress,
                 salt: new BigNumber(0),
-                expirationTimeSeconds: new BigNumber(0),
                 signerAddress: constants.NULL_ADDRESS,
                 data: '0x1234',
-                domain: {
-                    verifyingContract: exchangeAddress,
-                    chainId,
-                },
             };
             const expectedTxHash = transactionHashUtils.getTransactionHashHex(tx);
             const txHash = await coordinatorContract.getTransactionHash.callAsync(tx);
@@ -60,15 +53,11 @@ describe('Libs tests', () => {
     describe('getApprovalHash', () => {
         it('should return the correct approval hash', async () => {
             const signedTx = {
+                verifyingContractAddress: exchangeAddress,
                 salt: new BigNumber(0),
-                expirationTimeSeconds: new BigNumber(0),
                 signerAddress: constants.NULL_ADDRESS,
                 data: '0x1234',
                 signature: '0x5678',
-                domain: {
-                    verifyingContract: exchangeAddress,
-                    chainId,
-                },
             };
             const approvalExpirationTimeSeconds = new BigNumber(0);
             const txOrigin = constants.NULL_ADDRESS;

contracts/coordinator/test/mixins.ts

@@ -1,11 +1,10 @@
-import { constants as exchangeConstants, exchangeDataEncoder, ExchangeFunctionName } from '@0x/contracts-exchange';
 import {
+    addressUtils,
     chaiSetup,
-    constants,
+    constants as devConstants,
     expectContractCallFailedAsync,
     getLatestBlockTimestampAsync,
     provider,
-    randomAddress,
     TransactionFactory,
     txDefaults,
     web3Wrapper,
@@ -13,18 +12,17 @@ import {
 import { BlockchainLifecycle } from '@0x/dev-utils';
 import { transactionHashUtils } from '@0x/order-utils';
 import { RevertReason, SignatureType, SignedOrder } from '@0x/types';
-import { BigNumber, LibBytesRevertErrors, providerUtils } from '@0x/utils';
+import { BigNumber } from '@0x/utils';
 import * as chai from 'chai';
 import * as ethUtil from 'ethereumjs-util';
 
-import { ApprovalFactory, artifacts, CoordinatorContract } from '../src';
+import { ApprovalFactory, artifacts, constants, CoordinatorContract, exchangeDataEncoder } from '../src';
 
 chaiSetup.configure();
 const expect = chai.expect;
 const blockchainLifecycle = new BlockchainLifecycle(web3Wrapper);
 
 describe('Mixins tests', () => {
-    let chainId: number;
     let transactionSignerAddress: string;
     let approvalSignerAddress1: string;
     let approvalSignerAddress2: string;
@@ -33,7 +31,7 @@ describe('Mixins tests', () => {
     let approvalFactory1: ApprovalFactory;
     let approvalFactory2: ApprovalFactory;
     let defaultOrder: SignedOrder;
-    const exchangeAddress = randomAddress();
+    const exchangeAddress = addressUtils.generatePseudoRandomAddress();
 
     before(async () => {
         await blockchainLifecycle.startAsync();
@@ -42,40 +40,36 @@ describe('Mixins tests', () => {
         await blockchainLifecycle.revertAsync();
     });
     before(async () => {
-        chainId = await providerUtils.getChainIdAsync(provider);
         mixins = await CoordinatorContract.deployFrom0xArtifactAsync(
             artifacts.Coordinator,
             provider,
             txDefaults,
             artifacts,
             exchangeAddress,
-            new BigNumber(chainId),
         );
         const accounts = await web3Wrapper.getAvailableAddressesAsync();
         [transactionSignerAddress, approvalSignerAddress1, approvalSignerAddress2] = accounts.slice(0, 3);
         defaultOrder = {
-            makerAddress: constants.NULL_ADDRESS,
-            takerAddress: constants.NULL_ADDRESS,
+            exchangeAddress: devConstants.NULL_ADDRESS,
+            makerAddress: devConstants.NULL_ADDRESS,
+            takerAddress: devConstants.NULL_ADDRESS,
             senderAddress: mixins.address,
             feeRecipientAddress: approvalSignerAddress1,
-            makerAssetData: constants.NULL_BYTES,
-            takerAssetData: constants.NULL_BYTES,
-            makerAssetAmount: constants.ZERO_AMOUNT,
-            takerAssetAmount: constants.ZERO_AMOUNT,
-            makerFee: constants.ZERO_AMOUNT,
-            takerFee: constants.ZERO_AMOUNT,
-            makerFeeAssetData: constants.NULL_BYTES,
-            takerFeeAssetData: constants.NULL_BYTES,
-            expirationTimeSeconds: constants.ZERO_AMOUNT,
-            salt: constants.ZERO_AMOUNT,
-            signature: constants.NULL_BYTES,
-            exchangeAddress: constants.NULL_ADDRESS,
-            chainId,
+            makerAssetData: devConstants.NULL_BYTES,
+            takerAssetData: devConstants.NULL_BYTES,
+            makerAssetAmount: devConstants.ZERO_AMOUNT,
+            takerAssetAmount: devConstants.ZERO_AMOUNT,
+            makerFee: devConstants.ZERO_AMOUNT,
+            takerFee: devConstants.ZERO_AMOUNT,
+            expirationTimeSeconds: devConstants.ZERO_AMOUNT,
+            salt: devConstants.ZERO_AMOUNT,
+            signature: devConstants.NULL_BYTES,
         };
-        const transactionSignerPrivateKey = constants.TESTRPC_PRIVATE_KEYS[accounts.indexOf(transactionSignerAddress)];
-        const approvalSignerPrivateKey1 = constants.TESTRPC_PRIVATE_KEYS[accounts.indexOf(approvalSignerAddress1)];
-        const approvalSignerPrivateKey2 = constants.TESTRPC_PRIVATE_KEYS[accounts.indexOf(approvalSignerAddress2)];
-        transactionFactory = new TransactionFactory(transactionSignerPrivateKey, exchangeAddress, chainId);
+        const transactionSignerPrivateKey =
+            devConstants.TESTRPC_PRIVATE_KEYS[accounts.indexOf(transactionSignerAddress)];
+        const approvalSignerPrivateKey1 = devConstants.TESTRPC_PRIVATE_KEYS[accounts.indexOf(approvalSignerAddress1)];
+        const approvalSignerPrivateKey2 = devConstants.TESTRPC_PRIVATE_KEYS[accounts.indexOf(approvalSignerAddress2)];
+        transactionFactory = new TransactionFactory(transactionSignerPrivateKey, exchangeAddress);
         approvalFactory1 = new ApprovalFactory(approvalSignerPrivateKey1, mixins.address);
         approvalFactory2 = new ApprovalFactory(approvalSignerPrivateKey2, mixins.address);
     });
@@ -88,105 +82,101 @@ describe('Mixins tests', () => {
 
     describe('getSignerAddress', () => {
         it('should return the correct address using the EthSign signature type', async () => {
-            const data = constants.NULL_BYTES;
-            const transaction = await transactionFactory.newSignedTransactionAsync({ data }, SignatureType.EthSign);
+            const data = devConstants.NULL_BYTES;
+            const transaction = transactionFactory.newSignedTransaction(data, SignatureType.EthSign);
             const transactionHash = transactionHashUtils.getTransactionHashHex(transaction);
             const signerAddress = await mixins.getSignerAddress.callAsync(transactionHash, transaction.signature);
             expect(transaction.signerAddress).to.eq(signerAddress);
         });
         it('should return the correct address using the EIP712 signature type', async () => {
-            const data = constants.NULL_BYTES;
-            const transaction = await transactionFactory.newSignedTransactionAsync({ data }, SignatureType.EIP712);
+            const data = devConstants.NULL_BYTES;
+            const transaction = transactionFactory.newSignedTransaction(data, SignatureType.EIP712);
             const transactionHash = transactionHashUtils.getTransactionHashHex(transaction);
             const signerAddress = await mixins.getSignerAddress.callAsync(transactionHash, transaction.signature);
             expect(transaction.signerAddress).to.eq(signerAddress);
         });
         it('should revert with with the Illegal signature type', async () => {
-            const data = constants.NULL_BYTES;
-            const transaction = await transactionFactory.newSignedTransactionAsync({ data });
+            const data = devConstants.NULL_BYTES;
+            const transaction = transactionFactory.newSignedTransaction(data);
             const illegalSignatureByte = ethUtil.toBuffer(SignatureType.Illegal).toString('hex');
             transaction.signature = `${transaction.signature.slice(
                 0,
                 transaction.signature.length - 2,
             )}${illegalSignatureByte}`;
             const transactionHash = transactionHashUtils.getTransactionHashHex(transaction);
-            expect(mixins.getSignerAddress.callAsync(transactionHash, transaction.signature)).to.be.rejectedWith(
+            expectContractCallFailedAsync(
+                mixins.getSignerAddress.callAsync(transactionHash, transaction.signature),
                 RevertReason.SignatureIllegal,
             );
         });
         it('should revert with with the Invalid signature type', async () => {
-            const data = constants.NULL_BYTES;
-            const transaction = await transactionFactory.newSignedTransactionAsync({ data });
+            const data = devConstants.NULL_BYTES;
+            const transaction = transactionFactory.newSignedTransaction(data);
             const invalidSignatureByte = ethUtil.toBuffer(SignatureType.Invalid).toString('hex');
             transaction.signature = `0x${invalidSignatureByte}`;
             const transactionHash = transactionHashUtils.getTransactionHashHex(transaction);
-            expect(mixins.getSignerAddress.callAsync(transactionHash, transaction.signature)).to.be.rejectedWith(
+            expectContractCallFailedAsync(
+                mixins.getSignerAddress.callAsync(transactionHash, transaction.signature),
                 RevertReason.SignatureInvalid,
             );
         });
         it("should revert with with a signature type that doesn't exist", async () => {
-            const data = constants.NULL_BYTES;
-            const transaction = await transactionFactory.newSignedTransactionAsync({ data });
+            const data = devConstants.NULL_BYTES;
+            const transaction = transactionFactory.newSignedTransaction(data);
             const invalidSignatureByte = '04';
             transaction.signature = `${transaction.signature.slice(
                 0,
                 transaction.signature.length - 2,
             )}${invalidSignatureByte}`;
             const transactionHash = transactionHashUtils.getTransactionHashHex(transaction);
-            expect(mixins.getSignerAddress.callAsync(transactionHash, transaction.signature)).to.be.rejectedWith(
+            expectContractCallFailedAsync(
+                mixins.getSignerAddress.callAsync(transactionHash, transaction.signature),
                 RevertReason.SignatureUnsupported,
             );
         });
     });
 
     describe('decodeOrdersFromFillData', () => {
-        for (const fnName of exchangeConstants.SINGLE_FILL_FN_NAMES) {
+        for (const fnName of constants.SINGLE_FILL_FN_NAMES) {
             it(`should correctly decode the orders for ${fnName} data`, async () => {
                 const orders = [defaultOrder];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
                 const decodedOrders = await mixins.decodeOrdersFromFillData.callAsync(data);
                 const decodedSignedOrders = decodedOrders.map(order => ({
                     ...order,
-                    signature: constants.NULL_BYTES,
-                    exchangeAddress: constants.NULL_ADDRESS,
-                    chainId,
+                    exchangeAddress: devConstants.NULL_ADDRESS,
+                    signature: devConstants.NULL_BYTES,
                 }));
                 expect(orders).to.deep.eq(decodedSignedOrders);
             });
         }
-        for (const fnName of exchangeConstants.BATCH_FILL_FN_NAMES) {
+        for (const fnName of constants.BATCH_FILL_FN_NAMES) {
             it(`should correctly decode the orders for ${fnName} data`, async () => {
                 const orders = [defaultOrder, defaultOrder];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
                 const decodedOrders = await mixins.decodeOrdersFromFillData.callAsync(data);
                 const decodedSignedOrders = decodedOrders.map(order => ({
                     ...order,
-                    signature: constants.NULL_BYTES,
-                    exchangeAddress: constants.NULL_ADDRESS,
-                    chainId,
+                    exchangeAddress: devConstants.NULL_ADDRESS,
+                    signature: devConstants.NULL_BYTES,
                 }));
                 expect(orders).to.deep.eq(decodedSignedOrders);
             });
         }
-        for (const fnName of exchangeConstants.MARKET_FILL_FN_NAMES) {
+        for (const fnName of constants.MARKET_FILL_FN_NAMES) {
             it(`should correctly decode the orders for ${fnName} data`, async () => {
                 const orders = [defaultOrder, defaultOrder];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
                 const decodedOrders = await mixins.decodeOrdersFromFillData.callAsync(data);
                 const decodedSignedOrders = decodedOrders.map(order => ({
                     ...order,
-                    signature: constants.NULL_BYTES,
-                    exchangeAddress: constants.NULL_ADDRESS,
-                    chainId,
+                    exchangeAddress: devConstants.NULL_ADDRESS,
+                    signature: devConstants.NULL_BYTES,
                 }));
                 expect(orders).to.deep.eq(decodedSignedOrders);
             });
         }
-        for (const fnName of [
-            ExchangeFunctionName.CancelOrder,
-            ExchangeFunctionName.BatchCancelOrders,
-            ExchangeFunctionName.CancelOrdersUpTo,
-        ]) {
+        for (const fnName of [constants.CANCEL_ORDER, constants.BATCH_CANCEL_ORDERS, constants.CANCEL_ORDERS_UP_TO]) {
             it(`should correctly decode the orders for ${fnName} data`, async () => {
                 const orders = [defaultOrder, defaultOrder];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
@@ -203,21 +193,19 @@ describe('Mixins tests', () => {
         });
         it('should revert if data is less than 4 bytes long', async () => {
             const data = '0x010203';
-            const expectedError = new LibBytesRevertErrors.InvalidByteOperationError(
-                LibBytesRevertErrors.InvalidByteOperationErrorCodes.LengthGreaterThanOrEqualsFourRequired,
-                new BigNumber(3), // the length of data
-                new BigNumber(4),
+            await expectContractCallFailedAsync(
+                mixins.decodeOrdersFromFillData.callAsync(data),
+                RevertReason.LibBytesGreaterOrEqualTo4LengthRequired,
             );
-            return expect(mixins.decodeOrdersFromFillData.callAsync(data)).to.revertWith(expectedError);
         });
     });
 
     describe('Single order approvals', () => {
-        for (const fnName of exchangeConstants.SINGLE_FILL_FN_NAMES) {
+        for (const fnName of constants.SINGLE_FILL_FN_NAMES) {
             it(`Should be successful: function=${fnName}, caller=tx_signer, senderAddress=[verifier], approval_sig=[approver1], expiration=[valid]`, async () => {
                 const orders = [defaultOrder];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await transactionFactory.newSignedTransactionAsync({ data });
+                const transaction = transactionFactory.newSignedTransaction(data);
                 const currentTimestamp = await getLatestBlockTimestampAsync();
                 const approvalExpirationTimeSeconds = new BigNumber(currentTimestamp).plus(constants.TIME_BUFFER);
                 const approval = approvalFactory1.newSignedApproval(
@@ -237,11 +225,11 @@ describe('Mixins tests', () => {
             it(`Should be successful: function=${fnName}, caller=tx_signer, senderAddress=[null], approval_sig=[approver1], expiration=[valid]`, async () => {
                 const order = {
                     ...defaultOrder,
-                    senderAddress: constants.NULL_ADDRESS,
+                    senderAddress: devConstants.NULL_ADDRESS,
                 };
                 const orders = [order];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await transactionFactory.newSignedTransactionAsync({ data });
+                const transaction = transactionFactory.newSignedTransaction(data);
                 const currentTimestamp = await getLatestBlockTimestampAsync();
                 const approvalExpirationTimeSeconds = new BigNumber(currentTimestamp).plus(constants.TIME_BUFFER);
                 const approval = approvalFactory1.newSignedApproval(
@@ -261,7 +249,7 @@ describe('Mixins tests', () => {
             it(`Should be successful: function=${fnName}, caller=approver1, senderAddress=[verifier], approval_sig=[], expiration=[]`, async () => {
                 const orders = [defaultOrder];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await transactionFactory.newSignedTransactionAsync({ data });
+                const transaction = transactionFactory.newSignedTransaction(data);
                 await mixins.assertValidCoordinatorApprovals.callAsync(
                     transaction,
                     approvalSignerAddress1,
@@ -276,7 +264,7 @@ describe('Mixins tests', () => {
             it(`Should be successful: function=${fnName}, caller=approver1, senderAddress=[verifier], approval_sig=[approver1], expiration=[invalid]`, async () => {
                 const orders = [defaultOrder];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await transactionFactory.newSignedTransactionAsync({ data });
+                const transaction = transactionFactory.newSignedTransaction(data);
                 const currentTimestamp = await getLatestBlockTimestampAsync();
                 const approvalExpirationTimeSeconds = new BigNumber(currentTimestamp).plus(constants.TIME_BUFFER);
                 const approval = approvalFactory1.newSignedApproval(
@@ -296,7 +284,7 @@ describe('Mixins tests', () => {
             it(`Should be successful: function=${fnName}, caller=approver1, senderAddress=[verifier], approval_sig=[], expiration=[]`, async () => {
                 const orders = [defaultOrder];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await transactionFactory.newSignedTransactionAsync({ data });
+                const transaction = transactionFactory.newSignedTransaction(data);
                 await mixins.assertValidCoordinatorApprovals.callAsync(
                     transaction,
                     approvalSignerAddress1,
@@ -311,7 +299,7 @@ describe('Mixins tests', () => {
             it(`Should revert: function=${fnName}, caller=tx_signer, senderAddress=[verifier], approval_sig=[invalid], expiration=[valid]`, async () => {
                 const orders = [defaultOrder];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await transactionFactory.newSignedTransactionAsync({ data });
+                const transaction = transactionFactory.newSignedTransaction(data);
                 const currentTimestamp = await getLatestBlockTimestampAsync();
                 const approvalExpirationTimeSeconds = new BigNumber(currentTimestamp).plus(constants.TIME_BUFFER);
                 const approval = approvalFactory1.newSignedApproval(
@@ -335,7 +323,7 @@ describe('Mixins tests', () => {
             it(`Should revert: function=${fnName}, caller=tx_signer, senderAddress=[verifier], approval_sig=[approver1], expiration=[invalid]`, async () => {
                 const orders = [defaultOrder];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await transactionFactory.newSignedTransactionAsync({ data });
+                const transaction = transactionFactory.newSignedTransaction(data);
                 const currentTimestamp = await getLatestBlockTimestampAsync();
                 const approvalExpirationTimeSeconds = new BigNumber(currentTimestamp).minus(constants.TIME_BUFFER);
                 const approval = approvalFactory1.newSignedApproval(
@@ -358,7 +346,7 @@ describe('Mixins tests', () => {
             it(`Should revert: function=${fnName}, caller=approver2, senderAddress=[verifier], approval_sig=[approver1], expiration=[valid]`, async () => {
                 const orders = [defaultOrder];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await transactionFactory.newSignedTransactionAsync({ data });
+                const transaction = transactionFactory.newSignedTransaction(data);
                 const currentTimestamp = await getLatestBlockTimestampAsync();
                 const approvalExpirationTimeSeconds = new BigNumber(currentTimestamp).plus(constants.TIME_BUFFER);
                 const approval = approvalFactory1.newSignedApproval(
@@ -382,14 +370,14 @@ describe('Mixins tests', () => {
     });
     describe('Batch order approvals', () => {
         for (const fnName of [
-            ...exchangeConstants.BATCH_FILL_FN_NAMES,
-            ...exchangeConstants.MARKET_FILL_FN_NAMES,
-            ExchangeFunctionName.MatchOrders,
+            ...constants.BATCH_FILL_FN_NAMES,
+            ...constants.MARKET_FILL_FN_NAMES,
+            constants.MATCH_ORDERS,
         ]) {
             it(`Should be successful: function=${fnName} caller=tx_signer, senderAddress=[verifier,verifier], feeRecipient=[approver1,approver1], approval_sig=[approver1], expiration=[valid]`, async () => {
                 const orders = [defaultOrder, defaultOrder];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await transactionFactory.newSignedTransactionAsync({ data });
+                const transaction = transactionFactory.newSignedTransaction(data);
                 const currentTimestamp = await getLatestBlockTimestampAsync();
                 const approvalExpirationTimeSeconds = new BigNumber(currentTimestamp).plus(constants.TIME_BUFFER);
                 const approval = approvalFactory1.newSignedApproval(
@@ -409,10 +397,10 @@ describe('Mixins tests', () => {
             it(`Should be successful: function=${fnName} caller=tx_signer, senderAddress=[null,null], feeRecipient=[approver1,approver1], approval_sig=[approver1], expiration=[valid]`, async () => {
                 const orders = [defaultOrder, defaultOrder].map(order => ({
                     ...order,
-                    senderAddress: constants.NULL_ADDRESS,
+                    senderAddress: devConstants.NULL_ADDRESS,
                 }));
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await transactionFactory.newSignedTransactionAsync({ data });
+                const transaction = transactionFactory.newSignedTransaction(data);
                 const currentTimestamp = await getLatestBlockTimestampAsync();
                 const approvalExpirationTimeSeconds = new BigNumber(currentTimestamp).plus(constants.TIME_BUFFER);
                 const approval = approvalFactory1.newSignedApproval(
@@ -432,10 +420,10 @@ describe('Mixins tests', () => {
             it(`Should be successful: function=${fnName} caller=tx_signer, senderAddress=[null,null], feeRecipient=[approver1,approver1], approval_sig=[], expiration=[]`, async () => {
                 const orders = [defaultOrder, defaultOrder].map(order => ({
                     ...order,
-                    senderAddress: constants.NULL_ADDRESS,
+                    senderAddress: devConstants.NULL_ADDRESS,
                 }));
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await transactionFactory.newSignedTransactionAsync({ data });
+                const transaction = transactionFactory.newSignedTransaction(data);
                 await mixins.assertValidCoordinatorApprovals.callAsync(
                     transaction,
                     transactionSignerAddress,
@@ -446,9 +434,9 @@ describe('Mixins tests', () => {
                 );
             });
             it(`Should be successful: function=${fnName} caller=tx_signer, senderAddress=[verifier,null], feeRecipient=[approver1,approver1], approval_sig=[approver1], expiration=[valid]`, async () => {
-                const orders = [defaultOrder, { ...defaultOrder, senderAddress: constants.NULL_ADDRESS }];
+                const orders = [defaultOrder, { ...defaultOrder, senderAddress: devConstants.NULL_ADDRESS }];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await transactionFactory.newSignedTransactionAsync({ data });
+                const transaction = transactionFactory.newSignedTransaction(data);
                 const currentTimestamp = await getLatestBlockTimestampAsync();
                 const approvalExpirationTimeSeconds = new BigNumber(currentTimestamp).plus(constants.TIME_BUFFER);
                 const approval = approvalFactory1.newSignedApproval(
@@ -468,7 +456,7 @@ describe('Mixins tests', () => {
             it(`Should be successful: function=${fnName} caller=tx_signer, senderAddress=[verifier,verifier], feeRecipient=[approver1,approver2], approval_sig=[approver1,approver2], expiration=[valid,valid]`, async () => {
                 const orders = [defaultOrder, { ...defaultOrder, feeRecipientAddress: approvalSignerAddress2 }];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await transactionFactory.newSignedTransactionAsync({ data });
+                const transaction = transactionFactory.newSignedTransaction(data);
                 const currentTimestamp = await getLatestBlockTimestampAsync();
                 const approvalExpirationTimeSeconds = new BigNumber(currentTimestamp).plus(constants.TIME_BUFFER);
                 const approval1 = approvalFactory1.newSignedApproval(
@@ -493,7 +481,7 @@ describe('Mixins tests', () => {
             it(`Should be successful: function=${fnName} caller=approver1, senderAddress=[verifier,verifier], feeRecipient=[approver1,approver1], approval_sig=[], expiration=[]`, async () => {
                 const orders = [defaultOrder, defaultOrder];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await transactionFactory.newSignedTransactionAsync({ data });
+                const transaction = transactionFactory.newSignedTransaction(data);
                 await mixins.assertValidCoordinatorApprovals.callAsync(
                     transaction,
                     approvalSignerAddress1,
@@ -506,7 +494,7 @@ describe('Mixins tests', () => {
             it(`Should revert: function=${fnName} caller=approver1, senderAddress=[verifier,verifier], feeRecipient=[approver1,approver2], approval_sig=[approver2], expiration=[valid]`, async () => {
                 const orders = [defaultOrder, { ...defaultOrder, feeRecipientAddress: approvalSignerAddress2 }];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await transactionFactory.newSignedTransactionAsync({ data });
+                const transaction = transactionFactory.newSignedTransaction(data);
                 const currentTimestamp = await getLatestBlockTimestampAsync();
                 const approvalExpirationTimeSeconds = new BigNumber(currentTimestamp).plus(constants.TIME_BUFFER);
                 const approval2 = approvalFactory2.newSignedApproval(
@@ -529,7 +517,7 @@ describe('Mixins tests', () => {
             it(`Should revert: function=${fnName} caller=tx_signer, senderAddress=[verifier,verifier], feeRecipient=[approver1, approver1], approval_sig=[], expiration=[]`, async () => {
                 const orders = [defaultOrder, defaultOrder];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await transactionFactory.newSignedTransactionAsync({ data });
+                const transaction = transactionFactory.newSignedTransaction(data);
                 expectContractCallFailedAsync(
                     mixins.assertValidCoordinatorApprovals.callAsync(
                         transaction,
@@ -545,7 +533,7 @@ describe('Mixins tests', () => {
             it(`Should revert: function=${fnName} caller=tx_signer, senderAddress=[verifier,verifier], feeRecipient=[approver1, approver1], approval_sig=[invalid], expiration=[valid]`, async () => {
                 const orders = [defaultOrder, defaultOrder];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await transactionFactory.newSignedTransactionAsync({ data });
+                const transaction = transactionFactory.newSignedTransaction(data);
                 const currentTimestamp = await getLatestBlockTimestampAsync();
                 const approvalExpirationTimeSeconds = new BigNumber(currentTimestamp).plus(constants.TIME_BUFFER);
                 const approval = approvalFactory1.newSignedApproval(
@@ -569,7 +557,7 @@ describe('Mixins tests', () => {
             it(`Should revert: function=${fnName} caller=tx_signer, senderAddress=[verifier,verifier], feeRecipient=[approver1, approver2], approval_sig=[valid,invalid], expiration=[valid,valid]`, async () => {
                 const orders = [defaultOrder, { ...defaultOrder, feeRecipientAddress: approvalSignerAddress2 }];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await transactionFactory.newSignedTransactionAsync({ data });
+                const transaction = transactionFactory.newSignedTransaction(data);
                 const currentTimestamp = await getLatestBlockTimestampAsync();
                 const approvalExpirationTimeSeconds = new BigNumber(currentTimestamp).plus(constants.TIME_BUFFER);
                 const approval1 = approvalFactory1.newSignedApproval(
@@ -598,7 +586,7 @@ describe('Mixins tests', () => {
             it(`Should revert: function=${fnName} caller=approver1, senderAddress=[verifier,verifier], feeRecipient=[approver1, approver2], approval_sig=[invalid], expiration=[valid]`, async () => {
                 const orders = [defaultOrder, { ...defaultOrder, feeRecipientAddress: approvalSignerAddress2 }];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await transactionFactory.newSignedTransactionAsync({ data });
+                const transaction = transactionFactory.newSignedTransaction(data);
                 const currentTimestamp = await getLatestBlockTimestampAsync();
                 const approvalExpirationTimeSeconds = new BigNumber(currentTimestamp).plus(constants.TIME_BUFFER);
                 const approval2 = approvalFactory2.newSignedApproval(
@@ -622,7 +610,7 @@ describe('Mixins tests', () => {
             it(`Should revert: function=${fnName} caller=tx_signer, senderAddress=[verifier,verifier], feeRecipient=[approver1, approver2], approval_sig=[valid,valid], expiration=[valid,invalid]`, async () => {
                 const orders = [defaultOrder, { ...defaultOrder, feeRecipientAddress: approvalSignerAddress2 }];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await transactionFactory.newSignedTransactionAsync({ data });
+                const transaction = transactionFactory.newSignedTransaction(data);
                 const currentTimestamp = await getLatestBlockTimestampAsync();
                 const approvalExpirationTimeSeconds1 = new BigNumber(currentTimestamp).plus(constants.TIME_BUFFER);
                 const approvalExpirationTimeSeconds2 = new BigNumber(currentTimestamp).minus(constants.TIME_BUFFER);
@@ -651,7 +639,7 @@ describe('Mixins tests', () => {
             it(`Should revert: function=${fnName} caller=approver1, senderAddress=[verifier,verifier], feeRecipient=[approver1, approver2], approval_sig=[valid], expiration=[invalid]`, async () => {
                 const orders = [defaultOrder, { ...defaultOrder, feeRecipientAddress: approvalSignerAddress2 }];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await transactionFactory.newSignedTransactionAsync({ data });
+                const transaction = transactionFactory.newSignedTransaction(data);
                 const currentTimestamp = await getLatestBlockTimestampAsync();
                 const approvalExpirationTimeSeconds = new BigNumber(currentTimestamp).minus(constants.TIME_BUFFER);
                 const approval2 = approvalFactory2.newSignedApproval(
@@ -674,7 +662,7 @@ describe('Mixins tests', () => {
             it(`Should revert: function=${fnName} caller=approver2, senderAddress=[verifier,verifier], feeRecipient=[approver1, approver1], approval_sig=[valid], expiration=[valid]`, async () => {
                 const orders = [defaultOrder, defaultOrder];
                 const data = exchangeDataEncoder.encodeOrdersToExchangeData(fnName, orders);
-                const transaction = await transactionFactory.newSignedTransactionAsync({ data });
+                const transaction = transactionFactory.newSignedTransaction(data);
                 const currentTimestamp = await getLatestBlockTimestampAsync();
                 const approvalExpirationTimeSeconds = new BigNumber(currentTimestamp).plus(constants.TIME_BUFFER);
                 const approval1 = approvalFactory1.newSignedApproval(
@@ -699,8 +687,8 @@ describe('Mixins tests', () => {
     describe('cancels', () => {
         it('should allow the tx signer to call `cancelOrder` without approval', async () => {
             const orders = [defaultOrder];
-            const data = exchangeDataEncoder.encodeOrdersToExchangeData(ExchangeFunctionName.CancelOrder, orders);
-            const transaction = await transactionFactory.newSignedTransactionAsync({ data });
+            const data = exchangeDataEncoder.encodeOrdersToExchangeData(constants.CANCEL_ORDER, orders);
+            const transaction = transactionFactory.newSignedTransaction(data);
             await mixins.assertValidCoordinatorApprovals.callAsync(
                 transaction,
                 transactionSignerAddress,
@@ -712,8 +700,8 @@ describe('Mixins tests', () => {
         });
         it('should allow the tx signer to call `batchCancelOrders` without approval', async () => {
             const orders = [defaultOrder, defaultOrder];
-            const data = exchangeDataEncoder.encodeOrdersToExchangeData(ExchangeFunctionName.BatchCancelOrders, orders);
-            const transaction = await transactionFactory.newSignedTransactionAsync({ data });
+            const data = exchangeDataEncoder.encodeOrdersToExchangeData(constants.BATCH_CANCEL_ORDERS, orders);
+            const transaction = transactionFactory.newSignedTransaction(data);
             await mixins.assertValidCoordinatorApprovals.callAsync(
                 transaction,
                 transactionSignerAddress,
@@ -724,8 +712,9 @@ describe('Mixins tests', () => {
             );
         });
         it('should allow the tx signer to call `cancelOrdersUpTo` without approval', async () => {
-            const data = exchangeDataEncoder.encodeOrdersToExchangeData(ExchangeFunctionName.CancelOrdersUpTo);
-            const transaction = await transactionFactory.newSignedTransactionAsync({ data });
+            const orders: SignedOrder[] = [];
+            const data = exchangeDataEncoder.encodeOrdersToExchangeData(constants.CANCEL_ORDERS_UP_TO, orders);
+            const transaction = transactionFactory.newSignedTransaction(data);
             await mixins.assertValidCoordinatorApprovals.callAsync(
                 transaction,
                 transactionSignerAddress,

contracts/coordinator/test/utils/approval_factory.ts

@@ -9,9 +9,9 @@ export class ApprovalFactory {
     private readonly _privateKey: Buffer;
     private readonly _verifyingContractAddress: string;
 
-    constructor(privateKey: Buffer, verifyingContract: string) {
+    constructor(privateKey: Buffer, verifyingContractAddress: string) {
         this._privateKey = privateKey;
-        this._verifyingContractAddress = verifyingContract;
+        this._verifyingContractAddress = verifyingContractAddress;
     }
 
     public newSignedApproval(

contracts/coordinator/test/utils/constants.ts

@@ -0,0 +1,12 @@
+import { BigNumber } from '@0x/utils';
+
+export const constants = {
+    SINGLE_FILL_FN_NAMES: ['fillOrder', 'fillOrKillOrder', 'fillOrderNoThrow'],
+    BATCH_FILL_FN_NAMES: ['batchFillOrders', 'batchFillOrKillOrders', 'batchFillOrdersNoThrow'],
+    MARKET_FILL_FN_NAMES: ['marketBuyOrders', 'marketBuyOrdersNoThrow', 'marketSellOrders', 'marketSellOrdersNoThrow'],
+    MATCH_ORDERS: 'matchOrders',
+    CANCEL_ORDER: 'cancelOrder',
+    BATCH_CANCEL_ORDERS: 'batchCancelOrders',
+    CANCEL_ORDERS_UP_TO: 'cancelOrdersUpTo',
+    TIME_BUFFER: new BigNumber(1000),
+};

contracts/coordinator/test/utils/exchange_data_encoder.ts

@@ -0,0 +1,47 @@
+import { IExchangeContract } from '@0x/contracts-exchange';
+import { constants as devConstants, provider } from '@0x/contracts-test-utils';
+import { SignedOrder } from '@0x/types';
+
+import { constants } from './index';
+
+export const exchangeDataEncoder = {
+    encodeOrdersToExchangeData(fnName: string, orders: SignedOrder[]): string {
+        const exchangeInstance = new IExchangeContract(devConstants.NULL_ADDRESS, provider);
+        let data;
+        if (constants.SINGLE_FILL_FN_NAMES.indexOf(fnName) !== -1) {
+            data = (exchangeInstance as any)[fnName].getABIEncodedTransactionData(
+                orders[0],
+                orders[0].takerAssetAmount,
+                orders[0].signature,
+            );
+        } else if (constants.BATCH_FILL_FN_NAMES.indexOf(fnName) !== -1) {
+            data = (exchangeInstance as any)[fnName].getABIEncodedTransactionData(
+                orders,
+                orders.map(order => order.takerAssetAmount),
+                orders.map(order => order.signature),
+            );
+        } else if (constants.MARKET_FILL_FN_NAMES.indexOf(fnName) !== -1) {
+            data = (exchangeInstance as any)[fnName].getABIEncodedTransactionData(
+                orders,
+                orders.map(order => order.takerAssetAmount).reduce((prev, curr) => prev.plus(curr)),
+                orders.map(order => order.signature),
+            );
+        } else if (fnName === constants.MATCH_ORDERS) {
+            data = exchangeInstance.matchOrders.getABIEncodedTransactionData(
+                orders[0],
+                orders[1],
+                orders[0].signature,
+                orders[1].signature,
+            );
+        } else if (fnName === constants.CANCEL_ORDER) {
+            data = exchangeInstance.cancelOrder.getABIEncodedTransactionData(orders[0]);
+        } else if (fnName === constants.BATCH_CANCEL_ORDERS) {
+            data = exchangeInstance.batchCancelOrders.getABIEncodedTransactionData(orders);
+        } else if (fnName === constants.CANCEL_ORDERS_UP_TO) {
+            data = exchangeInstance.cancelOrdersUpTo.getABIEncodedTransactionData(devConstants.ZERO_AMOUNT);
+        } else {
+            throw new Error(`Error: ${fnName} not a supported function`);
+        }
+        return data;
+    },
+};

contracts/coordinator/test/utils/hash_utils.ts

@@ -6,13 +6,13 @@ import * as _ from 'lodash';
 export const hashUtils = {
     getApprovalHashBuffer(
         transaction: SignedZeroExTransaction,
-        verifyingContract: string,
+        verifyingContractAddress: string,
         txOrigin: string,
         approvalExpirationTimeSeconds: BigNumber,
     ): Buffer {
         const typedData = eip712Utils.createCoordinatorApprovalTypedData(
             transaction,
-            verifyingContract,
+            verifyingContractAddress,
             txOrigin,
             approvalExpirationTimeSeconds,
         );
@@ -21,12 +21,12 @@ export const hashUtils = {
     },
     getApprovalHashHex(
         transaction: SignedZeroExTransaction,
-        verifyingContract: string,
+        verifyingContractAddress: string,
         txOrigin: string,
         approvalExpirationTimeSeconds: BigNumber,
     ): string {
         const hashHex = `0x${hashUtils
-            .getApprovalHashBuffer(transaction, verifyingContract, txOrigin, approvalExpirationTimeSeconds)
+            .getApprovalHashBuffer(transaction, verifyingContractAddress, txOrigin, approvalExpirationTimeSeconds)
             .toString('hex')}`;
         return hashHex;
     },

contracts/coordinator/test/utils/index.ts

@@ -1,3 +1,5 @@
 export { hashUtils } from './hash_utils';
 export { ApprovalFactory } from './approval_factory';
+export { constants } from './constants';
+export { exchangeDataEncoder } from './exchange_data_encoder';
 export * from './types';

contracts/coordinator/truffle-config.js

@@ -1,96 +0,0 @@
-/**
- * Use this file to configure your truffle project. It's seeded with some
- * common settings for different networks and features like migrations,
- * compilation and testing. Uncomment the ones you need or modify
- * them to suit your project as necessary.
- *
- * More information about configuration can be found at:
- *
- * truffleframework.com/docs/advanced/configuration
- *
- * To deploy via Infura you'll need a wallet provider (like truffle-hdwallet-provider)
- * to sign your transactions before they're sent to a remote public node. Infura accounts
- * are available for free at: infura.io/register.
- *
- * You'll also need a mnemonic - the twelve word phrase the wallet uses to generate
- * public/private key pairs. If you're publishing your code to GitHub make sure you load this
- * phrase from a file you've .gitignored so it doesn't accidentally become public.
- *
- */
-
-// const HDWalletProvider = require('truffle-hdwallet-provider');
-// const infuraKey = "fj4jll3k.....";
-//
-// const fs = require('fs');
-// const mnemonic = fs.readFileSync(".secret").toString().trim();
-
-module.exports = {
-    /**
-     * Networks define how you connect to your ethereum client and let you set the
-     * defaults web3 uses to send transactions. If you don't specify one truffle
-     * will spin up a development blockchain for you on port 9545 when you
-     * run `develop` or `test`. You can ask a truffle command to use a specific
-     * network from the command line, e.g
-     *
-     * $ truffle test --network <network-name>
-     */
-
-    networks: {
-        // Useful for testing. The `development` name is special - truffle uses it by default
-        // if it's defined here and no other network is specified at the command line.
-        // You should run a client (like ganache-cli, geth or parity) in a separate terminal
-        // tab if you use this network and you must also set the `host`, `port` and `network_id`
-        // options below to some value.
-        //
-        // development: {
-        //  host: "127.0.0.1",     // Localhost (default: none)
-        //  port: 8545,            // Standard Ethereum port (default: none)
-        //  network_id: "*",       // Any network (default: none)
-        // },
-        // Another network with more advanced options...
-        // advanced: {
-        // port: 8777,             // Custom port
-        // network_id: 1342,       // Custom network
-        // gas: 8500000,           // Gas sent with each transaction (default: ~6700000)
-        // gasPrice: 20000000000,  // 20 gwei (in wei) (default: 100 gwei)
-        // from: <address>,        // Account to send txs from (default: accounts[0])
-        // websockets: true        // Enable EventEmitter interface for web3 (default: false)
-        // },
-        // Useful for deploying to a public network.
-        // NB: It's important to wrap the provider as a function.
-        // ropsten: {
-        // provider: () => new HDWalletProvider(mnemonic, `https://ropsten.infura.io/v3/YOUR-PROJECT-ID`),
-        // network_id: 3,       // Ropsten's id
-        // gas: 5500000,        // Ropsten has a lower block limit than mainnet
-        // confirmations: 2,    // # of confs to wait between deployments. (default: 0)
-        // timeoutBlocks: 200,  // # of blocks before a deployment times out  (minimum/default: 50)
-        // skipDryRun: true     // Skip dry run before migrations? (default: false for public nets )
-        // },
-        // Useful for private networks
-        // private: {
-        // provider: () => new HDWalletProvider(mnemonic, `https://network.io`),
-        // network_id: 2111,   // This network is yours, in the cloud.
-        // production: true    // Treats this network as if it was a public net. (default: false)
-        // }
-    },
-
-    // Set default mocha options here, use special reporters etc.
-    mocha: {
-        // timeout: 100000
-    },
-
-    // Configure your compilers
-    compilers: {
-        solc: {
-            version: '0.5.9',
-            settings: {
-                evmVersion: 'constantinople',
-                optimizer: {
-                    enabled: true,
-                    runs: 1000000,
-                    details: { yul: true, deduplicate: true, cse: true, constantOptimizer: true },
-                },
-            },
-        },
-    },
-};

contracts/dev-utils/CHANGELOG.json

@@ -1,44 +1,4 @@
 [
-    {
-        "version": "0.1.0-beta.0",
-        "changes": [
-            {
-                "note": "Use built in selectors instead of hard coded constants",
-                "pr": 2055
-            },
-            {
-                "note": "Compile and export all contracts, artifacts, and wrappers by default",
-                "pr": 2055
-            },
-            {
-                "note": "Add `marketBuy/SellOrdersNoThrow` and `marketBuy/SellOrdersFillOrKill` to `LibTransactionDecoder`.",
-                "pr": 2075
-            },
-            {
-                "note": "`run_mocha` package script runs with `UNLIMITED_CONTRACT_SIZE=true` environment variable.",
-                "pr": 2075
-            }
-        ],
-        "timestamp": 1570135330
-    },
-    {
-        "timestamp": 1568744790,
-        "version": "0.0.10",
-        "changes": [
-            {
-                "note": "Dependencies updated"
-            }
-        ]
-    },
-    {
-        "timestamp": 1567521715,
-        "version": "0.0.9",
-        "changes": [
-            {
-                "note": "Dependencies updated"
-            }
-        ]
-    },
     {
         "timestamp": 1566446343,
         "version": "0.0.8",
@@ -138,14 +98,6 @@
             {
                 "note": "Add support for StaticCallProxy",
                 "pr": 1863
-            },
-            {
-                "note": "Add `OrderTransferSimulationUtils` contract for simulating order transfers on-chain",
-                "pr": 1868
-            },
-            {
-                "note": "Updated to use the new rich error pattern from @0x/contracts-exchange",
-                "pr": 1913
             }
         ]
     }

contracts/dev-utils/CHANGELOG.md

@@ -5,21 +5,6 @@ Edit the package's CHANGELOG.json file only.
 
 CHANGELOG
 
-## v0.1.0-beta.0 - _October 3, 2019_
-
-    * Use built in selectors instead of hard coded constants (#2055)
-    * Compile and export all contracts, artifacts, and wrappers by default (#2055)
-    * Add `marketBuy/SellOrdersNoThrow` and `marketBuy/SellOrdersFillOrKill` to `LibTransactionDecoder`. (#2075)
-    * `run_mocha` package script runs with `UNLIMITED_CONTRACT_SIZE=true` environment variable. (#2075)
-
-## v0.0.10 - _September 17, 2019_
-
-    * Dependencies updated
-
-## v0.0.9 - _September 3, 2019_
-
-    * Dependencies updated
-
 ## v0.0.8 - _August 22, 2019_
 
     * Dependencies updated
@@ -60,5 +45,3 @@ CHANGELOG
     * Refactor `LibAssetData` balance/allowance checks to never revert (#1848)
     * Refactor `OrderValidationUtils` to calculate `fillableTakerAssetAmount` (#1848)
     * Add support for StaticCallProxy (#1863)
-    * Add `OrderTransferSimulationUtils` contract for simulating order transfers on-chain (#1868)
-    * Updated to use the new rich error pattern from @0x/contracts-exchange (#1913)

contracts/dev-utils/README.md

@@ -1,6 +1,6 @@
 ## Dev-Utils
 
-This package implements various utilities for developers. For example, the `DevUtils` contract can query batches of balances or allowances given some `assetData`, can validate batches of orders, and can decode 0x-specific calldata. Addresses of the deployed contracts can be found in this 0x [guide](https://0x.org/docs/guides/0x-cheat-sheet) or the [DEPLOYS](./DEPLOYS.json) file within this package.
+This package implements various utilities for developers. For example, the `DevUtils` contract can query batches of balances or allowances given some `assetData`, can validate batches of orders, and can decode 0x-specific calldata. Addresses of the deployed contracts can be found in the 0x [wiki](https://0xproject.com/wiki#Deployed-Addresses) or the [DEPLOYS](./DEPLOYS.json) file within this package.
 
 ## Installation
 
@@ -12,7 +12,7 @@ npm install @0x/contracts-dev-utils --save
 
 ## Bug bounty
 
-A bug bounty for the 2.0.0 contracts is ongoing! Instructions can be found [here](https://0x.org/docs/guides/bug-bounty-program).
+A bug bounty for the 2.0.0 contracts is ongoing! Instructions can be found [here](https://0xproject.com/wiki#Bug-Bounty).
 
 ## Contributing
 

contracts/dev-utils/compiler.json

@@ -7,7 +7,7 @@
         "evmVersion": "constantinople",
         "optimizer": {
             "enabled": true,
-            "runs": 10000,
+            "runs": 1000000,
             "details": { "yul": true, "deduplicate": true, "cse": true, "constantOptimizer": true }
         },
         "outputSelection": {
@@ -22,5 +22,11 @@
                 ]
             }
         }
-    }
+    },
+    "contracts": [
+        "src/DevUtils.sol",
+        "src/LibAssetData.sol",
+        "src/LibTransactionDecoder.sol",
+        "src/EthBalanceChecker.sol"
+    ]
 }

contracts/dev-utils/contracts/src/DevUtils.sol

@@ -1,6 +1,6 @@
 /*
 
-  Copyright 2019 ZeroEx Intl.
+  Copyright 2018 ZeroEx Intl.
 
   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
@@ -20,7 +20,6 @@ pragma solidity ^0.5.5;
 pragma experimental ABIEncoderV2;
 
 import "./OrderValidationUtils.sol";
-import "./OrderTransferSimulationUtils.sol";
 import "./LibTransactionDecoder.sol";
 import "./EthBalanceChecker.sol";
 
@@ -29,12 +28,10 @@ import "./EthBalanceChecker.sol";
 contract DevUtils is
     OrderValidationUtils,
     LibTransactionDecoder,
-    EthBalanceChecker,
-    OrderTransferSimulationUtils
+    EthBalanceChecker
 {
-    constructor (address _exchange)
+    constructor (address _exchange, bytes memory _zrxAssetData)
         public
-        OrderValidationUtils(_exchange)
-        OrderTransferSimulationUtils(_exchange)
+        OrderValidationUtils(_exchange, _zrxAssetData)
     {}
 }

contracts/dev-utils/contracts/src/EthBalanceChecker.sol

@@ -1,6 +1,6 @@
 /*
 
-  Copyright 2019 ZeroEx Intl.
+  Copyright 2018 ZeroEx Intl.
 
   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.

contracts/dev-utils/contracts/src/LibAssetData.sol

@@ -20,19 +20,32 @@ pragma solidity ^0.5.5;
 pragma experimental ABIEncoderV2;
 
 import "@0x/contracts-utils/contracts/src/LibBytes.sol";
+import "@0x/contracts-asset-proxy/contracts/src/libs/LibAssetProxyIds.sol";
 import "@0x/contracts-exchange/contracts/src/interfaces/IExchange.sol";
-import "@0x/contracts-asset-proxy/contracts/src/interfaces/IAssetData.sol";
-import "@0x/contracts-asset-proxy/contracts/src/interfaces/IAssetProxy.sol";
-import "@0x/contracts-erc20/contracts/src/interfaces/IERC20Token.sol";
-import "@0x/contracts-erc721/contracts/src/interfaces/IERC721Token.sol";
-import "@0x/contracts-erc1155/contracts/src/interfaces/IERC1155.sol";
 
 
-contract LibAssetData {
-
+contract LibAssetData is
+    LibAssetProxyIds
+{
     // 2^256 - 1
     uint256 constant internal _MAX_UINT256 = uint256(-1);
 
+    // ERC20 selectors
+    bytes4 constant internal _ERC20_BALANCE_OF_SELECTOR = 0x70a08231;
+    bytes4 constant internal _ERC20_ALLOWANCE_SELECTOR = 0xdd62ed3e;
+
+    // ERC721 selectors
+    bytes4 constant internal _ERC721_OWNER_OF_SELECTOR = 0x6352211e;
+    bytes4 constant internal _ERC721_IS_APPROVED_FOR_ALL_SELECTOR = 0xe985e9c5;
+    bytes4 constant internal _ERC721_GET_APPROVED_SELECTOR = 0x081812fc;
+
+    // ERC1155 selectors
+    bytes4 constant internal _ERC1155_BALANCE_OF_SELECTOR = 0x00fdd58e;
+    bytes4 constant internal _ERC1155_IS_APPROVED_FOR_ALL_SELECTOR = 0xe985e9c5;
+
+    // `transferFrom` selector for all AssetProxy contracts
+    bytes4 constant internal _ASSET_PROXY_TRANSFER_FROM_SELECTOR = 0xa85e59e4;
+
     using LibBytes for bytes;
 
     // solhint-disable var-name-mixedcase
@@ -47,10 +60,10 @@ contract LibAssetData {
         public
     {
         _EXCHANGE = IExchange(_exchange);
-        _ERC20_PROXY_ADDRESS = _EXCHANGE.getAssetProxy(IAssetData(address(0)).ERC20Token.selector);
-        _ERC721_PROXY_ADDRESS = _EXCHANGE.getAssetProxy(IAssetData(address(0)).ERC721Token.selector);
-        _ERC1155_PROXY_ADDRESS = _EXCHANGE.getAssetProxy(IAssetData(address(0)).ERC1155Assets.selector);
-        _STATIC_CALL_PROXY_ADDRESS = _EXCHANGE.getAssetProxy(IAssetData(address(0)).StaticCall.selector);
+        _ERC20_PROXY_ADDRESS = _EXCHANGE.getAssetProxy(ERC20_PROXY_ID);
+        _ERC721_PROXY_ADDRESS = _EXCHANGE.getAssetProxy(ERC721_PROXY_ID);
+        _ERC1155_PROXY_ADDRESS = _EXCHANGE.getAssetProxy(ERC1155_PROXY_ID);
+        _STATIC_CALL_PROXY_ADDRESS = _EXCHANGE.getAssetProxy(STATIC_CALL_PROXY_ID);
     }
 
     /// @dev Returns the owner's balance of the assets(s) specified in
@@ -68,33 +81,23 @@ contract LibAssetData {
         // Get id of AssetProxy contract
         bytes4 assetProxyId = assetData.readBytes4(0);
 
-        if (assetProxyId == IAssetData(address(0)).ERC20Token.selector) {
+        if (assetProxyId == ERC20_PROXY_ID) {
             // Get ERC20 token address
             address tokenAddress = assetData.readAddress(16);
 
             // Encode data for `balanceOf(ownerAddress)`
-            bytes memory balanceOfData = abi.encodeWithSelector(
-                IERC20Token(address(0)).balanceOf.selector,
-                ownerAddress
-            );
+            bytes memory balanceOfData = abi.encodeWithSelector(_ERC20_BALANCE_OF_SELECTOR, ownerAddress);
 
             // Query balance
             (bool success, bytes memory returnData) = tokenAddress.staticcall(balanceOfData);
             balance = success && returnData.length == 32 ? returnData.readUint256(0) : 0;
-        } else if (assetProxyId == IAssetData(address(0)).ERC721Token.selector) {
+        } else if (assetProxyId == ERC721_PROXY_ID) {
             // Get ERC721 token address and id
             (, address tokenAddress, uint256 tokenId) = decodeERC721AssetData(assetData);
 
             // Check if id is owned by ownerAddress
-            bytes memory ownerOfCalldata = abi.encodeWithSelector(
-                IERC721Token(address(0)).ownerOf.selector,
-                tokenId
-            );
-
-            (bool success, bytes memory returnData) = tokenAddress.staticcall(ownerOfCalldata);
-            address currentOwnerAddress = (success && returnData.length == 32) ? returnData.readAddress(12) : address(0);
-            balance = currentOwnerAddress == ownerAddress ? 1 : 0;
-        } else if (assetProxyId == IAssetData(address(0)).ERC1155Assets.selector) {
+            balance = getERC721TokenOwner(tokenAddress, tokenId) == ownerAddress ? 1 : 0;
+        } else if (assetProxyId == ERC1155_PROXY_ID) {
             // Get ERC1155 token address, array of ids, and array of values
             (, address tokenAddress, uint256[] memory tokenIds, uint256[] memory tokenValues,) = decodeERC1155AssetData(assetData);
 
@@ -102,7 +105,7 @@ contract LibAssetData {
             for (uint256 i = 0; i != length; i++) {
                 // Encode data for `balanceOf(ownerAddress, tokenIds[i])
                 bytes memory balanceOfData = abi.encodeWithSelector(
-                    IERC1155(address(0)).balanceOf.selector,
+                    _ERC1155_BALANCE_OF_SELECTOR,
                     ownerAddress,
                     tokenIds[i]
                 );
@@ -117,10 +120,10 @@ contract LibAssetData {
                     balance = scaledBalance;
                 }
             }
-        } else if (assetProxyId == IAssetData(address(0)).StaticCall.selector) {
+        } else if (assetProxyId == STATIC_CALL_PROXY_ID) {
             // Encode data for `staticCallProxy.transferFrom(assetData,...)`
             bytes memory transferFromData = abi.encodeWithSelector(
-                IAssetProxy(address(0)).transferFrom.selector,
+                _ASSET_PROXY_TRANSFER_FROM_SELECTOR,
                 assetData,
                 address(0),  // `from` address is not used
                 address(0),  // `to` address is not used
@@ -132,7 +135,7 @@ contract LibAssetData {
 
             // Success means that the staticcall can be made an unlimited amount of times
             balance = success ? _MAX_UINT256 : 0;
-        } else if (assetProxyId == IAssetData(address(0)).MultiAsset.selector) {
+        } else if (assetProxyId == MULTI_ASSET_PROXY_ID) {
             // Get array of values and array of assetDatas
             (, uint256[] memory assetAmounts, bytes[] memory nestedAssetData) = decodeMultiAssetData(assetData);
 
@@ -187,7 +190,7 @@ contract LibAssetData {
         // Get id of AssetProxy contract
         bytes4 assetProxyId = assetData.readBytes4(0);
 
-        if (assetProxyId == IAssetData(address(0)).MultiAsset.selector) {
+        if (assetProxyId == MULTI_ASSET_PROXY_ID) {
             // Get array of values and array of assetDatas
             (, uint256[] memory amounts, bytes[] memory nestedAssetData) = decodeMultiAssetData(assetData);
 
@@ -205,13 +208,13 @@ contract LibAssetData {
             return allowance;
         }
 
-        if (assetProxyId == IAssetData(address(0)).ERC20Token.selector) {
+        if (assetProxyId == ERC20_PROXY_ID) {
             // Get ERC20 token address
             address tokenAddress = assetData.readAddress(16);
 
             // Encode data for `allowance(ownerAddress, _ERC20_PROXY_ADDRESS)`
             bytes memory allowanceData = abi.encodeWithSelector(
-                IERC20Token(address(0)).allowance.selector,
+                _ERC20_ALLOWANCE_SELECTOR,
                 ownerAddress,
                 _ERC20_PROXY_ADDRESS
             );
@@ -219,13 +222,13 @@ contract LibAssetData {
             // Query allowance
             (bool success, bytes memory returnData) = tokenAddress.staticcall(allowanceData);
             allowance = success && returnData.length == 32 ? returnData.readUint256(0) : 0;
-        } else if (assetProxyId == IAssetData(address(0)).ERC721Token.selector) {
+        } else if (assetProxyId == ERC721_PROXY_ID) {
             // Get ERC721 token address and id
             (, address tokenAddress, uint256 tokenId) = decodeERC721AssetData(assetData);
 
             // Encode data for `isApprovedForAll(ownerAddress, _ERC721_PROXY_ADDRESS)`
             bytes memory isApprovedForAllData = abi.encodeWithSelector(
-                IERC721Token(address(0)).isApprovedForAll.selector,
+                _ERC721_IS_APPROVED_FOR_ALL_SELECTOR,
                 ownerAddress,
                 _ERC721_PROXY_ADDRESS
             );
@@ -235,7 +238,7 @@ contract LibAssetData {
             // If not approved for all, call `getApproved(tokenId)`
             if (!success || returnData.length != 32 || returnData.readUint256(0) != 1) {
                 // Encode data for `getApproved(tokenId)`
-                bytes memory getApprovedData = abi.encodeWithSelector(IERC721Token(address(0)).getApproved.selector, tokenId);
+                bytes memory getApprovedData = abi.encodeWithSelector(_ERC721_GET_APPROVED_SELECTOR, tokenId);
                 (success, returnData) = tokenAddress.staticcall(getApprovedData);
 
                 // Allowance is 1 if successful and the approved address is the ERC721Proxy
@@ -244,13 +247,13 @@ contract LibAssetData {
                 // Allowance is 2^256 - 1 if `isApprovedForAll` returned true
                 allowance = _MAX_UINT256;
             }
-        } else if (assetProxyId == IAssetData(address(0)).ERC1155Assets.selector) {
+        } else if (assetProxyId == ERC1155_PROXY_ID) {
             // Get ERC1155 token address
             (, address tokenAddress, , , ) = decodeERC1155AssetData(assetData);
 
             // Encode data for `isApprovedForAll(ownerAddress, _ERC1155_PROXY_ADDRESS)`
             bytes memory isApprovedForAllData = abi.encodeWithSelector(
-                IERC1155(address(0)).isApprovedForAll.selector,
+                _ERC1155_IS_APPROVED_FOR_ALL_SELECTOR,
                 ownerAddress,
                 _ERC1155_PROXY_ADDRESS
             );
@@ -258,7 +261,7 @@ contract LibAssetData {
             // Query allowance
             (bool success, bytes memory returnData) = tokenAddress.staticcall(isApprovedForAllData);
             allowance = success && returnData.length == 32 && returnData.readUint256(0) == 1 ? _MAX_UINT256 : 0;
-        } else if (assetProxyId == IAssetData(address(0)).StaticCall.selector) {
+        } else if (assetProxyId == STATIC_CALL_PROXY_ID) {
             // The StaticCallProxy does not require any approvals
             allowance = _MAX_UINT256;
         }
@@ -324,7 +327,7 @@ contract LibAssetData {
         pure
         returns (bytes memory assetData)
     {
-        assetData = abi.encodeWithSelector(IAssetData(address(0)).ERC20Token.selector, tokenAddress);
+        assetData = abi.encodeWithSelector(ERC20_PROXY_ID, tokenAddress);
         return assetData;
     }
 
@@ -343,7 +346,7 @@ contract LibAssetData {
         assetProxyId = assetData.readBytes4(0);
 
         require(
-            assetProxyId == IAssetData(address(0)).ERC20Token.selector,
+            assetProxyId == ERC20_PROXY_ID,
             "WRONG_PROXY_ID"
         );
 
@@ -361,7 +364,7 @@ contract LibAssetData {
         returns (bytes memory assetData)
     {
         assetData = abi.encodeWithSelector(
-            IAssetData(address(0)).ERC721Token.selector,
+            ERC721_PROXY_ID,
             tokenAddress,
             tokenId
         );
@@ -385,7 +388,7 @@ contract LibAssetData {
         assetProxyId = assetData.readBytes4(0);
 
         require(
-            assetProxyId == IAssetData(address(0)).ERC721Token.selector,
+            assetProxyId == ERC721_PROXY_ID,
             "WRONG_PROXY_ID"
         );
 
@@ -411,7 +414,7 @@ contract LibAssetData {
         returns (bytes memory assetData)
     {
         assetData = abi.encodeWithSelector(
-            IAssetData(address(0)).ERC1155Assets.selector,
+            ERC1155_PROXY_ID,
             tokenAddress,
             tokenIds,
             tokenValues,
@@ -443,7 +446,7 @@ contract LibAssetData {
         assetProxyId = assetData.readBytes4(0);
 
         require(
-            assetProxyId == IAssetData(address(0)).ERC1155Assets.selector,
+            assetProxyId == ERC1155_PROXY_ID,
             "WRONG_PROXY_ID"
         );
 
@@ -479,7 +482,7 @@ contract LibAssetData {
         returns (bytes memory assetData)
     {
         assetData = abi.encodeWithSelector(
-            IAssetData(address(0)).MultiAsset.selector,
+            MULTI_ASSET_PROXY_ID,
             amounts,
             nestedAssetData
         );
@@ -504,7 +507,7 @@ contract LibAssetData {
         assetProxyId = assetData.readBytes4(0);
 
         require(
-            assetProxyId == IAssetData(address(0)).MultiAsset.selector,
+            assetProxyId == MULTI_ASSET_PROXY_ID,
             "WRONG_PROXY_ID"
         );
 
@@ -515,4 +518,24 @@ contract LibAssetData {
         );
         // solhint-enable indent
     }
+
+    /// @dev Calls `asset.ownerOf(tokenId)`, but returns a null owner instead of reverting on an unowned asset.
+    /// @param tokenAddress Address of ERC721 asset.
+    /// @param tokenId The identifier for the specific NFT.
+    /// @return Owner of tokenId or null address if unowned.
+    function getERC721TokenOwner(address tokenAddress, uint256 tokenId)
+        public
+        view
+        returns (address ownerAddress)
+    {
+        bytes memory ownerOfCalldata = abi.encodeWithSelector(
+            _ERC721_OWNER_OF_SELECTOR,
+            tokenId
+        );
+
+        (bool success, bytes memory returnData) = tokenAddress.staticcall(ownerOfCalldata);
+
+        ownerAddress = (success && returnData.length == 32) ? returnData.readAddress(12) : address(0);
+        return ownerAddress;
+    }
 }

contracts/dev-utils/contracts/src/LibTransactionDecoder.sol

@@ -19,13 +19,14 @@
 pragma solidity ^0.5.5;
 pragma experimental ABIEncoderV2;
 
-import "@0x/contracts-exchange/contracts/src/interfaces/IExchange.sol";
+import "@0x/contracts-exchange-libs/contracts/src/LibExchangeSelectors.sol";
 import "@0x/contracts-exchange-libs/contracts/src/LibOrder.sol";
 import "@0x/contracts-utils/contracts/src/LibBytes.sol";
 
 
-contract LibTransactionDecoder {
-
+contract LibTransactionDecoder is
+    LibExchangeSelectors
+{
     using LibBytes for bytes;
 
     /// @dev Decodes the call data for an Exchange contract method call.
@@ -46,68 +47,72 @@ contract LibTransactionDecoder {
     {
         bytes4 functionSelector = transactionData.readBytes4(0);
 
-        if (functionSelector == IExchange(address(0)).batchCancelOrders.selector) {
+        if (functionSelector == BATCH_CANCEL_ORDERS_SELECTOR) {
             functionName = "batchCancelOrders";
-        } else if (functionSelector == IExchange(address(0)).batchFillOrders.selector) {
+        } else if (functionSelector == BATCH_FILL_ORDERS_SELECTOR) {
             functionName = "batchFillOrders";
-        } else if (functionSelector == IExchange(address(0)).batchFillOrdersNoThrow.selector) {
+        } else if (functionSelector == BATCH_FILL_ORDERS_NO_THROW_SELECTOR) {
             functionName = "batchFillOrdersNoThrow";
-        } else if (functionSelector == IExchange(address(0)).batchFillOrKillOrders.selector) {
+        } else if (functionSelector == BATCH_FILL_OR_KILL_ORDERS_SELECTOR) {
             functionName = "batchFillOrKillOrders";
-        } else if (functionSelector == IExchange(address(0)).cancelOrder.selector) {
+        } else if (functionSelector == CANCEL_ORDER_SELECTOR) {
             functionName = "cancelOrder";
-        } else if (functionSelector == IExchange(address(0)).fillOrder.selector) {
+        } else if (functionSelector == FILL_ORDER_SELECTOR) {
             functionName = "fillOrder";
-        } else if (functionSelector == IExchange(address(0)).fillOrKillOrder.selector) {
+        } else if (functionSelector == FILL_ORDER_NO_THROW_SELECTOR) {
+            functionName = "fillOrderNoThrow";
+        } else if (functionSelector == FILL_OR_KILL_ORDER_SELECTOR) {
             functionName = "fillOrKillOrder";
-        } else if (functionSelector == IExchange(address(0)).marketBuyOrdersNoThrow.selector) {
+        } else if (functionSelector == MARKET_BUY_ORDERS_SELECTOR) {
+            functionName = "marketBuyOrders";
+        } else if (functionSelector == MARKET_BUY_ORDERS_NO_THROW_SELECTOR) {
             functionName = "marketBuyOrdersNoThrow";
-        } else if (functionSelector == IExchange(address(0)).marketSellOrdersNoThrow.selector) {
+        } else if (functionSelector == MARKET_SELL_ORDERS_SELECTOR) {
+            functionName = "marketSellOrders";
+        } else if (functionSelector == MARKET_SELL_ORDERS_NO_THROW_SELECTOR) {
             functionName = "marketSellOrdersNoThrow";
-        } else if (functionSelector == IExchange(address(0)).marketBuyOrdersFillOrKill.selector) {
-            functionName = "marketBuyOrdersFillOrKill";
-        } else if (functionSelector == IExchange(address(0)).marketSellOrdersFillOrKill.selector) {
-            functionName = "marketSellOrdersFillOrKill";
-        } else if (functionSelector == IExchange(address(0)).matchOrders.selector) {
+        } else if (functionSelector == MATCH_ORDERS_SELECTOR) {
             functionName = "matchOrders";
         } else if (
-            functionSelector == IExchange(address(0)).cancelOrdersUpTo.selector ||
-            functionSelector == IExchange(address(0)).executeTransaction.selector
+            functionSelector == CANCEL_ORDERS_UP_TO_SELECTOR ||
+            functionSelector == EXECUTE_TRANSACTION_SELECTOR
+            // TODO: add new noThrow cancel functions when https://github.com/0xProject/ZEIPs/issues/35 is merged.
         ) {
             revert("UNIMPLEMENTED");
         } else {
             revert("UNKNOWN_FUNCTION_SELECTOR");
         }
 
-        if (functionSelector == IExchange(address(0)).batchCancelOrders.selector) {
+        if (functionSelector == BATCH_CANCEL_ORDERS_SELECTOR) {
             // solhint-disable-next-line indent
             orders = abi.decode(transactionData.slice(4, transactionData.length), (LibOrder.Order[]));
             takerAssetFillAmounts = new uint256[](0);
             signatures = new bytes[](0);
         } else if (
-            functionSelector == IExchange(address(0)).batchFillOrKillOrders.selector ||
-            functionSelector == IExchange(address(0)).batchFillOrders.selector ||
-            functionSelector == IExchange(address(0)).batchFillOrdersNoThrow.selector
+            functionSelector == BATCH_FILL_OR_KILL_ORDERS_SELECTOR ||
+            functionSelector == BATCH_FILL_ORDERS_NO_THROW_SELECTOR ||
+            functionSelector == BATCH_FILL_ORDERS_SELECTOR
         ) {
             (orders, takerAssetFillAmounts, signatures) = _makeReturnValuesForBatchFill(transactionData);
-        } else if (functionSelector == IExchange(address(0)).cancelOrder.selector) {
+        } else if (functionSelector == CANCEL_ORDER_SELECTOR) {
             orders = new LibOrder.Order[](1);
             orders[0] = abi.decode(transactionData.slice(4, transactionData.length), (LibOrder.Order));
             takerAssetFillAmounts = new uint256[](0);
             signatures = new bytes[](0);
         } else if (
-            functionSelector == IExchange(address(0)).fillOrKillOrder.selector ||
-            functionSelector == IExchange(address(0)).fillOrder.selector
+            functionSelector == FILL_OR_KILL_ORDER_SELECTOR ||
+            functionSelector == FILL_ORDER_SELECTOR ||
+            functionSelector == FILL_ORDER_NO_THROW_SELECTOR
         ) {
             (orders, takerAssetFillAmounts, signatures) = _makeReturnValuesForSingleOrderFill(transactionData);
         } else if (
-            functionSelector == IExchange(address(0)).marketBuyOrdersNoThrow.selector ||
-            functionSelector == IExchange(address(0)).marketSellOrdersNoThrow.selector ||
-            functionSelector == IExchange(address(0)).marketBuyOrdersFillOrKill.selector ||
-            functionSelector == IExchange(address(0)).marketSellOrdersFillOrKill.selector
+            functionSelector == MARKET_BUY_ORDERS_SELECTOR ||
+            functionSelector == MARKET_BUY_ORDERS_NO_THROW_SELECTOR ||
+            functionSelector == MARKET_SELL_ORDERS_SELECTOR ||
+            functionSelector == MARKET_SELL_ORDERS_NO_THROW_SELECTOR
         ) {
             (orders, takerAssetFillAmounts, signatures) = _makeReturnValuesForMarketFill(transactionData);
-        } else if (functionSelector == IExchange(address(0)).matchOrders.selector) {
+        } else if (functionSelector == MATCH_ORDERS_SELECTOR) {
             (
                 LibOrder.Order memory leftOrder,
                 LibOrder.Order memory rightOrder,

contracts/dev-utils/contracts/src/OrderTransferSimulationUtils.sol

@@ -1,160 +0,0 @@
-/*
-
-  Copyright 2019 ZeroEx Intl.
-
-  Licensed under the Apache License, Version 2.0 (the "License");
-  you may not use this file except in compliance with the License.
-  You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-  Unless required by applicable law or agreed to in writing, software
-  distributed under the License is distributed on an "AS IS" BASIS,
-  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-  See the License for the specific language governing permissions and
-  limitations under the License.
-
-*/
-
-pragma solidity ^0.5.5;
-pragma experimental ABIEncoderV2;
-
-
-import "@0x/contracts-exchange/contracts/src/interfaces/IExchange.sol";
-import "@0x/contracts-exchange/contracts/src/libs/LibExchangeRichErrorDecoder.sol";
-import "@0x/contracts-exchange-libs/contracts/src/LibExchangeRichErrors.sol";
-import "@0x/contracts-exchange-libs/contracts/src/LibOrder.sol";
-import "@0x/contracts-exchange-libs/contracts/src/LibFillResults.sol";
-import "@0x/contracts-utils/contracts/src/LibBytes.sol";
-
-
-contract OrderTransferSimulationUtils is
-    LibExchangeRichErrorDecoder
-{
-    using LibBytes for bytes;
-
-    enum OrderTransferResults {
-        TakerAssetDataFailed,     // Transfer of takerAsset failed
-        MakerAssetDataFailed,     // Transfer of makerAsset failed
-        TakerFeeAssetDataFailed,  // Transfer of takerFeeAsset failed
-        MakerFeeAssetDataFailed,  // Transfer of makerFeeAsset failed
-        TransfersSuccessful       // All transfers in the order were successful
-    }
-
-    // keccak256(abi.encodeWithSignature("Error(string)", "TRANSFERS_SUCCESSFUL"));
-    bytes32 constant internal _TRANSFERS_SUCCESSFUL_RESULT_HASH = 0xf43f26ea5a94b478394a975e856464913dc1a8a1ca70939d974aa7c238aa0ce0;
-
-    // solhint-disable var-name-mixedcase
-    IExchange internal _EXCHANGE;
-    // solhint-enable var-name-mixedcase
-
-    constructor (address _exchange)
-        public
-    {
-        _EXCHANGE = IExchange(_exchange);
-    }
-
-    /// @dev Simulates all of the transfers within an order and returns the index of the first failed transfer.
-    /// @param order The order to simulate transfers for.
-    /// @param takerAddress The address of the taker that will fill the order.
-    /// @param takerAssetFillAmount The amount of takerAsset that the taker wished to fill.
-    /// @return The index of the first failed transfer (or 4 if all transfers are successful).
-    function getSimulatedOrderTransferResults(
-        LibOrder.Order memory order,
-        address takerAddress,
-        uint256 takerAssetFillAmount
-    )
-        public
-        returns (OrderTransferResults orderTransferResults)
-    {
-        LibFillResults.FillResults memory fillResults = LibFillResults.calculateFillResults(
-            order,
-            takerAssetFillAmount,
-            _EXCHANGE.protocolFeeMultiplier(),
-            tx.gasprice
-        );
-
-        // Create input arrays
-        bytes[] memory assetData = new bytes[](4);
-        address[] memory fromAddresses = new address[](4);
-        address[] memory toAddresses = new address[](4);
-        uint256[] memory amounts = new uint256[](4);
-
-        // Transfer `takerAsset` from taker to maker
-        assetData[0] = order.takerAssetData;
-        fromAddresses[0] = takerAddress;
-        toAddresses[0] = order.makerAddress;
-        amounts[0] = takerAssetFillAmount;
-
-        // Transfer `makerAsset` from maker to taker
-        assetData[1] = order.makerAssetData;
-        fromAddresses[1] = order.makerAddress;
-        toAddresses[1] = takerAddress;
-        amounts[1] = fillResults.makerAssetFilledAmount;
-
-        // Transfer `takerFeeAsset` from taker to feeRecipient
-        assetData[2] = order.takerFeeAssetData;
-        fromAddresses[2] = takerAddress;
-        toAddresses[2] = order.feeRecipientAddress;
-        amounts[2] = fillResults.takerFeePaid;
-
-        // Transfer `makerFeeAsset` from maker to feeRecipient
-        assetData[3] = order.makerFeeAssetData;
-        fromAddresses[3] = order.makerAddress;
-        toAddresses[3] = order.feeRecipientAddress;
-        amounts[3] = fillResults.makerFeePaid;
-
-        // Encode data for `simulateDispatchTransferFromCalls(assetData, fromAddresses, toAddresses, amounts)`
-        bytes memory simulateDispatchTransferFromCallsData = abi.encodeWithSelector(
-            IExchange(address(0)).simulateDispatchTransferFromCalls.selector,
-            assetData,
-            fromAddresses,
-            toAddresses,
-            amounts
-        );
-
-        // Perform call and catch revert
-        (, bytes memory returnData) = address(_EXCHANGE).call(simulateDispatchTransferFromCallsData);
-
-        bytes4 selector = returnData.readBytes4(0);
-        if (selector == LibExchangeRichErrors.AssetProxyDispatchErrorSelector()) {
-            // Decode AssetProxyDispatchError and return index of failed transfer
-            (, bytes32 failedTransferIndex,) = decodeAssetProxyDispatchError(returnData);
-            return OrderTransferResults(uint8(uint256(failedTransferIndex)));
-        } else if (selector == LibExchangeRichErrors.AssetProxyTransferErrorSelector()) {
-            // Decode AssetProxyTransferError and return index of failed transfer
-            (bytes32 failedTransferIndex, ,) = decodeAssetProxyTransferError(returnData);
-            return OrderTransferResults(uint8(uint256(failedTransferIndex)));
-        } else if (keccak256(returnData) == _TRANSFERS_SUCCESSFUL_RESULT_HASH) {
-            // All transfers were successful
-            return OrderTransferResults.TransfersSuccessful;
-        } else {
-            revert("UNKNOWN_RETURN_DATA");
-        }
-    }
-
-    /// @dev Simulates all of the transfers for each given order and returns the indices of each first failed transfer.
-    /// @param orders Array of orders to individually simulate transfers for.
-    /// @param takerAddresses Array of addresses of takers that will fill each order.
-    /// @param takerAssetFillAmounts Array of amounts of takerAsset that will be filled for each order.
-    /// @return The indices of the first failed transfer (or 4 if all transfers are successful) for each order.
-    function getSimulatedOrdersTransferResults(
-        LibOrder.Order[] memory orders,
-        address[] memory takerAddresses,
-        uint256[] memory takerAssetFillAmounts
-    )
-        public
-        returns (OrderTransferResults[] memory orderTransferResults)
-    {
-        uint256 length = orders.length;
-        orderTransferResults = new OrderTransferResults[](length);
-        for (uint256 i = 0; i != length; i++) {
-            orderTransferResults[i] = getSimulatedOrderTransferResults(
-                orders[i],
-                takerAddresses[i],
-                takerAssetFillAmounts[i]
-            );
-        }
-        return orderTransferResults;
-    }
-}

contracts/dev-utils/contracts/src/OrderValidationUtils.sol

@@ -1,6 +1,6 @@
 /*
 
-  Copyright 2019 ZeroEx Intl.
+  Copyright 2018 ZeroEx Intl.
 
   Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
@@ -23,20 +23,25 @@ import "@0x/contracts-exchange/contracts/src/interfaces/IExchange.sol";
 import "@0x/contracts-exchange-libs/contracts/src/LibOrder.sol";
 import "@0x/contracts-exchange-libs/contracts/src/LibMath.sol";
 import "@0x/contracts-utils/contracts/src/LibBytes.sol";
-import "@0x/contracts-utils/contracts/src/LibSafeMath.sol";
 import "./LibAssetData.sol";
 
 
 contract OrderValidationUtils is
-    LibAssetData
+    LibAssetData,
+    LibMath
 {
     using LibBytes for bytes;
-    using LibSafeMath for uint256;
 
-    constructor (address _exchange)
+    // solhint-disable var-name-mixedcase
+    bytes internal _ZRX_ASSET_DATA;
+    // solhint-enable var-name-mixedcase
+
+    constructor (address _exchange, bytes memory _zrxAssetData)
         public
         LibAssetData(_exchange)
-    {}
+    {
+        _ZRX_ASSET_DATA = _zrxAssetData;
+    }
 
     /// @dev Fetches all order-relevant information needed to validate if the supplied order is fillable.
     /// @param order The order structure.
@@ -62,8 +67,9 @@ contract OrderValidationUtils is
 
         // Validate the maker's signature
         address makerAddress = order.makerAddress;
-        isValidSignature = _EXCHANGE.isValidOrderSignature(
-            order,
+        isValidSignature = _EXCHANGE.isValidSignature(
+            orderInfo.orderHash,
+            makerAddress,
             signature
         );
 
@@ -73,22 +79,26 @@ contract OrderValidationUtils is
         // Assign to stack variables to reduce redundant mloads/sloads
         uint256 takerAssetAmount = order.takerAssetAmount;
         uint256 makerFee = order.makerFee;
-
+        bytes memory zrxAssetData = _ZRX_ASSET_DATA;
+    
         // Get the amount of `takerAsset` that is transferable to maker given the transferability of `makerAsset`, `makerFeeAsset`,
         // and the total amounts specified in the order
         uint256 transferableTakerAssetAmount;
-        if (order.makerAssetData.equals(order.makerFeeAssetData)) {
+        if (order.makerAssetData.equals(zrxAssetData)) {
             // If `makerAsset` equals `makerFeeAsset`, the % that can be filled is
             // transferableMakerAssetAmount / (makerAssetAmount + makerFee)
-            transferableTakerAssetAmount = LibMath.getPartialAmountFloor(
+            transferableTakerAssetAmount = getPartialAmountFloor(
                 transferableMakerAssetAmount,
-                order.makerAssetAmount.safeAdd(makerFee),
+                safeAdd(order.makerAssetAmount, makerFee),
                 takerAssetAmount
             );
         } else {
+            // Get the transferable amount of the `makerFeeAsset`
+            uint256 transferableMakerFeeAssetAmount = getTransferableAssetAmount(makerAddress, zrxAssetData);
+
             // If `makerFee` is 0, the % that can be filled is (transferableMakerAssetAmount / makerAssetAmount)
             if (makerFee == 0) {
-                transferableTakerAssetAmount = LibMath.getPartialAmountFloor(
+                transferableTakerAssetAmount = getPartialAmountFloor(
                     transferableMakerAssetAmount,
                     order.makerAssetAmount,
                     takerAssetAmount
@@ -97,26 +107,23 @@ contract OrderValidationUtils is
             // If `makerAsset` does not equal `makerFeeAsset`, the % that can be filled is the lower of
             // (transferableMakerAssetAmount / makerAssetAmount) and (transferableMakerAssetFeeAmount / makerFee)
             } else {
-                // Get the transferable amount of the `makerFeeAsset`
-                uint256 transferableMakerFeeAssetAmount = getTransferableAssetAmount(makerAddress, order.makerFeeAssetData);
-
-                uint256 transferableMakerToTakerAmount = LibMath.getPartialAmountFloor(
+                uint256 transferableMakerToTakerAmount = getPartialAmountFloor(
                     transferableMakerAssetAmount,
                     order.makerAssetAmount,
                     takerAssetAmount
                 );
-                uint256 transferableMakerFeeToTakerAmount = LibMath.getPartialAmountFloor(
+                uint256 transferableMakerFeeToTakerAmount = getPartialAmountFloor(
                     transferableMakerFeeAssetAmount,
                     makerFee,
                     takerAssetAmount
                 );
-                transferableTakerAssetAmount = LibSafeMath.min256(transferableMakerToTakerAmount, transferableMakerFeeToTakerAmount);
+                transferableTakerAssetAmount = min256(transferableMakerToTakerAmount, transferableMakerFeeToTakerAmount);
             }
         }
 
         // `fillableTakerAssetAmount` is the lower of the order's remaining `takerAssetAmount` and the `transferableTakerAssetAmount`
-        fillableTakerAssetAmount = LibSafeMath.min256(
-            takerAssetAmount.safeSub(orderInfo.orderTakerAssetFilledAmount),
+        fillableTakerAssetAmount = min256(
+            safeSub(takerAssetAmount, orderInfo.orderTakerAssetFilledAmount),
             transferableTakerAssetAmount
         );
 
@@ -163,7 +170,7 @@ contract OrderValidationUtils is
     /// @return The amount of the asset tranferable by the owner.
     /// NOTE: If the `assetData` encodes data for multiple assets, the `transferableAssetAmount`
     /// will represent the amount of times the entire `assetData` can be transferred. To calculate
-    /// the total individual transferable amounts, this scaled `transferableAmount` must be multiplied by
+    /// the total individual transferable amounts, this scaled `transferableAmount` must be multiplied by 
     /// the individual asset amounts located within the `assetData`.
     function getTransferableAssetAmount(address ownerAddress, bytes memory assetData)
         public
@@ -171,7 +178,7 @@ contract OrderValidationUtils is
         returns (uint256 transferableAssetAmount)
     {
         (uint256 balance, uint256 allowance) = getBalanceAndAssetProxyAllowance(ownerAddress, assetData);
-        transferableAssetAmount = LibSafeMath.min256(balance, allowance);
+        transferableAssetAmount = min256(balance, allowance);
         return transferableAssetAmount;
     }
 }

contracts/dev-utils/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@0x/contracts-dev-utils",
-    "version": "0.1.0-beta.0",
+    "version": "0.0.8",
     "engines": {
         "node": ">=6.12"
     },
@@ -12,17 +12,17 @@
     "scripts": {
         "build": "yarn pre_build && tsc -b",
         "build:ci": "yarn build",
-        "pre_build": "run-s compile quantify_bytecode contracts:gen generate_contract_wrappers",
+        "pre_build": "run-s compile generate_contract_wrappers",
         "test": "yarn run_mocha",
         "rebuild_and_test": "run-s build test",
         "test:coverage": "SOLIDITY_COVERAGE=true run-s build run_mocha coverage:report:text coverage:report:lcov",
         "test:profiler": "SOLIDITY_PROFILER=true run-s build run_mocha profiler:report:html",
         "test:trace": "SOLIDITY_REVERT_TRACE=true run-s build run_mocha",
-        "run_mocha": "UNLIMITED_CONTRACT_SIZE=true mocha --require source-map-support/register --require make-promises-safe 'lib/test/**/*.js' --timeout 100000 --bail --exit",
+        "run_mocha": "mocha --require source-map-support/register --require make-promises-safe 'lib/test/**/*.js' --timeout 100000 --bail --exit",
         "compile": "sol-compiler",
         "watch": "sol-compiler -w",
         "clean": "shx rm -rf lib generated-artifacts generated-wrappers",
-        "generate_contract_wrappers": "abi-gen --abis  ${npm_package_config_abis} --output generated-wrappers --backend ethers",
+        "generate_contract_wrappers": "abi-gen --abis  ${npm_package_config_abis} --template ../../node_modules/@0x/abi-gen-templates/contract.handlebars --partials '../../node_modules/@0x/abi-gen-templates/partials/**/*.handlebars' --output generated-wrappers --backend ethers",
         "lint": "tslint --format stylish --project . --exclude ./generated-wrappers/**/* --exclude ./generated-artifacts/**/* --exclude **/lib/**/* && yarn lint-contracts",
         "fix": "tslint --fix --format stylish --project . --exclude ./generated-wrappers/**/* --exclude ./generated-artifacts/**/* --exclude **/lib/**/* && yarn lint-contracts",
         "coverage:report:text": "istanbul report text",
@@ -31,12 +31,10 @@
         "coverage:report:lcov": "istanbul report lcov",
         "test:circleci": "yarn test",
         "contracts:gen": "contracts-gen",
-        "lint-contracts": "solhint -c ../.solhint.json contracts/**/**/**/**/*.sol",
-        "quantify_bytecode": "echo EVM bytecode object lengths:;for i in ./generated-artifacts/*.json; do node -e \"console.log('$i\t' + (require('$i').compilerOutput.evm.bytecode.object.length - 2) / 2)\"; done",
-        "compile:truffle": "truffle compile"
+        "lint-contracts": "solhint -c ../.solhint.json contracts/**/**/**/**/*.sol"
     },
     "config": {
-        "abis": "./generated-artifacts/@(DevUtils|EthBalanceChecker|LibAssetData|LibTransactionDecoder|OrderTransferSimulationUtils|OrderValidationUtils).json",
+        "abis": "./generated-artifacts/@(DevUtils|LibAssetData|LibTransactionDecoder|EthBalanceChecker).json",
         "abis:comment": "This list is auto-generated by contracts-gen. Don't edit manually."
     },
     "repository": {
@@ -49,11 +47,11 @@
     },
     "homepage": "https://github.com/0xProject/0x-monorepo/contracts/dev-utils/README.md",
     "devDependencies": {
-        "@0x/abi-gen": "^4.3.0-beta.0",
-        "@0x/contracts-gen": "^1.1.0-beta.0",
-        "@0x/contracts-test-utils": "^3.2.0-beta.0",
-        "@0x/dev-utils": "^2.4.0-beta.0",
-        "@0x/sol-compiler": "^3.2.0-beta.0",
+        "@0x/abi-gen": "^4.1.1",
+        "@0x/contracts-gen": "^1.0.13",
+        "@0x/contracts-test-utils": "^3.1.14",
+        "@0x/dev-utils": "^2.3.1",
+        "@0x/sol-compiler": "^3.1.13",
         "@0x/tslint-config": "^3.0.1",
         "@types/lodash": "4.14.104",
         "@types/mocha": "^5.2.7",
@@ -67,25 +65,24 @@
         "npm-run-all": "^4.1.2",
         "shx": "^0.2.2",
         "solhint": "^1.4.1",
-        "truffle": "^5.0.32",
         "tslint": "5.11.0",
         "typescript": "3.0.1"
     },
     "dependencies": {
-        "@0x/base-contract": "^5.5.0-beta.0",
-        "@0x/contracts-asset-proxy": "^2.3.0-beta.0",
-        "@0x/contracts-erc1155": "^1.2.0-beta.0",
-        "@0x/contracts-erc20": "^2.3.0-beta.0",
-        "@0x/contracts-erc721": "^2.2.0-beta.0",
-        "@0x/contracts-exchange": "^2.2.0-beta.0",
-        "@0x/contracts-exchange-libs": "^3.1.0-beta.0",
-        "@0x/contracts-utils": "^3.3.0-beta.0",
-        "@0x/order-utils": "^8.5.0-beta.0",
-        "@0x/types": "^2.5.0-beta.0",
-        "@0x/typescript-typings": "^4.4.0-beta.0",
-        "@0x/utils": "^4.6.0-beta.0",
-        "@0x/web3-wrapper": "^6.1.0-beta.0",
-        "ethereum-types": "^2.2.0-beta.0",
+        "@0x/base-contract": "^5.3.2",
+        "@0x/contracts-asset-proxy": "^2.2.6",
+        "@0x/contracts-erc1155": "^1.1.13",
+        "@0x/contracts-erc20": "^2.2.12",
+        "@0x/contracts-erc721": "^2.1.13",
+        "@0x/contracts-exchange": "^2.1.12",
+        "@0x/contracts-exchange-libs": "^3.0.6",
+        "@0x/contracts-utils": "^3.2.2",
+        "@0x/order-utils": "^8.3.0",
+        "@0x/types": "^2.4.1",
+        "@0x/typescript-typings": "^4.2.4",
+        "@0x/utils": "^4.5.0",
+        "@0x/web3-wrapper": "^6.0.11",
+        "ethereum-types": "^2.1.4",
         "ethereumjs-util": "^5.1.1"
     },
     "publishConfig": {

contracts/dev-utils/src/artifacts.ts

@@ -6,16 +6,10 @@
 import { ContractArtifact } from 'ethereum-types';
 
 import * as DevUtils from '../generated-artifacts/DevUtils.json';
-import * as EthBalanceChecker from '../generated-artifacts/EthBalanceChecker.json';
 import * as LibAssetData from '../generated-artifacts/LibAssetData.json';
 import * as LibTransactionDecoder from '../generated-artifacts/LibTransactionDecoder.json';
-import * as OrderTransferSimulationUtils from '../generated-artifacts/OrderTransferSimulationUtils.json';
-import * as OrderValidationUtils from '../generated-artifacts/OrderValidationUtils.json';
 export const artifacts = {
     DevUtils: DevUtils as ContractArtifact,
-    EthBalanceChecker: EthBalanceChecker as ContractArtifact,
-    LibAssetData: LibAssetData as ContractArtifact,
     LibTransactionDecoder: LibTransactionDecoder as ContractArtifact,
-    OrderTransferSimulationUtils: OrderTransferSimulationUtils as ContractArtifact,
-    OrderValidationUtils: OrderValidationUtils as ContractArtifact,
+    LibAssetData: LibAssetData as ContractArtifact,
 };

contracts/dev-utils/src/wrappers.ts

@@ -4,8 +4,5 @@
  * -----------------------------------------------------------------------------
  */
 export * from '../generated-wrappers/dev_utils';
-export * from '../generated-wrappers/eth_balance_checker';
 export * from '../generated-wrappers/lib_asset_data';
 export * from '../generated-wrappers/lib_transaction_decoder';
-export * from '../generated-wrappers/order_transfer_simulation_utils';
-export * from '../generated-wrappers/order_validation_utils';

contracts/dev-utils/test/lib_asset_data.ts

@@ -22,7 +22,7 @@ import { chaiSetup, constants, LogDecoder, provider, txDefaults, web3Wrapper } f
 import { BlockchainLifecycle } from '@0x/dev-utils';
 import { assetDataUtils } from '@0x/order-utils';
 import { AssetProxyId } from '@0x/types';
-import { BigNumber, providerUtils } from '@0x/utils';
+import { BigNumber } from '@0x/utils';
 import * as ethUtil from 'ethereumjs-util';
 
 import { artifacts, LibAssetDataContract } from '../src';
@@ -87,13 +87,13 @@ describe('LibAssetData', () => {
 
     before(async () => {
         await blockchainLifecycle.startAsync();
-        const chainId = await providerUtils.getChainIdAsync(provider);
+
         exchange = await ExchangeContract.deployFrom0xArtifactAsync(
             exchangeArtifacts.Exchange,
             provider,
             txDefaults,
-            {},
-            new BigNumber(chainId),
+            artifacts,
+            constants.NULL_BYTES,
         );
 
         erc20Proxy = await ERC20ProxyContract.deployFrom0xArtifactAsync(
@@ -476,6 +476,20 @@ describe('LibAssetData', () => {
         });
     });
 
+    describe('getERC721TokenOwner', async () => {
+        it('should return the null address when tokenId is not owned', async () => {
+            const nonexistentTokenId = new BigNumber(1234567890);
+            expect(
+                await libAssetData.getERC721TokenOwner.callAsync(erc721Token.address, nonexistentTokenId),
+            ).to.be.equal(constants.NULL_ADDRESS);
+        });
+        it('should return the owner address when tokenId is owned', async () => {
+            expect(
+                await libAssetData.getERC721TokenOwner.callAsync(erc721Token.address, firstERC721TokenId),
+            ).to.be.equal(tokenOwnerAddress);
+        });
+    });
+
     describe('getBalanceAndAllowance', () => {
         it('should query balance and allowance together, from asset data', async () => {
             const allowance = new BigNumber(1);

contracts/dev-utils/test/lib_transaction_decoder.ts

@@ -23,8 +23,6 @@ const order = {
     salt: new BigNumber('66097384406870180066678463045003379626790660770396923976862707230261946348951'),
     makerAssetData: '0xf47261b000000000000000000000000034d402f14d58e001d8efbe6585051bf9706aa064',
     takerAssetData: '0xf47261b000000000000000000000000025b8fe1de9daf8ba351890744ff28cf7dfa8f5e3',
-    makerFeeAssetData: '0xf47261b000000000000000000000000034d402f14d58e001d8efbe6585051bf9706aa064',
-    takerFeeAssetData: '0xf47261b000000000000000000000000025b8fe1de9daf8ba351890744ff28cf7dfa8f5e3',
 };
 const takerAssetFillAmount = new BigNumber('100000000000000000000');
 const signature =
@@ -82,7 +80,7 @@ describe('LibTransactionDecoder', () => {
         ]);
     });
 
-    for (const func of ['fillOrder', 'fillOrKillOrder']) {
+    for (const func of ['fillOrder', 'fillOrderNoThrow', 'fillOrKillOrder']) {
         const input = (exchangeInterface as any)[func].getABIEncodedTransactionData(
             order,
             takerAssetFillAmount,
@@ -98,12 +96,7 @@ describe('LibTransactionDecoder', () => {
         });
     }
 
-    for (const func of [
-        'marketBuyOrdersNoThrow',
-        'marketSellOrdersNoThrow',
-        'marketBuyOrdersFillOrKill',
-        'marketSellOrdersFillOrKill',
-    ]) {
+    for (const func of ['marketBuyOrders', 'marketBuyOrdersNoThrow', 'marketSellOrders', 'marketSellOrdersNoThrow']) {
         const input = (exchangeInterface as any)[func].getABIEncodedTransactionData(
             [order, order],
             takerAssetFillAmount,
@@ -130,8 +123,6 @@ describe('LibTransactionDecoder', () => {
             takerAssetAmount: order.makerAssetAmount,
             makerFee: order.takerFee,
             takerFee: order.makerFee,
-            makerFeeAssetData: order.takerFeeAssetData,
-            takerFeeAssetData: order.makerFeeAssetData,
         };
         const input = exchangeInterface.matchOrders.getABIEncodedTransactionData(
             order,

contracts/dev-utils/test/order_validation_utils.ts

@@ -20,8 +20,8 @@ import {
 } from '@0x/contracts-test-utils';
 import { BlockchainLifecycle } from '@0x/dev-utils';
 import { assetDataUtils, orderHashUtils } from '@0x/order-utils';
-import { OrderTransferResults, SignedOrder } from '@0x/types';
-import { BigNumber, providerUtils } from '@0x/utils';
+import { SignedOrder } from '@0x/types';
+import { BigNumber } from '@0x/utils';
 import * as chai from 'chai';
 
 import { artifacts, DevUtilsContract } from '../src';
@@ -30,18 +30,18 @@ chaiSetup.configure();
 const expect = chai.expect;
 const blockchainLifecycle = new BlockchainLifecycle(web3Wrapper);
 
-describe('OrderValidationUtils/OrderTransferSimulatorUtils', () => {
+describe('OrderValidationUtils', () => {
     let makerAddress: string;
     let takerAddress: string;
     let owner: string;
     let erc20AssetData: string;
     let erc20AssetData2: string;
     let erc721AssetData: string;
-    let feeAssetData: string;
+    let zrxAssetData: string;
 
     let erc20Token: DummyERC20TokenContract;
     let erc20Token2: DummyERC20TokenContract;
-    let feeErc20Token: DummyERC20TokenContract;
+    let zrxToken: DummyERC20TokenContract;
     let erc721Token: DummyERC721TokenContract;
     let exchange: ExchangeContract;
     let devUtils: DevUtilsContract;
@@ -64,13 +64,12 @@ describe('OrderValidationUtils/OrderTransferSimulatorUtils', () => {
     before(async () => {
         const accounts = await web3Wrapper.getAvailableAddressesAsync();
         const usedAddresses = ([owner, makerAddress, takerAddress] = accounts.slice(0, 3));
-        const chainId = await providerUtils.getChainIdAsync(provider);
 
         const erc20Wrapper = new ERC20Wrapper(provider, usedAddresses, owner);
         const erc721Wrapper = new ERC721Wrapper(provider, usedAddresses, owner);
 
         const numDummyErc20ToDeploy = 3;
-        [erc20Token, erc20Token2, feeErc20Token] = await erc20Wrapper.deployDummyTokensAsync(
+        [erc20Token, zrxToken, erc20Token2] = await erc20Wrapper.deployDummyTokensAsync(
             numDummyErc20ToDeploy,
             constants.DUMMY_TOKEN_DECIMALS,
         );
@@ -79,13 +78,13 @@ describe('OrderValidationUtils/OrderTransferSimulatorUtils', () => {
         [erc721Token] = await erc721Wrapper.deployDummyTokensAsync();
         erc721Proxy = await erc721Wrapper.deployProxyAsync();
 
-        feeAssetData = assetDataUtils.encodeERC20AssetData(feeErc20Token.address);
+        zrxAssetData = assetDataUtils.encodeERC20AssetData(zrxToken.address);
         exchange = await ExchangeContract.deployFrom0xArtifactAsync(
             exchangeArtifacts.Exchange,
             provider,
             txDefaults,
-            {},
-            new BigNumber(chainId),
+            artifacts,
+            zrxAssetData,
         );
 
         multiAssetProxy = await MultiAssetProxyContract.deployFrom0xArtifactAsync(
@@ -94,7 +93,7 @@ describe('OrderValidationUtils/OrderTransferSimulatorUtils', () => {
             txDefaults,
             artifacts,
         );
-        const exchangeWrapper = new ExchangeWrapper(exchange);
+        const exchangeWrapper = new ExchangeWrapper(exchange, provider);
         await exchangeWrapper.registerAssetProxyAsync(erc20Proxy.address, owner);
         await exchangeWrapper.registerAssetProxyAsync(erc721Proxy.address, owner);
         await exchangeWrapper.registerAssetProxyAsync(multiAssetProxy.address, owner);
@@ -107,6 +106,7 @@ describe('OrderValidationUtils/OrderTransferSimulatorUtils', () => {
             txDefaults,
             artifacts,
             exchange.address,
+            zrxAssetData,
         );
 
         erc20AssetData = assetDataUtils.encodeERC20AssetData(erc20Token.address);
@@ -114,14 +114,11 @@ describe('OrderValidationUtils/OrderTransferSimulatorUtils', () => {
         erc721AssetData = assetDataUtils.encodeERC721AssetData(erc721Token.address, tokenId);
         const defaultOrderParams = {
             ...constants.STATIC_ORDER_PARAMS,
+            exchangeAddress: exchange.address,
             makerAddress,
             feeRecipientAddress: constants.NULL_ADDRESS,
             makerAssetData: erc20AssetData,
             takerAssetData: erc20AssetData2,
-            makerFeeAssetData: feeAssetData,
-            takerFeeAssetData: feeAssetData,
-            exchangeAddress: exchange.address,
-            chainId,
         };
         const privateKey = constants.TESTRPC_PRIVATE_KEYS[accounts.indexOf(makerAddress)];
         orderFactory = new OrderFactory(privateKey, defaultOrderParams);
@@ -248,11 +245,11 @@ describe('OrderValidationUtils/OrderTransferSimulatorUtils', () => {
                 from: makerAddress,
             });
             const divisor = 4;
-            await feeErc20Token.setBalance.awaitTransactionSuccessAsync(
+            await zrxToken.setBalance.awaitTransactionSuccessAsync(
                 makerAddress,
                 signedOrder.makerFee.dividedToIntegerBy(divisor),
             );
-            await feeErc20Token.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.makerFee, {
+            await zrxToken.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.makerFee, {
                 from: makerAddress,
             });
             const [, fillableTakerAssetAmount] = await devUtils.getOrderRelevantState.callAsync(
@@ -272,8 +269,8 @@ describe('OrderValidationUtils/OrderTransferSimulatorUtils', () => {
             await erc20Token.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.makerAssetAmount, {
                 from: makerAddress,
             });
-            await feeErc20Token.setBalance.awaitTransactionSuccessAsync(makerAddress, signedOrder.makerFee);
-            await feeErc20Token.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.makerFee, {
+            await zrxToken.setBalance.awaitTransactionSuccessAsync(makerAddress, signedOrder.makerFee);
+            await zrxToken.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.makerFee, {
                 from: makerAddress,
             });
             const [, fillableTakerAssetAmount] = await devUtils.getOrderRelevantState.callAsync(
@@ -294,8 +291,8 @@ describe('OrderValidationUtils/OrderTransferSimulatorUtils', () => {
             await erc20Token.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.makerAssetAmount, {
                 from: makerAddress,
             });
-            await feeErc20Token.setBalance.awaitTransactionSuccessAsync(makerAddress, signedOrder.makerFee);
-            await feeErc20Token.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.makerFee, {
+            await zrxToken.setBalance.awaitTransactionSuccessAsync(makerAddress, signedOrder.makerFee);
+            await zrxToken.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.makerFee, {
                 from: makerAddress,
             });
             const divisor = 4;
@@ -319,8 +316,8 @@ describe('OrderValidationUtils/OrderTransferSimulatorUtils', () => {
             );
         });
         it('should return a fillableTakerAssetAmount of 0 when non-fee balances/allowances are insufficient', async () => {
-            await feeErc20Token.setBalance.awaitTransactionSuccessAsync(makerAddress, signedOrder.makerFee);
-            await feeErc20Token.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.makerFee, {
+            await zrxToken.setBalance.awaitTransactionSuccessAsync(makerAddress, signedOrder.makerFee);
+            await zrxToken.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.makerFee, {
                 from: makerAddress,
             });
             const [, fillableTakerAssetAmount] = await devUtils.getOrderRelevantState.callAsync(
@@ -334,8 +331,8 @@ describe('OrderValidationUtils/OrderTransferSimulatorUtils', () => {
             await erc20Token.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.makerAssetAmount, {
                 from: makerAddress,
             });
-            await feeErc20Token.setBalance.awaitTransactionSuccessAsync(makerAddress, signedOrder.makerFee);
-            await feeErc20Token.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.makerFee, {
+            await zrxToken.setBalance.awaitTransactionSuccessAsync(makerAddress, signedOrder.makerFee);
+            await zrxToken.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.makerFee, {
                 from: makerAddress,
             });
             const [, fillableTakerAssetAmount] = await devUtils.getOrderRelevantState.callAsync(
@@ -346,14 +343,14 @@ describe('OrderValidationUtils/OrderTransferSimulatorUtils', () => {
         });
         it('should return the correct fillableTakerAssetAmount when balances/allowances are partially sufficient and makerAsset=makerFeeAsset', async () => {
             signedOrder = await orderFactory.newSignedOrderAsync({
-                makerAssetData: feeAssetData,
+                makerAssetData: zrxAssetData,
                 makerAssetAmount: new BigNumber(10),
                 takerAssetAmount: new BigNumber(20),
                 makerFee: new BigNumber(40),
             });
             const transferableMakerAssetAmount = new BigNumber(10);
-            await feeErc20Token.setBalance.awaitTransactionSuccessAsync(makerAddress, transferableMakerAssetAmount);
-            await feeErc20Token.approve.awaitTransactionSuccessAsync(erc20Proxy.address, transferableMakerAssetAmount, {
+            await zrxToken.setBalance.awaitTransactionSuccessAsync(makerAddress, transferableMakerAssetAmount);
+            await zrxToken.approve.awaitTransactionSuccessAsync(erc20Proxy.address, transferableMakerAssetAmount, {
                 from: makerAddress,
             });
             const expectedFillableTakerAssetAmount = transferableMakerAssetAmount
@@ -382,17 +379,17 @@ describe('OrderValidationUtils/OrderTransferSimulatorUtils', () => {
             await erc20Token.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.makerAssetAmount, {
                 from: makerAddress,
             });
-            await feeErc20Token.setBalance.awaitTransactionSuccessAsync(makerAddress, signedOrder.makerFee);
-            await feeErc20Token.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.makerFee, {
+            await zrxToken.setBalance.awaitTransactionSuccessAsync(makerAddress, signedOrder.makerFee);
+            await zrxToken.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.makerFee, {
                 from: makerAddress,
             });
             await erc20Token2.setBalance.awaitTransactionSuccessAsync(takerAddress, signedOrder.takerAssetAmount);
             await erc20Token2.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.takerAssetAmount, {
                 from: takerAddress,
             });
-            await feeErc20Token.setBalance.awaitTransactionSuccessAsync(takerAddress, signedOrder.takerFee);
+            await zrxToken.setBalance.awaitTransactionSuccessAsync(takerAddress, signedOrder.takerFee);
 
-            await feeErc20Token.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.takerFee, {
+            await zrxToken.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.takerFee, {
                 from: takerAddress,
             });
             const takerAssetFillAmount = signedOrder.takerAssetAmount.dividedToIntegerBy(4);
@@ -410,38 +407,15 @@ describe('OrderValidationUtils/OrderTransferSimulatorUtils', () => {
                 signedOrder.takerAssetAmount.minus(takerAssetFillAmount),
             );
         });
-        it('should return correct info even when there are no fees specified', async () => {
-            signedOrder = await orderFactory.newSignedOrderAsync({
-                makerFee: new BigNumber(0),
-                takerFee: new BigNumber(0),
-                makerFeeAssetData: '0x',
-                takerFeeAssetData: '0x',
-            });
-            await erc20Token.setBalance.awaitTransactionSuccessAsync(makerAddress, signedOrder.makerAssetAmount);
-            await erc20Token.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.makerAssetAmount, {
-                from: makerAddress,
-            });
-            const [
-                orderInfo,
-                fillableTakerAssetAmount,
-                isValidSignature,
-            ] = await devUtils.getOrderRelevantState.callAsync(signedOrder, signedOrder.signature);
-            expect(orderInfo.orderHash).to.equal(orderHashUtils.getOrderHashHex(signedOrder));
-            expect(orderInfo.orderStatus).to.equal(OrderStatus.Fillable);
-            expect(orderInfo.orderTakerAssetFilledAmount).to.bignumber.equal(constants.ZERO_AMOUNT);
-            expect(fillableTakerAssetAmount).to.bignumber.equal(signedOrder.takerAssetAmount);
-            expect(isValidSignature).to.equal(true);
-        });
     });
     describe('getOrderRelevantStates', async () => {
         it('should return the correct information for multiple orders', async () => {
-            signedOrder = await orderFactory.newSignedOrderAsync();
             await erc20Token.setBalance.awaitTransactionSuccessAsync(makerAddress, signedOrder.makerAssetAmount);
             await erc20Token.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.makerAssetAmount, {
                 from: makerAddress,
             });
-            await feeErc20Token.setBalance.awaitTransactionSuccessAsync(makerAddress, signedOrder.makerFee);
-            await feeErc20Token.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.makerFee, {
+            await zrxToken.setBalance.awaitTransactionSuccessAsync(makerAddress, signedOrder.makerFee);
+            await zrxToken.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.makerFee, {
                 from: makerAddress,
             });
             const signedOrder2 = await orderFactory.newSignedOrderAsync({ makerAssetData: erc721AssetData });
@@ -467,185 +441,5 @@ describe('OrderValidationUtils/OrderTransferSimulatorUtils', () => {
             expect(isValidSignature[1]).to.equal(false);
         });
     });
-    describe('getSimulatedOrderTransferResults', () => {
-        beforeEach(async () => {
-            signedOrder = await orderFactory.newSignedOrderAsync();
-        });
-        it('should return TakerAssetDataFailed if the takerAsset transfer fails', async () => {
-            const orderTransferResults = await devUtils.getSimulatedOrderTransferResults.callAsync(
-                signedOrder,
-                takerAddress,
-                signedOrder.takerAssetAmount,
-            );
-            expect(orderTransferResults).to.equal(OrderTransferResults.TakerAssetDataFailed);
-        });
-        it('should return MakerAssetDataFailed if the makerAsset transfer fails', async () => {
-            await erc20Token2.setBalance.awaitTransactionSuccessAsync(takerAddress, signedOrder.takerAssetAmount, {
-                from: owner,
-            });
-            await erc20Token2.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.takerAssetAmount, {
-                from: takerAddress,
-            });
-            const orderTransferResults = await devUtils.getSimulatedOrderTransferResults.callAsync(
-                signedOrder,
-                takerAddress,
-                signedOrder.takerAssetAmount,
-            );
-            expect(orderTransferResults).to.equal(OrderTransferResults.MakerAssetDataFailed);
-        });
-        it('should return TakerFeeAssetDataFailed if the takerFeeAsset transfer fails', async () => {
-            await erc20Token2.setBalance.awaitTransactionSuccessAsync(takerAddress, signedOrder.takerAssetAmount, {
-                from: owner,
-            });
-            await erc20Token2.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.takerAssetAmount, {
-                from: takerAddress,
-            });
-            await erc20Token.setBalance.awaitTransactionSuccessAsync(makerAddress, signedOrder.makerAssetAmount, {
-                from: owner,
-            });
-            await erc20Token.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.makerAssetAmount, {
-                from: makerAddress,
-            });
-            const orderTransferResults = await devUtils.getSimulatedOrderTransferResults.callAsync(
-                signedOrder,
-                takerAddress,
-                signedOrder.takerAssetAmount,
-            );
-            expect(orderTransferResults).to.equal(OrderTransferResults.TakerFeeAssetDataFailed);
-        });
-        it('should return MakerFeeAssetDataFailed if the makerFeeAsset transfer fails', async () => {
-            await erc20Token2.setBalance.awaitTransactionSuccessAsync(takerAddress, signedOrder.takerAssetAmount, {
-                from: owner,
-            });
-            await erc20Token2.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.takerAssetAmount, {
-                from: takerAddress,
-            });
-            await erc20Token.setBalance.awaitTransactionSuccessAsync(makerAddress, signedOrder.makerAssetAmount, {
-                from: owner,
-            });
-            await erc20Token.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.makerAssetAmount, {
-                from: makerAddress,
-            });
-            await feeErc20Token.setBalance.awaitTransactionSuccessAsync(takerAddress, signedOrder.takerFee, {
-                from: owner,
-            });
-            await feeErc20Token.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.takerFee, {
-                from: takerAddress,
-            });
-            const orderTransferResults = await devUtils.getSimulatedOrderTransferResults.callAsync(
-                signedOrder,
-                takerAddress,
-                signedOrder.takerAssetAmount,
-            );
-            expect(orderTransferResults).to.equal(OrderTransferResults.MakerFeeAssetDataFailed);
-        });
-        it('should return TransfersSuccessful if all transfers succeed', async () => {
-            await erc20Token2.setBalance.awaitTransactionSuccessAsync(takerAddress, signedOrder.takerAssetAmount, {
-                from: owner,
-            });
-            await erc20Token2.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.takerAssetAmount, {
-                from: takerAddress,
-            });
-            await erc20Token.setBalance.awaitTransactionSuccessAsync(makerAddress, signedOrder.makerAssetAmount, {
-                from: owner,
-            });
-            await erc20Token.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.makerAssetAmount, {
-                from: makerAddress,
-            });
-            await feeErc20Token.setBalance.awaitTransactionSuccessAsync(takerAddress, signedOrder.takerFee, {
-                from: owner,
-            });
-            await feeErc20Token.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.takerFee, {
-                from: takerAddress,
-            });
-            await feeErc20Token.setBalance.awaitTransactionSuccessAsync(makerAddress, signedOrder.makerFee, {
-                from: owner,
-            });
-            await feeErc20Token.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.makerFee, {
-                from: makerAddress,
-            });
-            const orderTransferResults = await devUtils.getSimulatedOrderTransferResults.callAsync(
-                signedOrder,
-                takerAddress,
-                signedOrder.takerAssetAmount,
-            );
-            expect(orderTransferResults).to.equal(OrderTransferResults.TransfersSuccessful);
-        });
-        it('should return TransfersSuccessful for a partial fill when taker has ample assets for the fill but not for the whole order', async () => {
-            await erc20Token2.setBalance.awaitTransactionSuccessAsync(
-                takerAddress,
-                signedOrder.takerAssetAmount.dividedBy(2),
-                {
-                    from: owner,
-                },
-            );
-            await erc20Token2.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.takerAssetAmount, {
-                from: takerAddress,
-            });
-            await erc20Token.setBalance.awaitTransactionSuccessAsync(makerAddress, signedOrder.makerAssetAmount, {
-                from: owner,
-            });
-            await erc20Token.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.makerAssetAmount, {
-                from: makerAddress,
-            });
-            await feeErc20Token.setBalance.awaitTransactionSuccessAsync(takerAddress, signedOrder.takerFee, {
-                from: owner,
-            });
-            await feeErc20Token.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.takerFee, {
-                from: takerAddress,
-            });
-            await feeErc20Token.setBalance.awaitTransactionSuccessAsync(makerAddress, signedOrder.makerFee, {
-                from: owner,
-            });
-            await feeErc20Token.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.makerFee, {
-                from: makerAddress,
-            });
-            const orderTransferResults = await devUtils.getSimulatedOrderTransferResults.callAsync(
-                signedOrder,
-                takerAddress,
-                signedOrder.takerAssetAmount.dividedBy(2),
-            );
-            expect(orderTransferResults).to.equal(OrderTransferResults.TransfersSuccessful);
-        });
-    });
-    describe('getSimulatedOrdersTransferResults', async () => {
-        it('should simulate the transfers of each order independently from one another', async () => {
-            // Set balances and allowances to exactly enough to fill a single order
-            await erc20Token2.setBalance.awaitTransactionSuccessAsync(takerAddress, signedOrder.takerAssetAmount, {
-                from: owner,
-            });
-            await erc20Token2.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.takerAssetAmount, {
-                from: takerAddress,
-            });
-            await erc20Token.setBalance.awaitTransactionSuccessAsync(makerAddress, signedOrder.makerAssetAmount, {
-                from: owner,
-            });
-            await erc20Token.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.makerAssetAmount, {
-                from: makerAddress,
-            });
-            await feeErc20Token.setBalance.awaitTransactionSuccessAsync(takerAddress, signedOrder.takerFee, {
-                from: owner,
-            });
-            await feeErc20Token.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.takerFee, {
-                from: takerAddress,
-            });
-            await feeErc20Token.setBalance.awaitTransactionSuccessAsync(makerAddress, signedOrder.makerFee, {
-                from: owner,
-            });
-            await feeErc20Token.approve.awaitTransactionSuccessAsync(erc20Proxy.address, signedOrder.makerFee, {
-                from: makerAddress,
-            });
-            const [
-                orderTransferResults1,
-                orderTransferResults2,
-            ] = await devUtils.getSimulatedOrdersTransferResults.callAsync(
-                [signedOrder, signedOrder],
-                [takerAddress, takerAddress],
-                [signedOrder.takerAssetAmount, signedOrder.takerAssetAmount],
-            );
-            expect(orderTransferResults1).to.equal(OrderTransferResults.TransfersSuccessful);
-            expect(orderTransferResults2).to.equal(OrderTransferResults.TransfersSuccessful);
-        });
-    });
 });
 // tslint:disable:max-file-line-count

contracts/dev-utils/truffle-config.js

@@ -1,96 +0,0 @@
-/**
- * Use this file to configure your truffle project. It's seeded with some
- * common settings for different networks and features like migrations,
- * compilation and testing. Uncomment the ones you need or modify
- * them to suit your project as necessary.
- *
- * More information about configuration can be found at:
- *
- * truffleframework.com/docs/advanced/configuration
- *
- * To deploy via Infura you'll need a wallet provider (like truffle-hdwallet-provider)
- * to sign your transactions before they're sent to a remote public node. Infura accounts
- * are available for free at: infura.io/register.
- *
- * You'll also need a mnemonic - the twelve word phrase the wallet uses to generate
- * public/private key pairs. If you're publishing your code to GitHub make sure you load this
- * phrase from a file you've .gitignored so it doesn't accidentally become public.
- *
- */
-
-// const HDWalletProvider = require('truffle-hdwallet-provider');
-// const infuraKey = "fj4jll3k.....";
-//
-// const fs = require('fs');
-// const mnemonic = fs.readFileSync(".secret").toString().trim();
-
-module.exports = {
-    /**
-     * Networks define how you connect to your ethereum client and let you set the
-     * defaults web3 uses to send transactions. If you don't specify one truffle
-     * will spin up a development blockchain for you on port 9545 when you
-     * run `develop` or `test`. You can ask a truffle command to use a specific
-     * network from the command line, e.g
-     *
-     * $ truffle test --network <network-name>
-     */
-
-    networks: {
-        // Useful for testing. The `development` name is special - truffle uses it by default
-        // if it's defined here and no other network is specified at the command line.
-        // You should run a client (like ganache-cli, geth or parity) in a separate terminal
-        // tab if you use this network and you must also set the `host`, `port` and `network_id`
-        // options below to some value.
-        //
-        // development: {
-        //  host: "127.0.0.1",     // Localhost (default: none)
-        //  port: 8545,            // Standard Ethereum port (default: none)
-        //  network_id: "*",       // Any network (default: none)
-        // },
-        // Another network with more advanced options...
-        // advanced: {
-        // port: 8777,             // Custom port
-        // network_id: 1342,       // Custom network
-        // gas: 8500000,           // Gas sent with each transaction (default: ~6700000)
-        // gasPrice: 20000000000,  // 20 gwei (in wei) (default: 100 gwei)
-        // from: <address>,        // Account to send txs from (default: accounts[0])
-        // websockets: true        // Enable EventEmitter interface for web3 (default: false)
-        // },
-        // Useful for deploying to a public network.
-        // NB: It's important to wrap the provider as a function.
-        // ropsten: {
-        // provider: () => new HDWalletProvider(mnemonic, `https://ropsten.infura.io/v3/YOUR-PROJECT-ID`),
-        // network_id: 3,       // Ropsten's id
-        // gas: 5500000,        // Ropsten has a lower block limit than mainnet
-        // confirmations: 2,    // # of confs to wait between deployments. (default: 0)
-        // timeoutBlocks: 200,  // # of blocks before a deployment times out  (minimum/default: 50)
-        // skipDryRun: true     // Skip dry run before migrations? (default: false for public nets )
-        // },
-        // Useful for private networks
-        // private: {
-        // provider: () => new HDWalletProvider(mnemonic, `https://network.io`),
-        // network_id: 2111,   // This network is yours, in the cloud.
-        // production: true    // Treats this network as if it was a public net. (default: false)
-        // }
-    },
-
-    // Set default mocha options here, use special reporters etc.
-    mocha: {
-        // timeout: 100000
-    },
-
-    // Configure your compilers
-    compilers: {
-        solc: {
-            version: '0.5.9',
-            settings: {
-                evmVersion: 'constantinople',
-                optimizer: {
-                    enabled: true,
-                    runs: 1000000,
-                    details: { yul: true, deduplicate: true, cse: true, constantOptimizer: true },
-                },
-            },
-        },
-    },
-};

contracts/dev-utils/tsconfig.json

@@ -4,11 +4,9 @@
     "include": ["./src/**/*", "./test/**/*", "./generated-wrappers/**/*"],
     "files": [
         "generated-artifacts/DevUtils.json",
-        "generated-artifacts/EthBalanceChecker.json",
         "generated-artifacts/LibAssetData.json",
         "generated-artifacts/LibTransactionDecoder.json",
-        "generated-artifacts/OrderTransferSimulationUtils.json",
-        "generated-artifacts/OrderValidationUtils.json"
+        "generated-artifacts/EthBalanceChecker.json"
     ],
     "exclude": ["./deploy/solc/solc_bin"]
 }

contracts/erc1155/CHANGELOG.json

@@ -1,32 +1,4 @@
 [
-    {
-        "version": "1.2.0-beta.0",
-        "changes": [
-            {
-                "note": "Add `mintKnownFungibleTokensAsync()`, `isNonFungibleItemAsync()`, `isFungibleItemAsync()`, `getOwnerOfAsync()`, `getBalanceAsync()` to `Erc1155Wrapper`.",
-                "pr": 1819
-            }
-        ],
-        "timestamp": 1570135330
-    },
-    {
-        "timestamp": 1568744790,
-        "version": "1.1.15",
-        "changes": [
-            {
-                "note": "Dependencies updated"
-            }
-        ]
-    },
-    {
-        "timestamp": 1567521715,
-        "version": "1.1.14",
-        "changes": [
-            {
-                "note": "Dependencies updated"
-            }
-        ]
-    },
     {
         "timestamp": 1566446343,
         "version": "1.1.13",