fix: snapshot.delete() no longer throws (#251)

googleapis · Jan 24, 2019 · dfd6507 · dfd6507
1 parent a089112
commit dfd6507
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 2 deletions.
diff --git a/src/snapshot.js b/src/snapshot.js
@@ -176,7 +176,7 @@ class Snapshot extends common.ServiceObject {
      * @name Snapshot#compute
      * @type {Compute}
      */
-    this.compute = isDisk ? scope.compute : scope;
+    this.compute = isDisk ? scope.zone.compute : scope;
     /**
      * @name Snapshot#name
      * @type {string}

diff --git a/system-test/compute.js b/system-test/compute.js
@@ -206,12 +206,13 @@ describe('Compute', () => {
       return zone.disk(DISK_NAME).getMetadata();
     });
 
-    it('should take a snapshot', async () => {
+    it('should take and delete a snapshot', async () => {
       const [snapshot, operation] = await disk
         .snapshot(generateName('snapshot'))
         .create();
       await operation.promise();
       await snapshot.getMetadata();
+      await snapshot.delete();
     });
 
     it('should run operation as a promise', async () => {

diff --git a/test/snapshot.js b/test/snapshot.js
@@ -95,11 +95,16 @@ describe('Snapshot', function() {
           assert.strictEqual(this, scope);
           done();
         },
+        zone: {
+          compute: COMPUTE,
+        },
       };
 
       const snapshot = new Snapshot(scope, SNAPSHOT_NAME);
       assert(snapshot instanceof ServiceObject);
 
+      assert.strictEqual(snapshot.compute, scope.zone.compute);
+
       const calledWith = snapshot.calledWith_[0];
       assert.strictEqual(calledWith.methods.create, true);