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

Unified Diff: extensions/common/api/printer_provider/usb_printer_manifest_unittest.cc

Issue 1123093003: Add a manifest property declaring USB printers supported by an app. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix interfaceProtocol documentation. Created 5 years, 7 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/common/api/printer_provider/usb_printer_manifest_unittest.cc
diff --git a/extensions/common/api/printer_provider/usb_printer_manifest_unittest.cc b/extensions/common/api/printer_provider/usb_printer_manifest_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..cd35af12c1d24057928020dd4dc8bf7628cb1d54
--- /dev/null
+++ b/extensions/common/api/printer_provider/usb_printer_manifest_unittest.cc
@@ -0,0 +1,45 @@
+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "device/usb/usb_device_filter.h"
+#include "extensions/common/api/printer_provider/usb_printer_manifest_data.h"
+#include "extensions/common/manifest_test.h"
+#include "extensions/common/value_builder.h"
+
+namespace extensions {
+
+class UsbPrinterManifestTest : public ManifestTest {
+ public:
+ UsbPrinterManifestTest() {}
+ ~UsbPrinterManifestTest() override {}
+};
+
+TEST_F(UsbPrinterManifestTest, Filters) {
+ scoped_refptr<Extension> extension =
+ LoadAndExpectSuccess("usb_printers_filters.json");
+ const UsbPrinterManifestData* manifest_data =
+ UsbPrinterManifestData::Get(extension.get());
+ ASSERT_TRUE(manifest_data);
+ EXPECT_EQ(2u, manifest_data->filters().size());
+ EXPECT_TRUE(DictionaryBuilder()
+ .Set("vendorId", 1)
+ .Set("productId", 2)
+ .Build()
+ ->Equals(manifest_data->filters()[0].ToValue().get()));
+ EXPECT_TRUE(DictionaryBuilder()
+ .Set("vendorId", 1)
+ .Set("interfaceClass", 2)
+ .Set("interfaceSubclass", 3)
+ .Set("interfaceProtocol", 4)
+ .Build()
+ ->Equals(manifest_data->filters()[1].ToValue().get()));
+}
+
+TEST_F(UsbPrinterManifestTest, InvalidFilter) {
+ LoadAndExpectError(
+ "usb_printers_invalid_filter.json",
+ "Only one of productId or interfaceClass may be specified.");
+}
+
+} // namespace extensions
« no previous file with comments | « extensions/common/api/printer_provider/usb_printer_manifest_handler.cc ('k') | extensions/common/common_manifest_handlers.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698