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

Incorrect xpath used in edit_activities feature.

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide

      Run following features using chrome and phantomjs

      1. blocks/social_activities/tests/behat/edit_activities.feature
      2. blocks/site_main_menu/tests/behat/edit_activities.feature
      Show
      Run following features using chrome and phantomjs blocks/social_activities/tests/behat/edit_activities.feature blocks/site_main_menu/tests/behat/edit_activities.feature
    • Affected Branches:
      MOODLE_31_STABLE
    • Fixed Branches:
      MOODLE_31_STABLE
    • Pull Master Branch:
      wip-mdl-53564

      Description

      MDL-52869 introduced xpath with //. which selects the current node under the dom tree. This seems to have worked fine for Firefox, but is not correct as

      1. // means: Selects nodes in the document from the current node that match the selection no matter where they are
      2. . means: Selects the current node
        And they both will lead to wrong results. If you want to select any element from the current node then you should be using *.

        Attachments

          Issue Links

            Activity

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  23/May/16