mooli tayer has uploaded a new change for review. Change subject: aaa: remove exceptional AdminDomain logic from domain sorting. ......................................................................
aaa: remove exceptional AdminDomain logic from domain sorting. Change-Id: I48ff93e74960c082bdd2dce1b15193272946d503 Topic: AAA Signed-off-by: Mooli Tayer <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetDomainListQuery.java D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/LoginDomainComparator.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/GetDomainListQueryTest.java D backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/LoginDomainComparatorTest.java 4 files changed, 5 insertions(+), 137 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/91/29991/1 diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetDomainListQuery.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetDomainListQuery.java index ccc1f19..ae16c7c 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetDomainListQuery.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetDomainListQuery.java @@ -7,23 +7,21 @@ import org.ovirt.engine.api.extensions.aaa.Authz; import org.ovirt.engine.core.aaa.AuthzUtils; -import org.ovirt.engine.core.common.config.Config; -import org.ovirt.engine.core.common.config.ConfigValues; import org.ovirt.engine.core.common.queries.GetDomainListParameters; import org.ovirt.engine.core.extensions.mgr.ExtensionProxy; import org.ovirt.engine.core.extensions.mgr.ExtensionsManager; import org.ovirt.engine.core.utils.extensionsmgr.EngineExtensionsManager; public class GetDomainListQuery<P extends GetDomainListParameters> extends QueriesCommandBase<P> { + + public static final String INTERNAL = "internal"; + public GetDomainListQuery(P parameters) { super(parameters); } @Override protected void executeQueryCommand() { - // Get the name of the internal domain: - String internal = Config.<String> getValue(ConfigValues.AdminDomain); - // Get the list of authentication profile names: List<ExtensionProxy> extensions = getExtensionsManager().getExtensionsByService(Authz.class.getName()); @@ -32,11 +30,9 @@ names.add(AuthzUtils.getName(extension)); } if (getParameters().getFilterInternalDomain()) { - names.remove(internal); + names.remove(INTERNAL); } - - // Sort it so that the internal profile is always the last: - sort(names, new LoginDomainComparator(internal)); + sort(names); // Return the sorted list: getQueryReturnValue().setReturnValue(names); diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/LoginDomainComparator.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/LoginDomainComparator.java deleted file mode 100644 index a7a2a1e..0000000 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/LoginDomainComparator.java +++ /dev/null @@ -1,30 +0,0 @@ -package org.ovirt.engine.core.bll; - -import java.io.Serializable; -import java.util.Comparator; - -/** - * Used to sort the login domain list on the login page. Business rule is sort alphabetically, - * but put "internal" at the end. - */ -public class LoginDomainComparator implements Comparator<String>, Serializable { - - private static final long serialVersionUID = -1679928871675563671L; - - private String internal; - - public LoginDomainComparator(String internal) { - this.internal = internal; - } - - @Override - public int compare(String a, String b) { - if (a == b) return 0; - if (a == null) return -1; - if (b == null) return 1; - if (a.equals(internal)) return 1; - if (b.equals(internal)) return -1; - return a.compareTo(b); - } - -} diff --git a/backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/GetDomainListQueryTest.java b/backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/GetDomainListQueryTest.java index ea63a7b..99dd921 100644 --- a/backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/GetDomainListQueryTest.java +++ b/backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/GetDomainListQueryTest.java @@ -3,7 +3,6 @@ import static org.junit.Assert.assertTrue; import static org.mockito.Mockito.doReturn; import static org.mockito.Mockito.mock; -import static org.ovirt.engine.core.utils.MockConfigRule.mockConfig; import java.util.ArrayList; import java.util.Arrays; @@ -12,15 +11,12 @@ import org.apache.commons.collections.CollectionUtils; import org.junit.Before; -import org.junit.ClassRule; import org.junit.Test; import org.ovirt.engine.api.extensions.Base; import org.ovirt.engine.api.extensions.ExtMap; import org.ovirt.engine.api.extensions.aaa.Authz; -import org.ovirt.engine.core.common.config.ConfigValues; import org.ovirt.engine.core.common.queries.GetDomainListParameters; import org.ovirt.engine.core.extensions.mgr.ExtensionProxy; -import org.ovirt.engine.core.utils.MockConfigRule; import org.ovirt.engine.core.utils.extensionsmgr.EngineExtensionsManager; /** @@ -58,11 +54,6 @@ }; private MockEngineExtensionsManager extMgr = new MockEngineExtensionsManager(); - - @ClassRule - public static final MockConfigRule MCR = new MockConfigRule( - mockConfig(ConfigValues.AdminDomain, INTERNAL) - ); @Before @Override diff --git a/backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/LoginDomainComparatorTest.java b/backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/LoginDomainComparatorTest.java deleted file mode 100644 index 6cdf81b..0000000 --- a/backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/LoginDomainComparatorTest.java +++ /dev/null @@ -1,89 +0,0 @@ -package org.ovirt.engine.core.bll; - -import static org.junit.Assert.assertEquals; - -import java.util.ArrayList; -import java.util.Collections; -import java.util.List; - -import org.junit.Test; -import org.junit.runner.RunWith; -import org.mockito.runners.MockitoJUnitRunner; - -@RunWith(MockitoJUnitRunner.class) -public class LoginDomainComparatorTest { - - @Test - public void testCompareWithoutInternal() { - List<String> list = new ArrayList<String>(); - list.add("a1"); - list.add("b1"); - list.add("c1"); - list.add("d1"); - list.add("a2"); - Collections.sort(list, new LoginDomainComparator("internal")); - assertEquals("a1", list.get(0)); - assertEquals("a2", list.get(1)); - assertEquals("b1", list.get(2)); - assertEquals("c1", list.get(3)); - assertEquals("d1", list.get(4)); - } - - @Test - public void testCompareWithInternal() { - List<String> list = new ArrayList<String>(); - list.add("a1"); - list.add("b1"); - list.add("internal"); - list.add("c1"); - list.add("d1"); - list.add("a2"); - Collections.sort(list, new LoginDomainComparator("internal")); - assertEquals("a1", list.get(0)); - assertEquals("a2", list.get(1)); - assertEquals("b1", list.get(2)); - assertEquals("c1", list.get(3)); - assertEquals("d1", list.get(4)); - assertEquals("internal", list.get(5)); - } - - @Test - public void testCompareWithTwoInternal() { - List<String> list = new ArrayList<String>(); - list.add("a1"); - list.add("b1"); - list.add("internal"); - list.add("c1"); - list.add("d1"); - list.add("internal"); - list.add("a2"); - Collections.sort(list, new LoginDomainComparator("internal")); - assertEquals("a1", list.get(0)); - assertEquals("a2", list.get(1)); - assertEquals("b1", list.get(2)); - assertEquals("c1", list.get(3)); - assertEquals("d1", list.get(4)); - assertEquals("internal", list.get(5)); - assertEquals("internal", list.get(6)); - } - - @Test - public void testCompareWithNulls() { - List<String> list = new ArrayList<String>(); - list.add(null); - list.add(null); - list.add("internal"); - list.add("c1"); - list.add("d1"); - list.add("internal"); - list.add("a2"); - Collections.sort(list, new LoginDomainComparator("internal")); - assertEquals(null, list.get(0)); - assertEquals(null, list.get(1)); - assertEquals("a2", list.get(2)); - assertEquals("c1", list.get(3)); - assertEquals("d1", list.get(4)); - assertEquals("internal", list.get(5)); - assertEquals("internal", list.get(6)); - } -} -- To view, visit http://gerrit.ovirt.org/29991 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I48ff93e74960c082bdd2dce1b15193272946d503 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: mooli tayer <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
