If WebJar version is a "-snapshot", overwrite existing files when ext…

…racing

src/main/java/org/webjars/WebJarExtractor.java

@@ -15,6 +15,7 @@
 import java.io.InputStream;
 import java.nio.charset.StandardCharsets;
 import java.nio.file.Files;
+import java.nio.file.StandardCopyOption;
 import java.nio.file.attribute.PosixFileAttributeView;
 import java.nio.file.attribute.PosixFilePermission;
 import java.util.Map;
@@ -138,10 +139,11 @@ private static void extractResource(@Nonnull String webJarName, @Nonnull WebJarI
             final String newPath = resource.getPath().substring(prefix.length());
             final String relativeName = String.format("%s%s%s", webJarId, File.separator, newPath);
             final File newFile = new File(to, relativeName);
-            if (!newFile.exists()) {
+            final boolean allowOverwrite = webJarInfo.getVersion() != null && webJarInfo.getVersion().toLowerCase().endsWith("-snapshot");
+            if (!newFile.exists() || allowOverwrite) {
                 try {
                     newFile.getParentFile().mkdirs();
-                    Files.copy(inputStream, newFile.toPath());
+                    Files.copy(inputStream, newFile.toPath(), StandardCopyOption.REPLACE_EXISTING);
                     inputStream.close();
                     Set<PosixFilePermission> resourcePerms = resource.getPosixFilePermissions();
                     if (resourcePerms != null && Files.getFileStore(newFile.toPath()).supportsFileAttributeView(PosixFileAttributeView.class)) {

src/test/java/org/webjars/WebJarAssetLocatorTest.java

@@ -248,12 +248,13 @@ public void invalid_webjar_path_should_return_null() {
     public void should_get_a_list_of_webjars() {
         Map<String, String> webjars = new WebJarAssetLocator().getWebJars();
 
-        assertThat(webjars.size(), is(38)); // this is the pom.xml ones plus the test resources (spaces, foo, bar-node, multiple)
+        assertThat(webjars.size(), is(39)); // this is the pom.xml ones plus the test resources (spaces, foo, bar-node, multiple)
         assertThat(webjars.get("bootstrap"), is("3.1.1"));
         assertThat(webjars.get("less-node"), is("1.6.0"));
         assertThat(webjars.get("jquery"), is("2.1.0"));
         assertThat(webjars.get("angularjs"), is("1.2.11"));
         assertThat(webjars.get("virtual-keyboard"), is("1.30.1"));
+        assertThat(webjars.get("wip"), is("1.0.0-SNAPSHOT"));
     }
 
     @Test

src/test/java/org/webjars/WebJarExtractorTest.java

@@ -63,6 +63,16 @@ public void extractWebJarShouldNotExtractWhenFileExists() throws Exception {
         assertFileContains(file, "Hello");
     }
 
+    @Test
+    public void extractWebJarShouldExtractWhenFileExistsAndWebJarVersionIsSnapshot() throws Exception {
+        WebJarExtractor extractor = new WebJarExtractor(createClassLoader());
+        File cacheDir = createTmpDir();
+        File file = new File(cacheDir, "wip/caramba.js");
+        createFile(file, "Hello");
+        extractor.extractWebJarTo("wip", cacheDir);
+        assertFileContains(file, "var just = 'do it';");
+    }
+
     @Test
     public void extractAllWebJarsShouldExtractWhenFileDoesntExist() throws Exception {
         WebJarExtractor extractor = new WebJarExtractor(createClassLoader());

src/test/resources/META-INF/resources/webjars/wip/1.0.0-SNAPSHOT/caramba.js

@@ -0,0 +1 @@
+var just = 'do it';