Improve tests

Author: alexander-yevsyukov

datastore/src/test/java/io/spine/server/storage/datastore/tenant/NamespaceIndexTest.java

@@ -32,6 +32,8 @@
 import io.spine.server.BoundedContext;
 import io.spine.server.BoundedContextBuilder;
 import io.spine.testing.TestValues;
+import org.junit.jupiter.api.AfterEach;
+import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
 
@@ -61,13 +63,27 @@ class NamespaceIndexTest {
 
     private static final NsConverterFactory converterFactory = NsConverterFactory.defaults();
 
+    private NamespaceIndex namespaceIndex;
+    private BoundedContext context;
+
     private static TenantId newTenantId() {
         return TenantId
                 .newBuilder()
                 .setValue(TestValues.randomString())
                 .vBuild();
     }
 
+    @BeforeEach
+    void createIndex() {
+        namespaceIndex = nsIndex();
+        context = BoundedContextBuilder.assumingTests().build();
+    }
+
+    @AfterEach
+    void closeContext() throws Exception {
+        context.close();
+    }
+
     @Test
     @DisplayName(NOT_ACCEPT_NULLS)
     void testNulls() {
@@ -77,17 +93,13 @@ void testNulls() {
         new NullPointerTester()
                 .setDefault(Namespace.class, defaultNamespace)
                 .setDefault(TenantId.class, tenantId)
-                .setDefault(BoundedContext.class, BoundedContextBuilder.assumingTests()
-                                                                       .build())
-                .testInstanceMethods(nsIndex(),
-                                     NullPointerTester.Visibility.PACKAGE);
+                .setDefault(BoundedContext.class, context)
+                .testInstanceMethods(namespaceIndex, NullPointerTester.Visibility.PACKAGE);
     }
 
     @Test
     @DisplayName("store tenant IDs")
     void testStore() {
-        NamespaceIndex namespaceIndex = nsIndex();
-
         Set<TenantId> initialEmptySet = namespaceIndex.all();
         assertTrue(initialEmptySet.isEmpty());
 
@@ -107,8 +119,6 @@ void testStore() {
     @Test
     @DisplayName("do nothing on close")
     void testClose() {
-        NamespaceIndex namespaceIndex = nsIndex();
-
         namespaceIndex.close();
         namespaceIndex.close();
         // No exception is thrown on the second call to #close() => no operation is performed
@@ -117,8 +127,6 @@ void testClose() {
     @Test
     @DisplayName("find existing namespaces")
     void testFindExisting() {
-        NamespaceIndex namespaceIndex = nsIndex();
-
         // Ensure no namespace has been kept
         Set<TenantId> initialEmptySet = namespaceIndex.all();
         assertTrue(initialEmptySet.isEmpty());
@@ -133,8 +141,6 @@ void testFindExisting() {
     @Test
     @DisplayName("not find non-existing namespaces")
     void testNotFindNonExisting() {
-        NamespaceIndex namespaceIndex = nsIndex();
-
         // Ensure no namespace has been kept
         Set<TenantId> initialEmptySet = namespaceIndex.all();
         assertTrue(initialEmptySet.isEmpty());
@@ -152,6 +158,14 @@ void testAsync() {
                       NamespaceIndexTest::testSynchronizeAccessMethods);
     }
 
+    @Test
+    @DisplayName("confirm registration")
+    void registered() {
+        namespaceIndex.registerWith(context);
+        assertThat(namespaceIndex.isRegistered())
+                .isTrue();
+    }
+
     @SuppressWarnings("OverlyLongMethod")
     private static void testSynchronizeAccessMethods() throws InterruptedException {
         // Initial data