impressionist 1.4.0 → 1.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 565d11269849faecb7c83d6cda308c6c1e9eba4c
-  data.tar.gz: ed373d6474fee728c3742b0d475ad5d425f9902e
+  metadata.gz: 7c1694b326962e72ed2f57841cb4bc799e1a8e4d
+  data.tar.gz: 35b5e4aa5feff7df8341022ec6b4195d35a7df18
 SHA512:
-  metadata.gz: 8a4160978bf2c001b1c8f2ea24f18bc563d3cd821de5294de0f5b2da8f1c87e8e3a7c2f0c210bd7420b8a7c216383f23ffb61875bac3930475dd163f90f6a5c6
-  data.tar.gz: bc2f41f6556664349216614eec6fdc9d366f40ac0e5fbd5bcbc1eab2fd470897104581f2b19ae31267067273a49a59693edff525a6c6c34596a8c00573932f65
+  metadata.gz: af17886e092d7fd62bbf70c768cb4b804f3087559f12fa2d2d68ee12f3ca6d859bcf7954790957094310ff76ed7376ae25707638779cbe0d894fd00191a5532b
+  data.tar.gz: 828a4f2a95741643e28e1ba943515fa9e715cee6a907f62b1a93950630433f27602b208a49f4d0c597d39818a09db9f1fd041c751080f4546926c7f141141e17

data/impressionist.gemspec

@@ -19,8 +19,12 @@ Gem::Specification.new do |s|
   s.required_rubygems_version = Gem::Requirement.new('>= 1.3.6') if s.respond_to? :required_rubygems_version=
 
   s.add_dependency 'httpclient', '~> 2.2'
-  s.add_dependency 'nokogiri', '~> 1.5'
-  s.add_development_dependency 'capybara'
+
+  # Nokogiri has dropped support for Ruby 1.8.7 onwards version 1.5.10
+  s.add_dependency 'nokogiri', (RUBY_VERSION.match("1.8.7") ? '1.5.10' : '~> 1.6.0')
+
+  # Capybara has dropped support for Ruby 1.8.7 onwards version 2.0.3
+  s.add_development_dependency 'capybara', '>= 2.0.3'
   s.add_development_dependency 'rake', '>= 0.9'
   s.add_development_dependency 'rails', '~> 3.1'
   s.add_development_dependency 'rdoc', '>= 2.4.2'

data/lib/impressionist/counter_cache.rb

@@ -42,8 +42,10 @@ module Impressionist
       end
 
       # Logs to log file, expects a message to be passed
+
       # default mode is ERROR
-      def impressionist_log(mode=:error, str)
+      # ruby 1.8.7 support
+      def impressionist_log(str, mode=:error)
         Rails.logger.send(mode.to_s, str)
       end
 

data/lib/impressionist/models/active_record/impression.rb

@@ -6,7 +6,7 @@
 
 class Impression < ActiveRecord::Base
   include Impressionist::CounterCache
-  include Impressionist::SetUpAssociation 
+  include Impressionist::SetUpAssociation
 
   after_save :impressionable_counter_cache_updatable?
 

data/lib/impressionist/models/active_record/impressionist/impressionable.rb

@@ -2,25 +2,23 @@ ActiveRecord::Base.send(:include, Impressionist::Impressionable)
 
 module Impressionist
   module Impressionable
-
     extend ActiveSupport::Concern
 
     module ClassMethods
+
       def is_impressionable(options={})
         define_association
-        imp_cache_options_set(options)
+
+        @impressionist_cache_options = options
       end
 
+private
       def define_association
         has_many(:impressions,
         :as => :impressionable,
         :dependent => :destroy)
       end
 
-      def imp_cache_options_set(options)
-        @impressionist_cache_options = options
-      end
-
     end
 
   end

data/lib/impressionist/models/mongo_mapper/impressionist/impressionable.rb

@@ -5,10 +5,17 @@ module Impressionist
     extend ActiveSupport::Concern
 
     module ClassMethods
-      def is_impressionable(options={})
-        many :impressions, :as => :impressionable, :dependent => :destroy
-        @cache_options = options[:counter_cache]
-      end
+
+        def is_impressionable(options={})
+          many(:impressions,
+          :as => :impressionable,
+          :dependent => :destroy)
+
+          @impressionist_cache_options = options
+        end
+
     end
+
   end
+
 end

data/lib/impressionist/models/mongoid/impressionist/impressionable.rb

@@ -1,3 +1,5 @@
+# TODO: Refactor this Entity
+# There's a lot of duplication
 Mongoid::Document.send(:include, Impressionist::Impressionable)
 
 module Impressionist

data/lib/impressionist/version.rb

@@ -1,3 +1,3 @@
 module Impressionist
-  VERSION = "1.4.0"
+  VERSION = "1.4.1"
 end

data/test_app/Gemfile

@@ -36,9 +36,9 @@ group :development, :test do
 end
 
 group :test do
-  gem 'capybara'
   gem 'simplecov'
   gem 'systemu'
+  gem 'capybara', '~> 2.0.0'
 end
 
 gem 'jquery-rails'

data/test_app/spec/controllers/articles_controller_spec.rb

@@ -0,0 +1,60 @@
+require 'spec_helper'
+
+describe ArticlesController do
+  fixtures :articles,:impressions,:posts,:widgets
+
+  render_views
+
+  it "should make the impressionable_hash available" do
+    get "index"
+    response.body.include?("false").should eq true
+  end
+
+  it "should log an impression with a message" do
+    get "index"
+    Impression.all.size.should eq 12
+    Article.first.impressions.last.message.should eq "this is a test article impression"
+    Article.first.impressions.last.controller_name.should eq "articles"
+    Article.first.impressions.last.action_name.should eq "index"
+  end
+
+  it "should log an impression without a message" do
+    get "show", :id=> 1
+    Impression.all.size.should eq 12
+    Article.first.impressions.last.message.should eq nil
+    Article.first.impressions.last.controller_name.should eq "articles"
+    Article.first.impressions.last.action_name.should eq "show"
+  end
+
+  it "should log the user_id if user is authenticated (@current_user before_filter method)" do
+    session[:user_id] = 123
+    get "show", :id=> 1
+    Article.first.impressions.last.user_id.should eq 123
+  end
+
+  it "should not log the user_id if user is authenticated" do
+    get "show", :id=> 1
+    Article.first.impressions.last.user_id.should eq nil
+  end
+
+  it "should log the request_hash, ip_address, referrer and session_hash" do
+    get "show", :id=> 1
+    Impression.last.request_hash.size.should eq 64
+    Impression.last.ip_address.should eq "0.0.0.0"
+    Impression.last.session_hash.size.should eq 32
+    Impression.last.referrer.should eq nil
+  end
+
+  # Capybara has change the way it works
+  # We need to pass :type options in order to make include helper methods
+  # see https://github.com/jnicklas/capybara#using-capybara-with-rspec
+  it "should log the referrer when you click a link", :type => :feature do
+    visit article_url(Article.first)
+    click_link "Same Page"
+    Impression.last.referrer.should eq "http://test.host/articles/1"
+  end
+end
+
+
+
+

data/test_app/spec/controllers/dummy_controller_spec.rb

@@ -0,0 +1,11 @@
+require 'spec_helper'
+
+describe DummyController do
+  fixtures :impressions
+  render_views
+
+  it "should log impression at the per action level on non-restful controller" do
+    get "index"
+    Impression.all.size.should eq 12
+  end
+end

data/test_app/spec/controllers/posts_controller_spec.rb

@@ -0,0 +1,19 @@
+require 'spec_helper'
+
+describe PostsController do
+  it "should log impression at the action level" do
+    get "show", :id=> 1
+    Impression.all.size.should eq 12
+    Impression.last.controller_name.should eq "posts"
+    Impression.last.action_name.should eq "show"
+    Impression.last.impressionable_type.should eq "Post"
+    Impression.last.impressionable_id.should eq 1
+  end
+
+  it "should log the user_id if user is authenticated (current_user helper method)" do
+    session[:user_id] = 123
+    get "show", :id=> 1
+    Post.first.impressions.last.user_id.should eq 123
+  end
+
+end

data/test_app/spec/controllers/widgets_controller_spec.rb

@@ -0,0 +1,60 @@
+require 'spec_helper'
+
+describe WidgetsController do
+
+  before(:each) do
+    @widget = Widget.find(1)
+    Widget.stub(:find).and_return(@widget)
+  end
+
+  it "should log impression at the per action level" do
+    get "show", :id=> 1
+    Impression.all.size.should eq 12
+    get "index"
+    Impression.all.size.should eq 13
+    get "new"
+    Impression.all.size.should eq 13
+  end
+
+  it "should not log impression when user-agent is in wildcard list" do
+    request.stub!(:user_agent).and_return('somebot')
+    get "show", :id=> 1
+    Impression.all.size.should eq 11
+  end
+
+  it "should not log impression when user-agent is in the bot list" do
+    request.stub!(:user_agent).and_return('Acoon Robot v1.50.001')
+    get "show", :id=> 1
+    Impression.all.size.should eq 11
+  end
+
+  context "impressionist unique options" do
+
+    it "should log unique impressions at the per action level" do
+      get "show", :id=> 1
+      Impression.all.size.should eq 12
+      get "show", :id=> 2
+      Impression.all.size.should eq 13
+      get "show", :id => 2
+      Impression.all.size.should eq 13
+      get "index"
+      Impression.all.size.should eq 14
+    end
+
+    it "should log unique impressions only once per id" do
+      get "show", :id=> 1
+      Impression.all.size.should eq 12
+
+      get "show", :id=> 2
+      Impression.all.size.should eq 13
+
+      get "show", :id => 2
+      Impression.all.size.should eq 13
+
+      get "index"
+      Impression.all.size.should eq 14
+    end
+
+  end
+
+end

data/test_app/spec/initializers/initializers_spec.rb

@@ -1,14 +1,16 @@
 require 'spec_helper'
 
 describe Impressionist do
+  let(:imp) { RUBY_VERSION.match("1.8") ? "is_impressionable" : :is_impressionable }
+
   it "should be extended from ActiveRecord::Base" do
-    method = RUBY_VERSION.match("1.8") ? "is_impressionable" : :is_impressionable
-    ActiveRecord::Base.methods.include?(method).should be_true
+    expect(ActiveRecord::Base).to respond_to(imp)
+    #ActiveRecord::Base.methods.include?(method).should be_true
   end
 
   it "should include methods in ApplicationController" do
     method = RUBY_VERSION.match("1.8") ? "impressionist" : :impressionist
-    ApplicationController.instance_methods.include?(method).should be_true
+    expect(ApplicationController).to respond_to(method)
   end
 
   it "should include the before_filter method in ApplicationController" do

data/test_app/spec/spec_helper.rb

@@ -5,6 +5,8 @@ unless ENV['CI']
 end
 require File.expand_path("../../config/environment", __FILE__)
 require 'rspec/rails'
+require 'capybara/rails'
+require 'capybara/rspec'
 
 # Requires supporting ruby files with custom matchers and macros, etc,
 # in spec/support/ and its subdirectories.

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: impressionist
 version: !ruby/object:Gem::Version
-  version: 1.4.0
+  version: 1.4.1
 platform: ruby
 authors:
 - johnmcaliley
@@ -30,28 +30,28 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: '1.5'
+        version: 1.6.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: '1.5'
+        version: 1.6.0
 - !ruby/object:Gem::Dependency
   name: capybara
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 2.0.3
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 2.0.3
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -267,8 +267,11 @@ files:
 - test_app/public/stylesheets/.gitkeep
 - test_app/script/cucumber
 - test_app/script/rails
-- test_app/spec/controllers/controller_spec.rb
+- test_app/spec/controllers/articles_controller_spec.rb
+- test_app/spec/controllers/dummy_controller_spec.rb
 - test_app/spec/controllers/impressionist_uniqueness_spec.rb
+- test_app/spec/controllers/posts_controller_spec.rb
+- test_app/spec/controllers/widgets_controller_spec.rb
 - test_app/spec/fixtures/articles.yml
 - test_app/spec/fixtures/impressions.yml
 - test_app/spec/fixtures/posts.yml

data/test_app/spec/controllers/controller_spec.rb

@@ -1,134 +0,0 @@
-require 'spec_helper'
-
-describe ArticlesController do
-  fixtures :articles,:impressions,:posts,:widgets
-  render_views
-
-  it "should make the impressionable_hash available" do
-    get "index"
-    response.body.include?("false").should eq true
-  end
-
-  it "should log an impression with a message" do
-    get "index"
-    Impression.all.size.should eq 12
-    Article.first.impressions.last.message.should eq "this is a test article impression"
-    Article.first.impressions.last.controller_name.should eq "articles"
-    Article.first.impressions.last.action_name.should eq "index"
-  end
-
-  it "should log an impression without a message" do
-    get "show", :id=> 1
-    Impression.all.size.should eq 12
-    Article.first.impressions.last.message.should eq nil
-    Article.first.impressions.last.controller_name.should eq "articles"
-    Article.first.impressions.last.action_name.should eq "show"
-  end
-
-  it "should log the user_id if user is authenticated (@current_user before_filter method)" do
-    session[:user_id] = 123
-    get "show", :id=> 1
-    Article.first.impressions.last.user_id.should eq 123
-  end
-
-  it "should not log the user_id if user is authenticated" do
-    get "show", :id=> 1
-    Article.first.impressions.last.user_id.should eq nil
-  end
-
-  it "should log the request_hash, ip_address, referrer and session_hash" do
-    get "show", :id=> 1
-    Impression.last.request_hash.size.should eq 64
-    Impression.last.ip_address.should eq "0.0.0.0"
-    Impression.last.session_hash.size.should eq 32
-    Impression.last.referrer.should eq nil
-  end
-
-  it "should log the referrer when you click a link" do
-    visit article_url(Article.first)
-    click_link "Same Page"
-    Impression.last.referrer.should eq "http://test.host/articles/1"
-  end
-end
-
-describe PostsController do
-  it "should log impression at the action level" do
-    get "show", :id=> 1
-    Impression.all.size.should eq 12
-    Impression.last.controller_name.should eq "posts"
-    Impression.last.action_name.should eq "show"
-    Impression.last.impressionable_type.should eq "Post"
-    Impression.last.impressionable_id.should eq 1
-  end
-
-  it "should log the user_id if user is authenticated (current_user helper method)" do
-    session[:user_id] = 123
-    get "show", :id=> 1
-    Post.first.impressions.last.user_id.should eq 123
-  end
-end
-
-describe WidgetsController do
-
-  before(:each) do
-    @widget = Widget.find(1)
-    Widget.stub(:find).and_return(@widget)
-  end
-
-  it "should log impression at the per action level" do
-    get "show", :id=> 1
-    Impression.all.size.should eq 12
-    get "index"
-    Impression.all.size.should eq 13
-    get "new"
-    Impression.all.size.should eq 13
-  end
-
-  it "should not log impression when user-agent is in wildcard list" do
-    request.stub!(:user_agent).and_return('somebot')
-    get "show", :id=> 1
-    Impression.all.size.should eq 11
-  end
-
-  it "should not log impression when user-agent is in the bot list" do
-    request.stub!(:user_agent).and_return('Acoon Robot v1.50.001')
-    get "show", :id=> 1
-    Impression.all.size.should eq 11
-  end
-
-  describe "impressionist unique options" do
-
-    it "should log unique impressions at the per action level" do
-      get "show", :id=> 1
-      Impression.all.size.should eq 12
-      get "show", :id=> 2
-      Impression.all.size.should eq 13
-      get "show", :id => 2
-      Impression.all.size.should eq 13
-      get "index"
-      Impression.all.size.should eq 14
-    end
-
-    it "should log unique impressions only once per id" do
-      get "show", :id=> 1
-      Impression.all.size.should eq 12
-      get "show", :id=> 2
-      Impression.all.size.should eq 13
-      get "show", :id => 2
-      Impression.all.size.should eq 13
-      get "index"
-      Impression.all.size.should eq 14
-    end
-
-  end
-
-end
-describe DummyController do
-  fixtures :impressions
-  render_views
-
-  it "should log impression at the per action level on non-restful controller" do
-    get "index"
-    Impression.all.size.should eq 12
-  end
-end