diff options
| author | tfarina@chromium.org <tfarina@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-16 03:34:25 +0000 |
|---|---|---|
| committer | tfarina@chromium.org <tfarina@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-16 03:34:25 +0000 |
| commit | e9032c6a2ea03e4a5081e0234eff857cf190e3e4 (patch) | |
| tree | a9e6ec385829730bc9442579c2fef525d1caf71e /base/env_var_unittest.cc | |
| parent | 3e108342206210d64d81368044d9503445a1a1f3 (diff) | |
| download | chromium_src-e9032c6a2ea03e4a5081e0234eff857cf190e3e4.zip chromium_src-e9032c6a2ea03e4a5081e0234eff857cf190e3e4.tar.gz chromium_src-e9032c6a2ea03e4a5081e0234eff857cf190e3e4.tar.bz2 | |
base: Return bool instead of void in EnvVarGetter::SetEnv.
Returning bool instead of void in this function is better, because with that
we can know if we had succeeded when tried to set an environment variable or not.
Also it makes this function more testable.
BUG=None
TEST=out/Debug/base_unittests --gtest_filter=EnvVarTest.*
Signed-off-by: Thiago Farina <tfarina@chromium.org>
Review URL: http://codereview.chromium.org/2891024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52602 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'base/env_var_unittest.cc')
| -rw-r--r-- | base/env_var_unittest.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/base/env_var_unittest.cc b/base/env_var_unittest.cc index 5587878..d80d997 100644 --- a/base/env_var_unittest.cc +++ b/base/env_var_unittest.cc @@ -27,7 +27,7 @@ TEST_F(EnvVarTest, SetEnvVar) { const char kFooUpper[] = "FOO"; const char kFooLower[] = "foo"; scoped_ptr<base::EnvVarGetter> env(base::EnvVarGetter::Create()); - env->SetEnv(kFooUpper, kFooLower); + EXPECT_TRUE(env->SetEnv(kFooUpper, kFooLower)); // Now verify that the environment has the new variable. EXPECT_TRUE(env->HasEnv(kFooUpper)); |