Added a unit test but since everything is mocked it doesn't actually call HttpBigQueryRpc.listTables (nor as any other listTables test). Please let me know if there is any other way to properly test this change.

There's a non-mocked integration test for list tables in ITBigQueryTest which you could add to, or write a new IT based on it

Thanks. I wasn't able to run the ITBigQueryTests due to ITBigQueryTest.beforeClass throws java.lang.IllegalArgumentException: A project ID is required for this service but could not be determined from the builder or the environment. Please set a project ID using the builder.. Trying things with a builder didn't help, getting null pointer exceptions. Are there any prerequisite for executing integration tests?

You'll need to set the environment variables GOOGLE_CLOUD_PROJECT and GOOGLE_APPLICATION_CREDENTIALS to specify the project for the test to run on and the credentials json file for that account

The BigtableIT tests may require configuration via system properties. For CI, we set them here. We set bigtable.env, bigtable.project, bigtable.instance, and bigtable.bigtable.

Thanks guys, I found the env variables needed but obviously I don't have credentials to access that project so the test fails on 401 Unauthorized

The idea would be to use your own project instead to test, setting the values of the environment variables to be your own project and credentials

@vvviiimmm, please run the following command to clean up the code formatting:

mvn com.coveo:fmt-maven-plugin:format

@vvviiimmm, please run the following command to clean up the code formatting:

mvn com.coveo:fmt-maven-plugin:format

Codecov Report

Merging #4306 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master    #4306      +/-   ##
============================================
- Coverage     49.17%   49.16%   -0.01%     
- Complexity    20988    21915     +927     
============================================
  Files          1996     2078      +82     
  Lines        194629   207084   +12455     
  Branches      21796    24084    +2288     
============================================
+ Hits          95702   101813    +6111     
- Misses        90817    97128    +6311     
- Partials       8110     8143      +33

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 395b016...c339196. Read the comment docs.

@JesseLovelace @kolea2 is this something we should:

• accept as is?
• try to fix it up?
• close it out?

@JesseLovelace, can you please finish this review?

@sduskis sure, i will add that to IT test and raise a new PR.

This is replaced by #4701 which includes these changes + integration tests