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

When there are failed simulations the error message prints all simulations instead of just the failed simulations #108

Closed
anoopgarlapati opened this issue May 5, 2020 · 3 comments

Comments

@anoopgarlapati
Copy link

I am using plugin version 3.3.1 and I observed that behavior of error message changed in a recent version. I have around 20 simulations as part of test suite and when there is a failure in one of them, the error message now reads "There're failed simulations: simulation1FQN, simulation2FQN, ..., simulation20FQN" when in fact only one simulation failed. This is tedious for us to check all the simulation reports to see which actually failed.
This used to work properly before where the error message was "Some simulations failed: failedSimulationFQN".
I've tracked down the problem to an enhancement commit 82ad702#diff-7bc60cc739fa8b7bb64a003d960de9ebR85-R86. If you now look at the marked lines, the message now logs all simulations instead of just failed simulations. I would like this behavior to be restored.

Thanks,
Anoop

@eshepelyuk
Copy link
Collaborator

eshepelyuk commented May 5, 2020

Thnx @anoopgarlapati thnx for the hint, seems as a regression.
I expect this to be fixed and released a new version in some days.

@eshepelyuk
Copy link
Collaborator

@anoopgarlapati release is plnned after fixing this one and #107
join our gitter channel to receive new release announcement earlier ;)

@anoopgarlapati
Copy link
Author

@eshepelyuk Thanks for the quick fix and release!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants