You are viewing a plain text version of this content. The canonical link for it is here.
Posted to log4j-dev@logging.apache.org by ce...@apache.org on 2002/07/26 20:23:35 UTC

cvs commit: jakarta-log4j/tests/witness socketServer.6 socketServer.7 socketServer.8

ceki        2002/07/26 11:23:35

  Modified:    tests    Tag: v1_2-branch build.xml
               tests/src/java/org/apache/log4j/net Tag: v1_2-branch
                        ShortSocketServer.java SocketServerTestCase.java
  Added:       tests/input Tag: v1_2-branch socketServer6.properties
               tests/witness Tag: v1_2-branch socketServer.6 socketServer.7
                        socketServer.8
  Log:
  Added test cases for MDC transmission over the wire and its handling by the receiver.
  
  Revision  Changes    Path
  No                   revision
  
  
  No                   revision
  
  
  1.16.2.4  +1 -1      jakarta-log4j/tests/build.xml
  
  Index: build.xml
  ===================================================================
  RCS file: /home/cvs/jakarta-log4j/tests/build.xml,v
  retrieving revision 1.16.2.3
  retrieving revision 1.16.2.4
  diff -u -r1.16.2.3 -r1.16.2.4
  --- build.xml	7 Jul 2002 06:47:43 -0000	1.16.2.3
  +++ build.xml	26 Jul 2002 18:23:34 -0000	1.16.2.4
  @@ -170,7 +170,7 @@
     <target name="SocketServer" depends="build">
       <parallel>
         <java classname="org.apache.log4j.net.ShortSocketServer" fork="yes">
  -	<arg value="5"/> 
  +	<arg value="8"/> 
   	<arg value="input/socketServer"/> 
   	<classpath refid="tests.classpath"/>
         </java>
  
  
  
  No                   revision
  
  
  No                   revision
  
  
  1.1.2.1   +8 -0      jakarta-log4j/tests/input/Attic/socketServer6.properties
  
  
  
  
  No                   revision
  
  
  No                   revision
  
  
  1.1.2.2   +3 -0      jakarta-log4j/tests/src/java/org/apache/log4j/net/ShortSocketServer.java
  
  Index: ShortSocketServer.java
  ===================================================================
  RCS file: /home/cvs/jakarta-log4j/tests/src/java/org/apache/log4j/net/ShortSocketServer.java,v
  retrieving revision 1.1.2.1
  retrieving revision 1.1.2.2
  diff -u -r1.1.2.1 -r1.1.2.2
  --- ShortSocketServer.java	17 May 2002 11:54:06 -0000	1.1.2.1
  +++ ShortSocketServer.java	26 Jul 2002 18:23:34 -0000	1.1.2.2
  @@ -15,6 +15,7 @@
   import org.apache.log4j.Logger;
   import org.apache.log4j.LogManager;
   import org.apache.log4j.PropertyConfigurator;
  +import org.apache.log4j.MDC;
   import org.apache.log4j.helpers.LogLog;
   import org.apache.log4j.net.SocketNode;
   import org.apache.log4j.net.SocketServer;
  @@ -50,6 +51,8 @@
   
         LogLog.debug("Listening on port " + SocketServerTestCase.PORT);
         ServerSocket serverSocket = new ServerSocket(SocketServerTestCase.PORT);
  +
  +      MDC.put("hostID", "shortSocketServer");
   
         for(int i = 1; i <= totalTests; i++) {
   	PropertyConfigurator.configure(prefix+i+".properties");
  
  
  
  1.2.2.2   +88 -5     jakarta-log4j/tests/src/java/org/apache/log4j/net/SocketServerTestCase.java
  
  Index: SocketServerTestCase.java
  ===================================================================
  RCS file: /home/cvs/jakarta-log4j/tests/src/java/org/apache/log4j/net/SocketServerTestCase.java,v
  retrieving revision 1.2.2.1
  retrieving revision 1.2.2.2
  diff -u -r1.2.2.1 -r1.2.2.2
  --- SocketServerTestCase.java	17 May 2002 11:54:06 -0000	1.2.2.1
  +++ SocketServerTestCase.java	26 Jul 2002 18:23:34 -0000	1.2.2.2
  @@ -58,6 +58,17 @@
     static String PAT5 = "^(DEBUG| INFO| WARN|ERROR|FATAL|LETHAL) some5 T5 MDC-TEST5 \\[main]\\"
                          + " (root|SocketServerTestCase) - Message \\d{1,2}";
   
  +  static String PAT6 = "^(DEBUG| INFO| WARN|ERROR|FATAL|LETHAL) some6 T6 client-test6 MDC-TEST6"
  +                       + " \\[main]\\ (root|SocketServerTestCase) - Message \\d{1,2}";
  +
  +  static String PAT7 = "^(DEBUG| INFO| WARN|ERROR|FATAL|LETHAL) some7 T7 client-test7 MDC-TEST7"
  +                       + " \\[main]\\ (root|SocketServerTestCase) - Message \\d{1,2}";
  +
  +  // DEBUG some8 T8 shortSocketServer MDC-TEST7 [main] SocketServerTestCase - Message 1
  +  static String PAT8 = "^(DEBUG| INFO| WARN|ERROR|FATAL|LETHAL) some8 T8 shortSocketServer"
  +                       + " MDC-TEST8 \\[main]\\ (root|SocketServerTestCase) - Message \\d{1,2}";
  +
  +
   
     static String EXCEPTION1 = "java.lang.Exception: Just testing";
     static String EXCEPTION2 = "\\s*at .*\\(.*:\\d{1,4}\\)";
  @@ -163,11 +174,11 @@
     }
   
     /**
  -   *  The pattern on the server side: %5p %x %X{key1}%X{key5} [%t] %c{1} - %m%n 
  +   * The pattern on the server side: %5p %x %X{key1}%X{key5} [%t] %c{1} - %m%n 
      *
  -   *  The test case uses wraps an AsyncAppender around the
  -   *  SocketAppender. This tests was written specifically for bug
  -   *  report #9155.  
  +   * The test case uses wraps an AsyncAppender around the
  +   * SocketAppender. This tests was written specifically for bug
  +   * report #9155.
   
      * Prior to the bug fix the output on the server did not contain the
      * MDC-TEST5 string because the MDC clone operation (in getMDCCopy
  @@ -192,10 +203,79 @@
   						       EXCEPTION2, EXCEPTION3});
       
       Transformer.transform(TEMP, FILTERED, new Filter[] {cf, new LineNumberFilter()});
  -
       assertTrue(Compare.compare(FILTERED, "witness/socketServer.5"));
     }
   
  +  /**
  +   * The pattern on the server side: %5p %x %X{hostID}${key6} [%t] %c{1} - %m%n 
  +   *
  +   * This test checks whether client-side MDC overrides the server side.
  +   * It uses an AsyncAppender encapsulating a SocketAppender
  +   */
  +  public void test6() throws Exception {
  +    socketAppender = new SocketAppender("localhost", PORT);
  +    socketAppender.setLocationInfo(true);
  +    AsyncAppender asyncAppender = new AsyncAppender();
  +    asyncAppender.setLocationInfo(true);
  +    asyncAppender.addAppender(socketAppender);
  +    rootLogger.addAppender(asyncAppender);
  +
  +    NDC.push("some6");
  +    MDC.put("hostID", "client-test6");
  +    common("T6", "key6", "MDC-TEST6");
  +    NDC.pop();
  +    MDC.remove("hostID");
  +    delay(2);
  +    ControlFilter cf = new ControlFilter(new String[]{PAT6, EXCEPTION1, 
  +						       EXCEPTION2, EXCEPTION3});
  +    
  +    Transformer.transform(TEMP, FILTERED, new Filter[] {cf, new LineNumberFilter()});
  +    assertTrue(Compare.compare(FILTERED, "witness/socketServer.6"));
  +  }
  +
  +  /**
  +   * The pattern on the server side: %5p %x %X{hostID}${key7} [%t] %c{1} - %m%n 
  +   *
  +   * This test checks whether client-side MDC overrides the server side.
  +   */
  +  public void test7() throws Exception {
  +    socketAppender = new SocketAppender("localhost", PORT);
  +    socketAppender.setLocationInfo(true);
  +    rootLogger.addAppender(socketAppender);
  +
  +    NDC.push("some7");
  +    MDC.put("hostID", "client-test7");
  +    common("T7", "key7", "MDC-TEST7");
  +    NDC.pop();
  +    MDC.remove("hostID"); 
  +    delay(2);
  +    ControlFilter cf = new ControlFilter(new String[]{PAT7, EXCEPTION1, 
  +						       EXCEPTION2, EXCEPTION3});
  +    
  +    Transformer.transform(TEMP, FILTERED, new Filter[] {cf, new LineNumberFilter()});
  +    assertTrue(Compare.compare(FILTERED, "witness/socketServer.7"));
  +  }
  +
  +  /**
  +   * The pattern on the server side: %5p %x %X{hostID}${key7} [%t] %c{1} - %m%n 
  +   *
  +   * This test checks whether server side MDC works.
  +   */
  +  public void test8() throws Exception {
  +    socketAppender = new SocketAppender("localhost", PORT);
  +    socketAppender.setLocationInfo(true);
  +    rootLogger.addAppender(socketAppender);
  +
  +    NDC.push("some8");
  +    common("T8", "key8", "MDC-TEST8");
  +    NDC.pop();
  +    delay(2);
  +    ControlFilter cf = new ControlFilter(new String[]{PAT8, EXCEPTION1, 
  +						       EXCEPTION2, EXCEPTION3});
  +    
  +    Transformer.transform(TEMP, FILTERED, new Filter[] {cf, new LineNumberFilter()});
  +    assertTrue(Compare.compare(FILTERED, "witness/socketServer.8"));
  +  }
   
     static 
     void common(String dc, String key, Object o) {
  @@ -230,6 +310,9 @@
       suite.addTest(new SocketServerTestCase("test3"));
       suite.addTest(new SocketServerTestCase("test4"));
       suite.addTest(new SocketServerTestCase("test5"));
  +    suite.addTest(new SocketServerTestCase("test6"));
  +    suite.addTest(new SocketServerTestCase("test7"));
  +    suite.addTest(new SocketServerTestCase("test8"));
       return suite;
     }
   }
  
  
  
  No                   revision
  
  
  No                   revision
  
  
  1.1.2.1   +35 -0     jakarta-log4j/tests/witness/Attic/socketServer.6
  
  
  
  
  1.1.2.1   +35 -0     jakarta-log4j/tests/witness/Attic/socketServer.7
  
  
  
  
  1.1.2.1   +35 -0     jakarta-log4j/tests/witness/Attic/socketServer.8
  
  
  
  

--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>