Comment 4 for bug 1692746

Revision history for this message
Rebecca Blundell (rjb-dev) wrote :

I have reviewed this more closely and think these are the things to focus on here:

test/behat/features/group/shared_views_to_group.feature
 - change name to share_with_group.feature
 - make as concise as possible
 - use new date step https://reviews.mahara.org/#/c/7962/ to add relative date/s

test/behat/features/group/group_view_block.feature
 - remove functionality that is tested in share_with_group.feature
 - This test is way too huge. Look at making it more concise.

test/behat/features/group/shared_pages_timestamp.feature
 - combine with share_with_group.feature

test/behat/features/user/search_users_by_share_pages.feature
 - don't confirm login
 - add pages in Background
 - consider combining with the share_with_group.feature

test/behat/features/view/sharepage.feature
 - could be a scenario in a related test, e.g, search_users_by_share_pages.feature