preparing for testing

Author: stephenplusplus

README.md

@@ -11,6 +11,7 @@
 This client supports the following Google Cloud Platform services:
 
 * [Google BigQuery](#google-bigquery)
+* [Google Compute Engine](#google-compute-engine)
 * [Google Cloud Datastore](#google-cloud-datastore)
 * [Google Cloud DNS](#google-cloud-dns)
 * [Google Cloud Storage](#google-cloud-storage)
@@ -113,6 +114,43 @@ job.getQueryResults().on('data', function(row) {});
 ```
 
 
+## Google Compute Engine
+
+- [API Documentation][gcloud-compute-docs]
+- [Official Documentation][cloud-compute-docs]
+
+#### Preview
+
+```js
+var gcloud = require('gcloud');
+
+// Authorizing on a per-API-basis. You don't need to do this if you auth on a
+// global basis (see Authorization section above).
+
+var gce = gcloud.compute({
+  projectId: 'my-project',
+  keyFilename: '/path/to/keyfile.json'
+});
+
+// Create a new instance using the latest OS image of your choice
+var zone = gce.zone('us-central1-a');
+
+var config = {
+  os: 'ubuntu'
+};
+
+zone.createInstance('ubuntu-http', config, function(err, instance, operation) {
+  // `operation` lets you check the status of long-running tasks.
+
+  operation.onComplete(function(err, metadata) {
+    if (!err) {
+      // Instance created!
+    }
+  });
+});
+```
+
+
 ## Google Cloud Datastore
 
 - [API Documentation][gcloud-datastore-docs]
@@ -359,6 +397,7 @@ Apache 2.0 - See [COPYING](COPYING) for more information.
 [gcloud-homepage]: https://googlecloudplatform.github.io/gcloud-node/
 [gcloud-docs]: https://googlecloudplatform.github.io/gcloud-node/#/docs
 [gcloud-bigquery-docs]: https://googlecloudplatform.github.io/gcloud-node/#/docs/bigquery
+[gcloud-compute-docs]: https://googlecloudplatform.github.io/gcloud-node/#/docs/compute
 [gcloud-datastore-docs]: https://googlecloudplatform.github.io/gcloud-node/#/docs/datastore
 [gcloud-dns-docs]: https://googlecloudplatform.github.io/gcloud-node/#/docs/dns
 [gcloud-pubsub-docs]: https://googlecloudplatform.github.io/gcloud-node/#/docs/pubsub
@@ -378,6 +417,8 @@ Apache 2.0 - See [COPYING](COPYING) for more information.
 
 [cloud-bigquery-docs]: https://cloud.google.com/bigquery/what-is-bigquery
 
+[cloud-compute-docs]: https://cloud.google.com/compute/docs
+
 [cloud-datastore-docs]: https://cloud.google.com/datastore/docs
 [cloud-datastore-activation]: https://cloud.google.com/datastore/docs/activate
 

docs/json/master/compute/.gitkeep

@@ -0,0 +1,7 @@
+<h3>Compute Engine Overview</h3>
+<p>
+  The object returned from <code>gcloud.compute</code> gives you complete control of your Compute Engine instances, snapshots, addresses, firewalls, and more.
+</p>
+<p>
+  To learn more about Compute Engine, see <a href="https://cloud.google.com/compute/docs">What is Google Compute Engine?</a>
+</p>

docs/site/components/docs/compute-overview.html

@@ -39,6 +39,49 @@ angular.module('gcloud.docs')
       ]
     },
 
+    compute: {
+      title: 'Compute',
+      _url: '{baseUrl}/compute',
+      pages: [
+        {
+          title: 'Address',
+          url: '/address'
+        },
+        {
+          title: 'Disk',
+          url: '/disk'
+        },
+        {
+          title: 'Firewall',
+          url: '/firewall'
+        },
+        {
+          title: 'Instance',
+          url: '/instance'
+        },
+        {
+          title: 'Network',
+          url: '/network'
+        },
+        {
+          title: 'Operation',
+          url: '/operation'
+        },
+        {
+          title: 'Region',
+          url: '/region'
+        },
+        {
+          title: 'Snapshot',
+          url: '/snapshot'
+        },
+        {
+          title: 'Zone',
+          url: '/zone'
+        }
+      ]
+    },
+
     datastore: {
       title: 'Datastore',
       _url: '{baseUrl}/datastore',
@@ -180,6 +223,9 @@ angular.module('gcloud.docs')
       '>=0.16.0': ['search'],
 
       // introduce dns api.
-      '>=0.18.0': ['dns']
+      '>=0.18.0': ['dns'],
+
+      // introduce compute api.
+      '>=0.19.0': ['compute']
     }
   });

docs/site/components/docs/docs-values.js

@@ -44,7 +44,7 @@ <h3 class="sub-heading">
     </article>
     <hr>
 
-    <article ng-repeat="service in ['bigquery', 'datastore', 'dns', 'pubsub', 'search', 'storage']"
+    <article ng-repeat="service in ['bigquery', 'compute', 'datastore', 'dns', 'pubsub', 'search', 'storage']"
              ng-if="isActiveDoc(service)"
              ng-include="'site/components/docs/' + service + '-overview.html'">
     </article>

docs/site/components/docs/docs.html

@@ -1,5 +1,5 @@
 /*!
- * Copyright 2014 Google Inc. All Rights Reserved.
+ * Copyright 2015 Google Inc. All Rights Reserved.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -26,70 +26,66 @@
  */
 var util = require('../common/util.js');
 
+/*! Developer Documentation
+ *
+ * @param {module:region} region - Region this address belongs to.
+ * @param {string} name - The name of the address.
+ */
 /**
- * Create an Address object to interact with a Google Compute Engine address.
+ * An Address object allows you to interact with a Google Compute Engine
+ * address.
  *
  * @constructor
- * @alias module:region/address
- *
- * @throws {Error} if an address name or a region are not provided.
- *
- * @param {module:region} region - The Google Compute Engine region this
- *     address belongs to.
- * @param {string} name - The name of the address.
- * @param {object=} metadata - Address metadata.
+ * @alias module:compute/address
  *
  * @example
  * var gcloud = require('gcloud')({
- *   keyFilename: '/path/to/keyfile.json'
+ *   keyFilename: '/path/to/keyfile.json',
+ *   projectId: 'grape-spaceship-123'
  * });
  *
  * var compute = gcloud.compute();
  *
- * var myRegion = compute.region('region-name');
+ * var region = compute.region('region-name');
  *
- * var address = myRegion.address('address1');
+ * var address = region.address('address1');
  */
-function Address(region, name, metadata) {
+function Address(region, name) {
   this.region = region;
   this.name = name;
-  this.metadata = metadata;
-
-  if (!util.is(this.name, 'string')) {
-    throw new Error('A name is needed to use a Compute Engine Address.');
-  }
-  if (!this.region) {
-    throw new Error('A region is needed to use a Compute Engine Address.');
-  }
 }
 
 /**
  * Delete the address.
  *
- * @param {function} callback - The callback function.
+ * @param {function=} callback - The callback function.
  *
  * @example
  * address.delete(function(err, operation) {
- *   // `operation` is an Operation object and can be used to check the status
- *   // of address deletion.
+ *   // `operation` is an Operation object that can be used to check the status
+ *   // of the request.
  * });
  */
 Address.prototype.delete = function(callback) {
   callback = callback || util.noop;
+
   var region = this.region;
-  this.makeReq_('DELETE', '', null, true, function(err, resp) {
+
+  this.makeReq_('DELETE', '', null, null, function(err, resp) {
     if (err) {
-      callback(err);
+      callback(err, null, resp);
       return;
     }
+
     var operation = region.operation(resp.name);
     operation.metadata = resp;
-    callback(null, operation);
+
+    callback(null, operation, resp);
   });
 };
 
 /**
- * Get the address' metadata.
+ * Get the metadata of this address.
  *
  * @param {function=} callback - The callback function.
  *
@@ -98,13 +94,17 @@ Address.prototype.delete = function(callback) {
  */
 Address.prototype.getMetadata = function(callback) {
   callback = callback || util.noop;
+
   var self = this;
+
   this.makeReq_('GET', '', null, null, function(err, resp) {
     if (err) {
-      callback(err);
+      callback(err, null, resp);
       return;
     }
+
     self.metadata = resp;
+
     callback(null, self.metadata, resp);
   });
 };
@@ -126,4 +126,4 @@ Address.prototype.makeReq_ = function(method, path, query, body, callback) {
   this.region.makeReq_(method, path, query, body, callback);
 };
 
-module.exports = Address;
+module.exports = Address;