Skip to content

Commit

Permalink
chore: ignore Update related gRPC tests which are failing due to b/27…
Browse files Browse the repository at this point in the history
…0215524 (#1917)

When b/270215524 is fixed and rolled out we should be able to revert this
  • Loading branch information
BenWhitehead authored Feb 28, 2023
1 parent dd71222 commit 69eaaf0
Show file tree
Hide file tree
Showing 7 changed files with 23 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -345,6 +345,7 @@ private void doTestUniformBucketLevelAccessAclImpact(IamConfiguration iamConfigu
}

@Test
@CrossRun.Ignore(transports = Transport.GRPC) // todo(b/270215524)
public void testEnableAndDisableUniformBucketLevelAccessOnExistingBucket() throws Exception {
String bpoBucket = generator.randomBucketName();
BucketInfo.IamConfiguration ublaDisabledIamConfiguration =
Expand Down Expand Up @@ -492,6 +493,7 @@ public void testUnspecifiedPublicAccessPreventionOnBucket() throws Exception {
}

@Test
@CrossRun.Ignore(transports = Transport.GRPC) // todo(b/270215524)
public void changingPAPDoesNotAffectUBLA() throws Exception {
String bucketName = generator.randomBucketName();
try (TemporaryBucket tempB =
Expand Down Expand Up @@ -534,6 +536,7 @@ public void changingPAPDoesNotAffectUBLA() throws Exception {
}

@Test
@CrossRun.Ignore(transports = Transport.GRPC) // todo(b/270215524)
public void changingUBLADoesNotAffectPAP() throws Exception {
String bucketName = generator.randomBucketName();
try (TemporaryBucket tempB =
Expand Down Expand Up @@ -615,6 +618,7 @@ public void testRetentionPolicyNoLock() throws Exception {

@Test
@SuppressWarnings({"unchecked", "deprecation"})
@CrossRun.Ignore(transports = Transport.GRPC) // todo(b/270215524)
public void testEnableAndDisableBucketPolicyOnlyOnExistingBucket() throws Exception {
String bpoBucket = generator.randomBucketName();
try (TemporaryBucket tempB =
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,7 @@ public final class ITBucketLifecycleRulesTest {
@Inject public Storage storage;

@Test
@CrossRun.Ignore(transports = Transport.GRPC) // todo(b/270215524)
public void deleteRule_addingALabelToABucketWithASingleDeleteRuleOnlyModifiesTheLabels()
throws Exception {
LifecycleRule d1 =
Expand Down Expand Up @@ -89,6 +90,7 @@ public void condition_ageDays_0_shouldWork() throws Exception {
}

@Test
@CrossRun.Ignore(transports = Transport.GRPC) // todo(b/270215524)
public void deleteRule_modifyingLifecycleRulesMatchesLastOperation() throws Exception {
BucketInfo info;
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -126,6 +126,7 @@ public void testGetBucketAbortMPULifecycle() throws Exception {
}

@Test
@CrossRun.Ignore(transports = Transport.GRPC) // todo(b/270215524)
public void testDeleteLifecycleRules() throws Exception {
String bucketName = generator.randomBucketName();
BucketInfo bucketInfo =
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -148,6 +148,7 @@ public void testBucketCustomPlacmentConfigDualRegion() throws Exception {
}

@Test
@CrossRun.Ignore(transports = Transport.GRPC) // todo(b/270215524)
public void testBucketLogging() throws Exception {
String logsBucketName = generator.randomBucketName();
String loggingBucketName = generator.randomBucketName();
Expand Down Expand Up @@ -189,6 +190,7 @@ public void testBucketLogging() throws Exception {
}

@Test
@CrossRun.Ignore(transports = Transport.GRPC) // todo(b/270215524)
public void testRemoveBucketCORS() {
String bucketName = generator.randomBucketName();
List<Cors.Origin> origins = ImmutableList.of(Cors.Origin.of("http://cloud.google.com"));
Expand Down Expand Up @@ -237,6 +239,7 @@ public void testRemoveBucketCORS() {
}

@Test
@CrossRun.Ignore(transports = Transport.GRPC) // todo(b/270215524)
public void testRpoConfig() {
String rpoBucket = generator.randomBucketName();
try {
Expand All @@ -259,6 +262,7 @@ public void testRetentionPolicyLock() {
}

@Test
@CrossRun.Ignore(transports = Transport.GRPC) // todo(b/270215524)
public void testRetentionPolicyLockRequesterPays() {
retentionPolicyLockRequesterPays(true);
}
Expand Down Expand Up @@ -386,6 +390,7 @@ public void testEnableDisableBucketDefaultEventBasedHold() {
}

@Test
@CrossRun.Ignore(transports = Transport.GRPC) // todo(b/270215524)
public void testCreateBucketWithAutoclass() {
String bucketName = generator.randomBucketName();
storage.create(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,7 @@ public class ITKmsTest {
@Inject public KmsFixture kms;

@Test
@CrossRun.Ignore(transports = Transport.GRPC) // todo(b/270215524)
public void testClearBucketDefaultKmsKeyName() {
String bucketName = generator.randomBucketName();
Bucket remoteBucket =
Expand All @@ -87,6 +88,7 @@ public void testClearBucketDefaultKmsKeyName() {
}

@Test
@CrossRun.Ignore(transports = Transport.GRPC) // todo(b/270215524)
public void testUpdateBucketDefaultKmsKeyName() {
String bucketName = generator.randomBucketName();
Bucket remoteBucket =
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -113,6 +113,7 @@ public ImmutableList<Param> parameters() {
}

@Test
@CrossRun.Ignore(transports = Transport.GRPC) // todo(b/270215524)
public void testBucketLabels() throws Exception {
BucketInfo bucket = newBucketInfo(param.initial);
try (TemporaryBucket tempB =
Expand All @@ -125,6 +126,7 @@ public void testBucketLabels() throws Exception {
}

@Test
@CrossRun.Ignore(transports = Transport.GRPC) // todo(b/270215524)
public void testBlobMetadata() {
BlobInfo blob = newBlobInfo(param.initial);
Blob gen1 = storage.create(blob, BlobTargetOption.doesNotExist());
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -571,6 +571,7 @@ public void testListBlobsMultiplePages() {
}

@Test
@CrossRun.Ignore(transports = Transport.GRPC) // todo(b/270215524)
public void testUpdateBlob() {
String blobName = "test-update-blob";
BlobInfo blob = BlobInfo.newBuilder(bucket, blobName).build();
Expand All @@ -584,6 +585,7 @@ public void testUpdateBlob() {
}

@Test
@CrossRun.Ignore(transports = Transport.GRPC) // todo(b/270215524)
public void testUpdateBlobReplaceMetadata() {
String blobName = "test-update-blob-replace-metadata";
ImmutableMap<String, String> metadata = ImmutableMap.of("k1", "a");
Expand All @@ -605,6 +607,7 @@ public void testUpdateBlobReplaceMetadata() {
}

@Test
@CrossRun.Ignore(transports = Transport.GRPC) // todo(b/270215524)
public void testUpdateBlobMergeMetadata() {
String blobName = "test-update-blob-merge-metadata";
ImmutableMap<String, String> metadata = ImmutableMap.of("k1", "a");
Expand All @@ -625,6 +628,7 @@ public void testUpdateBlobMergeMetadata() {
}

@Test
@CrossRun.Ignore(transports = Transport.GRPC) // todo(b/270215524)
public void testUpdateBlobUnsetMetadata() {

String blobName = "test-update-blob-unset-metadata";
Expand Down Expand Up @@ -1260,6 +1264,7 @@ public void testEnableDisableTemporaryHold() {
}

@Test
@CrossRun.Ignore(transports = Transport.GRPC) // todo(b/270215524)
public void testAttemptObjectDeleteWithEventBasedHold() {
String blobName = generator.randomObjectName();
BlobInfo blobInfo = BlobInfo.newBuilder(bucket, blobName).setEventBasedHold(true).build();
Expand All @@ -1276,6 +1281,7 @@ public void testAttemptObjectDeleteWithEventBasedHold() {
}

@Test
@CrossRun.Ignore(transports = Transport.GRPC) // todo(b/270215524)
public void testAttemptDeletionObjectTemporaryHold() {
String blobName = generator.randomObjectName();
BlobInfo blobInfo = BlobInfo.newBuilder(bucket, blobName).setTemporaryHold(true).build();
Expand Down Expand Up @@ -1404,6 +1410,7 @@ public void testAutoContentTypeWriter() throws IOException {
}

@Test
@CrossRun.Ignore(transports = Transport.GRPC) // todo(b/270215524)
public void testBlobTimeStorageClassUpdated() {

String blobName = generator.randomObjectName();
Expand Down

0 comments on commit 69eaaf0

Please sign in to comment.