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.

Sign up for GitHub

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

Jump to bottom

docs: Add Kinesis ingestion samples #1947

Merged
merged 10 commits into from
Mar 21, 2024
Merged

docs: Add Kinesis ingestion samples #1947

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
63e67b6
docs: Adding samples for creating and updating Kinesis ingestion topics
michaelpri10 Mar 7, 2024
f198317
style: Format AdmitIT
michaelpri10 Mar 7, 2024
4d8a9d8
style: Re-format AdmitIT
michaelpri10 Mar 7, 2024
66f39c4
docs: Fix ingestion topic deletion in test
michaelpri10 Mar 7, 2024
e4c8a94
docs: Add test for updating existing ingestion settings and verify th…
michaelpri10 Mar 7, 2024
0496f52
docs: Use response to verify ingestion settings update
michaelpri10 Mar 13, 2024
adc9a3d
style: fix formatting in tests
michaelpri10 Mar 13, 2024
5336089
docs: Update AdminIT test to check for correct topic name
michaelpri10 Mar 15, 2024
ba6f728
🦉 Updates from OwlBot post-processor
gcf-owl-bot[bot] Mar 18, 2024
7e81589
Merge branch 'main' into ingestion-samples
michaelpri10 Mar 21, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
Prev Previous commit
Next Next commit
style: fix formatting in tests
  • Loading branch information
@michaelpri10
michaelpri10 committed Mar 13, 2024
commit adc9a3dba08967cd8cf3204d3f2954b6c13de379
3 changes: 2 additions & 1 deletion samples/snippets/src/main/java/pubsub/UpdateTopicTypeExample.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,8 @@ public static void updateTopicTypeExample(

Topic response = topicAdminClient.updateTopic(request);

System.out.println("Updated topic with Kinesis ingestion settings: " + response.getAllFields());
System.out.println(
"Updated topic with Kinesis ingestion settings: " + response.getAllFields());
}
}
}
Expand Down
9 changes: 6 additions & 3 deletions samples/snippets/src/test/java/pubsub/AdminIT.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -290,7 +290,8 @@ public void testAdmin() throws Exception {
// Update topic type to Kinesis ingestion.
UpdateTopicTypeExample.updateTopicTypeExample(
projectId, topicId, streamArn, consumerArn, awsRoleArn, gcpServiceAccount);
assertThat(bout.toString()).contains("google.pubsub.v1.Topic.name=" + ingestionTopicName.toString());
assertThat(bout.toString())
.contains("google.pubsub.v1.Topic.name=" + ingestionTopicName.toString());
assertThat(bout.toString()).contains(streamArn);
assertThat(bout.toString()).contains(consumerArn);
assertThat(bout.toString()).contains(awsRoleArn);
Expand All @@ -305,7 +306,8 @@ public void testAdmin() throws Exception {
// Test create topic with Kinesis ingestion settings.
CreateTopicWithKinesisIngestionExample.createTopicWithKinesisIngestionExample(
projectId, ingestionTopicId, streamArn, consumerArn, awsRoleArn, gcpServiceAccount);
assertThat(bout.toString()).contains("google.pubsub.v1.Topic.name=" + ingestionTopicName.toString());
assertThat(bout.toString())
.contains("google.pubsub.v1.Topic.name=" + ingestionTopicName.toString());
assertThat(bout.toString()).contains(streamArn);
assertThat(bout.toString()).contains(consumerArn);
assertThat(bout.toString()).contains(awsRoleArn);
Expand All @@ -315,7 +317,8 @@ public void testAdmin() throws Exception {
// Test update existing Kinesis ingestion settings.
UpdateTopicTypeExample.updateTopicTypeExample(
projectId, ingestionTopicId, streamArn, consumerArn2, awsRoleArn, gcpServiceAccount);
assertThat(bout.toString()).contains("google.pubsub.v1.Topic.name=" + ingestionTopicName.toString());
assertThat(bout.toString())
.contains("google.pubsub.v1.Topic.name=" + ingestionTopicName.toString());
assertThat(bout.toString()).contains(streamArn);
assertThat(bout.toString()).contains(consumerArn2);
assertThat(bout.toString()).contains(awsRoleArn);
Expand Down