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

Fix CI #9270

Merged
merged 1 commit into from
Aug 2, 2018
Merged

Fix CI #9270

merged 1 commit into from
Aug 2, 2018

Conversation

cheme
Copy link
Contributor

@cheme cheme commented Aug 2, 2018

Quick fix for broken tests.
See : https://gitlab.parity.io/parity/parity/pipelines/21961 build errors in latest master

@parity-cla-bot
Copy link

It looks like @cheme hasn't signed our Contributor License Agreement, yet.

The purpose of a CLA is to ensure that the guardian of a project's outputs has the necessary ownership or grants of rights over all contributions to allow them to distribute under the chosen licence.
Wikipedia

You can read and sign our full Contributor License Agreement at the following URL: https://cla.parity.io

Once you've signed, please reply to this thread with [clabot:check] to prove it.

Many thanks,

Parity Technologies CLA Bot

@cheme
Copy link
Contributor Author

cheme commented Aug 2, 2018

[clabot:check]

@parity-cla-bot
Copy link

It looks like @cheme signed our Contributor License Agreement. 👍

Many thanks,

Parity Technologies CLA Bot

@dvdplm dvdplm added A0-pleasereview 🤓 Pull request needs code review. F4-tests 💻 Tests need fixing, improving or augmenting. M4-core ⛓ Core client code / Rust. labels Aug 2, 2018
Copy link
Collaborator

@dvdplm dvdplm left a comment

Choose a reason for hiding this comment

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

lgtm although I'm not sure how #9140 could have snuck by CI without failing.

@dvdplm dvdplm added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Aug 2, 2018
Copy link
Collaborator

@sorpaas sorpaas left a comment

Choose a reason for hiding this comment

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

LGTM!

@@ -612,7 +612,8 @@ mod tests {
let mut vm_tracer = NoopVMTracer;

let address = {
let mut ext = Externalities::new(state, &setup.env_info, &setup.machine, 0, get_test_origin(), &mut setup.sub_state, OutputPolicy::InitContract(None), &mut tracer, &mut vm_tracer, false);
let mut ext = Externalities::new(state, &setup.env_info, &setup.machine, &setup.schedule, 0, get_test_origin(), &mut setup.sub_state, OutputPolicy::InitContract(None), &mut tracer, &mut vm_tracer, false);

Copy link
Collaborator

Choose a reason for hiding this comment

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

Extra empty line. Definitely fine but not sure this is intentional or not.

@sorpaas sorpaas merged commit f442665 into openethereum:master Aug 2, 2018
ordian added a commit to ordian/parity that referenced this pull request Aug 2, 2018
…rp_sync_on_light_client

* 'master' of https://github.com/paritytech/parity:
  Propagate transactions for next 4 blocks. (openethereum#9265)
  decode block rlp less often (openethereum#9252)
  Fix eternalities tests can_create (missing parameter) (openethereum#9270)
  Update ref to `parity-common` and update `seek` behaviour (openethereum#9257)
  Comply EIP-86 with the new definition (openethereum#9140)
  Check if synced when using eth_getWork (openethereum#9193) (openethereum#9210)
@5chdn 5chdn added this to the 2.1 milestone Aug 21, 2018
@5chdn 5chdn added B0-patchthis and removed F4-tests 💻 Tests need fixing, improving or augmenting. labels Aug 21, 2018
This was referenced Aug 23, 2018
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. M4-core ⛓ Core client code / Rust.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants