Clean up MdcData and ImportMetadata

Removed duplicated class. Added assertions to check the logic.

Change-Id: Ia953ee0c88d71214cf620cdd286f6ecdd3f3d26d
Issue-ID: SDC-2326
Signed-off-by: Tomasz Golabek <tomasz.golabek@nokia.com>

3 files changed, 109 insertions, 156 deletions

diff --git a/catalog-fe/src/test/java/org/openecomp/sdc/fe/impl/ImportMetadataTest.java b/catalog-fe/src/test/java/org/openecomp/sdc/fe/impl/ImportMetadataTest.java
index 54e0392c75..2fd5b56b18 100644
--- a/catalog-fe/src/test/java/org/openecomp/sdc/fe/impl/ImportMetadataTest.java
+++ b/catalog-fe/src/test/java/org/openecomp/sdc/fe/impl/ImportMetadataTest.java
@@ -1,110 +1,53 @@
+/*-
+ * ============LICENSE_START=======================================================
+ * SDC
+ * ================================================================================
+ * Copyright (C) 2017 AT&T Intellectual Property. All rights reserved.
+ * ================================================================================
+ * 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.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ * ============LICENSE_END=========================================================
+ * Modifications copyright (c) 2019 Nokia
+ * ================================================================================
+ */
 package org.openecomp.sdc.fe.impl;
 
+import static com.google.code.beanmatchers.BeanMatchers.hasValidGettersAndSetters;
+import static org.hamcrest.core.IsEqual.equalTo;
+import static org.junit.Assert.assertThat;
+
 import org.junit.Test;
 
 public class ImportMetadataTest {
 
-	private ImportMetadata createTestSubject() {
-		return new ImportMetadata("", 1234567, "", "", "");
-	}
-
-	@Test
-	public void testGetName() throws Exception {
-		ImportMetadata testSubject;
-		String result;
-
-		// default test
-		testSubject = createTestSubject();
-		result = testSubject.getName();
-	}
-
-	@Test
-	public void testSetName() throws Exception {
-		ImportMetadata testSubject;
-		String name = "";
-
-		// default test
-		testSubject = createTestSubject();
-		testSubject.setName(name);
-	}
+	private static final String CHECKSUM = "CHECKSUM";
+	private static final String CREATOR = "CREATOR";
+	private static final String MIME = "MIME";
+	private static final long SIZE = 123L;
+	private static final String NAME = "name";
 
 	@Test
-	public void testGetSize() throws Exception {
-		ImportMetadata testSubject;
-		long result;
-
-		// default test
-		testSubject = createTestSubject();
-		result = testSubject.getSize();
+	public void shouldHaveValidGettersAndSetters() {
+		assertThat(ImportMetadata.class, hasValidGettersAndSetters());
 	}
 
 	@Test
-	public void testSetSize() throws Exception {
-		ImportMetadata testSubject;
-		long size = 1234567;
-
-		// default test
-		testSubject = createTestSubject();
-		testSubject.setSize(size);
+	public void testConstructor() {
+		ImportMetadata importMetadata = new ImportMetadata(NAME, SIZE, MIME, CREATOR, CHECKSUM);
+		assertThat(importMetadata.getCreator(), equalTo(CREATOR));
+		assertThat(importMetadata.getMd5Checksum(), equalTo(CHECKSUM));
+		assertThat(importMetadata.getMime(), equalTo(MIME));
+		assertThat(importMetadata.getName(), equalTo(NAME));
+		assertThat(importMetadata.getSize(), equalTo(SIZE));
 	}
 
-	@Test
-	public void testGetMime() throws Exception {
-		ImportMetadata testSubject;
-		String result;
-
-		// default test
-		testSubject = createTestSubject();
-		result = testSubject.getMime();
-	}
-
-	@Test
-	public void testSetMime() throws Exception {
-		ImportMetadata testSubject;
-		String mime = "";
-
-		// default test
-		testSubject = createTestSubject();
-		testSubject.setMime(mime);
-	}
-
-	@Test
-	public void testGetCreator() throws Exception {
-		ImportMetadata testSubject;
-		String result;
-
-		// default test
-		testSubject = createTestSubject();
-		result = testSubject.getCreator();
-	}
-
-	@Test
-	public void testSetCreator() throws Exception {
-		ImportMetadata testSubject;
-		String creator = "";
-
-		// default test
-		testSubject = createTestSubject();
-		testSubject.setCreator(creator);
-	}
-
-	@Test
-	public void testGetMd5Checksum() throws Exception {
-		ImportMetadata testSubject;
-		String result;
-
-		// default test
-		testSubject = createTestSubject();
-		result = testSubject.getMd5Checksum();
-	}
-
-	@Test
-	public void testSetMd5Checksum() throws Exception {
-		ImportMetadata testSubject;
-		String md5Checksum = "";
-
-		// default test
-		testSubject = createTestSubject();
-		testSubject.setMd5Checksum(md5Checksum);
-	}
 }
\ No newline at end of file
diff --git a/catalog-fe/src/test/java/org/openecomp/sdc/fe/impl/MdcDataTest.java b/catalog-fe/src/test/java/org/openecomp/sdc/fe/impl/MdcDataTest.java
new file mode 100644
index 0000000000..d07470668e
--- /dev/null
+++ b/catalog-fe/src/test/java/org/openecomp/sdc/fe/impl/MdcDataTest.java
@@ -0,0 +1,70 @@
+/*-
+ * ============LICENSE_START=======================================================
+ * SDC
+ * ================================================================================
+ * Copyright (C) 2018 AT&T Intellectual Property. All rights reserved.
+ * ================================================================================
+ * 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.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ * ============LICENSE_END=========================================================
+ * Modifications copyright (c) 2019 Nokia
+ * ================================================================================
+ */
+package org.openecomp.sdc.fe.impl;
+
+import static org.junit.Assert.assertEquals;
+
+import org.junit.Test;
+
+public class MdcDataTest {
+
+	private static final String INSTANCE_ID = "INSTANCE_ID";
+	private static final String USER_ID = "USER_ID";
+	private static final String REMOTE_ADDRESS = "REMOTE_ADDRESS";
+	private static final String LOCAL_ADDRESS = "LOCAL_ADDRESS";
+	private static final long TRANSACTION_START_TIME = 123L;
+
+	@Test
+	public void testGetTransactionStartTimeIsSetByConstructor() {
+		MdcData testSubject = createTestSubject();
+		assertEquals(testSubject.getTransactionStartTime(), Long.valueOf(TRANSACTION_START_TIME));
+	}
+
+	@Test
+	public void testGetUserIdIsSetByConstructor() {
+		MdcData testSubject = createTestSubject();
+		assertEquals(testSubject.getUserId(), USER_ID);
+	}
+
+	@Test
+	public void testGetRemoteAddrIsSetByConstructor() {
+		MdcData testSubject = createTestSubject();
+		assertEquals(testSubject.getRemoteAddr(), REMOTE_ADDRESS);
+	}
+
+	@Test
+	public void testGetLocalAddrIsSetByConstructor() {
+		MdcData testSubject = createTestSubject();
+		assertEquals(testSubject.getLocalAddr(), LOCAL_ADDRESS);
+	}
+
+	@Test
+	public void testGetServiceInstanceIDIsSetByConstructor(){
+		MdcData testSubject = createTestSubject();
+		assertEquals(testSubject.getServiceInstanceID(), INSTANCE_ID);
+	}
+
+	private MdcData createTestSubject() {
+		return new MdcData(INSTANCE_ID, USER_ID, REMOTE_ADDRESS, LOCAL_ADDRESS, TRANSACTION_START_TIME);
+	}
+
+}
\ No newline at end of file
diff --git a/catalog-fe/src/test/java/org/openecomp/sdc/fe/mdc/MdcDataTest.java b/catalog-fe/src/test/java/org/openecomp/sdc/fe/mdc/MdcDataTest.java
deleted file mode 100644
index d9414315e5..0000000000
--- a/catalog-fe/src/test/java/org/openecomp/sdc/fe/mdc/MdcDataTest.java
+++ /dev/null
@@ -1,60 +0,0 @@
-package org.openecomp.sdc.fe.mdc;
-
-import org.junit.Test;
-
-public class MdcDataTest {
-
-	private MdcData createTestSubject() {
-		return new MdcData("", "", "", "", null);
-	}
-
-	@Test
-	public void testGetTransactionStartTime() throws Exception {
-		MdcData testSubject;
-		Long result;
-
-		// default test
-		testSubject = createTestSubject();
-		result = testSubject.getTransactionStartTime();
-	}
-
-	@Test
-	public void testGetUserId() throws Exception {
-		MdcData testSubject;
-		String result;
-
-		// default test
-		testSubject = createTestSubject();
-		result = testSubject.getUserId();
-	}
-
-	@Test
-	public void testGetRemoteAddr() throws Exception {
-		MdcData testSubject;
-		String result;
-
-		// default test
-		testSubject = createTestSubject();
-		result = testSubject.getRemoteAddr();
-	}
-
-	@Test
-	public void testGetLocalAddr() throws Exception {
-		MdcData testSubject;
-		String result;
-
-		// default test
-		testSubject = createTestSubject();
-		result = testSubject.getLocalAddr();
-	}
-
-	@Test
-	public void testGetServiceInstanceID() throws Exception {
-		MdcData testSubject;
-		String result;
-
-		// default test
-		testSubject = createTestSubject();
-		result = testSubject.getServiceInstanceID();
-	}
-}
\ No newline at end of file