You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ranger.apache.org by sp...@apache.org on 2021/05/05 01:43:49 UTC

[ranger] branch master updated: RANGER-3254: Fixed unit test failures that are introduced by previous commit

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

spolavarapu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ranger.git


The following commit(s) were added to refs/heads/master by this push:
     new 775b612  RANGER-3254: Fixed unit test failures that are introduced by previous commit
775b612 is described below

commit 775b612f5939337834a3204a07a365195b802efa
Author: Sailaja Polavarapu <sp...@cloudera.com>
AuthorDate: Tue May 4 18:31:35 2021 -0700

    RANGER-3254: Fixed unit test failures that are introduced by previous commit
---
 .../apache/ranger/unixusersync/process/PolicyMgrUserGroupBuilder.java | 4 ++--
 .../ranger/unixusersync/process/TestFileSourceUserGroupBuilder.java   | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/ugsync/src/main/java/org/apache/ranger/unixusersync/process/PolicyMgrUserGroupBuilder.java b/ugsync/src/main/java/org/apache/ranger/unixusersync/process/PolicyMgrUserGroupBuilder.java
index 6420939..7e9a364 100644
--- a/ugsync/src/main/java/org/apache/ranger/unixusersync/process/PolicyMgrUserGroupBuilder.java
+++ b/ugsync/src/main/java/org/apache/ranger/unixusersync/process/PolicyMgrUserGroupBuilder.java
@@ -1836,8 +1836,8 @@ public class PolicyMgrUserGroupBuilder extends AbstractUserGroupSource implement
 	}
 
 	//Only for testing purpose
-	protected void setUserSyncNameValidationEnabled(boolean isNameValidationEnabled) {
-		config.setProperty(UserGroupSyncConfig.UGSYNC_NAME_VALIDATION_ENABLED, "isNameValidationEnabled");
+	protected void setUserSyncNameValidationEnabled(String isNameValidationEnabled) {
+		config.setProperty(UserGroupSyncConfig.UGSYNC_NAME_VALIDATION_ENABLED, isNameValidationEnabled);
 		this.isUserSyncNameValidationEnabled = config.isUserSyncNameValidationEnabled();
 	}
 }
diff --git a/ugsync/src/test/java/org/apache/ranger/unixusersync/process/TestFileSourceUserGroupBuilder.java b/ugsync/src/test/java/org/apache/ranger/unixusersync/process/TestFileSourceUserGroupBuilder.java
index 2f26677..6b01ba7 100644
--- a/ugsync/src/test/java/org/apache/ranger/unixusersync/process/TestFileSourceUserGroupBuilder.java
+++ b/ugsync/src/test/java/org/apache/ranger/unixusersync/process/TestFileSourceUserGroupBuilder.java
@@ -183,7 +183,7 @@ public class TestFileSourceUserGroupBuilder {
 
         PolicyMgrUserGroupBuilderTest sink = new PolicyMgrUserGroupBuilderTest();
         sink.init();
-        sink.setUserSyncNameValidationEnabled(true);
+        sink.setUserSyncNameValidationEnabled("true");
         fileBuilder.updateSink(sink);
 
         assertEquals(2, sink.getTotalInvalidGroups());