Contrib: Gradebook2
  1. Contrib: Gradebook2
  2. GRBK-1203

Find all cases where we display dialogs containing HTTP status codes to users and turn them into meaningful messages

    Details

    • Target Version:
    • QA Status:
      PASS
    • Previous Issue Keys:

      Description

      Find all cases where we display dialogs containing HTTP status codes to users and turn them into meaningful messages

        Issue Links

          Activity

          Hide
          Constance Fuller added a comment - - edited
          This issue is related to GRBK-1079. In the original ticket, ignoredBusinessRules were added to allow the re-importing of assignments on the import process. But those changes are affecting the display of the BusinessRuleExceptions when working solely thru the client.

          See also GRBK-407.
          Show
          Constance Fuller added a comment - - edited This issue is related to GRBK-1079 . In the original ticket, ignoredBusinessRules were added to allow the re-importing of assignments on the import process. But those changes are affecting the display of the BusinessRuleExceptions when working solely thru the client. See also GRBK-407 .
          Hide
          Constance Fuller added a comment -
          This message is also given. Will discuss what message should be provided instead:

          finalGradeSubmissionMessageText5a = Encountered a problem on the server : HTTP Response 500
          Show
          Constance Fuller added a comment - This message is also given. Will discuss what message should be provided instead: finalGradeSubmissionMessageText5a = Encountered a problem on the server : HTTP Response 500
          Hide
          Constance Fuller added a comment -
          Ready for QA
          Show
          Constance Fuller added a comment - Ready for QA
          Hide
          Peter Peterson added a comment -
          Hi Constance,

          Could you put a test plan out here. I'm not sure which dialogs generate this error.
          Show
          Peter Peterson added a comment - Hi Constance, Could you put a test plan out here. I'm not sure which dialogs generate this error.
          Hide
          Constance Fuller added a comment -
          Pete,

          The first instance found was in creating duplicate items thru the UI and imports. (The import error had been falling out even when processing manually.) See the review for GRBK-1167 for that one HTTP error message:

          QA Note from GRBK-1167

          For QA and unit testing, verify the duplication of items with "No Categories" and "Categories" (weighted or not). Also, test thru the UI for direct creation as well as thru imports. See GRBK-1079 AND GRBK-407 for additional test scenarios.

          The second instance found was a final grade submission error. It was coded into the properties files.

          QA for that one would be final grade submission with a forced server error.

          Constance
          Show
          Constance Fuller added a comment - Pete, The first instance found was in creating duplicate items thru the UI and imports. (The import error had been falling out even when processing manually.) See the review for GRBK-1167 for that one HTTP error message: QA Note from GRBK-1167 For QA and unit testing, verify the duplication of items with "No Categories" and "Categories" (weighted or not). Also, test thru the UI for direct creation as well as thru imports. See GRBK-1079 AND GRBK-407 for additional test scenarios. The second instance found was a final grade submission error. It was coded into the properties files. QA for that one would be final grade submission with a forced server error. Constance
          Hide
          Peter Peterson added a comment -
          QA UPDATE - Messages work as described. Need to work with Thomas to generate a server failure message then this will be PASS and CLOSE.
          Show
          Peter Peterson added a comment - QA UPDATE - Messages work as described. Need to work with Thomas to generate a server failure message then this will be PASS and CLOSE.
          Hide
          Peter Peterson added a comment -
          QA PASS: After discussing this with Thomas and Constance concluded that this is QA PASS without the server failure test. The reasoning is that the other passed test all work and they pull the same properties file as this would. Additionally there was no code change in this fix only properties file modification.
          Show
          Peter Peterson added a comment - QA PASS: After discussing this with Thomas and Constance concluded that this is QA PASS without the server failure test. The reasoning is that the other passed test all work and they pull the same properties file as this would. Additionally there was no code change in this fix only properties file modification.

            People

            • Assignee:
              Unassigned
              Reporter:
              Thomas Amsler
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: