Skip to content
This repository has been archived by the owner on Mar 3, 2021. It is now read-only.

test function with params will throw error #1438

Merged
merged 2 commits into from
Mar 27, 2020
Merged

Conversation

Aniket-Engg
Copy link
Collaborator

fixes #1432

@Aniket-Engg
Copy link
Collaborator Author

Aniket-Engg commented Mar 26, 2020

@yann300 I think we can not handle compilation error in separate manner as implemented in #1434 as there is a single error handler in IDE for all type of errors https://github.com/ethereum/remix-ide/blob/master/src/app/tabs/test-tab.js#L125 .
Either we need to use a generic interface for all errors generated from remix-tests or we have to update the error handling in IDE.

@Aniket-Engg
Copy link
Collaborator Author

Aniket-Engg commented Mar 27, 2020

I made one commit in this PR: ethereum/remix-ide#2656 to handle error like this one.

@Aniket-Engg Aniket-Engg merged commit 24bfc28 into master Mar 27, 2020
@Aniket-Engg Aniket-Engg deleted the testfn-with-params branch March 27, 2020 13:59
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.

'abi.encodeWithSignature' shows issue in remix-tests plugin
2 participants