add more test cases

Author: fengmk2

.travis.yml

@@ -1,6 +1,6 @@
 language: node_js
 node_js:
-  - 0.11
-  - 0.10
   - 0.8
-  - 0.6
+  - '0.10'
+  - '0.11'
+script: make test-coveralls

Makefile

@@ -1,7 +1,7 @@
 TESTS = test/*.test.js
 REPORTER = spec
 TIMEOUT = 1000
-JSCOVERAGE = ./node_modules/jscover/bin/jscover
+MOCHA_OPTS =
 
 install:
 	@npm install
@@ -10,14 +10,20 @@ test: install
 	@NODE_ENV=test ./node_modules/mocha/bin/mocha \
 		--reporter $(REPORTER) \
 		--timeout $(TIMEOUT) \
+		$(MOCHA_OPTS) \
 		$(TESTS)
 
-test-cov: install lib-cov
-	@UTILITY_COV=1 $(MAKE) test REPORTER=dot
-	@UTILITY_COV=1 $(MAKE) test REPORTER=html-cov > coverage.html
+test-cov:
+	@rm -f coverage.html
+	@$(MAKE) test MOCHA_OPTS='--require blanket' REPORTER=html-cov > coverage.html
+	@$(MAKE) test MOCHA_OPTS='--require blanket' REPORTER=travis-cov
+	@ls -lh coverage.html
 
-lib-cov:
-	@rm -rf $@
-	@$(JSCOVERAGE) lib $@
+test-coveralls:
+	@$(MAKE) test
+	@echo TRAVIS_JOB_ID $(TRAVIS_JOB_ID)
+	@$(MAKE) test MOCHA_OPTS='--require blanket' REPORTER=mocha-lcov-reporter | ./node_modules/coveralls/bin/coveralls.js
 
-.PHONY: install test test-cov lib-cov
+test-all: test test-cov
+
+.PHONY: test test-cov test-all test-coveralls

README.md

@@ -1,12 +1,10 @@
-utility [![Build Status](https://secure.travis-ci.org/fengmk2/utility.png)](http://travis-ci.org/fengmk2/utility)
+utility [![Build Status](https://secure.travis-ci.org/fengmk2/utility.png)](http://travis-ci.org/fengmk2/utility) [![Coverage Status](https://coveralls.io/repos/fengmk2/utility/badge.png)](https://coveralls.io/r/fengmk2/utility)
 =======
 
 ![logo](https://raw.github.com/fengmk2/utility/master/logo.png)
 
 A collection of useful utilities. 
 
-* jscoverage: [100%](http://fengmk2.github.com/coverage/utility.html)
-
 ## Install
 
 ```bash

package.json

@@ -4,15 +4,20 @@
   "description": "A collection of useful utilities.",
   "main": "index.js",
   "scripts": {
-    "test": "make test"
+    "test": "make test-all",
+    "blanket": { "pattern": "utility/lib" },
+    "travis-cov": { "threshold": 98 }
   },
   "dependencies": {
 
   },
   "devDependencies": {
     "should": "*",
     "moment": "*",
-    "jscover": "*",
+    "blanket": "*",
+    "travis-cov": "*",
+    "coveralls": "*",
+    "mocha-lcov-reporter": "*",
     "benchmark": "*",
     "mocha": "*"
   },

test/utility.test.js

@@ -96,13 +96,14 @@ Encode string s using a URL-safe alphabet, which substitutes - instead of + and
   describe('logDate()', function () {
     it('should return an log format date string', function () {
       utils.logDate().should.match(/^\d{4}\-\d{2}\-\d{2} \d{2}:\d{2}:\d{2}\.\d{3}$/);
+      utils.logDate(new Date(1372062988014)).should.match(/^\d{4}\-\d{2}\-\d{2} \d{2}:\d{2}:\d{2}\.\d{3}$/);
       for (var m = 1; m <= 12; m++) {
         for (var d = 1; d <= 28; d++) {
           for (var h = 0; h < 24; h++) {
             var ss = parseInt(Math.random() * 60, 10);
             var ds = '2013-' + m + '-' + d + ' ' + h + ':' + ss + ':' + ss;
-            var n = new Date();
-            utils.logDate().should.match(/^\d{4}\-\d{2}\-\d{2} \d{2}:\d{2}:\d{2}\.\d{3}$/);
+            var n = new Date(ds);
+            utils.logDate(n).should.match(/^\d{4}\-\d{2}\-\d{2} \d{2}:\d{2}:\d{2}\.\d{3}$/);
           }
         }
       }
@@ -145,6 +146,10 @@ Encode string s using a URL-safe alphabet, which substitutes - instead of + and
         }
       });
     });
+
+    it('should return source string when decode error', function () {
+      utils.decodeURIComponent('%').should.equal('%');
+    });
   });
 
   describe('datestruct()', function () {