Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

rpctest executable #757

Merged
merged 18 commits into from Mar 19, 2016
Merged

rpctest executable #757

merged 18 commits into from Mar 19, 2016

Conversation

debris
Copy link
Collaborator

@debris debris commented Mar 17, 2016

changes:

  • rpc-tests executable, used for running rpc-tests
  • declarative spec deserialization. It does not use unwraps and is partly used in new code.
  • fixed typos in frontier_test.json and frontier_like_test.json/
  • ethcore/pod_state is now public (it's used by rpctest executable).

todo in next prs:

  • [refactor] use new spec and blockchain deserialization instead of old one.

@debris debris added the A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. label Mar 17, 2016
@debris debris added A0-pleasereview 🤓 Pull request needs code review. and removed A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. labels Mar 17, 2016
#[derive(Debug, PartialEq, Deserialize)]
pub struct Builtin {
name: String,
linear: Linear,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be of type enum { LinearType(Linear) } (we might have things other than linear).

@gavofyork
Copy link
Contributor

pretty damn beautiful.

Copyright 2015, 2016 Ethcore (UK) Limited

Usage:
parity --json <test-file> --name <test-name> [options]
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rpctest

@gavofyork gavofyork added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Mar 18, 2016
gavofyork added a commit that referenced this pull request Mar 19, 2016
@gavofyork gavofyork merged commit 1d82213 into master Mar 19, 2016
@gavofyork gavofyork deleted the ethrpc_test branch March 19, 2016 07:45
@arkpar arkpar added this to the 1.0 Parity milestone Mar 20, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants