svetakvsundhar commented on pull request #17159: URL: https://github.com/apache/beam/pull/17159#issuecomment-1083695799
R: @TheNeuralBit Hi Brian, As we discussed, on testing this function, we'll need OAuth since we are reading from a BQ table, and thus it might make more sense to write an integration test for this. Currently to unit test this, we can 1) Write a mock schema and test the code after that (skipping getting the schema from a BQ table). We could ensure usertypes.__annotations is giving us what we expect it to. 2) I can write the other function (WritetoBQ) in this PR, and use it to help with the unit testing. Let me know your thoughts, I think 1) might be a good option for the unit test purpose. -- 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]
