Fix tests
This commit is contained in:
parent
95a6d404b5
commit
5c7baab570
|
|
@ -34,7 +34,7 @@ class EdoniqUserExportTestCase(TestCase):
|
|||
|
||||
def test_fetch_course_session_users(self):
|
||||
users = fetch_course_session_users([COURSE_TEST_ID])
|
||||
self.assertEqual(len(users), 3)
|
||||
self.assertEqual(len(users), 2)
|
||||
|
||||
def test_response_csv(self):
|
||||
users = fetch_course_session_users([COURSE_TEST_ID])
|
||||
|
|
@ -42,17 +42,17 @@ class EdoniqUserExportTestCase(TestCase):
|
|||
|
||||
user_data = [
|
||||
[
|
||||
user.user.id,
|
||||
user.user.first_name,
|
||||
user.user.last_name,
|
||||
str(user.id),
|
||||
user.first_name,
|
||||
user.last_name,
|
||||
"m",
|
||||
user.user.language,
|
||||
user.user.email,
|
||||
user.user.additional_json_data.get("Geburtsdatum", ""),
|
||||
user.language,
|
||||
user.email,
|
||||
user.additional_json_data.get("Geburtsdatum", ""),
|
||||
"Lernende",
|
||||
user.user.additional_json_data.get("Firmenname", ""),
|
||||
user.user.additional_json_data.get("Lehrvertragsnummer", ""),
|
||||
user.course_session.import_id,
|
||||
user.additional_json_data.get("Firmenname", ""),
|
||||
user.additional_json_data.get("Lehrvertragsnummer", ""),
|
||||
user.coursesessionuser_set.first().course_session.import_id,
|
||||
"",
|
||||
]
|
||||
for user in users
|
||||
|
|
|
|||
|
|
@ -26,8 +26,7 @@ def export_trainers(request):
|
|||
|
||||
|
||||
def fetch_course_session_users(courses: List[int], role=CourseSessionUser.Role.MEMBER):
|
||||
# if a user is in multiple courses, he should be exported multiple times
|
||||
# todo: check if this is the case otherwise use .distinct("user")
|
||||
# if users should be exported per course session, remove the distinct() call
|
||||
return (
|
||||
User.objects.filter(
|
||||
coursesessionuser__course_session__course__id__in=courses,
|
||||
|
|
|
|||
Loading…
Reference in New Issue