SONAR-13835 api/qualitygates/search is not returning deterministic results when many...

diff --git a/server/sonar-db-dao/src/main/resources/org/sonar/db/qualitygate/ProjectQgateAssociationMapper.xml b/server/sonar-db-dao/src/main/resources/org/sonar/db/qualitygate/ProjectQgateAssociationMapper.xml
index baa08b4dc62f399fe3934ebd125b2825fc64f347..6b31322cb8054fbbfd611e801935f2771417d434 100644 (file)
--- a/server/sonar-db-dao/src/main/resources/org/sonar/db/qualitygate/ProjectQgateAssociationMapper.xml
+++ b/server/sonar-db-dao/src/main/resources/org/sonar/db/qualitygate/ProjectQgateAssociationMapper.xml
@@ -25,7 +25,7 @@
     <if test="query.projectSearch() != null">
       and (proj.kee = #{query.projectSearch} or upper(proj.name) like #{query.projectSearchUpperLikeSql} escape '/')
     </if>
-    order by proj.name
+    order by proj.name, proj.kee
   </select>
 
   <select id="selectQGateUuidByProjectUuid" parameterType="String" resultType="string">

diff --git a/server/sonar-db-dao/src/test/java/org/sonar/db/qualitygate/ProjectQgateAssociationDaoTest.java b/server/sonar-db-dao/src/test/java/org/sonar/db/qualitygate/ProjectQgateAssociationDaoTest.java
index 890b72652ccf3f439d7fbdff31268f5e3d69804f..f2fe1826be1b39a9339f2abd851e66e940cca68c 100644 (file)
--- a/server/sonar-db-dao/src/test/java/org/sonar/db/qualitygate/ProjectQgateAssociationDaoTest.java
+++ b/server/sonar-db-dao/src/test/java/org/sonar/db/qualitygate/ProjectQgateAssociationDaoTest.java
@@ -64,6 +64,26 @@ public class ProjectQgateAssociationDaoTest {
         tuple(project3.uuid(), project3.getKey(), project3.name(), null));
   }
 
+  @Test
+  public void select_all_projects_by_query_should_have_deterministic_order() {
+    OrganizationDto organization = db.organizations().insert();
+    QGateWithOrgDto qualityGate1 = db.qualityGates().insertQualityGate(organization);
+    ComponentDto project1 = db.components().insertPrivateProject(organization, d -> d.setName("p1").setDbKey("key1"));
+    ComponentDto project2 = db.components().insertPrivateProject(organization, d -> d.setName("p1").setDbKey("key2"));
+    ComponentDto project3 = db.components().insertPrivateProject(organization, d -> d.setName("p2").setDbKey("key3"));
+    db.qualityGates().associateProjectToQualityGate(db.components().getProjectDto(project1), qualityGate1);
+    db.qualityGates().associateProjectToQualityGate(db.components().getProjectDto(project2), qualityGate1);
+    db.qualityGates().associateProjectToQualityGate(db.components().getProjectDto(project3), qualityGate1);
+
+    List<ProjectQgateAssociationDto> result = underTest.selectProjects(dbSession, ProjectQgateAssociationQuery.builder()
+      .qualityGate(qualityGate1)
+      .build());
+
+    assertThat(result)
+      .extracting(ProjectQgateAssociationDto::getUuid)
+      .containsExactlyInAnyOrder(project1.uuid(), project2.uuid(), project3.uuid());
+  }
+
   @Test
   public void select_projects_by_query() {
     OrganizationDto organization = db.organizations().insert();