Merge pull request #156 from domaframework/case-sensitive-path-check

SQLファイルの存在チェックでパスの大文字小文字を区別するように修正

Author: nakamura-to

src/main/java/org/seasar/doma/internal/apt/meta/AbstractSqlFileQueryMetaFactory.java

@@ -19,6 +19,7 @@
 
 import java.io.File;
 import java.io.IOException;
+import java.io.UncheckedIOException;
 import java.net.URI;
 import java.util.LinkedHashMap;
 
@@ -91,7 +92,7 @@ protected File getFile(M queryMeta, ExecutableElement method,
         if (!uri.isAbsolute()) {
             uri = new File(".").toURI().resolve(uri);
         }
-        File file = new File(uri);
+        File file = getCanonicalFile(new File(uri));
         if (!file.exists()) {
             throw new AptException(Message.DOMA4019, env, method, new Object[] {
                     filePath, file.getAbsolutePath() });
@@ -100,9 +101,21 @@ protected File getFile(M queryMeta, ExecutableElement method,
             throw new AptException(Message.DOMA4021, env, method, new Object[] {
                     filePath, file.getAbsolutePath() });
         }
+        if (!IOUtil.endsWith(file, filePath)) {
+            throw new AptException(Message.DOMA4309, env, method, new Object[] {
+                    filePath, file.getAbsolutePath() });
+        }
         return file;
     }
 
+    protected File getCanonicalFile(File file) {
+        try {
+            return file.getCanonicalFile();
+        } catch (IOException e) {
+            throw new UncheckedIOException(e);
+        }
+    }
+
     protected File[] getSiblingFiles(M queryMeta, ExecutableElement method,
             File file) {
         File dir = getDir(file);

src/main/java/org/seasar/doma/internal/util/IOUtil.java

@@ -15,7 +15,7 @@
  */
 package org.seasar.doma.internal.util;
 
-import static org.seasar.doma.internal.util.AssertionUtil.*;
+import static org.seasar.doma.internal.util.AssertionUtil.assertNotNull;
 
 import java.io.BufferedReader;
 import java.io.Closeable;
@@ -26,6 +26,8 @@
 import java.io.InputStream;
 import java.io.InputStreamReader;
 import java.nio.CharBuffer;
+import java.nio.file.Path;
+import java.nio.file.Paths;
 
 import org.seasar.doma.internal.Constants;
 import org.seasar.doma.internal.WrapException;
@@ -81,4 +83,20 @@ public static void close(Closeable closeable) {
         }
     }
 
+    public static boolean endsWith(File file, String pathname) {
+        Path path = file.toPath();
+        Path other = Paths.get(pathname);
+        int i = path.getNameCount() - 1;
+        int j = other.getNameCount() - 1;
+        for (; i >= 0 && j >= 0; i--, j--) {
+            // avoid Path#equals to make comparison case sensitive
+            String element = path.getName(i).toString();
+            String otherElement = other.getName(j).toString();
+            if (!element.equals(otherElement)) {
+                return false;
+            }
+        }
+        return true;
+    }
+
 }

src/main/java/org/seasar/doma/message/Message.java

@@ -449,6 +449,7 @@ public enum Message implements MessageResource {
     DOMA4306("@Columnは@Embeddableが注釈された型のプロパティに注釈できません。 at {0}.{1}"),
     DOMA4307("永続プロパティの型[{0}]とコンストラクタパラメータの型[{1}]が[{2}]番目で一致しません。Kotlinでエンベッダブルクラスを定義する場合、永続プロパティとコンストラクタのパラメータの宣言順序は一致しなければいけません。継承している場合、コンストラクタパラメータの宣言順序は親クラスに定義されたプロパティ順でなければいけません。 at {3}"),
     DOMA4308("永続プロパティの型[{0}]とコンストラクタパラメータの型[{1}]が[{2}]番目で一致しません。Kotlinでエンティティクラスを定義する場合、永続プロパティとコンストラクタのパラメータの宣言順序は一致しなければいけません。継承している場合、コンストラクタパラメータの宣言順序は親クラスに定義されたプロパティ順でなければいけません。 at {3}"),
+    DOMA4309("ファイルパス[{0}]の大文字小文字がファイルシステム上のパスと一致しません。ファイルシステム上のパスは\"{1}\"です。"),
 
     // other
     DOMA5001("JDBCドライバがロードされていない可能性があります。まず、JDBCドライバがクラスパスにあることを確認してください。次に、JDBCドライバが自動でロードされない場合は、Class.forNameで明示的にロードしてください。 ex) Class.forName(\"oracle.jdbc.driver.OracleDriver\")"),

src/test/java/org/seasar/doma/internal/util/IOUtilTest.java

@@ -16,6 +16,7 @@
 package org.seasar.doma.internal.util;
 
 import java.io.Closeable;
+import java.io.File;
 import java.io.IOException;
 
 import junit.framework.TestCase;
@@ -35,4 +36,17 @@ public void close() throws IOException {
             }
         });
     }
+
+    public void testEndWith_true() throws Exception {
+        File file = new File("/fuga/META-INF/piyo/HogeDao/selectById.sql");
+        String pathname = "META-INF/piyo/HogeDao/selectById.sql";
+        assertTrue(IOUtil.endsWith(file, pathname));
+    }
+
+    public void testEndWith_false() throws Exception {
+        File file = new File("/fuga/META-INF/piyo/hogeDao/selectById.sql");
+        String pathname = "META-INF/piyo/HogeDao/selectById.sql";
+        assertFalse(IOUtil.endsWith(file, pathname));
+    }
+
 }