You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2017/09/18 16:13:46 UTC

svn commit: r1808740 - /tomcat/tc7.0.x/trunk/test/org/apache/tomcat/websocket/TestPerMessageDeflate.java

Author: markt
Date: Mon Sep 18 16:13:46 2017
New Revision: 1808740

URL: http://svn.apache.org/viewvc?rev=1808740&view=rev
Log:
Unit tests always compile with Java 6 so avoid Java 7 features.
This is simpler than refactoring the build scripts to build the WebSocket unit tests with Java 7.

Modified:
    tomcat/tc7.0.x/trunk/test/org/apache/tomcat/websocket/TestPerMessageDeflate.java

Modified: tomcat/tc7.0.x/trunk/test/org/apache/tomcat/websocket/TestPerMessageDeflate.java
URL: http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/test/org/apache/tomcat/websocket/TestPerMessageDeflate.java?rev=1808740&r1=1808739&r2=1808740&view=diff
==============================================================================
--- tomcat/tc7.0.x/trunk/test/org/apache/tomcat/websocket/TestPerMessageDeflate.java (original)
+++ tomcat/tc7.0.x/trunk/test/org/apache/tomcat/websocket/TestPerMessageDeflate.java Mon Sep 18 16:13:46 2017
@@ -18,7 +18,7 @@ package org.apache.tomcat.websocket;
 
 import java.io.IOException;
 import java.nio.ByteBuffer;
-import java.nio.charset.StandardCharsets;
+import java.nio.charset.Charset;
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.List;
@@ -31,6 +31,13 @@ import org.junit.Test;
 public class TestPerMessageDeflate {
 
     /*
+     * This replaces StandardCharsets.UTF_8 as that requires Java 7 and this is
+     * simpler than refactoring the build script to build the WebSocket unit
+     * tests with Java 7.
+     */
+    private static final Charset UTF_8 = Charset.forName("UTF-8");
+
+    /*
      * https://bz.apache.org/bugzilla/show_bug.cgi?id=61491
      */
     @Test
@@ -44,14 +51,14 @@ public class TestPerMessageDeflate {
         PerMessageDeflate perMessageDeflate = PerMessageDeflate.negotiate(preferences, true);
         perMessageDeflate.setNext(new TesterTransformation());
 
-        ByteBuffer bb1 = ByteBuffer.wrap("A".getBytes(StandardCharsets.UTF_8));
+        ByteBuffer bb1 = ByteBuffer.wrap("A".getBytes(UTF_8));
         MessagePart mp1 = new MessagePart(true, 0, Constants.OPCODE_TEXT, bb1, null, null);
 
         List<MessagePart> uncompressedParts1 = new ArrayList<MessagePart>();
         uncompressedParts1.add(mp1);
         perMessageDeflate.sendMessagePart(uncompressedParts1);
 
-        ByteBuffer bb2 = ByteBuffer.wrap("".getBytes(StandardCharsets.UTF_8));
+        ByteBuffer bb2 = ByteBuffer.wrap("".getBytes(UTF_8));
         MessagePart mp2 = new MessagePart(true, 0, Constants.OPCODE_TEXT, bb2, null, null);
 
         List<MessagePart> uncompressedParts2 = new ArrayList<MessagePart>();



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