From: Evgeny Mandrikov <mandrikov@gmail.com>
Date: Wed, 16 Mar 2011 10:01:32 +0000 (+0300)
Subject: SONAR-2280 ProjectFileSystem.resolvePath should return canonical file
X-Git-Tag: 2.7~36
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=be5def62257b64e1c7ce9d722dfdf73e887a3b3d;p=sonarqube.git

SONAR-2280 ProjectFileSystem.resolvePath should return canonical file
---

diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/resources/DefaultProjectFileSystem.java b/sonar-plugin-api/src/main/java/org/sonar/api/resources/DefaultProjectFileSystem.java
index 0c2520fd226..0387bef2e5a 100644
--- a/sonar-plugin-api/src/main/java/org/sonar/api/resources/DefaultProjectFileSystem.java
+++ b/sonar-plugin-api/src/main/java/org/sonar/api/resources/DefaultProjectFileSystem.java
@@ -143,7 +143,11 @@ public class DefaultProjectFileSystem implements ProjectFileSystem {
   public File resolvePath(String path) {
     File file = new File(path);
     if (!file.isAbsolute()) {
-      file = new File(getBasedir(), path);
+      try {
+        file = new File(getBasedir(), path).getCanonicalFile();
+      } catch (IOException e) {
+        throw new SonarException("Unable to resolve path '" + path + "'", e);
+      }
     }
     return file;
   }
diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/resources/ProjectFileSystem.java b/sonar-plugin-api/src/main/java/org/sonar/api/resources/ProjectFileSystem.java
index de5bd42d4cc..c6840a40c91 100644
--- a/sonar-plugin-api/src/main/java/org/sonar/api/resources/ProjectFileSystem.java
+++ b/sonar-plugin-api/src/main/java/org/sonar/api/resources/ProjectFileSystem.java
@@ -93,8 +93,8 @@ public interface ProjectFileSystem extends BatchComponent {
   File getSonarWorkingDirectory();
 
   /**
-   * Get file from path. It can be absolute or relative to project basedir. For example resolvePath("pom.xml") or
-   * resolvePath("src/main/java")
+   * @return file in canonical form from specified path. Path can be absolute or relative to project basedir.
+   *         For example resolvePath("pom.xml") or resolvePath("src/main/java")
    */
   File resolvePath(String path);
 
diff --git a/sonar-plugin-api/src/test/java/org/sonar/api/resources/DefaultProjectFileSystemTest.java b/sonar-plugin-api/src/test/java/org/sonar/api/resources/DefaultProjectFileSystemTest.java
index a32c84bc3dd..72e88d2979e 100644
--- a/sonar-plugin-api/src/test/java/org/sonar/api/resources/DefaultProjectFileSystemTest.java
+++ b/sonar-plugin-api/src/test/java/org/sonar/api/resources/DefaultProjectFileSystemTest.java
@@ -19,8 +19,17 @@
  */
 package org.sonar.api.resources;
 
+import static org.hamcrest.CoreMatchers.is;
+import static org.hamcrest.Matchers.endsWith;
+import static org.hamcrest.Matchers.hasItem;
+import static org.hamcrest.Matchers.not;
+import static org.junit.Assert.assertThat;
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.when;
+
 import org.apache.commons.lang.StringUtils;
 import org.apache.commons.lang.SystemUtils;
+import org.apache.maven.project.MavenProject;
 import org.hamcrest.Description;
 import org.hamcrest.Matcher;
 import org.hamcrest.TypeSafeMatcher;
@@ -32,11 +41,6 @@ import org.sonar.api.test.MavenTestUtils;
 import java.io.File;
 import java.util.List;
 
-import static org.hamcrest.CoreMatchers.is;
-import static org.hamcrest.Matchers.hasItem;
-import static org.hamcrest.Matchers.not;
-import static org.junit.Assert.assertThat;
-
 public class DefaultProjectFileSystemTest {
 
   private Project project = null;
@@ -143,6 +147,20 @@ public class DefaultProjectFileSystemTest {
     assertThat(files.size(), is(2));
   }
 
+  /**
+   * See http://jira.codehaus.org/browse/SONAR-2280
+   */
+  @Test
+  public void resolvePathShouldReturnCanonicalFile() {
+    MavenProject pom = mock(MavenProject.class);
+    when(pom.getBasedir()).thenReturn(new File("/project"));
+    Project project = new Project("").setPom(pom);
+    DefaultProjectFileSystem fs = new DefaultProjectFileSystem(project, null);
+
+    assertThat(fs.resolvePath(".").getAbsolutePath(), endsWith("project"));
+    assertThat(fs.resolvePath("../project").getAbsolutePath(), endsWith("project"));
+  }
+
   /**
    * Example of hidden files/directories : .DSStore, .svn, .git
    */