Test and fix for cold initialisation with ids

Author: JohannesRabauer

spring-data-eclipse-store/src/main/java/software/xdev/spring/data/eclipse/store/repository/EclipseStoreStorage.java

@@ -280,10 +280,15 @@ public <T> IdSetter<T> ensureIdSetter(final Class<T> domainClass)
 				final String entityName = this.getEntityName(domainClass);
 				final Consumer<Object> idSetter = id ->
 				{
+					this.ensureEntitiesInRoot();
 					this.root.getLastIds().put(entityName, id);
 					this.storageManager.store(this.root.getLastIds());
 				};
-				final Supplier<Object> idGetter = () -> this.root.getLastIds().get(entityName);
+				final Supplier<Object> idGetter = () ->
+				{
+					this.ensureEntitiesInRoot();
+					return this.root.getLastIds().get(entityName);
+				};
 				return IdSetter.createIdSetter(domainClass, idSetter, idGetter);
 			}
 		);

spring-data-eclipse-store/src/test/java/software/xdev/spring/data/eclipse/store/integration/tests/IdTest.java

@@ -15,14 +15,15 @@
  */
 package software.xdev.spring.data.eclipse.store.integration.tests;
 
+import static software.xdev.spring.data.eclipse.store.helper.TestUtil.restartDatastore;
+
 import java.util.List;
 import java.util.Optional;
 
 import org.junit.jupiter.api.Assertions;
 import org.junit.jupiter.api.Test;
 import org.springframework.beans.factory.annotation.Autowired;
 
-import org.springframework.beans.factory.annotation.Autowired;
 import software.xdev.spring.data.eclipse.store.helper.TestData;
 import software.xdev.spring.data.eclipse.store.helper.TestUtil;
 import software.xdev.spring.data.eclipse.store.integration.DefaultTestAnnotations;
@@ -65,6 +66,27 @@ void testCreateSingleWithAutoIdInteger(@Autowired final CustomerWithIdIntegerRep
 		);
 	}
 
+	/**
+	 * In other tests {@link EclipseStoreStorage#clearData} is called. Here the datastore is restarted again to ensure
+	 * no previous method is called before the test.
+	 */
+	@Test
+	void testSaveSingleWithoutAnyPreviousCall(@Autowired final CustomerWithIdIntegerRepository customerRepository)
+	{
+		restartDatastore(this.storage);
+		final CustomerWithIdInteger customer1 = new CustomerWithIdInteger(TestData.FIRST_NAME, TestData.LAST_NAME);
+		customerRepository.save(customer1);
+		
+		TestUtil.doBeforeAndAfterRestartOfDatastore(
+			this.storage,
+			() -> {
+				final Optional<CustomerWithIdInteger> loadedCustomer = customerRepository.findById(0);
+				Assertions.assertTrue(loadedCustomer.isPresent());
+				Assertions.assertEquals(customer1, loadedCustomer.get());
+			}
+		);
+	}
+	
 	@Test
 	void testCreateSingleWithAutoIdIntegerWorkingCopyIdSet(
 		@Autowired final CustomerWithIdIntegerRepository customerRepository)