OLD | NEW |
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
4 | 4 |
5 #include "base/path_service.h" | 5 #include "base/path_service.h" |
6 | 6 |
7 #include "base/basictypes.h" | 7 #include "base/basictypes.h" |
8 #include "base/file_util.h" | 8 #include "base/file_util.h" |
9 #include "base/file_path.h" | 9 #include "base/file_path.h" |
10 #include "base/scoped_temp_dir.h" | 10 #include "base/scoped_temp_dir.h" |
11 #if defined(OS_WIN) | 11 #include "base/string_util.h" |
12 #include "base/win/windows_version.h" | 12 #include "build/build_config.h" |
13 #endif | |
14 #include "testing/gtest/include/gtest/gtest.h" | 13 #include "testing/gtest/include/gtest/gtest.h" |
15 #include "testing/gtest/include/gtest/gtest-spi.h" | 14 #include "testing/gtest/include/gtest/gtest-spi.h" |
16 #include "testing/platform_test.h" | 15 #include "testing/platform_test.h" |
17 | 16 |
| 17 #if defined(OS_WIN) |
| 18 #include <userenv.h> |
| 19 #include "base/win/windows_version.h" |
| 20 // userenv.dll is required for GetDefaultUserProfileDirectory(). |
| 21 #pragma comment(lib, "userenv.lib") |
| 22 #endif |
| 23 |
18 namespace { | 24 namespace { |
19 | 25 |
20 // Returns true if PathService::Get returns true and sets the path parameter | 26 // Returns true if PathService::Get returns true and sets the path parameter |
21 // to non-empty for the given PathService::DirType enumeration value. | 27 // to non-empty for the given PathService::DirType enumeration value. |
22 bool ReturnsValidPath(int dir_type) { | 28 bool ReturnsValidPath(int dir_type) { |
23 FilePath path; | 29 FilePath path; |
24 bool result = PathService::Get(dir_type, &path); | 30 bool result = PathService::Get(dir_type, &path); |
25 #if defined(OS_POSIX) | 31 #if defined(OS_POSIX) |
26 // If chromium has never been started on this account, the cache path may not | 32 // If chromium has never been started on this account, the cache path may not |
27 // exist. | 33 // exist. |
28 if (dir_type == base::DIR_CACHE) | 34 if (dir_type == base::DIR_CACHE) |
29 return result && !path.value().empty(); | 35 return result && !path.empty(); |
30 #endif | 36 #endif |
31 return result && !path.value().empty() && file_util::PathExists(path); | 37 #if defined(OS_LINUX) |
| 38 // On the linux try-bots: a path is returned (e.g. /home/chrome-bot/Desktop), |
| 39 // but it doesn't exist. |
| 40 if (dir_type == base::DIR_USER_DESKTOP) |
| 41 return result && !path.empty(); |
| 42 #endif |
| 43 #if defined(OS_WIN) |
| 44 // On Windows XP, the Quick Launch folder for the "Default User" doesn't exist |
| 45 // by default. At least confirm that the path returned begins with the |
| 46 // Default User's profile path. |
| 47 if (dir_type == base::DIR_DEFAULT_USER_QUICK_LAUNCH && |
| 48 base::win::GetVersion() < base::win::VERSION_VISTA) { |
| 49 wchar_t default_profile_path[MAX_PATH]; |
| 50 DWORD size = arraysize(default_profile_path); |
| 51 return (result && |
| 52 ::GetDefaultUserProfileDirectory(default_profile_path, &size) && |
| 53 StartsWith(path.value(), default_profile_path, false)); |
| 54 } |
| 55 #endif |
| 56 return result && !path.empty() && file_util::PathExists(path); |
32 } | 57 } |
33 | 58 |
34 #if defined(OS_WIN) | 59 #if defined(OS_WIN) |
35 // Function to test any directory keys that are not supported on some versions | 60 // Function to test any directory keys that are not supported on some versions |
36 // of Windows. Checks that the function fails and that the returned path is | 61 // of Windows. Checks that the function fails and that the returned path is |
37 // empty. | 62 // empty. |
38 bool ReturnsInvalidPath(int dir_type) { | 63 bool ReturnsInvalidPath(int dir_type) { |
39 FilePath path; | 64 FilePath path; |
40 bool result = PathService::Get(dir_type, &path); | 65 bool result = PathService::Get(dir_type, &path); |
41 return !result && path.empty(); | 66 return !result && path.empty(); |
42 } | 67 } |
43 #endif | 68 #endif |
44 | 69 |
45 } // namespace | 70 } // namespace |
46 | 71 |
47 // On the Mac this winds up using some autoreleased objects, so we need to | 72 // On the Mac this winds up using some autoreleased objects, so we need to |
48 // be a PlatformTest. | 73 // be a PlatformTest. |
49 typedef PlatformTest PathServiceTest; | 74 typedef PlatformTest PathServiceTest; |
50 | 75 |
51 // Test that all PathService::Get calls return a value and a true result | 76 // Test that all PathService::Get calls return a value and a true result |
52 // in the development environment. (This test was created because a few | 77 // in the development environment. (This test was created because a few |
53 // later changes to Get broke the semantics of the function and yielded the | 78 // later changes to Get broke the semantics of the function and yielded the |
54 // correct value while returning false.) | 79 // correct value while returning false.) |
55 TEST_F(PathServiceTest, Get) { | 80 TEST_F(PathServiceTest, Get) { |
56 for (int key = base::DIR_CURRENT; key < base::PATH_END; ++key) { | 81 for (int key = base::PATH_START + 1; key < base::PATH_END; ++key) { |
57 #if defined(OS_ANDROID) | 82 #if defined(OS_ANDROID) |
58 if (key == base::FILE_MODULE) | 83 if (key == base::FILE_MODULE || key == base::DIR_USER_DESKTOP) |
59 continue; // Android doesn't implement FILE_MODULE; | 84 continue; // Android doesn't implement FILE_MODULE and DIR_USER_DESKTOP; |
60 #endif | 85 #endif |
61 EXPECT_PRED1(ReturnsValidPath, key); | 86 EXPECT_PRED1(ReturnsValidPath, key); |
62 } | 87 } |
63 #if defined(OS_WIN) | 88 #if defined(OS_WIN) |
64 for (int key = base::PATH_WIN_START + 1; key < base::PATH_WIN_END; ++key) { | 89 for (int key = base::PATH_WIN_START + 1; key < base::PATH_WIN_END; ++key) { |
65 bool valid = true; | 90 bool valid = true; |
66 switch(key) { | 91 switch(key) { |
67 case base::DIR_LOCAL_APP_DATA_LOW: | 92 case base::DIR_LOCAL_APP_DATA_LOW: |
68 // DIR_LOCAL_APP_DATA_LOW is not supported prior Vista and is expected | 93 // DIR_LOCAL_APP_DATA_LOW is not supported prior Vista and is expected |
69 // to fail. | 94 // to fail. |
70 valid = base::win::GetVersion() >= base::win::VERSION_VISTA; | 95 valid = base::win::GetVersion() >= base::win::VERSION_VISTA; |
71 break; | 96 break; |
72 case base::DIR_APP_SHORTCUTS: | 97 case base::DIR_APP_SHORTCUTS: |
73 // DIR_APP_SHORTCUTS is not supported prior Windows 8 and is expected to | 98 // DIR_APP_SHORTCUTS is not supported prior Windows 8 and is expected to |
74 // fail. | 99 // fail. |
75 valid = base::win::GetVersion() >= base::win::VERSION_WIN8; | 100 valid = base::win::GetVersion() >= base::win::VERSION_WIN8; |
76 break; | 101 break; |
77 } | 102 } |
78 | 103 |
79 if (valid) | 104 if (valid) |
80 EXPECT_TRUE(ReturnsValidPath(key)) << key; | 105 EXPECT_TRUE(ReturnsValidPath(key)) << key; |
81 else | 106 else |
82 EXPECT_TRUE(ReturnsInvalidPath(key)) << key; | 107 EXPECT_TRUE(ReturnsInvalidPath(key)) << key; |
83 } | 108 } |
84 #elif defined(OS_MACOSX) | 109 #elif defined(OS_MACOSX) |
85 for (int key = base::PATH_MAC_START + 1; key < base::PATH_MAC_END; ++key) { | 110 for (int key = base::PATH_MAC_START + 1; key < base::PATH_MAC_END; ++key) { |
86 EXPECT_PRED1(ReturnsValidPath, key); | 111 EXPECT_PRED1(ReturnsValidPath, key); |
| 112 } |
| 113 #elif defined(OS_ANDROID) |
| 114 for (int key = base::PATH_ANDROID_START + 1; key < base::PATH_ANDROID_END; |
| 115 ++key) { |
| 116 EXPECT_PRED1(ReturnsValidPath, key); |
| 117 } |
| 118 #elif defined(OS_POSIX) |
| 119 for (int key = base::PATH_POSIX_START + 1; key < base::PATH_POSIX_END; |
| 120 ++key) { |
| 121 EXPECT_PRED1(ReturnsValidPath, key); |
87 } | 122 } |
88 #endif | 123 #endif |
89 } | 124 } |
90 | 125 |
91 // test that all versions of the Override function of PathService do what they | 126 // test that all versions of the Override function of PathService do what they |
92 // are supposed to do. | 127 // are supposed to do. |
93 TEST_F(PathServiceTest, Override) { | 128 TEST_F(PathServiceTest, Override) { |
94 int my_special_key = 666; | 129 int my_special_key = 666; |
95 ScopedTempDir temp_dir; | 130 ScopedTempDir temp_dir; |
96 ASSERT_TRUE(temp_dir.CreateUniqueTempDir()); | 131 ASSERT_TRUE(temp_dir.CreateUniqueTempDir()); |
97 FilePath fake_cache_dir(temp_dir.path().AppendASCII("cache")); | 132 FilePath fake_cache_dir(temp_dir.path().AppendASCII("cache")); |
98 // PathService::Override should always create the path provided if it doesn't | 133 // PathService::Override should always create the path provided if it doesn't |
99 // exist. | 134 // exist. |
100 EXPECT_TRUE(PathService::Override(my_special_key, fake_cache_dir)); | 135 EXPECT_TRUE(PathService::Override(my_special_key, fake_cache_dir)); |
101 EXPECT_TRUE(file_util::PathExists(fake_cache_dir)); | 136 EXPECT_TRUE(file_util::PathExists(fake_cache_dir)); |
102 | 137 |
103 FilePath fake_cache_dir2(temp_dir.path().AppendASCII("cache2")); | 138 FilePath fake_cache_dir2(temp_dir.path().AppendASCII("cache2")); |
104 // PathService::OverrideAndCreateIfNeeded should obey the |create| parameter. | 139 // PathService::OverrideAndCreateIfNeeded should obey the |create| parameter. |
105 PathService::OverrideAndCreateIfNeeded(my_special_key, | 140 PathService::OverrideAndCreateIfNeeded(my_special_key, |
106 fake_cache_dir2, | 141 fake_cache_dir2, |
107 false); | 142 false); |
108 EXPECT_FALSE(file_util::PathExists(fake_cache_dir2)); | 143 EXPECT_FALSE(file_util::PathExists(fake_cache_dir2)); |
109 EXPECT_TRUE(PathService::OverrideAndCreateIfNeeded(my_special_key, | 144 EXPECT_TRUE(PathService::OverrideAndCreateIfNeeded(my_special_key, |
110 fake_cache_dir2, | 145 fake_cache_dir2, |
111 true)); | 146 true)); |
112 EXPECT_TRUE(file_util::PathExists(fake_cache_dir2)); | 147 EXPECT_TRUE(file_util::PathExists(fake_cache_dir2)); |
113 } | 148 } |
OLD | NEW |