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

Gitcoin API tests are failing since API started rounding results #3434

Closed
LefterisJP opened this issue Sep 9, 2021 · 2 comments
Closed

Gitcoin API tests are failing since API started rounding results #3434

LefterisJP opened this issue Sep 9, 2021 · 2 comments

Comments

@LefterisJP
Copy link
Member

Problem Definition

2021-09-09_18-10

The gitcoin queries:

DEBUG    rotkehlchen.chain.ethereum.gitcoin.api:api.py:149 Greenlet-0: Querying gitcoin: https://gitcoin.co/api/v0.1/grants/contributions_rec_report/?id=149&from_timestamp=2017-09-25&to_timestamp=2017-09-25
DEBUG    rotkehlchen.chain.ethereum.gitcoin.api:api.py:149 Greenlet-0: Querying gitcoin: https://gitcoin.co/api/v0.1/grants/contributions_rec_report/?id=149&from_timestamp=2021-06-01&to_timestamp=2021-06-10&page=1&format=json
DEBUG    rotkehlchen.chain.ethereum.gitcoin.api:api.py:149 Greenlet-0: Querying gitcoin: https://gitcoin.co/api/v0.1/grants/contributions_rec_report/?id=149&from_timestamp=2021-06-01&to_timestamp=2021-06-10&page=2&format=json

Test in question:

def test_get_grant_events(rotkehlchen_api_server, start_with_valid_premium):

Example transaction that is rounded up: https://zkscan.io/explorer/transactions/8ee8bfe2d07a0a9192549725aa3d3dcf78b7913a6bf187895c2510fb058ed314

Correct amount: 0.000475 ETH
Returned amount: 0.0005 ETH

Lots of errors like that.

Task

Make gitcoin fix their API and then unskip the tests.

@kelsos
Copy link
Member

kelsos commented Jul 12, 2022

@LefterisJP this issue doesn't make sense anymore right, considering we removed the Gitcoin API integration?

@LefterisJP
Copy link
Member Author

@LefterisJP this issue doesn't make sense anymore right, considering we removed the Gitcoin API integration?

Yes closing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants