You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tapestry.apache.org by dr...@apache.org on 2011/10/19 11:51:11 UTC

svn commit: r1186062 - /tapestry/tapestry5/trunk/tapestry-core/src/test/java/org/apache/tapestry5/internal/TapestryAppInitializerTest.java

Author: drobiazko
Date: Wed Oct 19 09:51:10 2011
New Revision: 1186062

URL: http://svn.apache.org/viewvc?rev=1186062&view=rev
Log:
Fixed broken test. Please always run tests, even for very minor changes.

Modified:
    tapestry/tapestry5/trunk/tapestry-core/src/test/java/org/apache/tapestry5/internal/TapestryAppInitializerTest.java

Modified: tapestry/tapestry5/trunk/tapestry-core/src/test/java/org/apache/tapestry5/internal/TapestryAppInitializerTest.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/test/java/org/apache/tapestry5/internal/TapestryAppInitializerTest.java?rev=1186062&r1=1186061&r2=1186062&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/test/java/org/apache/tapestry5/internal/TapestryAppInitializerTest.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/test/java/org/apache/tapestry5/internal/TapestryAppInitializerTest.java Wed Oct 19 09:51:10 2011
@@ -16,17 +16,22 @@ package org.apache.tapestry5.internal;
 
 import org.apache.tapestry5.ioc.Registry;
 import org.apache.tapestry5.util.Transformer;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 import org.testng.Assert;
 import org.testng.annotations.Test;
 
 @Test
 public class TapestryAppInitializerTest extends Assert
 {
+
+    private Logger logger = LoggerFactory.getLogger(TapestryAppInitializerTest.class);
+
     @SuppressWarnings("unchecked")
     @Test
     public void testLoadAppModule()
     {
-        Registry registry = new TapestryAppInitializer(null, "org.apache.tapestry5.integration.app0",
+        Registry registry = new TapestryAppInitializer(logger, "org.apache.tapestry5.integration.app0",
                                                        "foo").createRegistry();
 
         Transformer<String> s1 = registry.getService("Service1", Transformer.class);
@@ -38,7 +43,7 @@ public class TapestryAppInitializerTest 
     {
         // Apparently just checking to see that it doesn't fail.
 
-        new TapestryAppInitializer(null, "non_existing.package", "foo").createRegistry();
+        new TapestryAppInitializer(logger, "non_existing.package", "foo").createRegistry();
     }
 
 }

Re: svn commit: r1186062 - /tapestry/tapestry5/trunk/tapestry-core/src/test/java/org/apache/tapestry5/internal/TapestryAppInitializerTest.java

Posted by Massimo Lusetti <ml...@gmail.com>.
On Wed, Oct 19, 2011 at 11:51 AM,  <dr...@apache.org> wrote:

> Author: drobiazko
> Date: Wed Oct 19 09:51:10 2011
> New Revision: 1186062
>
> URL: http://svn.apache.org/viewvc?rev=1186062&view=rev
> Log:
> Fixed broken test. Please always run tests, even for very minor changes.

Right... sorry for the noise.

Cheers
-- 
Massimo
http://meridio.blogspot.com

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tapestry.apache.org
For additional commands, e-mail: dev-help@tapestry.apache.org