review poitns finished
This commit is contained in:
parent
b3dcc17ccb
commit
88a916ece5
|
@ -297,7 +297,9 @@ public class JpaJobPersistenceImpl implements IJobPersistence {
|
||||||
public void enqueueWorkChunkForProcessing(String theChunkId, Consumer<Integer> theCallback) {
|
public void enqueueWorkChunkForProcessing(String theChunkId, Consumer<Integer> theCallback) {
|
||||||
int updated = myWorkChunkRepository.updateChunkStatus(
|
int updated = myWorkChunkRepository.updateChunkStatus(
|
||||||
theChunkId, WorkChunkStatusEnum.QUEUED, WorkChunkStatusEnum.READY);
|
theChunkId, WorkChunkStatusEnum.QUEUED, WorkChunkStatusEnum.READY);
|
||||||
theCallback.accept(updated);
|
if (updated == 1) {
|
||||||
|
theCallback.accept(updated);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
|
|
|
@ -55,6 +55,7 @@ import static org.mockito.ArgumentMatchers.any;
|
||||||
import static org.mockito.ArgumentMatchers.anyInt;
|
import static org.mockito.ArgumentMatchers.anyInt;
|
||||||
import static org.mockito.ArgumentMatchers.eq;
|
import static org.mockito.ArgumentMatchers.eq;
|
||||||
import static org.mockito.Mockito.doReturn;
|
import static org.mockito.Mockito.doReturn;
|
||||||
|
import static org.mockito.Mockito.never;
|
||||||
import static org.mockito.Mockito.times;
|
import static org.mockito.Mockito.times;
|
||||||
import static org.mockito.Mockito.verify;
|
import static org.mockito.Mockito.verify;
|
||||||
import static org.mockito.Mockito.verifyNoMoreInteractions;
|
import static org.mockito.Mockito.verifyNoMoreInteractions;
|
||||||
|
@ -476,6 +477,7 @@ public class JobCoordinatorImplTest extends BaseBatch2Test {
|
||||||
assertSame(jobDefinition, myJobDefinitionCaptor.getValue());
|
assertSame(jobDefinition, myJobDefinitionCaptor.getValue());
|
||||||
assertEquals(startRequest.getParameters(), myParametersJsonCaptor.getValue());
|
assertEquals(startRequest.getParameters(), myParametersJsonCaptor.getValue());
|
||||||
|
|
||||||
|
verify(myBatchJobSender, never()).sendWorkChannelMessage(any());
|
||||||
verifyNoMoreInteractions(myJobInstancePersister);
|
verifyNoMoreInteractions(myJobInstancePersister);
|
||||||
verifyNoMoreInteractions(myStep1Worker);
|
verifyNoMoreInteractions(myStep1Worker);
|
||||||
verifyNoMoreInteractions(myStep2Worker);
|
verifyNoMoreInteractions(myStep2Worker);
|
||||||
|
|
Loading…
Reference in New Issue