[Downloads] Add context parameter to help center article URL.

chrome/browser/download/download_commands_unittest.cc

 // 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 "chrome/browser/download/download_commands.h"
 
 #include <vector>
 
 #include "content/public/test/mock_download_item.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 using content::DownloadItem;
 using ::testing::Mock;
 using ::testing::NiceMock;
 using ::testing::Return;
 using ::testing::ReturnRefOfCopy;
 using ::testing::_;
 
 namespace {
 
 // Default target path for a mock download item in DownloadItemModelTest.
 const base::FilePath::CharType kDefaultTargetFilePath[] =
     FILE_PATH_LITERAL("/foo/bar/foo.bar");
 
 // Default URL for a mock download item in DownloadCommandsTest.
 const char kDefaultURL[] = "http://example.com/foo.bar";
 
+}  // namespace
+
 class DownloadCommandsTest : public testing::Test {
  public:
   DownloadCommandsTest() : commands_(&item_) {}
 
   virtual ~DownloadCommandsTest() {
   }
 
  protected:
   // Sets up defaults for the download item.
   void SetUp() override {
@@ skipping 34 matching lines @@
 
   DownloadCommands& commands() {
     return commands_;
   }
 
  private:
   NiceMock<content::MockDownloadItem> item_;
   DownloadCommands commands_;
 };
 
-}  // namespace
-
 TEST_F(DownloadCommandsTest, InProgress) {
   EXPECT_TRUE(IsCommandEnabled(DownloadCommands::SHOW_IN_FOLDER));
   EXPECT_TRUE(IsCommandEnabled(DownloadCommands::CANCEL));
   EXPECT_TRUE(IsCommandEnabled(DownloadCommands::PAUSE));
   EXPECT_FALSE(IsCommandEnabled(DownloadCommands::RESUME));
   EXPECT_TRUE(IsCommandEnabled(DownloadCommands::ALWAYS_OPEN_TYPE));
   EXPECT_TRUE(IsCommandEnabled(DownloadCommands::OPEN_WHEN_COMPLETE));
 
   EXPECT_FALSE(IsCommandChecked(DownloadCommands::OPEN_WHEN_COMPLETE));
   EXPECT_FALSE(IsCommandChecked(DownloadCommands::ALWAYS_OPEN_TYPE));
@@ skipping 52 matching lines @@
   // Pause.
   EXPECT_CALL(item(), Pause()).Times(1);
   commands().ExecuteCommand(DownloadCommands::PAUSE);
 }
 
 TEST_F(DownloadCommandsTest, DoResume) {
   // Resume.
   EXPECT_CALL(item(), Resume()).Times(1);
   commands().ExecuteCommand(DownloadCommands::RESUME);
 }
+
+TEST_F(DownloadCommandsTest, LearnMoreInterruptedURL) {
+  EXPECT_CALL(item(), GetLastReason())
+      .WillOnce(
+          Return(content::DOWNLOAD_INTERRUPT_REASON_NETWORK_DISCONNECTED));
+  GURL learn_more_url = commands().GetLearnMoreURLForInterruptedDownload();
+  EXPECT_EQ("p=ui_download_errors&ctx=22", learn_more_url.query());

[Randy Smith (Not in Mondays), 2015/04/21 17:28:37]

I'd rather compare against the download interrupt reason casted to an integer. 
We don't want this test to be dependent on the reason -> integer mappings, even
if we need to keep those fixed so that we don't mess up the help articles.

[asanka, 2015/04/22 16:58:47]

Done.

+}