GEODE-2430: Fix failing tests
Project: http://git-wip-us.apache.org/repos/asf/geode/repo Commit: http://git-wip-us.apache.org/repos/asf/geode/commit/584df772 Tree: http://git-wip-us.apache.org/repos/asf/geode/tree/584df772 Diff: http://git-wip-us.apache.org/repos/asf/geode/diff/584df772 Branch: refs/heads/feature/GEODE-2267 Commit: 584df7722617b5b80e5ca042098325492da57290 Parents: 80f8c26 Author: Jared Stewart <[email protected]> Authored: Tue Feb 7 09:35:10 2017 -0800 Committer: Jinmei Liao <[email protected]> Committed: Tue Feb 7 11:02:52 2017 -0800 ---------------------------------------------------------------------- .../management/internal/configuration/ClusterConfigBaseTest.java | 4 ++-- .../internal/configuration/ClusterConfigImportDUnitTest.java | 3 +-- 2 files changed, 3 insertions(+), 4 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/geode/blob/584df772/geode-core/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigBaseTest.java ---------------------------------------------------------------------- diff --git a/geode-core/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigBaseTest.java b/geode-core/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigBaseTest.java index 3a3c74c..cc0fff2 100644 --- a/geode-core/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigBaseTest.java +++ b/geode-core/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigBaseTest.java @@ -100,12 +100,12 @@ public class ClusterConfigBaseTest extends JUnit4DistributedTestCase { + "<region name=\"regionForGroup2\">\n" + " <region-attributes data-policy=\"replicate\" scope=\"distributed-ack\"/>\n" + " </region>\n" + "</cache>\n"; - writeFile(group2Dir, "group1.xml", group2Xml); + writeFile(group2Dir, "group2.xml", group2Xml); writeFile(group2Dir, "group2.properties", "log-file-size-limit=7000"); createJarFileWithClass("Group2", "group2.jar", group2Dir); - File clusterConfigZip = lsRule.getTempFolder().newFile("cluster_config.zip"); + File clusterConfigZip = new File(lsRule.getTempFolder().newFolder(), "cluster_config.zip"); ZipUtils.zipDirectory(clusterConfigDir.getCanonicalPath(), clusterConfigZip.getCanonicalPath()); FileUtils.deleteDirectory(clusterConfigDir); http://git-wip-us.apache.org/repos/asf/geode/blob/584df772/geode-core/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigImportDUnitTest.java ---------------------------------------------------------------------- diff --git a/geode-core/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigImportDUnitTest.java b/geode-core/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigImportDUnitTest.java index f68015e..f3c471e 100644 --- a/geode-core/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigImportDUnitTest.java +++ b/geode-core/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigImportDUnitTest.java @@ -214,9 +214,8 @@ public class ClusterConfigImportDUnitTest extends ClusterConfigBaseTest { Set<String> expectedZipEntries = new HashSet<>(); for (ConfigGroup group : expectedExportedClusterConfig.getGroups()) { - String groupDir = group.getName() + "/"; + String groupDir = group.getName() + File.separator; - expectedZipEntries.add(groupDir); for (String jarOrXmlOrPropFile : group.getAllFiles()) { expectedZipEntries.add(groupDir + jarOrXmlOrPropFile); }
