dan
/
jekyll
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix RuboCop Performance offenses in test files (#7839) 

* Replace select.first with find
* Replace select.count with count
This commit is contained in:
Takashi Udagawa 2019-10-06 15:30:25 +09:00 committed by Ashwin Maroli
parent de6548f1ba
commit 3eab32fad7
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
test/test_document.rb
View File

@ -175,7 +175,7 @@ class TestDocument < JekyllUnitTest
}] }]
) )
@site.process @site.process
@document = @site.collections["slides"].docs.select { |d| d.is_a?(Document) }.first @document = @site.collections["slides"].docs.find { |d| d.is_a?(Document) }
end end
should "know the front matter defaults" do should "know the front matter defaults" do

2
test/test_filters.rb
View File

@ -1036,7 +1036,7 @@ class TestFilters < JekyllUnitTest
posts = @filter.site.site_payload["site"]["posts"] posts = @filter.site.site_payload["site"]["posts"]
results = @filter.where_exp(posts, "post", results = @filter.where_exp(posts, "post",
"post.date > site.dont_show_posts_before") "post.date > site.dont_show_posts_before")
assert_equal posts.select { |p| p.date > @sample_time }.count, results.length assert_equal posts.count { |p| p.date > @sample_time }, results.length
end end
end end