Skip to content

Commit 2ef3f8d

Browse files
committed
Checks failed
1 parent 11ccfc0 commit 2ef3f8d

2 files changed

Lines changed: 31 additions & 2 deletions

File tree

Storage/tests/System/ManageBucketsTest.php

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -605,6 +605,13 @@ public function encryptionEnforcementConfigs()
605605
];
606606
}
607607

608+
/**
609+
* Creates a bucket with IP filter disabled.
610+
*
611+
* @group storage-ipfilter
612+
*
613+
* @return Bucket
614+
*/
608615
public function testCreateBucketWithIpFilterDisabled()
609616
{
610617
$projectId = self::getProjectId(getenv('GOOGLE_CLOUD_PHP_TESTS_KEY_PATH'));
@@ -647,7 +654,14 @@ public function testCreateBucketWithIpFilterDisabled()
647654
}
648655

649656
/**
657+
* Deletes an IP from the bucket IP filter.
658+
*
650659
* @depends testCreateBucketWithIpFilterDisabled
660+
* @group storage-ipfilter
661+
*
662+
* @param Bucket $bucket The bucket.
663+
*
664+
* @return void
651665
*/
652666
public function testDeleteBucketIpFilter(Bucket $bucket)
653667
{
@@ -665,6 +679,13 @@ public function testDeleteBucketIpFilter(Bucket $bucket)
665679
$this->assertArrayNotHasKey('allowedIpCidrRanges', $info['ipFilter']['publicNetworkSource']);
666680
}
667681

682+
/**
683+
* Tests creating a bucket with invalid IP filter fails.
684+
*
685+
* @group storage-ipfilter
686+
*
687+
* @return void
688+
*/
668689
public function testCreateBucketWithInvalidIpFilterFails()
669690
{
670691
$this->expectException(BadRequestException::class);

Storage/tests/Unit/BucketTest.php

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -808,7 +808,11 @@ public function testGetBucketWithIpFilter()
808808
$info['ipFilter']['publicNetworkSource']['allowedIpCidrRanges']
809809
);
810810
}
811-
811+
/**
812+
* Tests disabling a bucket IP filter.
813+
*
814+
* @return void
815+
*/
812816
public function testDisableBucketIpFilter()
813817
{
814818
$ipFilterConfig = [
@@ -834,7 +838,11 @@ public function testDisableBucketIpFilter()
834838
$this->assertArrayHasKey('ipFilter', $info);
835839
$this->assertEquals('Disabled', $info['ipFilter']['mode']);
836840
}
837-
841+
/**
842+
* Tests deleting an IP from a bucket IP filter.
843+
*
844+
* @return void
845+
*/
838846
public function testDeleteBucketIpFilter()
839847
{
840848
$ipFilterConfig = [

0 commit comments

Comments
 (0)