Clarify test intent

1 files changed, 5 insertions, 0 deletions

diff --git a/configserver/src/test/java/com/yahoo/vespa/config/server/session/RemoteSessionRepoTest.java b/configserver/src/test/java/com/yahoo/vespa/config/server/session/RemoteSessionRepoTest.java
index cbea7c286cc..e4972e0b9b3 100644
--- a/configserver/src/test/java/com/yahoo/vespa/config/server/session/RemoteSessionRepoTest.java
+++ b/configserver/src/test/java/com/yahoo/vespa/config/server/session/RemoteSessionRepoTest.java
@@ -87,6 +87,10 @@ public class RemoteSessionRepoTest extends TestWithCurator {
         assertNull(remoteSessionRepo.getSession(0l));
     }
 
+    // If reading a session throws an exception it should be handled and not prevent other applications
+    // from loading. In this test we just show that we end up with one session in remote session
+    // repo even if it had bad data (by making getSessionIdForApplication() in FailingTenantApplications
+    // throw an exception).
     @Test
     public void testBadApplicationRepoOnActivate() throws Exception {
         TenantApplications applicationRepo = new FailingTenantApplications();
@@ -97,6 +101,7 @@ public class RemoteSessionRepoTest extends TestWithCurator {
                 .withApplicationRepo(applicationRepo)
                 .build();
         remoteSessionRepo = tenant.getRemoteSessionRepo();
+        assertThat(remoteSessionRepo.listSessions().size(), is(0));
         createSession("/mytenant", 2l, true);
         assertThat(remoteSessionRepo.listSessions().size(), is(1));
     }