CUPS: Add a regression test for duplex detection.

chrome/browser/printing/print_system_task_proxy_unittest.cc

 // Copyright (c) 2011 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 <cups/cups.h>
 
 #include <cstring>
 #include <string>
 #include <vector>
 
@@ skipping 203 matching lines @@
        it != test_cases.end(); ++it) {
     num_options = 0;
     options = NULL;
     printing_internal::parse_lpoptions(userLpOptionsFile, it->printer_name,
                                        &num_options, &options);
     ASSERT_EQ(num_options, it->expected_option_count);
     EXPECT_EQ(num_options != 0, options != NULL);
     cupsFreeOptions(num_options, options);
   }
 }
+
+// Test duplex detection code, which regressed in http://crbug.com/103999.
+TEST(PrintSystemTaskProxyTest, DetectDuplexModeCUPS) {

[kmadhusu, 2011/11/16 20:43:29]

Can you add another test for windows duplex setting?

[Lei Zhang, 2011/11/17 00:20:16]

That's beyond the scope of this CL.

[kmadhusu, 2011/11/17 01:03:00]

From my understanding, the purpose of this CL to add tests to verify/test duplex
settings irrespective of the platform.   Since you are adding new tests, I
thought it is better to add test for windows duplex setting.

+  // Specifies the test ppd data.
+  printing::PrinterCapsAndDefaults printer_info;
+  printer_info.printer_capabilities.append(
+      "*PPD-Adobe: \"4.3\"\n\n"
+      "*OpenGroup: General/General\n\n"
+      "*OpenUI *Duplex/Double-Sided Printing: PickOne\n"
+      "*DefaultDuplex: None\n"
+      "*Duplex None/Off: "
+      "\"<</Duplex false>>setpagedevice\"\n"
+      "*Duplex DuplexNoTumble/Long Edge (Standard): "
+      "\"<</Duplex true/Tumble false>>setpagedevice\"\n"
+      "*Duplex DuplexTumble/Short Edge (Flip): "
+      "\"<</Duplex true/Tumble true>>setpagedevice\"\n"
+      "*CloseUI: *Duplex\n\n"
+      "*CloseGroup: General\n");
+

[kmadhusu, 2011/11/16 20:43:29]

nit: remove blank line

[Lei Zhang, 2011/11/17 00:20:16]

Done.

+
+  bool set_color_as_default = false;
+  bool set_duplex_as_default = false;
+  int printer_color_space_for_color = printing::UNKNOWN_COLOR_MODEL;
+  int printer_color_space_for_black = printing::UNKNOWN_COLOR_MODEL;
+  int default_duplex_setting_value = printing::UNKNOWN_DUPLEX_MODE;
+
+  scoped_refptr<PrintSystemTaskProxy> test_proxy;
+  bool res = test_proxy->GetPrinterCapabilitiesCUPS(
+      printer_info,
+      "InvalidPrinter",
+      &set_color_as_default,
+      &printer_color_space_for_color,
+      &printer_color_space_for_black,
+      &set_duplex_as_default,
+      &default_duplex_setting_value);
+  ASSERT_TRUE(res);
+  EXPECT_FALSE(set_duplex_as_default);
+  EXPECT_EQ(printing::SIMPLEX, default_duplex_setting_value);

[kmadhusu, 2011/11/16 20:43:29]

Can you add another test to verify that a ppd without duplex setting info will
return UNKNOWN_DUPLEX_MODE?

[Lei Zhang, 2011/11/17 00:20:16]

Done.

+}