1 files changed, 35 insertions, 75 deletions

diff --git a/src/test/java/org/onap/dcae/vestest/TestVESLogger.java b/src/test/java/org/onap/dcae/vestest/TestVESLogger.java
index 3d41eed6..484f7dc8 100644
--- a/src/test/java/org/onap/dcae/vestest/TestVESLogger.java
+++ b/src/test/java/org/onap/dcae/vestest/TestVESLogger.java
@@ -7,9 +7,9 @@
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
- * 
+ *
  *      http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -19,97 +19,57 @@
  */
 package org.onap.dcae.vestest;
 
-import static org.junit.Assert.*;
-
 import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.assertNotSame;
+import static org.onap.dcae.commonFunction.VESLogger.REQUEST_ID;
 
-import java.io.FileReader;
-import java.io.IOException;
-import java.util.UUID;
-
-import org.json.JSONObject;
-import com.google.gson.JsonParser;
 import com.att.nsa.logging.LoggingContext;
-import com.google.gson.JsonObject;
-
-import org.junit.After;
-import org.junit.Before;
+import com.att.nsa.logging.log4j.EcompFields;
+import java.util.UUID;
 import org.junit.Test;
-import org.onap.dcae.commonFunction.CommonStartup;
-import org.onap.dcae.commonFunction.DmaapPropertyReader;
-import org.onap.dcae.commonFunction.EventProcessor;
-import org.onap.dcae.commonFunction.EventPublisherHash;
 import org.onap.dcae.commonFunction.VESLogger;
-import org.onap.dcae.controller.FetchDynamicConfig;
-
 
 public class TestVESLogger {
 
-	LoggingContext threadLC;
-	
-	@Before
-	public void setUp() throws Exception {
+	@Test
+	public void shouldOnLoggingContextInitializationPutRandomUUIDAsRequestID() {
+		LoggingContext commonLoggingContext = VESLogger.getCommonLoggingContext();
+		String requestId = commonLoggingContext.get(REQUEST_ID, "default");
 
-		 threadLC = null;
-	}
+		assertNotNull(requestId);
+		assertNotSame(requestId, "default");
 
-	@After
-	public void tearDown() throws Exception {
 	}
-	
 
-	
 	@Test
-	public void testgetCommonLoggingContext() {
-		Boolean flag = false;
-		threadLC = VESLogger.getCommonLoggingContext();
-		if(!threadLC.equals(""))
-		{
-			flag = true;
-		}
-		
-		assertEquals(true, flag);
-		
-	}
-	@Test
-	public void testUuidgetLoggingContextForThread() {
-		Boolean flag = false;
+	public void shouldOnLoggingContextInitializationPutGivenUUIDAsRequestIDAndSupplyEndTimestamp() {
 		final UUID uuid = UUID.randomUUID();
-		threadLC = VESLogger.getLoggingContextForThread(uuid);
-		//if(threadLC.get("REQUEST_ID", null) != null)
-		if(!threadLC.equals(""))
-		{
-			flag = true;
-		}
-		
-		assertEquals(true, flag);
-		
-		
+		LoggingContext loggingContextForThread = VESLogger.getLoggingContextForThread(uuid);
+		String requestId = loggingContextForThread.get(REQUEST_ID, "default");
+		String endTimestamp = loggingContextForThread.get(EcompFields.kEndTimestamp, "default");
+
+		assertNotNull(requestId);
+		assertNotNull(endTimestamp);
+		assertNotSame(endTimestamp, "default");
+		assertEquals(requestId, uuid.toString());
 	}
+
 	@Test
-	public void testStringgetLoggingContextForThread() {
-		Boolean flag = false;
+	public void shouldOnLoggingContextInitializationPutGivenUUIDAsRequestIDAndSupplyEndTimestampAndCompleteStatusCode() {
 		final UUID uuid = UUID.randomUUID();
-		threadLC = VESLogger.getLoggingContextForThread(uuid.toString());
-		//if(threadLC.get("REQUEST_ID", null) != null)
-		if(!threadLC.equals(""))
-		{
-			flag = true;
-		}
-		
-		assertEquals(true, flag);
-		
-	}
+		LoggingContext loggingContextForThread = VESLogger.getLoggingContextForThread(uuid.toString());
+		String requestId = loggingContextForThread.get(REQUEST_ID, "default");
+		String statusCode = loggingContextForThread.get("statusCode", "default");
+		String endTimestamp = loggingContextForThread.get(EcompFields.kEndTimestamp, "default");
 
-	@Test
-	public void testsetUpEcompLogging() {
-		Boolean flag = false;
-		VESLogger.setUpEcompLogging();
-		
-		
-		assertEquals(true, true);
-		
+		assertNotNull(requestId);
+		assertNotNull(endTimestamp);
+		assertNotNull(statusCode);
+		assertNotSame(endTimestamp, "default");
+		assertEquals(requestId, uuid.toString());
+		assertEquals(statusCode, "COMPLETE");
 	}
-	
+
 }