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

behat/mink is not php74 compliant

    XMLWordPrintable

    Details

      Description

      Once the main problems in core has been solved, behat runs have revealed that current mink 1.7.1 is not php74 compliant.

      See: https://travis-ci.org/stronk7/Mink/builds/614672369

      That matches the errors we are getting in our runs:

      007 Scenario: Moving chapters up rearranges them properly                       # /var/www/html/mod/book/tests/behat/reorganize_chapters.feature:62
            Given I click on "Move chapter up \"3. There aren't 2 without 3\"" "link" # /var/www/html/mod/book/tests/behat/reorganize_chapters.feature:63
              8192: implode(): Passing glue string after array is deprecated. Swap the parameters in /var/www/html/vendor/behat/mink/src/Selector/Xpath/Escaper.php line 50
      

      This is about to send a PR to them, asking for a quick 1.7.2 if possible (if not, we'll need to fork and point to ours).

      WIP fix/PR: https://github.com/minkphp/Mink/pull/792

      That makes their own tests to pass:

      https://travis-ci.org/stronk7/Mink/builds/614678756

      Ciao

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              stronk7 Eloy Lafuente (stronk7)
              Reporter:
              stronk7 Eloy Lafuente (stronk7)
              Peer reviewer:
              Tim Hunt
              Integrator:
              Andrew Nicols
              Tester:
              CiBoT
              Participants:
              Component watchers:
              Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                11/May/20

                  Time Tracking

                  Estimated:
                  Original Estimate - 0 minutes
                  0m
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 3 hours, 10 minutes
                  3h 10m