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

Wasm math tests extended #6354

Merged
merged 6 commits into from
Aug 28, 2017
Merged

Wasm math tests extended #6354

merged 6 commits into from
Aug 28, 2017

Conversation

NikVolf
Copy link
Contributor

@NikVolf NikVolf commented Aug 22, 2017

No description provided.

@NikVolf NikVolf added A0-pleasereview 🤓 Pull request needs code review. M4-core ⛓ Core client code / Rust. labels Aug 22, 2017
@@ -415,36 +415,121 @@ fn storage_read() {
assert_eq!(Address::from(&result[12..32]), address);
}

macro_rules! reqrep_test {
Copy link
Collaborator

Choose a reason for hiding this comment

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

This could be just a function that accepts code and input

Copy link
Contributor Author

@NikVolf NikVolf Aug 28, 2017

Choose a reason for hiding this comment

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

but it in turn uses macro, and the testfile content should be known at compile time, no runtime loading there

Copy link
Collaborator

Choose a reason for hiding this comment

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

Right, that's exactly why I was suggesting making testfile content (code) an argument to the function.

@arkpar arkpar added A6-mustntgrumble 💦 Pull request has areas for improvement. The author need not address them before merging. and removed A0-pleasereview 🤓 Pull request needs code review. labels Aug 28, 2017
@gavofyork gavofyork merged commit abecd80 into master Aug 28, 2017
@gavofyork gavofyork deleted the wasm-math-extended branch August 28, 2017 12:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A6-mustntgrumble 💦 Pull request has areas for improvement. The author need not address them before merging. M4-core ⛓ Core client code / Rust.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants