You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tiles.apache.org by ap...@apache.org on 2011/02/02 13:38:06 UTC

svn commit: r1066446 [1/2] - in /tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request: ./ collection/ reflect/ render/ scope/ util/

Author: apetrelli
Date: Wed Feb  2 12:38:04 2011
New Revision: 1066446

URL: http://svn.apache.org/viewvc?rev=1066446&view=rev
Log:
TILESSB-38
Fixed tiles-request-api Checkstyle of test code.

Modified:
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/AbstractClientRequestTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/AbstractRequestTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/AbstractViewRequestTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/NotAvailableFeatureExceptionTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/RequestExceptionTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/AddableParameterMapTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/HeaderValuesCollectionTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/HeaderValuesMapEntrySetTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/HeaderValuesMapTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/IteratorEnumerationTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/KeySetTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/MapEntryArrayValuesTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/MapEntryTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/ReadOnlyEnumerationMapEntrySetTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/ReadOnlyEnumerationMapTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/ReadOnlyEnumerationMapValuesCollectionTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/RemovableKeySetTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/ScopeMapEntrySetTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/ScopeMapTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/reflect/CannotAccessMethodExceptionTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/reflect/CannotInstantiateObjectExceptionTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/reflect/ClassUtilTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/render/BasicRendererFactoryTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/render/ChainedDelegateRendererTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/render/DispatchRendererTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/render/NoSuchRendererExceptionTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/render/StringRendererTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/scope/NoSuchScopeExceptionTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/scope/ReflectionContextResolverTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/util/ApplicationAccessTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/util/ApplicationContextWrapperTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/util/RequestUtilTest.java
    tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/util/RequestWrapperTest.java

Modified: tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/AbstractClientRequestTest.java
URL: http://svn.apache.org/viewvc/tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/AbstractClientRequestTest.java?rev=1066446&r1=1066445&r2=1066446&view=diff
==============================================================================
--- tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/AbstractClientRequestTest.java (original)
+++ tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/AbstractClientRequestTest.java Wed Feb  2 12:38:04 2011
@@ -1,5 +1,22 @@
-/**
+/*
+ * $Id$
+ *
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
  */
 package org.apache.tiles.request;
 
@@ -22,12 +39,24 @@ import org.junit.Test;
  */
 public class AbstractClientRequestTest {
 
+    /**
+     * The request to test.
+     */
     private AbstractClientRequest request;
 
+    /**
+     * The application context.
+     */
     private ApplicationContext applicationContext;
 
+    /**
+     * The application scope.
+     */
     private Map<String, Object> applicationScope;
 
+    /**
+     * The context resolver.
+     */
     private ContextResolver contextResolver;
 
     /**

Modified: tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/AbstractRequestTest.java
URL: http://svn.apache.org/viewvc/tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/AbstractRequestTest.java?rev=1066446&r1=1066445&r2=1066446&view=diff
==============================================================================
--- tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/AbstractRequestTest.java (original)
+++ tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/AbstractRequestTest.java Wed Feb  2 12:38:04 2011
@@ -1,5 +1,22 @@
-/**
+/*
+ * $Id$
+ *
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
  */
 package org.apache.tiles.request;
 

Modified: tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/AbstractViewRequestTest.java
URL: http://svn.apache.org/viewvc/tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/AbstractViewRequestTest.java?rev=1066446&r1=1066445&r2=1066446&view=diff
==============================================================================
--- tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/AbstractViewRequestTest.java (original)
+++ tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/AbstractViewRequestTest.java Wed Feb  2 12:38:04 2011
@@ -1,5 +1,22 @@
-/**
+/*
+ * $Id$
+ *
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
  *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
  */
 package org.apache.tiles.request;
 
@@ -16,20 +33,35 @@ import org.junit.Before;
 import org.junit.Test;
 
 /**
- * @author antonio
+ * Tests {@link AbstractViewRequest}.
  *
  * @version $Rev$ $Date$
  */
 public class AbstractViewRequestTest {
 
+    /**
+     * The request to test.
+     */
     private AbstractViewRequest request;
 
+    /**
+     * The internal request.
+     */
     private Request wrappedRequest;
 
+    /**
+     * The application context.
+     */
     private ApplicationContext applicationContext;
 
+    /**
+     * The context resolver.
+     */
     private ContextResolver contextResolver;
 
+    /**
+     * The application scope.
+     */
     private Map<String, Object> applicationScope;
 
     /**

Modified: tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/NotAvailableFeatureExceptionTest.java
URL: http://svn.apache.org/viewvc/tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/NotAvailableFeatureExceptionTest.java?rev=1066446&r1=1066445&r2=1066446&view=diff
==============================================================================
--- tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/NotAvailableFeatureExceptionTest.java (original)
+++ tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/NotAvailableFeatureExceptionTest.java Wed Feb  2 12:38:04 2011
@@ -43,7 +43,7 @@ public class NotAvailableFeatureExceptio
     }
 
     /**
-     * Test method for {@link org.apache.tiles.request.NotAvailableFeatureException#NotAvailableFeatureException(java.lang.String)}.
+     * Test method for {@link NotAvailableFeatureException#NotAvailableFeatureException(String)}.
      */
     @Test
     public void testNotAvailableFeatureExceptionString() {
@@ -53,7 +53,7 @@ public class NotAvailableFeatureExceptio
     }
 
     /**
-     * Test method for {@link org.apache.tiles.request.NotAvailableFeatureException#NotAvailableFeatureException(java.lang.Throwable)}.
+     * Test method for {@link NotAvailableFeatureException#NotAvailableFeatureException(Throwable)}.
      */
     @Test
     public void testNotAvailableFeatureExceptionThrowable() {
@@ -64,7 +64,7 @@ public class NotAvailableFeatureExceptio
     }
 
     /**
-     * Test method for {@link org.apache.tiles.request.NotAvailableFeatureException#NotAvailableFeatureException(java.lang.String, java.lang.Throwable)}.
+     * Test method for {@link NotAvailableFeatureException#NotAvailableFeatureException(String, Throwable)}.
      */
     @Test
     public void testNotAvailableFeatureExceptionStringThrowable() {

Modified: tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/RequestExceptionTest.java
URL: http://svn.apache.org/viewvc/tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/RequestExceptionTest.java?rev=1066446&r1=1066445&r2=1066446&view=diff
==============================================================================
--- tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/RequestExceptionTest.java (original)
+++ tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/RequestExceptionTest.java Wed Feb  2 12:38:04 2011
@@ -64,7 +64,7 @@ public class RequestExceptionTest {
     }
 
     /**
-     * Test method for {@link org.apache.tiles.request.RequestException#RequestException(java.lang.String, java.lang.Throwable)}.
+     * Test method for {@link RequestException#RequestException(String, Throwable)}.
      */
     @Test
     public void testRequestExceptionStringThrowable() {

Modified: tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/AddableParameterMapTest.java
URL: http://svn.apache.org/viewvc/tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/AddableParameterMapTest.java?rev=1066446&r1=1066445&r2=1066446&view=diff
==============================================================================
--- tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/AddableParameterMapTest.java (original)
+++ tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/AddableParameterMapTest.java Wed Feb  2 12:38:04 2011
@@ -1,5 +1,22 @@
-/**
+/*
+ * $Id$
+ *
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
  *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
  */
 package org.apache.tiles.request.collection;
 
@@ -24,8 +41,14 @@ import org.junit.Test;
  */
 public class AddableParameterMapTest {
 
+    /**
+     * The object to test.
+     */
     private AddableParameterMap map;
 
+    /**
+     * The extractor to use.
+     */
     private HasAddableKeys<String> extractor;
 
     /**
@@ -46,7 +69,7 @@ public class AddableParameterMapTest {
         Set<Map.Entry<String, String>> entrySet = map.entrySet();
         MapEntry<String, String> entry1 = new MapEntry<String, String>("one", "value1", false);
         MapEntry<String, String> entry2 = new MapEntry<String, String>("two", "value2", false);
-        List<Map.Entry<String, String>> entries = new ArrayList<Map.Entry<String,String>>(2);
+        List<Map.Entry<String, String>> entries = new ArrayList<Map.Entry<String, String>>(2);
         entries.add(entry1);
         entries.add(entry2);
 
@@ -60,7 +83,7 @@ public class AddableParameterMapTest {
     }
 
     /**
-     * Test method for {@link org.apache.tiles.request.collection.AddableParameterMap#put(java.lang.String, java.lang.String)}.
+     * Test method for {@link AddableParameterMap#put(String, String)}.
      */
     @Test
     public void testPut() {

Modified: tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/HeaderValuesCollectionTest.java
URL: http://svn.apache.org/viewvc/tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/HeaderValuesCollectionTest.java?rev=1066446&r1=1066445&r2=1066446&view=diff
==============================================================================
--- tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/HeaderValuesCollectionTest.java (original)
+++ tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/HeaderValuesCollectionTest.java Wed Feb  2 12:38:04 2011
@@ -1,3 +1,23 @@
+/*
+ * $Id$
+ *
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
 package org.apache.tiles.request.collection;
 
 import static org.easymock.EasyMock.*;
@@ -22,10 +42,19 @@ import org.junit.Test;
 public class HeaderValuesCollectionTest {
 
 
+    /**
+     * The extractor to use.
+     */
     private EnumeratedValuesExtractor extractor;
 
+    /**
+     * The map to test.
+     */
     private HeaderValuesMap map;
 
+    /**
+     * The collection.
+     */
     private Collection<String[]> coll;
 
     /**
@@ -41,7 +70,7 @@ public class HeaderValuesCollectionTest 
     /**
      * Tests {@link Collection#add(Object)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testAdd() {
         coll.add(null);
     }
@@ -49,7 +78,7 @@ public class HeaderValuesCollectionTest 
     /**
      * Tests {@link Collection#addAll(Object)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testAddAll() {
         coll.addAll(null);
     }
@@ -57,7 +86,7 @@ public class HeaderValuesCollectionTest 
     /**
      * Tests {@link Collection#clear(Object)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testClear() {
         coll.clear();
     }
@@ -253,7 +282,7 @@ public class HeaderValuesCollectionTest 
      * Test method for {@link Collection#iterator()}.
      */
     @SuppressWarnings("unchecked")
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testIteratorRemove() {
         Enumeration<String> keys = createMock(Enumeration.class);
 
@@ -270,23 +299,23 @@ public class HeaderValuesCollectionTest 
     /**
      * Tests {@link Collection#remove(Object)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testRemove() {
         coll.remove(null);
     }
 
     /**
-     * Tests {@link Collection#removeAll(java.util.Collection)}
+     * Tests {@link Collection#removeAll(java.util.Collection)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testRemoveAll() {
         coll.removeAll(null);
     }
 
     /**
-     * Tests {@link Collection#retainAll(java.util.Collection)}
+     * Tests {@link Collection#retainAll(java.util.Collection)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testRetainAll() {
         coll.retainAll(null);
     }

Modified: tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/HeaderValuesMapEntrySetTest.java
URL: http://svn.apache.org/viewvc/tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/HeaderValuesMapEntrySetTest.java?rev=1066446&r1=1066445&r2=1066446&view=diff
==============================================================================
--- tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/HeaderValuesMapEntrySetTest.java (original)
+++ tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/HeaderValuesMapEntrySetTest.java Wed Feb  2 12:38:04 2011
@@ -1,3 +1,23 @@
+/*
+ * $Id$
+ *
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
 package org.apache.tiles.request.collection;
 
 import static org.easymock.EasyMock.*;
@@ -15,13 +35,26 @@ import org.apache.tiles.request.attribut
 import org.junit.Before;
 import org.junit.Test;
 
+/**
+ * Tests {@link HeaderValuesMap entry set}.
+ *
+ */
 public class HeaderValuesMapEntrySetTest {
 
 
+    /**
+     * The extractor to use.
+     */
     private EnumeratedValuesExtractor extractor;
 
+    /**
+     * The map to test.
+     */
     private HeaderValuesMap map;
 
+    /**
+     * The set to test.
+     */
     private Set<Map.Entry<String, String[]>> entrySet;
 
     /**
@@ -37,7 +70,7 @@ public class HeaderValuesMapEntrySetTest
     /**
      * Tests {@link Set#add(Object)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testAdd() {
         entrySet.add(null);
     }
@@ -45,7 +78,7 @@ public class HeaderValuesMapEntrySetTest
     /**
      * Tests {@link Set#addAll(Object)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testAddAll() {
         entrySet.addAll(null);
     }
@@ -53,7 +86,7 @@ public class HeaderValuesMapEntrySetTest
     /**
      * Tests {@link Set#clear(Object)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testClear() {
         entrySet.clear();
     }
@@ -111,7 +144,7 @@ public class HeaderValuesMapEntrySetTest
         expect(values2.hasMoreElements()).andReturn(false);
 
         replay(extractor, values1, values2, entry1, entry2);
-        List<Map.Entry<String, String[]>> coll = new ArrayList<Map.Entry<String,String[]>>();
+        List<Map.Entry<String, String[]>> coll = new ArrayList<Map.Entry<String, String[]>>();
         coll.add(entry1);
         coll.add(entry2);
         assertTrue(entrySet.containsAll(coll));
@@ -135,7 +168,7 @@ public class HeaderValuesMapEntrySetTest
         expect(values1.nextElement()).andReturn("value1");
 
         replay(extractor, values1, entry1);
-        List<Map.Entry<String, String[]>> coll = new ArrayList<Map.Entry<String,String[]>>();
+        List<Map.Entry<String, String[]>> coll = new ArrayList<Map.Entry<String, String[]>>();
         coll.add(entry1);
         assertFalse(entrySet.containsAll(coll));
         verify(extractor, values1, entry1);
@@ -190,7 +223,7 @@ public class HeaderValuesMapEntrySetTest
      * Test method for {@link Set#iterator()}.
      */
     @SuppressWarnings("unchecked")
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testIteratorRemove() {
         Enumeration<String> keys = createMock(Enumeration.class);
 
@@ -207,23 +240,23 @@ public class HeaderValuesMapEntrySetTest
     /**
      * Tests {@link Set#remove(Object)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testRemove() {
         entrySet.remove(null);
     }
 
     /**
-     * Tests {@link Set#removeAll(java.util.Collection)}
+     * Tests {@link Set#removeAll(java.util.Collection)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testRemoveAll() {
         entrySet.removeAll(null);
     }
 
     /**
-     * Tests {@link Set#retainAll(java.util.Collection)}
+     * Tests {@link Set#retainAll(java.util.Collection)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testRetainAll() {
         entrySet.retainAll(null);
     }

Modified: tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/HeaderValuesMapTest.java
URL: http://svn.apache.org/viewvc/tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/HeaderValuesMapTest.java?rev=1066446&r1=1066445&r2=1066446&view=diff
==============================================================================
--- tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/HeaderValuesMapTest.java (original)
+++ tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/HeaderValuesMapTest.java Wed Feb  2 12:38:04 2011
@@ -1,5 +1,22 @@
-/**
+/*
+ * $Id$
+ *
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
  *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
  */
 package org.apache.tiles.request.collection;
 
@@ -21,8 +38,14 @@ import org.junit.Test;
  */
 public class HeaderValuesMapTest {
 
+    /**
+     * The extractor to use.
+     */
     private EnumeratedValuesExtractor extractor;
 
+    /**
+     * The map to test.
+     */
     private HeaderValuesMap map;
 
     /**
@@ -64,8 +87,10 @@ public class HeaderValuesMapTest {
         expect(values2.hasMoreElements()).andReturn(false);
 
         replay(extractor, keys, values1, values2);
-        assertEquals(("one".hashCode() ^ "value1".hashCode()) +
-                ("two".hashCode() ^ ("value2".hashCode() + "value3".hashCode())),
+        assertEquals(
+                ("one".hashCode() ^ "value1".hashCode())
+                        + ("two".hashCode() ^ ("value2".hashCode() + "value3"
+                                .hashCode())),
                 map.hashCode());
         verify(extractor, keys, values1, values2);
     }
@@ -73,7 +98,7 @@ public class HeaderValuesMapTest {
     /**
      * Test method for {@link org.apache.tiles.request.collection.HeaderValuesMap#clear()}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testClear() {
         map.clear();
     }
@@ -274,9 +299,9 @@ public class HeaderValuesMapTest {
     }
 
     /**
-     * Test method for {@link org.apache.tiles.request.collection.HeaderValuesMap#put(java.lang.String, java.lang.String[])}.
+     * Test method for {@link HeaderValuesMap#put(String, String[])}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testPut() {
         map.put("one", new String[] {"value1", "value2"});
     }
@@ -284,7 +309,7 @@ public class HeaderValuesMapTest {
     /**
      * Test method for {@link org.apache.tiles.request.collection.HeaderValuesMap#putAll(java.util.Map)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testPutAll() {
         map.putAll(new HashMap<String, String[]>());
     }
@@ -292,7 +317,7 @@ public class HeaderValuesMapTest {
     /**
      * Test method for {@link org.apache.tiles.request.collection.HeaderValuesMap#remove(java.lang.Object)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testRemove() {
         map.remove("one");
     }

Modified: tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/IteratorEnumerationTest.java
URL: http://svn.apache.org/viewvc/tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/IteratorEnumerationTest.java?rev=1066446&r1=1066445&r2=1066446&view=diff
==============================================================================
--- tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/IteratorEnumerationTest.java (original)
+++ tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/IteratorEnumerationTest.java Wed Feb  2 12:38:04 2011
@@ -1,5 +1,22 @@
-/**
+/*
+ * $Id$
+ *
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
  *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
  */
 package org.apache.tiles.request.collection;
 
@@ -18,8 +35,14 @@ import org.junit.Test;
  */
 public class IteratorEnumerationTest {
 
+    /**
+     * The iterator to use.
+     */
     private Iterator<Integer> iterator;
 
+    /**
+     * The object to test.
+     */
     private IteratorEnumeration<Integer> enumeration;
 
     /**

Modified: tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/KeySetTest.java
URL: http://svn.apache.org/viewvc/tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/KeySetTest.java?rev=1066446&r1=1066445&r2=1066446&view=diff
==============================================================================
--- tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/KeySetTest.java (original)
+++ tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/KeySetTest.java Wed Feb  2 12:38:04 2011
@@ -1,3 +1,23 @@
+/*
+ * $Id$
+ *
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
 package org.apache.tiles.request.collection;
 
 import static org.easymock.EasyMock.*;
@@ -14,11 +34,21 @@ import org.apache.tiles.request.attribut
 import org.junit.Before;
 import org.junit.Test;
 
+/**
+ * Tests {@link KeySet}.
+ *
+ */
 public class KeySetTest {
 
 
+    /**
+     * The extractor to use.
+     */
     private HasKeys<Integer> extractor;
 
+    /**
+     * The key set.
+     */
     private Set<String> entrySet;
 
     /**
@@ -34,7 +64,7 @@ public class KeySetTest {
     /**
      * Tests {@link Set#add(Object)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testAdd() {
         entrySet.add(null);
     }
@@ -42,7 +72,7 @@ public class KeySetTest {
     /**
      * Tests {@link Set#addAll(Object)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testAddAll() {
         entrySet.addAll(null);
     }
@@ -50,7 +80,7 @@ public class KeySetTest {
     /**
      * Tests {@link Set#clear(Object)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testClear() {
         entrySet.clear();
     }
@@ -167,7 +197,7 @@ public class KeySetTest {
      * Test method for {@link Set#iterator()}.
      */
     @SuppressWarnings("unchecked")
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testIteratorRemove() {
         Enumeration<String> keys = createMock(Enumeration.class);
 
@@ -184,23 +214,23 @@ public class KeySetTest {
     /**
      * Tests {@link Set#remove(Object)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testRemove() {
         entrySet.remove(null);
     }
 
     /**
-     * Tests {@link Set#removeAll(java.util.Collection)}
+     * Tests {@link Set#removeAll(java.util.Collection)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testRemoveAll() {
         entrySet.removeAll(null);
     }
 
     /**
-     * Tests {@link Set#retainAll(java.util.Collection)}
+     * Tests {@link Set#retainAll(java.util.Collection)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testRetainAll() {
         entrySet.retainAll(null);
     }

Modified: tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/MapEntryArrayValuesTest.java
URL: http://svn.apache.org/viewvc/tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/MapEntryArrayValuesTest.java?rev=1066446&r1=1066445&r2=1066446&view=diff
==============================================================================
--- tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/MapEntryArrayValuesTest.java (original)
+++ tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/MapEntryArrayValuesTest.java Wed Feb  2 12:38:04 2011
@@ -1,11 +1,27 @@
-/**
+/*
+ * $Id$
+ *
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
  */
 package org.apache.tiles.request.collection;
 
 import static org.junit.Assert.*;
 
-import org.apache.tiles.request.collection.MapEntryArrayValues;
 import org.junit.Test;
 
 /**

Modified: tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/MapEntryTest.java
URL: http://svn.apache.org/viewvc/tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/MapEntryTest.java?rev=1066446&r1=1066445&r2=1066446&view=diff
==============================================================================
--- tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/MapEntryTest.java (original)
+++ tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/MapEntryTest.java Wed Feb  2 12:38:04 2011
@@ -1,11 +1,27 @@
-/**
+/*
+ * $Id$
+ *
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
  */
 package org.apache.tiles.request.collection;
 
 import static org.junit.Assert.*;
 
-import org.apache.tiles.request.collection.MapEntry;
 import org.junit.Test;
 
 /**
@@ -62,7 +78,7 @@ public class MapEntryTest {
     /**
      * Test method for {@link org.apache.tiles.request.collection.MapEntry#setValue(java.lang.Object)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testSetValueException() {
         MapEntry<String, String> entry = new MapEntry<String, String>("key", "value", false);
         assertEquals("value", entry.getValue());

Modified: tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/ReadOnlyEnumerationMapEntrySetTest.java
URL: http://svn.apache.org/viewvc/tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/ReadOnlyEnumerationMapEntrySetTest.java?rev=1066446&r1=1066445&r2=1066446&view=diff
==============================================================================
--- tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/ReadOnlyEnumerationMapEntrySetTest.java (original)
+++ tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/ReadOnlyEnumerationMapEntrySetTest.java Wed Feb  2 12:38:04 2011
@@ -1,3 +1,23 @@
+/*
+ * $Id$
+ *
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
 package org.apache.tiles.request.collection;
 
 import static org.easymock.EasyMock.*;
@@ -16,15 +36,25 @@ import org.junit.Before;
 import org.junit.Test;
 
 /**
- * Tests {@link ReadOnlyEnumerationMap#entrySet()}
+ * Tests {@link ReadOnlyEnumerationMap#entrySet()}.
  *
  * @version $Rev$ $Date$
  */
 public class ReadOnlyEnumerationMapEntrySetTest {
+
+    /**
+     * The extractor to use.
+     */
     private HasKeys<Integer> extractor;
 
+    /**
+     * The map to test.
+     */
     private ReadOnlyEnumerationMap<Integer> map;
 
+    /**
+     * The set to test.
+     */
     private Set<Map.Entry<String, Integer>> entrySet;
 
     /**
@@ -41,7 +71,7 @@ public class ReadOnlyEnumerationMapEntry
     /**
      * Tests {@link Set#add(Object)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testAdd() {
         entrySet.add(null);
     }
@@ -49,7 +79,7 @@ public class ReadOnlyEnumerationMapEntry
     /**
      * Tests {@link Set#addAll(Object)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testAddAll() {
         entrySet.addAll(null);
     }
@@ -57,7 +87,7 @@ public class ReadOnlyEnumerationMapEntry
     /**
      * Tests {@link Set#clear(Object)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testClear() {
         entrySet.clear();
     }
@@ -120,7 +150,7 @@ public class ReadOnlyEnumerationMapEntry
         expect(extractor.getValue("one")).andReturn(1);
 
         replay(extractor, entry1);
-        List<Map.Entry<String, String>> coll = new ArrayList<Map.Entry<String,String>>();
+        List<Map.Entry<String, String>> coll = new ArrayList<Map.Entry<String, String>>();
         coll.add(entry1);
         assertFalse(entrySet.containsAll(coll));
         verify(extractor, entry1);
@@ -169,7 +199,7 @@ public class ReadOnlyEnumerationMapEntry
      * Test method for {@link Set#iterator()}.
      */
     @SuppressWarnings("unchecked")
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testIteratorRemove() {
         Enumeration<String> keys = createMock(Enumeration.class);
 
@@ -186,23 +216,23 @@ public class ReadOnlyEnumerationMapEntry
     /**
      * Tests {@link Set#remove(Object)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testRemove() {
         entrySet.remove(null);
     }
 
     /**
-     * Tests {@link Set#removeAll(java.util.Collection)}
+     * Tests {@link Set#removeAll(java.util.Collection)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testRemoveAll() {
         entrySet.removeAll(null);
     }
 
     /**
-     * Tests {@link Set#retainAll(java.util.Collection)}
+     * Tests {@link Set#retainAll(java.util.Collection)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testRetainAll() {
         entrySet.retainAll(null);
     }

Modified: tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/ReadOnlyEnumerationMapTest.java
URL: http://svn.apache.org/viewvc/tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/ReadOnlyEnumerationMapTest.java?rev=1066446&r1=1066445&r2=1066446&view=diff
==============================================================================
--- tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/ReadOnlyEnumerationMapTest.java (original)
+++ tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/ReadOnlyEnumerationMapTest.java Wed Feb  2 12:38:04 2011
@@ -1,5 +1,22 @@
-/**
+/*
+ * $Id$
+ *
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
  *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
  */
 package org.apache.tiles.request.collection;
 
@@ -21,8 +38,14 @@ import org.junit.Test;
  */
 public class ReadOnlyEnumerationMapTest {
 
+    /**
+     * The extractor to use.
+     */
     private HasKeys<Integer> extractor;
 
+    /**
+     * The map to test.
+     */
     private ReadOnlyEnumerationMap<Integer> map;
 
     /**
@@ -38,7 +61,7 @@ public class ReadOnlyEnumerationMapTest 
     /**
      * Test method for {@link org.apache.tiles.request.collection.ReadOnlyEnumerationMap#clear()}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testClear() {
         map.clear();
     }
@@ -58,7 +81,7 @@ public class ReadOnlyEnumerationMapTest 
     }
 
     /**
-     * Test method for {@link org.apache.tiles.request.collection.ReadOnlyEnumerationMap#containsValue(java.lang.Object)}.
+     * Test method for {@link ReadOnlyEnumerationMap#containsValue(Object)}.
      */
     @SuppressWarnings("unchecked")
     @Test
@@ -80,7 +103,7 @@ public class ReadOnlyEnumerationMapTest 
     }
 
     /**
-     * Test method for {@link org.apache.tiles.request.collection.ReadOnlyEnumerationMap#containsValue(java.lang.Object)}.
+     * Test method for {@link ReadOnlyEnumerationMap#containsValue(Object)}.
      */
     @SuppressWarnings("unchecked")
     @Test
@@ -157,9 +180,9 @@ public class ReadOnlyEnumerationMapTest 
     }
 
     /**
-     * Test method for {@link org.apache.tiles.request.collection.ReadOnlyEnumerationMap#put(java.lang.String, java.lang.String[])}.
+     * Test method for {@link ReadOnlyEnumerationMap#put(String, String[])}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testPut() {
         map.put("one", 1);
     }
@@ -167,7 +190,7 @@ public class ReadOnlyEnumerationMapTest 
     /**
      * Test method for {@link org.apache.tiles.request.collection.ReadOnlyEnumerationMap#putAll(java.util.Map)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testPutAll() {
         map.putAll(new HashMap<String, Integer>());
     }
@@ -175,7 +198,7 @@ public class ReadOnlyEnumerationMapTest 
     /**
      * Test method for {@link org.apache.tiles.request.collection.ReadOnlyEnumerationMap#remove(java.lang.Object)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testRemove() {
         map.remove("one");
     }

Modified: tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/ReadOnlyEnumerationMapValuesCollectionTest.java
URL: http://svn.apache.org/viewvc/tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/ReadOnlyEnumerationMapValuesCollectionTest.java?rev=1066446&r1=1066445&r2=1066446&view=diff
==============================================================================
--- tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/ReadOnlyEnumerationMapValuesCollectionTest.java (original)
+++ tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/ReadOnlyEnumerationMapValuesCollectionTest.java Wed Feb  2 12:38:04 2011
@@ -1,3 +1,23 @@
+/*
+ * $Id$
+ *
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
 package org.apache.tiles.request.collection;
 
 import static org.easymock.EasyMock.*;
@@ -20,10 +40,19 @@ import org.junit.Test;
  * @version $Rev$ $Date$
  */
 public class ReadOnlyEnumerationMapValuesCollectionTest {
+    /**
+     * The extractor to use.
+     */
     private HasKeys<Integer> extractor;
 
+    /**
+     * The map to test.
+     */
     private ReadOnlyEnumerationMap<Integer> map;
 
+    /**
+     * The collection to test.
+     */
     private Collection<Integer> coll;
 
     /**
@@ -40,7 +69,7 @@ public class ReadOnlyEnumerationMapValue
     /**
      * Tests {@link Collection#add(Object)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testAdd() {
         coll.add(null);
     }
@@ -48,7 +77,7 @@ public class ReadOnlyEnumerationMapValue
     /**
      * Tests {@link Collection#addAll(Object)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testAddAll() {
         coll.addAll(null);
     }
@@ -56,7 +85,7 @@ public class ReadOnlyEnumerationMapValue
     /**
      * Tests {@link Collection#clear(Object)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testClear() {
         coll.clear();
     }
@@ -197,7 +226,7 @@ public class ReadOnlyEnumerationMapValue
      * Test method for {@link Collection#iterator()}.
      */
     @SuppressWarnings("unchecked")
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testIteratorRemove() {
         Enumeration<String> keys = createMock(Enumeration.class);
 
@@ -214,23 +243,23 @@ public class ReadOnlyEnumerationMapValue
     /**
      * Tests {@link Collection#remove(Object)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testRemove() {
         coll.remove(null);
     }
 
     /**
-     * Tests {@link Collection#removeAll(java.util.Collection)}
+     * Tests {@link Collection#removeAll(java.util.Collection)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testRemoveAll() {
         coll.removeAll(null);
     }
 
     /**
-     * Tests {@link Collection#retainAll(java.util.Collection)}
+     * Tests {@link Collection#retainAll(java.util.Collection)}.
      */
-    @Test(expected=UnsupportedOperationException.class)
+    @Test(expected = UnsupportedOperationException.class)
     public void testRetainAll() {
         coll.retainAll(null);
     }

Modified: tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/RemovableKeySetTest.java
URL: http://svn.apache.org/viewvc/tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/RemovableKeySetTest.java?rev=1066446&r1=1066445&r2=1066446&view=diff
==============================================================================
--- tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/RemovableKeySetTest.java (original)
+++ tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/RemovableKeySetTest.java Wed Feb  2 12:38:04 2011
@@ -1,5 +1,22 @@
-/**
+/*
+ * $Id$
+ *
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
  *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
  */
 package org.apache.tiles.request.collection;
 
@@ -22,8 +39,14 @@ import org.junit.Test;
  */
 public class RemovableKeySetTest {
 
+    /**
+     * The extractor to use.
+     */
     private HasRemovableKeys<Integer> extractor;
 
+    /**
+     * The key set to test.
+     */
     private RemovableKeySet entrySet;
 
     /**

Modified: tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/ScopeMapEntrySetTest.java
URL: http://svn.apache.org/viewvc/tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/ScopeMapEntrySetTest.java?rev=1066446&r1=1066445&r2=1066446&view=diff
==============================================================================
--- tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/ScopeMapEntrySetTest.java (original)
+++ tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/ScopeMapEntrySetTest.java Wed Feb  2 12:38:04 2011
@@ -1,3 +1,23 @@
+/*
+ * $Id$
+ *
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
 package org.apache.tiles.request.collection;
 
 
@@ -16,16 +36,25 @@ import org.junit.Before;
 import org.junit.Test;
 
 /**
- * Tests {@link ScopeMap#entrySet()}
+ * Tests {@link ScopeMap#entrySet()}.
  *
  * @version $Rev$ $Date$
  */
 public class ScopeMapEntrySetTest {
 
+    /**
+     * The map to test.
+     */
     private ScopeMap map;
 
+    /**
+     * The extractor to use.
+     */
     private AttributeExtractor extractor;
 
+    /**
+     * The entry set to test.
+     */
     private Set<Map.Entry<String, Object>> entrySet;
 
     /**
@@ -39,7 +68,7 @@ public class ScopeMapEntrySetTest {
     }
 
     /**
-     * Tests {@link Set#add(Object)}
+     * Tests {@link Set#add(Object)}.
      */
     @SuppressWarnings("unchecked")
     @Test
@@ -58,7 +87,7 @@ public class ScopeMapEntrySetTest {
     }
 
     /**
-     * Tests {@link Set#add(Object)}
+     * Tests {@link Set#add(Object)}.
      */
     @SuppressWarnings("unchecked")
     @Test
@@ -75,7 +104,7 @@ public class ScopeMapEntrySetTest {
     }
 
     /**
-     * Tests {@link Set#addAll(java.util.Collection)}
+     * Tests {@link Set#addAll(java.util.Collection)}.
      */
     @SuppressWarnings("unchecked")
     @Test
@@ -94,7 +123,7 @@ public class ScopeMapEntrySetTest {
         extractor.setValue("two", 2);
 
         replay(extractor, entry1, entry2);
-        List<Map.Entry<String, Object>> coll = new ArrayList<Map.Entry<String,Object>>();
+        List<Map.Entry<String, Object>> coll = new ArrayList<Map.Entry<String, Object>>();
         coll.add(entry1);
         coll.add(entry2);
         assertTrue(entrySet.addAll(coll));
@@ -125,7 +154,7 @@ public class ScopeMapEntrySetTest {
     }
 
     /**
-     * Tests {@link Set#remove(Object)}
+     * Tests {@link Set#remove(Object)}.
      */
     @SuppressWarnings("unchecked")
     @Test
@@ -143,7 +172,7 @@ public class ScopeMapEntrySetTest {
     }
 
     /**
-     * Tests {@link Set#remove(Object)}
+     * Tests {@link Set#remove(Object)}.
      */
     @SuppressWarnings("unchecked")
     @Test
@@ -159,7 +188,7 @@ public class ScopeMapEntrySetTest {
     }
 
     /**
-     * Tests {@link Set#addAll(java.util.Collection)}
+     * Tests {@link Set#addAll(java.util.Collection)}.
      */
     @SuppressWarnings("unchecked")
     @Test
@@ -177,7 +206,7 @@ public class ScopeMapEntrySetTest {
         extractor.removeValue("two");
 
         replay(extractor, entry1, entry2);
-        List<Map.Entry<String, Object>> coll = new ArrayList<Map.Entry<String,Object>>();
+        List<Map.Entry<String, Object>> coll = new ArrayList<Map.Entry<String, Object>>();
         coll.add(entry1);
         coll.add(entry2);
         assertTrue(entrySet.removeAll(coll));
@@ -185,7 +214,7 @@ public class ScopeMapEntrySetTest {
     }
 
     /**
-     * Tests {@link Set#addAll(java.util.Collection)}
+     * Tests {@link Set#addAll(java.util.Collection)}.
      */
     @SuppressWarnings("unchecked")
     @Test
@@ -211,7 +240,7 @@ public class ScopeMapEntrySetTest {
         extractor.removeValue("three");
 
         replay(extractor, keys);
-        List<Map.Entry<String, Object>> coll = new ArrayList<Map.Entry<String,Object>>();
+        List<Map.Entry<String, Object>> coll = new ArrayList<Map.Entry<String, Object>>();
         coll.add(entry1);
         coll.add(entry2);
         assertTrue(entrySet.retainAll(coll));

Modified: tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/ScopeMapTest.java
URL: http://svn.apache.org/viewvc/tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/ScopeMapTest.java?rev=1066446&r1=1066445&r2=1066446&view=diff
==============================================================================
--- tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/ScopeMapTest.java (original)
+++ tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/collection/ScopeMapTest.java Wed Feb  2 12:38:04 2011
@@ -1,5 +1,22 @@
-/**
+/*
+ * $Id$
+ *
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
  *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
  */
 package org.apache.tiles.request.collection;
 
@@ -22,8 +39,14 @@ import org.junit.Test;
  */
 public class ScopeMapTest {
 
+    /**
+     * The map tot est.
+     */
     private ScopeMap map;
 
+    /**
+     * The extractor to use.
+     */
     private AttributeExtractor extractor;
 
     /**

Modified: tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/reflect/CannotAccessMethodExceptionTest.java
URL: http://svn.apache.org/viewvc/tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/reflect/CannotAccessMethodExceptionTest.java?rev=1066446&r1=1066445&r2=1066446&view=diff
==============================================================================
--- tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/reflect/CannotAccessMethodExceptionTest.java (original)
+++ tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/reflect/CannotAccessMethodExceptionTest.java Wed Feb  2 12:38:04 2011
@@ -23,7 +23,6 @@ package org.apache.tiles.request.reflect
 
 import static org.junit.Assert.*;
 
-import org.apache.tiles.request.reflect.CannotAccessMethodException;
 import org.junit.Test;
 
 /**
@@ -44,7 +43,7 @@ public class CannotAccessMethodException
     }
 
     /**
-     * Test method for {@link org.apache.tiles.CannotAccessMethodException#CannotAccessMethodException(java.lang.String)}.
+     * Test method for {@link CannotAccessMethodException#CannotAccessMethodException(String)}.
      */
     @Test
     public void testCannotAccessMethodExceptionString() {
@@ -54,7 +53,7 @@ public class CannotAccessMethodException
     }
 
     /**
-     * Test method for {@link org.apache.tiles.CannotAccessMethodException#CannotAccessMethodException(java.lang.Throwable)}.
+     * Test method for {@link CannotAccessMethodException#CannotAccessMethodException(Throwable)}.
      */
     @Test
     public void testCannotAccessMethodExceptionThrowable() {
@@ -65,7 +64,7 @@ public class CannotAccessMethodException
     }
 
     /**
-     * Test method for {@link org.apache.tiles.CannotAccessMethodException#CannotAccessMethodException(java.lang.String, java.lang.Throwable)}.
+     * Test method for {@link CannotAccessMethodException#CannotAccessMethodException(String, Throwable)}.
      */
     @Test
     public void testCannotAccessMethodExceptionStringThrowable() {

Modified: tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/reflect/CannotInstantiateObjectExceptionTest.java
URL: http://svn.apache.org/viewvc/tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/reflect/CannotInstantiateObjectExceptionTest.java?rev=1066446&r1=1066445&r2=1066446&view=diff
==============================================================================
--- tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/reflect/CannotInstantiateObjectExceptionTest.java (original)
+++ tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/reflect/CannotInstantiateObjectExceptionTest.java Wed Feb  2 12:38:04 2011
@@ -21,10 +21,8 @@
 
 package org.apache.tiles.request.reflect;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertNull;
+import static org.junit.Assert.*;
 
-import org.apache.tiles.request.reflect.CannotInstantiateObjectException;
 import org.junit.Test;
 
 /**
@@ -45,7 +43,7 @@ public class CannotInstantiateObjectExce
     }
 
     /**
-     * Test method for {@link org.apache.tiles.CannotInstantiateObjectException#CannotInstantiateObjectException(java.lang.String)}.
+     * Test method for {@link CannotInstantiateObjectException#CannotInstantiateObjectException(String)}.
      */
     @Test
     public void testCannotInstantiateObjectExceptionString() {
@@ -55,7 +53,7 @@ public class CannotInstantiateObjectExce
     }
 
     /**
-     * Test method for {@link org.apache.tiles.CannotInstantiateObjectException#CannotInstantiateObjectException(java.lang.Throwable)}.
+     * Test method for {@link CannotInstantiateObjectException#CannotInstantiateObjectException(Throwable)}.
      */
     @Test
     public void testCannotInstantiateObjectExceptionThrowable() {
@@ -66,7 +64,7 @@ public class CannotInstantiateObjectExce
     }
 
     /**
-     * Test method for {@link org.apache.tiles.CannotInstantiateObjectException#CannotInstantiateObjectException(java.lang.String, java.lang.Throwable)}.
+     * Test method for {@link CannotInstantiateObjectException#CannotInstantiateObjectException(String, Throwable)}.
      */
     @Test
     public void testCannotInstantiateObjectExceptionStringThrowable() {

Modified: tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/reflect/ClassUtilTest.java
URL: http://svn.apache.org/viewvc/tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/reflect/ClassUtilTest.java?rev=1066446&r1=1066445&r2=1066446&view=diff
==============================================================================
--- tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/reflect/ClassUtilTest.java (original)
+++ tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/reflect/ClassUtilTest.java Wed Feb  2 12:38:04 2011
@@ -27,8 +27,6 @@ import java.beans.PropertyDescriptor;
 import java.util.HashMap;
 import java.util.Map;
 
-import org.apache.tiles.request.reflect.CannotInstantiateObjectException;
-import org.apache.tiles.request.reflect.ClassUtil;
 import org.junit.Test;
 
 /**
@@ -44,7 +42,7 @@ public class ClassUtilTest {
     private static final int MAP_SIZE = 3;
 
     /**
-     * Test method for {@link org.apache.tiles.request.reflect.ClassUtil#collectBeanInfo(java.lang.Class, java.util.Map)}.
+     * Test method for {@link ClassUtil#collectBeanInfo(Class, Map)}.
      */
     @Test
     public void testCollectBeanInfo() {
@@ -82,14 +80,13 @@ public class ClassUtilTest {
      * Test method for {@link org.apache.tiles.request.reflect.ClassUtil#getClass(String, Class)}.
      * @throws ClassNotFoundException If something goes wrong.
      */
-    @Test(expected=ClassNotFoundException.class)
+    @Test(expected = ClassNotFoundException.class)
     public void testGetClassException() throws ClassNotFoundException {
         ClassUtil.getClass("this.class.does.not.Exist", Object.class);
     }
 
     /**
      * Test method for {@link org.apache.tiles.request.reflect.ClassUtil#instantiate(String, boolean)}.
-     * @throws ClassNotFoundException If something goes wrong.
      */
     @Test
     public void testInstantiate() {
@@ -99,7 +96,6 @@ public class ClassUtilTest {
 
     /**
      * Test method for {@link org.apache.tiles.request.reflect.ClassUtil#instantiate(String, boolean)}.
-     * @throws ClassNotFoundException If something goes wrong.
      */
     @Test
     public void testInstantiateOneParameter() {
@@ -108,27 +104,24 @@ public class ClassUtilTest {
 
     /**
      * Test method for {@link org.apache.tiles.request.reflect.ClassUtil#instantiate(String)}.
-     * @throws ClassNotFoundException If something goes wrong.
      */
-    @Test(expected=CannotInstantiateObjectException.class)
+    @Test(expected = CannotInstantiateObjectException.class)
     public void testInstantiateOneParameterException() {
         assertNotNull(ClassUtil.instantiate("this.class.does.not.Exist"));
     }
 
     /**
      * Test method for {@link org.apache.tiles.request.reflect.ClassUtil#instantiate(String)}.
-     * @throws ClassNotFoundException If something goes wrong.
      */
-    @Test(expected=CannotInstantiateObjectException.class)
+    @Test(expected = CannotInstantiateObjectException.class)
     public void testInstantiateInstantiationException() {
         ClassUtil.instantiate(TestInterface.class.getName());
     }
 
     /**
      * Test method for {@link org.apache.tiles.request.reflect.ClassUtil#instantiate(String)}.
-     * @throws ClassNotFoundException If something goes wrong.
      */
-    @Test(expected=CannotInstantiateObjectException.class)
+    @Test(expected = CannotInstantiateObjectException.class)
     public void testInstantiateIllegalAccessException() {
         ClassUtil.instantiate(TestPrivateClass.class.getName());
     }
@@ -169,9 +162,15 @@ public class ClassUtilTest {
         void setValue3(String value3);
     }
 
+    /**
+     * A test static class.
+     */
     public static class TestClass {
     }
 
+    /**
+     * A test static private class.
+     */
     private static class TestPrivateClass {
     }
 }

Modified: tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/render/BasicRendererFactoryTest.java
URL: http://svn.apache.org/viewvc/tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/render/BasicRendererFactoryTest.java?rev=1066446&r1=1066445&r2=1066446&view=diff
==============================================================================
--- tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/render/BasicRendererFactoryTest.java (original)
+++ tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/render/BasicRendererFactoryTest.java Wed Feb  2 12:38:04 2011
@@ -82,7 +82,7 @@ public class BasicRendererFactoryTest {
      * Tests execution and
      * {@link BasicRendererFactory#getRenderer(String)}.
      */
-    @Test(expected=NoSuchRendererException.class)
+    @Test(expected = NoSuchRendererException.class)
     public void testGetRendererException() {
         Renderer renderer1 = createMock(Renderer.class);
         Renderer renderer2 = createMock(Renderer.class);

Modified: tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/render/ChainedDelegateRendererTest.java
URL: http://svn.apache.org/viewvc/tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/render/ChainedDelegateRendererTest.java?rev=1066446&r1=1066445&r2=1066446&view=diff
==============================================================================
--- tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/render/ChainedDelegateRendererTest.java (original)
+++ tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/render/ChainedDelegateRendererTest.java Wed Feb  2 12:38:04 2011
@@ -26,9 +26,6 @@ import java.io.IOException;
 import java.io.StringWriter;
 
 import org.apache.tiles.request.Request;
-import org.apache.tiles.request.render.CannotRenderException;
-import org.apache.tiles.request.render.ChainedDelegateRenderer;
-import org.apache.tiles.request.render.TypeDetectingRenderer;
 import org.easymock.EasyMock;
 import org.junit.Before;
 import org.junit.Test;
@@ -76,7 +73,7 @@ public class ChainedDelegateRendererTest
 
     /**
      * Tests
-     * {@link ChainedDelegateRenderer#render(Attribute, Request)}
+     * {@link ChainedDelegateRenderer#render(String, Request)}
      * writing a definition.
      *
      * @throws IOException If something goes wrong during rendition.
@@ -100,12 +97,12 @@ public class ChainedDelegateRendererTest
 
     /**
      * Tests
-     * {@link ChainedDelegateRenderer#render(Attribute, Request)}
+     * {@link ChainedDelegateRenderer#render(String, Request)}
      * writing a definition.
      *
      * @throws IOException If something goes wrong during rendition.
      */
-    @Test(expected=NullPointerException.class)
+    @Test(expected = NullPointerException.class)
     public void testWriteNull() throws IOException {
         StringWriter writer = new StringWriter();
         Request requestContext = EasyMock
@@ -124,12 +121,12 @@ public class ChainedDelegateRendererTest
 
     /**
      * Tests
-     * {@link ChainedDelegateRenderer#render(Attribute, Request)}
+     * {@link ChainedDelegateRenderer#render(String, Request)}
      * writing a definition.
      *
      * @throws IOException If something goes wrong during rendition.
      */
-    @Test(expected=CannotRenderException.class)
+    @Test(expected = CannotRenderException.class)
     public void testWriteNotRenderable() throws IOException {
         StringWriter writer = new StringWriter();
         Request requestContext = EasyMock
@@ -141,7 +138,7 @@ public class ChainedDelegateRendererTest
         expect(
                 templateRenderer.isRenderable("Result",
                         requestContext)).andReturn(Boolean.FALSE);
-        expect(stringRenderer.isRenderable("Result",requestContext))
+        expect(stringRenderer.isRenderable("Result", requestContext))
                 .andReturn(Boolean.FALSE);
 
         replay(requestContext, stringRenderer, templateRenderer,
@@ -157,7 +154,7 @@ public class ChainedDelegateRendererTest
 
     /**
      * Tests
-     * {@link ChainedDelegateRenderer#render(Attribute, Request)}
+     * {@link ChainedDelegateRenderer#render(String, Request)}
      * writing a string.
      *
      * @throws IOException If something goes wrong during rendition.
@@ -186,7 +183,7 @@ public class ChainedDelegateRendererTest
 
     /**
      * Tests
-     * {@link ChainedDelegateRenderer#render(Attribute, Request)}
+     * {@link ChainedDelegateRenderer#render(String, Request)}
      * writing a template.
      *
      * @throws IOException If something goes wrong during rendition.

Modified: tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/render/DispatchRendererTest.java
URL: http://svn.apache.org/viewvc/tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/render/DispatchRendererTest.java?rev=1066446&r1=1066445&r2=1066446&view=diff
==============================================================================
--- tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/render/DispatchRendererTest.java (original)
+++ tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/render/DispatchRendererTest.java Wed Feb  2 12:38:04 2011
@@ -68,7 +68,7 @@ public class DispatchRendererTest {
      *
      * @throws IOException If something goes wrong during rendition.
      */
-    @Test(expected=CannotRenderException.class)
+    @Test(expected = CannotRenderException.class)
     public void testWriteNull() throws IOException {
         Request requestContext = createMock(Request.class);
         replay(requestContext);
@@ -79,8 +79,6 @@ public class DispatchRendererTest {
     /**
      * Tests
      * {@link DispatchRenderer#isRenderable(String, Request)}.
-     *
-     * @throws IOException If something goes wrong during rendition.
      */
     @Test
     public void testIsRenderable() {

Modified: tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/render/NoSuchRendererExceptionTest.java
URL: http://svn.apache.org/viewvc/tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/render/NoSuchRendererExceptionTest.java?rev=1066446&r1=1066445&r2=1066446&view=diff
==============================================================================
--- tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/render/NoSuchRendererExceptionTest.java (original)
+++ tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/render/NoSuchRendererExceptionTest.java Wed Feb  2 12:38:04 2011
@@ -23,7 +23,6 @@ package org.apache.tiles.request.render;
 
 import static org.junit.Assert.*;
 
-import org.apache.tiles.request.render.NoSuchRendererException;
 import org.junit.Test;
 
 /**

Modified: tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/render/StringRendererTest.java
URL: http://svn.apache.org/viewvc/tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/render/StringRendererTest.java?rev=1066446&r1=1066445&r2=1066446&view=diff
==============================================================================
--- tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/render/StringRendererTest.java (original)
+++ tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/render/StringRendererTest.java Wed Feb  2 12:38:04 2011
@@ -27,7 +27,6 @@ import java.io.IOException;
 import java.io.StringWriter;
 
 import org.apache.tiles.request.Request;
-import org.apache.tiles.request.render.StringRenderer;
 import org.junit.Before;
 import org.junit.Test;
 
@@ -70,8 +69,6 @@ public class StringRendererTest {
     /**
      * Tests
      * {@link StringRenderer#isRenderable(String, Request)}.
-     *
-     * @throws IOException If something goes wrong.
      */
     @Test
     public void testIsRenderable() {

Modified: tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/scope/NoSuchScopeExceptionTest.java
URL: http://svn.apache.org/viewvc/tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/scope/NoSuchScopeExceptionTest.java?rev=1066446&r1=1066445&r2=1066446&view=diff
==============================================================================
--- tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/scope/NoSuchScopeExceptionTest.java (original)
+++ tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/scope/NoSuchScopeExceptionTest.java Wed Feb  2 12:38:04 2011
@@ -43,7 +43,7 @@ public class NoSuchScopeExceptionTest {
     }
 
     /**
-     * Test method for {@link org.apache.tiles.request.scope.NoSuchScopeException#NoSuchScopeException(java.lang.String)}.
+     * Test method for {@link NoSuchScopeException#NoSuchScopeException(String)}.
      */
     @Test
     public void testNoSuchScopeExceptionString() {
@@ -53,7 +53,7 @@ public class NoSuchScopeExceptionTest {
     }
 
     /**
-     * Test method for {@link org.apache.tiles.request.scope.NoSuchScopeException#NoSuchScopeException(java.lang.Throwable)}.
+     * Test method for {@link NoSuchScopeException#NoSuchScopeException(Throwable)}.
      */
     @Test
     public void testNoSuchScopeExceptionThrowable() {
@@ -64,7 +64,7 @@ public class NoSuchScopeExceptionTest {
     }
 
     /**
-     * Test method for {@link org.apache.tiles.request.scope.NoSuchScopeException#NoSuchScopeException(java.lang.String, java.lang.Throwable)}.
+     * Test method for {@link NoSuchScopeException#NoSuchScopeException(String, Throwable)}.
      */
     @Test
     public void testNoSuchScopeExceptionStringThrowable() {

Modified: tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/scope/ReflectionContextResolverTest.java
URL: http://svn.apache.org/viewvc/tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/scope/ReflectionContextResolverTest.java?rev=1066446&r1=1066445&r2=1066446&view=diff
==============================================================================
--- tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/scope/ReflectionContextResolverTest.java (original)
+++ tiles/sandbox/trunk/tiles-request/tiles-request-api/src/test/java/org/apache/tiles/request/scope/ReflectionContextResolverTest.java Wed Feb  2 12:38:04 2011
@@ -1,5 +1,22 @@
-/**
+/*
+ * $Id$
+ *
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
  */
 package org.apache.tiles.request.scope;
 
@@ -19,20 +36,38 @@ import org.junit.Before;
 import org.junit.Test;
 
 /**
- * Tests {@link ReflectionContextResolver}
+ * Tests {@link ReflectionContextResolver}.
  *
  * @version $Rev$ $Date$
  */
 public class ReflectionContextResolverTest {
 
+    /**
+     * The scopes.
+     */
     private static final String [] SCOPES = new String[] {"one", "two", "three"};
 
+    /**
+     * The resolver to test.
+     */
     private ReflectionContextResolver resolver;
 
+    /**
+     * One scope.
+     */
     private Map<String, Object> oneScope;
+    /**
+     * Two scope.
+     */
     private Map<String, Object> twoScope;
+    /**
+     * Three scope.
+     */
     private Map<String, Object> threeScope;
 
+    /**
+     * The request.
+     */
     private Request request;
 
     /**
@@ -49,7 +84,7 @@ public class ReflectionContextResolverTe
     }
 
     /**
-     * Test method for {@link org.apache.tiles.request.scope.ReflectionContextResolver#getContext(org.apache.tiles.request.Request, java.lang.String)}.
+     * Test method for {@link ReflectionContextResolver#getContext(org.apache.tiles.request.Request, java.lang.String)}.
      */
     @Test
     public void testGetContext() {
@@ -61,7 +96,7 @@ public class ReflectionContextResolverTe
     }
 
     /**
-     * Test method for {@link org.apache.tiles.request.scope.ReflectionContextResolver#getContext(org.apache.tiles.request.Request, java.lang.String)}.
+     * Test method for {@link ReflectionContextResolver#getContext(org.apache.tiles.request.Request, java.lang.String)}.
      */
     @Test
     public void testGetContextWrapped() {
@@ -74,31 +109,31 @@ public class ReflectionContextResolverTe
     }
 
     /**
-     * Test method for {@link org.apache.tiles.request.scope.ReflectionContextResolver#getContext(org.apache.tiles.request.Request, java.lang.String)}.
+     * Test method for {@link ReflectionContextResolver#getContext(org.apache.tiles.request.Request, java.lang.String)}.
      */
-    @Test(expected=NoSuchScopeException.class)
+    @Test(expected = NoSuchScopeException.class)
     public void testGetContextException() {
         resolver.getContext(request, "none");
     }
 
     /**
-     * Test method for {@link org.apache.tiles.request.scope.ReflectionContextResolver#getContext(org.apache.tiles.request.Request, java.lang.String)}.
+     * Test method for {@link ReflectionContextResolver#getContext(org.apache.tiles.request.Request, java.lang.String)}.
      */
-    @Test(expected=NoSuchScopeException.class)
+    @Test(expected = NoSuchScopeException.class)
     public void testGetContextException2() {
         resolver.getContext(request, "private");
     }
 
     /**
-     * Test method for {@link org.apache.tiles.request.scope.ReflectionContextResolver#getContext(org.apache.tiles.request.Request, java.lang.String)}.
+     * Test method for {@link ReflectionContextResolver#getContext(org.apache.tiles.request.Request, java.lang.String)}.
      */
-    @Test(expected=NoSuchScopeException.class)
+    @Test(expected = NoSuchScopeException.class)
     public void testGetContextException3() {
         resolver.getContext(request, "unavailable");
     }
 
     /**
-     * Test method for {@link org.apache.tiles.request.scope.ReflectionContextResolver#getAvailableScopes(org.apache.tiles.request.Request)}.
+     * Test method for {@link ReflectionContextResolver#getAvailableScopes(org.apache.tiles.request.Request)}.
      */
     @Test
     public void testGetAvailableScopes() {
@@ -108,7 +143,7 @@ public class ReflectionContextResolverTe
     }
 
     /**
-     * Test method for {@link org.apache.tiles.request.scope.ReflectionContextResolver#getAvailableScopes(org.apache.tiles.request.Request)}.
+     * Test method for {@link ReflectionContextResolver#getAvailableScopes(org.apache.tiles.request.Request)}.
      */
     @Test
     public void testGetAvailableScopesWrapped() {
@@ -118,12 +153,28 @@ public class ReflectionContextResolverTe
         verify(oneScope, twoScope, threeScope);
     }
 
+    /**
+     * A sample request.
+     */
     public static class SampleRequest implements Request {
 
+        /**
+         * The scopes.
+         */
         private static final String [] SCOPES = new String[] {"one", "two", "three"};
 
+        /**
+         * The scope maps.
+         */
         private Map<String, Object> oneScope, twoScope, threeScope;
 
+        /**
+         * Constructor.
+         *
+         * @param oneScope Scope one.
+         * @param twoScope Scope two.
+         * @param threeScope Scope three.
+         */
         public SampleRequest(Map<String, Object> oneScope,
                 Map<String, Object> twoScope, Map<String, Object> threeScope) {
             this.oneScope = oneScope;
@@ -141,23 +192,48 @@ public class ReflectionContextResolverTe
             return SCOPES;
         }
 
+        /**
+         * Returns one scope.
+         *
+         * @return One scope.
+         */
         public Map<String, Object> getOneScope() {
             return oneScope;
         }
 
+        /**
+         * Returns two scope.
+         *
+         * @return Two scope.
+         */
         public Map<String, Object> getTwoScope() {
             return twoScope;
         }
 
+        /**
+         * Returns three scope.
+         *
+         * @return Three scope.
+         */
         public Map<String, Object> getThreeScope() {
             return threeScope;
         }
 
+        /**
+         * Returns a private scope.
+         *
+         * @return A private, unused, scope.
+         */
         @SuppressWarnings("unused")
         private Map<String, Object> getPrivateScope() {
             return null;
         }
 
+        /**
+         * Returns an unavailable scope.
+         *
+         * @return The unavailable scope, raising exception.
+         */
         public Map<String, Object> getUnavailableScope() {
             throw new UnsupportedOperationException("No way!");
         }