when using the argNameBasedConstructorAutoMapping setting, @AutomapConstructor can only be used in one constructor.

Author: sogoagain

src/main/java/org/apache/ibatis/executor/resultset/DefaultResultSetHandler.java

@@ -710,10 +710,12 @@ private Optional<Constructor<?>> findConstructorForAutomapping(final Class<?> re
     if (constructors.length == 1) {
       return Optional.of(constructors[0]);
     }
-    for (final Constructor<?> constructor : constructors) {
-      if (constructor.isAnnotationPresent(AutomapConstructor.class)) {
-        return Optional.of(constructor);
-      }
+    List<Constructor<?>> automapConstructors = Arrays.stream(constructors).filter(x -> x.isAnnotationPresent(AutomapConstructor.class)).toList();
+    if (automapConstructors.size() > 1) {
+      throw new ExecutorException("@AutomapConstructor should be used in only one constructor.");
+    }
+    if (automapConstructors.size() == 1) {
+      return Optional.of(automapConstructors.get(0));
     }
     if (configuration.isArgNameBasedConstructorAutoMapping()) {
       // Finding-best-match type implementation is possible,

src/test/java/org/apache/ibatis/submitted/arg_name_baesd_constructor_automapping/ArgNameBasedConstructorAutoMappingTest.java

@@ -135,4 +135,17 @@ void shouldApplyColumnPrefix() {
       assertEquals(99, task.getAssignee().getTeam());
     }
   }
+
+  @Test
+  void shouldBeOnlyOneAutomapConstructorAnnotation() {
+    // This test requires -parameters compiler option
+    try (SqlSession sqlSession = sqlSessionFactory.openSession()) {
+      Mapper mapper = sqlSession.getMapper(Mapper.class);
+      mapper.selectNameAndIdWithAmbiguousConstructor(1);
+      fail("Exception should be thrown");
+    } catch (PersistenceException e) {
+      ExecutorException ex = (ExecutorException) e.getCause();
+      assertEquals("@AutomapConstructor should be used in only one constructor.", ex.getMessage());
+    }
+  }
 }

src/test/java/org/apache/ibatis/submitted/arg_name_baesd_constructor_automapping/Mapper.java

@@ -34,5 +34,8 @@ public interface Mapper {
   @Select("select name user_name, id user_id from users where id = #{id}")
   User2 selectUserIdAndUserNameUnderscore(Integer id);
 
+  @Select("select name, id from users where id = #{id}")
+  User3 selectNameAndIdWithAmbiguousConstructor(Integer id);
+
   Task selectTask(Integer id);
 }

src/test/java/org/apache/ibatis/submitted/arg_name_baesd_constructor_automapping/User3.java

@@ -0,0 +1,57 @@
+/*
+ *    Copyright 2009-2022 the original author or authors.
+ *
+ *    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.apache.ibatis.submitted.arg_name_baesd_constructor_automapping;
+
+import org.apache.ibatis.annotations.AutomapConstructor;
+
+public class User3 {
+
+  private Integer id;
+  private String name;
+  private Long team;
+
+  @AutomapConstructor
+  public User3(Integer id, String name, Long team) {
+    super();
+    this.id = id;
+    this.name = name;
+    this.team = team;
+  }
+
+  @AutomapConstructor
+  public User3(Integer id, String name) {
+    super();
+    this.id = id;
+    this.name = name;
+    this.team = 100L;
+  }
+
+  public Integer getId() {
+    return id;
+  }
+
+  public String getName() {
+    return name;
+  }
+
+  public void setTeam(Long team) {
+    this.team = team;
+  }
+
+  public Long getTeam() {
+    return team;
+  }
+}