From 5c93d07d2aeb60a7b31b235fe11bbcbc746149b8 Mon Sep 17 00:00:00 2001 From: Guo Xiang Tan Date: Fri, 2 Mar 2018 09:25:46 +0800 Subject: [PATCH] Remove duplication of params in tests. --- spec/requests/posts_controller_spec.rb | 1 - spec/requests/topics_controller_spec.rb | 3 +-- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/spec/requests/posts_controller_spec.rb b/spec/requests/posts_controller_spec.rb index c7f9d210a0..b0c4a61deb 100644 --- a/spec/requests/posts_controller_spec.rb +++ b/spec/requests/posts_controller_spec.rb @@ -22,7 +22,6 @@ RSpec.describe PostsController do category.set_permissions(staff: :full) category.save! - # strange API, why is topic id in here twice put "/posts/#{post.id}.json", params: { post: { category_id: category.id, raw: "this is a test edit to post" } } diff --git a/spec/requests/topics_controller_spec.rb b/spec/requests/topics_controller_spec.rb index 27bbd9a1d2..b4e00c54fe 100644 --- a/spec/requests/topics_controller_spec.rb +++ b/spec/requests/topics_controller_spec.rb @@ -14,8 +14,7 @@ RSpec.describe TopicsController do category.set_permissions(staff: :full) category.save! - # strange API, why is topic id in here twice - put "/t/#{post.topic_id}.json", params: { category_id: category.id, topic_id: post.topic_id } + put "/t/#{post.topic_id}.json", params: { category_id: category.id } expect(response.status).not_to eq(200) expect(post.topic.category_id).not_to eq(category.id)