Update Module with Tests and 100% Test Coverage

.gitignore

@@ -4,3 +4,4 @@
 nohup.out
 npm-debug.log
 node_modules
+/coverage

.travis.yml

@@ -1,3 +1,3 @@
 language: node_js
 node_js:
-  - 0.6
+  - 0.12

Readme.md

@@ -1,7 +1,12 @@
 Env - In your environment managing your variables.
 ===
 
-[![Build Status](https://secure.travis-ci.org/dshaw/env.png)](http://travis-ci.org/dshaw/env)
+[![Build Status](https://travis-ci.org/dwyl/env.svg)](https://travis-ci.org/dwyl/env)
+[![Test Coverage](https://codeclimate.com/github/dwyl/env/badges/coverage.svg)](https://codeclimate.com/github/dwyl/env/coverage)
+[![Code Climate](https://codeclimate.com/github/dwyl/env/badges/gpa.svg)](https://codeclimate.com/github/dwyl/env)
+[![Dependency Status](https://david-dm.org/dwyl/env.svg)](https://david-dm.org/dwyl/env)
+[![devDependency Status](https://david-dm.org/dwyl/env/dev-status.svg)](https://david-dm.org/dwyl/env#info=devDependencies)
+[![npm](https://img.shields.io/npm/v/env.svg)](https://www.npmjs.com/package/env)
 
 Managing environment variables can be a pain. Env helps make that better.
 

env.json

@@ -0,0 +1 @@
+{}

lib/env.js

@@ -8,7 +8,8 @@
  * Module dependencies
  */
 
-var path = require('path')
+var path = require('path'); // path.existsSync is depracated
+var fs   = require('fs');   // using fs.statSync instead for same check
 
 /**
  * Exports
@@ -21,8 +22,8 @@ exports = module.exports = createEnv;
  */
 
 function Env (envfile) {
-  this.envfile = path.join(process.cwd(), envfile || 'env.json')
-  this.evars = (path.existsSync(this.envfile)) ? require(this.envfile) : {}
+  this.envfile = envfile || path.resolve(process.cwd() + '/env.json');
+  this.evars = (fs.statSync(this.envfile)) && require(this.envfile)
   this.id = process.env['ENV_ID']
 }
 
@@ -102,4 +103,4 @@ Env.prototype.del = function (name) {
 
 function createEnv (envfile) {
   return new Env(envfile)
-}
+}

package.json

@@ -1,6 +1,6 @@
 {
     "name": "env"
-  , "version": "0.0.2"
+  , "version": "1.0.0"
   , "description": "Environment variable manager"
   , "keywords": ["process", "environment", "env"]
   , "author": "Daniel D. Shaw <dshaw@dshaw.com> (http://dshaw.com)"
@@ -17,10 +17,21 @@
       , "test" : "test"
     }
     , "devDependencies": {
-      "tap": "0.0.x"
+        "codeclimate-test-reporter": "0.1.0"
+      , "istanbul": "^0.3.17"
+      , "pre-commit": "1.0.10"
+      , "tape": "^4.0.0"
     }
   , "scripts" : {
-        "test" : "tap ./test/*.js"
+        "test" : "tape ./test/*.js"
+        , "coverage" : "istanbul cover tape ./test/env.test.js"
+        , "check-coverage" : "istanbul cover tape ./test/*.js && istanbul check-coverage --statements 100 --functions 100 --lines 100 --branches 100"
+        , "codeclimate": "CODECLIMATE_REPO_TOKEN=5eff89a1f915b11bf7665cf239a7cb9d972d93e8bd2604f67429ba946b5f06ee ./node_modules/codeclimate-test-reporter/bin/codeclimate.js < ./coverage/lcov.info"
     }
-  , "engines": { "node": ">= 0.5.9" }
+  ,  "pre-commit": [
+      "check-coverage",
+      "codeclimate"
+    ]
+  , "engines": { "node": ">= 0.10.0" }
+  , "keywords": [ "env", "environment", "variables", "config" ]
 }

test/env.test.js

@@ -1,7 +1,8 @@
 var Env = require('..')
-  , env = require('..')('./fixtures/env.json')
-  , tap = require('tap')
-  , test = tap.test
+  , Path = require('path')
+  , env_json = Path.resolve(__dirname + '/fixtures/env.json')
+  , env = require('..')(env_json)
+  , test = require('tape')
 
 test('env core', function (t) {
   t.ok(Env.version)
@@ -10,7 +11,7 @@ test('env core', function (t) {
   t.ok(env.set)
   t.ok(env.del)
 
-  t.equal(env.evars, require('./fixtures/env.json'))
+  t.equal(env.evars, require(env_json))
 
   t.end()
 })
@@ -76,7 +77,7 @@ test('env get all', function (t) {
 
 test('env id', function (t) {
   env.set('ENV_ID', '1')
-  var env2 = require('..')('./fixtures/env.json')
+  var env2 = require('..')(env_json)
   t.equal(env2.id, '1')
 
   t.end()
@@ -85,9 +86,8 @@ test('env id', function (t) {
 test('env with no env.json', function (t) {
   var env3 = require('..')()
   t.ok(env3.evars)
-  t.isa(env3.evars, 'object')
+  t.ok(typeof env3.evars  === 'object', 'env is an object');
   t.deepEqual(env3.evars, {})
   t.deepEqual(env3.get(), {})
-
   t.end()
 })