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

Side by Side Diff: chromeos/dbus/cros_disks_client_unittest.cc

Issue 2392503002: Allow to pass remount option to Mount method of cros_disks_client. (Closed)
Patch Set: Sync to head. Created 4 years, 2 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 "chromeos/dbus/cros_disks_client.h" 5 #include "chromeos/dbus/cros_disks_client.h"
6 6
7 #include <stdint.h> 7 #include <stdint.h>
8 8
9 #include <memory> 9 #include <memory>
10 10
(...skipping 141 matching lines...) Expand 10 before | Expand all | Expand 10 after
152 EXPECT_EQ(kIdUuid, result.uuid()); 152 EXPECT_EQ(kIdUuid, result.uuid());
153 EXPECT_EQ(kDeviceSize, result.total_size_in_bytes()); 153 EXPECT_EQ(kDeviceSize, result.total_size_in_bytes());
154 EXPECT_EQ(DEVICE_TYPE_SD, result.device_type()); 154 EXPECT_EQ(DEVICE_TYPE_SD, result.device_type());
155 EXPECT_EQ(kMountPath, result.mount_path()); 155 EXPECT_EQ(kMountPath, result.mount_path());
156 } 156 }
157 157
158 TEST(CrosDisksClientTest, ComposeMountOptions) { 158 TEST(CrosDisksClientTest, ComposeMountOptions) {
159 std::string kExpectedMountLabelOption = 159 std::string kExpectedMountLabelOption =
160 std::string("mountlabel=") + kMountLabel; 160 std::string("mountlabel=") + kMountLabel;
161 std::vector<std::string> rw_mount_options = 161 std::vector<std::string> rw_mount_options =
162 CrosDisksClient::ComposeMountOptions(kMountLabel, 162 CrosDisksClient::ComposeMountOptions(
163 MOUNT_ACCESS_MODE_READ_WRITE); 163 kMountLabel, MOUNT_ACCESS_MODE_READ_WRITE, REMOUNT_OPTION_DISABLED);
164 ASSERT_EQ(5U, rw_mount_options.size()); 164 ASSERT_EQ(5U, rw_mount_options.size());
165 EXPECT_EQ("nodev", rw_mount_options[0]); 165 EXPECT_EQ("nodev", rw_mount_options[0]);
166 EXPECT_EQ("noexec", rw_mount_options[1]); 166 EXPECT_EQ("noexec", rw_mount_options[1]);
167 EXPECT_EQ("nosuid", rw_mount_options[2]); 167 EXPECT_EQ("nosuid", rw_mount_options[2]);
168 EXPECT_EQ("rw", rw_mount_options[3]); 168 EXPECT_EQ("rw", rw_mount_options[3]);
169 EXPECT_EQ(kExpectedMountLabelOption, rw_mount_options[4]); 169 EXPECT_EQ(kExpectedMountLabelOption, rw_mount_options[4]);
170 170
171 std::vector<std::string> ro_mount_options = 171 std::vector<std::string> ro_mount_options =
172 CrosDisksClient::ComposeMountOptions(kMountLabel, 172 CrosDisksClient::ComposeMountOptions(
173 MOUNT_ACCESS_MODE_READ_ONLY); 173 kMountLabel, MOUNT_ACCESS_MODE_READ_ONLY, REMOUNT_OPTION_DISABLED);
174 ASSERT_EQ(5U, ro_mount_options.size()); 174 ASSERT_EQ(5U, ro_mount_options.size());
175 EXPECT_EQ("nodev", ro_mount_options[0]); 175 EXPECT_EQ("nodev", ro_mount_options[0]);
176 EXPECT_EQ("noexec", ro_mount_options[1]); 176 EXPECT_EQ("noexec", ro_mount_options[1]);
177 EXPECT_EQ("nosuid", ro_mount_options[2]); 177 EXPECT_EQ("nosuid", ro_mount_options[2]);
178 EXPECT_EQ("ro", ro_mount_options[3]); 178 EXPECT_EQ("ro", ro_mount_options[3]);
179 EXPECT_EQ(kExpectedMountLabelOption, ro_mount_options[4]); 179 EXPECT_EQ(kExpectedMountLabelOption, ro_mount_options[4]);
180
181 std::vector<std::string> remount_mount_options =
182 CrosDisksClient::ComposeMountOptions(
183 kMountLabel, MOUNT_ACCESS_MODE_READ_WRITE, REMOUNT_OPTION_ENABLED);
184 ASSERT_EQ(6U, remount_mount_options.size());
185 EXPECT_EQ("nodev", remount_mount_options[0]);
186 EXPECT_EQ("noexec", remount_mount_options[1]);
187 EXPECT_EQ("nosuid", remount_mount_options[2]);
188 EXPECT_EQ("rw", remount_mount_options[3]);
189 EXPECT_EQ("remount", remount_mount_options[4]);
190 EXPECT_EQ(kExpectedMountLabelOption, remount_mount_options[5]);
180 } 191 }
181 192
182 } // namespace chromeos 193 } // namespace chromeos
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698