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

Push notifications unit tests leave unused curl mock responses

    XMLWordPrintable

    Details

      Description

      Steps to replicate

      Run the following phpunit test suites: 

      root@bb4c351b316b:/siteroot# vendor/bin/phpunit --testsuite='message_airnotifier_testsuite,repository_url_testsuite'
      Moodle 4.0dev (Build: 20210723), 8c0853d0260f110a50b858edc95b480d7fa0013d
      Php: 7.4.3, mysqli: 5.7.31, OS: Linux 5.4.0-77-generic x86_64
      PHPUnit 9.5.4 by Sebastian Bergmann and contributors.
       
      ................E                                                 17 / 17 (100%)
       
      Time: 00:08.544, Memory: 75.00 MB
       
      There was 1 error:
       
      1) repository_url_lib_testcase::test_escape_url
      Undefined index: content_type
       
      /siteroot/repository/url/lib.php:157
      /siteroot/repository/url/lib.php:112
      /siteroot/repository/url/tests/lib_test.php:79
      /siteroot/lib/phpunit/classes/advanced_testcase.php:80
       
      ERRORS!
      Tests: 17, Assertions: 72, Errors: 1.
      

        

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              mikhailgolenkov Mikhail Golenkov
              Reporter:
              mikhailgolenkov Mikhail Golenkov
              Peer reviewer:
              Heena Agheda Heena Agheda
              Integrator:
              Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
              Tester:
              Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
              Participants:
              Component watchers:
              Andrew Lyons, Dongsheng Cai, Huong Nguyen, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                29/Jul/21

                  Time Tracking

                  Estimated:
                  Original Estimate - 0 minutes
                  0m
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 20 minutes
                  20m