This is an automated email from the ASF dual-hosted git repository. ggregory pushed a commit to annotated tag japicmp-base-0.0.1 in repository https://gitbox.apache.org/repos/asf/commons-vfs.git
commit 80d2a73e5f00b22800ccca649e3215365fb92f01 Author: siom79 <martin.m...@googlemail.com> AuthorDate: Mon Sep 30 22:28:27 2013 +0200 further test cases --- .../src/main/java/japicmp/test/Unchanged.java | 8 +++++++ .../src/main/java/japicmp/test/Unchanged.java | 8 +++++++ .../src/test/java/japicmp/test/BasicTest.java | 26 +++++++++++++++++----- 3 files changed, 37 insertions(+), 5 deletions(-) diff --git a/japicmp-testbase/japicmp-test-v1/src/main/java/japicmp/test/Unchanged.java b/japicmp-testbase/japicmp-test-v1/src/main/java/japicmp/test/Unchanged.java new file mode 100644 index 0000000..d499de8 --- /dev/null +++ b/japicmp-testbase/japicmp-test-v1/src/main/java/japicmp/test/Unchanged.java @@ -0,0 +1,8 @@ +package japicmp.test; + +public class Unchanged { + + public void unchangedMethod(String str) { + System.out.println(str); + } +} diff --git a/japicmp-testbase/japicmp-test-v2/src/main/java/japicmp/test/Unchanged.java b/japicmp-testbase/japicmp-test-v2/src/main/java/japicmp/test/Unchanged.java new file mode 100644 index 0000000..d499de8 --- /dev/null +++ b/japicmp-testbase/japicmp-test-v2/src/main/java/japicmp/test/Unchanged.java @@ -0,0 +1,8 @@ +package japicmp.test; + +public class Unchanged { + + public void unchangedMethod(String str) { + System.out.println(str); + } +} diff --git a/japicmp-testbase/japicmp-test/src/test/java/japicmp/test/BasicTest.java b/japicmp-testbase/japicmp-test/src/test/java/japicmp/test/BasicTest.java index 005dbe5..e650051 100644 --- a/japicmp-testbase/japicmp-test/src/test/java/japicmp/test/BasicTest.java +++ b/japicmp-testbase/japicmp-test/src/test/java/japicmp/test/BasicTest.java @@ -3,6 +3,7 @@ package japicmp.test; import japicmp.cmp.JarArchiveComparator; import japicmp.model.JApiChangeStatus; import japicmp.model.JApiClass; +import japicmp.model.JApiMethod; import org.junit.Test; import java.io.File; @@ -18,11 +19,17 @@ public class BasicTest { public void test() { JarArchiveComparator jarArchiveComparator = new JarArchiveComparator(); List<JApiClass> jApiClasses = jarArchiveComparator.compare(getArchive("japicmp-test-v1.jar"), getArchive("japicmp-test-v2.jar")); - assertThat(jApiClasses.size(), is(2)); - assertThat(getJApiClass(jApiClasses, Removed.class.getName()), is(notNullValue())); - assertThat(getJApiClass(jApiClasses, Added.class.getName()), is(notNullValue())); - assertThat(getJApiClass(jApiClasses, Removed.class.getName()).getChangeStatus(), is(JApiChangeStatus.REMOVED)); - assertThat(getJApiClass(jApiClasses, Added.class.getName()).getChangeStatus(), is(JApiChangeStatus.NEW)); + assertThat(jApiClasses.size(), is(3)); + JApiClass jApiClassRemoved = getJApiClass(jApiClasses, Removed.class.getName()); + JApiClass jApiClassAdded = getJApiClass(jApiClasses, Added.class.getName()); + JApiClass jApiClassUnchanged = getJApiClass(jApiClasses, Unchanged.class.getName()); + assertThat(jApiClassRemoved, is(notNullValue())); + assertThat(jApiClassAdded, is(notNullValue())); + assertThat(jApiClassUnchanged, is(notNullValue())); + assertThat(jApiClassRemoved.getChangeStatus(), is(JApiChangeStatus.REMOVED)); + assertThat(jApiClassAdded.getChangeStatus(), is(JApiChangeStatus.NEW)); + assertThat(jApiClassUnchanged.getChangeStatus(), is(JApiChangeStatus.UNCHANGED)); + assertThat(getJApiMethod(jApiClassUnchanged.getMethods(), "unchangedMethod"), is(notNullValue())); } private File getArchive(String filename) { @@ -37,4 +44,13 @@ public class BasicTest { } throw new IllegalArgumentException("No class found with name " + fqn + "."); } + + private JApiMethod getJApiMethod(List<JApiMethod> jApiMethods, String name) { + for(JApiMethod jApiMethod : jApiMethods) { + if(jApiMethod.getName().equals(name)) { + return jApiMethod; + } + } + throw new IllegalArgumentException("No method found with name " + name + "."); + } }