5
0
mirror of https://github.com/apache/sqoop.git synced 2025-05-09 03:41:06 +08:00

SQOOP-2715: Sqoop2: The test case ShowJobInOrderTest always failed

(Colin Ma via Jarek Jarcec Cecho)
This commit is contained in:
Jarek Jarcec Cecho 2015-12-05 13:02:24 +01:00
parent e9df0207e8
commit 1b228f308c
5 changed files with 48 additions and 16 deletions

View File

@ -459,4 +459,22 @@ public void createTableShortStories() {
public void createAndLoadTableShortStories() {
new ShortStories(getInfrastructureProvider(DatabaseInfrastructureProvider.class).getInstance(), getTableName()).createTables().loadBasicData();
}
/**
* Clear the test data for Job.
*/
public void clearJob() {
for(MJob job : getClient().getJobs()) {
getClient().deleteJob(job.getName());
}
}
/**
* Clear the test data for Link.
*/
public void clearLink() {
for(MLink link : getClient().getLinks()) {
getClient().deleteLink(link.getName());
}
}
}

View File

@ -30,6 +30,7 @@
import org.apache.sqoop.test.infrastructure.providers.HadoopInfrastructureProvider;
import org.apache.sqoop.test.infrastructure.providers.SqoopInfrastructureProvider;
import org.apache.sqoop.test.utils.ParametrizedUtils;
import org.testng.annotations.AfterMethod;
import org.testng.annotations.DataProvider;
import org.testng.annotations.Factory;
import org.testng.annotations.Test;
@ -146,4 +147,13 @@ private void verifyActionsForJob(String jobName) throws Exception {
// ignore the exception
}
}
@AfterMethod
public void dropTestData() {
if (TARGET_LINK.equals(target)) {
clearLink();
} else if (TARGET_JOB.equals(target)) {
clearJob();
}
}
}

View File

@ -22,9 +22,10 @@
import org.apache.sqoop.model.MLink;
import org.apache.sqoop.test.infrastructure.Infrastructure;
import org.apache.sqoop.test.infrastructure.SqoopTestCase;
import org.apache.sqoop.test.infrastructure.providers.DatabaseInfrastructureProvider;
import org.apache.sqoop.test.infrastructure.providers.HadoopInfrastructureProvider;
import org.apache.sqoop.test.infrastructure.providers.SqoopInfrastructureProvider;
import org.apache.sqoop.test.testcases.ConnectorTestCase;
import org.testng.annotations.AfterMethod;
import org.testng.annotations.Test;
import java.util.List;
@ -34,7 +35,7 @@
/**
* Ensure that jobs will be shown in order
*/
@Infrastructure(dependencies = {HadoopInfrastructureProvider.class, SqoopInfrastructureProvider.class})
@Infrastructure(dependencies = {HadoopInfrastructureProvider.class, SqoopInfrastructureProvider.class, DatabaseInfrastructureProvider.class})
public class ShowJobInOrderTest extends SqoopTestCase {
public ShowJobInOrderTest() {
@ -56,6 +57,7 @@ public void testShowJobInOrder() throws Exception {
// Job creation
MJob job = getClient().createJob(rdbmsLink.getName(), hdfsLink.getName());
job.setName("testJobName1");
// rdms "FROM" config
fillRdbmsFromConfig(job, "id");
@ -67,6 +69,7 @@ public void testShowJobInOrder() throws Exception {
// Job creation
job = getClient().createJob(hdfsLink.getName(), rdbmsLink.getName());
job.setName("testJobName2");
// rdms "To" config
fillRdbmsToConfig(job);
@ -78,6 +81,7 @@ public void testShowJobInOrder() throws Exception {
// Job creation
job = getClient().createJob(rdbmsLink.getName(), hdfsLink.getName());
job.setName("testJobName3");
// rdms "FROM" config
fillRdbmsFromConfig(job, "id");
@ -89,7 +93,7 @@ public void testShowJobInOrder() throws Exception {
// Job creation
job = getClient().createJob(hdfsLink.getName(), rdbmsLink.getName());
job.setName("testJobName4");
// hdfs "From" config
fillHdfsFromConfig(job);
@ -101,9 +105,14 @@ public void testShowJobInOrder() throws Exception {
List<MJob> jobs = getClient().getJobs();
assertEquals(1, jobs.get(0).getPersistenceId());
assertEquals(2, jobs.get(1).getPersistenceId());
assertEquals(3, jobs.get(2).getPersistenceId());
assertEquals(4, jobs.get(3).getPersistenceId());
assertEquals(jobs.get(0).getName(), "testJobName1");
assertEquals(jobs.get(1).getName(), "testJobName2");
assertEquals(jobs.get(2).getName(), "testJobName3");
assertEquals(jobs.get(3).getName(), "testJobName4");
}
@AfterMethod
public void dropTestData() {
clearJob();
}
}

View File

@ -72,6 +72,8 @@ public void setupRdbmsTable() {
@AfterMethod
public void tearDownRdbmsTable() {
dropTable();
clearJob();
clearLink();
}
@Test

View File

@ -20,16 +20,13 @@
import org.apache.log4j.Logger;
import org.apache.commons.io.IOUtils;
import org.apache.hadoop.security.token.delegation.web.DelegationTokenAuthenticatedURL;
import org.apache.sqoop.common.SqoopException;
import org.apache.sqoop.model.MConfigList;
import org.apache.sqoop.model.MJob;
import org.apache.sqoop.model.MLink;
import org.apache.sqoop.test.infrastructure.Infrastructure;
import org.apache.sqoop.test.infrastructure.SqoopTestCase;
import org.apache.sqoop.test.infrastructure.providers.HadoopInfrastructureProvider;
import org.apache.sqoop.test.infrastructure.providers.SqoopInfrastructureProvider;
import org.testng.annotations.AfterMethod;
import org.testng.annotations.BeforeMethod;
import org.testng.annotations.Test;
import java.io.DataOutputStream;
@ -133,12 +130,8 @@ public void createFirstLink() {
@AfterMethod
public void dropTestData() {
for(MJob job : getClient().getJobs()) {
getClient().deleteJob(job.getName());
}
for(MLink link : getClient().getLinks()) {
getClient().deleteLink(link.getName());
}
clearJob();
clearLink();
}
private TestDescription desc;