rating 0.11.0 → 0.12.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3478dbe5fac4bb47db0a3b730f0b21d49747912056447d9de55d5ec82f6072ec
-  data.tar.gz: 3d95a77107c053347876554c1f36121a62b9342dd4b585833cd12b2f32aadd0c
+  metadata.gz: 107d28dc3f0be409a1684eebea163b6dcf6cc57d801f7dcd88bca4f0bcf4f662
+  data.tar.gz: af591a933ee3bf23720cd054c33968ec9fbba3f21cbd9d117dd1628f0b6cebb5
 SHA512:
-  metadata.gz: caabeb2c0336c22a86d95a064688424d242c96f84c8e28bd93883a2472f15a22d937ab726f060f765332bb8277f19ffb2fde571a2e7019b91e02c8aced96867d
-  data.tar.gz: 5acbf2bb5e2280de7507e9c23785ecc75e31b62a8c434a48972f50e78d09fa38a8e4280d4272c91bb3109994cdab217f5766a01d94c5a384d9990aa080c18e06
+  metadata.gz: 3c3dc424ab1155b6a7f6aa309b461015c7d06b2786c172d55b03b29052a3643d098fe78dde4a9c7edf2b564ea324451e5ad1a12373769caa7b6520cb0b5def8d
+  data.tar.gz: b92eaadfdaa4cca67eca6e3a5550333080541d93b1a48e01dace83bc858969fc718f60123c40353c15bf3ec6c42166b2359d9d82e05b315454a4ca74b91b32be

data/CHANGELOG.md

@@ -1,3 +1,17 @@
+## v0.12.0
+
+### News
+
+- Officially Supports Postgres;
+
+### Updates
+
+- Migrates the CI to GitHub Actions;
+- This gem does not depends on `git` package anymore;
+- Makes this gem requires MFA for security;
+- Adds coverage test;
+- The method `rated?` now uses a better query;
+
 ## v0.11.0
 
 ### Updates
@@ -14,47 +28,47 @@
 
 ### News
 
-- Order index keys of template the same way Rails does the query;
+- Order index keys of the template the same way Rails does the query;
 - Limits the `_type` columns to avoid overflow bytes in DBs like MySQL < 5.7.
 
 ### Updates
 
 - Update Ruby to 2.6.5;
-- Update Activerecord to the last version;
+- Update ActiveRecord to the last version;
 
 ## v0.8.0
 
 ### News
 
-- Adds `unscoped_rating` option to calculate the rating counting all resource record ignoring the scope.
+- Adds `unscoped_rating` option to calculate the rating by counting all resource records ignoring the scope.
 
 ## v0.7.0
 
 ### News
 
 - Support to configure `uniqueness` validation via YAML into Rating::Rate model;
-- Support to multiple scopes via `extra_scopes` option.
+- Support multiple scopes via `extra_scopes` option.
 
 ### Updates
 
-- Reverts v0.6.0, since we need this validation because we cannot edit the Rate model by ourself.
+- Reverts v0.6.0, since we need this validation because we cannot edit the Rate model by ourselves.
 
 ## v0.6.0
 
 ### Updates
 
-- The author unique validations was removed to enable custom validations
+- The author's unique validations were removed to enable custom validations
 
 ## v0.5.0
 
 ### News
 
-- Adds `rating.yml` config to support to change the tables where Rating will write the data;
-- Adds `scoping` option to support to generates zero based rating via scope.
+- Adds `rating.yml` config to support changing the tables where Rating will write the data;
+- Adds `scoping` option to support generating zero-based rating via scope.
 
 ### Updates
 
-- The migrate was separated in two to improve in troubleshoot. (iondrimba)
+- The migration was separated in two to improve troubleshooting. [#1](https://github.com/wbotelhos/rating/pull/1) by [iondrimba](https://github.com/iondrimba)
 
 ## v0.4.0
 

data/README.md

@@ -1,9 +1,10 @@
 # Rating
 
-[![Build Status](https://travis-ci.org/wbotelhos/rating.svg)](https://travis-ci.org/wbotelhos/rating)
+[![CI](https://github.com/wbotelhos/rating/workflows/CI/badge.svg)](https://github.com/wbotelhos/rating/actions)
 [![Gem Version](https://badge.fury.io/rb/rating.svg)](https://badge.fury.io/rb/rating)
 [![Maintainability](https://api.codeclimate.com/v1/badges/cc5efe8b06bc1d5e9e8a/maintainability)](https://codeclimate.com/github/wbotelhos/rating/maintainability)
-[![Patreon](https://img.shields.io/badge/donate-%3C3-brightgreen.svg)](https://www.patreon.com/wbotelhos)
+[![codecov](https://codecov.io/gh/wbotelhos/rating/branch/master/graph/badge.svg?token=QJSHUOULEG)](https://codecov.io/gh/wbotelhos/rating)
+[![Sponsor](https://img.shields.io/badge/sponsor-%3C3-green)](https://www.patreon.com/wbotelhos)
 
 A true Bayesian rating system with scope and cache enabled.
 
@@ -51,15 +52,15 @@ Add the following code on your `Gemfile` and run `bundle install`:
 gem 'rating'
 ```
 
-Run the following task to create a Rating migration:
+Run the following task to create the Rating migration:
 
-```bash
+```sh
 rails g rating:install
 ```
 
 Then execute the migrations to create the to create tables `rating_rates` and `rating_ratings`:
 
-```bash
+```sh
 rake db:migrate
 ```
 
@@ -423,6 +424,16 @@ author.rate resource, 5, extra_scope: { approved: true }
 
 As you can see, now, only the rate with value `5` will be included on the final rating.
 
+### Specs
+
+To run all specs, run the following rakes:
+
+```sh
+bundle exec rake spec
+bundle exec rake spec_config
+bundle exec rake spec_config_with_extra_scopes
+```
+
 ### References
 
 - [Evan Miller](http://www.evanmiller.org/ranking-items-with-star-ratings.html)

data/lib/rating/models/rating/extension.rb

@@ -20,9 +20,8 @@ module Rating
         Rate.rate_for author: self, extra_scopes: extra_scopes, resource: resource, scopeable: scope
       end
 
-      # TODO: use exists for performance
       def rated?(resource, extra_scopes: {}, scope: nil)
-        !rate_for(resource, extra_scopes: extra_scopes, scope: scope).nil?
+        Rate.exists?(extra_scopes.merge(author: self, resource: resource, scopeable: scope))
       end
 
       def rates(extra_scopes: {}, scope: nil)

data/lib/rating/models/rating/rate.rb

@@ -19,7 +19,7 @@ module Rating
       scope:          ::Rating::Config.validations['rate']['scope'].map(&:to_sym),
     }
 
-    def self.create(author:, extra_scopes:, metadata:, resource:, scopeable: nil, value:)
+    def self.create(author:, extra_scopes:, metadata:, resource:, value:, scopeable: nil)
       attributes = { author: author, resource: resource, scopeable: scopeable }.merge(extra_scopes)
       record     = find_or_initialize_by(attributes)
 
@@ -31,7 +31,7 @@ module Rating
       record
     end
 
-    def self.rate_for(author:, extra_scopes: {}, resource:, scopeable: nil)
+    def self.rate_for(author:, resource:, extra_scopes: {}, scopeable: nil)
       find_by extra_scopes.merge(author: author, resource: resource, scopeable: scopeable)
     end
 

data/lib/rating/models/rating/rating.rb

@@ -93,8 +93,8 @@ module Rating
         resource_rating_avg      = values.rating_avg
         resource_rating_count    = values.rating_count.to_f
 
-        (resource_rating_count / (resource_rating_count + count_avg)) * resource_rating_avg +
-          (count_avg           / (resource_rating_count + count_avg)) * resource_type_rating_avg
+        ((resource_rating_count / (resource_rating_count + count_avg)) * resource_rating_avg) +
+          ((count_avg           / (resource_rating_count + count_avg)) * resource_type_rating_avg)
       end
 
       def execute_sql(sql)

data/lib/rating/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Rating
-  VERSION = '0.11.0'
+  VERSION = '0.12.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rating
 version: !ruby/object:Gem::Version
-  version: 0.11.0
+  version: 0.12.0
 platform: ruby
 authors:
 - Washington Botelho
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-01-16 00:00:00.000000000 Z
+date: 2022-08-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -24,6 +24,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: codecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: database_cleaner
   requirement: !ruby/object:Gem::Requirement
@@ -38,6 +52,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: debug
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: factory_bot_rails
   requirement: !ruby/object:Gem::Requirement
@@ -67,7 +95,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: pry-byebug
+  name: pg
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -108,6 +136,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: rubocop-rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
@@ -140,7 +182,10 @@ description: A true Bayesian rating system with scope and cache enabled.
 email: wbotelhos@gmail.com
 executables: []
 extensions: []
-extra_rdoc_files: []
+extra_rdoc_files:
+- CHANGELOG.md
+- LICENSE
+- README.md
 files:
 - CHANGELOG.md
 - LICENSE
@@ -154,71 +199,12 @@ files:
 - lib/rating/models/rating/rate.rb
 - lib/rating/models/rating/rating.rb
 - lib/rating/version.rb
-- spec/config/rate_table_spec.rb
-- spec/config/rating_table_spec.rb
-- spec/config/validations_spec.rb
-- spec/factories/article.rb
-- spec/factories/author.rb
-- spec/factories/category.rb
-- spec/factories/comment.rb
-- spec/factories/global.rb
-- spec/factories/rating/rate.rb
-- spec/factories/rating/rating.rb
-- spec/factories/toy.rb
-- spec/models/extension/after_create_spec.rb
-- spec/models/extension/order_by_rating_spec.rb
-- spec/models/extension/rate_for_spec.rb
-- spec/models/extension/rate_spec.rb
-- spec/models/extension/rated_question_spec.rb
-- spec/models/extension/rated_records_spec.rb
-- spec/models/extension/rated_spec.rb
-- spec/models/extension/rates_records_spec.rb
-- spec/models/extension/rates_spec.rb
-- spec/models/extension/rating_records_spec.rb
-- spec/models/extension/rating_spec.rb
-- spec/models/extension/rating_warm_up_spec.rb
-- spec/models/extension/unscoped_rating_spec.rb
-- spec/models/extension/where_spec.rb
-- spec/models/rate/create_spec.rb
-- spec/models/rate/rate_for_spec.rb
-- spec/models/rate_spec.rb
-- spec/models/rating/averager_data_spec.rb
-- spec/models/rating/data_spec.rb
-- spec/models/rating/update_rating_spec.rb
-- spec/models/rating/values_data_spec.rb
-- spec/models/rating_spec.rb
-- spec/rails_helper.rb
-- spec/support/common.rb
-- spec/support/database_cleaner.rb
-- spec/support/db/migrate/add_comment_on_rating_rates_table.rb
-- spec/support/db/migrate/add_extra_fields_on_rating_rates_table.rb
-- spec/support/db/migrate/create_articles_table.rb
-- spec/support/db/migrate/create_authors_table.rb
-- spec/support/db/migrate/create_categories_table.rb
-- spec/support/db/migrate/create_comments_table.rb
-- spec/support/db/migrate/create_globals_table.rb
-- spec/support/db/migrate/create_rates_table.rb
-- spec/support/db/migrate/create_rating_table.rb
-- spec/support/db/migrate/create_review_ratings_table.rb
-- spec/support/db/migrate/create_reviews_table.rb
-- spec/support/db/migrate/create_toys_table.rb
-- spec/support/factory_bot.rb
-- spec/support/migrate.rb
-- spec/support/models/article.rb
-- spec/support/models/author.rb
-- spec/support/models/category.rb
-- spec/support/models/comment.rb
-- spec/support/models/global.rb
-- spec/support/models/review.rb
-- spec/support/models/review_rating.rb
-- spec/support/models/toy.rb
-- spec/support/shared_context/with_database_records.rb
-- spec/support/shoulda.rb
 homepage: https://github.com/wbotelhos/rating
 licenses:
 - MIT
-metadata: {}
-post_install_message: 
+metadata:
+  rubygems_mfa_required: 'true'
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -233,68 +219,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
-signing_key: 
+rubygems_version: 3.3.12
+signing_key:
 specification_version: 4
 summary: A true Bayesian rating system with scope and cache enabled.
-test_files:
-- spec/config/rate_table_spec.rb
-- spec/config/validations_spec.rb
-- spec/config/rating_table_spec.rb
-- spec/models/rate/rate_for_spec.rb
-- spec/models/rate/create_spec.rb
-- spec/models/rating/averager_data_spec.rb
-- spec/models/rating/data_spec.rb
-- spec/models/rating/update_rating_spec.rb
-- spec/models/rating/values_data_spec.rb
-- spec/models/extension/rated_records_spec.rb
-- spec/models/extension/rating_spec.rb
-- spec/models/extension/rated_question_spec.rb
-- spec/models/extension/rating_warm_up_spec.rb
-- spec/models/extension/order_by_rating_spec.rb
-- spec/models/extension/unscoped_rating_spec.rb
-- spec/models/extension/where_spec.rb
-- spec/models/extension/rate_spec.rb
-- spec/models/extension/rate_for_spec.rb
-- spec/models/extension/after_create_spec.rb
-- spec/models/extension/rates_spec.rb
-- spec/models/extension/rating_records_spec.rb
-- spec/models/extension/rated_spec.rb
-- spec/models/extension/rates_records_spec.rb
-- spec/models/rating_spec.rb
-- spec/models/rate_spec.rb
-- spec/support/migrate.rb
-- spec/support/shoulda.rb
-- spec/support/factory_bot.rb
-- spec/support/models/article.rb
-- spec/support/models/category.rb
-- spec/support/models/toy.rb
-- spec/support/models/author.rb
-- spec/support/models/review.rb
-- spec/support/models/comment.rb
-- spec/support/models/global.rb
-- spec/support/models/review_rating.rb
-- spec/support/common.rb
-- spec/support/db/migrate/create_categories_table.rb
-- spec/support/db/migrate/create_authors_table.rb
-- spec/support/db/migrate/create_review_ratings_table.rb
-- spec/support/db/migrate/create_articles_table.rb
-- spec/support/db/migrate/create_rating_table.rb
-- spec/support/db/migrate/create_toys_table.rb
-- spec/support/db/migrate/add_comment_on_rating_rates_table.rb
-- spec/support/db/migrate/add_extra_fields_on_rating_rates_table.rb
-- spec/support/db/migrate/create_reviews_table.rb
-- spec/support/db/migrate/create_comments_table.rb
-- spec/support/db/migrate/create_globals_table.rb
-- spec/support/db/migrate/create_rates_table.rb
-- spec/support/database_cleaner.rb
-- spec/support/shared_context/with_database_records.rb
-- spec/factories/article.rb
-- spec/factories/category.rb
-- spec/factories/rating/rate.rb
-- spec/factories/rating/rating.rb
-- spec/factories/toy.rb
-- spec/factories/author.rb
-- spec/factories/comment.rb
-- spec/factories/global.rb
-- spec/rails_helper.rb
+test_files: []

data/spec/config/rate_table_spec.rb

@@ -1,17 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-
-RSpec.describe Rating::Config, '.rate_table' do
-  if ENV['CONFIG_ENABLED'] != 'true'
-    context 'when rating.yml does not exist' do
-      it { expect(subject.rate_table).to eq 'rating_rates' }
-    end
-  end
-
-  if ENV['CONFIG_ENABLED'] == 'true'
-    context 'when rating.yml exists' do
-      it { expect(subject.rate_table).to eq 'reviews' }
-    end
-  end
-end

data/spec/config/rating_table_spec.rb

@@ -1,17 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-
-RSpec.describe Rating::Config, '.rating_table' do
-  if ENV['CONFIG_ENABLED'] != 'true'
-    context 'when rating.yml does not exist' do
-      it { expect(subject.rating_table).to eq 'rating_ratings' }
-    end
-  end
-
-  if ENV['CONFIG_ENABLED'] == 'true'
-    context 'when rating.yml exists' do
-      it { expect(subject.rating_table).to eq 'review_ratings' }
-    end
-  end
-end

data/spec/config/validations_spec.rb

@@ -1,33 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-
-RSpec.describe Rating::Config, '.validations' do
-  if ENV['CONFIG_ENABLED_WITH_EXTRA_SCOPES'] != 'true'
-    context 'when rating.yml does not exist' do
-      it do
-        expect(subject.validations).to eq({
-          rate: {
-            case_sensitive: false,
-            scope:          %w[author_type resource_id resource_type scopeable_id scopeable_type],
-          },
-        }.deep_stringify_keys
-                                         )
-      end
-    end
-  end
-
-  if ENV['CONFIG_ENABLED_WITH_EXTRA_SCOPES'] == 'true'
-    context 'when rating.yml exists' do
-      it do
-        expect(subject.validations).to eq({
-          rate: {
-            case_sensitive: false,
-            scope:          %w[author_type resource_id resource_type scopeable_id scopeable_type scope_1 scope_2],
-          },
-        }.deep_stringify_keys
-                                         )
-      end
-    end
-  end
-end

data/spec/factories/article.rb

@@ -1,7 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :article do
-    sequence(:name) { |i| "Article #{i}" }
-  end
-end

data/spec/factories/author.rb

@@ -1,7 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :author do
-    sequence(:name) { |i| "Author #{i}" }
-  end
-end

data/spec/factories/category.rb

@@ -1,7 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :category do
-    sequence(:name) { |i| "Category #{i}" }
-  end
-end

data/spec/factories/comment.rb

@@ -1,5 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :comment
-end

data/spec/factories/global.rb

@@ -1,6 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :global do
-  end
-end

data/spec/factories/rating/rate.rb

@@ -1,10 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :rating_rate, class: Rating::Rate do
-    value { 100 }
-
-    author   { create :author }
-    resource { create :article }
-  end
-end

data/spec/factories/rating/rating.rb

@@ -1,12 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :rating_rating, class: Rating::Rating do
-    average  { 100 }
-    estimate { 100 }
-    sum      { 100 }
-    total    { 1 }
-
-    association :resource, factory: :article, strategy: :build
-  end
-end

data/spec/factories/toy.rb

@@ -1,6 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :toy do
-  end
-end

data/spec/models/extension/after_create_spec.rb

@@ -1,47 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-
-RSpec.describe Rating::Extension, 'after_create' do
-  context 'when record is author' do
-    let!(:record) { build :author }
-
-    it 'does not warm up the cache' do
-      expect(record).not_to receive(:rating_warm_up)
-
-      record.save
-    end
-  end
-
-  context 'when record is not author' do
-    context 'when record has scoping' do
-      let!(:record) { build :article }
-
-      it 'warms up the cache' do
-        expect(record).to receive(:rating_warm_up).with(scoping: :categories)
-
-        record.save
-      end
-    end
-
-    context 'when record has no scoping' do
-      let!(:record) { build :comment }
-
-      it 'warms up the cache' do
-        expect(record).to receive(:rating_warm_up).with(scoping: nil)
-
-        record.save
-      end
-    end
-
-    context 'when update is made' do
-      let!(:record) { create :comment }
-
-      it 'does not warm up the cache' do
-        expect(record).not_to receive(:rating_warm_up)
-
-        record.save
-      end
-    end
-  end
-end