issue #52: removed PowerMock agents as they are not needed and break badly in JDK8

Author: Luis Miranda

README.md

@@ -36,11 +36,4 @@ Select a Tomcat Server in order to deploy the server.
 
 ### Unit Tests
 
-Known issues while testing:
-
- * [ECLEmma coverage plugin incompatible with PowerMock](http://stackoverflow.com/questions/23363212/powermock-eclemma-coverage-issue)
-  
- * When using @PrepareForTest and PowerMockito the breakpoints not does [not stop when debugging](http://stackoverflow.com/questions/35140575/powermockito-junit-and-eclemma-debugging-dosent-work). The solution is define Rules.
-
- * Java7+ enforces bytecode verification and with Powermock you can experience a [java.lang.VerifyError](http://www.notonlyanecmplace.com/java-7-enforces-bytecode-verification/).
-   At Java8 bytecode verification is mandatory and there is no option to disable it :(
+The unit tests use PowerMock but do not run with the PowerMock javaagents enabled. This means you will not be able to mock out static methods, but other PowerMock features will work.

core/pom.xml

@@ -30,18 +30,6 @@
 			<version>1.6.2</version>
 			<scope>test</scope>
 		</dependency>
-		<dependency>
-			<groupId>org.powermock</groupId>
-			<artifactId>powermock-module-javaagent</artifactId>
-			<version>1.6.0</version>
-			<scope>test</scope>
-		</dependency>
-		<dependency>
-			<groupId>org.powermock</groupId>
-			<artifactId>powermock-module-junit4-rule-agent</artifactId>
-			<version>1.6.4</version>
-			<scope>test</scope>
-		</dependency>
 
 		<!-- for log -->
 		<dependency>
@@ -111,7 +99,7 @@
 				<artifactId>maven-surefire-plugin</artifactId>
 				<version>2.12</version>
 				<configuration>
-					<argLine>-XX:-UseSplitVerifier ${jacoco.agent.argLine}</argLine>
+					<argLine>${jacoco.agent.argLine}</argLine>
 				</configuration>
 			</plugin>
 		</plugins>

core/src/test/java/com/onelogin/saml2/test/AuthTest.java

@@ -32,50 +32,17 @@
 import javax.servlet.http.HttpSession;
 
 import org.junit.Test;
-import org.junit.runner.RunWith;
-import org.powermock.core.classloader.annotations.PrepareForTest;
 
 import com.onelogin.saml2.Auth;
-import com.onelogin.saml2.authn.SamlResponse;
 import com.onelogin.saml2.exception.SettingsException;
 import com.onelogin.saml2.exception.XMLEntityException;
 import com.onelogin.saml2.settings.Saml2Settings;
 import com.onelogin.saml2.settings.SettingsBuilder;
 import com.onelogin.saml2.util.Constants;
 import com.onelogin.saml2.util.Util;
 
-/*
- * Known issues while testing
- *
- * ECLEmma coverage plugin incompatible with PowerMock
- * http://stackoverflow.com/questions/23363212/powermock-eclemma-coverage-issue
- *
- * import org.junit.runner.RunWith;
- * import org.powermock.modules.junit4.PowerMockRunner;
- * @RunWith(PowerMockRunner.class)
- *
- */
-@PrepareForTest({Auth.class})
 public class AuthTest {
 
-	/*
-	 * When using @PrepareForTest and PowerMockito the breakpoints when debugging
-	 * does not work. Define Rules!
-	 * (http://stackoverflow.com/questions/35140575/powermockito-junit-and-eclemma-debugging-dosent-work)
-	 *
-	 * https://github.com/jayway/powermock/wiki/PowerMockRule
-	 *
-	 * import org.junit.Rule;
-	 * import org.powermock.modules.junit4.rule.PowerMockRule;
-	 * @Rule
-	 * public PowerMockRule rule = new PowerMockRule();
-	 *
-	 */
-
-	//import org.powermock.modules.junit4.rule.PowerMockRule;
-	//@Rule
-	//public PowerMockRule rule = new PowerMockRule();
-
 	/**
 	 * Tests the constructor of Auth
 	 * Case: No parameters

core/src/test/java/com/onelogin/saml2/test/MetadataTest.java

@@ -12,7 +12,6 @@
 import java.security.cert.CertificateEncodingException;
 import java.util.Calendar;
 
-import org.powermock.core.classloader.annotations.PrepareForTest;
 import org.w3c.dom.Document;
 import org.junit.Test;
 
@@ -25,7 +24,6 @@
 /**
  * Tests the com.onelogin.saml2.Metadata class
  */
-@PrepareForTest({Metadata.class})
 public class MetadataTest {
 	/**
 	 * Tests the constructor method of Metadata

core/src/test/java/com/onelogin/saml2/test/authn/AuthnRequestTest.java

@@ -4,38 +4,22 @@
 import static org.hamcrest.CoreMatchers.not;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertThat;
-import static org.mockito.Mockito.mock;
-import static org.mockito.Mockito.verify;
-import static org.mockito.Mockito.when;
 import static org.powermock.api.mockito.PowerMockito.when;
 import static org.powermock.api.support.membermodification.MemberMatcher.method;
 
 import java.util.ArrayList;
-import java.util.HashMap;
 import java.util.List;
-import java.util.Map;
 
-import javax.servlet.http.HttpServletRequest;
-import javax.servlet.http.HttpServletResponse;
-
-import org.junit.Rule;
 import org.junit.Test;
 import org.powermock.api.mockito.PowerMockito;
-import org.powermock.core.classloader.annotations.PrepareForTest;
-//import org.powermock.modules.junit4.PowerMockRunner;
-import org.powermock.modules.junit4.rule.PowerMockRule;
 
 import com.onelogin.saml2.authn.AuthnRequest;
 import com.onelogin.saml2.settings.Saml2Settings;
 import com.onelogin.saml2.settings.SettingsBuilder;
 import com.onelogin.saml2.util.Util;
 
-@PrepareForTest({AuthnRequest.class})
 public class AuthnRequestTest {
 
-	 @Rule
-	 public PowerMockRule rule = new PowerMockRule();
-
 	/**
 	 * Tests the getEncodedAuthnRequest method of AuthnRequest
 	 *
@@ -278,4 +262,4 @@ public void testAuthNDestination() throws Exception {
 		assertThat(authnRequestStr, containsString("<samlp:AuthnRequest"));
 		assertThat(authnRequestStr, not(containsString("Destination=\"http://idp.example.com/simplesaml/saml2/idp/SSOService.php\"")));
 	}
-}
+}

core/src/test/java/com/onelogin/saml2/test/authn/AuthnResponseTest.java

@@ -18,8 +18,6 @@
 
 import org.junit.Rule;
 import org.junit.Test;
-import org.powermock.core.classloader.annotations.PrepareForTest;
-import org.powermock.modules.junit4.rule.PowerMockRule;
 import org.w3c.dom.Document;
 import org.w3c.dom.Node;
 import org.w3c.dom.NodeList;
@@ -31,12 +29,8 @@
 import com.onelogin.saml2.util.Util;
 import com.onelogin.saml2.util.Constants;
 
-@PrepareForTest({SamlResponse.class})
 public class AuthnResponseTest {
-	
-	@Rule
-	public PowerMockRule rule = new PowerMockRule();
-	
+
 	/**
 	 * Tests the constructor of SamlResponse
 	 *

core/src/test/java/com/onelogin/saml2/test/logout/LogoutRequestTest.java

@@ -21,26 +21,20 @@
 
 import java.security.PrivateKey;
 
+import org.powermock.api.mockito.PowerMockito;
 import org.w3c.dom.Document;
 
 import org.junit.Rule;
 import org.junit.Test;
-import org.powermock.api.mockito.PowerMockito;
-import org.powermock.core.classloader.annotations.PrepareForTest;
-import org.powermock.modules.junit4.rule.PowerMockRule;
 
 import com.onelogin.saml2.exception.XMLEntityException;
 import com.onelogin.saml2.logout.LogoutRequest;
 import com.onelogin.saml2.settings.Saml2Settings;
 import com.onelogin.saml2.settings.SettingsBuilder;
 import com.onelogin.saml2.util.Util;
 
-@PrepareForTest({LogoutRequest.class})
 public class LogoutRequestTest {
 
-	@Rule
-	public PowerMockRule rule = new PowerMockRule();
-
 	/**
 	 * Tests the constructor and the getEncodedLogoutRequest method of LogoutRequest
 	 *
@@ -701,4 +695,4 @@ public void testGetError() throws URISyntaxException, IOException, XMLEntityExce
 		logoutRequest.isValid();
 		assertNull(logoutRequest.getError());
 	}
-}
+}

core/src/test/java/com/onelogin/saml2/test/logout/LogoutResponseTest.java

@@ -17,11 +17,8 @@
 import javax.servlet.http.HttpServletRequest;
 import javax.xml.xpath.XPathExpressionException;
 
-import org.junit.Rule;
 import org.junit.Test;
 import org.powermock.api.mockito.PowerMockito;
-import org.powermock.core.classloader.annotations.PrepareForTest;
-import org.powermock.modules.junit4.rule.PowerMockRule;
 
 import com.onelogin.saml2.exception.XMLEntityException;
 import com.onelogin.saml2.logout.LogoutResponse;
@@ -30,12 +27,8 @@
 import com.onelogin.saml2.util.Util;
 import com.onelogin.saml2.util.Constants;
 
-@PrepareForTest({LogoutResponse.class})
 public class LogoutResponseTest {
 
-	@Rule
-	public PowerMockRule rule = new PowerMockRule();
-
 	/**
 	 * Tests the constructor, the build and the getEncodedLogoutResponse method of LogoutResponse
 	 *
@@ -501,4 +494,4 @@ public void testGetError() throws URISyntaxException, IOException, XMLEntityExce
 		logoutResponse.isValid();
 		assertNull(logoutResponse.getError());
 	}
-}
+}

core/src/test/java/com/onelogin/saml2/test/model/ContactTest.java

@@ -1,7 +1,6 @@
 package com.onelogin.saml2.test.model;
 
 import org.junit.Test;
-import org.powermock.core.classloader.annotations.PrepareForTest;
 
 import static org.junit.Assert.assertEquals;
 
@@ -10,7 +9,6 @@
 /**
  * Tests the com.onelogin.saml2.model.Contact class
  */
-@PrepareForTest({Contact.class})
 public class ContactTest {
 
 	/**

core/src/test/java/com/onelogin/saml2/test/model/OrganizationTest.java

@@ -4,7 +4,6 @@
 import java.net.URL;
 
 import org.junit.Test;
-import org.powermock.core.classloader.annotations.PrepareForTest;
 
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
@@ -15,7 +14,6 @@
 /**
  * Tests the com.onelogin.saml2.model.Organization class
  */
-@PrepareForTest({Organization.class})
 public class OrganizationTest {
 
 	/**