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

Unit test failures (20_STABLE, 2.0.6)

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 2.0.6, 2.1.3, 2.2
    • Fix Version/s: 2.0.7, 2.1.4
    • Component/s: Unit tests
    • Labels:

      Description

      While rolling 2.0.6, executing unit tests I've got:

      123/128 test cases complete: 4294 passes, 1 fails and 0 exceptions.

      Fail: filter/mediaplugin/simpletest/testfiltermediaplugin.php / ▶ filter_mediaplugin_test / ▶ test_filter_mediaplugin_link
       
      Testing text: <a class="_blanktarget" href="http://moodle.org/testfile/test.flv?d=100x">test flv</a> at 
      [/Users/stronk7/git_moodle/integration/filter/mediaplugin/simpletest/testfiltermediaplugin.php line 110]

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              An exactly the same test is also failing under 21_STABLE.

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - An exactly the same test is also failing under 21_STABLE.
              Hide
              salvetore Michael de Raadt added a comment -

              Still an issue with master prior to 2.2 release.

              Show
              salvetore Michael de Raadt added a comment - Still an issue with master prior to 2.2 release.
              Hide
              abgreeve Adrian Greeve added a comment -

              Thanks to Rajesh for providing me with the fix for this. I couldn't find any other options for flv / f4v urls.

              Show
              abgreeve Adrian Greeve added a comment - Thanks to Rajesh for providing me with the fix for this. I couldn't find any other options for flv / f4v urls.
              Hide
              skodak Petr Skoda added a comment -

              ahh, this is caused by the recent fix that ignores the extra params after ?, we just need to remove the failing test and probably add a new one.

              Show
              skodak Petr Skoda added a comment - ahh, this is caused by the recent fix that ignores the extra params after ?, we just need to remove the failing test and probably add a new one.
              Hide
              skodak Petr Skoda added a comment -

              the proposed regex change above is not correct, please fix only the unittests to reflect changes from MDL-30303, that is anything invalid after "?" xxx.flv?blah=blah is ignored.

              Show
              skodak Petr Skoda added a comment - the proposed regex change above is not correct, please fix only the unittests to reflect changes from MDL-30303 , that is anything invalid after "?" xxx.flv?blah=blah is ignored.
              Hide
              abgreeve Adrian Greeve added a comment -

              The regular expression has been reverted back to the original and the test string has now been modified.

              Show
              abgreeve Adrian Greeve added a comment - The regular expression has been reverted back to the original and the test string has now been modified.
              Hide
              skodak Petr Skoda added a comment - - edited

              To integrators: please cherry pick to stable branches

              Show
              skodak Petr Skoda added a comment - - edited To integrators: please cherry pick to stable branches
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              (note: after reviewing this, it was considered that the best solution was to remove the test completely as far as had no sense anymore as invalid test)

              Integrated and backported to 21 and 20 stables.

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - (note: after reviewing this, it was considered that the best solution was to remove the test completely as far as had no sense anymore as invalid test) Integrated and backported to 21 and 20 stables.
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              All tests passing under 20 and 21 STABLE. Still some to fix for master (but @ MDL-30347). So passed.

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - All tests passing under 20 and 21 STABLE. Still some to fix for master (but @ MDL-30347 ). So passed.
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              The master fixes corresponding to this issue have been sent upstream. Fixes for other branches (19, 20, 21 stable) will be sent in the very-next days.

              Thanks for the hard work! Closing, ciao

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - The master fixes corresponding to this issue have been sent upstream. Fixes for other branches (19, 20, 21 stable) will be sent in the very-next days. Thanks for the hard work! Closing, ciao

                People

                • Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    9/Jan/12