wyvernprotocol / wyvern-v3

Wyvern Protocol v3.1, Ethereum implementation
https://wyvernprotocol.com
MIT License
298 stars 121 forks source link

Refactor tests #34

Closed MarvinJanssen closed 3 years ago

MarvinJanssen commented 3 years ago

First refactor for improved performance and readability. Replaces nested promises with async/await pattern. Tests should still be functionally equivalent.

The tests still rely on side-effects at times (contract state). The next step would be to make them self-contained. Tests may also benefit from beforeEach() hooks to cut down on repetition.