[ https://issues.apache.org/jira/browse/BEAM-520?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15576063#comment-15576063 ]
Geetha Bijjam commented on BEAM-520: ------------------------------------ Thanks for the info. I am working on multiple_output_pardo_test.py and will send a PR once the Travis Build succeeds. > Update Python SDK example tests to use assert_that > -------------------------------------------------- > > Key: BEAM-520 > URL: https://issues.apache.org/jira/browse/BEAM-520 > Project: Beam > Issue Type: Improvement > Components: sdk-py > Reporter: Ahmet Altay > Priority: Minor > Labels: starter > > Most of our examples use assert_that to test examples: > https://github.com/apache/incubator-beam/blob/python-sdk/sdks/python/apache_beam/examples/complete/autocomplete_test.py#L38 > Some of our examples use this pattern: > 1) Create a path(s) > 2) Construct fake command line arguments using these paths > 3) Construct an argparse object to parse these flags > 4) Do the (often trivial logic) > 5) Write to a file > 6) Manually open and read the file > 7) Compare results. > https://github.com/apache/incubator-beam/blob/python-sdk/sdks/python/apache_beam/examples/cookbook/multiple_output_pardo_test.py > As well as being cumbersome, this obscures the core of what is being > illustrated and tested. As many as possible tests should be updated to use > assert_that. -- This message was sent by Atlassian JIRA (v6.3.4#6332)