include checking symlinks

Benjamin Chess · Benjamin Chess · commit a70abd04e7ce · 2016-01-15T12:50:04.000-08:00
diff --git a/pre_commit/git.py b/pre_commit/git.py
@@ -89,7 +89,7 @@ def wrapper(include_expr, exclude_expr):
             if (
                 include_regex.search(filename) and
                 not exclude_regex.search(filename) and
-                os.path.exists(filename)
+                os.path.lexists(filename)
             )
         )
     return wrapper
diff --git a/testing/test_symlink b/testing/test_symlink
@@ -0,0 +1 @@
+does_not_exist
diff --git a/tests/git_test.py b/tests/git_test.py
@@ -61,6 +61,7 @@ def get_filenames():
             'pre_commit/main.py',
             'pre_commit/git.py',
             'im_a_file_that_doesnt_exist.py',
+            'testing/test_symlink',
             'hooks.yaml',
         )
 
@@ -73,6 +74,7 @@ def test_get_files_matching_base(get_files_matching_func):
         'pre_commit/main.py',
         'pre_commit/git.py',
         'hooks.yaml',
+        'testing/test_symlink'
     ])
 
 
@@ -96,7 +98,7 @@ def test_does_not_include_deleted_fileS(get_files_matching_func):
 
 def test_exclude_removes_files(get_files_matching_func):
     ret = get_files_matching_func('', '\\.py$')
-    assert ret == set(['hooks.yaml'])
+    assert ret == set(['hooks.yaml', 'testing/test_symlink'])
 
 
 def resolve_conflict():

Original file line number	Diff line number	Diff line change
`@@ -89,7 +89,7 @@ def wrapper(include_expr, exclude_expr):`
`89`	`89`	`if (`
`90`	`90`	`include_regex.search(filename) and`
`91`	`91`	`not exclude_regex.search(filename) and`
`92`		`- os.path.exists(filename)`
	`92`	`+ os.path.lexists(filename)`
`93`	`93`	`)`
`94`	`94`	`)`
`95`	`95`	`return wrapper`