Merge pull request GoogleCloudPlatform#50 from GoogleCloudPlatform/test-deploy

Added deployment tests

Author: jmdobry

.gitignore

@@ -6,4 +6,5 @@ coverage/
 test/encrypted/nodejs-docs-samples.json
 *.iml
 .idea/
-config.json
+config.json
+key.json

.travis.yml

@@ -14,8 +14,7 @@
 sudo: false
 language: node_js
 node_js:
-- "stable"
-- "0.12"
+- "4"
 
 cache:
   directories:
@@ -34,12 +33,6 @@ env:
     - GOOGLE_APPLICATION_CREDENTIALS=$TRAVIS_BUILD_DIR/test/encrypted/nodejs-docs-samples.json
     - CLOUD_BUCKET=nodejs-docs-samples
     - GCLOUD_PROJECT=nodejs-docs-samples
-    - OAUTH_CLIENT_ID=test-id
-    - OAUTH_CLIENT_SECRET=test-secret
-  matrix:
-    - DATA_BACKEND=datastore
-    - DATA_BACKEND=mongodb
-    - DATA_BACKEND=cloudsql
 
 before_install:
   - if [ ! -d $HOME/gcloud/google-cloud-sdk ]; then
@@ -57,4 +50,4 @@ before_install:
     fi
 
 after_success:
-- npm run coveralls
+- npm run codecov

1-hello-world/.gitignore

@@ -0,0 +1,3 @@
+node_modules
+*.log
+config.json

1-hello-world/app.js

@@ -28,10 +28,8 @@ if (module === require.main) {
   // [START server]
   // Start the server
   var server = app.listen(process.env.PORT || 8080, function () {
-    var host = server.address().address;
     var port = server.address().port;
-
-    console.log('App listening at http://%s:%s', host, port);
+    console.log('App listening on port %s', port);
   });
   // [END server]
 }

1-hello-world/package.json

@@ -8,8 +8,8 @@
     "start": "node app.js",
     "monitor": "nodemon app.js",
     "deploy": "gcloud preview app deploy app.yaml",
-    "lint": "jshint --exclude-path=../.gitignore .",
     "mocha": "mocha test/index.js -t 30000",
+    "lint": "semistandard \"**/*.js\"",
     "test": "npm run lint && npm run mocha"
   },
   "author": "Google Inc.",
@@ -28,12 +28,23 @@
     }
   ],
   "license": "Apache Version 2.0",
+  "semistandard": {
+    "globals": [
+      "after",
+      "afterEach",
+      "before",
+      "beforeEach",
+      "describe",
+      "it"
+    ]
+  },
   "dependencies": {
     "express": "^4.13.4"
   },
   "devDependencies": {
-    "jshint": "^2.9.1",
     "mocha": "^2.4.5",
+    "nodejs-repo-tools": "git+https://github.com/GoogleCloudPlatform/nodejs-repo-tools.git#21daa823090c43fb667157c8b5b0c3b7f45a8357",
+    "semistandard": "^7.0.5",
     "supertest": "^1.2.0"
   },
   "engines": {

1-hello-world/test/app.test.js

@@ -15,15 +15,17 @@
 
 var assert = require('assert');
 var config = require('./config');
-var request = require('supertest');
-var utils = require('../../test/utils');
+var utils = require('nodejs-repo-tools');
 
 describe('app.js', function () {
-  it('should run', function (done) {
-    utils.testLocalApp(config, done);
-  });
+  if (!process.env.E2E_TESTS) {
+    it('should run', function (done) {
+      utils.testLocalApp(config, done);
+    });
+  }
+
   it('should create an express app', function (done) {
-    request(require('../app'))
+    utils.getRequest(config)
       .get('/')
       .expect(200)
       .expect(function (response) {

1-hello-world/test/config.js

@@ -17,7 +17,7 @@ var path = require('path');
 
 module.exports = {
   test: '1-hello-world',
-  path: path.resolve(path.join(__dirname, '../')),
+  cwd: path.resolve(path.join(__dirname, '../')),
   cmd: 'node',
   args: ['app.js'],
   msg: 'Hello, world!'

1-hello-world/test/index.js

@@ -14,12 +14,14 @@
 'use strict';
 
 var config = require('./config');
-var utils = require('../../test/utils');
+var utils = require('nodejs-repo-tools');
 
 describe(config.test + '/', function () {
-  it('should install dependencies', function (done) {
-    this.timeout(60 * 1000); // Allow 1 minute to test installation
-    utils.testInstallation(config, done);
-  });
+  if (!process.env.E2E_TESTS) {
+    it('should install dependencies', function (done) {
+      this.timeout(120 * 1000); // Allow 2 minutes to test installation
+      utils.testInstallation(config, done);
+    });
+  }
   require('./app.test');
 });

2-structured-data/.gitignore

@@ -0,0 +1,3 @@
+node_modules
+*.log
+config.json

2-structured-data/app.js

@@ -50,10 +50,8 @@ app.use(function (err, req, res, next) {
 if (module === require.main) {
   // Start the server
   var server = app.listen(config.get('PORT'), function () {
-    var host = server.address().address;
     var port = server.address().port;
-
-    console.log('App listening at http://%s:%s', host, port);
+    console.log('App listening on port %s', port);
   });
 }