mongoid_atomic_votes 0.2.0 → 0.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 1471df3a89ef9456b35bc2e078fb8b73d0fc89a9
-  data.tar.gz: 561f2eac6abec232a1966ee23181bbb09b290325
+  metadata.gz: 474842edee2815d2d65750077a75894ac7f67dc0
+  data.tar.gz: 414e249a6a8bd8c9bfb4852381f7d47f6131bdf7
 SHA512:
-  metadata.gz: 3583b0c8a866dab57c14cfeef6e98555d7ceab20779e83d89e7722be0e076607ae13d8d0b2ba0a787866e33131892d608792366504bc12fffce836e1ac8da472
-  data.tar.gz: 0151bda82246b40cb8bc37e62fda203d66b2c95a9dcd1fee9f4bd34688e6d2495ba30c30efe3347d9cc4870316745cfd699c92469bf3a5a63b18cb49f7670c25
+  metadata.gz: d7e410e7eb861576dde400586ee450bf7514ce6202a4c14b5aa01b4c2b9b97339cfd6d06cde5006ca21ab6d62f2d9a616bcb76b7a916904cdaf509fa6582b6ad
+  data.tar.gz: 2bbbc2ff81776a89cef0f4878d6bd3ff6f72c7cc6338d97c96c2b8c71cabda3cfb653fe01cf0abc33a049814e0126f2427da34515ebad156ab7aa82b2ad8e875

data/Gemfile

@@ -6,7 +6,7 @@ gemspec
 group :test do
   gem 'rspec'
   gem 'factory_girl'
-  gem 'simplecov'
+  gem 'simplecov', require: false
   gem 'database_cleaner'
   gem 'pry'
 end

data/lib/mongoid_atomic_votes/atomic_votes.rb

@@ -10,7 +10,7 @@ module Mongoid
       private
       def define_relations(base)
         base.field :vote_count, type: Integer, default: 0
-        base.field :vote_value, type: Float,   default: nil
+        base.field :vote_value, type: Float, default: nil
         base.embeds_many :votes, class_name: 'Mongoid::AtomicVotes::Vote', as: :atomic_voteable
       end
 
@@ -59,21 +59,21 @@ module Mongoid
     private
     def update_votes(mark, retract=false)
       opts = {
-        '$inc' => {vote_count: retract ? -1 : 1},
-        '$set' => {vote_value: self.vote_value}
+        '$inc' => { vote_count: retract ? -1 : 1 },
+        '$set' => { vote_value: self.vote_value }
       }
 
       if retract
-        opts['$pull'] = {votes: {_id: mark.id}}
+        opts['$pull'] = { votes: { _id: mark.id } }
       else
-        opts['$push'] = {votes: mark.as_json}
+        opts['$push'] = { votes: mark.as_json }
       end
 
-      self.collection.find(_id: self.id).update(opts)['ok'] > 0
+      self.collection.find(_id: self.id).update_one(opts).modified_count > 0
     end
 
     def update_vote_value(mark, retract=false)
-      value, vote_count_diff = [mark.value, 1].map{|v| v * (retract ? -1 : 1)}
+      value, vote_count_diff = [mark.value, 1].map { |v| v * (retract ? -1 : 1) }
       self.vote_value = if self.vote_count == 1 && retract
                           nil
                         else
@@ -92,7 +92,7 @@ module Mongoid
 
     def remove_vote_mark(mark)
       _assigning do
-        self.votes.reject!{|v| v.id == mark.id}
+        self.votes.reject! { |v| v.id == mark.id }
         update_vote_value(mark, true)
       end
       update_votes(mark, true)

data/lib/mongoid_atomic_votes/version.rb

@@ -1,5 +1,5 @@
 module Mongoid
   module AtomicVotes
-    VERSION = "0.2.0"
+    VERSION = '0.2.1'
   end
 end

data/lib/mongoid_atomic_votes/vote.rb

@@ -4,14 +4,14 @@ class Mongoid::AtomicVotes::Vote
 
   embedded_in :atomic_voteable, polymorphic: true
 
-  field :value,       type: Integer
+  field :value, type: Integer
   field :voted_by_id, type: BSON::ObjectId
-  field :voter_type,  type: String
+  field :voter_type, type: String
 
-  validates_presence_of  :value, :voted_by_id, :voter_type
-  validates_inclusion_of :value, in: ->(vote){ vote.class.vote_range }, if: ->(vote){ vote.class.vote_range }
+  validates_presence_of :value, :voted_by_id, :voter_type
+  validates_inclusion_of :value, in: ->(vote) { vote.class.vote_range }, if: ->(vote) { vote.class.vote_range }
 
-  index({voted_by_id: 1, mark: 1}, {unique: true, background: true})
+  index({ voted_by_id: 1, mark: 1 }, unique: true, background: true)
 
   class << self
     @@vote_range = nil

data/lib/mongoid_atomic_votes.rb

@@ -1,3 +1,3 @@
-require "mongoid_atomic_votes/version"
-require "mongoid_atomic_votes/atomic_votes"
-require "mongoid_atomic_votes/vote"
+require 'mongoid_atomic_votes/version'
+require 'mongoid_atomic_votes/atomic_votes'
+require 'mongoid_atomic_votes/vote'

data/mongoid_atomic_votes.gemspec

@@ -14,5 +14,5 @@ Gem::Specification.new do |gem|
   gem.require_paths = ["lib"]
   gem.version       = Mongoid::AtomicVotes::VERSION
 
-  gem.add_runtime_dependency 'mongoid', ['>= 4.0']
+  gem.add_runtime_dependency 'mongoid', ['~> 5.0']
 end

data/spec/atomic_votes_spec.rb

@@ -7,7 +7,7 @@ describe Post do
     @post = FactoryGirl.create(:post)
   end
 
-  it "has Mongoid::AtomicVotes module" do
+  it 'has Mongoid::AtomicVotes module' do
     expect(subject.class.ancestors).to include(Mongoid::AtomicVotes)
   end
 
@@ -19,7 +19,7 @@ describe Post do
   it { expect(subject).to respond_to(:votes) }
 
   it { expect(subject.class).to respond_to(:set_vote_range) }
-  it { expect { subject.class.set_vote_range(1) }.to raise_exception }
+  it { expect { subject.class.set_vote_range(1) }.to raise_error('argument should be a Range') }
 
   describe '#votes' do
     it 'is array of votes' do
@@ -71,7 +71,7 @@ describe Post do
       end
 
       vote_value = @post.votes.map(&:value).sum.to_f/@post.votes.size
-      
+
       expect(@post.vote_value).to eq(vote_value)
       expect(@post.vote_count).to eq(@users.size)
       expect(@post.votes.size).to eq(@users.size)

data/spec/spec_helper.rb

@@ -1,17 +1,17 @@
-$LOAD_PATH.unshift(File.dirname(__FILE__))
-$LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
-
 MODELS = File.join(File.dirname(__FILE__), 'models')
 
+require 'rubygems'
+require 'database_cleaner'
+require 'factory_girl'
 require 'simplecov'
 
-SimpleCov.start
+SimpleCov.start do
+  add_filter '/.gems/'
+  add_filter '/.bundle/'
+end
 
-require 'rubygems'
 require 'mongoid'
 require 'mongoid_atomic_votes'
-require 'database_cleaner'
-require 'factory_girl'
 
 Dir["#{MODELS}/*.rb"].each { |f| require f }
 
@@ -19,11 +19,8 @@ Mongoid.configure do |config|
   config.connect_to 'mongoid_atomic_votes_test'
 end
 
-#Mongoid.logger = Logger.new($stdout)
-#Moped.logger = Logger.new($stdout)
-
-#Mongoid.logger.level = Logger::DEBUG
-#Moped.logger.level = Logger::DEBUG
+Mongoid.logger.level = Logger::ERROR
+Mongo::Logger.logger.level = Logger::ERROR
 
 FactoryGirl.definition_file_paths = [File.join(File.dirname(__FILE__), 'factories')]
 FactoryGirl.find_definitions
@@ -42,4 +39,4 @@ RSpec.configure do |config|
   config.after(:each) do
     DatabaseCleaner.clean
   end
-end
+end

metadata

@@ -1,29 +1,29 @@
 --- !ruby/object:Gem::Specification
 name: mongoid_atomic_votes
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.1
 platform: ruby
 authors:
 - Hck
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-07-29 00:00:00.000000000 Z
+date: 2015-11-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mongoid
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '5.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '5.0'
 description: Atomic votes for mongoid models
 email: 
 executables: []
@@ -66,7 +66,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 2.4.5
 signing_key: 
 specification_version: 4
 summary: Atomic votes implementation for mongoid