Repository: incubator-griffin
Updated Branches:
  refs/heads/master a792bdd97 -> 564d4ce21


Topics/refactor ut

Author: evanwan87 <evan.wa...@gmail.com>

Closes #258 from evanwan87/topics/refactor-ut.


Project: http://git-wip-us.apache.org/repos/asf/incubator-griffin/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-griffin/commit/564d4ce2
Tree: http://git-wip-us.apache.org/repos/asf/incubator-griffin/tree/564d4ce2
Diff: http://git-wip-us.apache.org/repos/asf/incubator-griffin/diff/564d4ce2

Branch: refs/heads/master
Commit: 564d4ce21f6eb7c089380e5bb71849a09be76522
Parents: a792bdd
Author: evanwan87 <evan.wa...@gmail.com>
Authored: Mon Apr 16 11:00:35 2018 +0800
Committer: William Guo <guo...@icloud.com>
Committed: Mon Apr 16 11:00:35 2018 +0800

----------------------------------------------------------------------
 .../java/org/apache/griffin/core/job/JobServiceImplTest.java     | 4 ++++
 1 file changed, 4 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-griffin/blob/564d4ce2/service/src/test/java/org/apache/griffin/core/job/JobServiceImplTest.java
----------------------------------------------------------------------
diff --git 
a/service/src/test/java/org/apache/griffin/core/job/JobServiceImplTest.java 
b/service/src/test/java/org/apache/griffin/core/job/JobServiceImplTest.java
index 1d2f0f5..e38f0fa 100644
--- a/service/src/test/java/org/apache/griffin/core/job/JobServiceImplTest.java
+++ b/service/src/test/java/org/apache/griffin/core/job/JobServiceImplTest.java
@@ -340,6 +340,7 @@ public class JobServiceImplTest {
         given(jobRepo.findByJobNameAndDeleted(jobName, false)).willReturn(new 
ArrayList<>());
 
         service.deleteJob(jobName);
+        verify(jobRepo, times(1)).findByJobNameAndDeleted(jobName, false);
     }
 
     @Test(expected = GriffinException.ServiceException.class)
@@ -352,6 +353,9 @@ public class JobServiceImplTest {
         
doThrow(SchedulerException.class).when(scheduler).pauseJob(Matchers.any(JobKey.class));
 
         service.deleteJob(job.getJobName());
+        verify(factory, times(1)).getScheduler();
+        verify(jobRepo, times(1)).findByJobNameAndDeleted(job.getJobName(), 
false);
+        verify(scheduler, times(1)).checkExists(Matchers.any(JobKey.class));
     }
 
     @Test

Reply via email to