varnitgoyal95/litmus-be
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #64 from medici/litmus-event-bus-client

Browse Source 
TP-12345|disabling experiment validator
This commit is contained in:
chandresh pancholi
2023-05-03 17:59:30 +05:30
committed by GitHub Enterprise
parent 5287d23951 497c9cc240
commit 3ddd01f89c
1 changed files with 5 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
litmus-core/src/main/java/com/navi/medici/validator/ExperimentValidator.java
View File

@@ -41,10 +41,11 @@ public class ExperimentValidator {
if (experimentQuery.findByName(request.getExperimentName()).isPresent()) {
throw new BadRequestException("Experiment already present with name : " + request.getExperimentName());
}
Optional<MetricEntity> primaryMetric = metricQuery.findByMetricName(request.getPrimaryMetric());
if (primaryMetric.isEmpty()) {
throw new BadRequestException("Primary metric not found");
}
// Optional<MetricEntity> primaryMetric = metricQuery.findByMetricName(request.getPrimaryMetric());
// if (primaryMetric.isEmpty()) {
// throw new BadRequestException("Primary metric not found");
// }
if (Objects.nonNull(request.getVariants()) && !request.getVariants().isEmpty()) {
int sum = 0;
for (VariantDefinition variant : request.getVariants()) {