forty_facets 0.1.9.1 → 0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 69545229909e0d3d59609f54cd23590099dc656eb9c0450e38ec28ea23001edd
-  data.tar.gz: c883ef1c6c6744a0066d006bed2ffe9810e32fadb0f0677f5b98d66564bd1e0e
+  metadata.gz: 145715b5814badb1b542b7eac19af6683c1c624e55aeeb4cd65a3f7e69d84d1a
+  data.tar.gz: bfc630bf7222be66578beca4923982e7172343fdf08f7b932819eb73f7072e7e
 SHA512:
-  metadata.gz: 669f899ed5ec16ada4907272603db2c9bbcd6a27b42555a80aaafc841c2eed0863452905ae72753b35652159a40a4fdd5c42931e780ed04a58f4dd9a0fdd975a
-  data.tar.gz: 3af3637c333eafe0723e8f695abcb0c1385c37bb089ac285375ea38b4975f7f1bec3e6381b1cfc53b53f280462d0aa4f52e23928b7ca47b199a22861b5ea1c2e
+  metadata.gz: 216b8f57082cd4d8b820484d089e47cde6e97ac21be0f9dd7761db01a150b30d110888d9826d617b6f9b8612880e446b5ffe86f0fddbdb7dddabe500d188d6c2
+  data.tar.gz: e635009e28fb7ec381295405d402f7026e9a3ca49eead6cd8541a93eea586734333eb9e60982e98fdd052869d4451fc32705d6045c328400cf3e3d7f8f07965c

data/.ruby-version

@@ -1 +1 @@
-2.5.1
+3.1.2

data/CHANGELOG

@@ -0,0 +1,8 @@
+# CHANGELOG
+
+## 0.2
+
+- introduction of CHANGELOG
+- upgraded to Ruby 3.1
+- upgraded to ActiveRecrd/Rails 7
+- merged #27

data/forty_facets.gemspec

@@ -18,11 +18,11 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ["lib"]
 
-  spec.add_development_dependency "bundler", "~> 1.5"
+  spec.add_development_dependency "bundler"
   spec.add_development_dependency "rake"
   spec.add_development_dependency "minitest"
   spec.add_development_dependency "sqlite3"
   spec.add_development_dependency "coveralls"
-  spec.add_development_dependency "activerecord", "~> 5.0"
+  spec.add_development_dependency "activerecord", "~> 7.0"
   spec.add_development_dependency "byebug" # travis doenst like byebug
 end

data/lib/forty_facets/filter/facet_filter_definition.rb

@@ -1,6 +1,7 @@
+# frozen_string_literal: true
+
 module FortyFacets
   class FacetFilterDefinition < FilterDefinition
-
     class FacetFilter < Filter
       def values
         @values ||= Array.wrap(value).sort.uniq
@@ -12,12 +13,12 @@ module FortyFacets
         order_accessor = definition.options[:order]
         if order_accessor
           if order_accessor.is_a?(Proc)
-            facet.sort_by!{|facet_value| order_accessor.call(facet_value.entity) }
+            facet.sort_by! { |facet_value| order_accessor.call(facet_value.entity) }
           else
-            facet.sort_by!{|facet_value| facet_value.entity.send(order_accessor) }
+            facet.sort_by! { |facet_value| facet_value.entity.send(order_accessor) }
           end
         else
-          facet.sort_by!{|facet_value| -facet_value.count }
+          facet.sort_by! { |facet_value| -facet_value.count }
         end
         facet
       end
@@ -49,12 +50,14 @@ module FortyFacets
       def selected
         entity = definition.origin_class
         column = entity.columns_hash[definition.attribute.to_s]
-        values.map{|v| entity.connection.type_cast(v, column)}
+        type = entity.connection.lookup_cast_type_from_column(column)
+        values.map { |value| type.serialize(value) }
       end
 
       def build_scope
-        return Proc.new { |base| base } if empty?
-        Proc.new do |base|
+        return proc { |base| base } if empty?
+
+        proc do |base|
           base.joins(definition.joins).where(definition.qualified_column_name => value)
         end
       end
@@ -90,8 +93,9 @@ module FortyFacets
 
     class BelongsToFilter < AssociationFacetFilter
       def build_scope
-        return Proc.new { |base| base } if empty?
-        Proc.new do |base|
+        return proc { |base| base } if empty?
+
+        proc do |base|
           base.joins(definition.joins).where(definition.qualified_column_name => values)
         end
       end
@@ -115,8 +119,9 @@ module FortyFacets
 
     class HasManyFilter < AssociationFacetFilter
       def build_scope
-        return Proc.new { |base| base } if empty?
-        Proc.new do |base|
+        return proc { |base| base } if empty?
+
+        proc do |base|
           base_table = definition.origin_class.table_name
 
           primary_key_column = "#{base_table}.#{definition.origin_class.primary_key}"
@@ -130,13 +135,13 @@ module FortyFacets
       def facet
         base_table = definition.search.root_class.table_name
         join_name =  [definition.association.name.to_s, base_table.to_s].sort.join('_')
-        foreign_id_col = definition.association.name.to_s.singularize + '_id'
-        my_column = join_name + '.' + foreign_id_col
+        foreign_id_col = "#{definition.association.name.to_s.singularize}_id"
+        my_column = "#{join_name}.#{foreign_id_col}"
         counts = without.result(skip_ordering: true)
-                  .distinct
-                  .joins(definition.joins)
-                  .select("#{my_column} as foreign_id, count(#{my_column}) as occurrences")
-                  .group(my_column)
+                        .distinct
+                        .joins(definition.joins)
+                        .select("#{my_column} as foreign_id, count(#{my_column}) as occurrences")
+                        .group(my_column)
         counts.includes_values = []
         entities_by_id = definition.association.klass.unscoped.find(counts.map(&:foreign_id)).group_by(&:id)
 
@@ -152,11 +157,12 @@ module FortyFacets
 
     def build_filter(search_instance, param_value)
       if association
-        if association.macro == :belongs_to
+        case association.macro
+        when :belongs_to
           BelongsToFilter.new(self, search_instance, param_value)
-        elsif association.macro == :has_many
+        when :has_many
           HasManyFilter.new(self, search_instance, param_value)
-        elsif association.macro == :has_and_belongs_to_many
+        when :has_and_belongs_to_many
           HasManyFilter.new(self, search_instance, param_value)
         else
           raise "Unsupported association type: #{association.macro}"

data/lib/forty_facets/filter/range_filter_definition.rb

@@ -1,23 +1,27 @@
 module FortyFacets
   class RangeFilterDefinition < FilterDefinition
     class RangeFilter < Filter
+      RANGE_REGEX = /(\d*) - (\d*)/.freeze
+
       def build_scope
         return Proc.new { |base| base } if empty?
 
         Proc.new do |base|
-          base.joins(definition.joins)
-            .where("#{definition.qualified_column_name} >= ? AND #{definition.qualified_column_name} <= ? ", min_value, max_value ) 
+          scope = base.joins(definition.joins)
+          scope = scope.where("#{definition.qualified_column_name} >= ?", min_value) if min_value.present?
+          scope = scope.where("#{definition.qualified_column_name} <= ?", max_value) if max_value.present?
+          scope
         end
       end
 
       def min_value
-        return nil if empty?
-        value.split(' - ').first
+        min, _max = range_values
+        min
       end
 
       def max_value
-        return nil if empty?
-        value.split(' - ').last
+        _min, max = range_values
+        max
       end
 
       def absolute_interval
@@ -32,6 +36,11 @@ module FortyFacets
         absolute_interval.max
       end
 
+      private
+
+      def range_values
+        value&.match(RANGE_REGEX)&.captures
+      end
     end
 
     def build_filter(search_instance, value)

data/lib/forty_facets/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module FortyFacets
-  VERSION = "0.1.9.1"
+  VERSION = '0.2'
 end

data/test/smoke_test.rb

@@ -1,47 +1,49 @@
+# frozen_string_literal: true
+
 require 'coveralls'
 Coveralls.wear!
 
-require "minitest/autorun"
+require 'minitest/autorun'
 require 'test_helper'
 require 'logger'
 # require 'byebug'# travis doenst like byebug
 require_relative '../lib/forty_facets'
 
-#silence_warnings do
+# silence_warnings do
 require_relative 'fixtures'
-#end
+# end
 
 class MovieSearch < FortyFacets::FacetSearch
   model 'Movie'
 
   text :title, name: 'Title'
   facet :studio, name: 'Studio'
-  facet :year, order: Proc.new {|year| -year}
+  facet :year, order: proc { |year| -year }
   facet :genres, name: 'Genre'
   facet :actors, name: 'Actor'
   range :price, name: 'Price'
   facet :writers, name: 'Writer'
-  facet [:studio, :country], name: 'Country'
-  facet [:studio, :status], name: 'Studio status'
-  facet [:studio, :producers], name: 'Producers'
+  facet %i[studio country], name: 'Country'
+  facet %i[studio status], name: 'Studio status'
+  facet %i[studio producers], name: 'Producers'
   sql_facet({ uschis: "studios.name = 'Uschi'", non_uschis: "studios.name != 'USCHI'" },
-            { name: "Uschis", path: [:studio, :uschis], joins: [:studio] })
-  sql_facet({ classic: "year <= 1980", non_classic: "year > 1980" },
-            { name: "Classic", path: :classic })
-  sql_facet({ classic: "year <= 1980", non_classic: "year > 1980" },
-            { name: "Classic" })
-  text [:studio, :description], name: 'Studio Description'
+            { name: 'Uschis', path: %i[studio uschis], joins: [:studio] })
+  sql_facet({ classic: 'year <= 1980', non_classic: 'year > 1980' },
+            { name: 'Classic', path: :classic })
+  sql_facet({ classic: 'year <= 1980', non_classic: 'year > 1980' },
+            { name: 'Classic' })
+  text %i[studio description], name: 'Studio Description'
   scope :classics, name: 'Name classics'
   scope :year_lte, name: 'Year less than or equal', pass_value: true
   custom :needs_complex_filtering
 end
 
 class SmokeTest < Minitest::Test
-
   def test_sql_facet_with_belongs_to
-    search = MovieSearch.new({'studio-uschis' => {}})
+    search = MovieSearch.new({ 'studio-uschis' => {} })
     assert_equal Movie.count, search.result.size
-    assert_equal search.filter([:studio, :uschis]).facet, [FortyFacets::FacetValue.new(:uschis, 0, false), FortyFacets::FacetValue.new(:non_uschis, 40, false)]
+    assert_equal search.filter(%i[studio uschis]).facet,
+                 [FortyFacets::FacetValue.new(:uschis, 0, false), FortyFacets::FacetValue.new(:non_uschis, 40, false)]
   end
 
   def test_it_finds_all_movies
@@ -50,57 +52,57 @@ class SmokeTest < Minitest::Test
   end
 
   def test_scope_filter
-    search = MovieSearch.new("search" => {})
+    search = MovieSearch.new('search' => {})
     assert_equal 40, search.result.size
     assert search.filter(:classic).facet.include? FortyFacets::FacetValue.new(:classic, 6, false)
     assert search.filter(:classic).facet.include? FortyFacets::FacetValue.new(:non_classic, 34, false)
 
-    search = MovieSearch.new("search" => { "classic" => "classic" })
+    search = MovieSearch.new('search' => { 'classic' => 'classic' })
     assert_equal 6, search.result.size
     assert search.filter(:classic).facet.include? FortyFacets::FacetValue.new(:classic, 6, true)
     assert search.filter(:classic).facet.include? FortyFacets::FacetValue.new(:non_classic, 34, false)
 
-    search = MovieSearch.new("search" => { "classic" => "non_classic" })
+    search = MovieSearch.new('search' => { 'classic' => 'non_classic' })
     assert_equal 34, search.result.size
     assert search.filter(:classic).facet.include? FortyFacets::FacetValue.new(:classic, 6, false)
     assert search.filter(:classic).facet.include? FortyFacets::FacetValue.new(:non_classic, 34, true)
 
-    search = MovieSearch.new("search" => { "classic" => ["non_classic", "classic"] })
+    search = MovieSearch.new('search' => { 'classic' => %w[non_classic classic] })
     assert_equal 40, search.result.size
     assert search.filter(:classic).facet.include? FortyFacets::FacetValue.new(:classic, 6, true)
     assert search.filter(:classic).facet.include? FortyFacets::FacetValue.new(:non_classic, 34, true)
   end
 
   def test_scope_filter_without_path
-    search = MovieSearch.new("search" => {})
+    search = MovieSearch.new('search' => {})
     assert_equal 40, search.result.size
-    assert search.filter([:classic, :non_classic]).facet.include? FortyFacets::FacetValue.new(:classic, 6, false)
-    assert search.filter([:classic, :non_classic]).facet.include? FortyFacets::FacetValue.new(:non_classic, 34, false)
+    assert search.filter(%i[classic non_classic]).facet.include? FortyFacets::FacetValue.new(:classic, 6, false)
+    assert search.filter(%i[classic non_classic]).facet.include? FortyFacets::FacetValue.new(:non_classic, 34, false)
 
-    search = MovieSearch.new("search" => { "classic-non_classic" => "classic" })
+    search = MovieSearch.new('search' => { 'classic-non_classic' => 'classic' })
     assert_equal 6, search.result.size
-    assert search.filter([:classic, :non_classic]).facet.include? FortyFacets::FacetValue.new(:classic, 6, true)
-    assert search.filter([:classic, :non_classic]).facet.include? FortyFacets::FacetValue.new(:non_classic, 34, false)
+    assert search.filter(%i[classic non_classic]).facet.include? FortyFacets::FacetValue.new(:classic, 6, true)
+    assert search.filter(%i[classic non_classic]).facet.include? FortyFacets::FacetValue.new(:non_classic, 34, false)
 
-    search = MovieSearch.new("search" => { "classic-non_classic" => "non_classic" })
+    search = MovieSearch.new('search' => { 'classic-non_classic' => 'non_classic' })
     assert_equal 34, search.result.size
-    assert search.filter([:classic, :non_classic]).facet.include? FortyFacets::FacetValue.new(:classic, 6, false)
-    assert search.filter([:classic, :non_classic]).facet.include? FortyFacets::FacetValue.new(:non_classic, 34, true)
+    assert search.filter(%i[classic non_classic]).facet.include? FortyFacets::FacetValue.new(:classic, 6, false)
+    assert search.filter(%i[classic non_classic]).facet.include? FortyFacets::FacetValue.new(:non_classic, 34, true)
 
-    search = MovieSearch.new("search" => { "classic-non_classic" => ["non_classic", "classic"] })
+    search = MovieSearch.new('search' => { 'classic-non_classic' => %w[non_classic classic] })
     assert_equal 40, search.result.size
-    assert search.filter([:classic, :non_classic]).facet.include? FortyFacets::FacetValue.new(:classic, 6, true)
-    assert search.filter([:classic, :non_classic]).facet.include? FortyFacets::FacetValue.new(:non_classic, 34, true)
+    assert search.filter(%i[classic non_classic]).facet.include? FortyFacets::FacetValue.new(:classic, 6, true)
+    assert search.filter(%i[classic non_classic]).facet.include? FortyFacets::FacetValue.new(:non_classic, 34, true)
   end
 
   def test_text_filter
-    search = MovieSearch.new({'search' => { 'title' => 'ipsum' }})
+    search = MovieSearch.new({ 'search' => { 'title' => 'ipsum' } })
     assert_equal 1, search.result.size
     assert_equal 'ipsum', search.result.first.title
   end
 
   def test_year_filter
-    search = MovieSearch.new({'search' => { 'year' => '2011' }})
+    search = MovieSearch.new({ 'search' => { 'year' => '2011' } })
     assert_equal [2011], search.result.map(&:year).uniq
 
     facet = search.filter(:year).facet
@@ -108,34 +110,43 @@ class SmokeTest < Minitest::Test
   end
 
   def test_range_filter
-    search = MovieSearch.new({'search' => {'price' => '0 - 20'}})
+    search = MovieSearch.new({ 'search' => { 'price' => '0 - 20' } })
     assert_equal Movie.count, search.result.size
 
-    search = MovieSearch.new({'search' => {'price' => '0 - 10'}})
-    assert_equal Movie.all.reject{|m| m.price > 10}.size, search.result.size
+    search = MovieSearch.new({ 'search' => { 'price' => '5 - 10' } })
+    assert_equal Movie.where('price >= ? AND price <= ?', 5, 10).count, search.result.size
+
+    search = MovieSearch.new({ 'search' => { 'price' => '5 - ' } })
+    assert_equal Movie.where('price >= ?', 5).count, search.result.size
+
+    search = MovieSearch.new({ 'search' => { 'price' => ' - 5' } })
+    assert_equal Movie.where('price <= ?', 5).count, search.result.size
+
+    search = MovieSearch.new({ 'search' => { 'price' => '' } })
+    assert_equal Movie.count, search.result.size
   end
 
   def test_text_filter_via_belongs_to
     description = Studio.first.description
-    search = MovieSearch.new({'search' => { 'studio-description' => description }})
+    search = MovieSearch.new({ 'search' => { 'studio-description' => description } })
 
-    assert_equal Movie.all.reject{|m| m.studio.description != description}.size, search.result.size
+    assert_equal Movie.all.select { |m| m.studio.description == description }.size, search.result.size
     assert_equal description, search.result.first.studio.description
   end
 
   def test_country_filter
-    search = MovieSearch.new('search' => { 'studio-country' => Country.first.id.to_s})
-    assert_equal [Country.first], search.result.map{|m| m.studio.country}.uniq
+    search = MovieSearch.new('search' => { 'studio-country' => Country.first.id.to_s })
+    assert_equal [Country.first], search.result.map { |m| m.studio.country }.uniq
     assert_equal Movie.count / 2, search.result.count
 
-    search = MovieSearch.new('search' => { 'studio-country' => Country.last.id.to_s})
-    assert_equal [Country.last], search.result.map{|m| m.studio.country}.uniq
+    search = MovieSearch.new('search' => { 'studio-country' => Country.last.id.to_s })
+    assert_equal [Country.last], search.result.map { |m| m.studio.country }.uniq
     assert_equal Movie.count / 2, search.result.count
   end
 
   def test_selected_country_filter
-    search = MovieSearch.new('search' => { 'studio-country' => Country.first.id.to_s})
-    filter = search.filter([:studio, :country])
+    search = MovieSearch.new('search' => { 'studio-country' => Country.first.id.to_s })
+    filter = search.filter(%i[studio country])
     assert_equal FortyFacets::FacetFilterDefinition::BelongsToFilter, filter.class
     assert_equal [Country.first], filter.selected
 
@@ -143,17 +154,16 @@ class SmokeTest < Minitest::Test
   end
 
   def test_studio_status_filter
-    search = MovieSearch.new('search' => { 'studio-status' => 'active'})
-    assert_equal ['active'], search.result.map{|m| m.studio.status}.uniq
+    search = MovieSearch.new('search' => { 'studio-status' => 'active' })
+    assert_equal ['active'], search.result.map { |m| m.studio.status }.uniq
     assert_equal Movie.count / 2, search.result.count
 
-    filter = search.filter([:studio, :status])
+    filter = search.filter(%i[studio status])
     assert_equal ['active'], filter.selected
   end
 
   def test_year_add_remove_filter
-
-    search = MovieSearch.new()
+    search = MovieSearch.new
 
     search = search.filter(:year).add(2010)
     assert_equal Movie.where(year: 2010).count, search.result.count
@@ -166,7 +176,7 @@ class SmokeTest < Minitest::Test
   end
 
   def test_selected_year_filter
-    search = MovieSearch.new()
+    search = MovieSearch.new
 
     search = search.filter(:year).add(2010)
     assert_equal [2010], search.filter(:year).selected
@@ -175,9 +185,9 @@ class SmokeTest < Minitest::Test
     assert_equal [2010, 2011], search.filter(:year).selected
 
     facet = search.filter(:year).facet
-    assert facet.find{|fv| fv.entity == 2010}.selected
-    assert facet.find{|fv| fv.entity == 2011}.selected
-    assert !facet.find{|fv| fv.entity == 2012}.selected
+    assert facet.find { |fv| fv.entity == 2010 }.selected
+    assert facet.find { |fv| fv.entity == 2011 }.selected
+    assert !facet.find { |fv| fv.entity == 2012 }.selected
   end
 
   def test_belongs_to_filter
@@ -186,7 +196,7 @@ class SmokeTest < Minitest::Test
     studio = first_facet_value.entity
     assert_kind_of Studio, studio
 
-    movies_with_studio = Movie.where(studio: studio)
+    movies_with_studio = Movie.where(studio:)
     search_with_studio = blank_search.filter(:studio).add(studio)
 
     assert_equal movies_with_studio.size, search_with_studio.result.size
@@ -196,7 +206,7 @@ class SmokeTest < Minitest::Test
   def test_belongs_to_filter_with_default_scope
     wrap_in_db_transaction do
       deleted_studio = Studio.create!(name: 'Deleted studio', status: 'active')
-      movie = Movie.create!(studio: deleted_studio)
+      Movie.create!(studio: deleted_studio)
       deleted_studio.update!(deleted_at: Time.now)
 
       blank_search = MovieSearch.new
@@ -216,7 +226,7 @@ class SmokeTest < Minitest::Test
   def test_has_many
     blank_search = MovieSearch.new
     genre = Genre.first
-    expected = Movie.order(:id).select{|m| m.genres.include?(genre)}
+    expected = Movie.order(:id).select { |m| m.genres.include?(genre) }
     assert blank_search.filter(:genres).is_a?(FortyFacets::FacetFilterDefinition::HasManyFilter)
     search = blank_search.filter(:genres).add(genre)
     actual = search.result
@@ -227,9 +237,9 @@ class SmokeTest < Minitest::Test
   def test_hast_many_via_belongs_to
     blank_search = MovieSearch.new
     producer = Producer.first
-    expected = Movie.order(:id).select{|m| m.studio.producers.include? producer}
-    assert blank_search.filter([:studio, :producers]).is_a?(FortyFacets::FacetFilterDefinition::HasManyFilter)
-    search = blank_search.filter([:studio, :producers]).add(producer)
+    expected = Movie.order(:id).select { |m| m.studio.producers.include? producer }
+    assert blank_search.filter(%i[studio producers]).is_a?(FortyFacets::FacetFilterDefinition::HasManyFilter)
+    search = blank_search.filter(%i[studio producers]).add(producer)
     actual = search.result
 
     assert_equal expected.size, actual.size
@@ -238,7 +248,7 @@ class SmokeTest < Minitest::Test
   def test_has_many_writers
     blank_search = MovieSearch.new
     writer = Writer.first
-    expected = Movie.order(:id).select{|m| m.writers.include?(writer)}
+    expected = Movie.order(:id).select { |m| m.writers.include?(writer) }
     assert blank_search.filter(:writers).is_a?(FortyFacets::FacetFilterDefinition::HasManyFilter)
     search = blank_search.filter(:writers).add(writer)
     actual = search.result
@@ -251,8 +261,8 @@ class SmokeTest < Minitest::Test
     genre = Genre.first
     actor = Actor.first
     expected = Movie.order(:id)
-                .select{|m| m.genres.include?(genre)}
-                .select{|m| m.actors.include?(actor)}
+                    .select { |m| m.genres.include?(genre) }
+                    .select { |m| m.actors.include?(actor) }
     assert blank_search.filter(:genres).is_a?(FortyFacets::FacetFilterDefinition::HasManyFilter)
     search_with_genre = blank_search.filter(:genres).add(genre)
     search_with_genre_and_actor = search_with_genre.filter(:actors).add(actor)
@@ -267,11 +277,11 @@ class SmokeTest < Minitest::Test
 
     search.filter(:writers).facet.each do |facet_value|
       writer = facet_value.entity
-      expected = Movie.order(:id).select{|m| m.writers.include?(writer)}.count
-      assert_equal expected, facet_value.count, "The amount of movies for a writer should match the number indicated in the facet"
+      expected = Movie.order(:id).select { |m| m.writers.include?(writer) }.count
+      assert_equal expected, facet_value.count,
+                   'The amount of movies for a writer should match the number indicated in the facet'
       assert_equal writer.id == selected_writer.id, facet_value.selected
     end
-
   end
 
   def test_has_many_facet_values_genres
@@ -280,11 +290,11 @@ class SmokeTest < Minitest::Test
 
     search.filter(:genres).facet.each do |facet_value|
       genre = facet_value.entity
-      expected = Movie.order(:id).select{|m| m.genres.include?(genre)}.count
-      assert_equal expected, facet_value.count, "The amount of movies for a genre should match the number indicated in the facet"
+      expected = Movie.order(:id).select { |m| m.genres.include?(genre) }.count
+      assert_equal expected, facet_value.count,
+                   'The amount of movies for a genre should match the number indicated in the facet'
       assert_equal genre.id == selected_genre.id, facet_value.selected
     end
-
   end
 
   def test_includes_does_not_blow_up
@@ -293,14 +303,15 @@ class SmokeTest < Minitest::Test
     search.filter(:studio).facet.reject(&:selected).to_a
   end
 
-  def test_scope_filter
-    search_with_scope = MovieSearch.new().filter(:classics).add('1')
+  def test_activating_scope_filter
+    search_with_scope = MovieSearch.new.filter(:classics).add('1')
     assert search_with_scope.result.count < Movie.count, 'Activating the scope should yield a smaller set of movies'
   end
 
   def test_scope_filter_with_params
-    search_with_scope = MovieSearch.new().filter(:year_lte).add(1980)
-    assert search_with_scope.result.count < Movie.count, 'Activating the scope with filter should yield a smaller set of movies'
+    search_with_scope = MovieSearch.new.filter(:year_lte).add(1980)
+    assert search_with_scope.result.count < Movie.count,
+           'Activating the scope with filter should yield a smaller set of movies'
   end
 
   def test_custom_filter
@@ -309,5 +320,4 @@ class SmokeTest < Minitest::Test
 
     assert_equal 'foo', new_search.filter(:needs_complex_filtering).value
   end
-
 end

metadata

@@ -1,29 +1,29 @@
 --- !ruby/object:Gem::Specification
 name: forty_facets
 version: !ruby/object:Gem::Version
-  version: 0.1.9.1
+  version: '0.2'
 platform: ruby
 authors:
 - Axel Tetzlaff
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-08-21 00:00:00.000000000 Z
+date: 2022-06-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.5'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.5'
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -86,14 +86,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '7.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '7.0'
 - !ruby/object:Gem::Dependency
   name: byebug
   requirement: !ruby/object:Gem::Requirement
@@ -121,6 +121,7 @@ files:
 - ".ruby-gemset"
 - ".ruby-version"
 - ".travis.yml"
+- CHANGELOG
 - Gemfile
 - LICENSE.txt
 - README.md
@@ -146,7 +147,7 @@ homepage: https://github.com/fortytools/forty_facets
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -161,8 +162,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.6
-signing_key: 
+rubygems_version: 3.3.7
+signing_key:
 specification_version: 4
 summary: Library for building facet searches for active_record models
 test_files: