redmine_crm 0.0.8 → 0.0.10

data/redmine_crm.gemspec

@@ -19,3 +19,4 @@ Gem::Specification.new do |spec|
   spec.require_paths = ["lib"]
 
 end
+

data/test/fixtures/issue.rb

@@ -1,8 +1,13 @@
 class Issue < ActiveRecord::Base
   rcrm_acts_as_taggable
+  rcrm_acts_as_viewed
   # default_scope {where(:closed => false)}
   belongs_to :user
   validates_presence_of :description
+
+  def save_without_validation
+    save
+  end
 end
 
 class SpecialIssue < Issue

data/test/fixtures/user.rb

@@ -1,3 +1,7 @@
 class User < ActiveRecord::Base
   has_many :issues
+
+  def anonymous?
+    false
+  end
 end

data/test/fixtures/votable_caches.yml

@@ -0,0 +1,2 @@
+votable_cache:
+  name: 'votiong model with cache'

data/test/fixtures/votables.yml

@@ -0,0 +1,4 @@
+votable:
+  name: 'a voting model'
+votable2:
+  name: 'another voting things'

data/test/fixtures/vote_classes.rb

@@ -0,0 +1,54 @@
+class Voter < ActiveRecord::Base
+  rcrm_acts_as_voter
+end
+
+class NotVoter < ActiveRecord::Base
+  
+end
+
+class Votable < ActiveRecord::Base
+  rcrm_acts_as_votable
+  validates_presence_of :name
+end
+
+class VotableVoter < ActiveRecord::Base
+  rcrm_acts_as_votable
+  rcrm_acts_as_voter
+end
+
+class StiVotable < ActiveRecord::Base
+  rcrm_acts_as_votable
+end
+
+class ChildOfStiVotable < StiVotable
+end
+
+class StiNotVotable < ActiveRecord::Base
+  validates_presence_of :name
+end
+
+class VotableChildOfStiNotVotable < StiNotVotable
+  rcrm_acts_as_votable
+end
+
+class NotVotable < ActiveRecord::Base
+end
+
+class VotableCache < ActiveRecord::Base
+  rcrm_acts_as_votable
+  validates_presence_of :name
+end
+
+class ABoringClass
+  def self.hw
+    'hello world'
+  end
+end
+
+
+# def clean_database
+#   models = [ActsAsVotable::Vote, Voter, NotVoter, Votable, NotVotable, VotableCache]
+#   models.each do |model|
+#     ActiveRecord::Base.connection.execute "DELETE FROM #{model.table_name}"
+#   end
+# end

data/test/fixtures/voters.yml

@@ -0,0 +1,6 @@
+voter:
+  name: 'i can vote!'
+voter2:
+  name: 'a new person'
+voter3:
+  name: 'another person'

data/test/schema.rb

@@ -21,5 +21,79 @@ ActiveRecord::Schema.define :version => 0 do
     t.column "closed", :boolean
     t.column "cached_tag_list", :string
     t.column "user_id", :integer
+    t.column "views", :integer, :default => 0
+  end
+
+  create_table "votes", :force => true do |t|
+    t.references "votable", :polymorphic => true
+    t.references "voter", :polymorphic => true
+
+    t.boolean "vote_flag"
+    t.string "vote_scope"
+    t.integer "vote_weight"
+
+    t.timestamps
+  end
+
+  # add_index :votes, [:votable_id, :votable_type]
+  # add_index :votes, [:voter_id, :voter_type]
+  # add_index :votes, [:voter_id, :voter_type, :vote_scope]
+  # add_index :votes, [:votable_id, :votable_type, :vote_scope]
+
+  create_table :voters, :force => true do |t|
+    t.string :name
+  end
+
+  create_table :not_voters, :force => true do |t|
+    t.string :name
+  end
+
+  create_table :votables, :force => true do |t|
+    t.string :name
+  end
+
+  create_table :votable_voters, :force => true do |t|
+    t.string :name
+  end
+
+  create_table :sti_votables, :force => true do |t|
+    t.string :name
+    t.string :type
+  end
+
+  create_table :sti_not_votables, :force => true do |t|
+    t.string :name
+    t.string :type
+  end
+
+  create_table :not_votables, :force => true do |t|
+    t.string :name
+  end
+
+  create_table :votable_caches, :force => true do |t|
+    t.string :name
+    t.integer :cached_votes_total
+    t.integer :cached_votes_score
+    t.integer :cached_votes_up
+    t.integer :cached_votes_down
+    t.integer :cached_weighted_total
+    t.integer :cached_weighted_score
+    t.float :cached_weighted_average
+
+    t.integer :cached_scoped_test_votes_total
+    t.integer :cached_scoped_test_votes_score
+    t.integer :cached_scoped_test_votes_up
+    t.integer :cached_scoped_test_votes_down
+    t.integer :cached_scoped_weighted_total
+    t.integer :cached_scoped_weighted_score
+    t.float :cached_scoped_weighted_average
+  end
+
+  create_table :viewings, :force => true do |t|
+    t.column :viewer_id,   :integer
+    t.column :viewed_id,   :integer
+    t.column :viewed_type, :string
+    t.column :ip, :string, :limit => '24'
+    t.column :created_at, :datetime
   end
 end

data/test/test_helper.rb

@@ -18,7 +18,9 @@ require File.dirname(__FILE__) + '/../lib/redmine_crm/rcrm_acts_as_taggable'
 require File.dirname(__FILE__) + '/../lib/redmine_crm/tag_list'
 require File.dirname(__FILE__) + '/../lib/redmine_crm/tag'
 require File.dirname(__FILE__) + '/../lib/redmine_crm/tagging'
-require File.dirname(__FILE__) + '/../lib/redmine_crm/tags_helper'
+require File.dirname(__FILE__) + '/../lib/redmine_crm/helpers/tags_helper'
+require File.dirname(__FILE__) + '/../lib/redmine_crm/rcrm_acts_as_votable'
+require File.dirname(__FILE__) + '/../lib/redmine_crm/rcrm_acts_as_voter'
 
 # require_dependency File.dirname(__FILE__) + '/../lib/redmine_crm/tags_helper'
 
@@ -27,8 +29,11 @@ ActiveRecord::Base.configurations = YAML::load(IO.read(File.dirname(__FILE__) +
 ActiveRecord::Base.establish_connection(ENV['DB'] || 'sqlite3')
 
 load(File.dirname(__FILE__) + '/schema.rb')
-load(File.dirname(__FILE__) + '/fixtures/issue.rb')
 load(File.dirname(__FILE__) + '/fixtures/user.rb')
+load(File.dirname(__FILE__) + '/fixtures/issue.rb')
+load(File.dirname(__FILE__) + "/fixtures/vote_classes.rb")
+
+# Dir["/fixtures/*.rb"].each{|file| load file}
 
 class ActiveSupport::TestCase #:nodoc:
   include ActiveRecord::TestFixtures
@@ -40,6 +45,7 @@ class ActiveSupport::TestCase #:nodoc:
   set_fixture_class :tags => RedmineCrm::Tag
   set_fixture_class :taggings => RedmineCrm::Tagging
   
+  set_fixture_class :votable_caches => VotableCache
   fixtures :all
 
   def assert_equivalent(expected, actual, message = nil)

data/test/viewed_test.rb

@@ -0,0 +1,42 @@
+require File.dirname(__FILE__) + '/test_helper'
+
+class ViewedTest < ActiveSupport::TestCase
+  
+  def user
+    user = users(:jonathan)
+  end
+
+  def issue
+    issue = issues(:first_issue)
+  end
+    
+  
+  def test_zero_of_view_count
+    assert_equal issue.view_count, 0
+  end
+    
+  def test_can_view
+    issue.view '127.0.0.1', user
+    assert_equal issue.view_count, 1
+  end
+
+  def test_viewed_by
+    assert !issue.viewed_by?('127.0.0.1', user)
+    issue.view '127.0.0.1', user
+    assert issue.viewed_by?('127.0.0.1', user)
+  end
+
+  def test_twice_view
+    issue.view '127.0.0.1', user
+    issue.view '127.0.0.1', user
+    assert_equal issue.view_count, 1
+  end
+    
+  def test_viewed?
+    assert !issue.viewed?
+    issue.view '127.0.0.1', user
+    assert issue.viewed?
+  end
+
+
+end

data/test/votable_model_test.rb

@@ -0,0 +1,478 @@
+require File.dirname(__FILE__) + '/test_helper'
+
+class VotableModelTest < ActiveSupport::TestCase
+  
+  def votable
+    votables(:votable)
+  end
+
+  def test_vote_without_voter
+    assert !votables(:votable).vote_by
+  end
+
+  def test_have_vote_when_saved
+    votable.vote_by(:voter => voters(:voter), :vote => "yes")
+    assert_equal votable.votes_for.size, 1
+  end
+
+  def test_voted_twice_by_same_person
+    votable.vote_by(:voter => voters(:voter), :vote => "yes")
+    votable.vote_by(:voter => voters(:voter), :vote => "no")
+    assert_equal votable.votes_for.size, 1
+  end
+
+  def test_voted_twice_by_same_person_with_duplicate_params
+    votable.vote_by(:voter => voters(:voter), :vote => "yes")
+    votable.vote_by(:voter => voters(:voter), :vote => "no", :duplicate => true)
+    assert_equal votable.votes_for.size, 2
+  end
+
+  def test_one_scoped_vote
+    votable.vote_by(:voter => voters(:voter), :vote => 'yes', :vote_scope => 'rank')
+    assert_equal votable.find_votes_for(:vote_scope => 'rank').size, 1
+  end
+
+  def test_one_scoped_vote_when_using_scope_by_same_person
+    votable.vote_by(:voter => voters(:voter), :vote => 'yes', :vote_scope => 'rank')
+    votable.vote_by(:voter => voters(:voter), :vote => 'yes', :vote_scope => 'rank')
+    assert_equal votable.find_votes_for(:vote_scope => 'rank').size, 1
+  end
+
+  def test_two_votes_for_when_voting_on_two_diff_scopes
+    votable.vote_by(:voter => voters(:voter), :vote => 'yes', :vote_scope => 'weekly_rank')
+    votable.vote_by(:voter => voters(:voter), :vote => 'yes', :vote_scope => 'monthly_rank')
+    assert_equal votable.votes_for.size, 2
+  end
+
+  def test_called_with_vote_up
+    votables(:votable).vote_up voters(:voter)
+    assert_equal votables(:votable).get_up_votes.first.voter, voters(:voter)
+  end
+
+  def test_called_with_vote_down
+    votables(:votable).vote_down voters(:voter)
+    assert_equal votables(:votable).get_down_votes.first.voter, voters(:voter)
+  end
+
+  def test_have_two_votes_when_voted_two_different_people
+    votable.vote_by(:voter => voters(:voter))
+    votable.vote_by(:voter => voters(:voter2))
+    assert_equal votable.votes_for.size, 2
+  end
+
+  def test_one_true_vote
+    votable.vote_by(:voter => voters(:voter))
+    votable.vote_by(:voter => voters(:voter2), :vote => "dislike")
+    assert_equal votable.get_up_votes.size, 1
+  end
+
+  def test_two_false_votes
+    votable.vote_by(:voter => voters(:voter), :vote => 'no')
+    votable.vote_by(:voter => voters(:voter2), :vote => "dislike")
+    assert_equal votable.get_down_votes.size, 2
+  end
+
+  def test_have_been_voted_on_by_voter2
+    votable.vote_by(:voter => voters(:voter2), :vote => true)
+    assert_equal votable.find_votes_for.first.voter.id, voters(:voter2).id
+  end
+
+  def test_count_the_vote_as_registered_if_this_the_voters_first_vote
+    votable.vote_by(:voter => voters(:voter))
+    assert votable.vote_registered?
+  end
+
+  def test_not_count_the_vote_as_being_registered_if_that_voter_has_alredy_voted_and_voted_has_not_chanded
+    votable.vote_by(:voter => voters(:voter), :vote => true)
+    votable.vote_by(:voter => voters(:voter), :vote => 'yes')
+    assert !votable.vote_registered?
+  end
+
+  def test_count_the_vote_as_registered_if_the_voter_has_voted_and_the_flag_has_changed
+    votable.vote_by(:voter => voters(:voter), :vote => true)
+    votable.vote_by(:voter => voters(:voter), :vote => 'dislike')
+    assert votable.vote_registered?
+  end
+
+  def test_count_the_vote_as_registered_if_the_voter_has_voted_and_vote_weight_has_changed
+    votable.vote_by(:voter => voters(:voter), :vote => true, :vote_weight => 1)
+    votable.vote_by(:voter => voters(:voter), :vote => true, :vote_weight => 2)
+    assert votable.vote_registered?
+  end
+
+  def test_voted_on_by_voter
+    votable.vote_by(:voter => voters(:voter))
+    assert votable.voted_on_by?(voters(:voter))
+  end
+
+  def test_unvoted
+    votable.liked_by(voters(:voter))
+    votable.unliked_by(voters(:voter))
+    assert !votable.voted_on_by?(voters(:voter))
+  end
+
+  def test_unvoted_positive_vote
+    votable.vote_by(:voter => voters(:voter))
+    votable.unvote(:voter => voters(:voter))
+    assert_equal votable.find_votes_for.count, 0
+  end
+
+  def test_set_the_votable_to_unregistered_after_unvoting
+    votable.vote_by(:voter => voters(:voter))
+    votable.unvote(:voter => voters(:voter))
+    assert !votable.vote_registered?
+  end
+
+  def test_unvote_a_negative_vote
+    votable.vote_by(:voter => voters(:voter), :vote => "no")
+    votable.unvote(:voter => voters(:voter))
+    assert_equal votable.find_votes_for.count, 0
+  end
+
+  def test_unvote_only_the_from_a_single_voter
+    votable.vote_by(:voter => voters(:voter))
+    votable.vote_by(:voter => voters(:voter2))
+    votable.unvote(:voter => voters(:voter))
+    assert_equal votable.find_votes_for.count, 1
+  end
+
+  def test_contained_to_instance
+    votable2 = Votable.new(:name => "2nd votable")
+    votable2.save
+
+    votable.vote_by(:voter => voters(:voter), :vote => false)
+    votable2.vote_by(:voter => voters(:voter), :vote => true)
+    votable2.vote_by(:voter => voters(:voter), :vote => true)
+
+    assert votable.vote_registered?
+    assert !votable2.vote_registered?
+  end
+
+  def test_default_weight_if_not_specified
+    votable.upvote_by(voters(:voter))
+    assert_equal votable.find_votes_for.first.vote_weight, 1
+  end
+
+  # with cached votes_for
+
+  def voter
+    voters(:voter)
+  end
+
+  def votable_cache
+    votable_caches(:votable_cache)
+  end
+
+  def test_not_update_cached_votes_for_if_there_are_no_colums
+    votable.vote_by(:voter => voter)
+  end
+
+  def test_update_chaced_votes_for_if_there_is_a_total_column
+    votable_cache.cached_votes_total = 50
+    votable_cache.vote_by(voter: voter)
+    assert_equal votable_cache.cached_votes_total, 1
+  end
+
+  def test_update_cached_total_votes_for_when_a_vote_up_is_removed
+    votable_cache.vote_by(:voter => voter, :vote => 'true')
+    votable_cache.unvote(:voter => voter)
+    assert_equal votable_cache.cached_votes_total, 0
+  end
+
+  def test_update_cachded_score_votes_for_if_there_is_a_score_column
+    votable_cache.cached_votes_score = 50
+    votable_cache.vote_by(:voter => voter)
+    assert_equal votable_cache.cached_votes_score, 1
+    votable_cache.vote_by(:voter => voters(:voter2), :vote => 'false')
+    assert_equal votable_cache.cached_votes_score, 0
+    votable_cache.vote_by(:voter => voter, :vote => 'false')
+    assert_equal votable_cache.cached_votes_score, -2
+  end
+
+  def test_update_cached_score_votef_for_when_a_vote_up_is_removed
+    votable_cache.vote_by(:voter => voter, :vote => 'true')
+    assert_equal votable_cache.cached_votes_score, 1
+    votable_cache.unvote(:voter => voter)
+    assert_equal votable_cache.cached_votes_score, 0
+  end
+
+  def test_update_cached_score_votef_for_when_a_vote_down_is_removed
+    votable_cache.vote_by(:voter => voter, :vote => 'false')
+    assert_equal votable_cache.cached_votes_score, -1
+    votable_cache.unvote(:voter => voter)
+    assert_equal votable_cache.cached_votes_score, 0
+  end
+
+  def test_updata_cached_weighted_total_if_there_is_a_weighted_total_column
+    votable_cache.cached_weighted_total = 50
+    votable_cache.vote_by(:voter => voter)
+    assert_equal votable_cache.cached_weighted_total, 1
+    votable_cache.vote_by(:voter => voters(:voter2), :vote => 'false')
+    assert_equal votable_cache.cached_weighted_total, 2
+  end
+
+  def test_update_cached_weighted_total_votes_for_when_a_vote_up_is_removed
+    votable_cache.vote_by(:voter => voter, :vote => 'true', :vote_weight => 3)
+    assert_equal votable_cache.cached_weighted_total, 3
+    votable_cache.unvote(:voter => voter)
+    assert_equal votable_cache.cached_weighted_total, 0
+  end
+
+  def test_update_cached_weighted_total_votes_for_when_a_vote_down_is_removed
+    votable_cache.vote_by(:voter => voter, :vote => 'false', :vote_weight => 4)
+    assert_equal votable_cache.cached_weighted_total, 4
+    votable_cache.unvote(:voter => voter)
+    assert_equal votable_cache.cached_weighted_total, 0
+  end
+
+  def test_update_cached_weighted_score_if_there_is_a_weighted_score_column
+    votable_cache.cached_weighted_score = 50
+    votable_cache.vote_by(:voter => voter, :vote_weight => 3)
+    assert_equal votable_cache.cached_weighted_score, 3
+    votable_cache.vote_by(:voter => voters(:voter2), :vote => 'false', :vote_weight => 5)
+    assert_equal votable_cache.cached_weighted_score, -2
+    # voter changes her vote from 3 to 5
+    votable_cache.vote_by(:voter => voter, :vote_weight => 5)
+    assert_equal votable_cache.cached_weighted_score, 0
+    votable_cache.vote_by :voter => voters(:voter3), :vote_weight => 4
+    assert_equal votable_cache.cached_weighted_score, 4
+  end
+
+  def test_update_cached_weighted_score_votes_for_when_a_vote_up_is_removed
+    votable_cache.vote_by(:voter => voter, :vote => 'true', :vote_weight => 3)
+    assert_equal votable_cache.cached_weighted_score, 3
+    votable_cache.unvote :voter => voter
+    assert_equal votable_cache.cached_weighted_score, 0
+  end
+
+  def test_update_cached_weighted_score_votes_for_when_a_vote_down_is_removed
+    votable_cache.vote_by(:voter => voter, :vote => 'false', :vote_weight => 4)
+    assert_equal votable_cache.cached_weighted_score, -4
+    votable_cache.unvote :voter => voter
+    assert_equal votable_cache.cached_weighted_score, 0
+  end
+
+  def test_update_cached_weighted_average_if_there_is_a_weighted_average_column
+    votable_cache.cached_weighted_average = 50.0
+    votable_cache.vote_by(:voter => voter, :vote => 'true', :vote_weight => 5)
+    assert_equal votable_cache.cached_weighted_average, 5.0
+    votable_cache.vote_by(:voter => voters(:voter2), :vote => 'true', :vote_weight => 3)
+    assert_equal votable_cache.cached_weighted_average, 4.0
+    # voter changes her vote from 5 to 4
+    votable_cache.vote_by(:voter => voter, :vote => 'true', :vote_weight => 4)
+    assert_equal votable_cache.cached_weighted_average, 3.5
+    votable_cache.vote_by(:voter => voters(:voter3), :vote => 'true', :vote_weight => 5)
+    assert_equal votable_cache.cached_weighted_average, 4.0
+  end
+
+  def test_update_cached_weighted_average_votes_for_when_a_vote_up_is_removed
+    votable_cache.vote_by(:voter => voter, :vote => 'true', :vote_weight => 5)
+    votable_cache.vote_by(:voter => voters(:voter2), :vote => 'true', :vote_weight => 3)
+    assert_equal votable_cache.cached_weighted_average, 4
+    votable_cache.unvote :voter => voter
+    assert_equal votable_cache.cached_weighted_average, 3
+  end
+
+  def test_update_cached_up_votes_for_if_there_is_an_up_vote_column
+    votable_cache.cached_votes_up = 50
+    votable_cache.vote_by(:voter => voter)
+    votable_cache.vote_by(:voter => voter)
+    assert_equal votable_cache.cached_votes_up, 1
+  end
+
+  def test_update_cached_down_votes_for_if_there_is_a_down_vote_column
+    votable_cache.cached_votes_down = 50
+    votable_cache.vote_by(:voter => voter, :vote => 'false')
+    assert_equal votable_cache.cached_votes_down, 1
+  end
+
+  def test_update_cached_up_votes_for_when_a_vote_up_is_removed
+    votable_cache.vote_by(:voter => voter, :vote => 'true')
+    votable_cache.unvote(:voter => voter)
+    assert_equal votable_cache.cached_votes_up, 0
+  end
+
+  def test_update_cached_down_votes_for_when_a_vote_down_is_removed
+    votable_cache.vote_by(:voter => voter, :vote => 'false')
+    votable_cache.unvote(:voter => voter)
+    assert_equal votable_cache.cached_votes_down, 0
+  end
+
+  def test_select_from_cached_total_votes_for_if_there_a_total_column
+    votable_cache.vote_by(:voter => voter)
+    votable_cache.cached_votes_total = 50
+    assert_equal votable_cache.count_votes_total, 50
+  end
+
+  def test_select_from_cached_up_votes_for_if_there_is_an_up_vote_column
+    votable_cache.vote_by(:voter => voter)
+    votable_cache.cached_votes_up = 50
+    assert_equal votable_cache.count_votes_up, 50
+  end
+
+  def test_select_from_cached_down_votes_for_if_there_is_a_down_vote_column
+    votable_cache.vote_by(:voter => voter, :vote => 'false')
+    votable_cache.cached_votes_down = 50
+    assert_equal votable_cache.count_votes_down, 50
+  end
+
+  def test_select_from_cached_weighted_total_if_there_is_a_weighted_total_column
+    votable_cache.vote_by(:voter => voter, :vote => 'false')
+    votable_cache.cached_weighted_total = 50
+    assert_equal votable_cache.weighted_total, 50
+  end
+
+  def test_select_from_cached_weighted_score_if_there_is_a_weighted_score_column
+    votable_cache.vote_by( :voter => voter, :vote => 'false')
+    votable_cache.cached_weighted_score = 50
+    assert_equal votable_cache.weighted_score, 50
+  end
+
+  def test_select_from_cached_weighted_average_if_there_is_a_weighted_average_column
+    votable_cache.vote_by( :voter => voter, :vote => 'false')
+    votable_cache.cached_weighted_average = 50
+    assert_equal votable_cache.weighted_average, 50
+  end
+
+  def test_update_cached_total_votes_for_when_voting_under_an_scope
+    votable_cache.vote_by( :voter => voter, :vote => 'true', :vote_scope => 'rank')
+    assert_equal votable_cache.cached_votes_total, 1
+  end
+
+  def test_update_cached_up_votes_for_when_voting_under_an_scope
+    votable_cache.vote_by( :voter => voter, :vote => 'true', :vote_scope => 'rank')
+    assert_equal votable_cache.cached_votes_up, 1
+  end
+
+  def test_update_cached_total_votes_for_when_a_scoped_vote_down_is_removed
+    votable_cache.vote_by(:voter => voter, :vote => 'true', :vote_scope => 'rank')
+    votable_cache.unvote( :voter => voter, :vote_scope => 'rank')
+    assert_equal votable_cache.cached_votes_total, 0
+  end
+
+  def test_update_cached_up_votes_for_when_a_scoped_vote_down_is_removed
+    votable_cache.vote_by(:voter => voter, :vote => 'true', :vote_scope => 'rank')
+    votable_cache.unvote(:voter => voter, :vote_scope => 'rank')
+    assert_equal votable_cache.cached_votes_up, 0
+  end
+
+  def test_update_cached_down_votes_for_when_downvoting_under_a_scope
+    votable_cache.vote_by(:voter => voter, :vote => 'false', :vote_scope => 'rank')
+    assert_equal votable_cache.cached_votes_down, 1
+  end
+
+  def test_update_cached_down_votes_for_when_a_scoped_vote_down_is_removed
+    votable_cache.vote_by(:voter => voter, :vote => 'false', :vote_scope => 'rank')
+    votable_cache.unvote(:voter => voter, :vote_scope => 'rank')
+    assert_equal votable_cache.cached_votes_down, 0
+  end
+
+  # describe "with scoped cached votes_for
+
+  def test_update_cached_total_votes_for_if_there_is_a_total_column
+    votable_cache.cached_scoped_test_votes_total = 50
+    votable_cache.vote_by(:voter => voter, :vote_scope => "test")
+    assert_equal votable_cache.cached_scoped_test_votes_total, 1
+  end
+
+  def test_update_cached_total_votes_for_when_a_vote_up_is_removed
+    votable_cache.vote_by(:voter => voter, :vote => 'true', :vote_scope => "test")
+    votable_cache.unvote(:voter => voter, :vote_scope => "test")
+    assert_equal votable_cache.cached_scoped_test_votes_total, 0
+  end
+
+  def test_update_cached_total_votes_for_when_a_vote_down_is_removed
+    votable_cache.vote_by(:voter => voter, :vote => 'false', :vote_scope => "test")
+    votable_cache.unvote(:voter => voter, :vote_scope => "test")
+    assert_equal votable_cache.cached_scoped_test_votes_total, 0
+  end
+
+  def test_update_cached_score_votes_for_if_there_is_a_score_column
+    votable_cache.cached_scoped_test_votes_score = 50
+    votable_cache.vote_by(:voter => voter, :vote_scope => "test")
+    assert_equal votable_cache.cached_scoped_test_votes_score, 1
+    votable_cache.vote_by(:voter => voters(:voter2), :vote => 'false', :vote_scope => "test")
+    assert_equal votable_cache.cached_scoped_test_votes_score, 0
+    votable_cache.vote_by(:voter => voter, :vote => 'false', :vote_scope => "test")
+    assert_equal votable_cache.cached_scoped_test_votes_score, -2
+  end
+
+  def test_update_cached_score_votes_for_when_a_vote_up_is_removed
+    votable_cache.vote_by(:voter => voter, :vote => 'true', :vote_scope => "test")
+    assert_equal votable_cache.cached_scoped_test_votes_score, 1
+    votable_cache.unvote(:voter => voter, :vote_scope => "test")
+    assert_equal votable_cache.cached_scoped_test_votes_score, 0
+  end
+
+  def test_update_cached_score_votes_for_when_a_vote_down_is_removed
+    votable_cache.vote_by(:voter => voter, :vote => 'false', :vote_scope => "test")
+    assert_equal votable_cache.cached_scoped_test_votes_score, -1
+    votable_cache.unvote(:voter => voter, :vote_scope => "test")
+    assert_equal votable_cache.cached_scoped_test_votes_score, 0
+  end
+
+  def test_update_cached_up_votes_for_if_there_is_an_up_vote_column
+    votable_cache.cached_scoped_test_votes_up = 50
+    votable_cache.vote_by(:voter => voter, :vote_scope => "test")
+    votable_cache.vote_by(:voter => voter, :vote_scope => "test")
+    assert_equal votable_cache.cached_scoped_test_votes_up, 1
+  end
+
+  def test_update_cached_down_votes_for_if_there_is_a_down_vote_column
+    votable_cache.cached_scoped_test_votes_down = 50
+    votable_cache.vote_by(:voter => voter, :vote => 'false', :vote_scope => "test")
+    assert_equal votable_cache.cached_scoped_test_votes_down, 1
+  end
+
+  def test_update_cached_up_votes_for_when_a_vote_up_is_removed
+    votable_cache.vote_by :voter => voter, :vote => 'true', :vote_scope => "test"
+    votable_cache.unvote :voter => voter, :vote_scope => "test"
+    assert_equal votable_cache.cached_scoped_test_votes_up, 0
+  end
+
+  def test_update_cached_down_votes_for_when_a_vote_down_is_removed
+    votable_cache.vote_by(:voter => voter, :vote => 'false', :vote_scope => "test")
+    votable_cache.unvote(:voter => voter, :vote_scope => "test")
+    assert_equal votable_cache.cached_scoped_test_votes_down, 0
+  end
+
+  def test_select_from_cached_total_votes_for_if_there_a_total_column
+    votable_cache.vote_by(:voter => voter, :vote_scope => "test")
+    votable_cache.cached_scoped_test_votes_total = 50
+    assert_equal votable_cache.count_votes_total(false, "test"), 50
+  end
+
+  def test_select_from_cached_up_votes_for_if_there_is_an_up_vote_column
+    votable_cache.vote_by(:voter => voter, :vote_scope => "test")
+    votable_cache.cached_scoped_test_votes_up = 50
+    assert_equal votable_cache.count_votes_up(false, "test"), 50
+  end
+
+  def test_select_from_cached_down_votes_for_if_there_is_a_down_vote_column
+    votable_cache.vote_by(:voter => voter, :vote => 'false', :vote_scope => "test")
+    votable_cache.cached_scoped_test_votes_down = 50
+    assert_equal votable_cache.count_votes_down(false, "test"), 50
+  end
+
+  # describe "sti models
+
+  def test_be_able_to_vote_on_a_votable_child_of_a_non_votable_sti_model
+    votable = VotableChildOfStiNotVotable.create(:name => 'sti child')
+
+    votable.vote_by(:voter => voter, :vote => 'yes')
+    assert_equal votable.votes_for.size, 1
+  end
+
+  def test_not_be_able_to_vote_on_a_parent_non_votable
+    assert !StiNotVotable.votable?
+  end
+
+  def test_be_able_to_vote_on_a_child_when_its_parent_is_votable
+    votable = ChildOfStiVotable.create(:name => 'sti child')
+
+    votable.vote_by(:voter => voter, :vote => 'yes')
+    assert_equal votable.votes_for.size, 1
+  end
+
+
+end