Add Istanbul for coverage, achieve 100% coverage. Add travis, add travis

badge; add coveralls, add coveralls badge

Author: pl12133

.gitignore

@@ -1 +1,2 @@
 node_modules
+coverage

.travis.yml

@@ -0,0 +1,8 @@
+sudo: false
+language: node_js
+node_js:
+  - stable
+script:
+  - npm run lint
+  - npm run test:cov
+  - npm run test:coveralls

README.md

@@ -1,3 +1,5 @@
+[![Build Status](https://travis-ci.org/pl12133/css-object-loader.svg?branch=master)](https://travis-ci.org/pl12133/css-object-loader)[![Coverage Status](https://coveralls.io/repos/github/pl12133/css-object-loader/badge.svg?branch=master)](https://coveralls.io/github/pl12133/css-object-loader?branch=master)
+
 <p align="center">
   <img src='http://i.imgur.com/PXYAzQE.png' title='CSS Object Loader' alt='CSS Object Loader'></img>
 </p>

package.json

@@ -8,7 +8,9 @@
     "lint": "./node_modules/.bin/eslint src test",
     "lint:fix": "./node_modules/.bin/eslint src test --fix",
     "test": "./node_modules/.bin/mocha",
-    "prepublish": "npm run lint && npm run test"
+    "test:cov": "node ./node_modules/.bin/istanbul cover --root src/ ./node_modules/.bin/_mocha",
+    "test:coveralls": "cat ./coverage/lcov.info | ./node_modules/coveralls/bin/coveralls.js",
+    "prepublish": "npm run lint && npm run test:cov"
   },
   "repository": {
     "type": "git",
@@ -21,11 +23,13 @@
   },
   "devDependencies": {
     "chai": "^3.5.0",
+    "coveralls": "^2.11.9",
     "eslint": "^2.9.0",
     "eslint-config-semistandard": "^6.0.1",
     "eslint-config-standard": "^5.2.0",
     "eslint-plugin-promise": "^1.1.0",
     "eslint-plugin-standard": "^1.3.2",
+    "istanbul": "^0.4.3",
     "mocha": "^2.4.5"
   }
 }

src/index.js

@@ -8,6 +8,7 @@ var camelCase = require('camelcase');
 //   1. Parse CSS stylesheet to AST
 //   2. Flatten AST to Object of shape { [rule.selector]: rule.declarations }
 module.exports = function cssObjectLoader (source) {
+  /* istanbul ignore next */
   this.cacheable && this.cacheable();
   // Step 1.
   var parsedStylesheet = getParsedStylesheet(source);

test/index.spec.js

@@ -23,10 +23,13 @@ describe('css-object-loader', () => {
 
   it('should be able to transform css', () => {
     expect({
-      p: {
-        fontSize: '14px',
-        fontWeight: 'bold'
-      }
-    }).to.deep.equal(actual);
+      fontSize: '14px',
+      fontWeight: 'bold'
+    }).to.deep.equal(actual['p']);
+
+    expect({
+      background: 'green',
+      textAlign: 'center'
+    }).to.deep.equal(actual['h1']);
   });
 });

test/style-spec.css

@@ -1,4 +1,16 @@
+/* An animation key to cover the `!isValidRule(...)` branch */
+@keyframes someanimation {
+  0%, 100% { left: 0; }
+  50% { left:100%; }
+}
+
 p {
+  /* the loader should be able to handle comments */
   font-size: 14px;
   font-weight: bold;
 }
+
+h1 {
+  background: green;
+  text-align: center;
+}