Fixing check_orphan_headers on Windows

tools_webrtc/presubmit_checks_lib/check_orphan_headers_test.py

Index: tools_webrtc/presubmit_checks_lib/check_orphan_headers_test.py
diff --git a/tools_webrtc/presubmit_checks_lib/check_orphan_headers_test.py b/tools_webrtc/presubmit_checks_lib/check_orphan_headers_test.py
index d6561b3d57370565b9d8edaad2125f416876cf1a..6587b8aa7456e0f4c76fef9471c4352ead563bc0 100755
--- a/tools_webrtc/presubmit_checks_lib/check_orphan_headers_test.py
+++ b/tools_webrtc/presubmit_checks_lib/check_orphan_headers_test.py
@@ -7,18 +7,34 @@
 # in the file PATENTS.  All contributing project authors may
 # be found in the AUTHORS file in the root of the source tree.
 
+import os
+import sys
 import unittest
 
 import check_orphan_headers
 
 
+def _GetRootBasedOnPlatform():
+  if sys.platform.startswith('win'):
+    return 'C:\\'
+  else:
+    return '/'
+
+
+def _GetPath(*path_chunks):
+  return os.path.join(_GetRootBasedOnPlatform(),
+                      *path_chunks)
+
+
 class GetBuildGnPathFromFilePathTest(unittest.TestCase):
 
   def testGetBuildGnFromSameDirectory(self):
-    file_path = '/home/projects/webrtc/base/foo.h'
-    expected_build_path = '/home/projects/webrtc/base/BUILD.gn'
-    file_exists = lambda p: p == '/home/projects/webrtc/base/BUILD.gn'
-    src_dir_path = '/home/projects/webrtc'
+    file_path = _GetPath('home', 'projects', 'webrtc', 'base', 'foo.h')
+    expected_build_path = _GetPath('home', 'projects', 'webrtc', 'base',
+                                   'BUILD.gn')
+    file_exists = lambda p: p == _GetPath('home', 'projects', 'webrtc',
+                                          'base', 'BUILD.gn')
+    src_dir_path = _GetPath('home', 'projects', 'webrtc')
     self.assertEqual(
       expected_build_path,
       check_orphan_headers.GetBuildGnPathFromFilePath(file_path,
@@ -26,10 +42,12 @@ class GetBuildGnPathFromFilePathTest(unittest.TestCase):
                                                       src_dir_path))
 
   def testGetBuildPathFromParentDirectory(self):
-    file_path = '/home/projects/webrtc/base/foo.h'
-    expected_build_path = '/home/projects/webrtc/BUILD.gn'
-    file_exists = lambda p: p == '/home/projects/webrtc/BUILD.gn'
-    src_dir_path = '/home/projects/webrtc'
+    file_path = _GetPath('home', 'projects', 'webrtc', 'base', 'foo.h')
+    expected_build_path = _GetPath('home', 'projects', 'webrtc',
+                                   'BUILD.gn')
+    file_exists = lambda p: p == _GetPath('home', 'projects', 'webrtc',
+                                          'BUILD.gn')
+    src_dir_path = _GetPath('home', 'projects', 'webrtc')
     self.assertEqual(
       expected_build_path,
       check_orphan_headers.GetBuildGnPathFromFilePath(file_path,
@@ -38,18 +56,18 @@ class GetBuildGnPathFromFilePathTest(unittest.TestCase):
 
   def testExceptionIfNoBuildGnFilesAreFound(self):
     with self.assertRaises(check_orphan_headers.NoBuildGnFoundError):
-      file_path = '/home/projects/webrtc/base/foo.h'
+      file_path = _GetPath('home', 'projects', 'webrtc', 'base', 'foo.h')
       file_exists = lambda p: False
-      src_dir_path = '/home/projects/webrtc'
+      src_dir_path = _GetPath('home', 'projects', 'webrtc')
       check_orphan_headers.GetBuildGnPathFromFilePath(file_path,
                                                       file_exists,
                                                       src_dir_path)
 
   def testExceptionIfFilePathIsNotAnHeader(self):
     with self.assertRaises(check_orphan_headers.WrongFileTypeError):
-      file_path = '/home/projects/webrtc/base/foo.cc'
+      file_path = _GetPath('home', 'projects', 'webrtc', 'base', 'foo.cc')
       file_exists = lambda p: False
-      src_dir_path = '/home/projects/webrtc'
+      src_dir_path = _GetPath('home', 'projects', 'webrtc')
       check_orphan_headers.GetBuildGnPathFromFilePath(file_path,
                                                       file_exists,
                                                       src_dir_path)
@@ -84,10 +102,15 @@ class GetHeadersInBuildGnFileSourcesTest(unittest.TestCase):
       sources = ["baz/foo.h"]
     }
     """
+    target_abs_path = _GetPath('a', 'b')
     self.assertEqual(
-      set(['/a/b/foo.h', '/a/b/bar.h', '/a/b/baz/foo.h']),
+      set([
+        _GetPath('a', 'b', 'foo.h'),
+        _GetPath('a', 'b', 'bar.h'),
+        _GetPath('a', 'b', 'baz', 'foo.h'),
+      ]),
       check_orphan_headers.GetHeadersInBuildGnFileSources(file_content,
-                                                          '/a/b'))
+                                                          target_abs_path))
 
 
 if __name__ == '__main__':