mongoid-votable 0.0.1 → 0.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 1eb71616709305c0a427619ad1cbe815707b4e5d
-  data.tar.gz: bc95294d564dc8a6580cdcc521994541fdb7735c
+  metadata.gz: a34ae76962b7b600d4201a76731e1bbdbea222cd
+  data.tar.gz: 4c64cf417ceab57073205364e07f71d6960924ed
 SHA512:
-  metadata.gz: 2231ad970b2338e088c26654746e44d65d7032b4ab800b3b9667fafeedf4e722d4a3fe3e2c80ca669e0b942bd9c4d015991242f37d71ad5ebd88c1a837bd4e20
-  data.tar.gz: 8198702962e25840c5b74a84760811b0aefacb71d48b6ab6d4b05ac4a3a2db8b1108ca0346c133abc65d5c0fad9090e3d82fd824584ab00d99c6b1891fed4c4a
+  metadata.gz: 1af78cf0eb8cb66304e50fe9f06a4a21a4f4e19179e1c7597c4b6c2c3ef857a1e92a6adea7c4de72e895f79a237a58c6b3b0b2214320c24f14e7a2df9bc91659
+  data.tar.gz: 9a9c15021bf6a66a3982bb6d8ec4f7f49d6221677376e61a4f3cef6748353d5bed1d8255b7d38b1b72599fd58d663d933512af59b68af72193a2488a4a016566

data/.rspec

@@ -0,0 +1,2 @@
+--color
+--format progress

data/README.md

@@ -18,7 +18,22 @@ Or install it yourself as:
 
 ## Usage
 
-
+~~~ruby
+class Topic
+  include Mongoid::Document
+  include Mongoid::Votable
+  
+end
+~~~
+
+~~~ruby
+topic.vote!(1, user)
+topic.vote!(-1, user)
+
+topic.vote!(10, user)
+
+topics = Topic.voted_by(user)
+~~~
 
 ## Contributing
 

data/lib/mongoid/votable.rb

@@ -2,7 +2,7 @@
 module Mongoid
   module Votable
     extend ActiveSupport::Concern
-    
+
     included do
       field :votes_count, type: Integer, default: 0
       field :votes_average, type: Integer, default: 0
@@ -13,7 +13,7 @@ module Mongoid
       embeds_many :votes, as: :votable, class_name: "Mongoid::Vote"
       
       scope :voted_by, ->(voter) do
-        where("votes.voter" => voter)
+        where("votes.voter_id" => voter.id)
       end
     end  
     
@@ -53,38 +53,39 @@ module Mongoid
       end
     end
     
-    def do_vote!(vote, value, status)
-      case status
-        when :created
-          if value > 0 
-            inc(:votes_up, 1)
-          else
-            inc(:votes_down, 1)
-          end
-          inc(:votes_count, 1)
-        when :updated
-          #Origin is vote down, but now is vote up
-          if vote.value - value < 0 && vote.value > 0
-            inc(:votes_up, 1)
-            inc(:votes_down, -1)
-          #Origin is vote up, but now is vote down
-          elsif vote.value - value > 0 && vote.value < 0 
-            inc(:votes_down, 1)
-            inc(:votes_up, -1)
-          end
-        when :destroyed
-          if value > 0
-            inc(:votes_down, -1)
+    private 
+      def do_vote!(vote, value, status)
+        case status
+          when :created
+            if value > 0 
+              inc(:votes_up, 1)
+            else
+              inc(:votes_down, 1)
+            end
+            inc(:votes_count, 1)
+          when :updated
+            #Origin is vote down, but now is vote up
+            if vote.value - value < 0 && vote.value > 0
+              inc(:votes_up, 1)
+              inc(:votes_down, -1)
+            #Origin is vote up, but now is vote down
+            elsif vote.value - value > 0 && vote.value < 0 
+              inc(:votes_down, 1)
+              inc(:votes_up, -1)
+            end
+          when :destroyed
+            if value > 0
+              inc(:votes_down, -1)
+            else
+              inc(:votes_up, -1)
+            end
+            inc(:votes_count, -1)          
           else
-            inc(:votes_up, -1)
-          end
-          inc(:votes_count, -1)          
-        else
-          raise "Not accept status"
-      end    
-      inc(:votes_average, value)
-      #set(:votes_average, votes.map(&:value).reduce(0, &:+))
-    end
+            raise "Not accept status"
+        end    
+        inc(:votes_average, value)
+        #set(:votes_average, votes.map(&:value).reduce(0, &:+))
+      end
     
     
     module ClassMethods  

data/lib/mongoid/votable/version.rb

@@ -1,5 +1,5 @@
 module Mongoid
   module Votable
-    VERSION = "0.0.1"
+    VERSION = "0.1.0"
   end
 end

data/mongoid-votable.gemspec

@@ -21,6 +21,7 @@ Gem::Specification.new do |spec|
   spec.add_dependency("mongoid",  [">= 3.0.0"])
   spec.add_dependency("activesupport", [">= 3.0.0"])
 
+  spec.add_development_dependency "factory_girl"
   spec.add_development_dependency "rspec"
   spec.add_development_dependency "bundler"
   spec.add_development_dependency "rake"

data/spec/config/mongoid.yml

@@ -0,0 +1,6 @@
+test:
+  sessions:
+    default:
+      database: mongoid-votable_test
+      hosts:
+        - localhost:27017

data/spec/factories/topics.rb

@@ -0,0 +1,5 @@
+FactoryGirl.define do
+  factory :topic do
+
+  end
+end

data/spec/factories/users.rb

@@ -0,0 +1,5 @@
+FactoryGirl.define do
+  factory :user do    
+    name     "User"
+  end 
+end

data/spec/models/topic.rb

@@ -0,0 +1,6 @@
+# encoding: UTF-8
+class Topic
+  include Mongoid::Document
+  include Mongoid::Votable
+
+end

data/spec/models/user.rb

@@ -0,0 +1,4 @@
+class User
+  include Mongoid::Document
+  field :name
+end

data/spec/mongoid/votable_spec.rb

@@ -0,0 +1,149 @@
+require 'spec_helper'
+
+module Mongoid
+  describe Votable do    
+    it { Topic.should respond_to(:voted_by) }
+    
+    let(:user) { create(:user) }
+    let(:user2) { create(:user) }
+    let(:topic) { create(:topic) }
+    
+    subject { topic }
+        
+    it { should respond_to(:votes, :votes_count, :votes_up, :votes_down, :votes_average) }
+    
+    its(:votes_count) { should == 0 }
+    its(:votes_up) { should == 0 }
+    its(:votes_down) { should == 0 }
+    its(:votes_average) { should == 0 }
+    its(:votes) { should be_empty }
+    
+    context "single user" do 
+    
+      context "user vote up" do
+        before(:each) { topic.vote!(1, user) }
+        its(:votes_count) { should == 1 }
+        its(:votes_up) { should == 1 }
+        its(:votes_down) { should == 0 }
+        its(:votes_average) { should == 1 }
+        its(:votes) { should_not be_empty }
+      end
+    
+      context "user vote down" do
+        before(:each) { topic.vote!(-1, user) }
+        its(:votes_count) { should == 1 }
+        its(:votes_up) { should == 0 }
+        its(:votes_down) { should == 1 }
+        its(:votes_average) { should == -1 }
+        its(:votes) { should_not be_empty }
+      end
+    
+      context "user vote up, then vote down with same value" do
+        before(:each) do 
+          topic.vote!(3, user) 
+          topic.vote!(-3, user) 
+        end
+        its(:votes_count) { should == 0 }
+        its(:votes_up) { should == 0 }
+        its(:votes_down) { should == 0 }
+        its(:votes_average) { should == 0 }
+        its(:votes) { should be_empty }
+      end
+    
+      context "user vote up, then vote down, (down > up)" do
+        before(:each) do 
+          topic.vote!(3, user) 
+          topic.vote!(-6, user) 
+        end
+        its(:votes_count) { should == 1 }
+        its(:votes_up) { should == 0 }
+        its(:votes_down) { should == 1 }
+        its(:votes_average) { should == -3 }
+        its(:votes) { should_not be_empty }
+      end    
+    
+      context "user vote up, then vote down, (down < up)" do
+        before(:each) do 
+          topic.vote!(3, user) 
+          topic.vote!(-1, user) 
+        end
+        its(:votes_count) { should == 1 }
+        its(:votes_up) { should == 1 }
+        its(:votes_down) { should == 0 }
+        its(:votes_average) { should == 2 }
+        its(:votes) { should_not be_empty }
+      end  
+    
+    end
+    
+    context "multiple users" do 
+    
+      context "users all vote up" do
+        before(:each) do
+          topic.vote!(1, user) 
+          topic.vote!(1, user2) 
+        end
+        its(:votes_count) { should == 2 }
+        its(:votes_up) { should == 2 }
+        its(:votes_down) { should == 0 }
+        its(:votes_average) { should == 2 }
+        its(:votes) { should have(2).items }
+      end
+      
+      context "users all vote down" do
+        before(:each) do
+          topic.vote!(-1, user) 
+          topic.vote!(-1, user2) 
+        end
+        its(:votes_count) { should == 2 }
+        its(:votes_up) { should == 0 }
+        its(:votes_down) { should == 2 }
+        its(:votes_average) { should == -2 }
+        its(:votes) { should have(2).items }
+      end
+      
+      context "users with different votes" do
+        before(:each) do
+          topic.vote!(1, user) 
+          topic.vote!(-1, user2) 
+        end
+        its(:votes_count) { should == 2 }
+        its(:votes_up) { should == 1 }
+        its(:votes_down) { should == 1 }
+        its(:votes_average) { should == 0 }
+        its(:votes) { should have(2).items }
+      end
+      
+      context "users with different votes" do
+        before(:each) do
+          topic.vote!(1, user) 
+          topic.vote!(-2, user2) 
+        end
+        its(:votes_count) { should == 2 }
+        its(:votes_up) { should == 1 }
+        its(:votes_down) { should == 1 }
+        its(:votes_average) { should == -1 }
+        its(:votes) { should have(2).items }
+      end
+    
+    end
+    
+    context "voted_by" do 
+      before(:each) do
+        topic.vote!(1, user) 
+        topic.vote!(1, user2) 
+      end
+      subject { Topic.voted_by(user) }
+      
+      it { should have(1).items }      
+      its(:first) { should be_instance_of Topic }      
+    end
+    
+    context "voted?" do
+      before(:each) { topic.vote!(1, user) }
+      it { topic.voted?(user).should be_true }
+      it { topic.voted?(user2).should be_false }
+    end    
+        
+  end 
+end

data/spec/spec_helper.rb

@@ -0,0 +1,35 @@
+# This file was generated by the `rspec --init` command. Conventionally, all
+# specs live under a `spec` directory, which RSpec adds to the `$LOAD_PATH`.
+# Require this file using `require "spec_helper"` to ensure that it is only
+# loaded once.
+#
+# See http://rubydoc.info/gems/rspec-core/RSpec/Core/Configuration
+require 'mongoid'
+require 'mongoid-votable'
+require 'active_support'
+require 'factory_girl'
+
+FactoryGirl.find_definitions
+
+ENV['MONGOID_ENV'] = 'test'
+
+Mongoid.load!("#{File.dirname(__FILE__)}/config/mongoid.yml")
+Mongoid.logger = nil
+
+RSpec.configure do |config|
+  config.treat_symbols_as_metadata_keys_with_true_values = true
+  config.run_all_when_everything_filtered = true
+  config.filter_run :focus
+  
+  config.mock_with :rspec
+
+  # Run specs in random order to surface order dependencies. If you find an
+  # order dependency and want to debug it, you can fix the order by providing
+  # the seed, which is printed after each run.
+  #     --seed 1234
+  config.order = 'random'
+  
+  config.include FactoryGirl::Syntax::Methods
+  
+  Dir['./spec/models/*.rb'].each { |f| require f }
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mongoid-votable
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.1.0
 platform: ruby
 authors:
 - James Zhan
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-03-16 00:00:00.000000000 Z
+date: 2013-03-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mongoid
@@ -38,6 +38,20 @@ dependencies:
     - - '>='
       - !ruby/object:Gem::Version
         version: 3.0.0
+- !ruby/object:Gem::Dependency
+  name: factory_girl
+  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: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -88,6 +102,7 @@ extensions: []
 extra_rdoc_files: []
 files:
 - .gitignore
+- .rspec
 - Gemfile
 - LICENSE.txt
 - README.md
@@ -97,6 +112,13 @@ files:
 - lib/mongoid/votable/version.rb
 - lib/mongoid/vote.rb
 - mongoid-votable.gemspec
+- spec/config/mongoid.yml
+- spec/factories/topics.rb
+- spec/factories/users.rb
+- spec/models/topic.rb
+- spec/models/user.rb
+- spec/mongoid/votable_spec.rb
+- spec/spec_helper.rb
 homepage: ''
 licenses:
 - MIT
@@ -121,5 +143,12 @@ rubygems_version: 2.0.3
 signing_key: 
 specification_version: 4
 summary: Votable for mongoid
-test_files: []
+test_files:
+- spec/config/mongoid.yml
+- spec/factories/topics.rb
+- spec/factories/users.rb
+- spec/models/topic.rb
+- spec/models/user.rb
+- spec/mongoid/votable_spec.rb
+- spec/spec_helper.rb
 has_rdoc: