diff --git a/client/src/test/java/org/apache/sqoop/client/TestSqoopClient.java b/client/src/test/java/org/apache/sqoop/client/TestSqoopClient.java index 91e34c19..0c6fa621 100644 --- a/client/src/test/java/org/apache/sqoop/client/TestSqoopClient.java +++ b/client/src/test/java/org/apache/sqoop/client/TestSqoopClient.java @@ -53,7 +53,7 @@ public class TestSqoopClient { SqoopResourceRequests resourceRequests; SqoopClient client; - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() { resourceRequests = mock(SqoopResourceRequests.class); client = new SqoopClient("my-cool-server"); diff --git a/common/src/test/java/org/apache/sqoop/validation/validators/TestHostAndPortValidator.java b/common/src/test/java/org/apache/sqoop/validation/validators/TestHostAndPortValidator.java index 401d60c6..1c726ac8 100644 --- a/common/src/test/java/org/apache/sqoop/validation/validators/TestHostAndPortValidator.java +++ b/common/src/test/java/org/apache/sqoop/validation/validators/TestHostAndPortValidator.java @@ -27,7 +27,7 @@ public class TestHostAndPortValidator { AbstractValidator validator = new HostAndPortValidator(); - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() { validator.reset(); assertEquals(0, validator.getMessages().size()); diff --git a/connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/GenericJdbcExecutorTest.java b/connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/GenericJdbcExecutorTest.java index 1734a162..c3b81714 100644 --- a/connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/GenericJdbcExecutorTest.java +++ b/connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/GenericJdbcExecutorTest.java @@ -37,7 +37,7 @@ public GenericJdbcExecutorTest() { GenericJdbcTestConstants.URL, null, null); } - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() { if(executor.existTable(emptyTable)) { executor.executeUpdate("DROP TABLE " + emptyTable); diff --git a/connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/TestExtractor.java b/connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/TestExtractor.java index 3014ed87..83599d31 100644 --- a/connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/TestExtractor.java +++ b/connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/TestExtractor.java @@ -54,7 +54,7 @@ public TestExtractor() { tableName = getClass().getSimpleName().toUpperCase(); } - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() { executor = new GenericJdbcExecutor(GenericJdbcTestConstants.DRIVER, GenericJdbcTestConstants.URL, null, null); @@ -73,7 +73,7 @@ public void setUp() { } } - @AfterMethod + @AfterMethod(alwaysRun = true) public void tearDown() { executor.close(); } diff --git a/connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/TestFromInitializer.java b/connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/TestFromInitializer.java index 5bdcd99c..29a528bc 100644 --- a/connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/TestFromInitializer.java +++ b/connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/TestFromInitializer.java @@ -59,7 +59,7 @@ public TestFromInitializer() { tableColumns = "ICOL,VCOL"; } - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() { executor = new GenericJdbcExecutor(GenericJdbcTestConstants.DRIVER, GenericJdbcTestConstants.URL, null, null); @@ -108,7 +108,7 @@ public Schema getSchema(String name) { ; } - @AfterMethod + @AfterMethod(alwaysRun = true) public void tearDown() { executor.close(); } diff --git a/connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/TestGenericJdbcConnectorUpgrader.java b/connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/TestGenericJdbcConnectorUpgrader.java index febb90fc..1b1fa7f0 100644 --- a/connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/TestGenericJdbcConnectorUpgrader.java +++ b/connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/TestGenericJdbcConnectorUpgrader.java @@ -42,7 +42,7 @@ public class TestGenericJdbcConnectorUpgrader { private GenericJdbcConnectorUpgrader upgrader; - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() { upgrader = new GenericJdbcConnectorUpgrader(); } diff --git a/connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/TestLoader.java b/connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/TestLoader.java index d299f1df..ba665103 100644 --- a/connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/TestLoader.java +++ b/connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/TestLoader.java @@ -56,7 +56,7 @@ public TestLoader(int numberOfRows) { this.tableName = getClass().getSimpleName().toUpperCase(); } - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() { executor = new GenericJdbcExecutor(GenericJdbcTestConstants.DRIVER, GenericJdbcTestConstants.URL, null, null); @@ -70,7 +70,7 @@ public void setUp() { } } - @AfterMethod + @AfterMethod(alwaysRun = true) public void tearDown() { executor.close(); } diff --git a/connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/TestToInitializer.java b/connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/TestToInitializer.java index 01dfab31..a61de7d7 100644 --- a/connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/TestToInitializer.java +++ b/connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/TestToInitializer.java @@ -57,7 +57,7 @@ public TestToInitializer() { tableColumns = "ICOL,VCOL"; } - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() { executor = new GenericJdbcExecutor(GenericJdbcTestConstants.DRIVER, GenericJdbcTestConstants.URL, null, null); @@ -74,7 +74,7 @@ public void setUp() { } } - @AfterMethod + @AfterMethod(alwaysRun = true) public void tearDown() { executor.close(); } diff --git a/connector/connector-hdfs/src/test/java/org/apache/sqoop/connector/hdfs/TestExtractor.java b/connector/connector-hdfs/src/test/java/org/apache/sqoop/connector/hdfs/TestExtractor.java index d761b6d1..9a5ec5e9 100644 --- a/connector/connector-hdfs/src/test/java/org/apache/sqoop/connector/hdfs/TestExtractor.java +++ b/connector/connector-hdfs/src/test/java/org/apache/sqoop/connector/hdfs/TestExtractor.java @@ -75,7 +75,7 @@ public static Object[][] data() { return parameters.toArray(new Object[0][]); } - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() throws Exception { FileUtils.mkdirs(inputDirectory); switch (this.outputFileType) { @@ -89,7 +89,7 @@ public void setUp() throws Exception { } } - @AfterMethod + @AfterMethod(alwaysRun = true) public void tearDown() throws IOException { FileUtils.delete(inputDirectory); } diff --git a/connector/connector-hdfs/src/test/java/org/apache/sqoop/connector/hdfs/TestLoader.java b/connector/connector-hdfs/src/test/java/org/apache/sqoop/connector/hdfs/TestLoader.java index 506dc823..b5ec6dac 100644 --- a/connector/connector-hdfs/src/test/java/org/apache/sqoop/connector/hdfs/TestLoader.java +++ b/connector/connector-hdfs/src/test/java/org/apache/sqoop/connector/hdfs/TestLoader.java @@ -85,10 +85,10 @@ public static Object[][] data() { return parameters.toArray(new Object[0][]); } - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() throws Exception {} - @AfterMethod + @AfterMethod(alwaysRun = true) public void tearDown() throws IOException { FileUtils.delete(outputDirectory); } diff --git a/connector/connector-hdfs/src/test/java/org/apache/sqoop/connector/hdfs/TestPartitioner.java b/connector/connector-hdfs/src/test/java/org/apache/sqoop/connector/hdfs/TestPartitioner.java index 952932cd..4c5b0a0d 100644 --- a/connector/connector-hdfs/src/test/java/org/apache/sqoop/connector/hdfs/TestPartitioner.java +++ b/connector/connector-hdfs/src/test/java/org/apache/sqoop/connector/hdfs/TestPartitioner.java @@ -60,7 +60,7 @@ public TestPartitioner(ToFormat outputFileType, Class topics = new ArrayList(1); @@ -49,7 +49,7 @@ public static void setup() throws IOException { loader = new KafkaLoader(); } - @AfterClass + @AfterClass(alwaysRun = true) public static void tearDown() throws IOException { testUtil.tearDown(); } diff --git a/connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteExecutor.java b/connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteExecutor.java index 3e7d1f11..eed6ee08 100644 --- a/connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteExecutor.java +++ b/connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteExecutor.java @@ -60,7 +60,7 @@ public IObjectFactory getObjectFactory() { return new org.powermock.modules.testng.PowerMockObjectFactory(); } - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() { initMocks(this); when(datasetMock.newWriter()).thenReturn(writerMock); @@ -73,7 +73,7 @@ public void setUp() { executor = new KiteDatasetExecutor(datasetMock); } - @AfterMethod + @AfterMethod(alwaysRun = true) public void tearDown() { executor.closeWriter(); executor.closeReader(); diff --git a/connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteExtractor.java b/connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteExtractor.java index d2c22499..08d2cb3e 100644 --- a/connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteExtractor.java +++ b/connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteExtractor.java @@ -56,7 +56,7 @@ public void writeRecord(Object obj) { } }; - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() { initMocks(this); diff --git a/connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteFromInitializer.java b/connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteFromInitializer.java index 027056ff..a60b2fc1 100644 --- a/connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteFromInitializer.java +++ b/connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteFromInitializer.java @@ -37,7 +37,7 @@ public class TestKiteFromInitializer extends PowerMockTestCase { private KiteFromInitializer initializer; - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() { initMocks(this); mockStatic(Datasets.class); diff --git a/connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteLoader.java b/connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteLoader.java index 8f3a5f80..b18c91ec 100644 --- a/connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteLoader.java +++ b/connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteLoader.java @@ -40,7 +40,7 @@ public class TestKiteLoader { @org.mockito.Mock private KiteDatasetExecutor executorMock; - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() { initMocks(this); diff --git a/connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteToDestroyer.java b/connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteToDestroyer.java index 8c61e759..3fcc3392 100644 --- a/connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteToDestroyer.java +++ b/connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteToDestroyer.java @@ -52,7 +52,7 @@ public class TestKiteToDestroyer extends PowerMockTestCase { @org.mockito.Mock private KiteDatasetExecutor executorMock; - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() { initMocks(this); mockStatic(KiteDatasetExecutor.class); diff --git a/connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteToInitializer.java b/connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteToInitializer.java index 93efa93c..5230ffe5 100644 --- a/connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteToInitializer.java +++ b/connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteToInitializer.java @@ -41,7 +41,7 @@ public class TestKiteToInitializer extends PowerMockTestCase { private KiteToInitializer initializer; - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() { initMocks(this); mockStatic(Datasets.class); diff --git a/connector/connector-sdk/src/test/java/org/apache/sqoop/connector/idf/TestAVROIntermediateDataFormat.java b/connector/connector-sdk/src/test/java/org/apache/sqoop/connector/idf/TestAVROIntermediateDataFormat.java index 5c7b444d..dd7a3e18 100644 --- a/connector/connector-sdk/src/test/java/org/apache/sqoop/connector/idf/TestAVROIntermediateDataFormat.java +++ b/connector/connector-sdk/src/test/java/org/apache/sqoop/connector/idf/TestAVROIntermediateDataFormat.java @@ -67,7 +67,7 @@ public class TestAVROIntermediateDataFormat { private final static org.joda.time.LocalTime time = new org.joda.time.LocalTime(12, 59, 59); private final static org.joda.time.LocalDate date = new org.joda.time.LocalDate(2014, 10, 01); - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() { createAvroIDF(); } diff --git a/connector/connector-sdk/src/test/java/org/apache/sqoop/connector/idf/TestCSVIntermediateDataFormat.java b/connector/connector-sdk/src/test/java/org/apache/sqoop/connector/idf/TestCSVIntermediateDataFormat.java index d2b0ae0b..25567115 100644 --- a/connector/connector-sdk/src/test/java/org/apache/sqoop/connector/idf/TestCSVIntermediateDataFormat.java +++ b/connector/connector-sdk/src/test/java/org/apache/sqoop/connector/idf/TestCSVIntermediateDataFormat.java @@ -53,7 +53,7 @@ public class TestCSVIntermediateDataFormat { private CSVIntermediateDataFormat dataFormat; - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() { } diff --git a/connector/connector-sdk/src/test/java/org/apache/sqoop/connector/idf/TestJSONIntermediateDataFormat.java b/connector/connector-sdk/src/test/java/org/apache/sqoop/connector/idf/TestJSONIntermediateDataFormat.java index 8a87c650..12c66c2a 100644 --- a/connector/connector-sdk/src/test/java/org/apache/sqoop/connector/idf/TestJSONIntermediateDataFormat.java +++ b/connector/connector-sdk/src/test/java/org/apache/sqoop/connector/idf/TestJSONIntermediateDataFormat.java @@ -52,7 +52,7 @@ public class TestJSONIntermediateDataFormat { private final static String dateTime = "'2014-10-01 12:00:00.000'"; private final static String time = "'12:59:59'"; - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() { createJSONIDF(); } diff --git a/connector/connector-sdk/src/test/java/org/apache/sqoop/connector/matcher/TestLocationMatcher.java b/connector/connector-sdk/src/test/java/org/apache/sqoop/connector/matcher/TestLocationMatcher.java index 624fa7b8..f23694d3 100644 --- a/connector/connector-sdk/src/test/java/org/apache/sqoop/connector/matcher/TestLocationMatcher.java +++ b/connector/connector-sdk/src/test/java/org/apache/sqoop/connector/matcher/TestLocationMatcher.java @@ -29,7 +29,7 @@ public class TestLocationMatcher { private LocationMatcher matcher; - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() { matcher = null; } diff --git a/connector/connector-sdk/src/test/java/org/apache/sqoop/connector/matcher/TestNameMatcher.java b/connector/connector-sdk/src/test/java/org/apache/sqoop/connector/matcher/TestNameMatcher.java index 76ff0da4..742a35c5 100644 --- a/connector/connector-sdk/src/test/java/org/apache/sqoop/connector/matcher/TestNameMatcher.java +++ b/connector/connector-sdk/src/test/java/org/apache/sqoop/connector/matcher/TestNameMatcher.java @@ -29,7 +29,7 @@ public class TestNameMatcher { private NameMatcher matcher; - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() { matcher = null; } diff --git a/core/src/test/java/org/apache/sqoop/core/TestSqoopConfiguration.java b/core/src/test/java/org/apache/sqoop/core/TestSqoopConfiguration.java index e15691eb..956e6abb 100644 --- a/core/src/test/java/org/apache/sqoop/core/TestSqoopConfiguration.java +++ b/core/src/test/java/org/apache/sqoop/core/TestSqoopConfiguration.java @@ -26,7 +26,7 @@ public class TestSqoopConfiguration { - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() throws Exception { // Unset any configuration dir if it is set by another test System.getProperties().remove(ConfigurationConstants.SYSPROP_CONFIG_DIR); diff --git a/core/src/test/java/org/apache/sqoop/driver/TestDriverConfigUpgrader.java b/core/src/test/java/org/apache/sqoop/driver/TestDriverConfigUpgrader.java index ec548d5d..d4522ae7 100644 --- a/core/src/test/java/org/apache/sqoop/driver/TestDriverConfigUpgrader.java +++ b/core/src/test/java/org/apache/sqoop/driver/TestDriverConfigUpgrader.java @@ -39,7 +39,7 @@ public class TestDriverConfigUpgrader { DriverUpgrader upgrader; - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void initializeUpgrader() { upgrader = new DriverUpgrader(); } diff --git a/core/src/test/java/org/apache/sqoop/driver/TestJobManager.java b/core/src/test/java/org/apache/sqoop/driver/TestJobManager.java index 926815d7..baca35f9 100644 --- a/core/src/test/java/org/apache/sqoop/driver/TestJobManager.java +++ b/core/src/test/java/org/apache/sqoop/driver/TestJobManager.java @@ -48,7 +48,7 @@ public class TestJobManager { private RepositoryManager repositoryManagerMock; private Repository jdbcRepoMock; - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() { jobManager = new JobManager(); connectorMgrMock = mock(ConnectorManager.class); diff --git a/core/src/test/java/org/apache/sqoop/driver/TestJobRequest.java b/core/src/test/java/org/apache/sqoop/driver/TestJobRequest.java index 26560fe8..4381b97f 100644 --- a/core/src/test/java/org/apache/sqoop/driver/TestJobRequest.java +++ b/core/src/test/java/org/apache/sqoop/driver/TestJobRequest.java @@ -33,7 +33,7 @@ public class TestJobRequest { private JobRequest jobRequest; - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void initializeSubmissionRequest() { jobRequest = new JobRequest(); } diff --git a/core/src/test/java/org/apache/sqoop/repository/TestJdbcRepository.java b/core/src/test/java/org/apache/sqoop/repository/TestJdbcRepository.java index f804a455..9b153fc5 100644 --- a/core/src/test/java/org/apache/sqoop/repository/TestJdbcRepository.java +++ b/core/src/test/java/org/apache/sqoop/repository/TestJdbcRepository.java @@ -76,7 +76,7 @@ public class TestJdbcRepository { private ConnectorConfigurableUpgrader connectorUpgraderMock; private DriverUpgrader driverUpgraderMock; - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() throws Exception { repoTransactionMock = mock(JdbcRepositoryTransaction.class); connectorMgrMock = mock(ConnectorManager.class); diff --git a/execution/mapreduce/src/test/java/org/apache/sqoop/job/io/TestSqoopWritable.java b/execution/mapreduce/src/test/java/org/apache/sqoop/job/io/TestSqoopWritable.java index 452e0857..85bf73d1 100644 --- a/execution/mapreduce/src/test/java/org/apache/sqoop/job/io/TestSqoopWritable.java +++ b/execution/mapreduce/src/test/java/org/apache/sqoop/job/io/TestSqoopWritable.java @@ -44,7 +44,7 @@ public class TestSqoopWritable { private SqoopWritable writable; private IntermediateDataFormat idfMock; - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() { idfMock = mock(IntermediateDataFormat.class); writable = new SqoopWritable(idfMock); diff --git a/execution/mapreduce/src/test/java/org/apache/sqoop/job/mr/TestMRConfigurationUtils.java b/execution/mapreduce/src/test/java/org/apache/sqoop/job/mr/TestMRConfigurationUtils.java index a3e6dee0..b83de3b2 100644 --- a/execution/mapreduce/src/test/java/org/apache/sqoop/job/mr/TestMRConfigurationUtils.java +++ b/execution/mapreduce/src/test/java/org/apache/sqoop/job/mr/TestMRConfigurationUtils.java @@ -44,7 +44,7 @@ public class TestMRConfigurationUtils { Job job; JobConf jobConfSpy; - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() throws Exception { setUpHadoopJob(); setUpHadoopJobConf(); diff --git a/execution/mapreduce/src/test/java/org/apache/sqoop/job/mr/TestSqoopOutputFormatLoadExecutor.java b/execution/mapreduce/src/test/java/org/apache/sqoop/job/mr/TestSqoopOutputFormatLoadExecutor.java index 41ea24aa..71c98db5 100644 --- a/execution/mapreduce/src/test/java/org/apache/sqoop/job/mr/TestSqoopOutputFormatLoadExecutor.java +++ b/execution/mapreduce/src/test/java/org/apache/sqoop/job/mr/TestSqoopOutputFormatLoadExecutor.java @@ -155,7 +155,7 @@ private Schema getSchema() { return new Schema("test").addColumn(new Text("t")); } - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() { conf = new Configuration(); conf.setIfUnset(MRJobConstants.TO_INTERMEDIATE_DATA_FORMAT, diff --git a/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/DerbyTestCase.java b/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/DerbyTestCase.java index 1b6bdc0d..be8c23eb 100644 --- a/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/DerbyTestCase.java +++ b/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/DerbyTestCase.java @@ -60,14 +60,14 @@ abstract public class DerbyTestCase { private Connection connection; - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() throws Exception { // Create link to the database Class.forName(DERBY_DRIVER).newInstance(); connection = DriverManager.getConnection(getStartJdbcUrl()); } - @AfterMethod + @AfterMethod(alwaysRun = true) public void tearDown() throws Exception { // Close active link if (connection != null) { diff --git a/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestConnectorHandling.java b/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestConnectorHandling.java index 248873bf..ca24398a 100644 --- a/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestConnectorHandling.java +++ b/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestConnectorHandling.java @@ -34,7 +34,7 @@ public class TestConnectorHandling extends DerbyTestCase { DerbyRepositoryHandler handler; - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() throws Exception { super.setUp(); handler = new DerbyRepositoryHandler(); diff --git a/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestDriverHandling.java b/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestDriverHandling.java index 91d8b929..e12bf46e 100644 --- a/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestDriverHandling.java +++ b/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestDriverHandling.java @@ -34,7 +34,7 @@ public class TestDriverHandling extends DerbyTestCase { private static final Object CURRENT_DRIVER_VERSION = "1"; DerbyRepositoryHandler handler; - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() throws Exception { super.setUp(); handler = new DerbyRepositoryHandler(); diff --git a/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestInputTypes.java b/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestInputTypes.java index 2cbe255d..fb071523 100644 --- a/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestInputTypes.java +++ b/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestInputTypes.java @@ -48,7 +48,7 @@ public class TestInputTypes extends DerbyTestCase { DerbyRepositoryHandler handler; - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() throws Exception { super.setUp(); diff --git a/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestJobHandling.java b/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestJobHandling.java index 3cc6928a..458450bd 100644 --- a/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestJobHandling.java +++ b/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestJobHandling.java @@ -48,7 +48,7 @@ public class TestJobHandling extends DerbyTestCase { DerbyRepositoryHandler handler; Connection derbyConnection; - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() throws Exception { super.setUp(); derbyConnection = getDerbyDatabaseConnection(); diff --git a/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestLinkHandling.java b/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestLinkHandling.java index 1c96deb7..5602c7c7 100644 --- a/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestLinkHandling.java +++ b/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestLinkHandling.java @@ -42,7 +42,7 @@ public class TestLinkHandling extends DerbyTestCase { DerbyRepositoryHandler handler; - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() throws Exception { super.setUp(); diff --git a/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestRepositoryUpgrade.java b/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestRepositoryUpgrade.java index b53aeffc..9255882a 100644 --- a/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestRepositoryUpgrade.java +++ b/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestRepositoryUpgrade.java @@ -30,7 +30,7 @@ public class TestRepositoryUpgrade extends DerbyTestCase { DerbyRepositoryHandler handler; - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() throws Exception { super.setUp(); handler = new TestDerbyRepositoryHandler(); diff --git a/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestSubmissionHandling.java b/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestSubmissionHandling.java index 523ddc7b..bb96e3c9 100644 --- a/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestSubmissionHandling.java +++ b/repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestSubmissionHandling.java @@ -38,7 +38,7 @@ public class TestSubmissionHandling extends DerbyTestCase { DerbyRepositoryHandler handler; - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() throws Exception { super.setUp(); diff --git a/repository/repository-postgresql/src/test/java/org/apache/sqoop/repository/postgresql/PostgresqlTestCase.java b/repository/repository-postgresql/src/test/java/org/apache/sqoop/repository/postgresql/PostgresqlTestCase.java index ea199d4b..ae546f30 100644 --- a/repository/repository-postgresql/src/test/java/org/apache/sqoop/repository/postgresql/PostgresqlTestCase.java +++ b/repository/repository-postgresql/src/test/java/org/apache/sqoop/repository/postgresql/PostgresqlTestCase.java @@ -21,8 +21,8 @@ import org.apache.sqoop.common.test.db.PostgreSQLProvider; import org.testng.SkipException; import org.testng.annotations.AfterMethod; +import org.testng.annotations.BeforeClass; import org.testng.annotations.BeforeMethod; -import org.testng.annotations.Test; /** * Abstract class with convenience methods for testing postgresql repository. @@ -33,13 +33,13 @@ abstract public class PostgresqlTestCase { public static PostgresqlTestUtils utils; public PostgresqlRepositoryHandler handler; - @Test - public static void setUpClass() { + @BeforeClass(alwaysRun = true) + public void setUpClass() { provider = new PostgreSQLProvider(); utils = new PostgresqlTestUtils(provider); } - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void setUp() throws Exception { try { provider.start(); @@ -51,7 +51,7 @@ public void setUp() throws Exception { handler.createOrUpgradeRepository(provider.getConnection()); } - @AfterMethod + @AfterMethod(alwaysRun = true) public void tearDown() throws Exception { provider.dropSchema("sqoop"); provider.stop(); diff --git a/test/src/main/java/org/apache/sqoop/test/testcases/ConnectorTestCase.java b/test/src/main/java/org/apache/sqoop/test/testcases/ConnectorTestCase.java index e98d0913..1124cd33 100644 --- a/test/src/main/java/org/apache/sqoop/test/testcases/ConnectorTestCase.java +++ b/test/src/main/java/org/apache/sqoop/test/testcases/ConnectorTestCase.java @@ -75,7 +75,7 @@ public void finished(MSubmission submission) { } }; - @BeforeClass + @BeforeClass(alwaysRun = true) public static void startHadoop() throws Exception { // Start Hadoop Clusters hadoopCluster = HadoopRunnerFactory.getHadoopCluster(System.getProperties(), HadoopMiniClusterRunner.class); @@ -88,14 +88,14 @@ public static void startHadoop() throws Exception { LOG.debug("HDFS Client: " + hdfsClient); } - @BeforeClass + @BeforeClass(alwaysRun = true) public static void startProvider() throws Exception { provider = DatabaseProviderFactory.getProvider(System.getProperties()); LOG.info("Starting database provider: " + provider.getClass().getName()); provider.start(); } - @AfterClass + @AfterClass(alwaysRun = true) public static void stopProvider() { LOG.info("Stopping database provider: " + provider.getClass().getName()); provider.stop(); diff --git a/test/src/main/java/org/apache/sqoop/test/testcases/KafkaConnectorTestCase.java b/test/src/main/java/org/apache/sqoop/test/testcases/KafkaConnectorTestCase.java index cc6724e3..a65d862b 100644 --- a/test/src/main/java/org/apache/sqoop/test/testcases/KafkaConnectorTestCase.java +++ b/test/src/main/java/org/apache/sqoop/test/testcases/KafkaConnectorTestCase.java @@ -38,13 +38,13 @@ public class KafkaConnectorTestCase extends ConnectorTestCase { private static TestUtil testUtil = TestUtil.getInstance(); private static final String TOPIC = "mytopic"; - @BeforeClass + @BeforeClass(alwaysRun = true) public static void startKafka() throws IOException { // starts Kafka server and its dependent zookeeper testUtil.prepare(); } - @AfterClass + @AfterClass(alwaysRun = true) public static void stopKafka() throws IOException { testUtil.tearDown(); } diff --git a/test/src/main/java/org/apache/sqoop/test/testcases/TomcatTestCase.java b/test/src/main/java/org/apache/sqoop/test/testcases/TomcatTestCase.java index ce748943..db88294d 100644 --- a/test/src/main/java/org/apache/sqoop/test/testcases/TomcatTestCase.java +++ b/test/src/main/java/org/apache/sqoop/test/testcases/TomcatTestCase.java @@ -88,7 +88,7 @@ abstract public class TomcatTestCase { */ private SqoopClient client; - @BeforeClass + @BeforeClass(alwaysRun = true) public static void startHadoop() throws Exception { // Start Hadoop Clusters hadoopCluster = HadoopRunnerFactory.getHadoopCluster(System.getProperties(), HadoopLocalRunner.class); @@ -101,12 +101,12 @@ public static void startHadoop() throws Exception { LOG.debug("HDFS Client: " + hdfsClient); } - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void findMethodName(Method method) { name = method.getName(); } - @BeforeMethod + @BeforeMethod(alwaysRun = true) public void startServer() throws Exception { // Get and set temporary path in hadoop cluster. tmpPath = HdfsUtils.joinPathFragments(TMP_PATH_BASE, getClass().getName(), name); @@ -122,12 +122,12 @@ public void startServer() throws Exception { client = new SqoopClient(getServerUrl()); } - @AfterMethod + @AfterMethod(alwaysRun = true) public void stopServer() throws Exception { cluster.stop(); } - @AfterClass + @AfterClass(alwaysRun = true) public static void stopHadoop() throws Exception { hadoopCluster.stop(); } diff --git a/test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/PartitionerTest.java b/test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/PartitionerTest.java index 352fb269..f69f08ce 100644 --- a/test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/PartitionerTest.java +++ b/test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/PartitionerTest.java @@ -34,6 +34,7 @@ /** * */ +@Test(groups = "slow") public class PartitionerTest extends ConnectorTestCase implements ITest { /**