You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@flink.apache.org by tr...@apache.org on 2016/10/04 12:28:06 UTC

flink git commit: [hotfix] Fix failing JobManagerRunnerMockTest

Repository: flink
Updated Branches:
  refs/heads/flip-6 d5c0092f3 -> 8f576209b


[hotfix] Fix failing JobManagerRunnerMockTest


Project: http://git-wip-us.apache.org/repos/asf/flink/repo
Commit: http://git-wip-us.apache.org/repos/asf/flink/commit/8f576209
Tree: http://git-wip-us.apache.org/repos/asf/flink/tree/8f576209
Diff: http://git-wip-us.apache.org/repos/asf/flink/diff/8f576209

Branch: refs/heads/flip-6
Commit: 8f576209bde06995d22c4bc8048f40daaa8ed65e
Parents: d5c0092
Author: Till Rohrmann <tr...@apache.org>
Authored: Tue Oct 4 14:27:37 2016 +0200
Committer: Till Rohrmann <tr...@apache.org>
Committed: Tue Oct 4 14:27:37 2016 +0200

----------------------------------------------------------------------
 .../jobmaster/JobManagerRunnerMockTest.java        | 17 +----------------
 1 file changed, 1 insertion(+), 16 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/flink/blob/8f576209/flink-runtime/src/test/java/org/apache/flink/runtime/jobmaster/JobManagerRunnerMockTest.java
----------------------------------------------------------------------
diff --git a/flink-runtime/src/test/java/org/apache/flink/runtime/jobmaster/JobManagerRunnerMockTest.java b/flink-runtime/src/test/java/org/apache/flink/runtime/jobmaster/JobManagerRunnerMockTest.java
index 3a769bb..f8a0b6a 100644
--- a/flink-runtime/src/test/java/org/apache/flink/runtime/jobmaster/JobManagerRunnerMockTest.java
+++ b/flink-runtime/src/test/java/org/apache/flink/runtime/jobmaster/JobManagerRunnerMockTest.java
@@ -131,21 +131,6 @@ public class JobManagerRunnerMockTest {
 	}
 
 	@Test
-	public void testJobFinishedByOtherBeforeGrantLeadership() throws Exception {
-		runner.start();
-
-		when(submittedJobGraphStore.contains(any(JobID.class))).thenReturn(false);
-		runner.grantLeadership(UUID.randomUUID());
-
-		// runner should shutdown automatic and informed the job completion
-		verify(leaderElectionService).stop();
-		verify(jobManager).shutDown();
-
-		assertTrue(jobCompletion.isJobFinished());
-		assertTrue(jobCompletion.isJobFinishedByOther());
-	}
-
-	@Test
 	public void testJobFinished() throws Exception {
 		runner.start();
 
@@ -211,7 +196,7 @@ public class JobManagerRunnerMockTest {
 
 		UUID leaderSessionID2 = UUID.randomUUID();
 		runner.grantLeadership(leaderSessionID2);
-		verify(jobManagerGateway, times(2)).startJob(leaderSessionID2);
+		verify(jobManagerGateway).startJob(leaderSessionID2);
 	}
 
 	private static class TestingOnCompletionActions implements OnCompletionActions {