Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(22)

Unified Diff: tests/standalone/io/web_socket_compression_test.dart

Issue 1605583002: Add tests for no context takeover configurations with multiple messages. (Closed) Base URL: https://github.com/dart-lang/sdk.git@master
Patch Set: Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/standalone/io/web_socket_compression_test.dart
diff --git a/tests/standalone/io/web_socket_compression_test.dart b/tests/standalone/io/web_socket_compression_test.dart
index 912b855da3d9e24cdc64b0a49d762e75b104cc88..e25bc1b2d4f6a28fc926e14a7d163a4793eea154 100644
--- a/tests/standalone/io/web_socket_compression_test.dart
+++ b/tests/standalone/io/web_socket_compression_test.dart
@@ -130,6 +130,43 @@ class SecurityConfiguration {
});
}
+ void testContextSupport({CompressionOptions serverOpts,
+ CompressionOptions clientOpts,
+ int messages}) {
+ asyncStart();
+
+ createServer().then((server) {
+ server.listen((request) {
+ Expect.isTrue(WebSocketTransformer.isUpgradeRequest(request));
+ WebSocketTransformer.upgrade(request, compression: serverOpts)
+ .then((webSocket) {
+ webSocket.listen((message) {
+ Expect.equals("Hello World", message);
+ webSocket.add(message);
+ });
+ });
+ });
+
+ var url = '${secure ? "wss" : "ws"}://$HOST_NAME:${server.port}/';
+ WebSocket.connect(url, compression: clientOpts).then((websocket) {
+ var i = 1;
+ websocket.listen((message) {
+ Expect.equals("Hello World", message);
+ if (i == messages) {
+ websocket.close();
+ return;
+ }
+ websocket.add("Hello World");
+ i++;
+ }, onDone: () {
+ server.close();
+ asyncEnd();
+ });
+ websocket.add("Hello World");
+ });
+ });
+ }
+
void testCompressionHeaders() {
asyncStart();
createServer().then((server) {
@@ -271,6 +308,31 @@ class SecurityConfiguration {
// Compression on server but not client.
testCompressionSupport(server: true);
+ // Test Multiple messages with various context takeover configurations.
+ // no context takeover on the server.
+ var serverComp = new CompressionOptions(serverNoContextTakeover: true);
+ testContextSupport(serverOpts: serverComp,
+ clientOpts: serverComp,
+ messages: 5);
+ // no contexttakeover on the client.
+ var clientComp = new CompressionOptions(clientNoContextTakeover: true);
+ testContextSupport(serverOpts: clientComp,
+ clientOpts: clientComp,
+ messages: 5);
+ // no context takeover enabled for both.
+ var compression = new CompressionOptions(serverNoContextTakeover: true,
+ clientNoContextTakeover: true);
+ testContextSupport(serverOpts: compression,
+ clientOpts: compression,
+ messages: 5);
+ // no context take over for opposing configurations.
+ testContextSupport(serverOpts: serverComp,
+ clientOpts: clientComp,
+ messages: 5);
+ testContextSupport(serverOpts: clientComp,
+ clientOpts: serverComp,
+ messages: 5);
+
testCompressionHeaders();
// Chrome headers
testReturnHeaders('permessage-deflate; client_max_window_bits',
@@ -289,7 +351,7 @@ class SecurityConfiguration {
'client_no_context_takeover',
'permessage-deflate; client_max_window_bits=15');
// Enable context Takeover and provide if requested.
- var compression = new CompressionOptions(clientNoContextTakeover: true,
+ compression = new CompressionOptions(clientNoContextTakeover: true,
serverNoContextTakeover: true);
testReturnHeaders('permessage-deflate; client_max_window_bits; '
'client_no_context_takeover',
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698