Newer pylint test versions expect additional columns in expected test outputs, so adding these in. Also, fixing the consider-using-f-string warnings from tests (#338)

Author: carlio

.gitignore

@@ -34,6 +34,7 @@ nosetests.xml
 .project
 .pydevproject
 .env
+.idea
 env.txt
 .venv
 

pylint_django/tests/input/external_factory_boy_noerror.py

@@ -2,7 +2,7 @@
 Test to validate that pylint_django doesn't produce
 Instance of 'SubFactory' has no 'pk' member (no-member) warnings
 """
-# pylint: disable=attribute-defined-outside-init, missing-docstring, too-few-public-methods
+# pylint: disable=attribute-defined-outside-init, missing-docstring, too-few-public-methods, consider-using-f-string
 import factory
 
 from django import test

pylint_django/tests/input/func_hard_coded_auth_user.txt

@@ -1,3 +1,3 @@
-imported-auth-user:5:0::User model imported from django.contrib.auth.models
-imported-auth-user:6:0::User model imported from django.contrib.auth.models
-hard-coded-auth-user:10:31:PullRequest:Hard-coded 'auth.User'
+imported-auth-user:5:0:5:40::User model imported from django.contrib.auth.models:UNDEFINED
+imported-auth-user:6:0:6:43::User model imported from django.contrib.auth.models:UNDEFINED
+hard-coded-auth-user:10:31:10:42:PullRequest:Hard-coded 'auth.User':UNDEFINED

pylint_django/tests/input/func_json_response.txt

@@ -1,4 +1,4 @@
-http-response-with-json-dumps:9:11:say_yes:Instead of HttpResponse(json.dumps(data)) use JsonResponse(data)
-http-response-with-json-dumps:14:11:say_yes2:Instead of HttpResponse(json.dumps(data)) use JsonResponse(data)
-redundant-content-type-for-json-response:23:11:redundant_content_type:Redundant content_type parameter for JsonResponse()
-http-response-with-content-type-json:28:11:content_type_json:Instead of HttpResponse(content_type='application/json') use JsonResponse()
+http-response-with-json-dumps:9:11:9:64:say_yes:Instead of HttpResponse(json.dumps(data)) use JsonResponse(data):UNDEFINED
+http-response-with-json-dumps:14:11:14:46:say_yes2:Instead of HttpResponse(json.dumps(data)) use JsonResponse(data):UNDEFINED
+redundant-content-type-for-json-response:23:11:23:67:redundant_content_type:Redundant content_type parameter for JsonResponse():UNDEFINED
+http-response-with-content-type-json:28:11:28:67:content_type_json:Instead of HttpResponse(content_type='application/json') use JsonResponse():UNDEFINED

pylint_django/tests/input/func_model_does_not_use_unicode_py33.txt

@@ -1 +1 @@
-model-has-unicode:9:0:SomeModel:Found __unicode__ method on model (SomeModel). Python3 uses __str__.
+model-has-unicode:9:0:19:29:SomeModel:Found __unicode__ method on model (SomeModel). Python3 uses __str__.:UNDEFINED

pylint_django/tests/input/func_model_no_explicit_unicode_str_compat.txt

@@ -1 +1 @@
-model-no-explicit-unicode:18:0:SomeModel:Model does not explicitly define __unicode__ (SomeModel)
+model-no-explicit-unicode:18:0:19:8:SomeModel:Model does not explicitly define __unicode__ (SomeModel):UNDEFINED

pylint_django/tests/input/func_modelform_exclude.txt

@@ -1 +1 @@
-modelform-uses-exclude:10:8:PersonForm.Meta:Use explicit fields instead of exclude in ModelForm
+modelform-uses-exclude:10:8:10:28:PersonForm.Meta:Use explicit fields instead of exclude in ModelForm:UNDEFINED

pylint_django/tests/input/func_noerror_foreign_key_attributes.py

@@ -1,7 +1,7 @@
 """
 Checks that Pylint does not complain about foreign key sets on models
 """
-#  pylint: disable=missing-docstring
+#  pylint: disable=missing-docstring consider-using-f-string
 
 from django.db import models
 

pylint_django/tests/input/func_unused_arguments.txt

@@ -1,2 +1,2 @@
-unused-argument:18:25:user_detail:Unused argument 'user_id':HIGH
-unused-argument:24:27:UserView.get:Unused argument 'user_id':INFERENCE
+unused-argument:18:25:18:32:user_detail:Unused argument 'user_id':HIGH
+unused-argument:24:27:24:34:UserView.get:Unused argument 'user_id':INFERENCE

pylint_django/tests/input/migrations/0002_new_column.txt

@@ -1 +1 @@
-new-db-field-with-default:28:8:Migration:pylint_django.tests.input.migrations.0002_new_column AddField with default value
+new-db-field-with-default:28:8:32:9:Migration:pylint_django.tests.input.migrations.0002_new_column AddField with default value:UNDEFINED

pylint_django/tests/input/migrations/0003_without_backwards.txt

@@ -1,4 +1,4 @@
-missing-backwards-migration-callable:12:8:Migration:Always include backwards migration callable
-missing-backwards-migration-callable:13:8:Migration:Always include backwards migration callable
-missing-backwards-migration-callable:15:8:Migration:Always include backwards migration callable
-missing-backwards-migration-callable:17:8:Migration:Always include backwards migration callable
+missing-backwards-migration-callable:12:8:12:30:Migration:Always include backwards migration callable:UNDEFINED
+missing-backwards-migration-callable:13:8:14:26:Migration:Always include backwards migration callable:UNDEFINED
+missing-backwards-migration-callable:15:8:16:31:Migration:Always include backwards migration callable:UNDEFINED
+missing-backwards-migration-callable:17:8:18:45:Migration:Always include backwards migration callable:UNDEFINED