You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by ha...@apache.org on 2015/03/31 08:01:22 UTC

directory-kerby git commit: DIRKRB-203 ConfigTest failed on windows. Contributed by Yaning

Repository: directory-kerby
Updated Branches:
  refs/heads/master a93a49afc -> 9ec7b32cf


DIRKRB-203 ConfigTest failed on windows. Contributed by Yaning


Project: http://git-wip-us.apache.org/repos/asf/directory-kerby/repo
Commit: http://git-wip-us.apache.org/repos/asf/directory-kerby/commit/9ec7b32c
Tree: http://git-wip-us.apache.org/repos/asf/directory-kerby/tree/9ec7b32c
Diff: http://git-wip-us.apache.org/repos/asf/directory-kerby/diff/9ec7b32c

Branch: refs/heads/master
Commit: 9ec7b32cf37df45226d0ad21d67231d75fadcc45
Parents: a93a49a
Author: Lin <li...@foxmail.com>
Authored: Tue Mar 31 13:58:13 2015 +0800
Committer: Lin <li...@foxmail.com>
Committed: Tue Mar 31 13:58:13 2015 +0800

----------------------------------------------------------------------
 .../org/apache/kerby/config/IniConfigTest.java  | 21 +++++++++++++++-----
 1 file changed, 16 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/directory-kerby/blob/9ec7b32c/lib/kerby-config/src/test/java/org/apache/kerby/config/IniConfigTest.java
----------------------------------------------------------------------
diff --git a/lib/kerby-config/src/test/java/org/apache/kerby/config/IniConfigTest.java b/lib/kerby-config/src/test/java/org/apache/kerby/config/IniConfigTest.java
index 3e0f04e..75d7fb6 100644
--- a/lib/kerby-config/src/test/java/org/apache/kerby/config/IniConfigTest.java
+++ b/lib/kerby-config/src/test/java/org/apache/kerby/config/IniConfigTest.java
@@ -19,6 +19,8 @@
  */
 package org.apache.kerby.config;
 
+import org.junit.After;
+import org.junit.Before;
 import org.junit.Test;
 
 import java.io.File;
@@ -30,9 +32,16 @@ import static org.assertj.core.api.Assertions.assertThat;
 
 public class IniConfigTest {
 
-    private final static String TEST_DIR = new File(System.getProperty(
-            "test.build.data", "/tmp")).getAbsolutePath();
-    private final static File TEST_FILE = new File(TEST_DIR, "test-ini-config");
+    private final File TEST_DIR = new File(System.getProperty("test.dir","target"));
+    private final File TEST_FILE = new File(TEST_DIR, "test-ini-config");
+
+    @Before
+    public void setUp() throws IOException {
+        if (TEST_FILE.exists()){
+            TEST_FILE.delete();
+        }
+        buildFile();
+    }
 
     /**
      * Build a INI format configuration file.
@@ -58,8 +67,6 @@ public class IniConfigTest {
 
     @Test
     public void testIniConfig() throws IOException {
-        buildFile();
-
         Conf conf = new Conf();
         conf.addIniConfig(TEST_FILE);
 
@@ -74,4 +81,8 @@ public class IniConfigTest {
         assertThat(config1.getBoolean("dns_lookup_realm")).isTrue();
     }
 
+    @After
+    public void tearDown() {
+        TEST_FILE.delete();
+    }
 }