← Back to team overview

mahara-contributors team mailing list archive

[Bug 1703751] Re: behat change_account_settings.feature doesn't test functionality

 

To clarify any potential ambiguities from the previous note:

- Take out all the "I should see" steps in test/behat/features/settings/notification_setting_order.feature
- Add more "should appear" steps to check that every notification type in the list is tested in their respective alphabetical order. Currently, not all notification types are checked if they are in the correct order. Only a few are checked but that wouldn't mean that the rest is in the correct order.
- Once these steps are added, take the content from that file and add it as scenario to the notification_choices.feature and make changes to the feature description as needed.
- In notification_choices.feature, after the 
I press "Save" step, add the step:
And I should see "Preferences saved"

The step to not see "Delete account" is not necessary as you are on a
different page.

- Rename notification_choices.feature as notification_settings.feature

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1703751

Title:
  behat change_account_settings.feature doesn't test functionality

Status in Mahara:
  In Progress

Bug description:
  This test is currently not testing actual functionality. It should be
  changed to check notification options are available and to check one
  or two actually work.

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1703751/+subscriptions


References