diff options
author | evan@chromium.org <evan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-08-22 18:48:05 +0000 |
---|---|---|
committer | evan@chromium.org <evan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-08-22 18:48:05 +0000 |
commit | a33efe35a17e107a9b18b5d9d1c430ddb780f942 (patch) | |
tree | ca0fd0f38ef17eaba79b0d4b405d508068a4dfbe /net/test | |
parent | d1b4b4593ef1f2416a7d1362af3cd8ed99e245ac (diff) | |
download | chromium_src-a33efe35a17e107a9b18b5d9d1c430ddb780f942.zip chromium_src-a33efe35a17e107a9b18b5d9d1c430ddb780f942.tar.gz chromium_src-a33efe35a17e107a9b18b5d9d1c430ddb780f942.tar.bz2 |
Fix sync integration tests by fixing Python path.
The generated Python proto output paths now match the input paths.
Review URL: http://codereview.chromium.org/7711005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@97696 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/test')
-rw-r--r-- | net/test/python_utils.cc | 2 | ||||
-rw-r--r-- | net/test/test_server.cc | 14 |
2 files changed, 10 insertions, 6 deletions
diff --git a/net/test/python_utils.cc b/net/test/python_utils.cc index 0eec816..f4fb487 100644 --- a/net/test/python_utils.cc +++ b/net/test/python_utils.cc @@ -16,6 +16,8 @@ const char kPythonPathEnv[] = "PYTHONPATH"; void AppendToPythonPath(const FilePath& dir) { + CHECK(file_util::DirectoryExists(dir)); + scoped_ptr<base::Environment> env(base::Environment::Create()); std::string old_path; std::string dir_path; diff --git a/net/test/test_server.cc b/net/test/test_server.cc index 8723215..d76430d 100644 --- a/net/test/test_server.cc +++ b/net/test/test_server.cc @@ -304,17 +304,19 @@ bool TestServer::SetPythonPath() { third_party_dir.AppendASCII("pyftpdlib").AppendASCII("src")); // Locate the Python code generated by the protocol buffers compiler. - FilePath pyproto_code_dir; - if (!GetPyProtoPath(&pyproto_code_dir)) { + FilePath pyproto_dir; + if (!GetPyProtoPath(&pyproto_dir)) { LOG(WARNING) << "Cannot find pyproto dir for generated code. " << "Testserver features that rely on it will not work"; return true; } - AppendToPythonPath(pyproto_code_dir); - AppendToPythonPath(pyproto_code_dir.Append(FILE_PATH_LITERAL("sync_pb"))); - AppendToPythonPath(pyproto_code_dir.Append( - FILE_PATH_LITERAL("device_management_pb"))); + AppendToPythonPath(pyproto_dir); + AppendToPythonPath(pyproto_dir.AppendASCII("chrome") + .AppendASCII("browser") + .AppendASCII("sync") + .AppendASCII("protocol")); + AppendToPythonPath(pyproto_dir.AppendASCII("device_management_pb")); return true; } |