Fix rendering logic of results panel controls#1082
Conversation
|
could you suggeset a test for this behaviour?, or this is while the simulation runs ? |
For starters, a completed job should not have a status notification. So we could test for this. Not sure how to test while running. Our workchain generation fixture is not so flexible I think. I'll add the first though. |
I meant to get this behaviour in qe app (error) , to check the logic and the response from the app you want |
Like I said, this is a monitoring feature, so we would need a running process. This is a good fixture to add. I would do this in a separate PR that handles the testing suite as a whole (planned). But at least for the static finished process, we can ensure that the status notification is empty. This would have caught this bug. |
30d5bbb to
3189607
Compare
Thie PR fixes incorrect logic in deciding which widgets to add to a results panel that was incorrectly rendering status notification for completed workflows.