HDFS-11736. OIV tests should not write outside 'target' directory. Contributed by Yiqun Lin.

(cherry picked from commit 71626fd0b9)
This commit is contained in:
Akira Ajisaka 2017-06-20 13:55:42 +09:00
parent 6032511239
commit 6a4a63b0fd
No known key found for this signature in database
GPG Key ID: C1EDBB9CA400FD50
3 changed files with 11 additions and 9 deletions

View File

@ -44,7 +44,6 @@ import java.util.Collection;
import java.util.List;
import java.util.Random;
import com.google.common.io.Files;
import org.apache.commons.cli.ParseException;
import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory;
@ -2438,7 +2437,8 @@ public class TestCheckpoint {
public void testLegacyOivImage() throws Exception {
MiniDFSCluster cluster = null;
SecondaryNameNode secondary = null;
File tmpDir = Files.createTempDir();
File tmpDir = GenericTestUtils.getTestDir("testLegacyOivImage");
tmpDir.mkdirs();
Configuration conf = new HdfsConfiguration();
conf.set(DFSConfigKeys.DFS_NAMENODE_LEGACY_OIV_IMAGE_DIR_KEY,
tmpDir.getAbsolutePath());

View File

@ -21,7 +21,6 @@ import com.google.common.base.Supplier;
import com.google.common.collect.ImmutableList;
import com.google.common.collect.ImmutableSet;
import com.google.common.collect.Lists;
import com.google.common.io.Files;
import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory;
import org.apache.hadoop.conf.Configuration;
@ -114,7 +113,8 @@ public class TestStandbyCheckpoints {
}
protected Configuration setupCommonConfig() {
tmpOivImgDir = Files.createTempDir();
tmpOivImgDir = GenericTestUtils.getTestDir("TestStandbyCheckpoints");
tmpOivImgDir.mkdirs();
Configuration conf = new Configuration();
conf.setInt(DFSConfigKeys.DFS_NAMENODE_CHECKPOINT_CHECK_PERIOD_KEY, 1);
@ -136,6 +136,10 @@ public class TestStandbyCheckpoints {
cluster.shutdown();
cluster = null;
}
if (tmpOivImgDir != null) {
FileUtil.fullyDelete(tmpOivImgDir);
}
}
@Test(timeout = 300000)

View File

@ -56,8 +56,6 @@ import javax.xml.parsers.ParserConfigurationException;
import javax.xml.parsers.SAXParser;
import javax.xml.parsers.SAXParserFactory;
import com.google.common.io.Files;
import org.apache.commons.io.FileUtils;
import org.apache.commons.io.output.NullOutputStream;
import org.apache.commons.logging.Log;
@ -87,9 +85,7 @@ import org.apache.log4j.Level;
import org.junit.AfterClass;
import org.junit.Assert;
import org.junit.BeforeClass;
import org.junit.Rule;
import org.junit.Test;
import org.junit.rules.TemporaryFolder;
import org.xml.sax.InputSource;
import org.xml.sax.SAXException;
import org.xml.sax.helpers.DefaultHandler;
@ -116,7 +112,9 @@ public class TestOfflineImageViewer {
// multiple tests.
@BeforeClass
public static void createOriginalFSImage() throws IOException {
tempDir = Files.createTempDir();
tempDir = new File(MiniDFSCluster.getBaseDirectory(), "name1");
tempDir.mkdirs();
MiniDFSCluster cluster = null;
try {
Configuration conf = new Configuration();