rating 0.1.0

data/lib/rating/version.rb

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

data/spec/factories/article.rb

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

data/spec/factories/rating/rate.rb

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

data/spec/factories/rating/rating.rb

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

data/spec/factories/user.rb

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

data/spec/models/extension/after_create_spec.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe Rating::Extension, ':after_create' do
+  context 'when creates object' do
+    let!(:user) { create :user }
+
+    it 'creates a record with zero values just to be easy to make the count' do
+      rating = Rating::Rating.find_by(resource: user)
+
+      expect(rating.average).to  eq 0
+      expect(rating.estimate).to eq 0
+      expect(rating.resource).to eq user
+      expect(rating.sum).to      eq 0
+      expect(rating.total).to    eq 0
+    end
+  end
+end

data/spec/models/extension/order_by_rating_spec.rb

@@ -0,0 +1,121 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe Rating::Extension, ':order_by_rating' do
+  let!(:user_1) { create :user }
+  let!(:user_2) { create :user }
+
+  let!(:article_1) { create :article }
+  let!(:article_2) { create :article }
+  let!(:article_3) { create :article }
+
+  before do
+    create :rating_rate, author: user_1, resource: article_1, value: 100
+    create :rating_rate, author: user_1, resource: article_2, value: 11
+    create :rating_rate, author: user_1, resource: article_3, value: 10
+    create :rating_rate, author: user_2, resource: article_1, value: 1
+  end
+
+  context 'with default filters' do
+    it 'sorts by :estimate :desc' do
+      expect(Article.order_by_rating).to eq [
+        article_1,
+        article_2,
+        article_3
+      ]
+    end
+  end
+
+  context 'filtering by :average' do
+    context 'as asc' do
+      it 'works' do
+        expect(Article.order_by_rating(:average, :asc)).to eq [
+          article_3,
+          article_2,
+          article_1
+        ]
+      end
+    end
+
+    context 'as desc' do
+      it 'works' do
+        expect(Article.order_by_rating(:average, :desc)).to eq [
+          article_1,
+          article_2,
+          article_3
+        ]
+      end
+    end
+  end
+
+  context 'filtering by :estimate' do
+    context 'as asc' do
+      it 'works' do
+        expect(Article.order_by_rating(:estimate, :asc)).to eq [
+          article_3,
+          article_2,
+          article_1
+        ]
+      end
+    end
+
+    context 'as desc' do
+      it 'works' do
+        expect(Article.order_by_rating(:estimate, :desc)).to eq [
+          article_1,
+          article_2,
+          article_3
+        ]
+      end
+    end
+  end
+
+  context 'filtering by :sum' do
+    context 'as asc' do
+      it 'works' do
+        expect(Article.order_by_rating(:sum, :asc)).to eq [
+          article_3,
+          article_2,
+          article_1
+        ]
+      end
+    end
+
+    context 'as desc' do
+      it 'works' do
+        expect(Article.order_by_rating(:sum, :desc)).to eq [
+          article_1,
+          article_2,
+          article_3
+        ]
+      end
+    end
+
+    context 'filtering by :total' do
+      context 'as asc' do
+        it 'works' do
+          result = Article.order_by_rating(:total, :asc)
+
+          expect(result[0..1]).to match_array [article_2, article_3]
+          expect(result.last).to  eq article_1
+        end
+      end
+
+      context 'as desc' do
+        it 'works' do
+          result = Article.order_by_rating(:total, :desc)
+
+          expect(result.first).to eq article_1
+          expect(result[1..2]).to match_array [article_2, article_3]
+        end
+      end
+    end
+  end
+
+  context 'with other resource' do
+    it 'works' do
+      expect(User.order_by_rating(:total, :desc)).to match_array [user_1, user_2]
+    end
+  end
+end

data/spec/models/extension/rate_for_spec.rb

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe Rating::Extension, ':rate_for' do
+  let!(:user)    { create :user }
+  let!(:article) { create :article }
+
+  it 'delegates to rate object' do
+    expect(Rating::Rate).to receive(:rate_for).with author: user, resource: article
+
+    user.rate_for article
+  end
+end

data/spec/models/extension/rate_spec.rb

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe Rating::Extension, ':rate' do
+  let!(:user)    { create :user }
+  let!(:article) { create :article }
+
+  it 'delegates to rate object' do
+    expect(Rating::Rate).to receive(:create).with author: user, resource: article, value: 3
+
+    user.rate article, 3
+  end
+end

data/spec/models/extension/rated_question_spec.rb

@@ -0,0 +1,20 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe Rating::Extension, ':rated?' do
+  let!(:user)    { create :user }
+  let!(:article) { create :article }
+
+  context 'when has no rate for the given resource' do
+    before { allow(user).to receive(:rate_for).with(article) { nil } }
+
+    specify { expect(user.rated?(article)).to eq false }
+  end
+
+  context 'when has rate for the given resource' do
+    before { allow(user).to receive(:rate_for).with(article) { double } }
+
+    specify { expect(user.rated?(article)).to eq true }
+  end
+end

data/spec/models/extension/rated_spec.rb

@@ -0,0 +1,38 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe Rating::Extension, ':rated' do
+  let!(:user)    { create :user }
+  let!(:article) { create :article }
+
+  before { user.rate article, 3 }
+
+  it 'returns rates made by the caller' do
+    expect(user.rated).to eq [Rating::Rate.find_by(resource: article)]
+  end
+
+  context 'when destroy author' do
+    before do
+      expect(Rating::Rate.where(author: user).count).to eq 1
+
+      user.destroy!
+    end
+
+    it 'destroys rates of this author' do
+      expect(Rating::Rate.where(author: user).count).to eq 0
+    end
+  end
+
+  context 'when destroy resource rated by author' do
+    before do
+      expect(Rating::Rate.where(resource: article).count).to eq 1
+
+      article.destroy!
+    end
+
+    it 'destroys rates for that resource' do
+      expect(Rating::Rate.where(resource: article).count).to eq 0
+    end
+  end
+end

data/spec/models/extension/rates_spec.rb

@@ -0,0 +1,38 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe Rating::Extension, ':rates' do
+  let!(:user)    { create :user }
+  let!(:article) { create :article }
+
+  before { user.rate article, 3 }
+
+  it 'returns rates record' do
+    expect(article.rates).to eq [Rating::Rate.last]
+  end
+
+  context 'when destroy author' do
+    before do
+      expect(Rating::Rate.where(resource: article).count).to eq 1
+
+      user.destroy!
+    end
+
+    it 'destroys rates of that resource' do
+      expect(Rating::Rate.where(resource: article).count).to eq 0
+    end
+  end
+
+  context 'when destroy resource' do
+    before do
+      expect(Rating::Rate.where(resource: article).count).to eq 1
+
+      article.destroy!
+    end
+
+    it 'destroys rates of that resource' do
+      expect(Rating::Rate.where(resource: article).count).to eq 0
+    end
+  end
+end

data/spec/models/extension/rating_spec.rb

@@ -0,0 +1,38 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe Rating::Extension, ':rating' do
+  let!(:user_1)  { create :user }
+  let!(:article) { create :article }
+
+  before { user_1.rate article, 1 }
+
+  it 'returns rating record' do
+    expect(article.rating).to eq Rating::Rating.last
+  end
+
+  context 'when destroy author' do
+    let!(:user_2) { create :user }
+
+    before { user_2.rate article, 2 }
+
+    it 'does not destroy resource rating' do
+      expect(Rating::Rating.where(resource: article).count).to eq 1
+
+      user_1.destroy!
+
+      expect(Rating::Rating.where(resource: article).count).to eq 1
+    end
+  end
+
+  context 'when destroy resource' do
+    it 'destroys resource rating too' do
+      expect(Rating::Rating.where(resource: article).count).to eq 1
+
+      article.destroy!
+
+      expect(Rating::Rating.where(resource: article).count).to eq 0
+    end
+  end
+end

data/spec/models/rate/create_spec.rb

@@ -0,0 +1,64 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe Rating::Rate, ':create' do
+  let!(:user)    { create :user }
+  let!(:article) { create :article }
+
+  before { create :rating_rate, author: user, resource: article, value: 3 }
+
+  context 'when rate does not exist yet' do
+    it 'creates a rate entry' do
+      rate = described_class.last
+
+      expect(rate.author).to   eq user
+      expect(rate.resource).to eq article
+      expect(rate.value).to    eq 3
+    end
+
+    it 'creates a rating entry' do
+      rating = Rating::Rating.last
+
+      expect(rating.average).to  eq 3
+      expect(rating.estimate).to eq 3
+      expect(rating.resource).to eq article
+      expect(rating.sum).to      eq 3
+      expect(rating.total).to    eq 1
+    end
+  end
+
+  context 'when rate already exists' do
+    let!(:user_2) { create :user }
+
+    before { create :rating_rate, author: user_2, resource: article, value: 4 }
+
+    it 'creates one more rate entry' do
+      rates = described_class.where(author: [user, user_2]).order('created_at asc')
+
+      expect(rates.size).to eq 2
+
+      rate = rates[0]
+
+      expect(rate.author).to   eq user
+      expect(rate.resource).to eq article
+      expect(rate.value).to    eq 3
+
+      rate = rates[1]
+
+      expect(rate.author).to   eq user_2
+      expect(rate.resource).to eq article
+      expect(rate.value).to    eq 4
+    end
+
+    it 'updates the unique rating entry' do
+      rating = Rating::Rating.find_by(resource: article)
+
+      expect(rating.average).to  eq 3.5
+      expect(rating.estimate).to eq 3.5
+      expect(rating.resource).to eq article
+      expect(rating.sum).to      eq 7
+      expect(rating.total).to    eq 2
+    end
+  end
+end

data/spec/models/rate/rate_for_spec.rb

@@ -0,0 +1,20 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe Rating::Rate, ':rate_for' do
+  let!(:user)    { create :user }
+  let!(:article) { create :article }
+
+  context 'when rate does not exist' do
+    specify { expect(described_class.rate_for(author: user, resource: article)).to eq nil }
+  end
+
+  context 'when rate does not exist' do
+    before { described_class.create author: user, resource: article, value: 3 }
+
+    it 'returns the record' do
+      expect(described_class.rate_for(author: user, resource: article)).to eq described_class.last
+    end
+  end
+end