From 8105011368dedb45275db17b1d0c13523486ff12 Mon Sep 17 00:00:00 2001 From: Sam Brannen Date: Wed, 22 Jan 2014 17:44:13 +0100 Subject: [PATCH] Delete unnecessary redeclaration of test methods This commit deletes methods in RollbackForRequiredEjbTxDaoTestNGTests that were unnecessarily redeclared. The redeclaration is required for JUnit's @FixMethodOrder support but not for TestNG's built-in support for dependent methods. Issue: SPR-6132 --- ...RollbackForRequiredEjbTxDaoTestNGTests.java | 18 ------------------ 1 file changed, 18 deletions(-) diff --git a/spring-test/src/test/java/org/springframework/test/context/testng/transaction/ejb/RollbackForRequiredEjbTxDaoTestNGTests.java b/spring-test/src/test/java/org/springframework/test/context/testng/transaction/ejb/RollbackForRequiredEjbTxDaoTestNGTests.java index cf00e5d5de..5a42c062fd 100644 --- a/spring-test/src/test/java/org/springframework/test/context/testng/transaction/ejb/RollbackForRequiredEjbTxDaoTestNGTests.java +++ b/spring-test/src/test/java/org/springframework/test/context/testng/transaction/ejb/RollbackForRequiredEjbTxDaoTestNGTests.java @@ -37,24 +37,6 @@ import static org.testng.AssertJUnit.*; @TransactionConfiguration(defaultRollback = true) public class RollbackForRequiredEjbTxDaoTestNGTests extends CommitForRequiredEjbTxDaoTestNGTests { - /** - * Redeclared to ensure test method execution order. Simply delegates to super. - */ - @Test - @Override - public void test1InitialState() { - super.test1InitialState(); - } - - /** - * Redeclared to ensure test method execution order. Simply delegates to super. - */ - @Test(dependsOnMethods = "test1InitialState") - @Override - public void test2IncrementCount1() { - super.test2IncrementCount1(); - } - /** * Overrides parent implementation in order to change expectations to align with * behavior associated with "required" transactions on repositories/DAOs and