Storage: File: add bucket check

lib/storage/bucket.js

@@ -198,6 +198,10 @@ Bucket.prototype.delete = function(callback) {
  * var file = bucket.file('my-existing-file.png');
  */
 Bucket.prototype.file = function(name) {
+  if (!name) {
+    throw Error('A file name must be specified.');
+  }
+
   return new File(this, name);
 };
 

lib/storage/file.js

@@ -64,10 +64,6 @@ var STORAGE_UPLOAD_BASE_URL = 'https://www.googleapis.com/upload/storage/v1/b';
  * @constructor
  */
 function File(bucket, name, metadata) {
-  if (!name) {
-    throw Error('A file name must be specified.');
-  }
-
   this.bucket = bucket;
   this.makeReq_ = bucket.makeReq_.bind(bucket);
   this.metadata = metadata || {};

test/storage/bucket.js

@@ -118,15 +118,19 @@ describe('Bucket', function() {
     var file;
     var metadata = { a: 'b' };
 
-    beforeEach(function() {
-      file = bucket.file(FILE_NAME, metadata);
+    it('should throw if no name was provided', function() {
+      assert.throws(function() {
+        file = bucket.file();
+      }, /A file name must be specified./);
     });
 
     it('should return a File object', function() {
+      file = bucket.file(FILE_NAME, metadata);
       assert(file instanceof FakeFile);
     });
 
     it('should pass filename to File object', function() {
+      file = bucket.file(FILE_NAME, metadata);
       assert.equal(file.name, FILE_NAME);
     });
   });

test/storage/file.js

@@ -125,12 +125,6 @@ describe('File', function() {
   });
 
   describe('initialization', function() {
-    it('should throw if no name is provided', function() {
-      assert.throws(function() {
-        new File(bucket);
-      }, /A file name must be specified/);
-    });
-
     it('should assign file name', function() {
       assert.equal(file.name, FILE_NAME);
     });