From a2e6a8a51e1479a6d05acbd7a0c05289d63f2ecf Mon Sep 17 00:00:00 2001 From: Guo Xiang Tan Date: Tue, 2 Apr 2019 07:43:13 +0800 Subject: [PATCH] DEV: Refactor test to assert for actual content. The other assertion is already asserting for `Post#url` and I think it makes the test clearer here. --- spec/serializers/reviewable_flagged_post_serializer_spec.rb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/spec/serializers/reviewable_flagged_post_serializer_spec.rb b/spec/serializers/reviewable_flagged_post_serializer_spec.rb index 1165fe18039..2c43bccb3ec 100644 --- a/spec/serializers/reviewable_flagged_post_serializer_spec.rb +++ b/spec/serializers/reviewable_flagged_post_serializer_spec.rb @@ -8,8 +8,8 @@ describe ReviewableFlaggedPostSerializer do p0 = Fabricate(:post) reviewable = PostActionCreator.spam(Fabricate(:user), p0).reviewable json = ReviewableFlaggedPostSerializer.new(reviewable, scope: Guardian.new(admin), root: nil).as_json - expect(json[:cooked]).to be_present - expect(json[:raw]).to be_present + expect(json[:cooked]).to eq(p0.cooked) + expect(json[:raw]).to eq(p0.raw) expect(json[:topic_url]).to eq(p0.url) end