[sling-org-apache-sling-distribution-core] branch master updated: SLING-4075 - Improve test coverage of SCD

[tmaret]

This is an automated email from the ASF dual-hosted git repository.

tmaret pushed a commit to branch master
in repository 
https://gitbox.apache.org/repos/asf/sling-org-apache-sling-distribution-core.git


The following commit(s) were added to refs/heads/master by this push:
     new 83b7c01  SLING-4075 - Improve test coverage of SCD
83b7c01 is described below

commit 83b7c010681172cd8500c525ac991c82930552eb
Author: tmaret <tma...@adobe.com>
AuthorDate: Thu Feb 28 15:08:37 2019 +0100

    SLING-4075 - Improve test coverage of SCD
    
    * Fix SingleQueueDistributionStrategyTest
---
 .../queue/impl/SingleQueueDistributionStrategyTest.java           | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git 
a/src/test/java/org/apache/sling/distribution/queue/impl/SingleQueueDistributionStrategyTest.java
 
b/src/test/java/org/apache/sling/distribution/queue/impl/SingleQueueDistributionStrategyTest.java
index 5cc3f0e..e1127dd 100644
--- 
a/src/test/java/org/apache/sling/distribution/queue/impl/SingleQueueDistributionStrategyTest.java
+++ 
b/src/test/java/org/apache/sling/distribution/queue/impl/SingleQueueDistributionStrategyTest.java
@@ -19,6 +19,7 @@
 package org.apache.sling.distribution.queue.impl;
 
 import java.util.Calendar;
+import java.util.Collections;
 import java.util.HashMap;
 import java.util.Iterator;
 
@@ -49,7 +50,8 @@ public class SingleQueueDistributionStrategyTest {
         DistributionQueue queue = mock(DistributionQueue.class);
         
when(queueProvider.getQueue(DistributionQueueDispatchingStrategy.DEFAULT_QUEUE_NAME)).thenReturn(queue);
         DistributionQueueItemStatus state = newDistributionQueueItemStatus();
-        when(queue.add(any(DistributionQueueItem.class))).thenReturn(new 
DistributionQueueEntry(null, null, state));
+        DistributionQueueItem queueItem = new 
DistributionQueueItem("packageId", Collections.<String, Object>emptyMap());
+        when(queue.add(any(DistributionQueueItem.class))).thenReturn(new 
DistributionQueueEntry("entryId", queueItem, state));
 
         Iterable<DistributionQueueItemStatus> returnedStates = 
singleQueueDistributionStrategy.add(distributionPackage, queueProvider);
         assertNotNull(returnedStates);
@@ -85,8 +87,8 @@ public class SingleQueueDistributionStrategyTest {
         DistributionQueue queue = mock(DistributionQueue.class);
         
when(queueProvider.getQueue(DistributionQueueDispatchingStrategy.DEFAULT_QUEUE_NAME)).thenReturn(queue);
         DistributionQueueItemStatus state = newDistributionQueueItemStatus();
-
-        when(queue.add(any(DistributionQueueItem.class))).thenReturn(new 
DistributionQueueEntry(null, null, state));
+        DistributionQueueItem queueItem = new 
DistributionQueueItem("packageId", Collections.<String, Object>emptyMap());
+        when(queue.add(any(DistributionQueueItem.class))).thenReturn(new 
DistributionQueueEntry("entryId", queueItem, state));
 
         Iterable<DistributionQueueItemStatus> returnedStates = 
singleQueueDistributionStrategy.add(distributionPackage, queueProvider);
         assertNotNull(returnedStates);