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

JS dialogs and drag & drop are not stable enough to consider them valid steps

    Details

    • Testing Instructions:
      Hide

      Test 1

      1. git pull git://github.com/dmonllao/moodle.git MDL-39717_master-testme
      2. Run behat with --tags @MDL-39717
      3. They SHOULD all pass

      Test 2

      1. Go to site administration -> development -> acceptance testing
      2. Check the description of the following steps definitions list:
        • I move "" activity to section ""
        • I delete "" activity
        • I accept the currently displayed dialog
        • I drag "" "" and I drop it in "" ""
      3. They SHOULD all include a statement about it's experimentality
      Show
      Test 1 git pull git://github.com/dmonllao/moodle.git MDL-39717 _master-testme Run behat with --tags @ MDL-39717 They SHOULD all pass Test 2 Go to site administration -> development -> acceptance testing Check the description of the following steps definitions list: I move "" activity to section "" I delete "" activity I accept the currently displayed dialog I drag "" "" and I drop it in "" "" They SHOULD all include a statement about it's experimentality
    • Affected Branches:
      MOODLE_25_STABLE, MOODLE_26_STABLE
    • Fixed Branches:
      MOODLE_25_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-39717_master

      Description

      Native JS modal dialogs and drag & drop support is not stable in all OS&browser combinations, we have MDL-39272 to improve drag & drop support but we will depend on selenium driver improvements to manage properly JS modal dialogs in all OS/browsers combinations.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            salvetore Michael de Raadt added a comment -

            I'm commenting on this issue to test that David gets the message.

            Show
            salvetore Michael de Raadt added a comment - I'm commenting on this issue to test that David gets the message.
            Hide
            salvetore Michael de Raadt added a comment -

            I've unassigned this and adding another message.

            Show
            salvetore Michael de Raadt added a comment - I've unassigned this and adding another message.
            Hide
            rajeshtaneja Rajesh Taneja added a comment -

            Thanks David,

            Patch looks good, pushing for integration.

            Show
            rajeshtaneja Rajesh Taneja added a comment - Thanks David, Patch looks good, pushing for integration.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Integrated (25 & master), thanks!

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Integrated (25 & master), thanks!
            Hide
            salvetore Michael de Raadt added a comment -

            I'm adding a note to say that I intend to test this issue, but I'm also taking it as an opportunity to establish a new testing environment, so it might take a bit of time.

            Show
            salvetore Michael de Raadt added a comment - I'm adding a note to say that I intend to test this issue, but I'm also taking it as an opportunity to establish a new testing environment, so it might take a bit of time.
            Hide
            salvetore Michael de Raadt added a comment -

            Test results under Windows...

            D:\xampp\htdocs\master_integration_behat>vendor\bin\behat --config D:\xampp\Data
            \master_integration_behat_behat\behat\behat.yml --format progress --tags @MDL-39
            717
            ...................................................................... 70
            ...................................................................... 140
            ...................................................................... 210
            ...................................................................... 280
            ...................................................................... 350
            ...................................................................... 420
            ...................................................................... 490
            ...................................................................... 560
            ...................................................................... 630
            ........................................................
             
            18 scenarios (18 passed)
            686 steps (686 passed)
            27m53.278s
             
            D:\xampp\htdocs\master_integration_behat>

            Linux results to come...

            Show
            salvetore Michael de Raadt added a comment - Test results under Windows... D:\xampp\htdocs\master_integration_behat>vendor\bin\behat --config D:\xampp\Data \master_integration_behat_behat\behat\behat.yml --format progress --tags @MDL-39 717 ...................................................................... 70 ...................................................................... 140 ...................................................................... 210 ...................................................................... 280 ...................................................................... 350 ...................................................................... 420 ...................................................................... 490 ...................................................................... 560 ...................................................................... 630 ........................................................   18 scenarios (18 passed) 686 steps (686 passed) 27m53.278s   D:\xampp\htdocs\master_integration_behat> Linux results to come...
            Hide
            salvetore Michael de Raadt added a comment -

            I just realised I started the Behat process without the specific tag set. I've restarted the testing with the flag (perhaps I should have let it continue), but I won't be here to see it end today.

            I will report the results first thing tomorrow morning.

            Testing continues...

            Show
            salvetore Michael de Raadt added a comment - I just realised I started the Behat process without the specific tag set. I've restarted the testing with the flag (perhaps I should have let it continue), but I won't be here to see it end today. I will report the results first thing tomorrow morning. Testing continues...
            Hide
            salvetore Michael de Raadt added a comment -

            Here are the Linux results...

            michael@salvetore-desktop:~/web/htdocs/master_integration_behat$ vendor/bin/behat --config /home/michael/web/data/master_integration_behat_behat:80/behat/behat.yml --tags @MDL-39717
            ...................................................................... 70
            ...................................................................... 140
            ...................................................................... 210
            ...................................................................... 280
            ...................................................................... 350
            ...................................................................... 420
            ...................................................................... 490
            ...................................................................... 560
            ...................................................................... 630
            ........................................................
             
            18 scenarios (18 passed)
            686 steps (686 passed)
            38m49.563s

            Show
            salvetore Michael de Raadt added a comment - Here are the Linux results... michael@salvetore-desktop:~/web/htdocs/master_integration_behat$ vendor/bin/behat --config /home/michael/web/data/master_integration_behat_behat:80/behat/behat.yml --tags @MDL-39717 ...................................................................... 70 ...................................................................... 140 ...................................................................... 210 ...................................................................... 280 ...................................................................... 350 ...................................................................... 420 ...................................................................... 490 ...................................................................... 560 ...................................................................... 630 ........................................................   18 scenarios (18 passed) 686 steps (686 passed) 38m49.563s
            Hide
            salvetore Michael de Raadt added a comment -

            Test result: Success!

            Show
            salvetore Michael de Raadt added a comment - Test result: Success!
            Hide
            damyon Damyon Wiese added a comment -

            Thanks for your contribution! This issue has been reviewed, integrated, tested and now released to everyone.

            Closing as Fixed!

            Show
            damyon Damyon Wiese added a comment - Thanks for your contribution! This issue has been reviewed, integrated, tested and now released to everyone. Closing as Fixed!

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  8/Jul/13