Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tests: Update the golden files' copyright to 2025 #3513

Merged
merged 6 commits into from
Jan 6, 2025
Merged

Conversation

blakeli0
Copy link
Collaborator

@blakeli0 blakeli0 commented Jan 3, 2025

Fixes #3512.

The copyright year for GAPIC generated files are automatically set to current year. This creates a problem that all the tests are going to automatically fail when new year comes. Updating all the golden files' copyright to 2025 in this PR for now.

There are golden test files need to be updated in spring-cloud-gcp as well since it is dependent on the generator.

In the future, we need to come up with a better strategy that the tests do not fail automatically, created an issue #3547 to track the future enhancement.

@product-auto-label product-auto-label bot added the size: xs Pull request size is extra small. label Jan 3, 2025
@product-auto-label product-auto-label bot added size: m Pull request size is medium. and removed size: xs Pull request size is extra small. labels Jan 3, 2025
@@ -61,6 +62,7 @@ void initialSanityCheck() {
assertTrue(context.containsServices());
}

@Disabled
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IIUC, this is a manual fix and this @Disabled is added temporarily and will be removed after this update is merged?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, I was testing if adding @Disabled to this test is going to fix most of the issues or not. It proves not, we need to update integration tests, showcase tests, hermetic build tests and spring-cloud-gcp tests. I'm going to update all the golden files in this repo for now to unblock the release, and create issues for improvements.

@blakeli0 blakeli0 changed the title Update ComposerTest.java tests: Update the golden files' copyright to 2025 Jan 6, 2025
Copy link

sonarqubecloud bot commented Jan 6, 2025

Copy link

sonarqubecloud bot commented Jan 6, 2025

Quality Gate Passed Quality Gate passed for 'java_showcase_integration_tests'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@blakeli0
Copy link
Collaborator Author

blakeli0 commented Jan 6, 2025

/gcbrun

1 similar comment
@blakeli0
Copy link
Collaborator Author

blakeli0 commented Jan 6, 2025

/gcbrun

@blakeli0 blakeli0 requested review from zhumin8 and lqiu96 January 6, 2025 21:38
@blakeli0 blakeli0 marked this pull request as ready for review January 6, 2025 21:38
@blakeli0 blakeli0 merged commit 8e5d662 into main Jan 6, 2025
50 of 51 checks passed
@blakeli0 blakeli0 deleted the blakeli0-patch-4 branch January 6, 2025 21:53
zhumin8 added a commit that referenced this pull request Jan 7, 2025
…yright to 2025 (#3550)

follow-up to #3513, 

Changes in this PR:
- generated by ` mvn test -DupdateUnitGoldens ` from
gapic-generator-java/. Note that unit tests without this change did not
fail. Filed #3556
to investigate and maybe remove these in the future.
- library-generation unit test golden files. They are causing test
failures in
https://github.com/googleapis/sdk-platform-java/actions/runs/12659566586/job/35278951040?pr=3462
These 2 files are used in unit tests to test logic in owlbot to insert
header when they are missing. e.g.
https://github.com/googleapis/sdk-platform-java/blob/67f5ea23f50eaf57925cdf34d51c7b0bfcc67cf3/hermetic_build/library_generation/owlbot/synthtool/languages/java.py#L113-L116
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size: m Pull request size is medium.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Nightly builds are failing due to copyright header update
4 participants