Improve test coverage. (GoogleCloudPlatform#139)

* Improve test coverage. * Experimenting with serial vs parallel tests. * Increase timeout

Author: jmdobry

package.json

@@ -45,9 +45,9 @@
  ]
  },
  "scripts": {
- "ava": "ava --match='!*: dependencies should install*' --timeout=1m --fail-fast --concurrency=5",
+ "ava": "ava --match='!*: dependencies should install*' --timeout=2m --fail-fast --concurrency=5",
  "ava:deps": "npm run deps_appengine && npm run ava",
- "cover": "npm run deps_appengine && nyc ava --match='!*: dependencies should install*'",
+ "cover": "npm run deps_appengine && nyc ava --match='!*: dependencies should install*' --timeout=2m --fail-fast --concurrency=5 --cache",
  "deps_appengine": "ava --match='*: dependencies should install*'",
  "deps_bigquery": "cd bigquery; npm i; cd ../",
  "deps_computeengine": "cd computeengine; npm i; cd ../",

pubsub/iam.js

@@ -185,8 +185,12 @@ function testSubscriptionPermissionsExample (subscriptionName, callback) {
 }
 // [END test_subscription_permissions]
 
+exports.getTopicPolicyExample = getTopicPolicyExample;
+exports.getSubscriptionPolicyExample = getSubscriptionPolicyExample;
 exports.setTopicPolicyExample = setTopicPolicyExample;
 exports.setSubscriptionPolicyExample = setSubscriptionPolicyExample;
+exports.testTopicPermissionsExample = testTopicPermissionsExample;
+exports.testSubscriptionPermissionsExample = testSubscriptionPermissionsExample;
 
 // Run the examples
 exports.main = function (cb) {

pubsub/subscription.js

@@ -290,6 +290,10 @@ exports.createTopicExample = createTopicExample;
 exports.deleteTopicExample = deleteTopicExample;
 exports.subscribeExample = subscribeExample;
 exports.deleteSubscriptionExample = deleteSubscriptionExample;
+exports.publishExample = publishExample;
+exports.getAllTopicsExample = getAllTopicsExample;
+exports.getAllSubscriptionsExample = getAllSubscriptionsExample;
+exports.subscribeExample = subscribeExample;
 exports.pubsub = pubsub;
 exports.main = function (cb) {
  var topicName = 'messageCenter';

test/appengine/all.test.js

@@ -317,7 +317,7 @@ sampleTests.forEach(function (sample) {
  sample.env.SERVER_URL = sample.env.SERVER_URL + port + '/parse';
  }
  port++;
- test.cb(sample.dir + ': dependencies should install', function (t) {
+ test.cb.serial(sample.dir + ': dependencies should install', function (t) {
  testInstallation(sample, t.end);
  });
 
@@ -330,13 +330,13 @@ sampleTests.forEach(function (sample) {
  return;
  }
 
- test.cb(sample.dir + ' should return 200 and Hello World', function (t) {
+ test.cb.serial(sample.dir + ' should return 200 and Hello World', function (t) {
  testLocalApp(sample, t.end);
  });
 });
 
 if (process.env.TRAVIS && process.env.DEPLOY_TESTS) {
- test.cb('should deploy all samples', function (t) {
+ test.cb.serial('should deploy all samples', function (t) {
  // 30 minutes because deployments are slow
  this.timeout(30 * 60 * 1000);
 

test/bigquery/dataset_size.test.js

@@ -16,7 +16,7 @@
 var test = require('ava');
 var datasetSizeExample = require('../../bigquery/dataset_size');
 
-test.cb('should return the size of a dataset', function (t) {
+test.cb.serial('should return the size of a dataset', function (t) {
  datasetSizeExample.main(
  'bigquery-public-data',
  'hacker_news',

test/bigquery/getting_started.test.js

@@ -16,7 +16,7 @@
 var test = require('ava');
 var gettingStartedExample = require('../../bigquery/getting_started');
 
-test.cb('should run a query', function (t) {
+test.cb.serial('should run a query', function (t) {
  gettingStartedExample.main(
  function (err, rows) {
  t.ifError(err);

test/bigquery/load_data_from_csv.test.js

@@ -22,7 +22,7 @@ var datasetId = 'nodejs_docs_samples';
 var tableName = 'test_' + new Date().getTime() + '_' +
  Math.floor(Math.random() * 10000);
 
-test.cb('should load data from a csv file', function (t) {
+test.cb.serial('should load data from a csv file', function (t) {
  async.series([
  function (cb) {
  loadDataFromCsvExample.createTable(datasetId, tableName, cb);

test/bigquery/load_data_from_gcs.test.js

@@ -23,7 +23,7 @@ var datasetId = 'nodejs_docs_samples';
 var tableName = 'test_' + new Date().getTime() + '_' +
  Math.floor(Math.random() * 10000);
 
-test.cb('should load data from a csv file in a GCS bucket', function (t) {
+test.cb.serial('should load data from a csv file in a GCS bucket', function (t) {
  async.series([
  function (cb) {
  loadDataFromCsvExample.createTable(datasetId, tableName, cb);

test/computeengine/mailjet.test.js

@@ -18,7 +18,7 @@ var proxyquire = require('proxyquire').noPreserveCache();
 process.env.MAILJET_API_KEY = 'foo';
 process.env.MAILJET_API_SECRET = 'bar';
 
-test.cb('should send an email', function (t) {
+test.cb.serial('should send an email', function (t) {
  proxyquire('../../computeengine/mailjet.js', {
  nodemailer: {
  createTransport: function (arg) {

test/computeengine/sendgrid.test.js

@@ -17,7 +17,7 @@ var test = require('ava');
 var proxyquire = require('proxyquire').noPreserveCache();
 process.env.SENDGRID_API_KEY = 'foo';
 
-test.cb('should send an email', function (t) {
+test.cb.serial('should send an email', function (t) {
  proxyquire('../../computeengine/sendgrid.js', {
  sendgrid: function (key) {
  t.is(key, 'foo');