Fix #31, false positive with 'input' as variable name

CHANGELOG.md

@@ -5,6 +5,8 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
 ## [Unreleased]
+### Fixed
+- False positive with 'input' as variable name ([#31](https://github.com/duo-labs/dlint/issues/31))
 
 ## [0.9.0] - 2019-10-13
 ### Added

dlint/linters/bad_input_use.py

@@ -7,6 +7,7 @@
     unicode_literals,
 )
 
+import ast
 import sys
 
 from . import base
@@ -27,12 +28,13 @@ def __init__(self, *args, **kwargs):
 
         super(BadInputUseLinter, self).__init__(*args, **kwargs)
 
-    def visit_Name(self, node):
+    def visit_Call(self, node):
         is_python_2 = sys.version_info < (3, 0)
 
         if (is_python_2
                 and self.unsafe_input_import
-                and node.id == 'input'):
+                and isinstance(node.func, ast.Name)
+                and node.func.id == 'input'):
             self.results.append(
                 base.Flake8Result(
                     lineno=node.lineno,

tests/test_bad_input_use.py

@@ -92,6 +92,21 @@ def test_no_input_usage(self):
 
         assert result == expected
 
+    def test_bad_input_variable_usage(self):
+        python_node = self.get_ast_node(
+            """
+            input = 1
+            """
+        )
+
+        linter = dlint.linters.BadInputUseLinter()
+        linter.visit(python_node)
+
+        result = linter.get_results()
+        expected = []
+
+        assert result == expected
+
 
 if __name__ == "__main__":
     unittest.main()