Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-47990

qtype_multichoice_single response import/export for testing broken

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.7.2, 2.8
    • Fix Version/s: 2.7.3
    • Component/s: Questions
    • Labels:
    • Testing Instructions:
      Hide

      These methods are only used for testing, so you could decide that the new unit test is sufficient testing.

      However, if you really want to test this properly:

      1. Install https://github.com/jamiepratt/moodle-quiz_simulate
      2. Create a quiz.
      3. Add a multiple choice question where the Choice contain HTML tags. E.g. when you view source in Atto the choice looks like '<p>A</p>'.
      4. Attempt this quiz as a student.
      5. Use the Simulate report to export the attempts at this quiz.
      6. Delete all the attempts at the quiz.
      7. Re-import the attept you exported, and verify that the response to the multiple choice question has been imported correctly.
      Show
      These methods are only used for testing, so you could decide that the new unit test is sufficient testing. However, if you really want to test this properly: Install https://github.com/jamiepratt/moodle-quiz_simulate Create a quiz. Add a multiple choice question where the Choice contain HTML tags. E.g. when you view source in Atto the choice looks like '<p>A</p>'. Attempt this quiz as a student. Use the Simulate report to export the attempts at this quiz. Delete all the attempts at the quiz. Re-import the attept you exported, and verify that the response to the multiple choice question has been imported correctly.
    • Affected Branches:
      MOODLE_27_STABLE, MOODLE_28_STABLE
    • Fixed Branches:
      MOODLE_27_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      prepare_simulated_post_data does a clean_param, but get_student_response_values_for_simulation does not, so things do not match as they should.

        Attachments

          Activity

            People

            Assignee:
            timhunt Tim Hunt
            Reporter:
            timhunt Tim Hunt
            Peer reviewer:
            John Okely
            Integrator:
            Eloy Lafuente (stronk7)
            Tester:
            Frédéric Massart
            Participants:
            Component watchers:
            Tim Hunt, Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              10/Nov/14