commit f69a5423cc9e8d1fbcbc7de020a1523c1220eab7
author Nasser Grainawi <nasser@codeaurora.org> Wed Aug 26 17:07:56 2020 -0600
committer Nasser Grainawi <nasser@codeaurora.org> Mon Aug 31 16:14:09 2020 -0600
tree 21dde27727d4db3cfc7b627122ccdd2370ef0042
parent 956fbbc9956aeba435d20267849b00466da48d1b

ReplicationTasksStorageTest: Add a test for start()

Also add missing asserts in the
instancesOfTheSameStorageHaveTheSameElements() test.

Change-Id: I68f533b2158326c9bfd035778d750c5cf5c35665

src/test/java/com/googlesource/gerrit/plugins/replication/ReplicationTasksStorageTest.java

diff --git a/src/test/java/com/googlesource/gerrit/plugins/replication/ReplicationTasksStorageTest.java b/src/test/java/com/googlesource/gerrit/plugins/replication/ReplicationTasksStorageTest.java
index 211eb34..02bf762 100644
--- a/src/test/java/com/googlesource/gerrit/plugins/replication/ReplicationTasksStorageTest.java
+++ b/src/test/java/com/googlesource/gerrit/plugins/replication/ReplicationTasksStorageTest.java
@@ -71,6 +71,14 @@
   }
 
   @Test
+  public void canStartWaitingUpdate() throws Exception {
+    storage.create(REF_UPDATE);
+    storage.start(uriUpdates);
+    assertThat(storage.listWaiting()).isEmpty();
+    assertContainsExactly(storage.listRunning(), REF_UPDATE);
+  }
+
+  @Test
   public void canFinishRunningUpdate() throws Exception {
     storage.create(REF_UPDATE);
     storage.start(uriUpdates);
@@ -93,6 +101,8 @@
     storage.start(uriUpdates);
     assertThat(storage.listWaiting()).isEmpty();
     assertThat(persistedView.listWaiting()).isEmpty();
+    assertContainsExactly(storage.listRunning(), REF_UPDATE);
+    assertContainsExactly(persistedView.listRunning(), REF_UPDATE);
 
     storage.finish(uriUpdates);
     assertThat(storage.listRunning()).isEmpty();