Ohne doCheckReferenceJob, dazu braucht es Mockito...

Ich habe noch keine Erfahrung mit Mockito. Deswegen würd ich auch mal das Issue bearbeiten. Assignee +1

In validateHealthReportConstraints() von IssuesRecorder Z.650 steht:

if (healthy >= unHealthy) {
return FormValidation.error(Messages.FieldValidator_Error_ThresholdOrder());
}

Die Methode wird aber sowohl von doCheckHealthy() als auch von doCheckUnHealthy() aufgerufen.

Falls healthy immer kleiner als unhealthy sein muss, würde

healthy = 1, unhealthy = 30 als healthy durchgehen (bei doCheckHealthy()).

Können Sie bitte erklären wofür genau diese Methoden da sind bzw. was genau diese Healthy/Unhealthy Reports darstellen?

Ein Build hat einen Gesundheitsstatus. Dieser ist bei 100%, falls die Anzahl der Warnings <= healthy ist. Er ist bei 0%, falls die Anzahl der Warnings >= unHealthy ist.

Die Validierungsmethode doCheckHealthy ist für das healthy zuständig. Die andere für unHealthy. Das Ergebnis erscheint jeweils unterhalb des Feldes. Neben der Validierung des einzelnen Feldes (d.h. healthy in doCheckHealthy und unhealthy in doCheckUnHealthy) gibt es noch eine Crossvalidation der beiden Felder zusammen. Daher haben beiden Methoden 2 Parameter, da diese für die Chrossvalidation gebraucht werden. Die Crossvalidation macht das von Ihnen angesprochene if:

if (healthy >= unHealthy) {
    return FormValidation.error(Messages.FieldValidator_Error_ThresholdOrder());
}

Diese Validieren schlägt an, wenn healthy nicht kleiner als unhealthy ist. Diese Validierung ist identisch für die beiden Felder und ist daher ausgelagert in die gemeinsame Methode.

Sie können ja mal in der Oberfläche die Daten einfüllen.