You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@creadur.apache.org by rd...@apache.org on 2013/12/06 19:15:00 UTC

svn commit: r1548633 - /creadur/rat/branches/gsoc/apache-rat-core/src/test/java/org/apache/rat/analysis/license/CopyrightHeaderTest.java

Author: rdonkin
Date: Fri Dec  6 18:15:00 2013
New Revision: 1548633

URL: http://svn.apache.org/r1548633
Log:
Apply PMD Rules.

Modified:
    creadur/rat/branches/gsoc/apache-rat-core/src/test/java/org/apache/rat/analysis/license/CopyrightHeaderTest.java

Modified: creadur/rat/branches/gsoc/apache-rat-core/src/test/java/org/apache/rat/analysis/license/CopyrightHeaderTest.java
URL: http://svn.apache.org/viewvc/creadur/rat/branches/gsoc/apache-rat-core/src/test/java/org/apache/rat/analysis/license/CopyrightHeaderTest.java?rev=1548633&r1=1548632&r2=1548633&view=diff
==============================================================================
--- creadur/rat/branches/gsoc/apache-rat-core/src/test/java/org/apache/rat/analysis/license/CopyrightHeaderTest.java (original)
+++ creadur/rat/branches/gsoc/apache-rat-core/src/test/java/org/apache/rat/analysis/license/CopyrightHeaderTest.java Fri Dec  6 18:15:00 2013
@@ -39,8 +39,8 @@ public class CopyrightHeaderTest {
 			"/*  Copyright 123a*/", "/*  Copyright 123f oobar*/",
 			"/*  Copyright 2013FooBar*/", "/*  Copyright 2012 2013 FooBar.*/" };
 
-	CopyrightHeader header;
-	Document subject = new MockLocation("subject");
+	private CopyrightHeader header;
+	private Document subject = new MockLocation("subject");
 
 	@Before
 	public void setUp() throws Exception {
@@ -52,7 +52,7 @@ public class CopyrightHeaderTest {
 	}
 
 	@Test
-	public void match() throws Exception {
+	public void match() {
 		for (String line : MATCHING_HEADERS) {
 			assertTrue("Copyright Header should be matched",
 					header.match(subject, line));
@@ -64,7 +64,7 @@ public class CopyrightHeaderTest {
 	}
 
 	@Test
-	public void noMatch() throws Exception {
+	public void noMatch() {
 		for (String line : NON_MATCHING_HEADERS) {
 			assertFalse("Copyright Header shouldn't be matched",
 					header.match(subject, line));