Skip to content

Commit

Permalink
Checkstyle fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
sydney-munro committed Mar 30, 2022
1 parent 9022a7c commit ab6979b
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 4 deletions.
Expand Up @@ -83,8 +83,15 @@ public void testCreateBucketPubSubNotification() {
assertNotNull("Unable to determine project", PROJECT);
assertNotNull("Topic Admin Client did not start up", topicAdminClient);

com.example.storage.bucket.CreateBucketPubSubNotification.createBucketPubSubNotification(bucketName, TOPIC,
CUSTOM_ATTRIBUTES, EVENT_TYPES, OBJECT_NAME_PREFIX, PAYLOAD_FORMAT, SELF_LINK, ETAG);
com.example.storage.bucket.CreateBucketPubSubNotification.createBucketPubSubNotification(
bucketName,
TOPIC,
CUSTOM_ATTRIBUTES,
EVENT_TYPES,
OBJECT_NAME_PREFIX,
PAYLOAD_FORMAT,
SELF_LINK,
ETAG);
assertThat(stdOut.getCapturedOutputAsUtf8String()).contains(TOPIC);
}

Expand Down
Expand Up @@ -81,7 +81,8 @@ public void testDeleteBucketPubSubNotification() {
assertNotNull("Topic Admin Client did not start up", topicAdminClient);

Notification notification = storage.createNotification(bucketName, notificationInfo);
com.example.storage.bucket.DeleteBucketPubSubNotification.deleteBucketPubSubNotification(bucketName,
com.example.storage.bucket.DeleteBucketPubSubNotification.deleteBucketPubSubNotification(
bucketName,
notification.getNotificationId());
assertThat(stdOut.getCapturedOutputAsUtf8String()).contains(
"Successfully deleted notification");
Expand All @@ -96,7 +97,8 @@ public void testNotificationNotFound() {
// Do a delete first.
storage.deleteNotification(bucketName, notification.getNotificationId());
// Now try to delete again.
com.example.storage.bucket.DeleteBucketPubSubNotification.deleteBucketPubSubNotification(bucketName,
com.example.storage.bucket.DeleteBucketPubSubNotification.deleteBucketPubSubNotification(
bucketName,
notification.getNotificationId());
assertThat(stdOut.getCapturedOutputAsUtf8String()).contains("Failed to find notification");
}
Expand Down

0 comments on commit ab6979b

Please sign in to comment.