You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by je...@apache.org on 2018/08/01 16:51:05 UTC

[geode] branch develop updated: GEODE-5497: Fix broken tests

This is an automated email from the ASF dual-hosted git repository.

jensdeppe pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git


The following commit(s) were added to refs/heads/develop by this push:
     new a7bf514  GEODE-5497: Fix broken tests
a7bf514 is described below

commit a7bf51429cc6ddb19da62fc1c90f09f242b9b018
Author: Jens Deppe <jd...@pivotal.io>
AuthorDate: Wed Aug 1 09:50:48 2018 -0700

    GEODE-5497: Fix broken tests
---
 .../geode/internal/cache/backup/BackupInspectorIntegrationTest.java  | 2 +-
 .../apache/geode/internal/cache/backup/UnixScriptGeneratorTest.java  | 5 +++--
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/geode-core/src/integrationTest/java/org/apache/geode/internal/cache/backup/BackupInspectorIntegrationTest.java b/geode-core/src/integrationTest/java/org/apache/geode/internal/cache/backup/BackupInspectorIntegrationTest.java
index 313567a..3f66f20 100644
--- a/geode-core/src/integrationTest/java/org/apache/geode/internal/cache/backup/BackupInspectorIntegrationTest.java
+++ b/geode-core/src/integrationTest/java/org/apache/geode/internal/cache/backup/BackupInspectorIntegrationTest.java
@@ -90,7 +90,7 @@ public class BackupInspectorIntegrationTest {
     assertThat(inspector.isIncremental()).isTrue();
     Set<String> oplogFiles = inspector.getIncrementalOplogFileNames();
     assertThat(oplogFiles.isEmpty()).isFalse();
-    assertThat(oplogFiles).hasSize(2);
+    assertThat(oplogFiles).hasSize(3);
     assertThat(oplogFiles.contains(CRF_FILE_NAME)).isTrue();
     assertThat(oplogFiles.contains(DRF_FILE_NAME)).isTrue();
     validateIncrementalBackupScript(inspector);
diff --git a/geode-core/src/integrationTest/java/org/apache/geode/internal/cache/backup/UnixScriptGeneratorTest.java b/geode-core/src/integrationTest/java/org/apache/geode/internal/cache/backup/UnixScriptGeneratorTest.java
index a3536e5..e6c4b57 100644
--- a/geode-core/src/integrationTest/java/org/apache/geode/internal/cache/backup/UnixScriptGeneratorTest.java
+++ b/geode-core/src/integrationTest/java/org/apache/geode/internal/cache/backup/UnixScriptGeneratorTest.java
@@ -83,8 +83,9 @@ public class UnixScriptGeneratorTest {
     scriptGenerator.writeCopyFile(writer, source, destinaiton);
     writer.flush();
     List<String> output = Files.readAllLines(outputFile.toPath());
-    assertThat(output).hasSize(1);
-    assertThat(output).containsExactly("cp -p '" + source + "' '" + destinaiton + "'");
+    assertThat(output).hasSize(2);
+    assertThat(output).containsExactly("mkdir -p '" + source.getParent() + "'",
+        "cp -p '" + source + "' '" + destinaiton + "'");
   }
 
   @Test