This is an automated email from the ASF dual-hosted git repository.
markt pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/master by this push:
new 3ece85c Update test for changes in Base64 decoder
3ece85c is described below
commit 3ece85cc63ece4dde1ec526f32d6362d165599da
Author: Mark Thomas <[email protected]>
AuthorDate: Thu Aug 1 23:37:59 2019 +0100
Update test for changes in Base64 decoder
---
test/org/apache/catalina/authenticator/TestBasicAuthParser.java | 7 +++----
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/test/org/apache/catalina/authenticator/TestBasicAuthParser.java
b/test/org/apache/catalina/authenticator/TestBasicAuthParser.java
index 4be4923..627a59a 100644
--- a/test/org/apache/catalina/authenticator/TestBasicAuthParser.java
+++ b/test/org/apache/catalina/authenticator/TestBasicAuthParser.java
@@ -387,18 +387,17 @@ public class TestBasicAuthParser {
/*
* "-" is not a legal base64 character. The RFC says it must be
* ignored by the decoder. This will scramble the decoded string
- * and eventually result in an authentication Assert.failure.
+ * and eventually result in an IllegalArgumentException.
*/
- @Test
+ @Test(expected = IllegalArgumentException.class)
public void testBadBase64Char() throws Exception {
final String BASE64_CRIB = "dXNlcmlkOnNl-3JldHM=";
final BasicAuthHeader AUTH_HEADER =
new BasicAuthHeader(NICE_METHOD, BASE64_CRIB);
+ @SuppressWarnings("unused")
BasicAuthenticator.BasicCredentials credentials =
new BasicAuthenticator.BasicCredentials(
AUTH_HEADER.getHeader(), StandardCharsets.UTF_8, true);
- Assert.assertEquals(USER_NAME, credentials.getUsername());
- Assert.assertNotSame(PASSWORD, credentials.getPassword());
}
/*
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
