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

[OraclizeQueryTest] Test converted TypeScript ABI #10

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

[OraclizeQueryTest] Test converted TypeScript ABI #10

rojosnow opened this issue Jun 4, 2018 · 5 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

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

This comment has been minimized.

Copy link

@gitcoinbot gitcoinbot commented Jun 7, 2018

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


This issue now has a funding of 0.05 ETH (30.33 USD @ $606.52/ETH) attached to it.

@gitcoinbot

This comment has been minimized.

Copy link

@gitcoinbot gitcoinbot commented Jun 26, 2018

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


Work has been started.

These users each claimed they can complete the work by 8 months, 4 weeks ago.
Please review their action plans below:

1) arku has started work.

I'll refer the existing tests and add tests for OrderLib and OraclizeQueryTest types.

Learn more on the Gitcoin Issue Details page.

@eswarasai eswarasai moved this from To do to In progress in MARKET Protocol Jun 26, 2018
@arku arku mentioned this issue Jun 26, 2018
3 of 3 tasks complete
@eswarasai eswarasai moved this from In progress to In review in MARKET Protocol Jun 26, 2018
@gitcoinbot

This comment has been minimized.

Copy link

@gitcoinbot gitcoinbot commented Jun 26, 2018

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


Work for 0.05 ETH (22.9 USD @ $458.08/ETH) has been submitted by:

  1. @arun1595
  2. @arun1595

@pelsasser please take a look at the submitted work:


MARKET Protocol automation moved this from In review to Done Jun 26, 2018
@eswarasai

This comment has been minimized.

Copy link
Contributor

@eswarasai eswarasai commented Jun 26, 2018

@arun1595 -- Thanks for quick turn around on this one. I'll let @pelsasser release the funds :)

@gitcoinbot

This comment has been minimized.

Copy link

@gitcoinbot gitcoinbot commented Jun 26, 2018

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


The funding of 0.05 ETH (22.24 USD @ $444.79/ETH) attached to this issue has been approved & issued to @arun1595.

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.

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