Merge pull request #291 from execut/fix-bug-with-minify

Added ability for redefine assets js and css via config

Author: unclead

src/assets/MultipleInputAsset.php

@@ -20,20 +20,20 @@ class MultipleInputAsset extends AssetBundle
         'yii\web\JqueryAsset'
     ];
 
-    public function init()
+    public function __construct($config = [])
     {
-        $this->sourcePath = __DIR__ . '/src/';
-
-        $this->js = [
-            YII_DEBUG ? 'js/jquery.multipleInput.js' : 'js/jquery.multipleInput.min.js'
-        ];
-
-        $this->css = [
-            YII_DEBUG ? 'css/multiple-input.css' : 'css/multiple-input.min.css'
-        ];
-
-        parent::init();
+        $config = array_merge([
+            'sourcePath' => __DIR__ . '/src/',
+            'js' => [
+                YII_DEBUG ? 'js/jquery.multipleInput.js' : 'js/jquery.multipleInput.min.js'
+            ],
+            'css' => [
+                YII_DEBUG ? 'css/multiple-input.css' : 'css/multiple-input.min.css'
+            ],
+        ], $config);
+
+        parent::__construct($config);
     }
 
 
-} 
+} 

tests/unit/MultipleInputAssetTest.php

@@ -0,0 +1,30 @@
+<?php
+
+namespace unclead\multipleinput\tests\unit;
+use unclead\multipleinput\assets\MultipleInputAsset;
+
+/**
+ * Class MultipleInputAssetTest
+ * @package unclead\multipleinput\tests\unit
+ */
+class MultipleInputAssetTest extends TestCase
+{
+    public function testConstuctor() {
+        $asset = new MultipleInputAsset();
+        $this->assertIsString($asset->sourcePath);
+        $this->assertCount(1, $asset->js);
+        $this->assertCount(1, $asset->css);
+    }
+
+    public function testSetEmptyJsViaConfig()
+    {
+        $asset = new MultipleInputAsset([
+            'js' => [],
+            'css' => [],
+            'sourcePath' => 'test',
+        ]);
+        $this->assertEquals('test', $asset->sourcePath);
+        $this->assertCount(0, $asset->js);
+        $this->assertCount(0, $asset->css);
+    }
+}