Uploaded image for project: '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

        Gliffy Diagrams

          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: