solidus_reviews 1.0.0 → 1.4.0

data/app/models/spree/product_decorator.rb

@@ -1,21 +0,0 @@
-# Add access to reviews/ratings to the product model
-Spree::Product.class_eval do
-  has_many :reviews
-
-  def stars
-    avg_rating.try(:round) || 0
-  end
-
-  def recalculate_rating
-    reviews_count = self.reviews.reload.approved.count
-
-    self.reviews_count = reviews_count
-    if reviews_count > 0
-      self.avg_rating = self.reviews.approved.sum(:rating).to_f / reviews_count
-    else
-      self.avg_rating = 0
-    end
-    self.save
-  end
-
-end

data/lib/spree_reviews/engine.rb

@@ -1,17 +0,0 @@
-module SpreeReviews
-  class Engine < Rails::Engine
-    require 'spree/core'
-    isolate_namespace Spree
-    engine_name 'solidus_reviews'
-
-    config.autoload_paths += %W(#{config.root}/lib)
-
-    def self.activate
-      Dir.glob(File.join(File.dirname(__FILE__), '../../app/**/*_decorator*.rb')) do |c|
-        Rails.configuration.cache_classes ? require(c) : load(c)
-      end
-      Spree::Ability.register_ability(Spree::ReviewsAbility)
-    end
-    config.to_prepare &method(:activate).to_proc
-  end
-end

data/spec/controllers/admin/review_settings_controller_spec.rb

@@ -1,57 +0,0 @@
-require 'spec_helper'
-
-describe Spree::Admin::ReviewSettingsController do
-  stub_authorization!
-
-  before do
-    user = create(:admin_user)
-    controller.stub(:try_spree_current_user => user)
-  end
-
-  context '#update' do
-    it 'redirects to edit-review-settings page' do
-      spree_put :update, preferences: { preview_size: 4 }
-      response.should redirect_to spree.edit_admin_review_settings_path
-    end
-
-    context 'For parameters:
-            preview_size: 4,
-            show_email: false,
-            feedback_rating: false,
-            require_login: true,
-            track_locale: true' do
-
-      it 'sets preferred_preview_size to 4' do
-        spree_put :update, preferences: { preview_size: 4 }
-        Spree::Reviews::Config.preferred_preview_size.should eq 4
-      end
-
-      it 'sets preferred_show_email to false' do
-        spree_put :update, preferences: { show_email: false }
-        Spree::Reviews::Config.preferred_show_email.should be false
-      end
-
-      it 'sets preferred_feedback_rating to false' do
-        spree_put :update, preferences: { feedback_rating: false }
-        Spree::Reviews::Config.preferred_feedback_rating.should be false
-      end
-
-      it 'sets preferred_require_login to true' do
-        spree_put :update, preferences: { require_login: true }
-        Spree::Reviews::Config.preferred_require_login.should be true
-      end
-
-      it 'sets preferred_track_locale to true' do
-        spree_put :update, preferences: { track_locale: true }
-        Spree::Reviews::Config.preferred_track_locale.should be true
-      end
-    end
-  end
-
-  context '#edit' do
-    it 'should render the edit template' do
-      spree_get :edit
-      response.should render_template(:edit)
-    end
-  end
-end

data/spec/controllers/admin/reviews_controller_spec.rb

@@ -1,63 +0,0 @@
-require 'spec_helper'
-
-describe Spree::Admin::ReviewsController do
-  stub_authorization!
-
-  let(:product) { create(:product) }
-  let(:review) { create(:review, approved: false) }
-
-  before do
-    user = create(:admin_user)
-    controller.stub(try_spree_current_user: user)
-  end
-
-  context '#index' do
-    it 'list reviews' do
-      reviews = [
-        create(:review, product: product),
-        create(:review, product: product)
-      ]
-      spree_get :index, product_id: product.slug
-      assigns[:reviews].should =~ reviews
-    end
-  end
-
-  context '#approve' do
-    it 'show notice message when approved' do
-      review.update_attribute(:approved, true)
-      spree_get :approve, id: review.id
-      response.should redirect_to spree.admin_reviews_path
-      flash[:notice].should eq Spree.t(:info_approve_review)
-    end
-
-    it 'show error message when not approved' do
-      Spree::Review.any_instance.stub(:update_attribute).and_return(false)
-      spree_get :approve, id: review.id
-      flash[:error].should eq Spree.t(:error_approve_review)
-    end
-  end
-
-  context '#edit' do
-    specify do
-      spree_get :edit, id: review.id
-      response.status.should eq(200)
-    end
-
-    context 'when product is nil' do
-      before do
-        review.product = nil
-        review.save!
-      end
-
-      it 'flash error' do
-        spree_get :edit, id: review.id
-        flash[:error].should eq Spree.t(:error_no_product)
-      end
-
-      it 'redirect to admin-reviews page' do
-        spree_get :edit, id: review.id
-        response.should redirect_to spree.admin_reviews_path
-      end
-    end
-  end
-end

data/spec/controllers/feedback_reviews_controller_spec.rb

@@ -1,71 +0,0 @@
-require 'spec_helper'
-
-describe Spree::FeedbackReviewsController do
-  let(:user) { create(:user) }
-  let(:product) { create(:product) }
-  let(:review) { create(:review, user: user) }
-  let(:valid_attributes) do
-    { review_id: review.id,
-      user_id: user.id,
-      feedback_review: {
-        rating: '4 stars',
-        comment: 'some comment'
-      }}
-  end
-
-  before do
-    controller.stub spree_current_user: user
-    controller.stub spree_user_signed_in?: true
-    request.env['HTTP_REFERER'] = '/'
-  end
-
-  describe '#create' do
-    it 'creates a new feedback review' do
-      rating = 4
-      comment = FFaker::Lorem.paragraphs(3).join("\n")
-      expect {
-        spree_post :create, { review_id: review.id,
-                              feedback_review: { comment: comment,
-                                                 rating: rating },
-                              format: :js }
-        response.status.should eq(200)
-        response.should render_template(:create)
-      }.to change(Spree::Review, :count).by(1)
-      feedback_review = Spree::FeedbackReview.last
-      feedback_review.comment.should eq(comment)
-      feedback_review.review.should eq(review)
-      feedback_review.rating.should eq(rating)
-      feedback_review.user.should eq(user)
-
-    end
-
-    it 'redirects back to the calling page' do
-      spree_post :create, valid_attributes
-      response.should redirect_to '/'
-    end
-
-    it 'sets locale on feedback-review if required by config' do
-      Spree::Reviews::Config.preferred_track_locale = true
-      spree_post :create, valid_attributes
-      assigns[:review].locale.should eq I18n.locale.to_s
-    end
-
-    it 'fails when user is not authorized' do
-      controller.stub(:authorize!) { raise }
-      expect {
-        spree_post :create, valid_attributes
-      }.to raise_error
-    end
-
-    it 'removes all non-numbers from ratings parameter' do
-      spree_post :create, valid_attributes
-      controller.params[:feedback_review][:rating].should eq '4'
-    end
-
-    it 'do not create feedback-review if review doesnt exist' do
-      expect {
-        spree_post :create, valid_attributes.merge!({review_id: nil})
-      }.to raise_error
-    end
-  end
-end

data/spec/controllers/products_controller_spec.rb

@@ -1,9 +0,0 @@
-require 'spec_helper'
-
-describe Spree::ProductsController do
-  [:avg_rating, :reviews_count].each do |attrib|
-    it "should add #{attrib} to the set of allowed attributes" do
-      controller.permitted_product_attributes.should include(attrib)
-    end
-  end
-end

data/spec/controllers/reviews_controller_spec.rb

@@ -1,136 +0,0 @@
-require 'spec_helper'
-
-describe Spree::ReviewsController, type: :controller do
-  let(:user) { create(:user) }
-  let(:product) { create(:product) }
-  let(:review_params) do
-    { product_id: product.slug,
-      review: { rating: 3,
-                name: 'Ryan Bigg',
-                title: 'Great Product',
-                review: 'Some big review text..' } }
-  end
-
-  before do
-    controller.stub :spree_current_user => user
-    controller.stub :spree_user_signed_in? => true
-  end
-
-  context '#index' do
-    context 'for a product that does not exist' do
-      it 'responds with a 404' do
-        spree_get :index, product_id: 'not_real'
-        response.status.should eq(404)
-      end
-    end
-
-    context 'for a valid product' do
-      it 'list approved reviews' do
-        approved_reviews = [
-          create(:review, :approved, product: product),
-          create(:review, :approved, product: product)
-        ]
-        spree_get :index, product_id: product.slug
-        assigns[:approved_reviews].should =~ approved_reviews
-      end
-    end
-  end
-
-  context '#new' do
-    context 'for a product that does not exist' do
-      it 'responds with a 404' do
-        spree_get :new, product_id: 'not_real'
-        response.status.should eq(404)
-      end
-    end
-
-    it 'fail if the user is not authorized to create a review' do
-      controller.stub(:authorize!) { raise }
-      expect {
-        spree_post :new, product_id: product.slug
-        assert_match 'ryanbig', response.body
-      }.to raise_error
-    end
-
-    it 'render the new template' do
-      spree_get :new, product_id: product.slug
-      response.status.should eq(200)
-      response.should render_template(:new)
-    end
-  end
-
-  context '#create' do
-    before { controller.stub spree_current_user: user }
-
-    context 'for a product that does not exist' do
-      it 'responds with a 404' do
-        spree_post :create, product_id: 'not_real'
-        response.status.should eq(404)
-      end
-    end
-
-    it 'creates a new review' do
-      expect {
-        spree_post :create, review_params
-      }.to change(Spree::Review, :count).by(1)
-    end
-
-    it 'sets the ip-address of the remote' do
-      request.stub(remote_ip: '127.0.0.1')
-      spree_post :create, review_params
-      assigns[:review].ip_address.should eq '127.0.0.1'
-    end
-
-    it 'fails if the user is not authorized to create a review' do
-      controller.stub(:authorize!) { raise }
-      expect{
-        spree_post :create, review_params
-      }.to raise_error
-    end
-
-    it 'flashes the notice' do
-      spree_post :create, review_params
-      flash[:notice].should eq Spree.t(:review_successfully_submitted)
-    end
-
-    it 'redirects to product page' do
-      spree_post :create, review_params
-      response.should redirect_to spree.product_path(product)
-    end
-
-    it 'removes all non-numbers from ratings param' do
-      spree_post :create, review_params
-      controller.params[:review][:rating].should eq '3'
-    end
-
-    it 'sets the current spree user as reviews user' do
-      spree_post :create, review_params
-      review_params[:review].merge!(user_id: user.id)
-      assigns[:review][:user_id] = user.id
-      assigns[:review][:user_id].should eq user.id
-    end
-
-    context 'with invalid params' do
-      it 'renders new when review.save fails' do
-        Spree::Review.any_instance.stub(:save).and_return(false)
-        spree_post :create, review_params
-        response.should render_template :new
-      end
-
-      it 'does not create a review' do
-        expect(Spree::Review.count).to eq 0
-        spree_post :create, review_params.merge({review: {rating: 'not_a_number'}})
-        expect(Spree::Review.count).to eq 0
-      end
-    end
-
-    # It always sets the locale so preference pointless
-    context 'when config requires locale tracking:' do
-      it 'sets the locale' do
-        Spree::Reviews::Config.preferred_track_locale = true
-        spree_post :create, review_params
-        assigns[:review].locale.should eq I18n.locale.to_s
-      end
-    end
-  end
-end