エンティティクラスの継承構造が2段階の場合に無限ループが発生する不具合を修正

2段階とは、親、子、孫のように3つのクラスからなる継承構造を意味します。
close #163

Author: nakamura-to

src/main/java/org/seasar/doma/internal/jdbc/entity/PropertyField.java

@@ -67,8 +67,7 @@ private Field getField(Class<?> clazz, String name) {
     }
 
     private Field findField(Class<?> clazz, String name) {
-        for (Class<?> cl = clazz; cl != Object.class; cl = clazz
-                .getSuperclass()) {
+        for (Class<?> cl = clazz; cl != Object.class; cl = cl.getSuperclass()) {
             try {
                 return cl.getDeclaredField(name);
             } catch (NoSuchFieldException ignored) {

src/test/java/org/seasar/doma/internal/jdbc/entity/Animal.java

@@ -19,7 +19,7 @@
  * @author nakamura-to
  *
  */
-public class Animal {
+public class Animal extends Life {
 
     public String kind;
 }

src/test/java/org/seasar/doma/internal/jdbc/entity/Life.java

@@ -0,0 +1,25 @@
+/*
+ * Copyright 2004-2010 the Seasar Foundation and the Others.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND,
+ * either express or implied. See the License for the specific language
+ * governing permissions and limitations under the License.
+ */
+package org.seasar.doma.internal.jdbc.entity;
+
+/**
+ * @author nakamura-to
+ *
+ */
+public class Life {
+
+    int weight;
+}

src/test/java/org/seasar/doma/internal/jdbc/entity/PropertyFieldTest.java

@@ -32,6 +32,11 @@ public void testConstructor_parentPath() throws Exception {
         assertEquals(1, path.fields.size());
     }
 
+    public void testConstructor_grandParentPath() throws Exception {
+        PropertyField<Person> path = new PropertyField<>("weight", Person.class);
+        assertEquals(1, path.fields.size());
+    }
+
     public void testConstructor_nestedPath() throws Exception {
         PropertyField<Person> path = new PropertyField<>("address.city",
                 Person.class);