X-Git-Url: https://gerrit.akraino.org/r/gitweb?a=blobdiff_plain;f=ui%2Fsrc%2Fmain%2Fjava%2Forg%2Fakraino%2Fvalidation%2Fui%2Fservice%2FIntegratedResultService.java;h=5ad22012ee53323ec9e98ba76a16cd0c36c2b936;hb=57c5345c26134129572612dca8382fe493054522;hp=965a78cbbd4f2ee663eaa5d57182e9924898bb66;hpb=5209bb26e2b70f6eefbccf61713dc2633751b86b;p=validation.git diff --git a/ui/src/main/java/org/akraino/validation/ui/service/IntegratedResultService.java b/ui/src/main/java/org/akraino/validation/ui/service/IntegratedResultService.java index 965a78c..5ad2201 100644 --- a/ui/src/main/java/org/akraino/validation/ui/service/IntegratedResultService.java +++ b/ui/src/main/java/org/akraino/validation/ui/service/IntegratedResultService.java @@ -147,11 +147,11 @@ public class IntegratedResultService { throw new IllegalArgumentException("Could not retrieve lab : " + lab.toString()); } ValidationDbTestResult vNexusResult = nexusService.getResult(name, version, labInfo.getSilo(), timestamp); - if (!dbAdapter.checkValidityOfNexusResult(vNexusResult)) { - return null; + if (vNexusResult != null && dbAdapter.checkValidityOfNexusResult(vNexusResult)) { + vNexusResult.setLab(labInfo); + return vNexusResult; } - vNexusResult.setLab(labInfo); - return vNexusResult; + return null; } public ValidationDbTestResult getLastResultBasedOnOutcomeFromNexus(@Nonnull String name, @Nonnull String version, @@ -164,11 +164,11 @@ public class IntegratedResultService { } ValidationDbTestResult vNexusResult = nexusService.getLastResultBasedOnOutcome(name, version, labInfo.getSilo(), allLayers, optional, outcome); - if (!dbAdapter.checkValidityOfNexusResult(vNexusResult)) { - return null; + if (vNexusResult != null && dbAdapter.checkValidityOfNexusResult(vNexusResult)) { + vNexusResult.setLab(labInfo); + return vNexusResult; } - vNexusResult.setLab(labInfo); - return vNexusResult; + return null; } public ValidationDbTestResult getLastResultBasedOnOutcomeFromNexus(@Nonnull String name, @Nonnull String version, @@ -181,11 +181,11 @@ public class IntegratedResultService { } ValidationDbTestResult vNexusResult = nexusService.getLastResultBasedOnOutcome(name, version, labInfo.getSilo(), layers, optional, outcome); - if (!dbAdapter.checkValidityOfNexusResult(vNexusResult)) { - return null; + if (vNexusResult != null && dbAdapter.checkValidityOfNexusResult(vNexusResult)) { + vNexusResult.setLab(labInfo); + return vNexusResult; } - vNexusResult.setLab(labInfo); - return vNexusResult; + return null; } public List getBasedOnDateFromNexus(@Nonnull String name, @Nonnull String version, @@ -198,7 +198,7 @@ public class IntegratedResultService { } List vNexusResults = new ArrayList(); List vResults = nexusService.getResults(name, version, labInfo.getSilo(), date); - if (vResults != null && vResults.size() > 1) { + if (vResults != null && vResults.size() >= 1) { for (ValidationDbTestResult vNexusResult : vResults) { if (dbAdapter.checkValidityOfNexusResult(vNexusResult)) { vNexusResult.setLab(labInfo);