Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[OrderLib] Test converted TypeScript ABI #11

Closed
rojosnow opened this issue Jun 4, 2018 · 0 comments · Fixed by #19
Closed

[OrderLib] Test converted TypeScript ABI #11

rojosnow opened this issue Jun 4, 2018 · 0 comments · Fixed by #19

Comments

@rojosnow
Copy link
Contributor

@rojosnow rojosnow commented Jun 4, 2018

Before you start work

Please read our contribution guidelines and if there is a bounty involved please also see here

If you have ongoing work from other bounties with us where funding has not been released, please do not pick up a new issue. We would like to involve as many contributors as possible and parallelize the work flow as much as possible.

Please make sure to comment in the issue here immediately after starting work so we know your plans for implementation and a timeline.

Please also note that in order for work to be accepted, all code must be accompanied by test cases as well.

User Story

As a developer, I need to ensure the MARKET API TypeScript ABIs work correctly so that I can correctly interface with the MARKET contracts.

Why Is this Needed?

Summary: We need to have automated test coverage for all TypeScript ABIs. This started as part of https://github.com/MARKETProtocol/MARKET.js/issues/14. Due to the size and scope of the testing, we created a new project and repo for the MARKET.js types.

Description

Type: Enhancement

Current Behavior

There is minimal test coverage.

Expected Behavior

We need 100% test coverage on the TypeScript ABIs.

Solution

Summary:
Typechain is being used to generate types and the ABIs from which to generate the types is specified as a glob pattern in the package.json under config.typechain.include which is currently set to: Market*|CollateralToken|OraclizeQueryTest|ERC20|OrderLib

Reference the work done in issue #2.

Definition of Done

  • Create tests for the OrderLib ABI
@eswarasai eswarasai added this to Backlog in MARKET Protocol via automation Jun 5, 2018
@eswarasai eswarasai moved this from Backlog to To do in MARKET Protocol Jun 5, 2018
@arku arku mentioned this issue Jun 26, 2018
3 of 3 tasks complete
@eswarasai eswarasai moved this from To do to In review in MARKET Protocol Jun 26, 2018
MARKET Protocol automation moved this from In review to Done Jun 26, 2018
eswarasai referenced this issue Jun 26, 2018
Add tests for OrderLib and OraclizeQueryTest types
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Linked pull requests

Successfully merging a pull request may close this issue.

3 participants
You can’t perform that action at this time.