Behat uploading a file step is currently broken

[Jinelle Foley-Barnes]

Hi Sonn,

There is an issue with the "Attaching a file" step.

Currently it passes and says it's a success but they file doesn't actually attach.

I have attached a test script below for you to test with.

Thanks,
Jinelle

[Son Nguyen (ngson2000)]

I have implemented 2 behat steps to deal with confirm popup

[Mahara Bot (dev-mahara)]

Patch for "master" branch: https://reviews.mahara.org/4580

[Son Nguyen (ngson2000)]

In order to attach files in behat tests, you need to
 specify the full path to the upload file on the testing server, or
 put the upload file into the folder test/behat/upload_files and use only the relative path to the file in the behat feature.

I attached a working feature file which uses the "Image2.png" file already available in the upload_files folder.

[Jinelle Foley-Barnes (jinelleb)]

Hi Sonn,

Does this test that you've attached make a dialogue box appear?

I don't know what area in Mahara has a dialogue box so I can write a test for it myself.

If this isn't a test that I can use can you please flesh out a test for me and I can do the rest.

Cheers,
Jinelle

[Son Nguyen (ngson2000)]

The confirmation dialogue box will be automatically accepted after each step. That's why you don't see the dialogue box when click delete button

The test file I attached does not make the box appear just include the new steps I implemented in https://reviews.mahara.org/#/c/4580.

There are many places in Mahara which have the alert or confirmation dialogue box when you try to delete something or navigate away without submit a change.

You can run the test I attached after checkout the patch https://reviews.mahara.org/#/c/4580.

I attach a new version of the test file as the previous one did not work

[Mahara Bot (dev-mahara)]

Patch for "master" branch: https://reviews.mahara.org/6217

[Robert Lyon (robertl-9)]

This actually works now - we can upload a file no problems.

I did notice we didn't actually test the deleting of a file though so have added a test for that

https://reviews.mahara.org/6217

[Mahara Bot (dev-mahara)]

Patch for "master" branch: https://reviews.mahara.org/6302

[Mahara Bot (dev-mahara)]

Patch for "master" branch: https://reviews.mahara.org/6301

[Mahara Bot (dev-mahara)]

Reviewed: https://reviews.mahara.org/6217
Committed: https://git.mahara.org/mahara/mahara/commit/cff3fbffd7f9142dc5d7be52d6fc033dd82b86d2
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit cff3fbffd7f9142dc5d7be52d6fc033dd82b86d2
Author: Robert Lyon <email address hidden>
Date: Tue Mar 22 11:59:57 2016 +1300

Bug 1415247: Adding behat step to confirm file deletion

Change-Id: Iaee6116f6220227325d5ead4ec548a8f8df26797
Signed-off-by: Robert Lyon <email address hidden>

[Mahara Bot (dev-mahara)]

Patch for "16.04_STABLE" branch: https://reviews.mahara.org/6450

[Mahara Bot (dev-mahara)]

Patch for "15.10_STABLE" branch: https://reviews.mahara.org/6451

[Mahara Bot (dev-mahara)]

Patch for "15.04_STABLE" branch: https://reviews.mahara.org/6452

[Mahara Bot (dev-mahara)]

Reviewed: https://reviews.mahara.org/6451
Committed: https://git.mahara.org/mahara/mahara/commit/2793c01e85ff682389108bc186b9737a3c87467a
Submitter: Robert Lyon (<email address hidden>)
Branch: 15.10_STABLE

commit 2793c01e85ff682389108bc186b9737a3c87467a
Author: Robert Lyon <email address hidden>
Date: Tue Mar 22 11:59:57 2016 +1300

Bug 1415247: Adding behat step to confirm file deletion

Change-Id: Iaee6116f6220227325d5ead4ec548a8f8df26797
Signed-off-by: Robert Lyon <email address hidden>
(cherry picked from commit cff3fbffd7f9142dc5d7be52d6fc033dd82b86d2)

[Mahara Bot (dev-mahara)]

Reviewed: https://reviews.mahara.org/6450
Committed: https://git.mahara.org/mahara/mahara/commit/c5e652b707b7be375eb5ffe74f3eb7634892f362
Submitter: Robert Lyon (<email address hidden>)
Branch: 16.04_STABLE

commit c5e652b707b7be375eb5ffe74f3eb7634892f362
Author: Robert Lyon <email address hidden>
Date: Tue Mar 22 11:59:57 2016 +1300

Bug 1415247: Adding behat step to confirm file deletion

Change-Id: Iaee6116f6220227325d5ead4ec548a8f8df26797
Signed-off-by: Robert Lyon <email address hidden>
(cherry picked from commit cff3fbffd7f9142dc5d7be52d6fc033dd82b86d2)

[Mahara Bot (dev-mahara)]

Reviewed: https://reviews.mahara.org/6452
Committed: https://git.mahara.org/mahara/mahara/commit/1e233c5e4970649433be8d8e6e12ca3d9d47a8b4
Submitter: Robert Lyon (<email address hidden>)
Branch: 15.04_STABLE

commit 1e233c5e4970649433be8d8e6e12ca3d9d47a8b4
Author: Robert Lyon <email address hidden>
Date: Tue Mar 22 11:59:57 2016 +1300

Bug 1415247: Adding behat step to confirm file deletion

Change-Id: Iaee6116f6220227325d5ead4ec548a8f8df26797
Signed-off-by: Robert Lyon <email address hidden>
(cherry picked from commit cff3fbffd7f9142dc5d7be52d6fc033dd82b86d2)