added:

 * include config
 * tests
 * excluding classes and ids

Author: JPeer264

README.md

@@ -47,7 +47,7 @@ What if you are using something such as Modernizr and you do not want to minify
 
 Just create a `.rcsrc` or you can add everything in your `package.json` with the value `rcs`
 
-The `.rcsrc`:
+The `.rcsrc` or the a own config file:
 
 ```json
 {
@@ -81,6 +81,7 @@ The `package.json`:
 - [processCss](#processcss)
 - [process](#process)
 - [generateLibraryFile](#generatelibraryfile)
+- [includeConfig](#includeconfig)
 
 ### processCss
 
@@ -171,3 +172,15 @@ var CSS_NAME_MAPPING_MIN = {
     't': 'another-class'
 };
 ```
+
+**includeConfig([pathLocation])**
+
+> All available configs [here](#rcs-config)
+
+Includes all configs. This function is synchronous.
+
+```js
+const rcs = require('rename-css-selectors')
+
+rcs.includeConfig()
+```

lib/cli.js

@@ -229,6 +229,19 @@ cli.generateLibraryFile = (pathString, options, cb) => {
 /**
  * includes .rcsrc - if not found it will include "rcs" in package.json
  */
-cli.includeConfig = () {
+cli.includeConfig = (pathString) => {
+    let configObject;
 
+    pathString   = pathString || path.join(process.cwd(), '.rcsrc');
+    configObject = rcs.helper.readJsonToObjSync(pathString);
+
+    if (!configObject) {
+        // package.json .rcs if no other config is found
+        configObject = rcs.helper.readJsonToObjSync(path.join(process.cwd(), 'package.json')).rcs;
+    }
+
+    if (configObject.exclude) {
+        rcs.selectorLibrary.setExclude(configObject.exclude);
+    }
+    // rcs.selectorLibrary.setExclude(configObject);
 };

lib/utils/options/selectorLibrary.js

@@ -77,10 +77,6 @@ class SelectorLibrary {
 
         options = _.merge(optionsDefault, options);
 
-        if (this.contains.call(this.excludes, matchedSelector)) {
-            return matchedSelector;
-        }
-
         // change the objects if origValues are set to false - to get information about the compressed selectors
         if (!options.origValues) {
             result = this.compressedSelectors[matchedSelector];
@@ -226,7 +222,11 @@ class SelectorLibrary {
                     continue;
                 }
 
-                selectorLibrarySelector = value[i].slice(1, value[i].length);
+                if (this.contains.call(this.excludes, value[i])) {
+                    continue;
+                }
+
+                selectorLibrarySelector = value[i].replace(/(\.|#)/g, '');
 
                 // save into this.selectors and this.compressedSelectors
                 this.selectors[selectorLibrarySelector] = this.setValue(value[i]);
@@ -241,6 +241,10 @@ class SelectorLibrary {
             return;
         }
 
+        if (this.contains.call(this.excludes, value)) {
+            return;
+        }
+
         selectorLibrarySelector = value.slice(1, value.length);
 
         // save css selector into this.selectors and this.compressedSelectors
@@ -256,14 +260,16 @@ class SelectorLibrary {
      * @param {String | Array} toExclude a string or array with string to exclude specific css selectors
      */
     setExclude(toExclude) {
+        if (!toExclude) return;
+
         if (typeof toExclude === 'string') {
-            this.excludes.append(toExclude);
+            (this.excludes).push(toExclude);
 
             return;
         }
 
         for (let e of toExclude) {
-            this.excludes.append(e);
+            (this.excludes).push(e);
         }
     }
 

package.json

@@ -8,6 +8,12 @@
     "coverage": "istanbul cover _mocha -- --recursive",
     "coveralls": "npm run coverage && coveralls < coverage/lcov.info"
   },
+  "rcs": {
+    "exclude": [
+      "js",
+      "no-js"
+    ]
+  },
   "repository": {
     "url": "https://www.github.com/JPeer264/rename-css-selectors",
     "type": "git"

test/cli.spec.js

@@ -12,7 +12,9 @@ const resultsCwd  = 'test/files/results';
 describe('cli.js', () => {
     beforeEach(() => {
         // reset counter and selectors for tests
-        rcs.selectorLibrary.selectors = {};
+        rcs.selectorLibrary.selectors           = {};
+        rcs.selectorLibrary.compressedSelectors = {};
+        rcs.selectorLibrary.excludes            = [];
         rcs.nameGenerator.resetCountForTests();
     });
 
@@ -243,4 +245,58 @@ describe('cli.js', () => {
             });
         });
     });
+
+    describe('include config', () => {
+        it('should set the config with package.json', done => {
+            // include config
+            cli.includeConfig();
+
+            // include new settings
+            rcs.selectorLibrary.set(['js', 'any-value']);
+
+            expect(rcs.selectorLibrary.get('js')).to.equal('js');
+            expect(rcs.selectorLibrary.get('any-value')).to.equal('a');
+
+            done();
+        });
+
+        it('should set the config with .rcsrc', done => {
+            const file = '.rcsrc';
+
+            fs.writeFileSync(file, `{
+                "exclude": [
+                    "flexbox",
+                    "no-js"
+                ]
+            }`, {
+                encoding: 'utf8'
+            });
+
+            // include config
+            cli.includeConfig();
+
+            // include new settings
+            rcs.selectorLibrary.set(['flexbox', 'any-value']);
+
+            expect(rcs.selectorLibrary.get('flexbox')).to.equal('flexbox');
+            expect(rcs.selectorLibrary.get('any-value')).to.equal('a');
+
+            fs.removeSync(file);
+
+            done();
+        });
+
+        it('should set the config with package.json', done => {
+            // include config
+            cli.includeConfig('test/files/config.json');
+
+            // include new settings
+            rcs.selectorLibrary.set(['own-file', 'any-value']);
+
+            expect(rcs.selectorLibrary.get('own-file')).to.equal('own-file');
+            expect(rcs.selectorLibrary.get('any-value')).to.equal('a');
+
+            done();
+        });
+    });
 });

test/files/.rcsrc renamed to test/files/config.json

@@ -1,6 +1,6 @@
 {
     "exclude": [
-        "js",
+        "own-file",
         "no-js"
     ]
 }

test/files/package.json

@@ -36,16 +36,15 @@ describe('helper.js', () => {
     });
 
     it('should read a json file and turn it to an object', done => {
-        const obj = rcs.helper.readJsonToObjSync('test/files/package.json');
+        const obj = rcs.helper.readJsonToObjSync('test/files/config.json');
 
         expect(obj).to.be.an('object');
-        expect(obj.rcs).to.be.an('object');
 
         done();
     });
 
     it('should fail to read a file', done => {
-        const obj = rcs.helper.readJsonToObjSync('not/existing/path/package.json');
+        const obj = rcs.helper.readJsonToObjSync('not/existing/path/config.json');
 
         expect(obj).to.be.false;
 

test/rcs/helper.spec.js

@@ -10,7 +10,10 @@ const resultsCwd  = 'test/files/results';
 describe('rcs file replace', () => {
     beforeEach(() => {
         // reset counter and selectors for tests
-        rcs.selectorLibrary.selectors = {};
+        rcs.selectorLibrary.selectors           = {};
+        rcs.selectorLibrary.compressedSelectors = {};
+        rcs.selectorLibrary.excludes            = [];
+
         rcs.nameGenerator.resetCountForTests();
     });
 

test/rcs/replace.spec.js

@@ -7,7 +7,10 @@ describe('rcs selector library', () => {
     describe('set new values', () => {
         beforeEach(() => {
             // reset counter and selectors for tests
-            rcs.selectorLibrary.selectors = {};
+            rcs.selectorLibrary.selectors           = {};
+            rcs.selectorLibrary.compressedSelectors = {};
+            rcs.selectorLibrary.excludes            = [];
+
             rcs.nameGenerator.resetCountForTests();
         });