beam-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Kenneth Knowles (JIRA)" <>
Subject [jira] [Resolved] (BEAM-2001) Verify PAssert execution in all runners
Date Fri, 26 Jan 2018 22:09:00 GMT


Kenneth Knowles resolved BEAM-2001.
       Resolution: Fixed
    Fix Version/s: Not applicable

All subtasks are resolved for the runners. And most likely since there are PAssertTest cases
that check it fails when it should there's not too much room for trouble anyhow.

> Verify PAssert execution in all runners
> ---------------------------------------
>                 Key: BEAM-2001
>                 URL:
>             Project: Beam
>          Issue Type: Test
>          Components: runner-core
>            Reporter: Aviem Zur
>            Priority: Blocker
>             Fix For: Not applicable
> Currently, {{PAssert}} assertions may not happen and tests will pass while silently hiding
> For {{ValidatesRunner}} tests to truly validate a runner supports the model we need to
verify the {{PAssert}} assertions actually ran.
> See [dev list discussion|].
> In order to reduce duplication, for runners which support metrics we could verify this
in {{TestPipeline}}, removing the need for the runner itself to make this assertion (See

This message was sent by Atlassian JIRA

View raw message