feat: Send xqueue submissions to edx-submission

fix: Restructuring to send course_id and score to edx submission

fix: Refactoring of sending information to sent_to_submission

fix: Elimination of unnecessary functions

fix: Added usage comment to ProblemBlock in XqueueInterface constructor

fix: update doc ADR

fix: setting for Quality Others test (ubuntu-24.04, 3.11, 20)

fix: Deprecation for django-trans-escape-filter-parse-error

test: Add @pytest.mark.django_db decorator to test functions

test: Fix for pylint being disabled

fix: updated changes for pylint disable

fix: update error from docs ADR-0005

update: xmodule/docs/decisions/0005-send-data-to-edx-submission.rst

Co-authored-by: Sarina Canelake <sarina@axim.org>

update: xmodule/docs/decisions/0005-send-data-to-edx-submission.rst

Co-authored-by: Sarina Canelake <sarina@axim.org>

fix: Adjusted correction

fix: update date for docs ADR

Revert "fix: update date for docs ADR"

This reverts commit 0b4229c.

fix: replace call created_submission to create_external_grader_detail

fix: update test xqueue_submission

fix: add docstring in test_xqueue_submission

fix: update date doc ADR

fix: update version edx-submission 3.8.6

fix: add @pytest.mark.xfail

fix: add 20 chances in test_capa_block:

fix: increase retry attempts for seed generation in ProblemBlockTest

fix: change version to edx-submission lib

fix: new version edx-submission in testings

fix: replace parameter file to files

fix: update variable grader_file_name and points_possible

fix: Adjustment in the is_flag_active function to always take the last record edited in the waffle

fix: wrap large line of code

fix: update function is_flag_active

fix: code style adjustment

fix: changes for 60 retry

feat: use CourseWaffleFlag to determine xqueue callback path

fix: Code style adjustment

fix: remove deprecated xqueue callback route and simplify callback type logic

fix: Deleting a comment in the ADR document

fix: add log in self.block is None

fix: Code style adjustment in log

Author: gabrielC1409

requirements/edx/base.txt

@@ -512,7 +512,7 @@ edx-search==4.1.3
     #   openedx-forum
 edx-sga==0.25.3
     # via -r requirements/edx/bundled.in
-edx-submissions==3.8.6
+edx-submissions==3.9.0
     # via
     #   -r requirements/edx/kernel.in
     #   ora2

requirements/edx/development.txt

@@ -812,7 +812,7 @@ edx-sga==0.25.3
     # via
     #   -r requirements/edx/doc.txt
     #   -r requirements/edx/testing.txt
-edx-submissions==3.8.6
+edx-submissions==3.9.0
     # via
     #   -r requirements/edx/doc.txt
     #   -r requirements/edx/testing.txt

requirements/edx/doc.txt

@@ -598,7 +598,7 @@ edx-search==4.1.3
     #   openedx-forum
 edx-sga==0.25.3
     # via -r requirements/edx/base.txt
-edx-submissions==3.8.6
+edx-submissions==3.9.0
     # via
     #   -r requirements/edx/base.txt
     #   ora2

requirements/edx/testing.txt

@@ -625,7 +625,7 @@ edx-search==4.1.3
     #   openedx-forum
 edx-sga==0.25.3
     # via -r requirements/edx/base.txt
-edx-submissions==3.8.6
+edx-submissions==3.9.0
     # via
     #   -r requirements/edx/base.txt
     #   ora2

scripts/xsslint_thresholds.json

@@ -37,4 +37,4 @@
         "underscore-not-escaped": 2
     },
     "total": 64
-}
+}

xmodule/capa/capa_problem.py

@@ -96,6 +96,7 @@ class LoncapaSystem(object):
     See :class:`DescriptorSystem` for documentation of other attributes.
 
     """
+
     def __init__(
         self,
         ajax_url,
@@ -130,6 +131,7 @@ class LoncapaProblem(object):
     """
     Main class for capa Problems.
     """
+
     def __init__(self, problem_text, id, capa_system, capa_block,  # pylint: disable=redefined-builtin
                  state=None, seed=None, minimal_init=False, extract_tree=True):
         """

xmodule/capa/errors.py

@@ -0,0 +1,56 @@
+# errors.py
+"""
+Custom error handling for the XQueue submission interface.
+"""
+
+
+class XQueueSubmissionError(Exception):
+    """Base class for all XQueue submission errors."""
+
+
+class JSONParsingError(XQueueSubmissionError):
+    """Raised when JSON parsing fails."""
+    MESSAGE = "Error parsing {name}: {error}"
+
+    def __init__(self, name, error):
+        super().__init__(self.MESSAGE.format(name=name, error=error))
+
+
+class MissingKeyError(XQueueSubmissionError):
+    """Raised when a required key is missing."""
+    MESSAGE = "Missing key: {key}"
+
+    def __init__(self, key):
+        super().__init__(self.MESSAGE.format(key=key))
+
+
+class ValidationError(XQueueSubmissionError):
+    """Raised when a validation check fails."""
+    MESSAGE = "Validation error: {error}"
+
+    def __init__(self, error):
+        super().__init__(self.MESSAGE.format(error=error))
+
+
+class TypeErrorSubmission(XQueueSubmissionError):
+    """Raised when an invalid type is encountered."""
+    MESSAGE = "Type error: {error}"
+
+    def __init__(self, error):
+        super().__init__(self.MESSAGE.format(error=error))
+
+
+class RuntimeErrorSubmission(XQueueSubmissionError):
+    """Raised for runtime errors."""
+    MESSAGE = "Runtime error: {error}"
+
+    def __init__(self, error):
+        super().__init__(self.MESSAGE.format(error=error))
+
+
+class GetSubmissionParamsError(XQueueSubmissionError):
+    """Raised when there is an issue getting submission parameters."""
+    MESSAGE = "Submission parameters error: {error}"
+
+    def __init__(self, error="Block instance is not defined!"):
+        super().__init__(self.MESSAGE.format(error=error))

xmodule/capa/inputtypes.py

@@ -66,8 +66,6 @@
 
 log = logging.getLogger(__name__)
 
-#########################################################################
-
 registry = TagRegistry()  # pylint: disable=invalid-name
 
 
@@ -408,7 +406,7 @@ class OptionInput(InputTypeBase):
 
     Example:
 
-    <optioninput options="('Up','Down')" correct="Up"/><text>The location of the sky</text>
+    <optioninput options="('Up', 'Down')" correct="Up"/><text>The location of the sky</text>
 
     # TODO: allow ordering to be randomized
     """
@@ -505,7 +503,7 @@ def setup(self):
             raise Exception(msg)
 
         self.choices = self.extract_choices(self.xml, i18n)
-        self._choices_map = dict(self.choices,)
+        self._choices_map = dict(self.choices, )
 
     @classmethod
     def get_attributes(cls):
@@ -602,16 +600,16 @@ def get_attributes(cls):
         Register the attributes.
         """
         return [
-            Attribute('params', None),       # extra iframe params
+            Attribute('params', None),  # extra iframe params
             Attribute('html_file', None),
             Attribute('gradefn', "gradefn"),
             Attribute('get_statefn', None),  # Function to call in iframe
-                                             #   to get current state.
+            #   to get current state.
             Attribute('initial_state', None),  # JSON string to be used as initial state
             Attribute('set_statefn', None),  # Function to call iframe to
-                                             #   set state
-            Attribute('width', "400"),       # iframe width
-            Attribute('height', "300"),      # iframe height
+            #   set state
+            Attribute('width', "400"),  # iframe width
+            Attribute('height', "300"),  # iframe height
             # Title for the iframe, which should be supplied by the author of the problem. Not translated
             # because we are in a class method and therefore do not have access to capa_system.i18n.
             # Note that the default "display name" for the problem is also not translated.
@@ -1626,7 +1624,7 @@ class ChoiceTextGroup(InputTypeBase):
     CheckboxProblem:
     <problem>
       <startouttext/>
-        A person randomly selects 100 times, with replacement, from the list of numbers \(\sqrt{2}\) , 2, 3, 4 ,5 ,6
+        A person randomly selects 100 times, with replacement, from the list of numbers \(\sqrt{2}\), 2, 3, 4, 5, 6
         and records the results. The first number they pick is \(\sqrt{2}\) Given this information
         select the correct choices and fill in numbers to make them accurate.
       <endouttext/>

xmodule/capa/registry.py

@@ -7,6 +7,7 @@ class TagRegistry(object):
 
     (A dictionary with some extra error checking.)
     """
+
     def __init__(self):
         self._mapping = {}
 

xmodule/capa/tests/test_answer_pool.py

@@ -13,6 +13,7 @@
 
 class CapaAnswerPoolTest(unittest.TestCase):
     """Capa Answer Pool Test"""
+
     def setUp(self):
         super(CapaAnswerPoolTest, self).setUp()  # lint-amnesty, pylint: disable=super-with-arguments
         self.system = test_capa_system()