Merge pull request #2 from jshiell/master

Sort order for static factory methods is now stable via sorting by arity,methodName, so as to avoid indeterminate results when constructor a parameter class with multiple static methods with the same arity

Author: jshiell

src/com/googlecode/yadic/resolvers/StaticMethodResolver.java

@@ -20,10 +20,13 @@
 import static com.googlecode.totallylazy.reflection.Methods.modifier;
 import static com.googlecode.totallylazy.Option.none;
 import static com.googlecode.totallylazy.Option.some;
+import static com.googlecode.totallylazy.comparators.Comparators.ascending;
+import static com.googlecode.totallylazy.comparators.Comparators.comparators;
 import static com.googlecode.totallylazy.predicates.Predicates.not;
 import static com.googlecode.totallylazy.predicates.Predicates.where;
 import static com.googlecode.totallylazy.Sequences.sequence;
 import static com.googlecode.yadic.generics.TypeConverter.convertParametersToInstances;
+import static com.googlecode.totallylazy.reflection.Methods.methodName;
 import static com.googlecode.totallylazy.reflection.Types.classOf;
 import static com.googlecode.totallylazy.reflection.Types.matches;
 import static java.lang.String.format;
@@ -49,7 +52,7 @@ public T resolve(Type type) throws Exception {
                 filter(modifier(PUBLIC).and(modifier(STATIC)).
                         and(where(genericReturnType(), matches(type)).
                                 and(where(genericParameterTypes(), not(exists(matches(type))))))).
-                sortBy(descending(arity()));
+                sortBy(comparators(descending(arity()), ascending(methodName())));
 
         if (methods.isEmpty()) {
             throw new ContainerException(concreteClass.getName() + " does not have any public static methods that return " + type);

test/com/googlecode/yadic/examples/MyStaticMethodClass.java

@@ -11,6 +11,10 @@ public static MyStaticMethodClass myStaticMethodClass1(String parameter) {
         return new MyStaticMethodClass("myStaticMethodClass1");
     }
 
+    public static MyStaticMethodClass myStaticMethodClass2a(String parameter1, Boolean parameter2) {
+        return new MyStaticMethodClass("myStaticMethodClass2a");
+    }
+
     public static MyStaticMethodClass myStaticMethodClass2(String parameter1, Integer parameter2) {
         return new MyStaticMethodClass("myStaticMethodClass2");
     }

test/com/googlecode/yadic/resolvers/StaticMethodResolverTest.java

@@ -44,6 +44,17 @@ public void choosesLargestArity() throws Exception {
         assertThat(staticMethodClass.constructedBy, is("myStaticMethodClass2"));
     }
 
+    @Test
+    public void sortOrderIsStableWhenMultipleMethodsExistWithTheSameArity() throws Exception {
+        StaticMethodResolver<MyStaticMethodClass> resolver = StaticMethodResolver
+            .staticMethodResolver(
+                containerWith("foobar").addInstance(Integer.class, 1).addInstance(Boolean.class, Boolean.TRUE),
+                MyStaticMethodClass.class);
+        MyStaticMethodClass staticMethodClass = resolver.resolve(MyStaticMethodClass.class);
+        assertThat(staticMethodClass, is(notNullValue()));
+        assertThat(staticMethodClass.constructedBy, is("myStaticMethodClass2"));
+    }
+
     @Test(expected = ContainerException.class)
     public void ignoresSelfReferencingStaticMethods() throws Exception {
         Container resolver = new SimpleContainer();