code cleanup and improvements after suggestions

Author: imomaliev

CHANGELOG.rst

@@ -4,7 +4,7 @@ Changelog
 Version 2.0.9 (dev)
 -----------------------------
 
-- Fix ``UnboundLocalError: local variable 'key_cls' referenced before assignment`` for FK when using ``to`` as string pointing to model when models is a package
+- Fix ``UnboundLocalError: local variable 'key_cls' referenced before assignment`` for cases when models is a python package, the ``to`` argument is a string that is used in this pattern ``app.Model`` and also there is some other ``bool`` const like ``null=True`` right after ``to``. (Sardorbek Imomaliev)
 
 Version 2.0.8 (18 April 2019)
 -----------------------------

pylint_django/tests/input/models/func_noerror_foreign_key_key_cls_unbound.py renamed to pylint_django/tests/input/func_noerror_foreign_key_key_cls_unbound.py

@@ -2,7 +2,7 @@
 Checks that Pylint does not complain about ForeignKey pointing to model
 in module of models package
 """
-# pylint: disable=missing-docstring,wrong-import-position
+# pylint: disable=missing-docstring
 from django.db import models
 
 

pylint_django/tests/input/models/func_noerror_foreign_key_package.py renamed to pylint_django/tests/input/func_noerror_foreign_key_package.py

@@ -2,7 +2,7 @@
 Checks that Pylint does not complain about ForeignKey pointing to model
 in module of models package
 """
-# pylint: disable=missing-docstring,wrong-import-position
+# pylint: disable=missing-docstring
 from django.db import models
 
 

pylint_django/tests/test_func.py

@@ -56,7 +56,6 @@ def _file_name(test):
 
 
 TESTS = get_tests()
-TESTS.extend(get_tests('input/models'))
 TESTS_NAMES = [t.base for t in TESTS]