Merged in bugfix/VBV-644-trainer-vorschau-read-only (pull request #288)
fix: allow learning content for trainers Approved-by: Christian Cueni
This commit is contained in:
commit
a76e23743b
|
|
@ -209,10 +209,16 @@ def can_view_course_completions(
|
||||||
)
|
)
|
||||||
|
|
||||||
|
|
||||||
|
def can_complete_learning_content(user: User, course_session_id: int) -> bool:
|
||||||
|
return is_course_session_member(
|
||||||
|
user, course_session_id
|
||||||
|
) or is_course_session_expert(user, course_session_id)
|
||||||
|
|
||||||
|
|
||||||
def course_session_permissions(user: User, course_session_id: int) -> list[str]:
|
def course_session_permissions(user: User, course_session_id: int) -> list[str]:
|
||||||
return _action_list(
|
return _action_list(
|
||||||
{
|
{
|
||||||
"complete-learning-content": is_course_session_member(
|
"complete-learning-content": can_complete_learning_content(
|
||||||
user, course_session_id
|
user, course_session_id
|
||||||
),
|
),
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -33,12 +33,21 @@ class ActionTestCase(TestCase):
|
||||||
role=CourseSessionUser.Role.MEMBER,
|
role=CourseSessionUser.Role.MEMBER,
|
||||||
)
|
)
|
||||||
|
|
||||||
|
trainer = create_user("trainer")
|
||||||
|
add_course_session_user(
|
||||||
|
self.course_session,
|
||||||
|
trainer,
|
||||||
|
role=CourseSessionUser.Role.EXPERT,
|
||||||
|
)
|
||||||
|
|
||||||
# WHEN
|
# WHEN
|
||||||
mentor_actions = course_session_permissions(lm, self.course_session.id)
|
mentor_actions = course_session_permissions(lm, self.course_session.id)
|
||||||
participant_actions = course_session_permissions(
|
participant_actions = course_session_permissions(
|
||||||
participant, self.course_session.id
|
participant, self.course_session.id
|
||||||
)
|
)
|
||||||
|
trainer_actions = course_session_permissions(trainer, self.course_session.id)
|
||||||
|
|
||||||
# THEN
|
# THEN
|
||||||
self.assertEqual(len(mentor_actions), 0)
|
self.assertEqual(len(mentor_actions), 0)
|
||||||
self.assertEqual(participant_actions, ["complete-learning-content"])
|
self.assertEqual(participant_actions, ["complete-learning-content"])
|
||||||
|
self.assertEqual(trainer_actions, ["complete-learning-content"])
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue