Upgrade NUglify 1.6.4 and add SupportNewSyntax() test case. (#501)

Co-authored-by: johnny <johnny@wantgoo.com>
madskristensen · Jul 7, 2020 · 8b1de34 · 8b1de34
1 parent d8c2ca6
commit 8b1de34
Show file tree

Hide file tree

Showing 7 changed files with 30 additions and 5 deletions.
diff --git a/src/BundlerMinifier.Core/BundlerMinifier.Core.csproj b/src/BundlerMinifier.Core/BundlerMinifier.Core.csproj
@@ -29,7 +29,7 @@
   </ItemGroup>
 
   <ItemGroup>
-    <PackageReference Include="NUglify" Version="1.5.14" />
+    <PackageReference Include="NUglify" Version="1.6.4" />
     <PackageReference Include="Newtonsoft.Json" Version="9.0.1" />
   </ItemGroup>
 

diff --git a/src/BundlerMinifier/BundlerMinifier.csproj b/src/BundlerMinifier/BundlerMinifier.csproj
@@ -30,7 +30,7 @@
     <PackageReference Include="Microsoft.Build.Framework" Version="15.1.0-*" PrivateAssets="All" />
     <PackageReference Include="Microsoft.Build.Utilities.Core" Version="15.1.0-*" PrivateAssets="All" />
     <PackageReference Include="Newtonsoft.Json" Version="9.0.1" PrivateAssets="All" />
-    <PackageReference Include="NUglify" Version="1.5.14" PrivateAssets="All" />
+    <PackageReference Include="NUglify" Version="1.6.4" PrivateAssets="All" />
   </ItemGroup>
 
   <ItemGroup Condition="'$(TargetFramework)' == 'netstandard1.3'">

diff --git a/src/BundlerMinifierTest/BundlerTest.cs b/src/BundlerMinifierTest/BundlerTest.cs
@@ -40,6 +40,7 @@ public void Cleanup()
             File.Delete("../../../artifacts/file3.min.js");
             File.Delete("../../../artifacts/file4.min.html");
             File.Delete("../../../artifacts/test7.min.js");
+            File.Delete("../../../artifacts/test8.min.js");
         }
 
         [TestMethod]
@@ -200,5 +201,15 @@ public void PreventDoubleProcessing()
             var secondFileTime = File.GetLastWriteTimeUtc(filePath);
             Assert.AreEqual(firstFileTime, secondFileTime);
         }
+
+        [TestMethod]
+        public void SupportNewSyntax()
+        {
+            _processor.Process(TEST_BUNDLE.Replace("test1", "test8"));
+
+            string jsResult = File.ReadAllText("../../../artifacts/test8.min.js");
+
+            Assert.AreEqual("function test(n){for(const t of n)console.log(t)}test([1,2,3,4]);", jsResult);
+        }
     }
 }
diff --git a/src/BundlerMinifierTest/artifacts/file4.js b/src/BundlerMinifierTest/artifacts/file4.js
@@ -0,0 +1,6 @@
+function test(arr) {
+    for (const abc of arr) {
+        console.log(abc);
+    }
+}
+test([1, 2, 3, 4]);
diff --git a/src/BundlerMinifierTest/artifacts/test8.json b/src/BundlerMinifierTest/artifacts/test8.json
@@ -0,0 +1,8 @@
+[
+  {
+    "outputFileName": "test8.min.js",
+    "inputFiles": [
+      "file4.js"
+    ]
+  }
+]
diff --git a/src/BundlerMinifierVsix/BundlerMinifierVsix.csproj b/src/BundlerMinifierVsix/BundlerMinifierVsix.csproj
@@ -258,8 +258,8 @@
       <HintPath>..\..\packages\NuGet.VisualStudio.3.5.0\lib\net45\NuGet.VisualStudio.dll</HintPath>
       <EmbedInteropTypes>True</EmbedInteropTypes>
     </Reference>
-    <Reference Include="NUglify, Version=1.5.14.0, Culture=neutral, PublicKeyToken=15bc7810aec21b5e, processorArchitecture=MSIL">
-      <HintPath>..\..\packages\NUglify.1.5.14\lib\net40\NUglify.dll</HintPath>
+    <Reference Include="NUglify, Version=1.6.4.0, Culture=neutral, PublicKeyToken=15bc7810aec21b5e, processorArchitecture=MSIL">
+      <HintPath>..\..\packages\NUglify.1.6.4\lib\net40\NUglify.dll</HintPath>
     </Reference>
     <Reference Include="PresentationCore" />
     <Reference Include="PresentationFramework" />

diff --git a/src/BundlerMinifierVsix/packages.config b/src/BundlerMinifierVsix/packages.config
@@ -2,5 +2,5 @@
 <packages>
   <package id="Newtonsoft.Json" version="9.0.1" targetFramework="net46" />
   <package id="NuGet.VisualStudio" version="3.5.0" targetFramework="net46" />
-  <package id="NUglify" version="1.5.14" targetFramework="net46" />
+  <package id="NUglify" version="1.6.4" targetFramework="net46" />
 </packages>