bullet 2.0.0.rc1 → 2.0.0.rc2

data/spec/bullet/counter_spec.rb

@@ -1,136 +0,0 @@
-require File.dirname(__FILE__) + '/../spec_helper'
-
-ActiveRecord::Base.establish_connection(:adapter => 'sqlite3', :database => ':memory:')
-
-describe Bullet::Detector::Counter do
-  def setup_db
-    ActiveRecord::Schema.define(:version => 1) do
-      create_table :countries do |t|
-        t.string :name
-      end
-
-      create_table :cities do |t|
-        t.string :name
-        t.integer :country_id
-      end
-    end
-  end
-
-  def teardown_db
-    ActiveRecord::Base.connection.tables.each do |table|
-      ActiveRecord::Base.connection.drop_table(table)
-    end
-  end
-
-  class Country < ActiveRecord::Base
-    has_many :cities
-  end
-
-  class City < ActiveRecord::Base
-    belongs_to :country
-  end
-
-  before(:all) do
-    setup_db
-
-    country1 = Country.create(:name => 'first')
-    country2 = Country.create(:name => 'second')
-    
-    country1.cities.create(:name => 'first')
-    country1.cities.create(:name => 'second')
-    country2.cities.create(:name => 'third')
-    country2.cities.create(:name => 'fourth')
-  end
-
-  after(:all) do
-    teardown_db
-  end
-
-  before(:each) do
-    Bullet.start_request
-  end
-  
-  after(:each) do
-    Bullet.end_request
-  end
-
-  it "should need counter cache with all cities" do
-    Country.all.each do |country|
-      country.cities.size
-    end
-    Bullet.collected_counter_cache_notifications.should_not be_empty
-  end
-  
-  it "should not need coounter cache with only one object" do
-    Country.first.cities.size
-    Bullet.collected_counter_cache_notifications.should be_empty
-  end
-
-  it "should not need counter cache with part of cities" do
-    Country.all.each do |country|
-      country.cities.where(:name => 'first').size
-    end
-    Bullet.collected_counter_cache_notifications.should be_empty
-  end
-end
-
-describe Bullet::Detector::Counter do
-  def setup_db
-    ActiveRecord::Schema.define(:version => 1) do
-      create_table :people do |t|
-        t.string :name
-        t.integer :pets_count
-      end
-
-      create_table :pets do |t|
-        t.string :name
-        t.integer :person_id
-      end
-    end
-  end
-
-  def teardown_db
-    ActiveRecord::Base.connection.tables.each do |table|
-      ActiveRecord::Base.connection.drop_table(table)
-    end
-  end
-
-  class Person < ActiveRecord::Base
-    has_many :pets
-  end
-
-  class Pet < ActiveRecord::Base
-    belongs_to :person, :counter_cache => true
-  end
-
-  before(:all) do
-    setup_db
-
-    person1 = Person.create(:name => 'first')
-    person2 = Person.create(:name => 'second')
-    
-    person1.pets.create(:name => 'first')
-    person1.pets.create(:name => 'second')
-    person2.pets.create(:name => 'third')
-    person2.pets.create(:name => 'fourth')
-  end
-
-  after(:all) do
-    teardown_db
-  end
-
-  before(:each) do
-    Bullet.start_request
-  end
-  
-  after(:each) do
-    Bullet.end_request
-  end
-
-  it "should not need counter cache" do
-    Person.all.each do |person|
-      person.pets.size
-    end
-    Bullet.collected_counter_cache_notifications.should be_empty
-  end
-end

data/spec/spec_helper.rb

@@ -1,79 +0,0 @@
-require 'rubygems'
-require 'rspec'
-require 'rspec/autorun'
-require 'rails'
-require 'active_record'
-require 'action_controller'
-
-module Rails
-  class <<self
-    def root
-      File.expand_path(__FILE__).split('/')[0..-3].join('/')
-    end
-  end
-end
-
-$LOAD_PATH.unshift(File.expand_path(File.dirname(__FILE__) + "/../lib"))
-require 'bullet'
-Bullet.enable = true
-ActiveRecord::Migration.verbose = false
-
-module Bullet
-  def self.collected_notifications_of_class( notification_class )
-    Bullet.notification_collector.collection.select do |notification| 
-      notification.is_a? notification_class 
-    end
-  end
-
-  def self.collected_counter_cache_notifications
-    collected_notifications_of_class Bullet::Notification::CounterCache
-  end
-
-  def self.collected_n_plus_one_query_notifications
-    collected_notifications_of_class Bullet::Notification::NPlusOneQuery
-  end
-
-  def self.collected_unused_eager_association_notifications
-    collected_notifications_of_class Bullet::Notification::UnusedEagerLoading
-  end
-end
-
-module Bullet
-  module Detector
-    class Association
-      class <<self
-        # returns true if all associations are preloaded
-        def completely_preloading_associations?
-          Bullet.collected_n_plus_one_query_notifications.empty?
-        end
-
-        def has_unused_preload_associations?
-          Bullet.collected_unused_eager_association_notifications.present?
-        end
-
-        # returns true if a given object has a specific association
-        def creating_object_association_for?(object, association)
-          object_associations[object].present? && object_associations[object].include?(association)
-        end
-
-        # returns true if a given class includes the specific unpreloaded association
-        def detecting_unpreloaded_association_for?(klass, association)
-          for_class_and_assoc = Bullet.collected_n_plus_one_query_notifications.select do |notification|
-            notification.base_class == klass and 
-            notification.associations.include?( association )
-          end
-          for_class_and_assoc.present?
-        end
-
-        # returns true if the given class includes the specific unused preloaded association
-        def unused_preload_associations_for?(klazz, association)
-          for_class_and_assoc = Bullet.collected_unused_eager_association_notifications.select do |notification|
-            notification.base_class == klass and 
-            notification.associations.include?( association )
-          end
-          for_class_and_assoc.present?
-        end
-      end
-    end
-  end
-end

data/tasks/bullet_tasks.rake

@@ -1,9 +0,0 @@
-namespace :bullet do
-  namespace :log do
-    desc "Truncates the bullet log file to zero bytes"
-    task :clear do
-      f = File.open("log/bullet.log", "w")
-      f.close
-    end
-  end
-end