From 2ce363abb0e3e720005e345a76cae3c969a073cf Mon Sep 17 00:00:00 2001 From: Jawayria Date: Mon, 1 Feb 2021 20:09:02 +0500 Subject: [PATCH 1/2] Applied pylint-amnesty to coursewarehistoryextended --- lms/djangoapps/coursewarehistoryextended/apps.py | 2 +- lms/djangoapps/coursewarehistoryextended/tests.py | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/lms/djangoapps/coursewarehistoryextended/apps.py b/lms/djangoapps/coursewarehistoryextended/apps.py index 91257be909..aaf50f2e96 100644 --- a/lms/djangoapps/coursewarehistoryextended/apps.py +++ b/lms/djangoapps/coursewarehistoryextended/apps.py @@ -1,4 +1,4 @@ -from django.apps import AppConfig +from django.apps import AppConfig # lint-amnesty, pylint: disable=missing-module-docstring class CoursewareHistoryExtendedConfig(AppConfig): diff --git a/lms/djangoapps/coursewarehistoryextended/tests.py b/lms/djangoapps/coursewarehistoryextended/tests.py index 1dedabae9d..47473c5260 100644 --- a/lms/djangoapps/coursewarehistoryextended/tests.py +++ b/lms/djangoapps/coursewarehistoryextended/tests.py @@ -22,10 +22,10 @@ from lms.djangoapps.courseware.tests.factories import StudentModuleFactory, cour class TestStudentModuleHistoryBackends(TestCase): """ Tests of data in CSMH and CSMHE """ # Tell Django to clean out all databases, not just default - databases = {alias for alias in connections} + databases = {alias for alias in connections} # lint-amnesty, pylint: disable=unnecessary-comprehension def setUp(self): - super(TestStudentModuleHistoryBackends, self).setUp() + super(TestStudentModuleHistoryBackends, self).setUp() # lint-amnesty, pylint: disable=super-with-arguments for record in (1, 2, 3): # This will store into CSMHE via the post_save signal csm = StudentModuleFactory.create(module_state_key=location('usage_id'), From 462158b3da4d8dc8217e9dbfed569ab0ce91edaf Mon Sep 17 00:00:00 2001 From: Jawayria Date: Wed, 3 Feb 2021 16:57:55 +0500 Subject: [PATCH 2/2] Resolved quality issue --- lms/djangoapps/coursewarehistoryextended/apps.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/lms/djangoapps/coursewarehistoryextended/apps.py b/lms/djangoapps/coursewarehistoryextended/apps.py index aaf50f2e96..2491566ce7 100644 --- a/lms/djangoapps/coursewarehistoryextended/apps.py +++ b/lms/djangoapps/coursewarehistoryextended/apps.py @@ -1,4 +1,5 @@ -from django.apps import AppConfig # lint-amnesty, pylint: disable=missing-module-docstring +# lint-amnesty, pylint: disable=missing-module-docstring +from django.apps import AppConfig class CoursewareHistoryExtendedConfig(AppConfig):