OOZIE-2685 fixing tests in TestHdfsOperations

Change-Id: I8eb502980b9e5c5a4e8d80510ffe240c4d5a12e4


Project: http://git-wip-us.apache.org/repos/asf/oozie/repo
Commit: http://git-wip-us.apache.org/repos/asf/oozie/commit/4a9542ca
Tree: http://git-wip-us.apache.org/repos/asf/oozie/tree/4a9542ca
Diff: http://git-wip-us.apache.org/repos/asf/oozie/diff/4a9542ca

Branch: refs/heads/oya
Commit: 4a9542caf1c894714a26a9886a5669b85b94e781
Parents: 050ba57
Author: Peter Bacsko <[email protected]>
Authored: Tue Oct 18 00:19:40 2016 +0200
Committer: Peter Bacsko <[email protected]>
Committed: Tue Oct 18 00:19:40 2016 +0200

----------------------------------------------------------------------
 .../org/apache/oozie/action/hadoop/TestHdfsOperations.java    | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/oozie/blob/4a9542ca/sharelib/oozie/src/test/java/org/apache/oozie/action/hadoop/TestHdfsOperations.java
----------------------------------------------------------------------
diff --git 
a/sharelib/oozie/src/test/java/org/apache/oozie/action/hadoop/TestHdfsOperations.java
 
b/sharelib/oozie/src/test/java/org/apache/oozie/action/hadoop/TestHdfsOperations.java
index 3a60f63..058babc 100644
--- 
a/sharelib/oozie/src/test/java/org/apache/oozie/action/hadoop/TestHdfsOperations.java
+++ 
b/sharelib/oozie/src/test/java/org/apache/oozie/action/hadoop/TestHdfsOperations.java
@@ -69,13 +69,11 @@ public class TestHdfsOperations {
     @Before
     public void setup() throws IOException {
         configureMocksForHappyPath();
+        actionData.put("testKey", "testValue");
     }
 
     @Test
     public void testActionDataUploadToHdfsSucceeds() throws IOException {
-        configureMocksForHappyPath();
-        actionData.put("testKey", "testValue");
-
         hdfsOperations.uploadActionDataToHDFS(configurationMock, path, 
actionData);
 
         
verify(seqFileWriterFactoryMock).createSequenceFileWriter(eq(configurationMock),
@@ -89,7 +87,6 @@ public class TestHdfsOperations {
 
     @Test(expected = IOException.class)
     public void testActionDataUploadToHdfsFailsWhenAppendingToWriter() throws 
IOException {
-        configureMocksForHappyPath();
         willThrow(new IOException()).given(writerMock).append(any(Text.class), 
any(Text.class));
 
         hdfsOperations.uploadActionDataToHDFS(configurationMock, path, 
actionData);
@@ -97,8 +94,6 @@ public class TestHdfsOperations {
 
     @Test(expected = IOException.class)
     public void testActionDataUploadToHdfsFailsWhenWriterIsNull() throws 
IOException {
-        configureMocksForHappyPath();
-        actionData.put("testKey", "testValue");
         
given(seqFileWriterFactoryMock.createSequenceFileWriter(eq(configurationMock),
                 any(Path.class), eq(Text.class), 
eq(Text.class))).willReturn(null);
 

Reply via email to