Addressed codacy issues in DNS. by mderka · Pull Request #850 · googleapis/google-cloud-java

mderka · 2016-04-04T17:49:07Z

This includes

removing static formatter from example
removing and unifying imports and full qualification
removing unused serializable import from Dns
replacing assert not true with assertFalse in ITDnsTest
removing unused method from LocalDnsHelperTest.

I deliberately skipped some items which I discussed before with @ajkannan and we concluded that they should be ignored.

coveralls · 2016-04-04T17:55:17Z

Coverage remained the same at 85.537% when pulling 5be34a9 on mderka:codacy into d17e325 on GoogleCloudPlatform:master.

Sign in to view

    System.out.printf("Description: %s%n", zone.description());
-    System.out.printf("Created: %s%n", FORMATTER.format(new Date(zone.creationTimeMillis())));
+    DateFormat formatter = new SimpleDateFormat("YYYY-MM-dd HH:mm:ss");
+    System.out.printf("Created: %s%n", formatter.format(new Date(zone.creationTimeMillis())));


aozarov · 2016-04-04T19:33:21Z

Some more issues:

ZoneTest.java:352:64: error: ',' is not followed by whitespace.
LocalDnsHelper.CallRegex should have its member variables be final.
LocalDnsHelper delayChange member can be final.
LocalDnsHelper create method Javadoc is pointing to its contained class (Using {@code} is preferred in this case).
OptionParsers should have a private constructor.
ITDnsTest line 156 could use '==' instead of equals for enum value.
ProjectInfo.fromPb can directly return the created Quota.
pom.xml redundnt groupId (inherit from parent).

Sign in to view

  static Function<Change, ChangeRequest> fromPbFunction(final Dns dns, final String zoneName) {
    return new Function<Change, ChangeRequest>() {
      @Override
-      public ChangeRequest apply(com.google.api.services.dns.model.Change pb) {


- removing static formatter from example - removing and unifying imports and full qualification - removing unused serializable import from Dns - replacing assert not true with assertFalse in ITDnsTest - removing unused method from LocalDnsHelperTest.

- added joda formatter to example - style in ZoneTest - visibility in LocalDnsHelper - added private constructor to OptionParsers - adjusted pom.xml

mderka · 2016-04-04T22:03:53Z

I addressed the pointed out issues. I rebased this PR due to conflict with some recently merged PR.

aozarov · 2016-04-04T22:16:09Z

Looks good. Feel free to merge after fixing the ProjectInfo.toPb and also cleaning up the other fully-qualified imports (and making sure build is green).

coveralls · 2016-04-04T23:25:23Z

Coverage decreased (-0.001%) to 85.543% when pulling ff65582 on mderka:codacy into 560d0b6 on GoogleCloudPlatform:master.

Sign in to view

          System.out.printf("%nID: %s%n", change.generatedId());
          System.out.printf("Status: %s%n", change.status());
-          System.out.printf("Started: %s%n", FORMATTER.format(change.startTimeMillis()));
+          System.out.printf("Started: %s%n", FORMATTER.print(change.startTimeMillis()));


…onfig to v1.5.1 (#850) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [com.google.cloud:google-cloud-shared-config](https://togithub.com/googleapis/java-shared-config) | `1.5.0` -> `1.5.1` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-shared-config/1.5.1/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-shared-config/1.5.1/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-shared-config/1.5.1/compatibility-slim/1.5.0)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-shared-config/1.5.1/confidence-slim/1.5.0)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>googleapis/java-shared-config</summary> ### [`v1.5.1`](https://togithub.com/googleapis/java-shared-config/blob/HEAD/CHANGELOG.md#151-httpsgithubcomgoogleapisjava-shared-configcomparev150v151-2022-06-30) [Compare Source](https://togithub.com/googleapis/java-shared-config/compare/v1.5.0...v1.5.1) ##### Dependencies - update dependency org.graalvm.buildtools:junit-platform-native to v0.9.12 ([#482](https://togithub.com/googleapis/java-shared-config/issues/482)) ([fbfc6dc](https://togithub.com/googleapis/java-shared-config/commit/fbfc6dc1329faaead3a3114c8599d9267722e7f0)) - update dependency org.graalvm.buildtools:native-maven-plugin to v0.9.12 ([#483](https://togithub.com/googleapis/java-shared-config/issues/483)) ([336cb78](https://togithub.com/googleapis/java-shared-config/commit/336cb7827b36583228c9e2b85871ae72f4c55975)) </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, click this checkbox. --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-document-ai).

- [ ] Regenerate this pull request now. chore(bazel): update gax-java to 2.18.4 PiperOrigin-RevId: 463115700 Source-Link: googleapis/googleapis@52130a9 Source-Link: https://github.com/googleapis/googleapis-gen/commit/6a4d9d9bb3afb20b0f5fa4f5d9f6740b1d0eb19a Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiNmE0ZDlkOWJiM2FmYjIwYjBmNWZhNGY1ZDlmNjc0MGIxZDBlYjE5YSJ9

…-plugin to v3.4.1 (#850) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [org.apache.maven.plugins:maven-javadoc-plugin](https://maven.apache.org/plugins/) ([source](https://togithub.com/apache/maven-javadoc-plugin)) | `3.4.0` -> `3.4.1` | [![age](https://badges.renovateapi.com/packages/maven/org.apache.maven.plugins:maven-javadoc-plugin/3.4.1/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/org.apache.maven.plugins:maven-javadoc-plugin/3.4.1/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/org.apache.maven.plugins:maven-javadoc-plugin/3.4.1/compatibility-slim/3.4.0)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/org.apache.maven.plugins:maven-javadoc-plugin/3.4.1/confidence-slim/3.4.0)](https://docs.renovatebot.com/merge-confidence/) | --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Renovate will not automatically rebase this PR, because other commits have been found. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, click this checkbox. ⚠ **Warning**: custom changes will be lost. --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-datacatalog).

…ator_java versions (#850) - [ ] Regenerate this pull request now. PiperOrigin-RevId: 472750037 Source-Link: googleapis/googleapis@88f2ea3 Source-Link: https://github.com/googleapis/googleapis-gen/commit/230a5588306aae18fe8f2a57f14d4039ad72c901 Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiMjMwYTU1ODgzMDZhYWUxOGZlOGYyYTU3ZjE0ZDQwMzlhZDcyYzkwMSJ9

🤖 I have created a release *beep* *boop* --- ## [2.3.3](googleapis/java-trace@v2.3.2...v2.3.3) (2022-09-15) ### Dependencies * Update dependency com.google.cloud:google-cloud-shared-dependencies to v3.0.3 ([#850](googleapis/java-trace#850)) ([2f4c81e](googleapis/java-trace@2f4c81e)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).

…#850)

…1.3 (googleapis#850) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [com.google.cloud:libraries-bom](https://cloud.google.com/java/docs/bom) ([source](https://togithub.com/googleapis/java-cloud-bom)) | `26.1.2` -> `26.1.3` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/26.1.3/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/26.1.3/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/26.1.3/compatibility-slim/26.1.2)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/26.1.3/confidence-slim/26.1.2)](https://docs.renovatebot.com/merge-confidence/) | --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Renovate will not automatically rebase this PR, because other commits have been found. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, click this checkbox. ⚠ **Warning**: custom changes will be lost. --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-recommender).

…s#1687) (googleapis#850) * chore(java): add a note in README for migrated split repos (googleapis#1687) * chore(java): add a note in README for migrated split repos Disable renovate bot and flaky bot for split repositories that have moved to the Java monorepo. The Java monorepo will pass the "monorepo=True" parameter to java.common_templates method in its owlbot.py files so that the migration note will not appear in the README in the monorepo. Co-authored-by: Jeff Ching <chingor@google.com> Source-Link: googleapis/synthtool@d4b2916 Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-java:latest@sha256:edae91ccdd2dded2f572ec341a768ad180305a3e8fbfd93064b28e237d35920a * 🦉 Updates from OwlBot post-processor See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com> Co-authored-by: Jeff Ching <chingor@google.com> Co-authored-by: Deepankar Dixit <90280028+ddixit14@users.noreply.github.com>

🤖 I have created a release *beep* *boop* --- ### Updating meta-information for bleeding-edge SNAPSHOT release. --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).

* build: upgrade Java microgenerator to v2.2.1 # v2.2.1 ## Bug Fixes - gax batching regression (#863) (3a6f168) # v2.2.0 ## Features - Add REST AIP-151 LRO suport (cb1b534) - enable self signed JWT for http (#850) (aba0ec0) - Implement DIREGAPIC LRO annotations (#832) (d7b29e0) - REGAPIC initial implementation (#824) (fdcfe70) - REGAPIC Multitransport implementation (grpc+rest) (#833) (445daf4) ## Bug Fixes - [bazel] fix rest transport handling in assembly rule (#835) (92f7c1c) - DIREGAPIC LRO generated tests logic (#838) (8ae8e6f) - fix diregapic-lro logic (#834) (957f69a) PiperOrigin-RevId: 405492216 Source-Link: googleapis/googleapis@4381242 Source-Link: https://github.com/googleapis/googleapis-gen/commit/8d1ecc73009d949803fe1745f4617546a8cb74fc Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiOGQxZWNjNzMwMDlkOTQ5ODAzZmUxNzQ1ZjQ2MTc1NDZhOGNiNzRmYyJ9 * 🦉 Updates from OwlBot See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

🤖 I have created a release *beep* *boop* --- ### [3.6.2](googleapis/java-logging@v3.6.1...v3.6.2) (2022-02-03) ### Dependencies * **java:** update actions/github-script action to v5 ([#1339](googleapis/java-logging#1339)) ([#851](googleapis/java-logging#851)) ([7a5ee11](googleapis/java-logging@7a5ee11)) * update actions/github-script action to v5 ([#849](googleapis/java-logging#849)) ([286728a](googleapis/java-logging@286728a)) * update dependency com.google.cloud:google-cloud-shared-dependencies to v2.7.0 ([#852](googleapis/java-logging#852)) ([eb2eef5](googleapis/java-logging@eb2eef5)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).

* feat: Add AWS Session Token to Metadata Requests * Adding testing functionality for requests made. Refactoring header passing logic

🤖 I have created a release *beep* *boop* --- ## [1.6.0](googleapis/google-auth-library-java@v1.5.3...v1.6.0) (2022-03-15) ### Features * Add AWS Session Token to Metadata Requests ([#850](googleapis/google-auth-library-java#850)) ([577e9a5](googleapis/google-auth-library-java@577e9a5)) ### Bug Fixes * ImmutableSet converted to List for Impersonated Credentials ([#732](googleapis/google-auth-library-java#732)) ([7dcd549](googleapis/google-auth-library-java@7dcd549)) * update library docs ([#868](googleapis/google-auth-library-java#868)) ([a081015](googleapis/google-auth-library-java@a081015)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).

* build: upgrade Java microgenerator to v2.2.1 # v2.2.1 ## Bug Fixes - gax batching regression (googleapis#863) (3a6f168) # v2.2.0 ## Features - Add REST AIP-151 LRO suport (cb1b534) - enable self signed JWT for http (googleapis#850) (aba0ec0) - Implement DIREGAPIC LRO annotations (googleapis#832) (d7b29e0) - REGAPIC initial implementation (googleapis#824) (fdcfe70) - REGAPIC Multitransport implementation (grpc+rest) (googleapis#833) (445daf4) ## Bug Fixes - [bazel] fix rest transport handling in assembly rule (googleapis#835) (92f7c1c) - DIREGAPIC LRO generated tests logic (googleapis#838) (8ae8e6f) - fix diregapic-lro logic (googleapis#834) (957f69a) PiperOrigin-RevId: 405492216 Source-Link: googleapis/googleapis@4381242 Source-Link: https://github.com/googleapis/googleapis-gen/commit/8d1ecc73009d949803fe1745f4617546a8cb74fc Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiOGQxZWNjNzMwMDlkOTQ5ODAzZmUxNzQ1ZjQ2MTc1NDZhOGNiNzRmYyJ9 * 🦉 Updates from OwlBot See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

* feat: Add AWS Session Token to Metadata Requests * Adding testing functionality for requests made. Refactoring header passing logic

🤖 I have created a release *beep* *boop* --- ## [1.6.0](googleapis/google-auth-library-java@v1.5.3...v1.6.0) (2022-03-15) ### Features * Add AWS Session Token to Metadata Requests ([#850](googleapis/google-auth-library-java#850)) ([0713bf2](googleapis/google-auth-library-java@0713bf2)) ### Bug Fixes * ImmutableSet converted to List for Impersonated Credentials ([#732](googleapis/google-auth-library-java#732)) ([8f11708](googleapis/google-auth-library-java@8f11708)) * update library docs ([#868](googleapis/google-auth-library-java#868)) ([d964089](googleapis/google-auth-library-java@d964089)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).

Fixes a NullPointerException introduced in #841 which occured when trying to check the the requesterPays status of a file while authenticated with an invalid service account. Refs: #849, #841

googlebot added the cla: yes This human has signed the Contributor License Agreement. label Apr 4, 2016

mderka added the api: dns Issues related to the Cloud DNS API. label Apr 4, 2016

mderka reviewed Apr 4, 2016
View reviewed changes

aozarov reviewed Apr 4, 2016
View reviewed changes

mderka added 2 commits April 4, 2016 15:00

Fixed some other minor issues:

c178259

- added joda formatter to example - style in ZoneTest - visibility in LocalDnsHelper - added private constructor to OptionParsers - adjusted pom.xml

mderka force-pushed the codacy branch from b236529 to c178259 Compare April 4, 2016 22:02

Unified imports of model objects.

ff65582

mziccard reviewed Apr 5, 2016
View reviewed changes

mderka merged commit 3e7069a into googleapis:master Apr 5, 2016

github-actions bot pushed a commit that referenced this pull request Oct 4, 2022

deps: update dependency com.google.cloud:google-cloud-core to v2.8.18 (…

6053905

…#850)

chingor13 pushed a commit that referenced this pull request Feb 24, 2026

feat: Add AWS Session Token to Metadata Requests (#850)

577e9a5

* feat: Add AWS Session Token to Metadata Requests * Adding testing functionality for requests made. Refactoring header passing logic

chingor13 pushed a commit that referenced this pull request Mar 12, 2026

feat: Add AWS Session Token to Metadata Requests (#850)

0713bf2

* feat: Add AWS Session Token to Metadata Requests * Adding testing functionality for requests made. Refactoring header passing logic

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Addressed codacy issues in DNS. #850

Addressed codacy issues in DNS. #850
mderka merged 3 commits intogoogleapis:masterfrom
mderka:codacy

mderka commented Apr 4, 2016

Uh oh!

coveralls commented Apr 4, 2016

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

aozarov commented Apr 4, 2016

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

mderka commented Apr 4, 2016

Uh oh!

aozarov commented Apr 4, 2016

Uh oh!

coveralls commented Apr 4, 2016

Uh oh!

This comment was marked as spam.

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Conversation

mderka commented Apr 4, 2016

Uh oh!

coveralls commented Apr 4, 2016

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

aozarov commented Apr 4, 2016

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

mderka commented Apr 4, 2016

Uh oh!

aozarov commented Apr 4, 2016

Uh oh!

coveralls commented Apr 4, 2016

Uh oh!

This comment was marked as spam.

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants