Merge "Show history of test results by date"
diff --git a/src/main/java/com/googlesource/gerrit/plugins/verifystatus/client/JobsScreen.java b/src/main/java/com/googlesource/gerrit/plugins/verifystatus/client/JobsScreen.java
index 810085f..508ed1a 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/verifystatus/client/JobsScreen.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/verifystatus/client/JobsScreen.java
@@ -44,7 +44,7 @@
JobsScreen(String changeNumber, String revisionNumber) {
new RestApi("changes").id(changeNumber).view("revisions").id(revisionNumber)
.view(Plugin.get().getPluginName(), "verifications")
- .addParameter("sort", "REPORTER")
+ .addParameter("sort", "DATE")
.get(new AsyncCallback<NativeMap<VerificationInfo>>() {
@Override
public void onSuccess(NativeMap<VerificationInfo> result) {
diff --git a/src/main/java/com/googlesource/gerrit/plugins/verifystatus/server/GetVerifications.java b/src/main/java/com/googlesource/gerrit/plugins/verifystatus/server/GetVerifications.java
index 275e8ca..42be9b2 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/verifystatus/server/GetVerifications.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/verifystatus/server/GetVerifications.java
@@ -95,6 +95,16 @@
.toComparison();
}
});
+ } else if (order.equals("DATE")) {
+ // sort the jobs list by Granted(Z-A)
+ Collections.sort(jobs, new Comparator<PatchSetVerification>() {
+ @Override
+ public int compare(PatchSetVerification a, PatchSetVerification b) {
+ return new CompareToBuilder()
+ .append(b.getGranted(),a.getGranted())
+ .toComparison();
+ }
+ });
}
}
@@ -148,6 +158,8 @@
sortJobs(jobs, "REPORTER");
} else if (sort.equals("NAME")) {
sortJobs(jobs, "NAME");
+ } else if (sort.equals("DATE")) {
+ sortJobs(jobs, "DATE");
}
}
