Author: ceki
Date: Tue Oct 28 17:04:09 2008
New Revision: 1894

Modified:
   
logback/trunk/logback-core/src/test/java/ch/qos/logback/core/joran/action/PropertyActionTest.java

Log:
- small improvements to test case

Modified: 
logback/trunk/logback-core/src/test/java/ch/qos/logback/core/joran/action/PropertyActionTest.java
==============================================================================
--- 
logback/trunk/logback-core/src/test/java/ch/qos/logback/core/joran/action/PropertyActionTest.java
   (original)
+++ 
logback/trunk/logback-core/src/test/java/ch/qos/logback/core/joran/action/PropertyActionTest.java
   Tue Oct 28 17:04:09 2008
@@ -14,6 +14,7 @@
 import ch.qos.logback.core.joran.spi.InterpretationContext;
 import ch.qos.logback.core.status.ErrorStatus;
 import ch.qos.logback.core.util.Constants;
+import ch.qos.logback.core.util.StatusPrinter;
 
 public class PropertyActionTest  {
 
@@ -38,7 +39,7 @@
   }
   
   @Test
-  public void testBegin() {
+  public void nameValuePair() {
     atts.setValue("name", "v1");
     atts.setValue("value", "work");
     spAction.begin(ec, null, atts);
@@ -46,7 +47,16 @@
   }
   
   @Test
-  public void testBeginNoValue() {
+  public void nameValuePairWithPrerequisiteSubsitution() {
+    context.putProperty("w", "wor");
+    atts.setValue("name", "v1");
+    atts.setValue("value", "${w}k");
+    spAction.begin(ec, null, atts);
+    assertEquals("work", ec.getSubstitutionProperty("v1"));
+  }
+  
+  @Test
+  public void noValue() {
     atts.setValue("name", "v1");
     spAction.begin(ec, null, atts);
     assertEquals(1, context.getStatusManager().getCount());
@@ -54,7 +64,7 @@
   }
 
   @Test
-  public void testBeginNoName() {
+  public void noName() {
     atts.setValue("value", "v1");
     spAction.begin(ec, null, atts);
     assertEquals(1, context.getStatusManager().getCount());
@@ -62,10 +72,11 @@
   }
   
   @Test
-  public void testBeginNothing() {
+  public void noAttributes() {
     spAction.begin(ec, null, atts);
     assertEquals(1, context.getStatusManager().getCount());
     assertTrue(checkError());
+    StatusPrinter.print(context);
   } 
   
   @Test
_______________________________________________
logback-dev mailing list
[email protected]
http://qos.ch/mailman/listinfo/logback-dev

Reply via email to