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

etherlime compile and deploy aren't failing correctly #306

Open
elenadimitrova opened this issue Feb 17, 2020 · 4 comments
Open

etherlime compile and deploy aren't failing correctly #306

elenadimitrova opened this issue Feb 17, 2020 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@elenadimitrova
Copy link

elenadimitrova commented Feb 17, 2020

etherlime compile and etherlime deploy seem to be "hiding" errors causing CI service to think failing steps have passed. Sample green builds below for

Bad etherlime compile - see step "Compiling contracts"
https://circleci.com/gh/argentlabs/argent-contracts/77

Bad etherlime deploy - see step "Testing deployment scripts" and search for Your deployment script finished with failure!
https://circleci.com/gh/argentlabs/argent-contracts/78

EDIT: This is not the case with etherlime test which does fail correctly, example build https://circleci.com/gh/argentlabs/argent-contracts/56

EDIT: I just came upon an example where even etherlime test reports a false positive https://circleci.com/gh/argentlabs/argent-contracts/97

@ochikov
Copy link
Collaborator

ochikov commented Feb 18, 2020

Thank you for the issue. We are going to investigate it.

@elenadimitrova
Copy link
Author

Investigating a bit more it appears you are not throwing an error on compilation errors but just putting the error on the stack and rejecting the compilation promise.

@ochikov
Copy link
Collaborator

ochikov commented May 6, 2020

Thank you. I will investigate it as well and fix it in the next releases.

@elenadimitrova
Copy link
Author

Unfortunately due to this we managed to merge some changes that broke our development branch https://circleci.com/gh/argentlabs/argent-contracts/1238 :(

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants