Repository: falcon
Updated Branches:
  refs/heads/master 0692b2415 -> ee5213caf


FALCON-1018 Enable and fix FalconClientTest#badClusterSubmit. Contributed by 
Paul Isaychuk


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

Branch: refs/heads/master
Commit: ee5213caf4fd04850ecd318086d679928244d373
Parents: 0692b24
Author: Ruslan Ostafiychuk <[email protected]>
Authored: Mon Feb 9 15:30:33 2015 +0200
Committer: Ruslan Ostafiychuk <[email protected]>
Committed: Mon Feb 9 15:30:33 2015 +0200

----------------------------------------------------------------------
 falcon-regression/CHANGES.txt                                  | 2 ++
 .../apache/falcon/regression/security/FalconClientTest.java    | 6 ++++--
 2 files changed, 6 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/falcon/blob/ee5213ca/falcon-regression/CHANGES.txt
----------------------------------------------------------------------
diff --git a/falcon-regression/CHANGES.txt b/falcon-regression/CHANGES.txt
index 404920f..d571441 100644
--- a/falcon-regression/CHANGES.txt
+++ b/falcon-regression/CHANGES.txt
@@ -215,6 +215,8 @@ Trunk (Unreleased)
    FALCON-681 delete duplicate feed retention test from falcon regression 
(SamarthG)
 
   BUG FIXES
+   FALCON-1018 Enable and fix FalconClientTest#badClusterSubmit (Paul Isaychuk 
via Ruslan Ostafiychuk)
+
    FALCON-1005 In ProcessInstanceSuspendTest clean test dir after each method 
execution
    (Paul Isaychuk via Ruslan Ostafiychuk)
 

http://git-wip-us.apache.org/repos/asf/falcon/blob/ee5213ca/falcon-regression/merlin/src/test/java/org/apache/falcon/regression/security/FalconClientTest.java
----------------------------------------------------------------------
diff --git 
a/falcon-regression/merlin/src/test/java/org/apache/falcon/regression/security/FalconClientTest.java
 
b/falcon-regression/merlin/src/test/java/org/apache/falcon/regression/security/FalconClientTest.java
index 514ca55..fec9d83 100644
--- 
a/falcon-regression/merlin/src/test/java/org/apache/falcon/regression/security/FalconClientTest.java
+++ 
b/falcon-regression/merlin/src/test/java/org/apache/falcon/regression/security/FalconClientTest.java
@@ -64,13 +64,15 @@ public class FalconClientTest extends BaseTestClass {
      * Test error thrown by falcon client, when acl of the submitted cluster 
has bad values.
      * @throws Exception
      */
-    @Test (enabled = false)
+    @Test (enabled = true)
     public void badClusterSubmit() throws Exception {
         bundles[0].setCLusterACL(MerlinConstants.DIFFERENT_USER_NAME,
             MerlinConstants.CURRENT_USER_GROUP, "*");
         final String clusterXml = bundles[0].getClusters().get(0);
         final ExecResult execResult = 
prism.getClusterHelper().clientSubmit(clusterXml);
-        AssertUtil.assertFailed(execResult, "cluster submission failed");
+        AssertUtil.assertFailed(execResult, String.format(
+            "Invalid acl owner %s, does not exist or does not belong to group: 
%s",
+            MerlinConstants.DIFFERENT_USER_NAME, 
MerlinConstants.CURRENT_USER_GROUP));
     }
 
     /**

Reply via email to