feat(unassert-cli): unassert-cli begins

twada · twada · commit 52686f7237a6 · 2016-01-26T23:32:43.000+09:00
diff --git a/.gitignore b/.gitignore
@@ -0,0 +1,3 @@
+node_modules/
+npm-debug.log
+.idea/
diff --git a/bin/cmd.js b/bin/cmd.js
@@ -0,0 +1,40 @@
+#!/usr/bin/env node
+
+/**
+ * unassert-cli
+ *   Command line tool to encourage reliable programming by writing assertions in production code, and compiling them away from release
+ *
+ * https://github.com/twada/unassert-cli
+ *
+ * Copyright (c) 2016 Takuto Wada
+ * Licensed under the MIT license.
+ *   http://twada.mit-license.org/
+ */
+'use strict';
+
+var fs = require('fs');
+var concat = require('concat-stream');
+var esprima = require('esprima');
+var escodegen = require('escodegen');
+var unassert = require('unassert');
+
+function transform (code) {
+    var ast = esprima.parse(code, { sourceType: 'module' });
+    return escodegen.generate(unassert(ast));
+}
+
+var args = process.argv.slice(2);
+var file = args[0];
+var input, filepath;
+
+if (file && file !== '-') {
+    filepath = file;
+    input = fs.createReadStream(file);
+} else {
+    input = process.stdin;
+} 
+
+input.pipe(concat(function(buf) {
+    console.log(transform(buf.toString('utf8')));
+}));
+
diff --git a/package.json b/package.json
@@ -0,0 +1,44 @@
+{
+  "name": "unassert-cli",
+  "description": "Command line tool for unassert",
+  "version": "0.0.1",
+  "author": {
+    "name": "Takuto Wada",
+    "email": "takuto.wada@gmail.com",
+    "url": "https://github.com/twada"
+  },
+  "bin": {
+    "unassert": "./bin/cmd.js"
+  },
+  "bugs": "https://github.com/twada/unassert-cli/issues",
+  "dependencies": {
+    "concat-stream": "^1.5.1",
+    "escodegen": "^1.8.0",
+    "esprima": "^2.7.1",
+    "unassert": "^1.3.1"
+  },
+  "devDependencies": {
+    "mocha": "^2.3.4"
+  },
+  "files": [
+    "CHANGELOG.md",
+    "README.md",
+    "bin",
+    "package.json"
+  ],
+  "homepage": "https://github.com/twada/unassert-cli",
+  "keywords": [
+    "DbC",
+    "assert",
+    "assertion",
+    "unassert"
+  ],
+  "license": "MIT",
+  "repository": {
+    "type": "git",
+    "url": "https://github.com/twada/unassert-cli.git"
+  },
+  "scripts": {
+    "test": "mocha"
+  }
+}
diff --git a/test/fixtures/assignment/expected.js b/test/fixtures/assignment/expected.js
@@ -0,0 +1,4 @@
+var assert;
+function add(a, b) {
+    return a + b;
+}
diff --git a/test/fixtures/assignment/fixture.js b/test/fixtures/assignment/fixture.js
@@ -0,0 +1,9 @@
+var assert;
+assert = require('assert');
+function add (a, b) {
+    console.assert(typeof a === 'number');
+    assert(!isNaN(a));
+    assert.equal(typeof b, 'number');
+    assert.ok(!isNaN(b));
+    return a + b;
+}
diff --git a/test/fixtures/assignment_singlevar/expected.js b/test/fixtures/assignment_singlevar/expected.js
@@ -0,0 +1,4 @@
+var add, assert;
+add = function (a, b) {
+    return a + b;
+};
diff --git a/test/fixtures/assignment_singlevar/fixture.js b/test/fixtures/assignment_singlevar/fixture.js
@@ -0,0 +1,9 @@
+var add, assert;
+assert = require('assert');
+add = function (a, b) {
+    console.assert(typeof a === 'number');
+    assert(!isNaN(a));
+    assert.equal(typeof b, 'number');
+    assert.ok(!isNaN(b));
+    return a + b;
+};
diff --git a/test/fixtures/commonjs/expected.js b/test/fixtures/commonjs/expected.js
@@ -0,0 +1,4 @@
+'use strict';
+function add(a, b) {
+    return a + b;
+}
diff --git a/test/fixtures/commonjs/fixture.js b/test/fixtures/commonjs/fixture.js
@@ -0,0 +1,10 @@
+'use strict';
+
+var assert = require('assert');
+
+function add (a, b) {
+    assert(!isNaN(a));
+    assert.equal(typeof b, 'number');
+    assert.ok(!isNaN(b));
+    return a + b;
+}
diff --git a/test/fixtures/commonjs_powerassert/expected.js b/test/fixtures/commonjs_powerassert/expected.js
@@ -0,0 +1,4 @@
+'use strict';
+function add(a, b) {
+    return a + b;
+}
diff --git a/test/fixtures/commonjs_powerassert/fixture.js b/test/fixtures/commonjs_powerassert/fixture.js
@@ -0,0 +1,10 @@
+'use strict';
+
+var assert = require('power-assert');
+
+function add (a, b) {
+    assert(!isNaN(a));
+    assert.equal(typeof b, 'number');
+    assert.ok(!isNaN(b));
+    return a + b;
+}
diff --git a/test/fixtures/commonjs_singlevar/expected.js b/test/fixtures/commonjs_singlevar/expected.js
@@ -0,0 +1,5 @@
+'use strict';
+var foo = 'FOO', bar = 'BAR';
+function add(a, b) {
+    return a + b;
+}
diff --git a/test/fixtures/commonjs_singlevar/fixture.js b/test/fixtures/commonjs_singlevar/fixture.js
@@ -0,0 +1,12 @@
+'use strict';
+
+var foo = 'FOO',
+    assert = require('assert'),
+    bar = 'BAR';
+
+function add (a, b) {
+    assert(!isNaN(a));
+    assert.equal(typeof b, 'number');
+    assert.ok(!isNaN(b));
+    return a + b;
+}
diff --git a/test/fixtures/es6module/expected.js b/test/fixtures/es6module/expected.js
@@ -0,0 +1,3 @@
+function add(a, b) {
+    return a + b;
+}
diff --git a/test/fixtures/es6module/fixture.js b/test/fixtures/es6module/fixture.js
@@ -0,0 +1,8 @@
+import assert from 'assert';
+
+function add (a, b) {
+    assert(!isNaN(a));
+    assert.equal(typeof b, 'number');
+    assert.ok(!isNaN(b));
+    return a + b;
+}
diff --git a/test/fixtures/es6module_powerassert/expected.js b/test/fixtures/es6module_powerassert/expected.js
@@ -0,0 +1,3 @@
+function add(a, b) {
+    return a + b;
+}
diff --git a/test/fixtures/es6module_powerassert/fixture.js b/test/fixtures/es6module_powerassert/fixture.js
@@ -0,0 +1,8 @@
+import assert from 'power-assert';
+
+function add (a, b) {
+    assert(!isNaN(a));
+    assert.equal(typeof b, 'number');
+    assert.ok(!isNaN(b));
+    return a + b;
+}
diff --git a/test/fixtures/func/expected.js b/test/fixtures/func/expected.js
@@ -0,0 +1,4 @@
+'use strict';
+function add(a, b) {
+    return a + b;
+}
diff --git a/test/fixtures/func/fixture.js b/test/fixtures/func/fixture.js
@@ -0,0 +1,56 @@
+'use strict';
+
+function add (a, b) {
+    console.assert(typeof a === 'number');
+
+    assert(!isNaN(a));
+    assert(!isNaN(a), 'assertion message');
+
+    assert.ok(!isNaN(b));
+    assert.ok(!isNaN(b), 'assertion message');
+
+    assert.equal(typeof b, 'number');
+    assert.equal(typeof b, 'number', 'assertion message');
+
+    assert.strictEqual(typeof b, 'number');
+    assert.strictEqual(typeof b, 'number', 'assertion message');
+
+    assert.deepEqual(typeof b, 'number');
+    assert.deepEqual(typeof b, 'number', 'assertion message');
+
+    assert.deepStrictEqual(typeof b, 'number');
+    assert.deepStrictEqual(typeof b, 'number', 'assertion message');
+
+    assert.notEqual(typeof a, 'object');
+    assert.notEqual(typeof a, 'object', 'assertion message');
+
+    assert.notStrictEqual(typeof a, 'object');
+    assert.notStrictEqual(typeof a, 'object', 'assertion message');
+
+    assert.notDeepEqual(typeof a, 'object');
+    assert.notDeepEqual(typeof a, 'object', 'assertion message');
+
+    assert.notDeepStrictEqual(typeof a, 'object');
+    assert.notDeepStrictEqual(typeof a, 'object', 'assertion message');
+
+    assert.throws(function () {
+        validate(a);
+    });
+    assert.throws(function () {
+        validate(a);
+    }, 'assertion message');
+    assert.throws(function () {
+        validate(a);
+    }, Error, 'assertion message');
+
+    assert.doesNotThrow(function () {
+        validate(b);
+    }, 'assertion message');
+    assert.doesNotThrow(function () {
+        validate(b);
+    });
+
+    assert.ifError(a);
+    assert.fail(a, b, 'assertion message', '==');
+    return a + b;
+}
diff --git a/test/fixtures/not_an_expression_statement/expected.js b/test/fixtures/not_an_expression_statement/expected.js
@@ -0,0 +1,9 @@
+function add(a, b) {
+    if (assert(a)) {
+        return null;
+    }
+    if (!assert(b)) {
+        return null;
+    }
+    return a + b;
+}
diff --git a/test/fixtures/not_an_expression_statement/fixture.js b/test/fixtures/not_an_expression_statement/fixture.js
@@ -0,0 +1,11 @@
+function add(a, b) {
+    assert(!isNaN(a), 'assertion message');
+    if (assert(a)) {
+        return null;
+    }
+    console.assert(a);
+    if (!assert(b)) {
+        return null;
+    }
+    return a + b;
+}
diff --git a/test/test.js b/test/test.js
@@ -0,0 +1,33 @@
+'use strict';
+
+var child = require('child_process');
+var fs = require('fs');
+var path = require('path');
+var concat = require('concat-stream');
+var assert = require('assert');
+var unassertCommand = path.join(__dirname, '..', 'bin', 'cmd.js');
+
+function testUnassertCommand (fixtureName) {
+    it(fixtureName, function (done) {
+        var fixtureFilepath = path.resolve(__dirname, 'fixtures', fixtureName, 'fixture.js');
+        var expectedFilepath = path.resolve(__dirname, 'fixtures', fixtureName, 'expected.js');
+        var expected = fs.readFileSync(expectedFilepath).toString();
+        var proc = child.spawn(unassertCommand, [fixtureFilepath]);
+        proc.stdout.pipe(concat(function (output) {
+            assert.equal(output.toString('utf8'), expected);
+            done();
+        }));
+    });
+}
+
+describe('unassert-cli', function () {
+    testUnassertCommand('func');
+    testUnassertCommand('commonjs');
+    testUnassertCommand('commonjs_singlevar');
+    testUnassertCommand('commonjs_powerassert');
+    testUnassertCommand('assignment');
+    testUnassertCommand('assignment_singlevar');
+    testUnassertCommand('es6module');
+    testUnassertCommand('es6module_powerassert');
+    testUnassertCommand('not_an_expression_statement');
+});

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+node_modules/`
	`2`	`+npm-debug.log`
	`3`	`+.idea/`