- logback-dev - mailman3.qos.ch

svn commit: r1628 - in logback/trunk/logback-core/src: main/java/ch/qos/logback/core/joran/action main/java/ch/qos/logback/core/util test/input/joran/inclusion test/java/ch/qos/logback/core/joran/action
by noreply.ceki＠qos.ch 04 Mar '08

04 Mar '08

Author: ceki Date: Tue Mar 4 17:38:27 2008 New Revision: 1628 Added: logback/trunk/logback-core/src/test/input/joran/inclusion/multiIncludeByFile.xml logback/trunk/logback-core/src/test/input/joran/inclusion/second.xml logback/trunk/logback-core/src/test/input/joran/inclusion/subByFile.xml logback/trunk/logback-core/src/test/input/joran/inclusion/topByFile.xml - copied, changed from r1622, /logback/trunk/logback-core/src/test/input/joran/inclusion/includeByFile.xml logback/trunk/logback-core/src/test/input/joran/inclusion/topByResource.xml - copied, changed from r1622, /logback/trunk/logback-core/src/test/input/joran/inclusion/includeByResource.xml logback/trunk/logback-core/src/test/input/joran/inclusion/topByUrl.xml - copied, changed from r1622, /logback/trunk/logback-core/src/test/input/joran/inclusion/includeByUrl.xml Removed: logback/trunk/logback-core/src/test/input/joran/inclusion/includeByFile.xml logback/trunk/logback-core/src/test/input/joran/inclusion/includeByResource.xml logback/trunk/logback-core/src/test/input/joran/inclusion/includeByUrl.xml Modified: logback/trunk/logback-core/src/main/java/ch/qos/logback/core/joran/action/IncludeAction.java logback/trunk/logback-core/src/main/java/ch/qos/logback/core/util/OptionHelper.java logback/trunk/logback-core/src/test/java/ch/qos/logback/core/joran/action/IncAction.java logback/trunk/logback-core/src/test/java/ch/qos/logback/core/joran/action/IncludeActionTest.java Log: - fixing bug #129 see also http://bugzilla.qos.ch/show_bug.cgi?id=129\ - refactored the IncludeAction for clarity - IncludeActionTest has been enhanced to check for bug 129 Modified: logback/trunk/logback-core/src/main/java/ch/qos/logback/core/joran/action/IncludeAction.java ============================================================================== --- logback/trunk/logback-core/src/main/java/ch/qos/logback/core/joran/action/IncludeAction.java (original) +++ logback/trunk/logback-core/src/main/java/ch/qos/logback/core/joran/action/IncludeAction.java Tue Mar 4 17:38:27 2008 @@ -14,10 +14,10 @@ import java.io.InputStream; import java.net.MalformedURLException; import java.net.URL; +import java.util.List; import org.xml.sax.Attributes; -import ch.qos.logback.core.joran.action.Action; import ch.qos.logback.core.joran.event.SaxEvent; import ch.qos.logback.core.joran.event.SaxEventRecorder; import ch.qos.logback.core.joran.spi.ActionException; @@ -32,41 +32,27 @@ private static final String FILE_ATTR = "file"; private static final String URL_ATTR = "url"; private static final String RESOURCE_ATTR = "resource"; - private SaxEventRecorder recorder = new SaxEventRecorder();; + + + private String attributeInUse; @Override public void begin(InterpretationContext ec, String name, Attributes attributes) throws ActionException { - String fileAttribute = attributes.getValue(FILE_ATTR); - String urlAttribute = attributes.getValue(URL_ATTR); - String resourceAttribute = attributes.getValue(RESOURCE_ATTR); - String attributeInUse = null; + SaxEventRecorder recorder = new SaxEventRecorder(); + + this.attributeInUse = null; - if(!checkAttributes(fileAttribute, urlAttribute, resourceAttribute)) { + if (!checkAttributes(attributes)) { return; } - InputStream in = null; - - if (!OptionHelper.isEmpty(fileAttribute)) { - attributeInUse = ec.subst(fileAttribute); - in = getInputStreamByFilePath(attributeInUse); - } - - if (!OptionHelper.isEmpty(urlAttribute)) { - attributeInUse = ec.subst(urlAttribute); - in = getInputStreamByUrl(attributeInUse); - } - - if (!OptionHelper.isEmpty(resourceAttribute)) { - attributeInUse = ec.subst(resourceAttribute); - in = getInputStreamByResource(attributeInUse); - } + InputStream in = getInputStream(ec, attributes); try { if (in != null) { - parseAndRecord(in); + parseAndRecord(in, recorder); in.close(); } } catch (JoranException e) { @@ -75,27 +61,17 @@ // called if in.close did not work } - if (recorder.saxEventList.size() == 0) { - return; - } - - // Let's remove the two <included> events before - // adding the events to the player. - SaxEvent first = recorder.saxEventList.get(0); - if (first != null && first.qName.equalsIgnoreCase(INCLUDED_TAG)) { - recorder.saxEventList.remove(0); - } - - SaxEvent last = recorder.saxEventList.get(recorder.saxEventList.size() - 1); - if (last != null && last.qName.equalsIgnoreCase(INCLUDED_TAG)) { - recorder.saxEventList.remove(recorder.saxEventList.size() - 1); - } - + // remove the <included> tag from the beginning and </included> from the end + trimHeadAndTail(recorder); + ec.getJoranInterpreter().addEvents(recorder.saxEventList); } - private boolean checkAttributes(String fileAttribute, - String urlAttribute, String resourceAttribute) { + private boolean checkAttributes(Attributes attributes) { + String fileAttribute = attributes.getValue(FILE_ATTR); + String urlAttribute = attributes.getValue(URL_ATTR); + String resourceAttribute = attributes.getValue(RESOURCE_ATTR); + int count = 0; if (!OptionHelper.isEmpty(fileAttribute)) { @@ -117,7 +93,8 @@ } else if (count == 1) { return true; } - throw new IllegalStateException("Count value ["+count+"] is not expected"); + throw new IllegalStateException("Count value [" + count + + "] is not expected"); } private InputStream getInputStreamByFilePath(String pathToFile) { @@ -163,7 +140,51 @@ return openURL(url); } - private void parseAndRecord(InputStream inputSource) throws JoranException { + InputStream getInputStream(InterpretationContext ec, Attributes attributes) { + String fileAttribute = attributes.getValue(FILE_ATTR); + String urlAttribute = attributes.getValue(URL_ATTR); + String resourceAttribute = attributes.getValue(RESOURCE_ATTR); + + if (!OptionHelper.isEmpty(fileAttribute)) { + attributeInUse = ec.subst(fileAttribute); + return getInputStreamByFilePath(attributeInUse); + } + + if (!OptionHelper.isEmpty(urlAttribute)) { + attributeInUse = ec.subst(urlAttribute); + return getInputStreamByUrl(attributeInUse); + } + + if (!OptionHelper.isEmpty(resourceAttribute)) { + attributeInUse = ec.subst(resourceAttribute); + return getInputStreamByResource(attributeInUse); + } + // given previous checkAttributes() check we cannot reach this line + throw new IllegalStateException("A input stream should have been returned"); + } + + private void trimHeadAndTail(SaxEventRecorder recorder) { + // Let's remove the two <included> events before + // adding the events to the player. + + List<SaxEvent> saxEventList = recorder.saxEventList; + + if (saxEventList.size() == 0) { + return; + } + + SaxEvent first = saxEventList.get(0); + if (first != null && first.qName.equalsIgnoreCase(INCLUDED_TAG)) { + saxEventList.remove(0); + } + + SaxEvent last = saxEventList.get(recorder.saxEventList.size() - 1); + if (last != null && last.qName.equalsIgnoreCase(INCLUDED_TAG)) { + saxEventList.remove(recorder.saxEventList.size() - 1); + } + } + + private void parseAndRecord(InputStream inputSource, SaxEventRecorder recorder) throws JoranException { recorder.setContext(context); recorder.recordEvents(inputSource); } Modified: logback/trunk/logback-core/src/main/java/ch/qos/logback/core/util/OptionHelper.java ============================================================================== --- logback/trunk/logback-core/src/main/java/ch/qos/logback/core/util/OptionHelper.java (original) +++ logback/trunk/logback-core/src/main/java/ch/qos/logback/core/util/OptionHelper.java Tue Mar 4 17:38:27 2008 @@ -90,16 +90,16 @@ * * If no value could be found for the specified key in the context map, then * the system properties are searched, if that fails, then substitution defaults - * to the empty string. + * to appending "_IS_UNDEFINED" to the key name. * * - * For example, if system properties contains no value for the key + * For example, if not the context not the system properties contains no value for the key * "inexistentKey", then the call * * <pre> * String s = OptionConverter.subsVars( * "Value of inexistentKey is [${inexistentKey}]", context);</pre> - * will set <code>s</code> to "Value of inexistentKey is []". + * will set <code>s</code> to "Value of inexistentKey is [inexistentKey_IS_UNDEFINED]". * * * Nevertheless, it is possible to specify a default substitution value using @@ -108,13 +108,14 @@ * <pre> * String s = OptionConverter.subsVars("Value of key is [${key2:-val2}]", context);</pre> * will set <code>s</code> to "Value of key is [val2]" even if the "key2" - * property is unset. + * property is not set. * * * An {@link java.lang.IllegalArgumentException} is thrown if <code>val</code> * contains a start delimeter "${" which is not balanced by a stop delimeter * "}". * + * * @param val * The string on which variable substitution is performed. Added: logback/trunk/logback-core/src/test/input/joran/inclusion/multiIncludeByFile.xml ============================================================================== --- (empty file) +++ logback/trunk/logback-core/src/test/input/joran/inclusion/multiIncludeByFile.xml Tue Mar 4 17:38:27 2008 @@ -0,0 +1,7 @@ +<?xml version="1.0" encoding="UTF-8" ?> +<!DOCTYPE x> + +<x> + <include file="${includeKey}" /> + <include file="${secondFileKey}" /> +</x> Added: logback/trunk/logback-core/src/test/input/joran/inclusion/second.xml ============================================================================== --- (empty file) +++ logback/trunk/logback-core/src/test/input/joran/inclusion/second.xml Tue Mar 4 17:38:27 2008 @@ -0,0 +1,8 @@ +<?xml version="1.0" encoding="UTF-8" ?> +<!DOCTYPE included> + +<included> + + <inc increment="1"/> + +</included> \ No newline at end of file Added: logback/trunk/logback-core/src/test/input/joran/inclusion/subByFile.xml ============================================================================== --- (empty file) +++ logback/trunk/logback-core/src/test/input/joran/inclusion/subByFile.xml Tue Mar 4 17:38:27 2008 @@ -0,0 +1,6 @@ +<?xml version="1.0" encoding="UTF-8" ?> +<!DOCTYPE included> + +<included> + <include file="${includeKey}" /> +</<included> Copied: logback/trunk/logback-core/src/test/input/joran/inclusion/topByFile.xml (from r1622, /logback/trunk/logback-core/src/test/input/joran/inclusion/includeByFile.xml) ============================================================================== --- /logback/trunk/logback-core/src/test/input/joran/inclusion/includeByFile.xml (original) +++ logback/trunk/logback-core/src/test/input/joran/inclusion/topByFile.xml Tue Mar 4 17:38:27 2008 @@ -2,5 +2,5 @@ <!DOCTYPE x> <x> - <include file="${testing}" /> + <include file="${includeKey}" /> </x> Copied: logback/trunk/logback-core/src/test/input/joran/inclusion/topByResource.xml (from r1622, /logback/trunk/logback-core/src/test/input/joran/inclusion/includeByResource.xml) ============================================================================== --- /logback/trunk/logback-core/src/test/input/joran/inclusion/includeByResource.xml (original) +++ logback/trunk/logback-core/src/test/input/joran/inclusion/topByResource.xml Tue Mar 4 17:38:27 2008 @@ -3,6 +3,6 @@ <x> - <include resource="${testing}" /> + <include resource="${includeKey}" /> </x> Copied: logback/trunk/logback-core/src/test/input/joran/inclusion/topByUrl.xml (from r1622, /logback/trunk/logback-core/src/test/input/joran/inclusion/includeByUrl.xml) ============================================================================== --- /logback/trunk/logback-core/src/test/input/joran/inclusion/includeByUrl.xml (original) +++ logback/trunk/logback-core/src/test/input/joran/inclusion/topByUrl.xml Tue Mar 4 17:38:27 2008 @@ -2,5 +2,5 @@ <!DOCTYPE x> <x> - <include url="${testing}" /> + <include url="${includeKey}" /> </x> Modified: logback/trunk/logback-core/src/test/java/ch/qos/logback/core/joran/action/IncAction.java ============================================================================== --- logback/trunk/logback-core/src/test/java/ch/qos/logback/core/joran/action/IncAction.java (original) +++ logback/trunk/logback-core/src/test/java/ch/qos/logback/core/joran/action/IncAction.java Tue Mar 4 17:38:27 2008 @@ -24,6 +24,11 @@ static public int endCount; static public int errorCount; + static public void reset() { + beginCount = 0; + endCount = 0; + errorCount = 0; + } /** * Instantiates an layout of the given class and sets its name. * Modified: logback/trunk/logback-core/src/test/java/ch/qos/logback/core/joran/action/IncludeActionTest.java ============================================================================== --- logback/trunk/logback-core/src/test/java/ch/qos/logback/core/joran/action/IncludeActionTest.java (original) +++ logback/trunk/logback-core/src/test/java/ch/qos/logback/core/joran/action/IncludeActionTest.java Tue Mar 4 17:38:27 2008 @@ -24,20 +24,34 @@ public class IncludeActionTest { - final static String FILE_KEY = "testing"; - + final static String INCLUDE_KEY = "includeKey"; + final static String SUB_FILE_KEY = "subFileKey"; + final static String SECOND_FILE_KEY = "secondFileKey"; + + Context context = new ContextBase(); TrivialConfigurator tc; static final String INCLUSION_DIR_PREFIX = "src/test/input/joran/inclusion/"; - static final String INCLUDE_BY_FILE = INCLUSION_DIR_PREFIX - + "includeByFile.xml"; - static final String INCLUDE_BY_URL = INCLUSION_DIR_PREFIX - + "includeByUrl.xml"; + static final String TOP_BY_FILE = INCLUSION_DIR_PREFIX + + "topByFile.xml"; + + static final String SUB_FILE = INCLUSION_DIR_PREFIX + + "subByFile.xml"; + + static final String MULTI_INCLUDE_BY_FILE = INCLUSION_DIR_PREFIX + + "multiIncludeByFile.xml"; + + static final String SECOND_FILE = INCLUSION_DIR_PREFIX + + "second.xml"; + + + static final String TOP_BY_URL = INCLUSION_DIR_PREFIX + + "topByUrl.xml"; static final String INCLUDE_BY_RESOURCE = INCLUSION_DIR_PREFIX - + "includeByResource.xml"; + + "topByResource.xml"; static final String INCLUDED_FILE = INCLUSION_DIR_PREFIX + "included.xml"; static final String URL_TO_INCLUDE = "file:./" + INCLUDED_FILE; @@ -66,19 +80,20 @@ @After public void tearDown() throws Exception { context = null; - System.clearProperty(FILE_KEY); + System.clearProperty(INCLUDE_KEY); + IncAction.reset(); } @Test public void basicFile() throws JoranException { - System.setProperty(FILE_KEY, INCLUDED_FILE); - tc.doConfigure(INCLUDE_BY_FILE); + System.setProperty(INCLUDE_KEY, INCLUDED_FILE); + tc.doConfigure(TOP_BY_FILE); verifyConfig(2); } @Test public void basicResource() throws JoranException { - System.setProperty(FILE_KEY, INCLUDED_AS_RESOURCE); + System.setProperty(INCLUDE_KEY, INCLUDED_AS_RESOURCE); tc.doConfigure(INCLUDE_BY_RESOURCE); StatusPrinter.print(context); verifyConfig(2); @@ -86,16 +101,16 @@ @Test public void testBasicURL() throws JoranException { - System.setProperty(FILE_KEY, URL_TO_INCLUDE); - tc.doConfigure(INCLUDE_BY_URL); + System.setProperty(INCLUDE_KEY, URL_TO_INCLUDE); + tc.doConfigure(TOP_BY_URL); StatusPrinter.print(context); verifyConfig(2); } @Test public void noFileFound() throws JoranException { - System.setProperty(FILE_KEY, "toto"); - tc.doConfigure(INCLUDE_BY_FILE); + System.setProperty(INCLUDE_KEY, "toto"); + tc.doConfigure(TOP_BY_FILE); assertEquals(Status.ERROR, context.getStatusManager().getLevel()); StatusChecker sc = new StatusChecker(context.getStatusManager()); assertTrue(sc.containsException(FileNotFoundException.class)); @@ -103,8 +118,8 @@ @Test public void withCorruptFile() throws JoranException { - System.setProperty(FILE_KEY, INVALID); - tc.doConfigure(INCLUDE_BY_FILE); + System.setProperty(INCLUDE_KEY, INVALID); + tc.doConfigure(TOP_BY_FILE); assertEquals(Status.ERROR, context.getStatusManager().getLevel()); StatusChecker sc = new StatusChecker(context.getStatusManager()); assertTrue(sc.containsException(SAXParseException.class)); @@ -112,22 +127,42 @@ @Test public void malformedURL() throws JoranException { - System.setProperty(FILE_KEY, "htp://logback.qos.ch"); - tc.doConfigure(INCLUDE_BY_URL); + System.setProperty(INCLUDE_KEY, "htp://logback.qos.ch"); + tc.doConfigure(TOP_BY_URL); assertEquals(Status.ERROR, context.getStatusManager().getLevel()); StatusChecker sc = new StatusChecker(context.getStatusManager()); assertTrue(sc.containsException(MalformedURLException.class)); } @Test - public void testUnknownURL() throws JoranException { - System.setProperty(FILE_KEY, "http://logback2345.qos.ch"); - tc.doConfigure(INCLUDE_BY_URL); + public void unknownURL() throws JoranException { + System.setProperty(INCLUDE_KEY, "http://logback2345.qos.ch"); + tc.doConfigure(TOP_BY_URL); assertEquals(Status.ERROR, context.getStatusManager().getLevel()); StatusChecker sc = new StatusChecker(context.getStatusManager()); assertTrue(sc.containsException(UnknownHostException.class)); } + @Test + public void nestedInclude() throws JoranException { + System.setProperty(SUB_FILE_KEY, INCLUDED_FILE); + System.setProperty(INCLUDE_KEY, SECOND_FILE); + tc.doConfigure(TOP_BY_FILE); + StatusPrinter.print(context); + verifyConfig(1); + + } + + @Test + public void multiInclude() throws JoranException { + System.setProperty(INCLUDE_KEY, INCLUDED_FILE); + System.setProperty(SECOND_FILE_KEY, SECOND_FILE); + tc.doConfigure(MULTI_INCLUDE_BY_FILE); + verifyConfig(3); + } + + + void verifyConfig(int expected) { assertEquals(expected, IncAction.beginCount); assertEquals(expected, IncAction.endCount);

1 0

[Bug 132] New: Logback manual refers to a not existing function in org.slf4j.MDC
by bugzilla-daemon＠pixie.qos.ch 04 Mar '08

04 Mar '08

http://bugzilla.qos.ch/show_bug.cgi?id=132 Summary: Logback manual refers to a not existing function in org.slf4j.MDC Product: logback-site Version: unspecified Platform: All OS/Version: All Status: NEW Severity: normal Priority: P1 Component: Documentation AssignedTo: logback-dev(a)qos.ch ReportedBy: mfranz(a)intershop.de The logback manual chapter 7 says: package org.slf4j; public class MDC { ... //Returns the keys in the MDC as a Set. The returned value can be null. public static Set<String> getKeys(); } But the method getKeys() is not part of the released slf4j API (versions 1.4.3 and 1.5.0). This method exists only within the LogbackMDCAdapter. -- Configure bugmail: http://bugzilla.qos.ch/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee.

1 0

svn commit: r1627 - in logback/trunk/logback-core/src: main/java/ch/qos/logback/core/util test/java/ch/qos/logback/core/util
by noreply.ceki＠qos.ch 04 Mar '08

04 Mar '08

Author: ceki Date: Tue Mar 4 16:48:11 2008 New Revision: 1627 Modified: logback/trunk/logback-core/src/main/java/ch/qos/logback/core/util/OptionHelper.java logback/trunk/logback-core/src/test/java/ch/qos/logback/core/util/OptionHelperTest.java Log: - undefined substitution variables are now marked as _IS_UNDEFINED Modified: logback/trunk/logback-core/src/main/java/ch/qos/logback/core/util/OptionHelper.java ============================================================================== --- logback/trunk/logback-core/src/main/java/ch/qos/logback/core/util/OptionHelper.java (original) +++ logback/trunk/logback-core/src/main/java/ch/qos/logback/core/util/OptionHelper.java Tue Mar 4 16:48:11 2008 @@ -9,8 +9,6 @@ */ package ch.qos.logback.core.util; -import java.util.Map; - import ch.qos.logback.core.Context; import ch.qos.logback.core.CoreGlobal; @@ -71,7 +69,8 @@ final static char DELIM_STOP = '}'; final static int DELIM_START_LEN = 2; final static int DELIM_STOP_LEN = 1; - + final static String _IS_UNDEFINED = "_IS_UNDEFINED"; + /** * Perform variable substitution in string <code>val</code> from the values * of keys found in context property map, and if that fails, then in the @@ -185,6 +184,9 @@ // x2=${x1} String recursiveReplacement = substVars(replacement, context); sbuf.append(recursiveReplacement); + } else { + // if we could not find a replacement, then signal the error + sbuf.append(key+"_IS_UNDEFINED"); } i = k + DELIM_STOP_LEN; Modified: logback/trunk/logback-core/src/test/java/ch/qos/logback/core/util/OptionHelperTest.java ============================================================================== --- logback/trunk/logback-core/src/test/java/ch/qos/logback/core/util/OptionHelperTest.java (original) +++ logback/trunk/logback-core/src/test/java/ch/qos/logback/core/util/OptionHelperTest.java Tue Mar 4 16:48:11 2008 @@ -28,26 +28,33 @@ super.tearDown(); } - public void testSubstVarsNoSubstitution() { - String noSubst = "testing if it works"; - - String result = OptionHelper.substVars(noSubst, null); + public void testLiteral() { + String noSubst = "hello world"; + String result = OptionHelper.substVars(noSubst, context); assertEquals(noSubst, result); } - + + + public void testUndefinedValues() { + String withUndefinedValues = "${axyz}"; + + String result = OptionHelper.substVars(withUndefinedValues, context); + assertEquals("axyz"+OptionHelper._IS_UNDEFINED, result); + } + public void testSubstVarsVariableNotClosed() { String noSubst = "testing if ${v1 works"; try { @SuppressWarnings("unused") - String result = OptionHelper.substVars(noSubst, null); + String result = OptionHelper.substVars(noSubst, context); fail(); } catch (IllegalArgumentException e) { //ok } } - public void testSubstVarsPrimaryOnly() { + public void testSubstVarsContextOnly() { context.putProperty("v1", "if"); context.putProperty("v2", "works");

1 0

svn commit: r1626 - logback/trunk/logback-core/src/test/java/ch/qos/logback/core/joran/action
by noreply.ceki＠qos.ch 04 Mar '08

04 Mar '08

Author: ceki Date: Tue Mar 4 15:21:26 2008 New Revision: 1626 Modified: logback/trunk/logback-core/src/test/java/ch/qos/logback/core/joran/action/IncludeActionTest.java logback/trunk/logback-core/src/test/java/ch/qos/logback/core/joran/action/PackageTest.java Log: Converted IncludeActionTest Junit 4 Modified: logback/trunk/logback-core/src/test/java/ch/qos/logback/core/joran/action/IncludeActionTest.java ============================================================================== --- logback/trunk/logback-core/src/test/java/ch/qos/logback/core/joran/action/IncludeActionTest.java (original) +++ logback/trunk/logback-core/src/test/java/ch/qos/logback/core/joran/action/IncludeActionTest.java Tue Mar 4 15:21:26 2008 @@ -1,12 +1,16 @@ package ch.qos.logback.core.joran.action; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertTrue; + import java.io.FileNotFoundException; import java.net.MalformedURLException; import java.net.UnknownHostException; import java.util.HashMap; -import junit.framework.TestCase; - +import org.junit.After; +import org.junit.Before; +import org.junit.Test; import org.xml.sax.SAXParseException; import ch.qos.logback.core.Context; @@ -18,7 +22,7 @@ import ch.qos.logback.core.status.StatusChecker; import ch.qos.logback.core.util.StatusPrinter; -public class IncludeActionTest extends TestCase { +public class IncludeActionTest { final static String FILE_KEY = "testing"; @@ -42,8 +46,7 @@ static final String INCLUDED_AS_RESOURCE = "input/joran/inclusion/includedAsResource.xml"; - public IncludeActionTest(String arg0) { - super(arg0); + public IncludeActionTest() { HashMap<Pattern, Action> rulesMap = new HashMap<Pattern, Action>(); rulesMap.put(new Pattern("x"), new NOPAction()); rulesMap.put(new Pattern("x/inc"), new IncAction()); @@ -53,34 +56,35 @@ tc.setContext(context); } - @Override - protected void setUp() throws Exception { - super.setUp(); + @Before + public void setUp() throws Exception { IncAction.beginCount = 0; IncAction.errorCount = 0; IncAction.endCount = 0; } - @Override - protected void tearDown() throws Exception { - super.tearDown(); + @After + public void tearDown() throws Exception { context = null; System.clearProperty(FILE_KEY); } - public void testBasicFile() throws JoranException { + @Test + public void basicFile() throws JoranException { System.setProperty(FILE_KEY, INCLUDED_FILE); tc.doConfigure(INCLUDE_BY_FILE); verifyConfig(2); } - public void testBasicResource() throws JoranException { + @Test + public void basicResource() throws JoranException { System.setProperty(FILE_KEY, INCLUDED_AS_RESOURCE); tc.doConfigure(INCLUDE_BY_RESOURCE); StatusPrinter.print(context); verifyConfig(2); } + @Test public void testBasicURL() throws JoranException { System.setProperty(FILE_KEY, URL_TO_INCLUDE); tc.doConfigure(INCLUDE_BY_URL); @@ -88,7 +92,8 @@ verifyConfig(2); } - public void testNoFileFound() throws JoranException { + @Test + public void noFileFound() throws JoranException { System.setProperty(FILE_KEY, "toto"); tc.doConfigure(INCLUDE_BY_FILE); assertEquals(Status.ERROR, context.getStatusManager().getLevel()); @@ -96,7 +101,8 @@ assertTrue(sc.containsException(FileNotFoundException.class)); } - public void testWithCorruptFile() throws JoranException { + @Test + public void withCorruptFile() throws JoranException { System.setProperty(FILE_KEY, INVALID); tc.doConfigure(INCLUDE_BY_FILE); assertEquals(Status.ERROR, context.getStatusManager().getLevel()); @@ -104,7 +110,8 @@ assertTrue(sc.containsException(SAXParseException.class)); } - public void testMalformedURL() throws JoranException { + @Test + public void malformedURL() throws JoranException { System.setProperty(FILE_KEY, "htp://logback.qos.ch"); tc.doConfigure(INCLUDE_BY_URL); assertEquals(Status.ERROR, context.getStatusManager().getLevel()); @@ -112,6 +119,7 @@ assertTrue(sc.containsException(MalformedURLException.class)); } + @Test public void testUnknownURL() throws JoranException { System.setProperty(FILE_KEY, "http://logback2345.qos.ch"); tc.doConfigure(INCLUDE_BY_URL); Modified: logback/trunk/logback-core/src/test/java/ch/qos/logback/core/joran/action/PackageTest.java ============================================================================== --- logback/trunk/logback-core/src/test/java/ch/qos/logback/core/joran/action/PackageTest.java (original) +++ logback/trunk/logback-core/src/test/java/ch/qos/logback/core/joran/action/PackageTest.java Tue Mar 4 15:21:26 2008 @@ -9,6 +9,7 @@ */ package ch.qos.logback.core.joran.action; +import junit.framework.JUnit4TestAdapter; import junit.framework.Test; import junit.framework.TestCase; import junit.framework.TestSuite; @@ -18,7 +19,7 @@ public static Test suite() { TestSuite suite = new TestSuite(); suite.addTestSuite(PropertyActionTest.class); - suite.addTestSuite(IncludeActionTest.class); + suite.addTest(new JUnit4TestAdapter(IncludeActionTest.class)); return suite; }

1 0

svn commit: r1625 - logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/spi
by noreply.ceki＠qos.ch 04 Mar '08

04 Mar '08

Author: ceki Date: Tue Mar 4 15:09:28 2008 New Revision: 1625 Modified: logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/spi/PackageTest.java Log: Adding LoggerComparatorTest to the PackageSuite Modified: logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/spi/PackageTest.java ============================================================================== --- logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/spi/PackageTest.java (original) +++ logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/spi/PackageTest.java Tue Mar 4 15:09:28 2008 @@ -10,6 +10,7 @@ package ch.qos.logback.classic.spi; +import junit.framework.JUnit4TestAdapter; import junit.framework.Test; import junit.framework.TestCase; import junit.framework.TestSuite; @@ -19,7 +20,8 @@ public static Test suite() { TestSuite suite = new TestSuite(); suite.addTestSuite(ContextListenerTest.class); - suite.addTestSuite(CallerDataTest.class); + suite.addTestSuite(CallerDataTest.class); + suite.addTest(new JUnit4TestAdapter (LoggerComparatorTest.class)); return suite; } } \ No newline at end of file

1 0

svn commit: r1624 - in logback/trunk/logback-classic/src: main/java/ch/qos/logback/classic/spi test/java/ch/qos/logback/classic/spi
by noreply.ceki＠qos.ch 04 Mar '08

04 Mar '08

Author: ceki Date: Tue Mar 4 11:51:19 2008 New Revision: 1624 Added: logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/spi/LoggerComparatorTest.java Modified: logback/trunk/logback-classic/src/main/java/ch/qos/logback/classic/spi/LoggerComparator.java Log: - fixing bug #127 See http://bugzilla.qos.ch/show_bug.cgi?id=127 for further details Modified: logback/trunk/logback-classic/src/main/java/ch/qos/logback/classic/spi/LoggerComparator.java ============================================================================== --- logback/trunk/logback-classic/src/main/java/ch/qos/logback/classic/spi/LoggerComparator.java (original) +++ logback/trunk/logback-classic/src/main/java/ch/qos/logback/classic/spi/LoggerComparator.java Tue Mar 4 11:51:19 2008 @@ -8,9 +8,15 @@ public class LoggerComparator implements Comparator<Logger> { public int compare(Logger l1, Logger l2) { + if (l1.getName().equals(l2.getName())) { + return 0; + } if (l1.getName().equals(LoggerContext.ROOT_NAME)) { return -1; } + if (l2.getName().equals(LoggerContext.ROOT_NAME)) { + return 1; + } return l1.getName().compareTo(l2.getName()); } Added: logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/spi/LoggerComparatorTest.java ============================================================================== --- (empty file) +++ logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/spi/LoggerComparatorTest.java Tue Mar 4 11:51:19 2008 @@ -0,0 +1,38 @@ +package ch.qos.logback.classic.spi; + +import org.junit.Before; +import org.junit.Test; +import static org.junit.Assert.*; + +import ch.qos.logback.classic.Logger; +import ch.qos.logback.classic.LoggerContext; + + +public class LoggerComparatorTest { + + LoggerComparator comparator = new LoggerComparator(); + LoggerContext lc = new LoggerContext(); + + Logger root = lc.getLogger("root"); + + Logger a = lc.getLogger("a"); + Logger b = lc.getLogger("b"); + + @Before + public void setUp() throws Exception { + + } + + + + @Test + public void testSmoke() { + assertEquals(0, comparator.compare(a, a)); + assertEquals(-1, comparator.compare(a, b)); + assertEquals(1, comparator.compare(b, a)); + assertEquals(-1, comparator.compare(root, a)); + // following two tests failed before bug #127 was fixed + assertEquals(1, comparator.compare(a, root)); + assertEquals(0, comparator.compare(root, root)); + } +}

1 0

svn commit: r1623 - logback/trunk/logback-classic/src/main/java/ch/qos/logback/classic/pattern
by noreply.ceki＠qos.ch 04 Mar '08

04 Mar '08

Author: ceki Date: Tue Mar 4 10:43:09 2008 New Revision: 1623 Modified: logback/trunk/logback-classic/src/main/java/ch/qos/logback/classic/pattern/ClassNameAbbreviator.java Log: it is slightly better if computeIndexes method is static Modified: logback/trunk/logback-classic/src/main/java/ch/qos/logback/classic/pattern/ClassNameAbbreviator.java ============================================================================== --- logback/trunk/logback-classic/src/main/java/ch/qos/logback/classic/pattern/ClassNameAbbreviator.java (original) +++ logback/trunk/logback-classic/src/main/java/ch/qos/logback/classic/pattern/ClassNameAbbreviator.java Tue Mar 4 10:43:09 2008 @@ -66,7 +66,7 @@ return buf.toString(); } - int computeIndexes(final String className, int[] dotArray) { + static int computeIndexes(final String className, int[] dotArray) { int dotCount = 0; int k = 0; while (true) {

1 0

[Bug 131] New: Application name for a web-app
by bugzilla-daemon＠pixie.qos.ch 02 Mar '08

02 Mar '08

http://bugzilla.qos.ch/show_bug.cgi?id=131 Summary: Application name for a web-app Product: logback-classic Version: unspecified Platform: PC OS/Version: Windows Status: NEW Severity: enhancement Priority: P1 Component: Other AssignedTo: logback-dev(a)qos.ch ReportedBy: noreply.ceki(a)qos.ch Logback should provide support for setting the application name of a web-app (or a JEE application) out of the box. The LoggingContext is a good place for this. See also: http://www.qos.ch/pipermail/logback-user/2008-March/000400.html -- Configure bugmail: http://bugzilla.qos.ch/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee.

1 0

svn commit: r1622 - logback/trunk
by noreply.ceki＠qos.ch 29 Feb '08

29 Feb '08

Author: ceki Date: Fri Feb 29 19:05:49 2008 New Revision: 1622 Modified: logback/trunk/pom.xml Log: - use slf4j 1.5.0 Modified: logback/trunk/pom.xml ============================================================================== --- logback/trunk/pom.xml (original) +++ logback/trunk/pom.xml Fri Feb 29 19:05:49 2008 @@ -36,7 +36,7 @@ <properties>  - <slf4j.version>1.5.0-SNAPSHOT</slf4j.version> + <slf4j.version>1.5.0</slf4j.version> <consolePlugin.version>1.0.1</consolePlugin.version> <retrotranslator.verify>false</retrotranslator.verify> </properties>

1 0

[Bug 130] New: Errors are hard to find
by bugzilla-daemon＠pixie.qos.ch 29 Feb '08

29 Feb '08

http://bugzilla.qos.ch/show_bug.cgi?id=130 Summary: Errors are hard to find Product: logback-access Version: unspecified Platform: PC OS/Version: Windows Status: NEW Severity: major Priority: P1 Component: Other AssignedTo: logback-dev(a)qos.ch ReportedBy: noreply.ceki(a)qos.ch Even small errors are hard to debug since there is no output shown by the requestLog implementations. It would be nice to access the Context statii thourgh a servlet or other means. Ex: an error in the Layout pattern results in no access logs without any other information -- Configure bugmail: http://bugzilla.qos.ch/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee.

1 0