deleteById() / deleteAll() not working with indexed properties

I have found another bug which prevents me from switching from fongo to mongo-java-server.

If you modify an indexed property of a document and then delete the document, no new document can be inserted with the value of the originally indexed property.

Here is a test case to reproduce the problem:

@RunWith(SpringRunner.class)
@SpringBootTest(classes={Issue.TestConfiguration.class})
public class Issue {

	@Autowired private TestRepository repository;

	@Test
	public void testDeleteWithUniqueIndexes() throws Exception {
		TestEntity document = repository.save(new TestEntity("DOC_1", "Text1"));

		// update value of indexed property
		document.setText("Text1 (updated)");
		repository.save(document);

		// delete document (deleteAll() does not work either)
		repository.deleteById("DOC_1");

		// duplicate key error
		repository.save(new TestEntity("DOC_1", "Text1"));
	}


	@Configuration
	@EnableMongoRepositories(basePackageClasses={TestRepository.class})
	protected static class TestConfiguration {
		@Bean
		public MongoTemplate mongoTemplate(MongoClient mongoClient) {
			return new MongoTemplate(mongoDbFactory(mongoClient));
		}

		@Bean
		public MongoDbFactory mongoDbFactory(MongoClient mongoClient) {
			return new SimpleMongoDbFactory(mongoClient, "test");
		}

		@Bean(destroyMethod="shutdown")
		public MongoServer mongoServer() {
			MongoServer mongoServer = new MongoServer(new MemoryBackend());
			mongoServer.bind();
			return mongoServer;
		}

		@Bean(destroyMethod="close")
		public MongoClient mongoClient(MongoServer mongoServer) {
			return new MongoClient(new ServerAddress(mongoServer.getLocalAddress()));
		}
	}
}

@Document(collection="test")
public class TestEntity {
	@Id private String id;
	@Indexed(unique=true) private String text;
	public TestEntity() {
	}
	public TestEntity(String id, String text) {
		this.id = id;
		this.text = text;
	}
	public String getId() {
		return id;
	}
	public String getText() {
		return text;
	}
	public void setText(String text) {
		this.text = text;
	}
}

public interface TestRepository extends MongoRepository<TestEntity, String> {
}

[bwaldvogel]

It’s caused by missing support for sparse indices. I’m currently working on it and hope to get it pushed soon.

[bwaldvogel]

That’s great to hear 😀

I’ve just released version 1.11.0