v2.0.2

Author: mishanga

.gitignore

@@ -1,4 +1,6 @@
+lib-cov
 node_modules
+test/test-coverage.html
 
 .idea
 *.iml

.jscs.json

@@ -1,6 +1,7 @@
 {
     "excludeFiles": [
-        "node_modules/**"
+        "node_modules/**",
+        "lib-cov/**"
     ],
     "requireCurlyBraces": [
         "else",

CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## 2.0.2 - 2014-01-09
+- Added test coverage (#138)
+- Added test helpers (#147)
+- Fixed config file recursive searching (#151)
+- Fixed `block-indent` detecting (#148)
+- Fixed `quote` value setter (#149)
+- Fixed detection integral tests (#150)
+
 ## 2.0.1 - 2013-12-23
 - Fix for `remove-empty-rulesets` option (#133)
 

README.md

@@ -721,6 +721,14 @@ a
 
 Run `npm test` for tests.
 
+To check test coverage, you need `jscoverage` tool.
+Once it is installed ([see instructions here](https://github.com/visionmedia/node-jscoverage#installation)), run:
+
+```
+npm run-script test-cov
+open ./test/test-coverage.html
+```
+
 ## Contributing
 
 Anyone and everyone is welcome to contribute. Please take a moment to

lib/cli.js

@@ -34,20 +34,26 @@ function getConfigPath(configPath) {
     var HOME = process.env.HOME || process.env.HOMEPATH || process.env.USERPROFILE;
     // Since `process.cwd()` can be absolutely anything, build default path
     // relative to current directory:
-    var defaultConfigPath = __dirname + '/../config/csscomb.json';
+    var defaultConfigPath = path.join(__dirname, '../config/csscomb.json');
 
-    configPath = configPath || process.cwd() + '/.csscomb.json';
+    configPath = configPath || path.join(process.cwd(), '.csscomb.json');
 
     // If we've finally found a config, return its path:
     if (fs.existsSync(configPath)) return configPath;
 
     // If we are in HOME dir already and yet no config file, return a default
-    // one from our package:
-    if (path.dirname(configPath) === HOME) return defaultConfigPath;
+    // one from our package.
+    // If project is located not under HOME, compare to root instead.
+    // Since there appears to be no good way to get root path in
+    // Windows, assume that if current dir has no parent dir, we're in
+    // root.
+    var dirname = path.dirname(configPath);
+    var parentDirname = path.dirname(dirname);
+    if (dirname === HOME || dirname === parentDirname) return defaultConfigPath;
 
     // If there is no config in this directory, go one level up and look for
     // a config there:
-    configPath = path.dirname(path.dirname(configPath)) + '/.csscomb.json';
+    configPath = path.join(parentDirname, '.csscomb.json');
     return getConfigPath(configPath);
 }
 

lib/options/block-indent.js

@@ -83,7 +83,7 @@ module.exports = {
     detect: function(nodeType, node, level) {
         var result = null;
         if (nodeType === 'atrulers' || nodeType === 'block') {
-            if (node[node.length - 1][0] === 's' && level > 0) {
+            if (node.length && node[node.length - 1][0] === 's' && level > 0) {
                 result = node[node.length - 1][1].replace(/\s*\n/g, '');
 
                 if (this._prev !== undefined && this._prev[0] < level) {

lib/options/quotes.js

@@ -7,6 +7,8 @@ module.exports = {
      * @returns {Object|undefined}
      */
     setValue: function(value) {
+        delete this._value;
+
         if (value === 'single' || value === 'double' ) {
             this._value = value;
         }

package.json

@@ -1,7 +1,7 @@
 {
     "name": "csscomb",
     "description": "CSS coding style formatter",
-    "version": "2.0.1",
+    "version": "2.0.2",
     "homepage": "http://csscomb.com/",
     "author": "Mikhail Troshev <mishanga@yandex-team.ru>",
     "repository": "https://github.com/csscomb/csscomb.js",
@@ -45,7 +45,7 @@
     },
     "dependencies": {
         "commander": "2.0.0",
-        "gonzales-pe": "2.0.1",
+        "gonzales-pe": "2.0.x",
         "minimatch": "0.2.12",
         "vow": "0.3.11",
         "vow-fs": "0.2.3"
@@ -61,6 +61,7 @@
         "csscomb": "./bin/csscomb"
     },
     "scripts": {
-        "test": "./node_modules/.bin/jshint-groups && ./node_modules/.bin/jscs . && ./node_modules/.bin/mocha -u bdd -R dot"
+        "test": "./node_modules/.bin/jshint-groups && ./node_modules/.bin/jscs . && node test/mocha",
+        "test-cov": "rm -rf lib-cov && jscoverage lib lib-cov && TEST_COV=true node test/mocha > ./test/test-coverage.html"
     }
 }