From af0a8b35ba4ddc8a0039c3e2cc702d0e4a39a3d1 Mon Sep 17 00:00:00 2001
From: Ned Batchelder <ned@nedbatchelder.com>
Date: Thu, 24 Sep 2020 17:22:39 -0400
Subject: [PATCH] Staff and instructors shouldn't have the Student role. (#107)

---
 lti_consumer/lti_1p3/constants.py           | 2 --
 lti_consumer/lti_1p3/tests/test_consumer.py | 1 -
 2 files changed, 3 deletions(-)

diff --git a/lti_consumer/lti_1p3/constants.py b/lti_consumer/lti_1p3/constants.py
index dcf4c42..de29ef7 100644
--- a/lti_consumer/lti_1p3/constants.py
+++ b/lti_consumer/lti_1p3/constants.py
@@ -22,11 +22,9 @@ LTI_1P3_ROLE_MAP = {
     'staff': [
         'http://purl.imsglobal.org/vocab/lis/v2/system/person#Administrator',
         'http://purl.imsglobal.org/vocab/lis/v2/institution/person#Instructor',
-        'http://purl.imsglobal.org/vocab/lis/v2/institution/person#Student',
     ],
     'instructor': [
         'http://purl.imsglobal.org/vocab/lis/v2/institution/person#Instructor',
-        'http://purl.imsglobal.org/vocab/lis/v2/institution/person#Student'
     ],
     'student': [
         'http://purl.imsglobal.org/vocab/lis/v2/institution/person#Student'
diff --git a/lti_consumer/lti_1p3/tests/test_consumer.py b/lti_consumer/lti_1p3/tests/test_consumer.py
index f7e3fb9..9ffc157 100644
--- a/lti_consumer/lti_1p3/tests/test_consumer.py
+++ b/lti_consumer/lti_1p3/tests/test_consumer.py
@@ -128,7 +128,6 @@ class TestLti1p3Consumer(TestCase):
             [
                 'http://purl.imsglobal.org/vocab/lis/v2/system/person#Administrator',
                 'http://purl.imsglobal.org/vocab/lis/v2/institution/person#Instructor',
-                'http://purl.imsglobal.org/vocab/lis/v2/institution/person#Student',
             ]
         )
     )
-- 
GitLab