[jboss-cvs] JBossAS SVN: r71332 - projects/microcontainer/trunk/kernel/src/tests/org/jboss/test/kernel/config/test.

[jboss-cvs-commits at lists.jboss.org]

Author: scott.stark at jboss.org
Date: 2008-03-27 00:48:41 -0400 (Thu, 27 Mar 2008)
New Revision: 71332

Modified:
   projects/microcontainer/trunk/kernel/src/tests/org/jboss/test/kernel/config/test/PropertyReplaceTestCase.java
Log:
Ensure the replacement property is set for all tests

Modified: projects/microcontainer/trunk/kernel/src/tests/org/jboss/test/kernel/config/test/PropertyReplaceTestCase.java
===================================================================
--- projects/microcontainer/trunk/kernel/src/tests/org/jboss/test/kernel/config/test/PropertyReplaceTestCase.java	2008-03-27 04:46:49 UTC (rev 71331)
+++ projects/microcontainer/trunk/kernel/src/tests/org/jboss/test/kernel/config/test/PropertyReplaceTestCase.java	2008-03-27 04:48:41 UTC (rev 71332)
@@ -84,6 +84,7 @@
       SecurityManager sm = suspendSecurity();
       try
       {
+         replaceProperty();
          SimpleAnnotation replAnn = instantiateAnnotation(true);
          assertEquals("test." + CONST + ".Name", replAnn.name());
 
@@ -109,13 +110,18 @@
       return singlePropertyCreator(false);
    }
 
+   private void replaceProperty()
+   {
+      // set property to be replaced
+      System.setProperty(PROP_NAME, CONST);
+      getLog().debug("Set "+PROP_NAME+" to: "+System.getProperty(PROP_NAME));      
+   }
    private void doTestProperty(boolean replace, List<PropertyReplaceTestCase.ObjectCreator> ocs) throws Throwable
    {
       SecurityManager sm = suspendSecurity();
       try
       {
-         // set property to be replaced
-         System.setProperty(PROP_NAME, CONST);
+         replaceProperty();
          for(PropertyReplaceTestCase.ObjectCreator oc : ocs)
          {
             // get property