From 55f21cee67b36dd6352a775a50ef048a0ff35140 Mon Sep 17 00:00:00 2001 From: Eylon Malin Date: Thu, 24 Oct 2019 12:43:12 +0300 Subject: fix ApacheClientMetricInterceptor to return expected ResponseCode Issue-ID: VID-253 Signed-off-by: Eylon Malin Change-Id: I5c90f5bcbf34eee28c192128333ca940558ecb30 --- .../test/java/org/onap/vid/more/LoggerFormatTest.java | 17 ++++++++++------- 1 file changed, 10 insertions(+), 7 deletions(-) (limited to 'vid-automation/src/test') diff --git a/vid-automation/src/test/java/org/onap/vid/more/LoggerFormatTest.java b/vid-automation/src/test/java/org/onap/vid/more/LoggerFormatTest.java index 6a4bf950f..50c3db861 100644 --- a/vid-automation/src/test/java/org/onap/vid/more/LoggerFormatTest.java +++ b/vid-automation/src/test/java/org/onap/vid/more/LoggerFormatTest.java @@ -8,6 +8,7 @@ import static org.hamcrest.MatcherAssert.assertThat; import static org.hamcrest.Matchers.allOf; import static org.hamcrest.Matchers.contains; import static org.hamcrest.Matchers.containsInRelativeOrder; +import static org.hamcrest.Matchers.greaterThan; import static org.hamcrest.Matchers.greaterThanOrEqualTo; import static org.hamcrest.Matchers.hasSize; import static vid.automation.test.services.SimulatorApi.retrieveRecordedRequests; @@ -27,7 +28,6 @@ import org.onap.vid.api.BaseApiTest; import org.springframework.web.client.RestTemplate; import org.testng.annotations.BeforeClass; import org.testng.annotations.Test; -import vid.automation.test.infra.SkipTestUntil; import vid.automation.test.services.SimulatorApi; import vid.automation.test.services.SimulatorApi.RecordedRequests; @@ -50,7 +50,6 @@ public class LoggerFormatTest extends BaseApiTest { SimulatorApi.registerExpectationFromPreset(new PresetAAIGetSubscribersGet(), SimulatorApi.RegistrationStrategy.CLEAR_THEN_SET); } - @SkipTestUntil("2019-09-24") @Test public void validateAuditLogsFormat() { validateLogsFormat(LogName.audit); @@ -66,12 +65,16 @@ public class LoggerFormatTest extends BaseApiTest { validateLogsFormat(LogName.error); } - @SkipTestUntil("2019-09-24") @Test public void validateMetricsLogsFormat() { validateLogsFormat(LogName.metrics, "metric"); } + @Test + public void validateMetrics2019LogsFormat() { + validateLogsFormat(LogName.metrics2019, "metric-ELS-2019.11"); + } + private void validateLogsFormat(LogName logName) { validateLogsFormat(logName, logName.name()); } @@ -86,12 +89,12 @@ public class LoggerFormatTest extends BaseApiTest { logger.info("logLines are: "+logLines); JsonNode response = getCheckerResults(logType, logLines); logger.info("Response is:" + response.toString()); - double fieldscore = response.path("summary").path("score").path("fieldscore").asDouble(); - double overall = response.path("summary").path("score").path("overallscore").asDouble(); - assertThat(fieldscore, is(greaterThanOrEqualTo(score))); - assertThat(overall, is(greaterThanOrEqualTo(score))); + int total_records = response.path("summary").path("total_records").asInt(); + int valid_records = response.path("summary").path("valid_records").asInt(); + assertThat(total_records, greaterThan(30)); //make sure we have at least 30 total records + assertThat((double)valid_records/total_records, is(greaterThanOrEqualTo(score))); } private String getLogLines(LogName logname) { -- cgit 1.2.3-korg