Applications should not delete entire User records in tests.

[mshima]

Overview of the issue

Tests removes every User and Authorities in tests making more difficult to implement and test more advanced security based on Users.
Liquibase and any other bootstrapped data are removed.
We should avoid removing Users and Authorities in tests.

generator-jhipster/generators/server/templates/src/test/java/_package_/security/DomainUserDetailsServiceIT.java.ejs

Line 87 in f0cea14

userRepository.deleteAll()<% if (reactive) { %>.block()<% } %>;

generator-jhipster/generators/server/templates/src/test/java/_package_/web/rest/AccountResourceIT.java.ejs

Line 150 in f0cea14

userRepository.deleteAll()<% if (reactive) { %>.block()<% } %>;

generator-jhipster/generators/spring-boot/templates/src/test/java/_package_/_entityPackage_/web/rest/UserResourceIT.java.ejs

Line 269 in f0cea14

userRepository.deleteAll()<% if (reactive) { %>.block()<% } %>;

Related issues

Suggest a Fix

JHipster Version(s)

JHipster configuration

Entity configuration(s) entityName.json files generated in the .jhipster directory

Browsers and Operating System

• Checking this box is mandatory (this is just to show you read everything)

[mshima]

It’s partially implemented. Missing reactive sql and mongodb.

[mraible]

Since it's related, I recently had to workaround this in my 21-Points Health project. In case others run into this with v8.4.0 and previous, here's a fix: mraible/21-points@caa91ba.

[mshima]

Cache seems to be influencing in tests.
Tests requiring workaround increased:

generator-jhipster/generators/spring-boot/generator.ts

Lines 289 to 294 in fb2b56d

	requiresDeleteAllUsers: data =>
	data.authenticationTypeOauth2 ||
	data.databaseTypeNeo4j ||
	(reactive && data.databaseTypeSql) ||
	(!reactive && data.databaseTypeMongodb) ||
	(!reactive && data.databaseTypeCassandra),