Damon

Results 39 comments of Damon

@seaneagan, Instead of hardcoding every testing case, I want to intelligently autogenerate them. I've been working on `install` first as a model for the others. I want to develop a...

@seaneagan , Next week I will provide my first PR for this. Just letting you know I'm still active and excited to work on this.

@seaneagan , I'm struggling with executing unittests with async methods. I need to step away for a while and come back with a fresh mind.

@seaneagan , I finally figured out how to test within an async context. At some point I am going to do another PR. Meanwhile, please see my progress: https://github.com/damondouglas/den/tree/master/test

@seaneagan do you think pub-dartlang should have a discovery endpoint api? For this particular use case, we could call `pub.packages.list` providing the `q` parameter and limit the response to names...

> Can we add or update existing tests in [StorageApiSinkRowUpdateIT](https://github.com/apache/beam/blob/master/sdks/java/io/google-cloud-platform/src/test/java/org/apache/beam/sdk/io/gcp/bigquery/StorageApiSinkRowUpdateIT.java)? That class contains integration tests for RowMutationInformation. Would be good to have some real end-to-end tests to check this new...

> @damondouglas Sorry I mean the new behavior of having multiple sections in the CSN (e.g. `a\b\c\d`). I don't think that's covered in our integration tests. @ahmedabu98 I added the...

Thank you, @ahmedabu98 . I will merge after checks pass.

> Thank you, @ahmedabu98 . I will merge after checks pass. FYI failing checks have nothing to do with code in the PR. Ran additional internal tests (see internal chat)...

Good day, @talatuyarer. Thank you again for making this contribution! Would you like to still work on this PR or should I close it? We appreciate what you do for...