Commit 4ead0fd5 authored by Yusuke Sato's avatar Yusuke Sato Committed by Commit Bot

arcvm: Simplify unit tests

This is a follow-up CL for CL:2016290.

BUG=None
TEST=try

Change-Id: I7965ab61243b5de40c7364ce9678eb403303ffea
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2019803Reviewed-by: default avatarHidehiko Abe <hidehiko@chromium.org>
Commit-Queue: Yusuke Sato <yusukes@chromium.org>
Cr-Commit-Position: refs/heads/master@{#735293}
parent f818f212
......@@ -4,7 +4,6 @@
#include "components/arc/session/arc_vm_client_adapter.h"
#include <iterator>
#include <memory>
#include <string>
#include <utility>
......@@ -753,10 +752,7 @@ TEST_F(ArcVmClientAdapterTest, KernelParam_RO) {
// Check "rw" is not in |params|.
auto request = GetTestConciergeClient()->start_arc_vm_request();
const std::vector<std::string> params(
std::make_move_iterator(request.mutable_params()->begin()),
std::make_move_iterator(request.mutable_params()->end()));
EXPECT_FALSE(base::Contains(params, "rw"));
EXPECT_FALSE(base::Contains(request.params(), "rw"));
}
// Tests that the kernel parameter does include "rw" when '/' is writable and
......@@ -771,10 +767,7 @@ TEST_F(ArcVmClientAdapterTest, KernelParam_RW) {
// Check "rw" is in |params|.
auto request = GetTestConciergeClient()->start_arc_vm_request();
const std::vector<std::string> params(
std::make_move_iterator(request.mutable_params()->begin()),
std::make_move_iterator(request.mutable_params()->end()));
EXPECT_TRUE(base::Contains(params, "rw"));
EXPECT_TRUE(base::Contains(request.params(), "rw"));
}
// Tests that CreateArcVmClientAdapter(), the non-testing version, doesn't
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment