add multi-selector rules test cases, fix #5

CHANGELOG.md

@@ -1,8 +1,9 @@
-2.2 2017/2/15
+2.3 2017/2/15
 
 * refactor
 * add npm badge
 * support rules with non-bidi declarations
+* support multiple selectors in a rule
 
 2.1 2017/2/8
 

index.js

@@ -150,9 +150,17 @@ module.exports = postcss.plugin('postcss-bidirection', function (opts) {
                 updateRtlItem(item);
 
                 root.insertAfter(item.rule, item.rtlRule);
+
                 // overwrite rtlRule.raws.before since its been lazy evaluated
                 item.rtlRule.raws.before = '\n\nhtml[dir="rtl"] ';
 
+                // multiple selectors in a rule
+                let selectors = item.rtlRule.selector.split(/\s*[,\n]+\s*/);
+                if (selectors.length) {
+                    item.rtlRule.selector =
+                      selectors.join(',\nhtml[dir="rtl"] ');
+                }
+
                 log('<', item.rule.raws.before,
                     '>', item.rtlRule.raws.before);
             }

package.json

@@ -1,6 +1,6 @@
 {
   "name": "postcss-bidirection",
-  "version": "2.2.0",
+  "version": "2.3.0",
   "description": "PostCSS plugin that polyfill Mozilla's Bi-directional CSS proposal to suppot direction-sensitive rules, a.k.a Left-To-Right (LTR) and Right-To-Left (RTL), in single syntax",
   "keywords": [
     "postcss",

tests/fixtures/multi-selectors-input.css

@@ -0,0 +1,8 @@
+.file svg,
+.domain svg,
+.folder svg,
+.refresh svg,
+.worker svg {
+  top: 7px;
+  margin-inline-end: 5px;
+}

tests/fixtures/multi-selectors-output.css

@@ -0,0 +1,16 @@
+.file svg,
+.domain svg,
+.folder svg,
+.refresh svg,
+.worker svg {
+  top: 7px;
+  margin-right: 5px;
+}
+
+html[dir="rtl"] .file svg,
+html[dir="rtl"] .domain svg,
+html[dir="rtl"] .folder svg,
+html[dir="rtl"] .refresh svg,
+html[dir="rtl"] .worker svg {
+  margin-left: 5px;
+}

tests/test.js

@@ -270,3 +270,8 @@ test('text-align start', t => {
     const { input, output } = read('text-align');
     return run(t, input, output, { });
 });
+
+test('should render rtl before each multi-selector rule', t => {
+    const { input, output } = read('multi-selectors');
+    return run(t, input, output, { });
+});