fix: fix #2

Author: yibn2008

lib/css-replace.js

@@ -35,11 +35,16 @@ function cssReplace (content, replacer) {
         throw new Error(`parse rule ${statement} failed`)
       }
 
+      // add suffix for @import statement
+      if (!parsed[0].rule.endsWith(';')) {
+        parsed[0].rule += ';'
+      }
+
       /**
        * parsed[0]: {
        *   path: 'foobar.css',
        *   condition: 'print',
-       *   rule: '@import url("foobar.css") print'
+       *   rule: '@import url("foobar.css") print;'
        * }
        */
       return replacer(Object.assign({

test/fixtures/simple/src/page/index.css

@@ -1,3 +1,5 @@
+@import "./refer.css";
+
 body {
   background: url(../img/xxx.png);
 }

test/fixtures/simple/src/page/refer.css

@@ -0,0 +1,3 @@
+body {
+  color: #333;
+}

test/index.test.js

@@ -4,12 +4,13 @@ const fs = require('fs')
 const path = require('path')
 const assert = require('assert')
 const webpack = require('webpack')
+const cssReplace = require('../lib/css-replace')
 
 describe('test plugin', function () {
   this.timeout(60000)
+  const basedir = path.join(__dirname, 'fixtures/simple')
 
   it('should compile with simple config', function (done) {
-    const basedir = path.join(__dirname, 'fixtures/simple')
     const config = require(path.join(basedir, 'webpack.config.js'))
     webpack(config).run((err, stats) => {
       if (err) {
@@ -35,4 +36,13 @@ describe('test plugin', function () {
       }
     })
   })
+
+  it('should css replace @import correctly', function (done) {
+    const original = fs.readFileSync(path.join(basedir, 'src/page/index.css'), 'utf-8')
+    const replaced = cssReplace(original, refer => {
+      return refer.rule;
+    })
+    assert.strictEqual(original, replaced)
+    done()
+  })
 })