Skip to content
This repository has been archived by the owner on Aug 8, 2018. It is now read-only.

closes #99 eth_gasLimit missing #103

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

4gn3s
Copy link

@4gn3s 4gn3s commented Feb 18, 2016

No description provided.

@4gn3s
Copy link
Author

4gn3s commented Mar 29, 2016

This feels really foolish, but what am I supposed to do to make the test work? I thought I should use a call similar to this: https://github.com/ethereum/pyethapp/blob/develop/pyethapp/rpc_client.py#L101
And yet when I dogenesis_block_info = client.call('eth_getBlockByNumber', 'earliest', False), it just returns a string "earliest". Is there something different about the testing environment that I'm missing?

@RomanZacharia
Copy link

@4gn3s It returns the 'earliest' because the 'call' method is redefined on line 6 75a2572#diff-e92b78f8db16a62992c87795ab883c44L4

@RomanZacharia
Copy link

@4gn3s There is nothing wrong with your code, It's all about the incorrect tests. You need starting the RPC Server before trying to connect to it.

4gn3s added 2 commits July 13, 2016 15:17
# Conflicts:
#	pyethapp/rpc_client.py
#	pyethapp/tests/test_rpc_client.py
@coveralls
Copy link

Coverage Status

Coverage increased (+0.4%) to 51.631% when pulling 02397d1 on 4gn3s:fix_eth_gasLimit_missing into 97c78c6 on ethereum:develop.

@4gn3s
Copy link
Author

4gn3s commented Jul 13, 2016

Thanks @RomanZacharia, I've fixed the tests

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

Successfully merging this pull request may close these issues.

3 participants