which() does not suppport multivariate anymore.

Author: panhongx

improveai/src/main/java/ai/improve/DecisionContext.java

@@ -186,10 +186,8 @@ public Object which(Object... variants) {
         if(variants.length == 1) {
             if(variants[0] instanceof List) {
                 return chooseFrom((List)variants[0]).get();
-            } else if(variants[0] instanceof Map) {
-                return optimize((Map)variants[0]).get();
             }
-            throw new IllegalArgumentException("If only one argument, it must be a List or Map");
+            throw new IllegalArgumentException("If only one argument, it must be a List.");
         }
 
         return chooseFrom(Arrays.asList(variants)).get();

improveai/src/main/java/ai/improve/DecisionModel.java

@@ -352,13 +352,11 @@ public Decision optimize(Map<String, ?> variants) {
      * @param variants See chooseFrom().
      *                 When the only argument is a List, it's equivalent to calling
      *                 chooseFrom(variants).get();
-     *                 When the only argument is a Map, it's equivalent to calling
-     *                 optimize(variants).get();
      *                 When there are two or more arguments, all the arguments would form a
      *                 list and be passed to chooseFrom();
      * @return Returns the chosen variant
-     * @throws IllegalArgumentException Thrown if variants is null or empty; or if there's only one
-     * variant and it's not a List or Map.
+     * @throws IllegalArgumentException Thrown if variants is null or empty; Thrown if there's only one
+     * variant and it's not a List.
      * */
     public Object which(Object... variants) {
         return given(null).which(variants);
@@ -379,7 +377,7 @@ public <T> Decision<T> chooseFirst(List<T> variants) {
      * This is a short hand of chooseFirst().get().
      * @param variants See chooseFrom(). If only one argument, it must be a non-empty list.
      * @return If multiple arguments is passed to first(), the first argument would be returned;
-     * If only one argument(a non-empty list), then the fist member of it would be returned.
+     * If only one argument(a nonempty list), then the fist member of it would be returned.
      * @throws IllegalArgumentException Thrown if variants is null; Thrown if variants is empty;
      * Thrown if the there's only one argument and it's not a non-empty list.
      */
@@ -398,7 +396,7 @@ public <T> Decision<T> chooseRandom(List<T> variants) {
 
     /**
      * A shorthand of chooseRandom(variants).get()
-     * @param variants See chooseFrom(). If only one argument, it must be a non-empty list. Expect
+     * @param variants See chooseFrom(). If only one argument, it must be a nonempty list. Expect
      *                 at least one argument.
      * @return A random variant.
      * @throws IllegalArgumentException Thrown if variants is empty or null; Thrown if there's only

improveai/src/test/java/ai/improve/DecisionModelTest.java

@@ -751,12 +751,13 @@ public void testWhich_1_argument_map() {
         variants.put("color", Arrays.asList("#000000"));
         variants.put("size", 3);
         DecisionModel decisionModel = new DecisionModel("theme");
-        Object best = decisionModel.which(variants);
-        assertEquals(new HashMap<String, Object>(){{
-            put("font", "Italic");
-            put("color", "#000000");
-            put("size", 3);
-        }}, best);
+        try {
+            decisionModel.which(variants);
+        } catch (IllegalArgumentException t) {
+            t.printStackTrace();
+            return ;
+        }
+        fail(DefaultFailMessage);
     }
 
     @Test