mirror of
https://github.com/discourse/discourse.git
synced 2025-02-25 18:55:32 -06:00
DEV: Delete redundant bookmark test (#24680)
Cases were identical and also already covered on L63
This commit is contained in:
@@ -159,36 +159,4 @@ describe "Bookmarking posts and topics", type: :system do
|
|||||||
expect(bookmark.reload.name).to eq("test name")
|
expect(bookmark.reload.name).to eq("test name")
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
context "when the user has a bookmark auto_delete_preference" do
|
|
||||||
before do
|
|
||||||
current_user.user_option.update!(
|
|
||||||
bookmark_auto_delete_preference: Bookmark.auto_delete_preferences[:on_owner_reply],
|
|
||||||
)
|
|
||||||
end
|
|
||||||
|
|
||||||
it "is respected when the user creates a new bookmark" do
|
|
||||||
visit_topic_and_open_bookmark_modal(post)
|
|
||||||
|
|
||||||
bookmark_modal.save
|
|
||||||
expect(topic_page).to have_post_bookmarked(post)
|
|
||||||
|
|
||||||
bookmark = Bookmark.find_by(bookmarkable: post, user: current_user)
|
|
||||||
expect(bookmark.auto_delete_preference).to eq(
|
|
||||||
Bookmark.auto_delete_preferences[:on_owner_reply],
|
|
||||||
)
|
|
||||||
end
|
|
||||||
|
|
||||||
it "allows the user to choose a different auto delete preference for a bookmark" do
|
|
||||||
visit_topic_and_open_bookmark_modal(post)
|
|
||||||
|
|
||||||
bookmark_modal.save
|
|
||||||
expect(topic_page).to have_post_bookmarked(post)
|
|
||||||
|
|
||||||
bookmark = Bookmark.find_by(bookmarkable: post, user: current_user)
|
|
||||||
expect(bookmark.auto_delete_preference).to eq(
|
|
||||||
Bookmark.auto_delete_preferences[:on_owner_reply],
|
|
||||||
)
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
end
|
||||||
|
|||||||
Reference in New Issue
Block a user