Skip to content

Add SerializationTest for logging and minor fixes#1116

Merged
mziccard merged 1 commit intogoogleapis:logging-alphafrom
mziccard:logging-serialization-test
Jul 19, 2016
Merged

Add SerializationTest for logging and minor fixes#1116
mziccard merged 1 commit intogoogleapis:logging-alphafrom
mziccard:logging-serialization-test

Conversation

@mziccard
Copy link
Copy Markdown
Contributor

No description provided.

@mziccard mziccard added the api: logging Issues related to the Cloud Logging API. label Jul 18, 2016
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jul 18, 2016
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+1.1%) to 85.508% when pulling 22a1862 on mziccard:logging-serialization-test into e3321a7 on GoogleCloudPlatform:logging-alpha.

@mziccard
Copy link
Copy Markdown
Contributor Author

/cc @lesv

.build();
private final Logging serviceMockReturnsOptions = createStrictMock(Logging.class);
private final LoggingOptions mockOptions = createMock(LoggingOptions.class);
private Logging pubsub;

This comment was marked as spam.

@lesv
Copy link
Copy Markdown
Contributor

lesv commented Jul 18, 2016

LGTM

@mziccard mziccard merged commit b1860bf into googleapis:logging-alpha Jul 19, 2016
mziccard added a commit to mziccard/gcloud-java that referenced this pull request Jul 20, 2016
meltsufin pushed a commit that referenced this pull request Dec 22, 2025
* fix: Add flush() calls in samples

* Add optional in comment

* 🦉 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>
chingor13 pushed a commit that referenced this pull request Jan 6, 2026
…a 8 (#1800) (#1116)

* chore: Java 8 unit test to build code in Java 17 and run tests on Java 8 (#1800)

* Our Java projects are configured to produce Java 8-compatible bytecode via https://github.com/googleapis/java-shared-config/blob/main/pom.xml#L848. This unit test change ensures this config is applied correctly.

* For the repositories that mark "dependencies (8)" and "dependencies (11)" as required, they should point to only "dependencies (17)" via `.github/sync-repo-settings.yaml` and repo's Settings tab.
Source-Link: googleapis/synthtool@cbe0100
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-java:latest@sha256:32851debfefed2b66038e0141f1b5c2103bb59ba80b7475adbc10ef7abab3de7

* dependencies (17) and build.sh

---------

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: losalex <[email protected]>
Co-authored-by: Tomo Suzuki <[email protected]>
suztomo pushed a commit to suztomo/google-cloud-java that referenced this pull request Mar 11, 2026
* fix: Add flush() calls in samples

* Add optional in comment

* 🦉 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>
suztomo pushed a commit that referenced this pull request Mar 30, 2026
🤖 I have created a release *beep* *boop*
---


### Updating meta-information for bleeding-edge SNAPSHOT release.

---
This PR was generated with [Release Please](https://togithub.com/googleapis/release-please). See [documentation](https://togithub.com/googleapis/release-please#release-please).
lqiu96 pushed a commit that referenced this pull request Apr 1, 2026
🤖 I have created a release *beep* *boop*
---


### Updating meta-information for bleeding-edge SNAPSHOT release.

---
This PR was generated with [Release Please](https://togithub.com/googleapis/release-please). See [documentation](https://togithub.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: logging Issues related to the Cloud Logging API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants