Unit tests for publishing app shortcuts on Mac

chrome/browser/web_applications/web_app_mac_unittest.mm

Index: chrome/browser/web_applications/web_app_mac_unittest.mm
diff --git a/chrome/browser/web_applications/web_app_mac_unittest.mm b/chrome/browser/web_applications/web_app_mac_unittest.mm
new file mode 100644
index 0000000000000000000000000000000000000000..9d967df393550c0283f42efaa8a3bde5b92fb5ef
--- /dev/null
+++ b/chrome/browser/web_applications/web_app_mac_unittest.mm
@@ -0,0 +1,72 @@
+// Copyright (c) 2012 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/web_applications/web_app_mac.h"
+
+#include "base/file_util.h"
+#include "base/mac/foundation_util.h"
+#include "base/scoped_temp_dir.h"
+#include "base/sys_string_conversions.h"
+#include "base/utf_string_conversions.h"
+#include "chrome/common/mac/app_mode_common.h"
+#include "testing/gmock/include/gmock/gmock.h"
+#include "testing/gtest/include/gtest/gtest.h"
+#import "testing/gtest_mac.h"
+
+using ::testing::_;
+using ::testing::Return;
+using ::testing::NiceMock;
+
+namespace {
+
+class WebAppShortcutCreatorMock : public web_app::WebAppShortcutCreator {
+ public:
+  explicit WebAppShortcutCreatorMock(
+      const ShellIntegration::ShortcutInfo& shortcut_info)
+      : WebAppShortcutCreator(shortcut_info) {
+  }
+
+  MOCK_CONST_METHOD1(GetDstPath, FilePath(const FilePath&));
+};
+
+ShellIntegration::ShortcutInfo GetShortcutInfo() {
+  ShellIntegration::ShortcutInfo info;
+  info.extension_id = "extension_id";
+  info.title = ASCIIToUTF16("Shortcut Title");
+  info.url = GURL("http://example.com/");
+  return info;
+}
+
+// This test currently fails because the Mac app loader isn't built yet.
+TEST(WebAppShortcutCreatorTest, FAILS_CreateShortcut) {
+  ScopedTempDir scoped_temp_dir;
+  EXPECT_TRUE(scoped_temp_dir.CreateUniqueTempDir());
+  FilePath dst_path = scoped_temp_dir.path().Append("a.app");
+
+  ShellIntegration::ShortcutInfo info = GetShortcutInfo();
+  NiceMock<WebAppShortcutCreatorMock> shortcut_creator(info);
+  EXPECT_CALL(shortcut_creator, GetDstPath(_))
+      .WillOnce(Return(dst_path));
+  EXPECT_TRUE(shortcut_creator.CreateShortcut());
+  EXPECT_TRUE(file_util::PathExists(dst_path));
+
+  FilePath plist_path = dst_path.Append("Contents").Append("Info.plist");
+  NSDictionary* plist = [NSDictionary dictionaryWithContentsOfFile:
+      base::mac::FilePathToNSString(plist_path)];
+  EXPECT_NSEQ(base::SysUTF8ToNSString(info.extension_id),
+              [plist objectForKey:app_mode::kCrAppModeShortcutIDKey]);
+  EXPECT_NSEQ(base::SysUTF16ToNSString(info.title),
+              [plist objectForKey:app_mode::kCrAppModeShortcutNameKey]);
+  EXPECT_NSEQ(base::SysUTF8ToNSString(info.url.spec()),
+              [plist objectForKey:app_mode::kCrAppModeShortcutURLKey]);

[jeremy, 2012/02/08 09:14:51]

Would be good to test that the function that needs to fill in the plist is
filling in all fields.

[sail, 2012/02/09 23:04:39]

I added a TODO for this. Currently there are a couple of values that we don't
fill in.

+}
+
+TEST(WebAppShortcutCreatorTest, CreateFailure) {
+  NiceMock<WebAppShortcutCreatorMock> shortcut_creator(GetShortcutInfo());
+  EXPECT_CALL(shortcut_creator, GetDstPath(_))
+      .WillOnce(Return(FilePath("/non-existant/path/")));
+  EXPECT_FALSE(shortcut_creator.CreateShortcut());
+}
+
+}  // namespace