Chore: Support non-HNS buckets in moveObjectAtomic

Removed HNS-only restriction comments.

Restructured tests for non-HNS coverage.

Author: thiyaguk09

storage/src/move_object_atomic.php

@@ -27,7 +27,7 @@
 use Google\Cloud\Storage\StorageClient;
 
 /**
- * Move an object to a new name within HNS-enabled bucket.
+ * Move an object to a new name within bucket.
  *
  * @param string $bucketName The name of your Cloud Storage bucket.
  *        (e.g. 'my-bucket')

storage/test/ObjectsTest.php

@@ -153,29 +153,27 @@ public function testManageObject()
 
     public function testMoveObjectAtomic()
     {
-        $bucketName = self::$bucketName . '-hns';
         $objectName = 'test-object-' . time();
         $newObjectName = $objectName . '-moved';
-        $bucket = self::$storage->createBucket($bucketName, [
-            'hierarchicalNamespace' => ['enabled' => true],
+        $bucket = self::$storage->createBucket(self::$bucketName, [
             'iamConfiguration' => ['uniformBucketLevelAccess' => ['enabled' => true]]
         ]);
 
         $object = $bucket->upload('test', ['name' => $objectName]);
         $this->assertTrue($object->exists());
 
         $output = self::runFunctionSnippet('move_object_atomic', [
-            $bucketName,
+            self::$bucketName,
             $objectName,
             $newObjectName
         ]);
 
         $this->assertEquals(
             sprintf(
                 'Moved gs://%s/%s to gs://%s/%s' . PHP_EOL,
-                $bucketName,
+                self::$bucketName,
                 $objectName,
-                $bucketName,
+                self::$bucketName,
                 $newObjectName
             ),
             $output