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

Changes to the (now behat only) role_assign generator

    XMLWordPrintable

Details

    • MOODLE_25_STABLE
    • MOODLE_26_STABLE
    • MDL-38863_master
    • Hide
      1. Pull the .feature file to test the new generator
        • git fetch git://github.com/dmonllao/moodle.git MDL-38863_master-testme
        • git cherry-pick 0a34fc13c748b2a543bb79b1ef53e2e579f2a41c
      2. Update the features files list
        • php admin/tool/behat/cli/util.php --enable
      3. Run the tests
      4. vendor/bin/behat...... with --tags @MDL-38863
      5. You SHOULD see a failure for the following scenario:
        • No references, IS NORMAL THAT IT FAILS
      Show
      Pull the .feature file to test the new generator git fetch git://github.com/dmonllao/moodle.git MDL-38863 _master-testme git cherry-pick 0a34fc13c748b2a543bb79b1ef53e2e579f2a41c Update the features files list php admin/tool/behat/cli/util.php --enable Run the tests vendor/bin/behat...... with --tags @ MDL-38863 You SHOULD see a failure for the following scenario: No references, IS NORMAL THAT IT FAILS

    Description

      While integrating MDL-38482 it was agreed that the role_assign generator (only available for behat, should:

      1) require a 3rd param, context.
      2) optionally, start using any "commons" generator (under /testing, to be shared with phpunit).

      So this is about to review that generator and improve it.

      Attachments

        Issue Links

          Activity

            People

              dmonllao David Monllaó
              stronk7 Eloy Lafuente (stronk7)
              Damyon Wiese Damyon Wiese
              Michael de Raadt Michael de Raadt
              Andrew Lyons, Huong Nguyen, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                18/Nov/13