You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@falcon.apache.org by ro...@apache.org on 2014/09/14 13:44:30 UTC

[3/3] git commit: FALCON-723 Fix ProcessInstanceResumeTest and ProcessInstanceSuspendTest as per FALCON-166. Contributed by Ruslan Ostafiychuk

FALCON-723 Fix ProcessInstanceResumeTest and ProcessInstanceSuspendTest as per FALCON-166. Contributed by Ruslan Ostafiychuk


Project: http://git-wip-us.apache.org/repos/asf/incubator-falcon/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-falcon/commit/4ed6d855
Tree: http://git-wip-us.apache.org/repos/asf/incubator-falcon/tree/4ed6d855
Diff: http://git-wip-us.apache.org/repos/asf/incubator-falcon/diff/4ed6d855

Branch: refs/heads/master
Commit: 4ed6d855eac460068ff0246aaf1be93a8f0e081f
Parents: a617305
Author: Ruslan Ostafiychuk <ro...@apache.org>
Authored: Sun Sep 14 14:42:28 2014 +0300
Committer: Ruslan Ostafiychuk <ro...@apache.org>
Committed: Sun Sep 14 14:42:28 2014 +0300

----------------------------------------------------------------------
 falcon-regression/CHANGES.txt                                    | 2 ++
 .../org/apache/falcon/regression/ProcessInstanceResumeTest.java  | 4 ++--
 .../org/apache/falcon/regression/ProcessInstanceSuspendTest.java | 4 ++--
 3 files changed, 6 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-falcon/blob/4ed6d855/falcon-regression/CHANGES.txt
----------------------------------------------------------------------
diff --git a/falcon-regression/CHANGES.txt b/falcon-regression/CHANGES.txt
index 5fe65c2..c88af6d 100644
--- a/falcon-regression/CHANGES.txt
+++ b/falcon-regression/CHANGES.txt
@@ -64,6 +64,8 @@ Trunk (Unreleased)
    FALCON-681 delete duplicate feed retention test from falcon regression (SamarthG)
 
   BUG FIXES
+   FALCON-723 Fix ProcessInstanceResumeTest and ProcessInstanceSuspendTest as per FALCON-166 (Ruslan Ostafiychuk)
+
    FALCON-715 "end" parameter should be greater than last instances startTime (Ruslan Ostafiychuk)
 
    FALCON-716 PrismFeedSuspendTest tests retagged as distributed (Ruslan Ostafiychuk)

http://git-wip-us.apache.org/repos/asf/incubator-falcon/blob/4ed6d855/falcon-regression/merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceResumeTest.java
----------------------------------------------------------------------
diff --git a/falcon-regression/merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceResumeTest.java b/falcon-regression/merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceResumeTest.java
index 2e774c2..a4c9619 100644
--- a/falcon-regression/merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceResumeTest.java
+++ b/falcon-regression/merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceResumeTest.java
@@ -115,7 +115,7 @@ public class ProcessInstanceResumeTest extends BaseTestClass {
             wholeRange);
         InstanceUtil.validateResponse(r, 6, 2, 4, 0, 0);
         r = prism.getProcessHelper().getProcessInstanceResume(processName, "?end=2010-01-02T01:15Z");
-        InstanceUtil.validateSuccessWithStatusCode(r, ResponseKeys.UNPARSEABLE_DATE);
+        InstanceUtil.validateSuccessWithStatusCode(r, 0);
     }
 
     /**
@@ -205,7 +205,7 @@ public class ProcessInstanceResumeTest extends BaseTestClass {
     public void testProcessInstanceResumeNoParams() throws Exception {
         bundles[0].submitFeedsScheduleProcess(prism);
         InstancesResult r = prism.getProcessHelper().getProcessInstanceResume(processName, null);
-        InstanceUtil.validateSuccessWithStatusCode(r, ResponseKeys.UNPARSEABLE_DATE);
+        InstanceUtil.validateSuccessWithStatusCode(r, 0);
     }
 
     /**

http://git-wip-us.apache.org/repos/asf/incubator-falcon/blob/4ed6d855/falcon-regression/merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceSuspendTest.java
----------------------------------------------------------------------
diff --git a/falcon-regression/merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceSuspendTest.java b/falcon-regression/merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceSuspendTest.java
index b0739e6..9497a49 100644
--- a/falcon-regression/merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceSuspendTest.java
+++ b/falcon-regression/merlin/src/test/java/org/apache/falcon/regression/ProcessInstanceSuspendTest.java
@@ -121,7 +121,7 @@ public class ProcessInstanceSuspendTest extends BaseTestClass {
                 "?start=2010-01-02T00:00Z&end=2010-01-02T01:30Z");
         result = prism.getProcessHelper().getProcessInstanceStatus(processName,
                 "?start=2010-01-02T00:00Z&end=2010-01-02T01:30Z");
-        InstanceUtil.validateSuccessWithStatusCode(result, 400);
+        InstanceUtil.validateSuccessWithStatusCode(result, 0);
     }
 
     /**
@@ -180,7 +180,7 @@ public class ProcessInstanceSuspendTest extends BaseTestClass {
         AssertUtil.checkStatus(clusterOC, EntityType.PROCESS, bundles[0].getProcessData(),
             Job.Status.RUNNING);
         InstancesResult r = prism.getProcessHelper().getProcessInstanceSuspend(processName, null);
-        InstanceUtil.validateSuccessWithStatusCode(r, ResponseKeys.UNPARSEABLE_DATE);
+        InstanceUtil.validateSuccessWithStatusCode(r, 0);
     }
 
     /**