Problem/Motivation

Some basic functionality is missing test coverage.

Proposed resolution

Create tests for

  • unsubscribing aka disabling
  • tokens
  • user preferences
  • user/node/comment deletion
  • mail body
  • extra field display
  • settings form

I don't think we want to create all these tests at once, it's more helpful to commit them in parts. I think unsubscribing, tokens and user preferences are top priority, because they are basic non-admin features, and the rest can be done in separate issues?

Remaining tasks

User interface changes

API changes

Data model changes

Comments

greggles created an issue. See original summary.

gnuget’s picture

Alright, I wrote the tests for unsubscribing aka disabling as part of #2879914: Notifications not sent when user chooses "Replies to my comment"

I renamed the test file to be more clear about what it is testing (the anonymous comments).

It now tests:

And that's it.

gnuget’s picture

Issue summary: View changes
gnuget’s picture

gnuget’s picture

Issue summary: View changes
gnuget’s picture

Status: Active » Fixed

I created:

So we can work on the missing tests on different issues.

I'm going to close this one :-)

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.