chore: recursive comparison in hermetic build IT

[diegomarquezp]

Fixes #2540

Sample error message when running library_generation/test/integration_tests.py

Generation finished successfully. Will now compare differences between generated and existing libraries
******************************
Checking for differences in 'java-apigee-connect'.
  The expected library is in /usr/local/google/home/diegomarquezp/Desktop/sdk-platform-java/library_generation/test/resources/integration/golden/java-apigee-connect.
  The actual library is in /usr/local/google/home/diegomarquezp/Desktop/sdk-platform-java/library_generation/output/google-cloud-java/java-apigee-connect. 
  Some files (found in both folders) are differing:
   -  README.md
   -  proto-google-cloud-apigee-connect-v1/src/main/java/com/google/cloud/apigeeconnect/v1/ConnectionProto.java

[JoeWang1127]

Hi @diegomarquezp, thanks for fixing integration test.

Could you also try to fix the difference in the generated files by changing committish in configuration file?

[diegomarquezp]

Hi @diegomarquezp, thanks for fixing integration test.

Could you also try to fix the difference in the generated files by changing committish in configuration file?

Thanks @JoeWang1127, this definitely went right for google-cloud-java, but I'm seeing some strange behavior in java-bigtable that doesn't seem to only depend on the commitish. I'm tracking the issue in #2549 , I suspect is a special treatment case that should be done in generate_library.sh (googleapis and googleapis-gen protos are different from java-bigtable protos)

@JoeWang1127 @blakeli0 what if we go for fixing java-bigtable as a follow up since the purpose of this PR is to fix the testing logic? I have disabled the bigtable test for now.

[sonarcloud]

Please retry analysis of this Pull-Request directly on SonarCloud

[JoeWang1127]

I added document-ai to the test configuration because I didn't find a commit in alloydb, alloydb-connectors and apigee-connect.

Since we have comments in alloydb, should we remove this entry?

[diegomarquezp]

Agreed, I'm removing document-ai to speed the test up

[JoeWang1127]

@JoeWang1127 @blakeli0 what if we go for fixing java-bigtable as a follow up since the purpose of this PR is to fix the testing logic? I have disabled the bigtable test for now.

LGTM.

[sonarcloud]

Quality Gate passed for 'gapic-generator-java-root'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

[sonarcloud]

Quality Gate passed for 'java_showcase_integration_tests'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud