From 8aa2970c519cc5fc59a0f003f8b5967964d1a802 Mon Sep 17 00:00:00 2001 From: usamasadiq Date: Mon, 13 Oct 2025 22:53:37 +0500 Subject: [PATCH] fix: fix pycodestyle error --- .../contentstore/views/tests/test_block.py | 8 +- .../djangoapps/student/tests/test_events.py | 150 +++++++++--------- .../certificates/tests/test_events.py | 108 ++++++------- lms/djangoapps/grades/tests/test_events.py | 90 +++++------ .../course_groups/tests/test_events.py | 28 ++-- .../user_authn/views/tests/test_events.py | 32 ++-- .../tests/test_mixed_modulestore.py | 16 +- 7 files changed, 216 insertions(+), 216 deletions(-) diff --git a/cms/djangoapps/contentstore/views/tests/test_block.py b/cms/djangoapps/contentstore/views/tests/test_block.py index e01263259b..2b21a9b9b9 100644 --- a/cms/djangoapps/contentstore/views/tests/test_block.py +++ b/cms/djangoapps/contentstore/views/tests/test_block.py @@ -870,10 +870,10 @@ class TestDuplicateItem(ItemTest, DuplicateHelper, OpenEdxEventsTestMixin): "signal": XBLOCK_DUPLICATED, "sender": None, "xblock_info": DuplicatedXBlockData( - usage_key=usage_key, - block_type=usage_key.block_type, - source_usage_key=self.vert_usage_key, - ), + usage_key=usage_key, + block_type=usage_key.block_type, + source_usage_key=self.vert_usage_key, + ), }, event_receiver.call_args.kwargs, ) diff --git a/common/djangoapps/student/tests/test_events.py b/common/djangoapps/student/tests/test_events.py index 9d57c8ca85..7c9d625118 100644 --- a/common/djangoapps/student/tests/test_events.py +++ b/common/djangoapps/student/tests/test_events.py @@ -278,23 +278,23 @@ class EnrollmentEventsTest(SharedModuleStoreTestCase, OpenEdxEventsTestMixin): "signal": COURSE_ENROLLMENT_CREATED, "sender": None, "enrollment": CourseEnrollmentData( - user=UserData( - pii=UserPersonalData( - username=self.user.username, - email=self.user.email, - name=self.user.profile.name, - ), - id=self.user.id, - is_active=self.user.is_active, - ), - course=CourseData( - course_key=self.course.id, - display_name=self.course.display_name, - ), - mode=enrollment.mode, - is_active=enrollment.is_active, - creation_date=enrollment.created, - ), + user=UserData( + pii=UserPersonalData( + username=self.user.username, + email=self.user.email, + name=self.user.profile.name, + ), + id=self.user.id, + is_active=self.user.is_active, + ), + course=CourseData( + course_key=self.course.id, + display_name=self.course.display_name, + ), + mode=enrollment.mode, + is_active=enrollment.is_active, + creation_date=enrollment.created, + ), }, event_receiver.call_args.kwargs, ) @@ -322,23 +322,23 @@ class EnrollmentEventsTest(SharedModuleStoreTestCase, OpenEdxEventsTestMixin): "signal": COURSE_ENROLLMENT_CHANGED, "sender": None, "enrollment": CourseEnrollmentData( - user=UserData( - pii=UserPersonalData( - username=self.user.username, - email=self.user.email, - name=self.user.profile.name, - ), - id=self.user.id, - is_active=self.user.is_active, - ), - course=CourseData( - course_key=self.course.id, - display_name=self.course.display_name, - ), - mode=enrollment.mode, - is_active=enrollment.is_active, - creation_date=enrollment.created, - ), + user=UserData( + pii=UserPersonalData( + username=self.user.username, + email=self.user.email, + name=self.user.profile.name, + ), + id=self.user.id, + is_active=self.user.is_active, + ), + course=CourseData( + course_key=self.course.id, + display_name=self.course.display_name, + ), + mode=enrollment.mode, + is_active=enrollment.is_active, + creation_date=enrollment.created, + ), }, event_receiver.call_args.kwargs, ) @@ -366,23 +366,23 @@ class EnrollmentEventsTest(SharedModuleStoreTestCase, OpenEdxEventsTestMixin): "signal": COURSE_UNENROLLMENT_COMPLETED, "sender": None, "enrollment": CourseEnrollmentData( - user=UserData( - pii=UserPersonalData( - username=self.user.username, - email=self.user.email, - name=self.user.profile.name, - ), - id=self.user.id, - is_active=self.user.is_active, - ), - course=CourseData( - course_key=self.course.id, - display_name=self.course.display_name, - ), - mode=enrollment.mode, - is_active=False, - creation_date=enrollment.created, - ), + user=UserData( + pii=UserPersonalData( + username=self.user.username, + email=self.user.email, + name=self.user.profile.name, + ), + id=self.user.id, + is_active=self.user.is_active, + ), + course=CourseData( + course_key=self.course.id, + display_name=self.course.display_name, + ), + mode=enrollment.mode, + is_active=False, + creation_date=enrollment.created, + ), }, event_receiver.call_args.kwargs, ) @@ -440,18 +440,18 @@ class TestCourseAccessRoleEvents(TestCase, OpenEdxEventsTestMixin): "signal": COURSE_ACCESS_ROLE_ADDED, "sender": None, "course_access_role_data": CourseAccessRoleData( - user=UserData( - pii=UserPersonalData( - username=self.user.username, - email=self.user.email, - ), - id=self.user.id, - is_active=self.user.is_active, - ), - course_key=self.course_key, - org_key=self.course_key.org, - role=role._role_name, # pylint: disable=protected-access - ), + user=UserData( + pii=UserPersonalData( + username=self.user.username, + email=self.user.email, + ), + id=self.user.id, + is_active=self.user.is_active, + ), + course_key=self.course_key, + org_key=self.course_key.org, + role=role._role_name, # pylint: disable=protected-access + ), }, event_receiver.call_args.kwargs, ) @@ -479,18 +479,18 @@ class TestCourseAccessRoleEvents(TestCase, OpenEdxEventsTestMixin): "signal": COURSE_ACCESS_ROLE_REMOVED, "sender": None, "course_access_role_data": CourseAccessRoleData( - user=UserData( - pii=UserPersonalData( - username=self.user.username, - email=self.user.email, - ), - id=self.user.id, - is_active=self.user.is_active, - ), - course_key=self.course_key, - org_key=self.course_key.org, - role=role._role_name, # pylint: disable=protected-access - ), + user=UserData( + pii=UserPersonalData( + username=self.user.username, + email=self.user.email, + ), + id=self.user.id, + is_active=self.user.is_active, + ), + course_key=self.course_key, + org_key=self.course_key.org, + role=role._role_name, # pylint: disable=protected-access + ), }, event_receiver.call_args.kwargs, ) diff --git a/lms/djangoapps/certificates/tests/test_events.py b/lms/djangoapps/certificates/tests/test_events.py index fb890c53dd..eb1f215e49 100644 --- a/lms/djangoapps/certificates/tests/test_events.py +++ b/lms/djangoapps/certificates/tests/test_events.py @@ -101,24 +101,24 @@ class CertificateEventTest(SharedModuleStoreTestCase, OpenEdxEventsTestMixin): "signal": CERTIFICATE_CREATED, "sender": None, "certificate": CertificateData( - user=UserData( - pii=UserPersonalData( - username=certificate.user.username, - email=certificate.user.email, - name=certificate.user.profile.name, - ), - id=certificate.user.id, - is_active=certificate.user.is_active, - ), - course=CourseData( - course_key=certificate.course_id, - ), - mode=certificate.mode, - grade=certificate.grade, - current_status=certificate.status, - download_url=certificate.download_url, - name=certificate.name, - ), + user=UserData( + pii=UserPersonalData( + username=certificate.user.username, + email=certificate.user.email, + name=certificate.user.profile.name, + ), + id=certificate.user.id, + is_active=certificate.user.is_active, + ), + course=CourseData( + course_key=certificate.course_id, + ), + mode=certificate.mode, + grade=certificate.grade, + current_status=certificate.status, + download_url=certificate.download_url, + name=certificate.name, + ), }, event_receiver.call_args.kwargs, ) @@ -155,24 +155,24 @@ class CertificateEventTest(SharedModuleStoreTestCase, OpenEdxEventsTestMixin): "signal": CERTIFICATE_CHANGED, "sender": None, "certificate": CertificateData( - user=UserData( - pii=UserPersonalData( - username=certificate.user.username, - email=certificate.user.email, - name=certificate.user.profile.name, - ), - id=certificate.user.id, - is_active=certificate.user.is_active, - ), - course=CourseData( - course_key=certificate.course_id, - ), - mode=certificate.mode, - grade=certificate.grade, - current_status=certificate.status, - download_url=certificate.download_url, - name=certificate.name, - ), + user=UserData( + pii=UserPersonalData( + username=certificate.user.username, + email=certificate.user.email, + name=certificate.user.profile.name, + ), + id=certificate.user.id, + is_active=certificate.user.is_active, + ), + course=CourseData( + course_key=certificate.course_id, + ), + mode=certificate.mode, + grade=certificate.grade, + current_status=certificate.status, + download_url=certificate.download_url, + name=certificate.name, + ), }, event_receiver.call_args.kwargs, ) @@ -208,24 +208,24 @@ class CertificateEventTest(SharedModuleStoreTestCase, OpenEdxEventsTestMixin): "signal": CERTIFICATE_REVOKED, "sender": None, "certificate": CertificateData( - user=UserData( - pii=UserPersonalData( - username=certificate.user.username, - email=certificate.user.email, - name=certificate.user.profile.name, - ), - id=certificate.user.id, - is_active=certificate.user.is_active, - ), - course=CourseData( - course_key=certificate.course_id, - ), - mode=certificate.mode, - grade=certificate.grade, - current_status=certificate.status, - download_url=certificate.download_url, - name=certificate.name, - ), + user=UserData( + pii=UserPersonalData( + username=certificate.user.username, + email=certificate.user.email, + name=certificate.user.profile.name, + ), + id=certificate.user.id, + is_active=certificate.user.is_active, + ), + course=CourseData( + course_key=certificate.course_id, + ), + mode=certificate.mode, + grade=certificate.grade, + current_status=certificate.status, + download_url=certificate.download_url, + name=certificate.name, + ), }, event_receiver.call_args.kwargs, ) diff --git a/lms/djangoapps/grades/tests/test_events.py b/lms/djangoapps/grades/tests/test_events.py index 93c440bdb5..4b3063265a 100644 --- a/lms/djangoapps/grades/tests/test_events.py +++ b/lms/djangoapps/grades/tests/test_events.py @@ -97,17 +97,17 @@ class PersistentGradeEventsTest(SharedModuleStoreTestCase, OpenEdxEventsTestMixi "signal": PERSISTENT_GRADE_SUMMARY_CHANGED, "sender": None, "grade": PersistentCourseGradeData( - user_id=self.params["user_id"], - course=CourseData( - course_key=self.params["course_id"], - ), - course_edited_timestamp=self.params["course_edited_timestamp"], - course_version=self.params["course_version"], - grading_policy_hash='', - percent_grade=self.params["percent_grade"], - letter_grade=self.params["letter_grade"], - passed_timestamp=grade.passed_timestamp - ) + user_id=self.params["user_id"], + course=CourseData( + course_key=self.params["course_id"], + ), + course_edited_timestamp=self.params["course_edited_timestamp"], + course_version=self.params["course_version"], + grading_policy_hash='', + percent_grade=self.params["percent_grade"], + letter_grade=self.params["letter_grade"], + passed_timestamp=grade.passed_timestamp + ) }, event_receiver.call_args.kwargs, ) @@ -159,20 +159,20 @@ class CoursePassingStatusEventsTest(SharedModuleStoreTestCase, OpenEdxEventsTest "signal": COURSE_PASSING_STATUS_UPDATED, "sender": None, "course_passing_status": CoursePassingStatusData( - is_passing=True, - user=UserData( - pii=UserPersonalData( - username=self.user.username, - email=self.user.email, - name=self.user.get_full_name(), - ), - id=self.user.id, - is_active=self.user.is_active, - ), - course=CourseData( - course_key=self.course.id, - ), - ), + is_passing=True, + user=UserData( + pii=UserPersonalData( + username=self.user.username, + email=self.user.email, + name=self.user.get_full_name(), + ), + id=self.user.id, + is_active=self.user.is_active, + ), + course=CourseData( + course_key=self.course.id, + ), + ), }, event_receiver.call_args.kwargs, ) @@ -233,26 +233,26 @@ class CCXCoursePassingStatusEventsTest( "signal": CCX_COURSE_PASSING_STATUS_UPDATED, "sender": None, "course_passing_status": CcxCoursePassingStatusData( - is_passing=True, - user=UserData( - pii=UserPersonalData( - username=self.user.username, - email=self.user.email, - name=self.user.get_full_name(), - ), - id=self.user.id, - is_active=self.user.is_active, - ), - course=CcxCourseData( - ccx_course_key=self.ccx_locator, - master_course_key=self.course.id, - display_name="", - coach_email="", - start=None, - end=None, - max_students_allowed=self.ccx.max_student_enrollments_allowed, - ), - ), + is_passing=True, + user=UserData( + pii=UserPersonalData( + username=self.user.username, + email=self.user.email, + name=self.user.get_full_name(), + ), + id=self.user.id, + is_active=self.user.is_active, + ), + course=CcxCourseData( + ccx_course_key=self.ccx_locator, + master_course_key=self.course.id, + display_name="", + coach_email="", + start=None, + end=None, + max_students_allowed=self.ccx.max_student_enrollments_allowed, + ), + ), }, event_receiver.call_args.kwargs, ) diff --git a/openedx/core/djangoapps/course_groups/tests/test_events.py b/openedx/core/djangoapps/course_groups/tests/test_events.py index 40a04dea8c..f11e95088f 100644 --- a/openedx/core/djangoapps/course_groups/tests/test_events.py +++ b/openedx/core/djangoapps/course_groups/tests/test_events.py @@ -97,20 +97,20 @@ class CohortEventTest(SharedModuleStoreTestCase, OpenEdxEventsTestMixin): "signal": COHORT_MEMBERSHIP_CHANGED, "sender": None, "cohort": CohortData( - user=UserData( - pii=UserPersonalData( - username=cohort_membership.user.username, - email=cohort_membership.user.email, - name=cohort_membership.user.profile.name, - ), - id=cohort_membership.user.id, - is_active=cohort_membership.user.is_active, - ), - course=CourseData( - course_key=cohort_membership.course_id, - ), - name=cohort_membership.course_user_group.name, - ), + user=UserData( + pii=UserPersonalData( + username=cohort_membership.user.username, + email=cohort_membership.user.email, + name=cohort_membership.user.profile.name, + ), + id=cohort_membership.user.id, + is_active=cohort_membership.user.is_active, + ), + course=CourseData( + course_key=cohort_membership.course_id, + ), + name=cohort_membership.course_user_group.name, + ), }, event_receiver.call_args.kwargs, ) diff --git a/openedx/core/djangoapps/user_authn/views/tests/test_events.py b/openedx/core/djangoapps/user_authn/views/tests/test_events.py index 4460e8c627..7efd4e4cf5 100644 --- a/openedx/core/djangoapps/user_authn/views/tests/test_events.py +++ b/openedx/core/djangoapps/user_authn/views/tests/test_events.py @@ -90,14 +90,14 @@ class RegistrationEventTest(UserAPITestCase, OpenEdxEventsTestMixin): "signal": STUDENT_REGISTRATION_COMPLETED, "sender": None, "user": UserData( - pii=UserPersonalData( - username=user.username, - email=user.email, - name=user.profile.name, - ), - id=user.id, - is_active=user.is_active, - ), + pii=UserPersonalData( + username=user.username, + email=user.email, + name=user.profile.name, + ), + id=user.id, + is_active=user.is_active, + ), }, event_receiver.call_args.kwargs, ) @@ -173,14 +173,14 @@ class LoginSessionEventTest(UserAPITestCase, OpenEdxEventsTestMixin): "signal": SESSION_LOGIN_COMPLETED, "sender": None, "user": UserData( - pii=UserPersonalData( - username=user.username, - email=user.email, - name=user.profile.name, - ), - id=user.id, - is_active=user.is_active, - ), + pii=UserPersonalData( + username=user.username, + email=user.email, + name=user.profile.name, + ), + id=user.id, + is_active=user.is_active, + ), }, event_receiver.call_args.kwargs, ) diff --git a/xmodule/modulestore/tests/test_mixed_modulestore.py b/xmodule/modulestore/tests/test_mixed_modulestore.py index a7fafaf8ad..f82d1e57c5 100644 --- a/xmodule/modulestore/tests/test_mixed_modulestore.py +++ b/xmodule/modulestore/tests/test_mixed_modulestore.py @@ -820,8 +820,8 @@ class TestMixedModuleStore(CommonMixedModuleStoreSetup): "signal": COURSE_CREATED, "sender": None, "course": CourseData( - course_key=test_course.id, - ), + course_key=test_course.id, + ), }, event_receiver.call_args.kwargs, ) @@ -899,9 +899,9 @@ class TestMixedModuleStore(CommonMixedModuleStoreSetup): "signal": XBLOCK_PUBLISHED, "sender": None, "xblock_info": XBlockData( - usage_key=sequential.location, - block_type=sequential.location.block_type, - ), + usage_key=sequential.location, + block_type=sequential.location.block_type, + ), }, event_receiver.call_args.kwargs, ) @@ -934,9 +934,9 @@ class TestMixedModuleStore(CommonMixedModuleStoreSetup): "signal": XBLOCK_DELETED, "sender": None, "xblock_info": XBlockData( - usage_key=vertical.location, - block_type=vertical.location.block_type, - ), + usage_key=vertical.location, + block_type=vertical.location.block_type, + ), }, event_receiver.call_args.kwargs, )