pass name for view parameter in table.exists (#280)

googleapis · Aug 31, 2018 · af01dcf · af01dcf
1 parent 6338364
commit af01dcf
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 5 deletions.
diff --git a/src/table.js b/src/table.js
@@ -689,7 +689,12 @@ Please use the format 'prezzy' or '${instance.name}/tables/prezzy'.`
       gaxOptions = {};
     }
 
-    this.getMetadata(gaxOptions, err => {
+    const reqOpts = {
+      view: 'name',
+      gaxOptions: gaxOptions,
+    };
+
+    this.getMetadata(reqOpts, err => {
       if (err) {
         if (err.code === 5) {
           callback(null, false);

diff --git a/test/table.js b/test/table.js
@@ -1134,8 +1134,8 @@ describe('Bigtable/Table', function() {
 
   describe('exists', function() {
     it('should not require gaxOptions', function(done) {
-      table.getMetadata = function(gaxOptions) {
-        assert.deepStrictEqual(gaxOptions, {});
+      table.getMetadata = function(options_) {
+        assert.deepStrictEqual(options_.gaxOptions, {});
         done();
       };
 
@@ -1144,9 +1144,18 @@ describe('Bigtable/Table', function() {
 
     it('should pass gaxOptions to getMetadata', function(done) {
       let gaxOptions = {};
+      table.getMetadata = function(options_) {
+        assert.strictEqual(options_.gaxOptions, gaxOptions);
+        done();
+      };
 
-      table.getMetadata = function(gaxOptions_) {
-        assert.strictEqual(gaxOptions_, gaxOptions);
+      table.exists(gaxOptions, assert.ifError);
+    });
+
+    it('should pass view = name to getMetadata', function(done) {
+      let gaxOptions = {};
+      table.getMetadata = function(options_) {
+        assert.strictEqual(options_.view, 'name');
         done();
       };