2023-02-22 11:55:31 +11:00
|
|
|
# frozen_string_literal: true
|
|
|
|
|
2017-05-07 07:06:52 +10:00
|
|
|
require 'rails_helper'
|
|
|
|
|
2024-03-13 19:39:26 +11:00
|
|
|
RSpec.describe NotifyService do
|
2022-03-28 21:43:58 +11:00
|
|
|
subject { described_class.new.call(recipient, type, activity) }
|
2017-05-07 07:06:52 +10:00
|
|
|
|
|
|
|
let(:user) { Fabricate(:user) }
|
|
|
|
let(:recipient) { user.account }
|
Account domain blocks (#2381)
* Add <ostatus:conversation /> tag to Atom input/output
Only uses ref attribute (not href) because href would be
the alternate link that's always included also.
Creates new conversation for every non-reply status. Carries
over conversation for every reply. Keeps remote URIs verbatim,
generates local URIs on the fly like the rest of them.
* Conversation muting - prevents notifications that reference a conversation
(including replies, favourites, reblogs) from being created. API endpoints
/api/v1/statuses/:id/mute and /api/v1/statuses/:id/unmute
Currently no way to tell when a status/conversation is muted, so the web UI
only has a "disable notifications" button, doesn't work as a toggle
* Display "Dismiss notifications" on all statuses in notifications column, not just own
* Add "muted" as a boolean attribute on statuses JSON
For now always false on contained reblogs, since it's only relevant for
statuses returned from the notifications endpoint, which are not nested
Remove "Disable notifications" from detailed status view, since it's
only relevant in the notifications column
* Up max class length
* Remove pending test for conversation mute
* Add tests, clean up
* Rename to "mute conversation" and "unmute conversation"
* Raise validation error when trying to mute/unmute status without conversation
* Adding account domain blocks that filter notifications and public timelines
* Add tests for domain blocks in notifications, public timelines
Filter reblogs of blocked domains from home
* Add API for listing and creating account domain blocks
* API for creating/deleting domain blocks, tests for Status#ancestors
and Status#descendants, filter domain blocks from them
* Filter domains in streaming API
* Update account_domain_block_spec.rb
2017-05-19 09:14:30 +10:00
|
|
|
let(:sender) { Fabricate(:account, domain: 'example.com') }
|
Feature conversations muting (#3017)
* Add <ostatus:conversation /> tag to Atom input/output
Only uses ref attribute (not href) because href would be
the alternate link that's always included also.
Creates new conversation for every non-reply status. Carries
over conversation for every reply. Keeps remote URIs verbatim,
generates local URIs on the fly like the rest of them.
* Conversation muting - prevents notifications that reference a conversation
(including replies, favourites, reblogs) from being created. API endpoints
/api/v1/statuses/:id/mute and /api/v1/statuses/:id/unmute
Currently no way to tell when a status/conversation is muted, so the web UI
only has a "disable notifications" button, doesn't work as a toggle
* Display "Dismiss notifications" on all statuses in notifications column, not just own
* Add "muted" as a boolean attribute on statuses JSON
For now always false on contained reblogs, since it's only relevant for
statuses returned from the notifications endpoint, which are not nested
Remove "Disable notifications" from detailed status view, since it's
only relevant in the notifications column
* Up max class length
* Remove pending test for conversation mute
* Add tests, clean up
* Rename to "mute conversation" and "unmute conversation"
* Raise validation error when trying to mute/unmute status without conversation
2017-05-15 11:04:13 +10:00
|
|
|
let(:activity) { Fabricate(:follow, account: sender, target_account: recipient) }
|
2020-09-19 01:26:45 +10:00
|
|
|
let(:type) { :follow }
|
2017-05-07 07:06:52 +10:00
|
|
|
|
2022-03-28 21:43:58 +11:00
|
|
|
it { expect { subject }.to change(Notification, :count).by(1) }
|
2017-05-07 07:06:52 +10:00
|
|
|
|
Feature conversations muting (#3017)
* Add <ostatus:conversation /> tag to Atom input/output
Only uses ref attribute (not href) because href would be
the alternate link that's always included also.
Creates new conversation for every non-reply status. Carries
over conversation for every reply. Keeps remote URIs verbatim,
generates local URIs on the fly like the rest of them.
* Conversation muting - prevents notifications that reference a conversation
(including replies, favourites, reblogs) from being created. API endpoints
/api/v1/statuses/:id/mute and /api/v1/statuses/:id/unmute
Currently no way to tell when a status/conversation is muted, so the web UI
only has a "disable notifications" button, doesn't work as a toggle
* Display "Dismiss notifications" on all statuses in notifications column, not just own
* Add "muted" as a boolean attribute on statuses JSON
For now always false on contained reblogs, since it's only relevant for
statuses returned from the notifications endpoint, which are not nested
Remove "Disable notifications" from detailed status view, since it's
only relevant in the notifications column
* Up max class length
* Remove pending test for conversation mute
* Add tests, clean up
* Rename to "mute conversation" and "unmute conversation"
* Raise validation error when trying to mute/unmute status without conversation
2017-05-15 11:04:13 +10:00
|
|
|
it 'does not notify when sender is blocked' do
|
|
|
|
recipient.block!(sender)
|
2022-03-28 21:43:58 +11:00
|
|
|
expect { subject }.to_not change(Notification, :count)
|
Feature conversations muting (#3017)
* Add <ostatus:conversation /> tag to Atom input/output
Only uses ref attribute (not href) because href would be
the alternate link that's always included also.
Creates new conversation for every non-reply status. Carries
over conversation for every reply. Keeps remote URIs verbatim,
generates local URIs on the fly like the rest of them.
* Conversation muting - prevents notifications that reference a conversation
(including replies, favourites, reblogs) from being created. API endpoints
/api/v1/statuses/:id/mute and /api/v1/statuses/:id/unmute
Currently no way to tell when a status/conversation is muted, so the web UI
only has a "disable notifications" button, doesn't work as a toggle
* Display "Dismiss notifications" on all statuses in notifications column, not just own
* Add "muted" as a boolean attribute on statuses JSON
For now always false on contained reblogs, since it's only relevant for
statuses returned from the notifications endpoint, which are not nested
Remove "Disable notifications" from detailed status view, since it's
only relevant in the notifications column
* Up max class length
* Remove pending test for conversation mute
* Add tests, clean up
* Rename to "mute conversation" and "unmute conversation"
* Raise validation error when trying to mute/unmute status without conversation
2017-05-15 11:04:13 +10:00
|
|
|
end
|
|
|
|
|
2017-11-15 13:56:41 +11:00
|
|
|
it 'does not notify when sender is muted with hide_notifications' do
|
|
|
|
recipient.mute!(sender, notifications: true)
|
2022-03-28 21:43:58 +11:00
|
|
|
expect { subject }.to_not change(Notification, :count)
|
2017-11-15 13:56:41 +11:00
|
|
|
end
|
|
|
|
|
|
|
|
it 'does notify when sender is muted without hide_notifications' do
|
|
|
|
recipient.mute!(sender, notifications: false)
|
2022-03-28 21:43:58 +11:00
|
|
|
expect { subject }.to change(Notification, :count)
|
2017-11-15 13:56:41 +11:00
|
|
|
end
|
|
|
|
|
Account domain blocks (#2381)
* Add <ostatus:conversation /> tag to Atom input/output
Only uses ref attribute (not href) because href would be
the alternate link that's always included also.
Creates new conversation for every non-reply status. Carries
over conversation for every reply. Keeps remote URIs verbatim,
generates local URIs on the fly like the rest of them.
* Conversation muting - prevents notifications that reference a conversation
(including replies, favourites, reblogs) from being created. API endpoints
/api/v1/statuses/:id/mute and /api/v1/statuses/:id/unmute
Currently no way to tell when a status/conversation is muted, so the web UI
only has a "disable notifications" button, doesn't work as a toggle
* Display "Dismiss notifications" on all statuses in notifications column, not just own
* Add "muted" as a boolean attribute on statuses JSON
For now always false on contained reblogs, since it's only relevant for
statuses returned from the notifications endpoint, which are not nested
Remove "Disable notifications" from detailed status view, since it's
only relevant in the notifications column
* Up max class length
* Remove pending test for conversation mute
* Add tests, clean up
* Rename to "mute conversation" and "unmute conversation"
* Raise validation error when trying to mute/unmute status without conversation
* Adding account domain blocks that filter notifications and public timelines
* Add tests for domain blocks in notifications, public timelines
Filter reblogs of blocked domains from home
* Add API for listing and creating account domain blocks
* API for creating/deleting domain blocks, tests for Status#ancestors
and Status#descendants, filter domain blocks from them
* Filter domains in streaming API
* Update account_domain_block_spec.rb
2017-05-19 09:14:30 +10:00
|
|
|
it 'does not notify when sender\'s domain is blocked' do
|
|
|
|
recipient.block_domain!(sender.domain)
|
2022-03-28 21:43:58 +11:00
|
|
|
expect { subject }.to_not change(Notification, :count)
|
Account domain blocks (#2381)
* Add <ostatus:conversation /> tag to Atom input/output
Only uses ref attribute (not href) because href would be
the alternate link that's always included also.
Creates new conversation for every non-reply status. Carries
over conversation for every reply. Keeps remote URIs verbatim,
generates local URIs on the fly like the rest of them.
* Conversation muting - prevents notifications that reference a conversation
(including replies, favourites, reblogs) from being created. API endpoints
/api/v1/statuses/:id/mute and /api/v1/statuses/:id/unmute
Currently no way to tell when a status/conversation is muted, so the web UI
only has a "disable notifications" button, doesn't work as a toggle
* Display "Dismiss notifications" on all statuses in notifications column, not just own
* Add "muted" as a boolean attribute on statuses JSON
For now always false on contained reblogs, since it's only relevant for
statuses returned from the notifications endpoint, which are not nested
Remove "Disable notifications" from detailed status view, since it's
only relevant in the notifications column
* Up max class length
* Remove pending test for conversation mute
* Add tests, clean up
* Rename to "mute conversation" and "unmute conversation"
* Raise validation error when trying to mute/unmute status without conversation
* Adding account domain blocks that filter notifications and public timelines
* Add tests for domain blocks in notifications, public timelines
Filter reblogs of blocked domains from home
* Add API for listing and creating account domain blocks
* API for creating/deleting domain blocks, tests for Status#ancestors
and Status#descendants, filter domain blocks from them
* Filter domains in streaming API
* Update account_domain_block_spec.rb
2017-05-19 09:14:30 +10:00
|
|
|
end
|
|
|
|
|
2017-05-20 05:05:32 +10:00
|
|
|
it 'does still notify when sender\'s domain is blocked but sender is followed' do
|
|
|
|
recipient.block_domain!(sender.domain)
|
|
|
|
recipient.follow!(sender)
|
2022-03-28 21:43:58 +11:00
|
|
|
expect { subject }.to change(Notification, :count)
|
2017-05-20 05:05:32 +10:00
|
|
|
end
|
|
|
|
|
Feature conversations muting (#3017)
* Add <ostatus:conversation /> tag to Atom input/output
Only uses ref attribute (not href) because href would be
the alternate link that's always included also.
Creates new conversation for every non-reply status. Carries
over conversation for every reply. Keeps remote URIs verbatim,
generates local URIs on the fly like the rest of them.
* Conversation muting - prevents notifications that reference a conversation
(including replies, favourites, reblogs) from being created. API endpoints
/api/v1/statuses/:id/mute and /api/v1/statuses/:id/unmute
Currently no way to tell when a status/conversation is muted, so the web UI
only has a "disable notifications" button, doesn't work as a toggle
* Display "Dismiss notifications" on all statuses in notifications column, not just own
* Add "muted" as a boolean attribute on statuses JSON
For now always false on contained reblogs, since it's only relevant for
statuses returned from the notifications endpoint, which are not nested
Remove "Disable notifications" from detailed status view, since it's
only relevant in the notifications column
* Up max class length
* Remove pending test for conversation mute
* Add tests, clean up
* Rename to "mute conversation" and "unmute conversation"
* Raise validation error when trying to mute/unmute status without conversation
2017-05-15 11:04:13 +10:00
|
|
|
it 'does not notify when sender is silenced and not followed' do
|
2019-05-15 03:05:02 +10:00
|
|
|
sender.silence!
|
2024-03-08 01:53:37 +11:00
|
|
|
subject
|
|
|
|
expect(Notification.find_by(activity: activity).filtered?).to be true
|
Feature conversations muting (#3017)
* Add <ostatus:conversation /> tag to Atom input/output
Only uses ref attribute (not href) because href would be
the alternate link that's always included also.
Creates new conversation for every non-reply status. Carries
over conversation for every reply. Keeps remote URIs verbatim,
generates local URIs on the fly like the rest of them.
* Conversation muting - prevents notifications that reference a conversation
(including replies, favourites, reblogs) from being created. API endpoints
/api/v1/statuses/:id/mute and /api/v1/statuses/:id/unmute
Currently no way to tell when a status/conversation is muted, so the web UI
only has a "disable notifications" button, doesn't work as a toggle
* Display "Dismiss notifications" on all statuses in notifications column, not just own
* Add "muted" as a boolean attribute on statuses JSON
For now always false on contained reblogs, since it's only relevant for
statuses returned from the notifications endpoint, which are not nested
Remove "Disable notifications" from detailed status view, since it's
only relevant in the notifications column
* Up max class length
* Remove pending test for conversation mute
* Add tests, clean up
* Rename to "mute conversation" and "unmute conversation"
* Raise validation error when trying to mute/unmute status without conversation
2017-05-15 11:04:13 +10:00
|
|
|
end
|
|
|
|
|
|
|
|
it 'does not notify when recipient is suspended' do
|
2019-05-15 03:05:02 +10:00
|
|
|
recipient.suspend!
|
2022-03-28 21:43:58 +11:00
|
|
|
expect { subject }.to_not change(Notification, :count)
|
Feature conversations muting (#3017)
* Add <ostatus:conversation /> tag to Atom input/output
Only uses ref attribute (not href) because href would be
the alternate link that's always included also.
Creates new conversation for every non-reply status. Carries
over conversation for every reply. Keeps remote URIs verbatim,
generates local URIs on the fly like the rest of them.
* Conversation muting - prevents notifications that reference a conversation
(including replies, favourites, reblogs) from being created. API endpoints
/api/v1/statuses/:id/mute and /api/v1/statuses/:id/unmute
Currently no way to tell when a status/conversation is muted, so the web UI
only has a "disable notifications" button, doesn't work as a toggle
* Display "Dismiss notifications" on all statuses in notifications column, not just own
* Add "muted" as a boolean attribute on statuses JSON
For now always false on contained reblogs, since it's only relevant for
statuses returned from the notifications endpoint, which are not nested
Remove "Disable notifications" from detailed status view, since it's
only relevant in the notifications column
* Up max class length
* Remove pending test for conversation mute
* Add tests, clean up
* Rename to "mute conversation" and "unmute conversation"
* Raise validation error when trying to mute/unmute status without conversation
2017-05-15 11:04:13 +10:00
|
|
|
end
|
|
|
|
|
Allow hiding of reblogs from followed users (#5762)
* Allow hiding of reblogs from followed users
This adds a new entry to the account menu to allow users to hide
future reblogs from a user (and then if they've done that, to show
future reblogs instead).
This does not remove or add historical reblogs from/to the user's
timeline; it only affects new statuses.
The API for this operates by sending a "reblogs" key to the follow
endpoint. If this is sent when starting a new follow, it will be
respected from the beginning of the follow relationship (even if
the follow request must be approved by the followee). If this is
sent when a follow relationship already exists, it will simply
update the existing follow relationship. As with the notification
muting, this will now return an object ({reblogs: [true|false]}) or
false for each follow relationship when requesting relationship
information for an account. This should cause few issues due to an
object being truthy in many languages, but some modifications may
need to be made in pickier languages.
Database changes: adds a show_reblogs column (default true,
non-nullable) to the follows and follow_requests tables. Because
these are non-nullable, we use the existing MigrationHelpers to
perform this change without locking those tables, although the
tables are likely to be small anyway.
Tests included.
See also <https://github.com/glitch-soc/mastodon/pull/212>.
* Rubocop fixes
* Code review changes
* Test fixes
This patchset closes #648 and resolves #3271.
* Rubocop fix
* Revert reblogs defaulting in argument, fix tests
It turns out we needed this for the same reason we needed it in muting:
if nil gets passed in somehow (most usually by an API client not passing
any value), we need to detect and handle it.
We could specify a default in the parameter and then also catch nil, but
there's no great reason to duplicate the default value.
2017-11-29 01:00:35 +11:00
|
|
|
describe 'reblogs' do
|
|
|
|
let(:status) { Fabricate(:status, account: Fabricate(:account)) }
|
|
|
|
let(:activity) { Fabricate(:status, account: sender, reblog: status) }
|
2020-09-19 01:26:45 +10:00
|
|
|
let(:type) { :reblog }
|
Allow hiding of reblogs from followed users (#5762)
* Allow hiding of reblogs from followed users
This adds a new entry to the account menu to allow users to hide
future reblogs from a user (and then if they've done that, to show
future reblogs instead).
This does not remove or add historical reblogs from/to the user's
timeline; it only affects new statuses.
The API for this operates by sending a "reblogs" key to the follow
endpoint. If this is sent when starting a new follow, it will be
respected from the beginning of the follow relationship (even if
the follow request must be approved by the followee). If this is
sent when a follow relationship already exists, it will simply
update the existing follow relationship. As with the notification
muting, this will now return an object ({reblogs: [true|false]}) or
false for each follow relationship when requesting relationship
information for an account. This should cause few issues due to an
object being truthy in many languages, but some modifications may
need to be made in pickier languages.
Database changes: adds a show_reblogs column (default true,
non-nullable) to the follows and follow_requests tables. Because
these are non-nullable, we use the existing MigrationHelpers to
perform this change without locking those tables, although the
tables are likely to be small anyway.
Tests included.
See also <https://github.com/glitch-soc/mastodon/pull/212>.
* Rubocop fixes
* Code review changes
* Test fixes
This patchset closes #648 and resolves #3271.
* Rubocop fix
* Revert reblogs defaulting in argument, fix tests
It turns out we needed this for the same reason we needed it in muting:
if nil gets passed in somehow (most usually by an API client not passing
any value), we need to detect and handle it.
We could specify a default in the parameter and then also catch nil, but
there's no great reason to duplicate the default value.
2017-11-29 01:00:35 +11:00
|
|
|
|
|
|
|
it 'shows reblogs by default' do
|
|
|
|
recipient.follow!(sender)
|
2022-03-28 21:43:58 +11:00
|
|
|
expect { subject }.to change(Notification, :count)
|
Allow hiding of reblogs from followed users (#5762)
* Allow hiding of reblogs from followed users
This adds a new entry to the account menu to allow users to hide
future reblogs from a user (and then if they've done that, to show
future reblogs instead).
This does not remove or add historical reblogs from/to the user's
timeline; it only affects new statuses.
The API for this operates by sending a "reblogs" key to the follow
endpoint. If this is sent when starting a new follow, it will be
respected from the beginning of the follow relationship (even if
the follow request must be approved by the followee). If this is
sent when a follow relationship already exists, it will simply
update the existing follow relationship. As with the notification
muting, this will now return an object ({reblogs: [true|false]}) or
false for each follow relationship when requesting relationship
information for an account. This should cause few issues due to an
object being truthy in many languages, but some modifications may
need to be made in pickier languages.
Database changes: adds a show_reblogs column (default true,
non-nullable) to the follows and follow_requests tables. Because
these are non-nullable, we use the existing MigrationHelpers to
perform this change without locking those tables, although the
tables are likely to be small anyway.
Tests included.
See also <https://github.com/glitch-soc/mastodon/pull/212>.
* Rubocop fixes
* Code review changes
* Test fixes
This patchset closes #648 and resolves #3271.
* Rubocop fix
* Revert reblogs defaulting in argument, fix tests
It turns out we needed this for the same reason we needed it in muting:
if nil gets passed in somehow (most usually by an API client not passing
any value), we need to detect and handle it.
We could specify a default in the parameter and then also catch nil, but
there's no great reason to duplicate the default value.
2017-11-29 01:00:35 +11:00
|
|
|
end
|
|
|
|
|
|
|
|
it 'shows reblogs when explicitly enabled' do
|
|
|
|
recipient.follow!(sender, reblogs: true)
|
2022-03-28 21:43:58 +11:00
|
|
|
expect { subject }.to change(Notification, :count)
|
Allow hiding of reblogs from followed users (#5762)
* Allow hiding of reblogs from followed users
This adds a new entry to the account menu to allow users to hide
future reblogs from a user (and then if they've done that, to show
future reblogs instead).
This does not remove or add historical reblogs from/to the user's
timeline; it only affects new statuses.
The API for this operates by sending a "reblogs" key to the follow
endpoint. If this is sent when starting a new follow, it will be
respected from the beginning of the follow relationship (even if
the follow request must be approved by the followee). If this is
sent when a follow relationship already exists, it will simply
update the existing follow relationship. As with the notification
muting, this will now return an object ({reblogs: [true|false]}) or
false for each follow relationship when requesting relationship
information for an account. This should cause few issues due to an
object being truthy in many languages, but some modifications may
need to be made in pickier languages.
Database changes: adds a show_reblogs column (default true,
non-nullable) to the follows and follow_requests tables. Because
these are non-nullable, we use the existing MigrationHelpers to
perform this change without locking those tables, although the
tables are likely to be small anyway.
Tests included.
See also <https://github.com/glitch-soc/mastodon/pull/212>.
* Rubocop fixes
* Code review changes
* Test fixes
This patchset closes #648 and resolves #3271.
* Rubocop fix
* Revert reblogs defaulting in argument, fix tests
It turns out we needed this for the same reason we needed it in muting:
if nil gets passed in somehow (most usually by an API client not passing
any value), we need to detect and handle it.
We could specify a default in the parameter and then also catch nil, but
there's no great reason to duplicate the default value.
2017-11-29 01:00:35 +11:00
|
|
|
end
|
|
|
|
|
2018-10-30 10:47:31 +11:00
|
|
|
it 'shows reblogs when disabled' do
|
2019-06-03 02:08:26 +10:00
|
|
|
recipient.follow!(sender, reblogs: false)
|
2022-03-28 21:43:58 +11:00
|
|
|
expect { subject }.to change(Notification, :count)
|
Allow hiding of reblogs from followed users (#5762)
* Allow hiding of reblogs from followed users
This adds a new entry to the account menu to allow users to hide
future reblogs from a user (and then if they've done that, to show
future reblogs instead).
This does not remove or add historical reblogs from/to the user's
timeline; it only affects new statuses.
The API for this operates by sending a "reblogs" key to the follow
endpoint. If this is sent when starting a new follow, it will be
respected from the beginning of the follow relationship (even if
the follow request must be approved by the followee). If this is
sent when a follow relationship already exists, it will simply
update the existing follow relationship. As with the notification
muting, this will now return an object ({reblogs: [true|false]}) or
false for each follow relationship when requesting relationship
information for an account. This should cause few issues due to an
object being truthy in many languages, but some modifications may
need to be made in pickier languages.
Database changes: adds a show_reblogs column (default true,
non-nullable) to the follows and follow_requests tables. Because
these are non-nullable, we use the existing MigrationHelpers to
perform this change without locking those tables, although the
tables are likely to be small anyway.
Tests included.
See also <https://github.com/glitch-soc/mastodon/pull/212>.
* Rubocop fixes
* Code review changes
* Test fixes
This patchset closes #648 and resolves #3271.
* Rubocop fix
* Revert reblogs defaulting in argument, fix tests
It turns out we needed this for the same reason we needed it in muting:
if nil gets passed in somehow (most usually by an API client not passing
any value), we need to detect and handle it.
We could specify a default in the parameter and then also catch nil, but
there's no great reason to duplicate the default value.
2017-11-29 01:00:35 +11:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2023-06-06 23:51:42 +10:00
|
|
|
context 'with muted and blocked users' do
|
Feature conversations muting (#3017)
* Add <ostatus:conversation /> tag to Atom input/output
Only uses ref attribute (not href) because href would be
the alternate link that's always included also.
Creates new conversation for every non-reply status. Carries
over conversation for every reply. Keeps remote URIs verbatim,
generates local URIs on the fly like the rest of them.
* Conversation muting - prevents notifications that reference a conversation
(including replies, favourites, reblogs) from being created. API endpoints
/api/v1/statuses/:id/mute and /api/v1/statuses/:id/unmute
Currently no way to tell when a status/conversation is muted, so the web UI
only has a "disable notifications" button, doesn't work as a toggle
* Display "Dismiss notifications" on all statuses in notifications column, not just own
* Add "muted" as a boolean attribute on statuses JSON
For now always false on contained reblogs, since it's only relevant for
statuses returned from the notifications endpoint, which are not nested
Remove "Disable notifications" from detailed status view, since it's
only relevant in the notifications column
* Up max class length
* Remove pending test for conversation mute
* Add tests, clean up
* Rename to "mute conversation" and "unmute conversation"
* Raise validation error when trying to mute/unmute status without conversation
2017-05-15 11:04:13 +10:00
|
|
|
let(:asshole) { Fabricate(:account, username: 'asshole') }
|
|
|
|
let(:reply_to) { Fabricate(:status, account: asshole) }
|
|
|
|
let(:activity) { Fabricate(:mention, account: recipient, status: Fabricate(:status, account: sender, thread: reply_to)) }
|
2020-09-19 01:26:45 +10:00
|
|
|
let(:type) { :mention }
|
Feature conversations muting (#3017)
* Add <ostatus:conversation /> tag to Atom input/output
Only uses ref attribute (not href) because href would be
the alternate link that's always included also.
Creates new conversation for every non-reply status. Carries
over conversation for every reply. Keeps remote URIs verbatim,
generates local URIs on the fly like the rest of them.
* Conversation muting - prevents notifications that reference a conversation
(including replies, favourites, reblogs) from being created. API endpoints
/api/v1/statuses/:id/mute and /api/v1/statuses/:id/unmute
Currently no way to tell when a status/conversation is muted, so the web UI
only has a "disable notifications" button, doesn't work as a toggle
* Display "Dismiss notifications" on all statuses in notifications column, not just own
* Add "muted" as a boolean attribute on statuses JSON
For now always false on contained reblogs, since it's only relevant for
statuses returned from the notifications endpoint, which are not nested
Remove "Disable notifications" from detailed status view, since it's
only relevant in the notifications column
* Up max class length
* Remove pending test for conversation mute
* Add tests, clean up
* Rename to "mute conversation" and "unmute conversation"
* Raise validation error when trying to mute/unmute status without conversation
2017-05-15 11:04:13 +10:00
|
|
|
|
|
|
|
it 'does not notify when conversation is muted' do
|
|
|
|
recipient.mute_conversation!(activity.status.conversation)
|
2022-03-28 21:43:58 +11:00
|
|
|
expect { subject }.to_not change(Notification, :count)
|
Feature conversations muting (#3017)
* Add <ostatus:conversation /> tag to Atom input/output
Only uses ref attribute (not href) because href would be
the alternate link that's always included also.
Creates new conversation for every non-reply status. Carries
over conversation for every reply. Keeps remote URIs verbatim,
generates local URIs on the fly like the rest of them.
* Conversation muting - prevents notifications that reference a conversation
(including replies, favourites, reblogs) from being created. API endpoints
/api/v1/statuses/:id/mute and /api/v1/statuses/:id/unmute
Currently no way to tell when a status/conversation is muted, so the web UI
only has a "disable notifications" button, doesn't work as a toggle
* Display "Dismiss notifications" on all statuses in notifications column, not just own
* Add "muted" as a boolean attribute on statuses JSON
For now always false on contained reblogs, since it's only relevant for
statuses returned from the notifications endpoint, which are not nested
Remove "Disable notifications" from detailed status view, since it's
only relevant in the notifications column
* Up max class length
* Remove pending test for conversation mute
* Add tests, clean up
* Rename to "mute conversation" and "unmute conversation"
* Raise validation error when trying to mute/unmute status without conversation
2017-05-15 11:04:13 +10:00
|
|
|
end
|
|
|
|
|
|
|
|
it 'does not notify when it is a reply to a blocked user' do
|
|
|
|
recipient.block!(asshole)
|
2022-03-28 21:43:58 +11:00
|
|
|
expect { subject }.to_not change(Notification, :count)
|
Feature conversations muting (#3017)
* Add <ostatus:conversation /> tag to Atom input/output
Only uses ref attribute (not href) because href would be
the alternate link that's always included also.
Creates new conversation for every non-reply status. Carries
over conversation for every reply. Keeps remote URIs verbatim,
generates local URIs on the fly like the rest of them.
* Conversation muting - prevents notifications that reference a conversation
(including replies, favourites, reblogs) from being created. API endpoints
/api/v1/statuses/:id/mute and /api/v1/statuses/:id/unmute
Currently no way to tell when a status/conversation is muted, so the web UI
only has a "disable notifications" button, doesn't work as a toggle
* Display "Dismiss notifications" on all statuses in notifications column, not just own
* Add "muted" as a boolean attribute on statuses JSON
For now always false on contained reblogs, since it's only relevant for
statuses returned from the notifications endpoint, which are not nested
Remove "Disable notifications" from detailed status view, since it's
only relevant in the notifications column
* Up max class length
* Remove pending test for conversation mute
* Add tests, clean up
* Rename to "mute conversation" and "unmute conversation"
* Raise validation error when trying to mute/unmute status without conversation
2017-05-15 11:04:13 +10:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2023-06-06 23:51:42 +10:00
|
|
|
context 'with sender as recipient' do
|
Feature conversations muting (#3017)
* Add <ostatus:conversation /> tag to Atom input/output
Only uses ref attribute (not href) because href would be
the alternate link that's always included also.
Creates new conversation for every non-reply status. Carries
over conversation for every reply. Keeps remote URIs verbatim,
generates local URIs on the fly like the rest of them.
* Conversation muting - prevents notifications that reference a conversation
(including replies, favourites, reblogs) from being created. API endpoints
/api/v1/statuses/:id/mute and /api/v1/statuses/:id/unmute
Currently no way to tell when a status/conversation is muted, so the web UI
only has a "disable notifications" button, doesn't work as a toggle
* Display "Dismiss notifications" on all statuses in notifications column, not just own
* Add "muted" as a boolean attribute on statuses JSON
For now always false on contained reblogs, since it's only relevant for
statuses returned from the notifications endpoint, which are not nested
Remove "Disable notifications" from detailed status view, since it's
only relevant in the notifications column
* Up max class length
* Remove pending test for conversation mute
* Add tests, clean up
* Rename to "mute conversation" and "unmute conversation"
* Raise validation error when trying to mute/unmute status without conversation
2017-05-15 11:04:13 +10:00
|
|
|
let(:sender) { recipient }
|
|
|
|
|
|
|
|
it 'does not notify when recipient is the sender' do
|
2022-03-28 21:43:58 +11:00
|
|
|
expect { subject }.to_not change(Notification, :count)
|
Feature conversations muting (#3017)
* Add <ostatus:conversation /> tag to Atom input/output
Only uses ref attribute (not href) because href would be
the alternate link that's always included also.
Creates new conversation for every non-reply status. Carries
over conversation for every reply. Keeps remote URIs verbatim,
generates local URIs on the fly like the rest of them.
* Conversation muting - prevents notifications that reference a conversation
(including replies, favourites, reblogs) from being created. API endpoints
/api/v1/statuses/:id/mute and /api/v1/statuses/:id/unmute
Currently no way to tell when a status/conversation is muted, so the web UI
only has a "disable notifications" button, doesn't work as a toggle
* Display "Dismiss notifications" on all statuses in notifications column, not just own
* Add "muted" as a boolean attribute on statuses JSON
For now always false on contained reblogs, since it's only relevant for
statuses returned from the notifications endpoint, which are not nested
Remove "Disable notifications" from detailed status view, since it's
only relevant in the notifications column
* Up max class length
* Remove pending test for conversation mute
* Add tests, clean up
* Rename to "mute conversation" and "unmute conversation"
* Raise validation error when trying to mute/unmute status without conversation
2017-05-15 11:04:13 +10:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2017-05-07 07:06:52 +10:00
|
|
|
describe 'email' do
|
|
|
|
before do
|
2023-03-30 23:44:00 +11:00
|
|
|
user.settings.update('notification_emails.follow': enabled)
|
|
|
|
user.save
|
2017-05-07 07:06:52 +10:00
|
|
|
end
|
|
|
|
|
|
|
|
context 'when email notification is enabled' do
|
|
|
|
let(:enabled) { true }
|
|
|
|
|
2024-07-09 02:01:08 +10:00
|
|
|
it 'sends email', :inline_jobs do
|
2024-02-20 02:57:47 +11:00
|
|
|
emails = capture_emails { subject }
|
|
|
|
|
|
|
|
expect(emails.size)
|
|
|
|
.to eq(1)
|
|
|
|
expect(emails.first)
|
|
|
|
.to have_attributes(
|
|
|
|
to: contain_exactly(user.email),
|
|
|
|
subject: eq(I18n.t('notification_mailer.follow.subject', name: sender.acct))
|
|
|
|
)
|
2017-05-07 07:06:52 +10:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when email notification is disabled' do
|
|
|
|
let(:enabled) { false }
|
|
|
|
|
|
|
|
it "doesn't send email" do
|
2024-02-20 02:57:47 +11:00
|
|
|
emails = capture_emails { subject }
|
|
|
|
|
|
|
|
expect(emails).to be_empty
|
2017-05-07 07:06:52 +10:00
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
2024-03-08 01:53:37 +11:00
|
|
|
|
2024-08-07 16:52:10 +10:00
|
|
|
context 'when the blocked sender has a role' do
|
|
|
|
let(:sender) { Fabricate(:user, role: sender_role).account }
|
|
|
|
let(:activity) { Fabricate(:mention, status: Fabricate(:status, account: sender)) }
|
|
|
|
let(:type) { :mention }
|
|
|
|
|
|
|
|
before do
|
|
|
|
recipient.block!(sender)
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when the role is a visible moderator' do
|
|
|
|
let(:sender_role) { Fabricate(:user_role, highlighted: true, permissions: UserRole::FLAGS[:manage_users]) }
|
|
|
|
|
|
|
|
it 'does notify' do
|
|
|
|
expect { subject }.to change(Notification, :count)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when the role is a non-visible moderator' do
|
|
|
|
let(:sender_role) { Fabricate(:user_role, highlighted: false, permissions: UserRole::FLAGS[:manage_users]) }
|
|
|
|
|
|
|
|
it 'does not notify' do
|
|
|
|
expect { subject }.to_not change(Notification, :count)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when the role is a visible non-moderator' do
|
|
|
|
let(:sender_role) { Fabricate(:user_role, highlighted: true) }
|
|
|
|
|
|
|
|
it 'does not notify' do
|
|
|
|
expect { subject }.to_not change(Notification, :count)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2024-07-12 22:09:52 +10:00
|
|
|
context 'with filtered notifications' do
|
|
|
|
let(:unknown) { Fabricate(:account, username: 'unknown') }
|
|
|
|
let(:status) { Fabricate(:status, account: unknown) }
|
|
|
|
let(:activity) { Fabricate(:mention, account: recipient, status: status) }
|
|
|
|
let(:type) { :mention }
|
|
|
|
|
|
|
|
before do
|
|
|
|
Fabricate(:notification_policy, account: recipient, filter_not_following: true)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'creates a filtered notification' do
|
|
|
|
expect { subject }.to change(Notification, :count)
|
|
|
|
expect(Notification.last).to be_filtered
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when no notification request exists' do
|
|
|
|
it 'creates a notification request' do
|
|
|
|
expect { subject }.to change(NotificationRequest, :count)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when a notification request exists' do
|
|
|
|
let!(:notification_request) do
|
|
|
|
Fabricate(:notification_request, account: recipient, from_account: unknown, last_status: Fabricate(:status, account: unknown))
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'updates the existing notification request' do
|
|
|
|
expect { subject }.to_not change(NotificationRequest, :count)
|
|
|
|
expect(notification_request.reload.last_status).to eq status
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2024-08-09 23:30:55 +10:00
|
|
|
describe NotifyService::DropCondition do
|
2024-06-12 23:50:38 +10:00
|
|
|
subject { described_class.new(notification) }
|
|
|
|
|
|
|
|
let(:activity) { Fabricate(:mention, status: Fabricate(:status)) }
|
|
|
|
let(:notification) { Fabricate(:notification, type: :mention, activity: activity, from_account: activity.status.account, account: activity.account) }
|
|
|
|
|
2024-08-09 23:30:55 +10:00
|
|
|
describe '#drop' do
|
|
|
|
context 'when sender is silenced and recipient has a default policy' do
|
|
|
|
before do
|
|
|
|
notification.from_account.silence!
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns false' do
|
|
|
|
expect(subject.drop?).to be false
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when sender is silenced and recipient has a policy to ignore silenced accounts' do
|
2024-06-12 23:50:38 +10:00
|
|
|
before do
|
|
|
|
notification.from_account.silence!
|
2024-08-09 23:30:55 +10:00
|
|
|
notification.account.create_notification_policy!(for_limited_accounts: :drop)
|
2024-06-12 23:50:38 +10:00
|
|
|
end
|
|
|
|
|
2024-08-09 23:30:55 +10:00
|
|
|
it 'returns true' do
|
|
|
|
expect(subject.drop?).to be true
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when sender is new and recipient has a default policy' do
|
2024-06-12 23:50:38 +10:00
|
|
|
it 'returns false' do
|
2024-08-09 23:30:55 +10:00
|
|
|
expect(subject.drop?).to be false
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when sender is new and recipient has a policy to ignore silenced accounts' do
|
|
|
|
before do
|
|
|
|
notification.account.create_notification_policy!(for_new_accounts: :drop)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns true' do
|
|
|
|
expect(subject.drop?).to be true
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when sender is new and followed and recipient has a policy to ignore silenced accounts' do
|
|
|
|
before do
|
|
|
|
notification.account.create_notification_policy!(for_new_accounts: :drop)
|
|
|
|
notification.account.follow!(notification.from_account)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns false' do
|
|
|
|
expect(subject.drop?).to be false
|
2024-06-12 23:50:38 +10:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when recipient has blocked sender' do
|
|
|
|
before do
|
|
|
|
notification.account.block!(notification.from_account)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns true' do
|
2024-08-09 23:30:55 +10:00
|
|
|
expect(subject.drop?).to be true
|
2024-06-12 23:50:38 +10:00
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2024-03-08 01:53:37 +11:00
|
|
|
describe NotifyService::FilterCondition do
|
|
|
|
subject { described_class.new(notification) }
|
|
|
|
|
|
|
|
let(:activity) { Fabricate(:mention, status: Fabricate(:status)) }
|
|
|
|
let(:notification) { Fabricate(:notification, type: :mention, activity: activity, from_account: activity.status.account, account: activity.account) }
|
|
|
|
|
|
|
|
describe '#filter?' do
|
|
|
|
context 'when sender is silenced' do
|
|
|
|
before do
|
|
|
|
notification.from_account.silence!
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns true' do
|
|
|
|
expect(subject.filter?).to be true
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when recipient follows sender' do
|
|
|
|
before do
|
|
|
|
notification.account.follow!(notification.from_account)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns false' do
|
|
|
|
expect(subject.filter?).to be false
|
|
|
|
end
|
|
|
|
end
|
2024-08-09 23:30:55 +10:00
|
|
|
|
|
|
|
context 'when recipient is allowing limited accounts' do
|
|
|
|
before do
|
|
|
|
notification.account.create_notification_policy!(for_limited_accounts: :accept)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns false' do
|
|
|
|
expect(subject.filter?).to be false
|
|
|
|
end
|
|
|
|
end
|
2024-03-08 01:53:37 +11:00
|
|
|
end
|
|
|
|
|
|
|
|
context 'when recipient is filtering not-followed senders' do
|
|
|
|
before do
|
|
|
|
Fabricate(:notification_policy, account: notification.account, filter_not_following: true)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns true' do
|
|
|
|
expect(subject.filter?).to be true
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when sender has permission' do
|
|
|
|
before do
|
|
|
|
Fabricate(:notification_permission, account: notification.account, from_account: notification.from_account)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns false' do
|
|
|
|
expect(subject.filter?).to be false
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when sender is followed by recipient' do
|
|
|
|
before do
|
|
|
|
notification.account.follow!(notification.from_account)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns false' do
|
|
|
|
expect(subject.filter?).to be false
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when recipient is filtering not-followers' do
|
|
|
|
before do
|
|
|
|
Fabricate(:notification_policy, account: notification.account, filter_not_followers: true)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns true' do
|
|
|
|
expect(subject.filter?).to be true
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when sender has permission' do
|
|
|
|
before do
|
|
|
|
Fabricate(:notification_permission, account: notification.account, from_account: notification.from_account)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns false' do
|
|
|
|
expect(subject.filter?).to be false
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when sender follows recipient' do
|
|
|
|
before do
|
|
|
|
notification.from_account.follow!(notification.account)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns true' do
|
|
|
|
expect(subject.filter?).to be true
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when sender follows recipient for longer than 3 days' do
|
|
|
|
before do
|
|
|
|
follow = notification.from_account.follow!(notification.account)
|
|
|
|
follow.update(created_at: 4.days.ago)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns false' do
|
|
|
|
expect(subject.filter?).to be false
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when recipient is filtering new accounts' do
|
|
|
|
before do
|
|
|
|
Fabricate(:notification_policy, account: notification.account, filter_new_accounts: true)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns true' do
|
|
|
|
expect(subject.filter?).to be true
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when sender has permission' do
|
|
|
|
before do
|
|
|
|
Fabricate(:notification_permission, account: notification.account, from_account: notification.from_account)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns false' do
|
|
|
|
expect(subject.filter?).to be false
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when sender is older than 30 days' do
|
|
|
|
before do
|
|
|
|
notification.from_account.update(created_at: 31.days.ago)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns false' do
|
|
|
|
expect(subject.filter?).to be false
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when recipient is not filtering anyone' do
|
|
|
|
before do
|
|
|
|
Fabricate(:notification_policy, account: notification.account)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns false' do
|
|
|
|
expect(subject.filter?).to be false
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when recipient is filtering unsolicited private mentions' do
|
|
|
|
before do
|
|
|
|
Fabricate(:notification_policy, account: notification.account, filter_private_mentions: true)
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when notification is not a private mention' do
|
|
|
|
it 'returns false' do
|
|
|
|
expect(subject.filter?).to be false
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when notification is a private mention' do
|
|
|
|
before do
|
|
|
|
notification.target_status.update(visibility: :direct)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns true' do
|
|
|
|
expect(subject.filter?).to be true
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when the message chain is initiated by recipient, but sender is not mentioned' do
|
|
|
|
before do
|
|
|
|
original_status = Fabricate(:status, account: notification.account, visibility: :direct)
|
|
|
|
notification.target_status.update(thread: original_status)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns true' do
|
|
|
|
expect(subject.filter?).to be true
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when the message chain is initiated by recipient, and sender is mentioned' do
|
|
|
|
before do
|
|
|
|
original_status = Fabricate(:status, account: notification.account, visibility: :direct)
|
|
|
|
notification.target_status.update(thread: original_status)
|
|
|
|
Fabricate(:mention, status: original_status, account: notification.from_account)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns false' do
|
|
|
|
expect(subject.filter?).to be false
|
|
|
|
end
|
|
|
|
end
|
2024-05-30 22:03:13 +10:00
|
|
|
|
|
|
|
context 'when the sender is mentioned in an unrelated message chain' do
|
|
|
|
before do
|
|
|
|
original_status = Fabricate(:status, visibility: :direct)
|
|
|
|
intermediary_status = Fabricate(:status, visibility: :direct, thread: original_status)
|
|
|
|
notification.target_status.update(thread: intermediary_status)
|
|
|
|
Fabricate(:mention, status: original_status, account: notification.from_account)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns true' do
|
|
|
|
expect(subject.filter?).to be true
|
|
|
|
end
|
|
|
|
end
|
2024-03-08 01:53:37 +11:00
|
|
|
end
|
2017-05-07 07:06:52 +10:00
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|