diff --git a/impl/src/main/java/org/jboss/forge/roaster/model/impl/FieldImpl.java b/impl/src/main/java/org/jboss/forge/roaster/model/impl/FieldImpl.java index 909f9eb8..912a2dee 100644 --- a/impl/src/main/java/org/jboss/forge/roaster/model/impl/FieldImpl.java +++ b/impl/src/main/java/org/jboss/forge/roaster/model/impl/FieldImpl.java @@ -291,17 +291,7 @@ public FieldSource setVisibility(final Visibility scope) @Override public String getName() { - String result = null; - for (Object f : field.fragments()) - { - if (f instanceof VariableDeclarationFragment) - { - VariableDeclarationFragment frag = (VariableDeclarationFragment) f; - result = frag.getName().getFullyQualifiedName(); - break; - } - } - return result; + return fragment.getName().getFullyQualifiedName(); } @Override @@ -499,4 +489,4 @@ public int getColumnNumber() { return cu.getColumnNumber(getStartPosition()); } -} \ No newline at end of file +} diff --git a/tests/src/test/java/org/jboss/forge/test/roaster/model/FieldMultipleTest.java b/tests/src/test/java/org/jboss/forge/test/roaster/model/FieldMultipleTest.java index f573742b..312c88cb 100644 --- a/tests/src/test/java/org/jboss/forge/test/roaster/model/FieldMultipleTest.java +++ b/tests/src/test/java/org/jboss/forge/test/roaster/model/FieldMultipleTest.java @@ -13,6 +13,7 @@ import org.jboss.forge.roaster.model.source.JavaClassSource; import org.junit.jupiter.api.Test; +import static org.assertj.core.api.Assertions.assertThat; import static org.junit.jupiter.api.Assertions.assertEquals; import static org.junit.jupiter.api.Assertions.assertFalse; import static org.junit.jupiter.api.Assertions.assertTrue; @@ -125,4 +126,10 @@ public void testMultipleFieldDeclarationWithInitializers() assertEquals(1, fields.get(2).getType().getArrayDimensions()); } + @Test + public void testMultipleFieldDeclarationParse() + { + JavaClassSource javaClassSource = Roaster.parse(JavaClassSource.class, "public class Test { private String a,b; }"); + assertThat(javaClassSource.getFields()).hasSize(2).extracting(FieldSource::getName).containsExactly("a", "b"); + } }