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 _______________________________________________ Mailing list: https://launchpad.net/~mahara-contributors Post to : [email protected] Unsubscribe : https://launchpad.net/~mahara-contributors More help : https://help.launchpad.net/ListHelp
