diff options
author | Harald Musum <musum@verizonmedia.com> | 2019-07-30 08:25:46 +0200 |
---|---|---|
committer | Harald Musum <musum@verizonmedia.com> | 2019-07-30 08:25:46 +0200 |
commit | 6868522340cf32563ae2edb239a90705c6c10f80 (patch) | |
tree | 0d9a2735e6002bc2a8aa9e0f5ba85bd847443a24 /configgen/src/test/java | |
parent | e02e35f6dea6b31af7e8726c9d898ca7602c370d (diff) |
Cleanup MakeConfig
Languages 'cppng' and 'cpp' are identical now, remove use of 'cppng'.
Remove unused property 'config.requireNamespace'.
Diffstat (limited to 'configgen/src/test/java')
-rw-r--r-- | configgen/src/test/java/com/yahoo/config/codegen/MakeConfigTest.java | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/configgen/src/test/java/com/yahoo/config/codegen/MakeConfigTest.java b/configgen/src/test/java/com/yahoo/config/codegen/MakeConfigTest.java index 501d7778fd7..7486d464e43 100644 --- a/configgen/src/test/java/com/yahoo/config/codegen/MakeConfigTest.java +++ b/configgen/src/test/java/com/yahoo/config/codegen/MakeConfigTest.java @@ -12,7 +12,7 @@ import org.junit.Test; public class MakeConfigTest { - File dest; + private File dest; @Before public void setUp() { @@ -29,8 +29,9 @@ public class MakeConfigTest { if (dir.isDirectory()) { String[] children = dir.list(); - for (int i = 0; i < children.length; i++) { - boolean success = recursiveDeleteDir(new File(dir, children[i])); + assert children != null; + for (String child : children) { + boolean success = recursiveDeleteDir(new File(dir, child)); if (!success) return false; } @@ -42,10 +43,8 @@ public class MakeConfigTest { @Test public void testProps() throws PropertyException { - long ts = System.currentTimeMillis(); System.setProperty("config.dumpTree", "true"); System.setProperty("config.useFramework", "true"); - System.setProperty("config.requireNamespace", "true"); System.setProperty("config.dest", dest.getAbsolutePath()); System.setProperty("config.spec", "src/test/resources/allfeatures.def"); MakeConfigProperties p = new MakeConfigProperties(); @@ -57,7 +56,6 @@ public class MakeConfigTest { System.setProperty("config.dumpTree", "false"); System.setProperty("config.useFramework", "false"); - System.setProperty("config.requireNamespace", "false"); System.setProperty("config.dest", dest.getAbsolutePath()); System.setProperty("config.spec", "src/test/resources/allfeatures.def,src/test/resources/bar.foo.def"); p = new MakeConfigProperties(); @@ -71,7 +69,6 @@ public class MakeConfigTest { public void testMake() throws IOException, InterruptedException { System.setProperty("config.dumpTree", "true"); System.setProperty("config.useFramework", "true"); - System.setProperty("config.requireNamespace", "true"); System.setProperty("config.dest", dest.getAbsolutePath()); System.setProperty("config.spec", "src/test/resources/allfeatures.def"); MakeConfig.main(new String[]{}); |