Loading...

XML

Word

Printable

Type: Bug
Resolution: Fixed
Priority: Minor
Fix Version/s: 2.5.4, 2.6.1
Affects Version/s: 2.5.3, 2.6, 2.7
Component/s: Automated functional tests (behat)
Labels:
- triaged

Affected Branches:
MOODLE_25_STABLE, MOODLE_26_STABLE, MOODLE_27_STABLE
Fixed Branches:
MOODLE_25_STABLE, MOODLE_26_STABLE
Pull from Repository:
git://github.com/dmonllao/moodle.git
Pull Main Branch:
~~MDL-43404~~_master
Pull Main Diff URL:
https://github.com/dmonllao/moodle/compare/moodle:master...MDL-43404_master
Testing Instructions:
Hide

Test 1

Install or update the behat site

git fetch git://github.com/dmonllao/moodle.git ~~MDL-43404~~_testme

git cherry-pick be4ccca82a30b2fcef5730501453e04226330886

php admin/tool/behat/cli/util.php --enable

vendor/bin/behat --config /path/provided/by/last/step/behat/behat.yml --tags @~~MDL-43404~~

It SHOULD all pass

Test 2

Log into the integration site

Go to your user profile

Edit your firstname

Click on "Change password"

You SHOULD see a JS modal window (the one you are probably used to see) stating that you are going to leave the page without submitting the form
Show
Test 1 Install or update the behat site git fetch git://github.com/dmonllao/moodle.git MDL-43404 _testme git cherry-pick be4ccca82a30b2fcef5730501453e04226330886 php admin/tool/behat/cli/util.php --enable vendor/bin/behat --config /path/provided/by/last/step/behat/behat.yml --tags @ MDL-43404 It SHOULD all pass Test 2 Log into the integration site Go to your user profile Edit your firstname Click on "Change password" You SHOULD see a JS modal window (the one you are probably used to see) stating that you are going to leave the page without submitting the form

After ~~MDL-42625~~ we can review all the currently supported browser+os combinations and make sure we are not auto accepting alerts. Scenarios must fail if there are unhandled alerts.

has been marked as being related by

MDL-42518 Add behat testing for form change checker

Closed

will be (partly) resolved by

MDL-43434 Moodle form auto-save instead of "unsaved changes" alerts.

Closed

Assignee:: David Monllaó

Reporter:: David Monllaó

Peer reviewer:: Eloy Lafuente (stronk7)

Integrator:: Eloy Lafuente (stronk7)

Tester:: Rajesh Taneja

Participants:: Andrew Lyons, Damyon Wiese, Dan Poltawski, David Monllaó, Eloy Lafuente (stronk7), Rajesh Taneja

Votes:: 0 Vote for this issue

Watchers:: 4 Start watching this issue

Created:: 16/Dec/13 6:36 PM

Updated:: 19/May/14 10:00 PM

Resolved:: 20/Dec/13 3:31 PM

Details

Description

Attachments

Issue Links

Activity

People

Dates

Clockify