consolidate behat messages tests

[Rebecca Blundell]

test/behat/features/settings/visible_asterix_message.feature
Is there another send message feature that this could be part of?

Check out these tests and see how we are testing messages and how it can be improved:
./features/social/send_multiple_messages.feature
./features/social/send_message.feature
./features/site_content/activity_type_messages.feature
./features/settings/visible_asterix_message.feature

[Rebecca Blundell (rjb-dev)]

What the tests currently do:

test/behat/features/site_content/activity_type_messages.feature

Background:
Admin sends UserA a message
UserA replies
Scenario:
Check the kinds of message filters available

test/behat/features/social/send_message.feature

Admin sends UserA a message including < & >
Check that sent message includes the same chars

test/behat/features/social/send_multiple_messages.feature

Admin logs in and searches for people
Sends a message to UserA
Requests friendship from UserA with a message that's too long
Checks it's rejected and sends another message
UserA logs in and sees message
Admin logs in and then masquerades as UserB
Requests friendship from Admin
Becomes Admin user and accepts friend request
Masquerades as UserB and checks friend accept options

test/behat/features/settings/visible_asterix_message.feature

Checks that on the send message form there are asterisks on the required fields

[Rebecca Blundell (rjb-dev)]

What needs to happen:

send_multiple_messages.feature is already quite complex, so let's leave it as is.

activity_type_messages.feature needs fixing so that the only thing in the background is the user creation.

Add the visible asterisk check to the send_message.feature and delete visible_asterix_message.feature

[Mahara Bot (dev-mahara)]

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

[Mahara Bot (dev-mahara)]

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

commit 8413ce2ec8807921d9b108db5ad6cfe41b4a35b3
Author: Niranjan Bandi <email address hidden>
Date: Fri Jan 26 15:11:27 2018 +1300

Bug 1719222- Background should have only user creation

- Made the changes according to the code review
Everything in the background is now moved to the scenario.
Visible_asterix_message.feature is now moved to send_message.feature.
visible_asterix_nessage.feature is now deleted.

Change-Id: I98c915048d1eff40dce4cd0eb0d62fc1b42913da