jonathansudhakar1 commented on issue #20876: URL: https://github.com/apache/beam/issues/20876#issuecomment-6583337647
This looks fixed on master. All three problems from the report have since been corrected in the Python snippet under "An Example Test for a Composite Transform": - The `TestPipeline` import was fixed to `from apache_beam.testing.test_pipeline import TestPipeline`, and the `WORDS` input was moved into the test, in https://github.com/apache/beam/commit/3c3bfa36eb92e104229db9e827d723bed8ae0840. - The missing `import unittest` and `import apache_beam as beam` were added in #25425 (https://github.com/apache/beam/commit/ad5e17edb8f500955f8eced56f4624ef833ccee0). As a check, I copied the current snippet from `website/www/site/content/en/documentation/pipelines/test-your-pipeline.md` (at 9fb00aacf21) into a file, added a `unittest.main()` call, and ran it against the Python SDK at that commit. `test_count` passed (`Ran 1 test ... OK`). Unless I have missed something, this issue can be closed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
