Spring/spring-batch
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Remove ignored test

Browse Source 
Based on the commit message of df1783a7, this
test has no added value.
This commit is contained in:
Mahmoud Ben Hassine
2020-12-03 15:58:48 +01:00
parent ac9a0d2718
commit 12926b46c9
1 changed files with 2 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
spring-batch-infrastructure/src/test/java/org/springframework/batch/support/transaction/ConcurrentTransactionAwareProxyTests.java
View File

@@ -1,5 +1,5 @@
/*
* Copyright 2006-2009 the original author or authors.
* Copyright 2006-2020 the original author or authors.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
@@ -47,6 +47,7 @@ import org.springframework.util.Assert;
/**
* @author Dave Syer
* @author Mahmoud Ben Hassine
*
*/
public class ConcurrentTransactionAwareProxyTests {
@@ -92,24 +93,6 @@ public class ConcurrentTransactionAwareProxyTests {
testList(list, false);
}
@Ignore("This fails too often and is a false negative")
@Test
public void testConcurrentTransactionalList() throws Exception {
List<String> list = TransactionAwareProxyFactory.createTransactionalList();
try {
testList(list, true);
fail("Expected ExecutionException or AssertionError (but don't panic if it didn't happen: it probably just means we got lucky for a change)");
}
catch (ExecutionException e) {
String message = e.getCause().getMessage();
assertTrue("Wrong message: " + message, message.startsWith("Lost update"));
}
catch (AssertionError e) {
String message = e.getMessage();
assertTrue("Wrong message: " + message, message.startsWith("Wrong number of results"));
}
}
@Test
public void testConcurrentTransactionalAppendOnlyMap() throws Exception {
Map<Long, Map<String, String>> map = TransactionAwareProxyFactory.createAppendOnlyTransactionalMap();