SoftwareSecure does not allow colons in exam names - replace them with dashes

edx_proctoring/backends/software_secure.py

@@ -355,6 +355,8 @@ class SoftwareSecureBackendProvider(ProctoringBackendProvider):
         now = datetime.datetime.utcnow()
         start_time_str = now.strftime("%a, %d %b %Y %H:%M:%S GMT")
         end_time_str = (now + datetime.timedelta(minutes=time_limit_mins)).strftime("%a, %d %b %Y %H:%M:%S GMT")
+        # note, SoftwareSecure does not appear to allow colons in the exam names
+        exam_name = exam['exam_name'].replace(':', '-')
         return {
             "examCode": attempt_code,
             "organization": self.organization,
@@ -365,7 +367,7 @@ class SoftwareSecureBackendProvider(ProctoringBackendProvider):
             "reviewerNotes": reviewer_notes,
             "examPassword": self._encrypt_password(self.crypto_key, attempt_code),
             "examSponsor": self.exam_sponsor,
-            "examName": exam['exam_name'],
+            "examName": exam_name,
             "ssiProduct": 'rp-now',
             # need to pass in a URL to the LMS?
             "examUrl": callback_url,

edx_proctoring/backends/tests/test_software_secure.py

@@ -219,7 +219,7 @@ class SoftwareSecureTests(TestCase):
         exam_id = create_exam(
             course_id='foo/bar/baz',
             content_id='content',
-            exam_name='Sample Exam',
+            exam_name='Sample Exam with : Colon',
             time_limit_mins=10,
             is_proctored=True
         )
@@ -236,6 +236,13 @@ class SoftwareSecureTests(TestCase):
 
             # assert that we use the default that is defined in system configuration
             self.assertEqual(result['reviewerNotes'], constants.DEFAULT_SOFTWARE_SECURE_REVIEW_POLICY)
+
+            # the check that if a colon was passed in for the exam name, then the colon was changed to
+            # a dash. This is because SoftwareSecure cannot handle a colon in the exam name
+            if ':' in exam['exam_name']:
+                self.assertIn('-', result['examName'])
+                self.assertNotIn(':', result['examName'])
+
             return result
 
         with HTTMock(mock_response_content):

setup.py

@@ -34,7 +34,7 @@ def load_requirements(*requirements_paths):
 
 setup(
     name='edx-proctoring',
-    version='0.10.17',
+    version='0.10.18',
     description='Proctoring subsystem for Open edX',
     long_description=open('README.md').read(),
     author='edX',