i dont think get_link_path should normcase paths

pekkaklarck · pekkaklarck · commit aed93cdc2843 · 2010-12-09T16:55:03.000Z
diff --git a/src/robot/utils/misc.py b/src/robot/utils/misc.py
@@ -114,8 +114,8 @@ def get_link_path(target, base):
     return url.replace('%5C', '/').replace('%3A', ':').replace('|', ':')
 
 def _get_pathname(target, base):
-    target = normpath(target)
-    base = normpath(base)
+    target = normpath(target, normcase=False)
+    base = normpath(base, normcase=False)
     if os.path.isfile(base):
         base = os.path.dirname(base)
     if base == target:
diff --git a/utest/utils/test_misc.py b/utest/utils/test_misc.py
@@ -54,15 +54,12 @@ def test_get_link_path(self):
                 ( 'c:\\path\\to', 'c:\\path\\to\\result_in_same_dir.html', 'result_in_same_dir.html' ),
                 ( 'c:\\path\\to\\dir', 'c:\\path\\to\\result_in_parent_dir.html', '../result_in_parent_dir.html' ),
                 ( 'c:\\path\\to', 'c:\\path\\to\\dir\\result_in_sub_dir.html', 'dir/result_in_sub_dir.html' ),
-                ( 'c:\\commonprefix\\sucks\\baR', 'c:\\commonprefix\\sucks\\baZ.txt', '../baz.txt' ),
+                ( 'c:\\commonprefix\\sucks\\baR', 'c:\\commonprefix\\sucks\\baZ.txt', '../baZ.txt' ),
                 ( 'c:\\a\\very\\long\\path', 'c:\\no\\depth\\limitation', '../../../../no/depth/limitation' ),
                 ( 'c:\\windows\\explorer.exe', 'c:\\windows\\path\\to\\existing\\file', 'path/to/existing/file' ),
                 ( 'c:\\path\\to\\identity', 'c:\\path\\to\\identity', 'identity' ),
             ]
-        import robot.utils.normalizing
         for basedir, target, expected in inputs:
-            if robot.utils.normalizing._CASE_INSENSITIVE_FILESYSTEM :
-                expected = expected.lower()
             assert_equals(get_link_path(target, basedir).replace('R:', 'r:'), expected,
                          '%s -> %s' % (target, basedir))
 
