Merge -r 1370811:1370812 from trunk to branch. FIXES: HADOOP-8660
git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/branches/branch-2@1370815 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
parent
92a99de731
commit
221e491f5d
|
@ -13,6 +13,7 @@
|
||||||
*/
|
*/
|
||||||
package org.apache.hadoop.security.authentication.client;
|
package org.apache.hadoop.security.authentication.client;
|
||||||
|
|
||||||
|
import junit.framework.Assert;
|
||||||
import org.apache.hadoop.security.authentication.server.AuthenticationFilter;
|
import org.apache.hadoop.security.authentication.server.AuthenticationFilter;
|
||||||
import junit.framework.TestCase;
|
import junit.framework.TestCase;
|
||||||
import org.mockito.Mockito;
|
import org.mockito.Mockito;
|
||||||
|
@ -114,6 +115,18 @@ public abstract class AuthenticatorTestCase extends TestCase {
|
||||||
return "http://" + host + ":" + port + "/foo/bar";
|
return "http://" + host + ":" + port + "/foo/bar";
|
||||||
}
|
}
|
||||||
|
|
||||||
|
private static class TestConnectionConfigurator
|
||||||
|
implements ConnectionConfigurator {
|
||||||
|
boolean invoked;
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public HttpURLConnection configure(HttpURLConnection conn)
|
||||||
|
throws IOException {
|
||||||
|
invoked = true;
|
||||||
|
return conn;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
private String POST = "test";
|
private String POST = "test";
|
||||||
|
|
||||||
protected void _testAuthentication(Authenticator authenticator, boolean doPost) throws Exception {
|
protected void _testAuthentication(Authenticator authenticator, boolean doPost) throws Exception {
|
||||||
|
@ -121,11 +134,10 @@ public abstract class AuthenticatorTestCase extends TestCase {
|
||||||
try {
|
try {
|
||||||
URL url = new URL(getBaseURL());
|
URL url = new URL(getBaseURL());
|
||||||
AuthenticatedURL.Token token = new AuthenticatedURL.Token();
|
AuthenticatedURL.Token token = new AuthenticatedURL.Token();
|
||||||
ConnectionConfigurator connConf =
|
TestConnectionConfigurator connConf = new TestConnectionConfigurator();
|
||||||
Mockito.mock(ConnectionConfigurator.class);
|
|
||||||
AuthenticatedURL aUrl = new AuthenticatedURL(authenticator, connConf);
|
AuthenticatedURL aUrl = new AuthenticatedURL(authenticator, connConf);
|
||||||
HttpURLConnection conn = aUrl.openConnection(url, token);
|
HttpURLConnection conn = aUrl.openConnection(url, token);
|
||||||
Mockito.verify(connConf).configure(Mockito.<HttpURLConnection>any());
|
Assert.assertTrue(connConf.invoked);
|
||||||
String tokenStr = token.toString();
|
String tokenStr = token.toString();
|
||||||
if (doPost) {
|
if (doPost) {
|
||||||
conn.setRequestMethod("POST");
|
conn.setRequestMethod("POST");
|
||||||
|
|
|
@ -208,6 +208,8 @@ Release 2.0.1-alpha - UNRELEASED
|
||||||
|
|
||||||
HADOOP-8405. ZKFC tests leak ZK instances. (todd)
|
HADOOP-8405. ZKFC tests leak ZK instances. (todd)
|
||||||
|
|
||||||
|
HADOOP-8660. TestPseudoAuthenticator failing with NPE. (tucu)
|
||||||
|
|
||||||
Release 2.0.0-alpha - 05-23-2012
|
Release 2.0.0-alpha - 05-23-2012
|
||||||
|
|
||||||
INCOMPATIBLE CHANGES
|
INCOMPATIBLE CHANGES
|
||||||
|
|
Loading…
Reference in New Issue