Merge pull request #319 from saguijs/clean-only-on-build-action

Run clean only when the action is BUILD 🏗

src/webpack/presets/clean.js

@@ -1,15 +1,16 @@
 import CleanWebpackPlugin from 'clean-webpack-plugin'
+import actions from '../../actions'
 
 export default {
   name: 'clean',
-  configure ({ projectPath }) {
+  configure ({ projectPath, action }) {
     return {
-      plugins: [
+      plugins: action === actions.BUILD ? [
         new CleanWebpackPlugin(['dist'], {
           root: projectPath,
           verbose: false
         })
-      ]
+      ] : []
     }
   }
 }

src/webpack/presets/clean.spec.js

@@ -1,11 +1,12 @@
 import { expect } from 'chai'
 import CleanWebpackPlugin from 'clean-webpack-plugin'
 import preset from './clean'
+import actions from '../../actions'
 
 describe('clean webpack preset', function () {
-  it('should configure CleanWebpackPlugin', function () {
+  it('should configure CleanWebpackPlugin when action is BUILD', function () {
     const projectPath = 'a/demo/path'
-    const config = preset.configure({ projectPath })
+    const config = preset.configure({ projectPath, action: actions.BUILD })
 
     const commons = config.plugins.filter((preset) => preset instanceof CleanWebpackPlugin)
     expect(commons.length).equal(1)
@@ -15,4 +16,12 @@ describe('clean webpack preset', function () {
     expect(cleanWebpackPlugin.options.verbose).to.eql(false)
     expect(cleanWebpackPlugin.options.root).to.eql(projectPath)
   })
+
+  it('should not configure CleanWebpackPlugin by default', function () {
+    const projectPath = 'a/demo/path'
+    const config = preset.configure({ projectPath })
+
+    const commons = config.plugins.filter((preset) => preset instanceof CleanWebpackPlugin)
+    expect(commons.length).equal(0)
+  })
 })