Revert "Skip non-matching insert-css statements"

ahdinosaur · ahdinosaur · commit 54a4c645a04a · 2016-03-24T19:40:41.000+13:00
reverts commit fbb0963 in favor of #3 (comment).
diff --git a/index.js b/index.js
@@ -57,8 +57,6 @@ function extract (chunk) {
     if (!node.arguments) return
     if (!node.arguments[0]) return
     if (node.arguments[0].value !== 'insert-css') return
-    if (!node.parent.arguments || !node.parent.arguments[0]) return
-
     css.push(node.parent.arguments[0].value)
     node.parent.update('0')
   }
diff --git a/test/index.js b/test/index.js
@@ -29,25 +29,4 @@ test('css-extract', function (t) {
       })
     }
   })
-
-  t.test('should leave non-matching insert-css statements inline', function (t) {
-    t.plan(4)
-
-    browserify(path.join(__dirname, 'manual-insert-source.js'))
-      .plugin(cssExtract, { out: readCss })
-      .bundle(readJs)
-
-    function readCss () {
-      return bl(function (err, data) {
-        t.ifError(err, 'no error')
-        t.equal(String(data), '', 'no css extracted')
-      })
-    }
-
-    function readJs (err, data) {
-      t.ifError(err, 'no error')
-      const source = fs.readFileSync(path.join(__dirname, 'manual-insert-source.js'), 'utf8')
-      t.ok(String(data).indexOf(String(source)) !== -1, 'source is still in built bundle')
-    }
-  })
 })
diff --git a/test/manual-insert-source.js b/test/manual-insert-source.js

Original file line number	Diff line number	Diff line change
`@@ -57,8 +57,6 @@ function extract (chunk) {`
`57`	`57`	`if (!node.arguments) return`
`58`	`58`	`if (!node.arguments[0]) return`
`59`	`59`	`if (node.arguments[0].value !== 'insert-css') return`
`60`		`- if (!node.parent.arguments \|\| !node.parent.arguments[0]) return`
`61`		`-`
`62`	`60`	`css.push(node.parent.arguments[0].value)`
`63`	`61`	`node.parent.update('0')`
`64`	`62`	`}`