ahmedabu98 opened a new pull request, #27434: URL: https://github.com/apache/beam/pull/27434
Most of our BigQuery write connector tests live in [BigQueryIOWriteTest](https://github.com/ahmedabu98/beam/blob/master/sdks/java/io/google-cloud-platform/src/test/java/org/apache/beam/sdk/io/gcp/bigquery/BigQueryIOWriteTest.java). I took a closer look recently and saw that it's in need of a cleanup. There are some tests that are duplicates, some that could be written more concisely, some have test names that are too vague (some names are even incorrect), some that run with an incorrect write method. This PR seeks to take care of some of these issues. This also fixes a long-standing flake `testTriggeredFileLoadsWithTempTablesToExistingNullSchemaTable[1]` ([example run](https://ci-beam.apache.org/job/beam_PreCommit_Java_GCP_IO_Direct_Cron/1482/testReport/org.apache.beam.sdk.io.gcp.bigquery/BigQueryIOWriteTest/testTriggeredFileLoadsWithTempTablesToExistingNullSchemaTable_1_/)) -- 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]
