Merge pull request #110 from Financial-Times/about-task

made about.json generation into a task

.travis.yml

@@ -1,6 +1,6 @@
 sudo: false
 language: node_js
-script: make test
+script: make verify test
 node_js:
 - '0.12.2'
 before_deploy:

README.md

@@ -1,12 +1,11 @@
-# next-build-tools [![Build Status](https://travis-ci.org/Financial-Times/next-build-tools.svg?branch=v2.4.2)](https://travis-ci.org/Financial-Times/next-build-tools)
+# next-build-tools
 
-```
   Usage: next-build-tools [options] [command]
 
 
   Commands:
-    deploy [
-    app]                                        runs haikro deployment scripts with sensible defaults for Next projects
+
+    deploy [app]                                        runs haikro deployment scripts with sensible defaults for Next projects
     configure [options] [source] [target]               downloads environment variables from next-config-vars and uploads them to the current app
     scale [source] [target]                             downloads process information from next-service-registry and scales/sizes the application servers
     provision [app]                                     provisions a new instance of an application server
@@ -18,15 +17,14 @@
     purge [options] [url]                               purges the given url from the Fastly cache.  Requires a FASTLY_KEY environment variable set to your fastly api key
     deploy-vcl [options] [folder]                       Deploys VCL in [folder] to the specified fastly service.  Requires FASTLY_KEY env var
     run [options]                                       Runs the local app through the router
+    about                                               Creates an __about.json file for the app
     deploy-static [options] <source> [otherSources...]  Deploys static <source> to [destination] on S3 (where [destination] is a full S3 URL).  Requires AWS_ACCESS and AWS_SECRET env vars
-    *
+    *                                                   
 
   Options:
 
     -h, --help     output usage information
     -V, --version  output the version number
-  
-```
 
 ## Development
 Warning the README.md is automatically generated.  Run `make docs` to update.

bin/next-build-tools.js

@@ -18,6 +18,7 @@ var nightwatch = require('../tasks/nightwatch');
 var deployHashedAssets = require('../tasks/deploy-hashed-assets');
 var deployStatic = require('../tasks/deploy-static');
 var run = require('../tasks/run');
+var about = require('../tasks/about');
 
 function list(val) {
 	return val.split(',');
@@ -152,6 +153,13 @@ program
 		run(opts).catch(exit);
 	});
 
+program
+	.command('about')
+	.description('Creates an __about.json file for the app')
+	.action(function(){
+		about().catch(exit);
+	});
+
 program
 	.command('deploy-static <source> [otherSources...]')
 	.description('Deploys static <source> to [destination] on S3 (where [destination] is a full S3 URL).  Requires AWS_ACCESS and AWS_SECRET env vars')

tasks/about.js

@@ -0,0 +1,10 @@
+'use strict';
+
+var about = require('../lib/about');
+var packageJson = require(process.cwd() + '/package.json');
+var normalizeName = require('../lib/normalize-name');
+module.exports = function() {
+	return about({
+		name: 'ft-next-' + normalizeName(packageJson.name)
+	});
+};

test/deploy-vcl.test.js

@@ -1,3 +1,4 @@
+/* global describe, it, before, after */
 'use strict';
 var sinon = require('sinon');
 var expect = require('chai').expect;
@@ -8,10 +9,6 @@ var fastlyMock = require('./mocks/fastly.mock');
 mockery.registerMock('fastly', fastlyMock);
 var path = require('path');
 
-
-
-
-
 describe.only('Deploy VCL', function(){
 
 	var deployVcl;
@@ -29,7 +26,7 @@ describe.only('Deploy VCL', function(){
 		deployVcl(path.resolve(__dirname, './fixtures/vcl')+'/', {service:fastlyMock.fakeServiceId}).then(function(){
 			sinon.assert.called(fastlyMock().updateVcl);
 			done();
-		})
+		});
 	});
 
 	it('Should replace placeholders with environment vars', function(done){
@@ -44,7 +41,7 @@ describe.only('Deploy VCL', function(){
 			expect(vcl).to.contain(value);
 				expect(vcl).not.to.contain('${AUTH_KEY}');
 			done();
-		})
-	})
+		});
+	});
 
 });

test/purge.test.js

@@ -1,3 +1,4 @@
+/* global describe, it */
 'use strict';
 var expect = require('chai').expect;