##// END OF EJS Templates
jro_apps » redmine
RSS
not use unnecessary instance variable at unit message test...
Toshi MARUYAMA -
r10969:10c96dba8b79
parent child
Show More
Side by Side Unified
Context file:
r10969:10c96dba8b79 Loading diff...:
Show file before | Show file after | Hide comments
@@ -47,13 +47,13 class MessageTest < ActiveSupport::TestCase
47 47 def test_reply
48 48 topics_count = @board.topics_count
49 49 messages_count = @board.messages_count
50 @message = Message.find(1)
51 replies_count = @message.replies_count
50 message = Message.find(1)
51 replies_count = message.replies_count
52 52
53 53 reply_author = User.find(2)
54 54 reply = Message.new(:board => @board, :subject => 'Test reply',
55 55 :content => 'Test reply content',
56 :parent => @message, :author => reply_author)
56 :parent => message, :author => reply_author)
57 57 assert reply.save
58 58 @board.reload
59 59 # same topics count
@@ -61,42 +61,42 class MessageTest < ActiveSupport::TestCase
61 61 # messages count incremented
62 62 assert_equal messages_count+1, @board[:messages_count]
63 63 assert_equal reply, @board.last_message
64 @message.reload
64 message.reload
65 65 # replies count incremented
66 assert_equal replies_count+1, @message[:replies_count]
67 assert_equal reply, @message.last_reply
66 assert_equal replies_count+1, message[:replies_count]
67 assert_equal reply, message.last_reply
68 68 # author should be watching the message
69 assert @message.watched_by?(reply_author)
69 assert message.watched_by?(reply_author)
70 70 end
71 71
72 72 def test_cannot_reply_to_locked_topic
73 73 topics_count = @board.topics_count
74 74 messages_count = @board.messages_count
75 @message = Message.find(1)
76 replies_count = @message.replies_count
77 assert_equal false, @message.locked
78 @message.locked = true
79 assert @message.save
80 assert_equal true, @message.locked
75 message = Message.find(1)
76 replies_count = message.replies_count
77 assert_equal false, message.locked
78 message.locked = true
79 assert message.save
80 assert_equal true, message.locked
81 81
82 82 reply_author = User.find(2)
83 83 reply = Message.new(:board => @board, :subject => 'Test reply',
84 84 :content => 'Test reply content',
85 :parent => @message, :author => reply_author)
85 :parent => message, :author => reply_author)
86 86 reply.save
87 87 assert_equal 1, reply.errors.count
88 88 end
89 89
90 90 def test_moving_message_should_update_counters
91 @message = Message.find(1)
91 message = Message.find(1)
92 92 assert_no_difference 'Message.count' do
93 93 # Previous board
94 94 assert_difference 'Board.find(1).topics_count', -1 do
95 assert_difference 'Board.find(1).messages_count', -(1 + @message.replies_count) do
95 assert_difference 'Board.find(1).messages_count', -(1 + message.replies_count) do
96 96 # New board
97 97 assert_difference 'Board.find(2).topics_count' do
98 assert_difference 'Board.find(2).messages_count', (1 + @message.replies_count) do
99 @message.update_attributes(:board_id => 2)
98 assert_difference 'Board.find(2).messages_count', (1 + message.replies_count) do
99 message.update_attributes(:board_id => 2)
100 100 end
101 101 end
102 102 end
General Comments 0
You need to be logged in to leave comments. Login now