aIbrahiim commented on PR #39655: URL: https://github.com/apache/beam/pull/39655#issuecomment-5222011184
> Hi @aIbrahiim, can you add a test here in yaml - https://github.com/apache/beam/tree/master/sdks/python/apache_beam/yaml/tests long as the setup isn't heavy. If it is, then move to extended_tests. Thanks Thanks @derrickaw , DicomSearch needs a real GCP Healthcare DICOM store, so setup is heavy and not a good fit for tests/. and I already have unit tests with a mock in yaml_io_test.py but I can add a YAML test under extended_tests/ instead -- 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]
