count should be a string / trailing comma

Alice Bartlett · Alice Bartlett · commit 6c3d93f52a51 · 2015-04-09T12:24:53.000+01:00
`count` should be a string not an integer, though there should only
ever be 1 result when searching by link anyway.

Add a trailing comma for cleaner diffs(tm).
diff --git a/lib/search_api.rb b/lib/search_api.rb
@@ -33,7 +33,7 @@ def scope_info
{
 scope:{
 title: scope_object.title,
- }
+ },
 }
 else
{}
@@ -45,7 +45,7 @@ def rummager_params
 end
 
 def scope_object
- @scope_object ||= api.unified_search(filter_link: scope_object_link, count: 1, fields: %w{title}).results.first
+ @scope_object ||= api.unified_search(filter_link: scope_object_link, count: "1", fields: %w{title}).results.first
 end
 
 def is_scoped?
diff --git a/test/unit/search_api_test.rb b/test/unit/search_api_test.rb
@@ -33,7 +33,7 @@ class SearchAPITest < ActiveSupport::TestCase
 @search_params.expects(:filter).with('manual').returns([@manual_link])
 @manual_search_response = stub(results: [stub(title: @manual_title)])
 
- @rummager_api.expects(:unified_search).with(filter_link: @manual_link, count: 1, fields: %w{title}).returns(@manual_search_response)
+ @rummager_api.expects(:unified_search).with(filter_link: @manual_link, count: "1", fields: %w{title}).returns(@manual_search_response)
 end
 
 should "returns search results from rummager" do

-Original file line number
+Diff line change
+{
 scope: {
 title: scope_object.title,
 -}
 +},
+}
 else
 {}
 end
 defscope_object
 -@scope_object ||= api.unified_search(filter_link: scope_object_link,count: 1,fields: %w{title}).results.first
 +@scope_object ||= api.unified_search(filter_link: scope_object_link,count: "1",fields: %w{title}).results.first
 end
 defis_scoped?
-Original file line number
+Diff line change
 @search_params.expects(:filter).with('manual').returns([@manual_link])
 @manual_search_response=stub(results: [stub(title: @manual_title)])
 -@rummager_api.expects(:unified_search).with(filter_link: @manual_link,count: 1,fields: %w{title}).returns(@manual_search_response)
 +@rummager_api.expects(:unified_search).with(filter_link: @manual_link,count: "1",fields: %w{title}).returns(@manual_search_response)
 end
 should"returns search results from rummager"do