Merge "added test case to TestFileRecorder"

1 files changed, 32 insertions, 18 deletions

diff --git a/sli/recording/src/test/java/org/onap/ccsdk/sli/core/sli/recording/TestFileRecorder.java b/sli/recording/src/test/java/org/onap/ccsdk/sli/core/sli/recording/TestFileRecorder.java
index c879d8f62..d0cc83182 100644
--- a/sli/recording/src/test/java/org/onap/ccsdk/sli/core/sli/recording/TestFileRecorder.java
+++ b/sli/recording/src/test/java/org/onap/ccsdk/sli/core/sli/recording/TestFileRecorder.java
@@ -3,8 +3,11 @@
  */
 package org.onap.ccsdk.sli.core.sli.recording;
 
-import static org.junit.Assert.*;
+import static org.junit.Assert.fail;
+
 import java.util.HashMap;
+
+import org.junit.Before;
 import org.junit.Test;
 import org.onap.ccsdk.sli.core.sli.SvcLogicException;
 
@@ -13,23 +16,34 @@ import org.onap.ccsdk.sli.core.sli.SvcLogicException;
  *
  */
 public class TestFileRecorder {
+	private FileRecorder recorder;
+
+	@Before
+	public void setUp() {
+		recorder = new FileRecorder();
+	}
+
+	/**
+	 * Test method for
+	 * {@link org.onap.ccsdk.sli.core.sli.recording.FileRecorder#record(java.util.Map)}.
+	 */
+	@Test
+	public void testRecord() {
+		HashMap<String, String> parms = new HashMap<>();
+		parms.put("file", "/dev/null");
+		parms.put("field1", "hi");
+		try {
+			recorder.record(parms);
+		} catch (SvcLogicException e) {
+			fail("Caught SvcLogicException : " + e.getMessage());
+		}
+	}
 
-    /**
-     * Test method for {@link org.onap.ccsdk.sli.core.sli.recording.FileRecorder#record(java.util.Map)}.
-     */
-    @Test
-    public void testRecord() {
-
-        FileRecorder recorder = new FileRecorder();
-
-        HashMap<String,String> parms = new HashMap<>();
-        parms.put("file", "/dev/null");
-        parms.put("field1","hi");
-        try {
-            recorder.record(parms);
-        } catch (SvcLogicException e) {
-            fail("Caught SvcLogicException : "+e.getMessage());
-        }
-    }
+	@Test(expected = Exception.class)
+	public void testRecordForEmptyFileName() throws Exception {
+		HashMap<String, String> parms = new HashMap<>();
+		parms.put("field1", "hi");
+		recorder.record(parms);
+	}
 
 }