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

Unified Diff: extensions/renderer/api/serial/data_receiver_unittest.cc

Issue 571333002: Add serialization support to the JS DataSender and DataReceiver. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@stash-service
Patch Set: address comments Created 6 years, 3 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
Index: extensions/renderer/api/serial/data_receiver_unittest.cc
diff --git a/extensions/renderer/api/serial/data_receiver_unittest.cc b/extensions/renderer/api/serial/data_receiver_unittest.cc
index f223cb3afba5d92c4fb8b2d049dfea72f66d8bcf..567fdc42e55c9222068c541fa0f8e8459beadc10 100644
--- a/extensions/renderer/api/serial/data_receiver_unittest.cc
+++ b/extensions/renderer/api/serial/data_receiver_unittest.cc
@@ -22,6 +22,8 @@ class DataReceiverTest : public ApiTestBase {
env()->RegisterModule("data_receiver", IDR_DATA_RECEIVER_JS);
env()->RegisterModule("device/serial/data_stream.mojom",
IDR_DATA_STREAM_MOJOM_JS);
+ env()->RegisterModule("device/serial/data_stream_serialization.mojom",
+ IDR_DATA_STREAM_SERIALIZATION_MOJOM_JS);
service_provider()->AddService(base::Bind(
&DataReceiverTest::CreateDataSource, base::Unretained(this)));
}
@@ -103,6 +105,37 @@ TEST_F(DataReceiverTest, ReceiveErrorThenData) {
RunTest("data_receiver_unittest.js", "testReceiveErrorThenData");
}
+TEST_F(DataReceiverTest, ReceiveBeforeAndAfterSerialization) {
+ data_to_send_.push("a");
+ data_to_send_.push("b");
+ RunTest("data_receiver_unittest.js",
+ "testReceiveBeforeAndAfterSerialization");
+}
+
+TEST_F(DataReceiverTest, ReceiveErrorSerialization) {
+ error_to_send_.push(1);
+ error_to_send_.push(3);
+ RunTest("data_receiver_unittest.js", "testReceiveErrorSerialization");
+}
+
+TEST_F(DataReceiverTest, ReceiveDataAndErrorSerialization) {
+ data_to_send_.push("a");
+ data_to_send_.push("b");
+ error_to_send_.push(1);
+ error_to_send_.push(3);
+ RunTest("data_receiver_unittest.js", "testReceiveDataAndErrorSerialization");
+}
+
+TEST_F(DataReceiverTest, SerializeDuringReceive) {
+ data_to_send_.push("a");
+ RunTest("data_receiver_unittest.js", "testSerializeDuringReceive");
+}
+
+TEST_F(DataReceiverTest, SerializeAfterClose) {
+ data_to_send_.push("a");
+ RunTest("data_receiver_unittest.js", "testSerializeAfterClose");
+}
+
TEST_F(DataReceiverTest, SourceShutdown) {
error_to_send_.push(2);
RunTest("data_receiver_unittest.js", "testSourceShutdown");

Powered by Google App Engine
This is Rietveld 408576698