Skip to content

Commit fe20b5b

Browse files
committed
add comment to explain test result invert
Signed-off-by: David BRAQUART <[email protected]>
1 parent 966741e commit fe20b5b

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

src/test/java/org/gridsuite/modification/server/ModificationControllerTest.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -450,7 +450,8 @@ void testNetworkModificationsWithErrorOnNetworkFlush() throws Exception {
450450
String groovyScriptInfosJson = getJsonBody(groovyScriptInfos, TEST_NETWORK_WITH_FLUSH_ERROR_ID, NetworkCreation.VARIANT_ID);
451451

452452
mockMvc.perform(post(NETWORK_MODIFICATION_URI).content(groovyScriptInfosJson).contentType(MediaType.APPLICATION_JSON)).andExpect(status().is5xxServerError());
453-
453+
// TODO because creating a modification is now a unique Tx, and because network.flush() is not covered by the apply() exception catch,
454+
// this test result has changed : the modification save is not committed because of the flush exception.
454455
assertEquals(0, modificationRepository.getModifications(TEST_GROUP_ID, true, false).size());
455456
}
456457

0 commit comments

Comments
 (0)