diff --git a/src/test/java/org/jboss/ejb/client/test/byteman/MixedModeServiceURLTestCase.java b/src/test/java/org/jboss/ejb/client/test/byteman/MixedModeServiceURLTestCase.java index ccafd9f73..86afae0cf 100644 --- a/src/test/java/org/jboss/ejb/client/test/byteman/MixedModeServiceURLTestCase.java +++ b/src/test/java/org/jboss/ejb/client/test/byteman/MixedModeServiceURLTestCase.java @@ -23,7 +23,6 @@ import java.util.HashSet; import java.util.List; import java.util.Map; -import java.util.Set; import org.jboss.byteman.contrib.bmunit.BMRule; import org.jboss.byteman.contrib.bmunit.BMRules; @@ -219,8 +218,8 @@ private void validateResults(Map> results) { } } } - Assert.assertEquals("ServiceURLs do not match for node1", actual_node1_urls, EXPECTED_NODE1_URLS); - Assert.assertEquals("ServiceURLs do not match for node2", actual_node2_urls, EXPECTED_NODE2_URLS); + Assert.assertEquals("ServiceURLs do not match for node1", EXPECTED_NODE1_URLS, actual_node1_urls); + Assert.assertEquals("ServiceURLs do not match for node2", EXPECTED_NODE2_URLS, actual_node2_urls); } /** diff --git a/src/test/java/org/jboss/ejb/client/test/byteman/MixedModeTestHelper.java b/src/test/java/org/jboss/ejb/client/test/byteman/MixedModeTestHelper.java index 311f4c3d1..09084a39c 100644 --- a/src/test/java/org/jboss/ejb/client/test/byteman/MixedModeTestHelper.java +++ b/src/test/java/org/jboss/ejb/client/test/byteman/MixedModeTestHelper.java @@ -6,11 +6,14 @@ import org.jboss.byteman.rule.Rule; import org.jboss.byteman.rule.helper.Helper; +import org.jboss.ejb.client.EJBClientContext; +import org.wildfly.discovery.FilterSpec; import org.wildfly.discovery.ServiceURL; public class MixedModeTestHelper extends Helper { private static final String NODE_LIST_MAP_NAME = "nodeListMap"; + private static final FilterSpec EJB_MODULE_FILTER_SPEC = FilterSpec.hasAttribute(EJBClientContext.FILTER_ATTR_EJB_MODULE); public MixedModeTestHelper(Rule rule) { super(rule); @@ -21,6 +24,22 @@ public void createNodeListMap() { } public void addServiceURLCacheToMap(String node, List list) { + if (list.size() < 4) { + System.out.printf("Ignoring partial ServiceURLs: %s%n", list); + return; + } + boolean hasEjbModule = true; + for (ServiceURL u : list) { + if (!u.satisfies(EJB_MODULE_FILTER_SPEC)) { + hasEjbModule = false; + break; + } + } + if (!hasEjbModule) { + System.out.printf("Ignoring invalid temp ServiceURLs: %s%n", list); + return; + } + System.out.println("** Adding serviceURL to map: node = " + node + ", list = " + list); List oldList = (List) link(NODE_LIST_MAP_NAME, node, list); if (oldList != null) {