fix: whitelist takes precedence over blacklist

src/utilities-data.js

@@ -49,6 +49,9 @@ export function wildcardToRegExp (input) {
  * @return {RegExp}
  */
 export function convertMatchListToRegExp (list = []) {
+  if (list.length === 0) {
+    return new RegExp('.^');
+  }
   const combined_re = list
     .map((item) => {
       return (typeof item === 'string')

src/utilities-options.js

@@ -6,12 +6,5 @@ import {DEFAULT_OPTIONS} from './constants';
  * @return {css_selector_generator_options}
  */
 export function sanitizeOptions (custom_options = {}) {
-  const options = Object.assign({}, DEFAULT_OPTIONS, custom_options);
-
-  // add never-matching item to blacklist so that it will always fail if empty
-  if (options.blacklist.length === 0) {
-    options.blacklist.push('.^');
-  }
-
-  return options;
+  return Object.assign({}, DEFAULT_OPTIONS, custom_options);
 }

src/utilities-selectors.js

@@ -65,10 +65,14 @@ export function getSelectorsByType (element, selector_type) {
  * Remove blacklisted selectors from list.
  * @param {Array.<string>} list
  * @param {RegExp} blacklist_re
+ * @param {RegExp} whitelist_re
  * @return {Array.<string>}
  */
-export function filterSelectors (list = [], blacklist_re) {
-  return list.filter((item) => !blacklist_re.test(item));
+export function filterSelectors (list = [], blacklist_re, whitelist_re) {
+  return list.filter((item) => (
+    whitelist_re.test(item)
+    || !blacklist_re.test(item)
+  ));
 }
 
 /**
@@ -126,7 +130,8 @@ export function getSelectorsList (element, options) {
 
   const reducer = (data, selector_type) => {
     const selectors_by_type = getSelectorsByType(element, selector_type);
-    const filtered_selectors = filterSelectors(selectors_by_type, blacklist_re);
+    const filtered_selectors =
+      filterSelectors(selectors_by_type, blacklist_re, whitelist_re);
     const found_selectors = orderSelectors(filtered_selectors, whitelist_re);
 
     data[selector_type] = combineWithinSelector

test/options-whitelist.spec.js

@@ -37,6 +37,15 @@ describe('options: whitelist', function () {
     assert.equal(result, '.bbb');
   });
 
+  it('should include whitelisted even if matched by blacklist', function () {
+    root.innerHTML = '<div class="aaa"></div>';
+    const result = getCssSelector(root.firstChild, {
+      whitelist: ['.aaa'],
+      blacklist: ['.aaa']
+    });
+    assert.equal(result, '.aaa');
+  });
+
   // TODO
   it.skip('should prioritise regardless of selector type', function () {
     root.innerHTML = '<p class="aaa"></p>';