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

commit bd42058e37c6239cfa1dae90a637e018ab62f8ea
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Fri Mar 27 09:37:51 2020 +0000

    Fix deprecation warnings with newer JUnit 4 in 2020.03 Eclipse IDE
---
 test/org/apache/catalina/core/TestStandardContext.java            | 3 ++-
 .../tribes/group/interceptors/TestEncryptInterceptor.java         | 3 ++-
 .../apache/catalina/valves/rewrite/TestQuotedStringTokenizer.java | 8 ++++----
 test/org/apache/coyote/http2/TestHttp2Limits.java                 | 5 +++--
 test/org/apache/tomcat/util/net/openssl/TestOpenSSLConf.java      | 5 +++--
 5 files changed, 14 insertions(+), 10 deletions(-)

diff --git a/test/org/apache/catalina/core/TestStandardContext.java 
b/test/org/apache/catalina/core/TestStandardContext.java
index f7e5793..b28905a 100644
--- a/test/org/apache/catalina/core/TestStandardContext.java
+++ b/test/org/apache/catalina/core/TestStandardContext.java
@@ -46,6 +46,7 @@ import jakarta.servlet.http.HttpServletRequest;
 import jakarta.servlet.http.HttpServletResponse;
 
 import org.hamcrest.CoreMatchers;
+import org.hamcrest.MatcherAssert;
 
 import org.junit.Assert;
 import org.junit.Test;
@@ -972,7 +973,7 @@ public class TestStandardContext extends TomcatBaseTest {
         Context context = new StandardContext();
 
         context.setResourceOnlyServlets("a,b,c");
-        
Assert.assertThat(Arrays.asList(context.getResourceOnlyServlets().split(",")),
+        
MatcherAssert.assertThat(Arrays.asList(context.getResourceOnlyServlets().split(",")),
                 CoreMatchers.hasItems("a", "b", "c"));
     }
 
diff --git 
a/test/org/apache/catalina/tribes/group/interceptors/TestEncryptInterceptor.java
 
b/test/org/apache/catalina/tribes/group/interceptors/TestEncryptInterceptor.java
index 5b76df7..172ca20 100644
--- 
a/test/org/apache/catalina/tribes/group/interceptors/TestEncryptInterceptor.java
+++ 
b/test/org/apache/catalina/tribes/group/interceptors/TestEncryptInterceptor.java
@@ -26,6 +26,7 @@ import java.util.Collection;
 
 import javax.crypto.Cipher;
 
+import org.hamcrest.MatcherAssert;
 import org.hamcrest.core.IsEqual;
 import org.hamcrest.core.IsNot;
 
@@ -371,7 +372,7 @@ public class TestEncryptInterceptor {
 
         byte[] cipherText2 = 
((ValueCaptureInterceptor)src.getNext()).getValue();
 
-        Assert.assertThat("Two identical cleartexts encrypt to the same 
ciphertext",
+        MatcherAssert.assertThat("Two identical cleartexts encrypt to the same 
ciphertext",
                 cipherText1, IsNot.not(IsEqual.equalTo(cipherText2)));
     }
 
diff --git 
a/test/org/apache/catalina/valves/rewrite/TestQuotedStringTokenizer.java 
b/test/org/apache/catalina/valves/rewrite/TestQuotedStringTokenizer.java
index ec228ae..2104155 100644
--- a/test/org/apache/catalina/valves/rewrite/TestQuotedStringTokenizer.java
+++ b/test/org/apache/catalina/valves/rewrite/TestQuotedStringTokenizer.java
@@ -23,8 +23,8 @@ import java.util.Collections;
 import java.util.List;
 
 import org.hamcrest.CoreMatchers;
+import org.hamcrest.MatcherAssert;
 
-import org.junit.Assert;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.junit.runners.Parameterized;
@@ -60,12 +60,12 @@ public class TestQuotedStringTokenizer {
         List<String> result = new ArrayList<>();
         int count = tokens.size();
         while (tokenizer.hasMoreTokens()) {
-            Assert.assertThat(Integer.valueOf(tokenizer.countTokens()), 
CoreMatchers.is(Integer.valueOf(count)));
+            MatcherAssert.assertThat(Integer.valueOf(tokenizer.countTokens()), 
CoreMatchers.is(Integer.valueOf(count)));
             result.add(tokenizer.nextToken());
             count--;
         }
-        Assert.assertThat(Integer.valueOf(tokenizer.countTokens()), 
CoreMatchers.is(Integer.valueOf(0)));
-        Assert.assertThat(tokens, CoreMatchers.is(result));
+        MatcherAssert.assertThat(Integer.valueOf(tokenizer.countTokens()), 
CoreMatchers.is(Integer.valueOf(0)));
+        MatcherAssert.assertThat(tokens, CoreMatchers.is(result));
     }
 
 }
diff --git a/test/org/apache/coyote/http2/TestHttp2Limits.java 
b/test/org/apache/coyote/http2/TestHttp2Limits.java
index d3ad8c8..69f6438 100644
--- a/test/org/apache/coyote/http2/TestHttp2Limits.java
+++ b/test/org/apache/coyote/http2/TestHttp2Limits.java
@@ -23,6 +23,7 @@ import java.util.List;
 import java.util.Random;
 
 import org.hamcrest.Description;
+import org.hamcrest.MatcherAssert;
 import org.hamcrest.TypeSafeMatcher;
 
 import org.junit.Assert;
@@ -265,7 +266,7 @@ public class TestHttp2Limits extends Http2TestBase {
             //       above.
             try {
                 parser.readFrame(true);
-                Assert.assertThat(output.getTrace(), RegexMatcher.matchesRegex(
+                MatcherAssert.assertThat(output.getTrace(), 
RegexMatcher.matchesRegex(
                         "0-Goaway-\\[1\\]-\\[11\\]-\\[" + limitMessage + 
"\\]"));
             } catch (IOException se) {
                 // Expected on some platforms
@@ -505,7 +506,7 @@ public class TestHttp2Limits extends Http2TestBase {
             // the escaped values.
             String limitMessage = sm.getString("http2Parser.headerLimitSize", 
"\\d++", "3");
             limitMessage = limitMessage.replace("[", "\\[").replace("]", 
"\\]");
-            Assert.assertThat(output.getTrace(), RegexMatcher.matchesRegex(
+            MatcherAssert.assertThat(output.getTrace(), 
RegexMatcher.matchesRegex(
                     "0-Goaway-\\[3\\]-\\[11\\]-\\[" + limitMessage + "\\]"));
             break;
         }
diff --git a/test/org/apache/tomcat/util/net/openssl/TestOpenSSLConf.java 
b/test/org/apache/tomcat/util/net/openssl/TestOpenSSLConf.java
index 2c66678..07eb9ad 100644
--- a/test/org/apache/tomcat/util/net/openssl/TestOpenSSLConf.java
+++ b/test/org/apache/tomcat/util/net/openssl/TestOpenSSLConf.java
@@ -21,6 +21,7 @@ import java.util.HashSet;
 import java.util.Set;
 
 import org.hamcrest.CoreMatchers;
+import org.hamcrest.MatcherAssert;
 
 import org.junit.Assert;
 import org.junit.Assume;
@@ -101,10 +102,10 @@ public class TestOpenSSLConf extends TomcatBaseTest {
             sslHostConfig = initOpenSSLConfCmd("CipherString", ENABLED_CIPHER);
         }
         String[] ciphers = sslHostConfig.getEnabledCiphers();
-        Assert.assertThat("Wrong HostConfig ciphers", ciphers,
+        MatcherAssert.assertThat("Wrong HostConfig ciphers", ciphers,
                 CoreMatchers.is(EXPECTED_CIPHERS));
         ciphers = 
SSLContext.getCiphers(sslHostConfig.getOpenSslContext().longValue());
-        Assert.assertThat("Wrong native SSL context ciphers", ciphers,
+        MatcherAssert.assertThat("Wrong native SSL context ciphers", ciphers,
                 CoreMatchers.is(EXPECTED_CIPHERS));
     }
 


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to