Culprits from the headline are listed in alphabetical order so that the list is easier to read.

Author: jan-molak

src/main/java/com/smartcodeltd/jenkinsci/plugins/buildmonitor/viewmodel/BuildView.java

@@ -13,10 +13,7 @@
 import hudson.model.Run;
 import hudson.model.User;
 
-import java.util.Date;
-import java.util.HashSet;
-import java.util.List;
-import java.util.Set;
+import java.util.*;
 
 public class BuildView implements BuildViewModel {
 
@@ -123,7 +120,7 @@ public BuildViewModel previousBuild() {
 
     @Override
     public Set<String> culprits() {
-        Set<String> culprits = new HashSet<String>();
+        Set<String> culprits = new TreeSet<String>();
 
         if (build instanceof AbstractBuild<?, ?>) {
             AbstractBuild<?, ?> jenkinsBuild = (AbstractBuild<?, ?>) build;

src/test/java/com/smartcodeltd/jenkinsci/plugins/buildmonitor/viewmodel/JobViewTest.java

@@ -349,11 +349,11 @@ public void should_tell_who_broke_the_build() throws Exception {
     }
 
     @Test
-    public void should_list_committers_who_broke_the_build() throws Exception {
+    public void should_list_committers_who_broke_the_build_in_alphabetical_order() throws Exception {
         view = a(jobView().of(
                 a(job().whereTheLast(build().wasBrokenBy("Adam", "Ben")))));
 
-        assertThat(view.headline(), is("Failed after Ben and Adam committed their changes"));
+        assertThat(view.headline(), is("Failed after Adam and Ben committed their changes"));
     }
 
     @Test
@@ -380,10 +380,10 @@ public void should_tell_the_number_of_broken_builds_since_the_last_broken_build(
     public void should_tell_the_number_of_broken_builds_since_the_last_build_broken_by_multiple_committers() throws Exception {
         view = a(jobView().of(
                 a(job().whereTheLast(build().wasBrokenBy("Adam")).
-                        andThePrevious(build().wasBrokenBy("Ben", "Connor")).
-                        andThePrevious(build().succeededThanksTo("Daniel")))));
+                        andThePrevious(build().wasBrokenBy("Daniel", "Ben", "Connor")).
+                        andThePrevious(build().succeededThanksTo("Errol")))));
 
-        assertThat(view.headline(), is("1 build has failed since Ben and Connor committed their changes"));
+        assertThat(view.headline(), is("1 build has failed since Ben, Connor and Daniel committed their changes"));
     }
 
     /*