jaculabilis/danbooru
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

add tests to check if approver/rejector is mentioned in moderated bulk update requests

Browse Source
This commit is contained in:
r888888888
2017-11-16 11:41:07 -08:00
parent cd5d9cdaeb
commit 2c620f205b
1 changed files with 9 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
test/unit/bulk_update_request_test.rb
View File

@@ -27,6 +27,10 @@ class BulkUpdateRequestTest < ActiveSupport::TestCase
@ti = TagImplication.where(:antecedent_name => "bar", :consequent_name => "baz").first @ti = TagImplication.where(:antecedent_name => "bar", :consequent_name => "baz").first
end end
should "reference the approver in the automated message" do
assert_match(Regexp.compile(@admin.name), @bur.forum_post.body)
end
should "set the BUR approver" do should "set the BUR approver" do
assert_equal(@admin.id, @bur.approver.id) assert_equal(@admin.id, @bur.approver.id)
end end
@@ -135,6 +139,11 @@ class BulkUpdateRequestTest < ActiveSupport::TestCase
assert_match(/\[REJECTED\]/, @topic.title) assert_match(/\[REJECTED\]/, @topic.title)
end end
should "reference the rejector in the automated message" do
@req.reject!(@admin)
assert_match(Regexp.compile(@admin.name), @req.forum_post.body)
end
should "not send @mention dmails to the approver" do should "not send @mention dmails to the approver" do
assert_no_difference("Dmail.count") do assert_no_difference("Dmail.count") do
@req.approve!(@admin) @req.approve!(@admin)