Bug 345729 change example around to call as opposed to set

git-svn-id: svn+ssh://dev.eclipse.org/svnroot/rt/org.eclipse.jetty/jetty/trunk@3209 7e9141cc-0065-0410-87d8-b60c137991c4

2 files changed, 17 insertions, 13 deletions

diff --git a/jetty-deploy/src/test/java/org/eclipse/jetty/deploy/bindings/GlobalJettyXmlBindingTest.java b/jetty-deploy/src/test/java/org/eclipse/jetty/deploy/bindings/GlobalJettyXmlBindingTest.java
index 26fa919281..ecf5587280 100644
--- a/jetty-deploy/src/test/java/org/eclipse/jetty/deploy/bindings/GlobalJettyXmlBindingTest.java
+++ b/jetty-deploy/src/test/java/org/eclipse/jetty/deploy/bindings/GlobalJettyXmlBindingTest.java
@@ -72,7 +72,7 @@ public class GlobalJettyXmlBindingTest
         
         Assert.assertNotNull(context);
         Assert.assertEquals(context.getDefaultServerClasses().length, context.getServerClasses().length - 1); // added a pattern
-        Assert.assertEquals(context.getDefaultSystemClasses().length,context.getSystemClasses().length + 1); // removed a patter
+        //Assert.assertEquals(context.getDefaultSystemClasses().length,context.getSystemClasses().length + 1); // removed a patter
         
         boolean fooPackage = false;
         
@@ -87,19 +87,19 @@ public class GlobalJettyXmlBindingTest
         
         Assert.assertTrue(fooPackage);
         
-        boolean jndiPackage = false;
+      //  boolean jndiPackage = false;
         
         // this test overrides and we removed the jndi from the list so it
         // should test false
-        for (String entry : context.getSystemClasses())
-        {
-            if ("org.eclipse.jetty.jndi.".equals(entry))
-            {
-                jndiPackage = true;
-            }
-        }
-        
-        Assert.assertFalse(jndiPackage);
+//        for (String entry : context.getSystemClasses())
+//        {
+//            if ("org.eclipse.jetty.jndi.".equals(entry))
+//            {
+//                jndiPackage = true;
+//            }
+//        }
+//        
+//        Assert.assertFalse(jndiPackage);
     }
 }
 
diff --git a/jetty-deploy/src/test/resources/context-binding-test-1.xml b/jetty-deploy/src/test/resources/context-binding-test-1.xml
index 846208e641..925f2578d6 100644
--- a/jetty-deploy/src/test/resources/context-binding-test-1.xml
+++ b/jetty-deploy/src/test/resources/context-binding-test-1.xml
@@ -1,6 +1,10 @@
 <Configure class="org.eclipse.jetty.webapp.WebAppContext">
 
-  <Array id="serverClasses" type="java.lang.String">
+  <Call name="addServerClass">
+    <Arg>org.eclipse.foo.</Arg>
+  </Call>
+
+  <!-- Array id="serverClasses" type="java.lang.String">
     <Item>-org.eclipse.jetty.continuation.</Item>
     <Item>-org.eclipse.jetty.jndi.</Item>
     <Item>-org.eclipse.jetty.plus.jaas.</Item>
@@ -23,5 +27,5 @@
   </Array>
 
   <Set name="serverClasses"><Ref id="serverClasses"/></Set>
-  <Set name="systemClasses"><Ref id="systemClasses"/></Set>
+  <Set name="systemClasses"><Ref id="systemClasses"/></Set-->
 </Configure>