virility 0.1.4 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 192acfdf2d757a8fd21934d481e5b76879bce823
-  data.tar.gz: 71f7a4ff20a49e39d5228132ed10513485afff84
+  metadata.gz: 554a235ea250af95894241a6e9355737a26bee66
+  data.tar.gz: c3a69e3f7d9b7e54ee8eac49e49eb1e488c29ab1
 SHA512:
-  metadata.gz: 86ac8277dcb48e7559f035b24e3d736e3c37c53fbd70dfcd17cf62452de1fb020f5e37a83824881cab6462f428d8140f9dce3ec3c331e1671dd94f53f9be9b8d
-  data.tar.gz: 0fe93108823ec634512bb5810524fa8a8a384a7c3df89bb38efc72959958abefccaa77ef96c5925f20484df0224612adab85129e6edcbe740f946b302556d977
+  metadata.gz: cead613eadceaec0a0f9d2535e9ab44c585e96a17a6906b22c911b20625494db85a4b80b87e9515392eda63b39d04c801ad096fb816c675ee78fbbbdd8f29beb
+  data.tar.gz: 07c0df627f75818bce04941e27fdb88d754a0b66c36e3dd917da84028d705650911ad4ff2d3df4b40e24c15e4b881712fad5dba375d28f58d743c6e303d6afc7

data/.ruby-version

@@ -1 +1 @@
-ruby-2.0.0-p451
+ruby-2.2.4

data/README.md

@@ -2,6 +2,8 @@
 
 Virility calls upon the API's of many popular social services such as Facebook, Twitter and Pinterest to collect the number of likes, tweets, pins etc. of a particular URL.  Written with a modular construction, Virility makes it easy to drop new data collection strategies into the framework so that you can collect all of your statistics in one easy location.
 
+View a demo online: http://virility.herokuapp.com/
+
 ## Installation
 
 Add this line to your application's Gemfile:
@@ -39,7 +41,6 @@ The Virility::Excitation object does the heavy lifting of collecting the data fr
 Currently there is support for the following social resources:
 * Facebook
 * Twitter
-* Delicious
 * Pinterest
 * Google Plus One
 * Stumble Upon
@@ -131,7 +132,7 @@ On this particular day, there was a 5,495 count difference between the two value
     Virility::Excitation.new("http://www.ruby-lang.org/en/").counts
     # => {:delicious=>4314, :facebook=>813, :pinterest=>22, :plusone=>406, :stumbleupon=>246937, :twitter=>698}
 
-Stumbleupon and Twitter are consistent while Delicious, Facebook, Pinterest and Google Plus One return different results. Depending on your needs, you could craft an algorithm that takes all of this into account and attempts to deliver an accurate number by combining the data sets that are different and trusting the ones that are the same.
+Stumbleupon and Twitter are consistent while Facebook, Pinterest and Google Plus One return different results. Depending on your needs, you could craft an algorithm that takes all of this into account and attempts to deliver an accurate number by combining the data sets that are different and trusting the ones that are the same.
 
 Based on this logic, it is possible to consider that the true total share count is closer to _253,250_. Not only is this an opinionated number, it's accuracy is questionable based on assumptions, however if you are just trying to get a ballpark feeling of the virility of your content, this number should suffice.
 
@@ -146,4 +147,4 @@ Based on this logic, it is possible to consider that the true total share count
 ## Copyright
 
 Copyright (c) 2012 Jay Sanders. See LICENSE.txt for
-further details.
+further details.

data/lib/virility/strategy.rb

@@ -3,13 +3,14 @@ module Virility
     include HTTParty
     include Virility::Supporter
 
-    attr_accessor :url, :response, :results
+    attr_accessor :url, :response, :results, :original_url
 
     def initialize url
-      @url = encode url
+      @original_url = url
+      @url = encode(url)
       @results = {}
     end
-    
+
     #
     # Abstract Methods - Delete eventually
     #
@@ -21,20 +22,20 @@ module Virility
     def count
       raise "Abstract Method count called on #{self.class} - Please define this method"
     end
-    
+
     #
     # Poll
     #
-    
+
     def poll
       call_strategy
       collect_results
     end
-    
+
     #
     # Call Strategy
     #
-    
+
     def call_strategy
       @response = census
     end
@@ -42,7 +43,7 @@ module Virility
     #
     # Results
     #
-    
+
     def collect_results
       if respond_to?(:outcome)
         @results = valid_response_test ? outcome : {}

data/lib/virility/version.rb

@@ -1,3 +1,3 @@
 module Virility
-  VERSION = "0.1.4"
+  VERSION = "0.2.0"
 end

data/spec/excitation_spec.rb

@@ -11,7 +11,7 @@ describe "Excitation" do
 
   context "initialization" do
     it "should raise an error if a URL is not set" do
-      lambda {Virility::Excitation.new}.should raise_error
+      expect{Virility::Excitation.new}.to raise_error(ArgumentError, "wrong number of arguments (0 for 1)")
     end
   end
 
@@ -19,11 +19,11 @@ describe "Excitation" do
   # Get Virility
   #
 
-#   context "poll" do
-#     it "should not raise an error" do
-#       lambda {Virility::Excitation.new(@url).poll}.should_not raise_error
-#     end
-#   end
+  context "poll" do
+    it "should not raise an error" do
+      expect{Virility::Excitation.new(@url).poll}.not_to raise_error
+    end
+  end
 
   #
   # Collect Strategies
@@ -31,15 +31,15 @@ describe "Excitation" do
 
   context "collect_strategies" do
     it "should assign a hash to the strategies variable" do
-      Virility::Excitation.new(@url).strategies.should be_a_kind_of Hash
+      expect(Virility::Excitation.new(@url).strategies).to be_a_kind_of Hash
     end
 
     it "strategies should be inherited from the Strategy" do
-      Virility::Excitation.new(@url).strategies.first.last.should be_a_kind_of Virility::Strategy
+      expect(Virility::Excitation.new(@url).strategies.first.last).to be_a_kind_of Virility::Strategy
     end
 
     it "should load all of the strategies" do
-      Virility::Excitation.new(@url).strategies.count.should == Dir[File.join('lib', 'virility', 'strategies', '**', '*')].count { |file| File.file?(file) }
+      expect(Virility::Excitation.new(@url).strategies.count).to eq(Dir[File.join('lib', 'virility', 'strategies', '**', '*')].count { |file| File.file?(file) })
     end
   end
 
@@ -50,7 +50,7 @@ describe "Excitation" do
   context "encode" do
     it "should encode the url" do
       v = Virility::Excitation.new(@url)
-      v.encode(@url).should == "http%3A%2F%2Fcreativeallies.com"
+      expect(v.encode(@url)).to eq("http%3A%2F%2Fcreativeallies.com")
     end
   end
 
@@ -60,7 +60,7 @@ describe "Excitation" do
 
   context "symbolize_for_key" do
     it "should return a symbol with the name of the class" do
-      Virility::Excitation.new(@url).symbolize_for_key(Virility::Excitation.new(@url)).should == :excitation
+      expect(Virility::Excitation.new(@url).symbolize_for_key(Virility::Excitation.new(@url))).to eq(:excitation)
     end
   end
 
@@ -72,13 +72,13 @@ describe "Excitation" do
     context "overall testing" do
       Virility::TESTING_STRATEGIES.each do |method, klass|
         it "should return a #{klass} object when the method #{method} is called" do
-          Virility::Excitation.new(@url).send(method).should be_a_kind_of klass
+          expect(Virility::Excitation.new(@url).send(method)).to be_a_kind_of(klass)
         end
       end
 
       Virility::FAKE_TESTING_STRATEGIES.each do |method|
         it "should raise an error if the strategy (#{method}) does not exist" do
-          lambda { Virility::Excitation.new(@url).send(method) }.should raise_error(Virility::UnknownStrategy, "#{method} Is Not A Known Strategy")
+          expect{ Virility::Excitation.new(@url).send(method) }.to raise_error(Virility::UnknownStrategy, "#{method} Is Not A Known Strategy")
         end
       end
     end
@@ -86,13 +86,13 @@ describe "Excitation" do
     context "strategy_exists?" do
       Virility::TESTING_STRATEGIES.keys.each do |strategy|
         it "should return true for #{strategy}" do
-          Virility::Excitation.new(@url).strategy_exists?(strategy).should be true
+          expect(Virility::Excitation.new(@url).strategy_exists?(strategy)).to be(true)
         end
       end
 
       Virility::FAKE_TESTING_STRATEGIES.each do |strategy|
         it "should return false for #{strategy}" do
-          Virility::Excitation.new(@url).strategy_exists?(strategy).should be false
+          expect(Virility::Excitation.new(@url).strategy_exists?(strategy)).to be(false)
         end
       end
     end
@@ -100,17 +100,15 @@ describe "Excitation" do
     context "get_strategy" do
       Virility::TESTING_STRATEGIES.each do |method, klass|
         it "should return a #{klass} object when get_strategy is called with #{method}" do
-          Virility::Excitation.new(@url).get_strategy(method).should be_a_kind_of klass
+          expect(Virility::Excitation.new(@url).get_strategy(method)).to be_a_kind_of(klass)
         end
       end
 
       Virility::FAKE_TESTING_STRATEGIES.each do |method|
         it "should raise an error if the strategy (#{method}) does not exist" do
-          lambda { Virility::Excitation.new(@url).get_strategy(method) }.should raise_error(Virility::UnknownStrategy, "#{method} Is Not A Known Strategy")
+          expect(lambda { Virility::Excitation.new(@url).get_strategy(method) }).to raise_error(Virility::UnknownStrategy, "#{method} Is Not A Known Strategy")
         end
       end
     end
-
   end
-
 end

data/spec/spec_helper.rb

@@ -1,6 +1,7 @@
 $LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
 $LOAD_PATH.unshift(File.dirname(__FILE__))
 require 'rspec'
+require 'pry'
 require 'virility'
 
 # Requires supporting files with custom matchers and macros, etc,
@@ -10,7 +11,7 @@ Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each {|f| require f}
 RSpec.configure do |config|
   config.filter_run :focus => true
   config.run_all_when_everything_filtered = true
-  config.mock_with :rspec 
+  config.mock_with :rspec
 end
 
 #
@@ -18,24 +19,29 @@ end
 #
 
 module Virility
-  TESTING_STRATEGIES = {:facebook => Virility::Facebook, :twitter => Virility::Twitter, :delicious => Virility::Delicious, :pinterest => Virility::Pinterest, :plus_one => Virility::PlusOne, :stumble_upon => Virility::StumbleUpon}
+  TESTING_STRATEGIES = {
+    :facebook => Virility::Facebook,
+    :twitter => Virility::Twitter,
+    :pinterest => Virility::Pinterest,
+    :plus_one => Virility::PlusOne,
+    :stumble_upon => Virility::StumbleUpon
+  }
   FAKE_TESTING_STRATEGIES = [:digg, :reddit, :linked_in, :instagram, :tumblr]
-  
+
   FB_RESULTS = {"like_count"=>"19", "click_count"=>"0", "share_count"=>"3", "comment_count"=>"0", "commentsbox_count"=>"0", "total_count"=>"22"}
   FAKE_FB_RESULTS = [:face_count, :pages, :friends]
 end
 
-
 #
 # Example Groups
 #
 
-share_examples_for "no context results" do
+RSpec.shared_examples "no context results" do
   it "should not raise an error" do
-    lambda { @virility.poll }.should_not raise_error
+    expect{ @virility.poll }.not_to raise_error
   end
 
   it "should return 0 for count" do
-    @virility.count.should == 0
+    expect(@virility.count).to eq(0)
   end
 end

data/spec/strategies/facebook_spec.rb

@@ -4,24 +4,24 @@ describe "Virility::Facebook" do
   before(:each) do
     @url = "http://creativeallies.com"
   end
-  
-  share_examples_for "no facebook results" do
+
+  RSpec.shared_examples "no facebook results" do
     it "should not raise an error" do
-      lambda { @virility.poll }.should_not raise_error
+      expect{ @virility.poll }.not_to raise_error
     end
 
     ["like_count", "click_count", "share_count", "comment_count", "commentsbox_count", "total_count"].each do |attribute|
       it "should return 0 for #{attribute}" do
-        @virility.send(attribute.to_sym).should == 0
+        expect(@virility.send(attribute.to_sym)).to eq(0)
       end
-    end  
+    end
   end
 
   describe "poll" do
     context "when there is not a valid result" do
       before(:each) do
         response = double("HTTParty::Response", :parsed_response => {"fql_query_response"=>{"list"=>"true"}})
-        Virility::Facebook.stub(:get).and_return(response)
+        allow(Virility::Facebook).to receive(:get) { response }
         @virility = Virility::Facebook.new(@url)
       end
 
@@ -31,7 +31,7 @@ describe "Virility::Facebook" do
     context "when there is no result" do
       before(:each) do
         response = double("HTTParty::Response")
-        Virility::Facebook.stub(:get).and_return(response)
+        allow(Virility::Facebook).to receive(:get) { response }
         @virility = Virility::Facebook.new(@url)
       end
 
@@ -41,7 +41,7 @@ describe "Virility::Facebook" do
     context "when there is a result but no fql_query_response" do
       before(:each) do
         response = double("HTTParty::Response", :parsed_response => {})
-        Virility::Facebook.stub(:get).and_return(response)
+        allow(Virility::Facebook).to receive(:get) { response }
         @virility = Virility::Facebook.new(@url)
       end
 
@@ -51,7 +51,7 @@ describe "Virility::Facebook" do
     context "when there is a result but parsed_response is weird" do
       before(:each) do
         response = double("HTTParty::Response", :parsed_response => Object.new)
-        Virility::Facebook.stub(:get).and_return(response)
+        allow(Virility::Facebook).to receive(:get) { response }
         @virility = Virility::Facebook.new(@url)
       end
 
@@ -61,17 +61,17 @@ describe "Virility::Facebook" do
     context "when there is a valid result" do
       before(:each) do
         response = double("HTTParty::Response", :parsed_response => {"fql_query_response"=>{"list"=>"true", "link_stat"=>{"like_count"=>"977662", "click_count"=>"265614", "share_count"=>"3020040", "comment_count"=>"1118601", "commentsbox_count"=>"0", "total_count"=>"5116303"}}})
-        Virility::Facebook.stub(:get).and_return(response)
+        allow(Virility::Facebook).to receive(:get) { response }
         @virility = Virility::Facebook.new(@url)
       end
 
       it "should not raise an error" do
-        lambda { @virility.poll }.should_not raise_error
+        expect{ @virility.poll }.not_to raise_error
       end
 
       {"like_count"=>"977662", "click_count"=>"265614", "share_count"=>"3020040", "comment_count"=>"1118601", "commentsbox_count"=>"0", "total_count"=>"5116303"}.each do |key, value|
         it "should return #{value} for #{key}" do
-          @virility.send(key.to_sym).should == value
+          expect(@virility.send(key.to_sym)).to eq(value)
         end
       end
     end
@@ -79,17 +79,17 @@ describe "Virility::Facebook" do
     context "when there is a valid result, but not all fields are present" do
       before(:each) do
         response = double("HTTParty::Response", :parsed_response => {"fql_query_response"=>{"list"=>"true", "link_stat"=>{"like_count"=>"977662", "comment_count"=>"1118601", "commentsbox_count"=>"0", "total_count"=>"5116303"}}})
-        Virility::Facebook.stub(:get).and_return(response)
+        allow(Virility::Facebook).to receive(:get) { response }
         @virility = Virility::Facebook.new(@url)
       end
 
       it "should not raise an error" do
-        lambda { @virility.poll }.should_not raise_error
+        expect{ @virility.poll }.not_to raise_error
       end
 
       {"like_count"=>"977662", "click_count"=>0, "share_count"=>0, "comment_count"=>"1118601", "commentsbox_count"=>"0", "total_count"=>"5116303"}.each do |key, value|
         it "should return #{value} for #{key}" do
-          @virility.send(key.to_sym).should == value
+          expect(@virility.send(key.to_sym)).to eq(value)
         end
       end
     end

data/spec/strategies/pinterest_spec.rb

@@ -4,12 +4,12 @@ describe "Virility::Pinterest" do
   before(:each) do
     @url = "http://creativeallies.com"
   end
-  
+
   describe "poll" do
     context "when there is not a valid result" do
       before(:each) do
         response = double("HTTParty::Response", :parsed_response => {"fake_return_value"=> "OICU812"})
-        Virility::Pinterest.stub(:get).and_return(response)
+        allow(Virility::Pinterest).to receive(:get) { response }
         @virility = Virility::Pinterest.new(@url)
       end
 
@@ -19,7 +19,7 @@ describe "Virility::Pinterest" do
     context "when there is no result" do
       before(:each) do
         response = double("HTTParty::Response")
-        Virility::Pinterest.stub(:get).and_return(response)
+        allow(Virility::Pinterest).to receive(:get) { response }
         @virility = Virility::Pinterest.new(@url)
       end
 
@@ -29,7 +29,7 @@ describe "Virility::Pinterest" do
     context "when there is a result but no specific hash value" do
       before(:each) do
         response = double("HTTParty::Response", :parsed_response => {})
-        Virility::Pinterest.stub(:get).and_return(response)
+        allow(Virility::Pinterest).to receive(:get) { response }
         @virility = Virility::Pinterest.new(@url)
       end
 
@@ -39,7 +39,7 @@ describe "Virility::Pinterest" do
     context "when there is a result but parsed_response is weird" do
       before(:each) do
         response = double("HTTParty::Response", :parsed_response => Object.new)
-        Virility::Pinterest.stub(:get).and_return(response)
+        allow(Virility::Pinterest).to receive(:get) { response }
         @virility = Virility::Pinterest.new(@url)
       end
 
@@ -49,16 +49,16 @@ describe "Virility::Pinterest" do
     context "when there is a valid result" do
       before(:each) do
         response = double("HTTParty::Response", :parsed_response => {"count"=>1, "url"=>"http://creativeallies.com"})
-        Virility::Pinterest.stub(:get).and_return(response)
+        allow(Virility::Pinterest).to receive(:get) { response }
         @virility = Virility::Pinterest.new(@url)
       end
 
       it "should not raise an error" do
-        lambda { @virility.poll }.should_not raise_error
+        expect{ @virility.poll }.not_to raise_error
       end
 
       it "should return 1 for the count" do
-        @virility.count.should == 1
+        expect(@virility.count).to eq(1)
       end
     end
   end

data/spec/strategies/plus_one_spec.rb

@@ -4,12 +4,12 @@ describe "Virility::PlusOne" do
   before(:each) do
     @url = "http://creativeallies.com"
   end
-  
+
   describe "poll" do
     context "when there is not a valid result" do
       before(:each) do
         response = double("HTTParty::Response", :parsed_response => {"fake_return_value"=> "OICU812"})
-        Virility::PlusOne.stub(:get).and_return(response)
+        allow(Virility::PlusOne).to receive(:get) { response }
         @virility = Virility::PlusOne.new(@url)
       end
 
@@ -19,7 +19,7 @@ describe "Virility::PlusOne" do
     context "when there is no result" do
       before(:each) do
         response = double("HTTParty::Response")
-        Virility::PlusOne.stub(:get).and_return(response)
+        allow(Virility::PlusOne).to receive(:get) { response }
         @virility = Virility::PlusOne.new(@url)
       end
 
@@ -29,7 +29,7 @@ describe "Virility::PlusOne" do
     context "when there is a result but no specific hash value" do
       before(:each) do
         response = double("HTTParty::Response", :parsed_response => {})
-        Virility::PlusOne.stub(:get).and_return(response)
+        allow(Virility::PlusOne).to receive(:get) { response }
         @virility = Virility::PlusOne.new(@url)
       end
 
@@ -39,7 +39,7 @@ describe "Virility::PlusOne" do
     context "when there is a result but parsed_response is weird" do
       before(:each) do
         response = double("HTTParty::Response", :parsed_response => Object.new)
-        Virility::PlusOne.stub(:get).and_return(response)
+        allow(Virility::PlusOne).to receive(:get) { response }
         @virility = Virility::PlusOne.new(@url)
       end
 
@@ -49,16 +49,16 @@ describe "Virility::PlusOne" do
     context "when there is a valid result" do
       before(:each) do
         response = double("HTTParty::Response", :parsed_response => {"shares"=>"8"})
-        Virility::PlusOne.stub(:get).and_return(response)
+        allow(Virility::PlusOne).to receive(:get) { response }
         @virility = Virility::PlusOne.new(@url)
       end
 
       it "should not raise an error" do
-        lambda { @virility.poll }.should_not raise_error
+        expect{ @virility.poll }.not_to raise_error
       end
 
       it "should return 8 for the count" do
-        @virility.count.should == 8
+        expect(@virility.count).to eq(8)
       end
     end
   end

data/spec/strategies/stumble_upon_spec.rb

@@ -4,12 +4,12 @@ describe "Virility::StumbleUpon" do
   before(:each) do
     @url = "http://creativeallies.com"
   end
-  
+
   describe "poll" do
     context "when there is not a valid result" do
       before(:each) do
         response = double("HTTParty::Response", :parsed_response => {"fake_return_value"=> "OICU812"})
-        Virility::StumbleUpon.stub(:get).and_return(response)
+        allow(Virility::StumbleUpon).to receive(:get) { response }
         @virility = Virility::StumbleUpon.new(@url)
       end
 
@@ -19,7 +19,7 @@ describe "Virility::StumbleUpon" do
     context "when there is no result" do
       before(:each) do
         response = double("HTTParty::Response")
-        Virility::StumbleUpon.stub(:get).and_return(response)
+        allow(Virility::StumbleUpon).to receive(:get) { response }
         @virility = Virility::StumbleUpon.new(@url)
       end
 
@@ -29,7 +29,7 @@ describe "Virility::StumbleUpon" do
     context "when there is a result but no specific hash value" do
       before(:each) do
         response = double("HTTParty::Response", :parsed_response => {})
-        Virility::StumbleUpon.stub(:get).and_return(response)
+        allow(Virility::StumbleUpon).to receive(:get) { response }
         @virility = Virility::StumbleUpon.new(@url)
       end
 
@@ -39,7 +39,7 @@ describe "Virility::StumbleUpon" do
     context "when there is a result but parsed_response is weird" do
       before(:each) do
         response = double("HTTParty::Response", :parsed_response => Object.new)
-        Virility::StumbleUpon.stub(:get).and_return(response)
+        allow(Virility::StumbleUpon).to receive(:get) { response }
         @virility = Virility::StumbleUpon.new(@url)
       end
 
@@ -49,16 +49,16 @@ describe "Virility::StumbleUpon" do
     context "when there is a valid result" do
       before(:each) do
         response = double("HTTParty::Response", :parsed_response => {"url"=>"http://creativeallies.com/", "in_index"=>true, "publicid"=>"2UhTwK", "views"=>4731, "title"=>"Creative Allies | Create Art For Rockstars | Upload For A Chance To Win", "thumbnail"=>"http://cdn.stumble-upon.com/mthumb/388/49348388.jpg", "thumbnail_b"=>"http://cdn.stumble-upon.com/images/nobthumb.png", "submit_link"=>"http://www.stumbleupon.com/submit/?url=http://creativeallies.com/", "badge_link"=>"http://www.stumbleupon.com/badge/?url=http://creativeallies.com/", "info_link"=>"http://www.stumbleupon.com/url/creativeallies.com/"})
-        Virility::StumbleUpon.stub(:get).and_return(response)
+        allow(Virility::StumbleUpon).to receive(:get) { response }
         @virility = Virility::StumbleUpon.new(@url)
       end
 
       it "should not raise an error" do
-        lambda { @virility.poll }.should_not raise_error
+        expect{ @virility.poll }.not_to raise_error
       end
 
       it "should return 4731 for the count" do
-        @virility.count.should == 4731
+        expect(@virility.count).to eq(4731)
       end
     end
   end

data/spec/strategies/twitter_spec.rb

@@ -4,12 +4,12 @@ describe "Virility::Twitter" do
   before(:each) do
     @url = "http://creativeallies.com"
   end
-  
+
   describe "poll" do
     context "when there is not a valid result" do
       before(:each) do
         response = double("HTTParty::Response", :parsed_response => {"fake_return_value"=> "OICU812"})
-        Virility::Twitter.stub(:get).and_return(response)
+        allow(Virility::Twitter).to receive(:get) { response }
         @virility = Virility::Twitter.new(@url)
       end
 
@@ -19,7 +19,7 @@ describe "Virility::Twitter" do
     context "when there is no result" do
       before(:each) do
         response = double("HTTParty::Response")
-        Virility::Twitter.stub(:get).and_return(response)
+        allow(Virility::Twitter).to receive(:get) { response }
         @virility = Virility::Twitter.new(@url)
       end
 
@@ -29,7 +29,7 @@ describe "Virility::Twitter" do
     context "when there is a result but no specific hash value" do
       before(:each) do
         response = double("HTTParty::Response", :parsed_response => {})
-        Virility::Twitter.stub(:get).and_return(response)
+        allow(Virility::Twitter).to receive(:get) { response }
         @virility = Virility::Twitter.new(@url)
       end
 
@@ -39,7 +39,7 @@ describe "Virility::Twitter" do
     context "when there is a result but parsed_response is weird" do
       before(:each) do
         response = double("HTTParty::Response", :parsed_response => Object.new)
-        Virility::Twitter.stub(:get).and_return(response)
+        allow(Virility::Twitter).to receive(:get) { response }
         @virility = Virility::Twitter.new(@url)
       end
 
@@ -49,16 +49,16 @@ describe "Virility::Twitter" do
     context "when there is a valid result" do
       before(:each) do
         response = double("HTTParty::Response", :parsed_response => {"count"=>121, "url"=>"http://creativeallies.com/"})
-        Virility::Twitter.stub(:get).and_return(response)
+        allow(Virility::Twitter).to receive(:get) { response }
         @virility = Virility::Twitter.new(@url)
       end
 
       it "should not raise an error" do
-        lambda { @virility.poll }.should_not raise_error
+        expect{ @virility.poll }.not_to raise_error
       end
 
       it "should return 121 for the count" do
-        @virility.count.should == 121
+        expect(@virility.count).to eq(121)
       end
     end
   end

data/spec/strategy_spec.rb

@@ -11,11 +11,11 @@ describe "Strategy" do
 
   context "initialization" do
     it "should raise an error if a URL is not set" do
-      lambda {Virility::Strategy.new}.should raise_error
+      expect{Virility::Strategy.new}.to raise_error(ArgumentError, "wrong number of arguments (0 for 1)")
     end
 
     it "should set and encode the url" do
-      Virility::Facebook.new(@url).url.should == "http%3A%2F%2Fcreativeallies.com"
+      expect(Virility::Facebook.new(@url).url).to eq("http%3A%2F%2Fcreativeallies.com")
     end
   end
 
@@ -25,11 +25,11 @@ describe "Strategy" do
 
   context "interface" do
     it "should raise an error on poll" do
-      lambda { Virility::Strategy.new(@url).poll }.should raise_error
+      expect{ Virility::Strategy.new(@url).poll }.to raise_error(RuntimeError, "Abstract Method census called on Virility::Strategy - Please define this method")
     end
 
     it "should raise an error on count" do
-      lambda { Virility::Strategy.new(@url).count }.should raise_error
+      expect{ Virility::Strategy.new(@url).count }.to raise_error(RuntimeError, "Abstract Method count called on Virility::Strategy - Please define this method")
     end
   end
 
@@ -40,23 +40,23 @@ describe "Strategy" do
   describe "dynamic methods" do
     before(:each) do
       @virility = Virility::Facebook.new(@url)
-      @virility.stub(:results).and_return(Virility::FB_RESULTS)
+      allow(@virility).to receive(:results) { Virility::FB_RESULTS }
     end
 
     context "overall testing" do
       Virility::FB_RESULTS.each do |key, value|
         it "should return #{value} when get_result is called with #{key}" do
-          @virility.send(key).should == value
+          expect(@virility.send(key)).to eq(value)
         end
       end
 
       Virility::FAKE_FB_RESULTS.each do |key|
         it "should_not raise an error if the result (#{key}) does not exist" do
-          lambda { @virility.send(key) }.should_not raise_error
+          expect{ @virility.send(key) }.not_to raise_error
         end
-        
+
         it "should return 0 if the result (#{key}) does not exist" do
-          @virility.send(key).should == 0
+          expect(@virility.send(key)).to eq(0)
         end
       end
     end
@@ -64,18 +64,18 @@ describe "Strategy" do
     context "result_exists?" do
       before(:each) do
         @virility = Virility::Facebook.new(@url)
-        @virility.stub(:results).and_return(Virility::FB_RESULTS)
+        allow(@virility).to receive(:results) { Virility::FB_RESULTS }
       end
 
       Virility::FB_RESULTS.keys.each do |result|
         it "should return true for #{result}" do
-          @virility.result_exists?(result).should be true
+          expect(@virility.result_exists?(result)).to eq(true)
         end
       end
 
       Virility::FAKE_FB_RESULTS.each do |result|
         it "should return false for #{result}" do
-          @virility.result_exists?(result).should be false
+          expect(@virility.result_exists?(result)).to eq(false)
         end
       end
     end
@@ -83,17 +83,17 @@ describe "Strategy" do
     context "get_result" do
       Virility::FB_RESULTS.each do |key, value|
         it "should return #{value} when get_result is called with #{key}" do
-          @virility.get_result(key).should == value
+          expect(@virility.get_result(key)).to eq(value)
         end
       end
 
       Virility::FAKE_FB_RESULTS.each do |key|
         it "should_not raise an error if the result (#{key}) does not exist" do
-          lambda { @virility.send(key) }.should_not raise_error
+          expect{ @virility.send(key) }.not_to raise_error
         end
-        
+
         it "should return 0 if the result (#{key}) does not exist" do
-          @virility.send(key).should == 0
+          expect(@virility.send(key)).to eq(0)
         end
       end
     end

data/spec/virility_spec.rb

@@ -10,7 +10,7 @@ describe "Virility" do
     context "valid strategies" do
       Virility::TESTING_STRATEGIES.each do |strategy, object|
         it "#{strategy} should create and return a #{object} object" do
-          Virility.factory(strategy, "http://creativeallies.com").should be_a_kind_of object
+          expect(Virility.factory(strategy, "http://creativeallies.com")).to be_a_kind_of(object)
         end
       end
     end
@@ -18,7 +18,7 @@ describe "Virility" do
     context "invalid strategies" do
       Virility::FAKE_TESTING_STRATEGIES.each do |strategy|
         it "#{strategy} should raise an error" do
-          lambda { Virility.factory(strategy, "http://creativeallies.com") }.should raise_error
+          expect{ Virility.factory(strategy, "http://creativeallies.com") }.to raise_error(Virility::UnknownStrategy, "#{strategy} Is Not A Known Strategy")
         end
       end
     end
@@ -31,44 +31,51 @@ describe "Virility" do
   describe "Public API testing" do
     before(:each) do
       @url = "http://creativeallies.com"
-      Virility::Delicious.stub(:get).and_return(double("HTTParty::Response", :parsed_response => {"url"=>"http://creativeallies.com/", "total_posts"=>50, "top_tags"=>{"graphic"=>1, "art"=>1, "contest"=>1, "photography"=>1, "creativity"=>1, "design"=>1, "online"=>1, "music"=>1, "contests"=>1, "freelance"=>1}, "hash"=>"f9468b2d2842d4a9685af46e1b8e9349", "title"=>"Creative Allies | Create Art For Rockstars"}))
-      Virility::Facebook.stub(:get).and_return(double("HTTParty::Response", :parsed_response => {"fql_query_response"=>{"list"=>"true", "link_stat"=>{"like_count"=>"977662", "click_count"=>"265614", "share_count"=>"3020040", "comment_count"=>"1118601", "commentsbox_count"=>"0", "total_count"=>"5116303"}}}))
-      Virility::Pinterest.stub(:get).and_return(double("HTTParty::Response", :parsed_response => {"count"=>1, "url"=>"http://creativeallies.com"}))
-      Virility::PlusOne.stub(:get).and_return(double("HTTParty::Response", :parsed_response => {"shares"=>"8"}))
-      Virility::StumbleUpon.stub(:get).and_return(double("HTTParty::Response", :parsed_response => {"url"=>"http://creativeallies.com/", "in_index"=>true, "publicid"=>"2UhTwK", "views"=>4731, "title"=>"Creative Allies | Create Art For Rockstars | Upload For A Chance To Win", "thumbnail"=>"http://cdn.stumble-upon.com/mthumb/388/49348388.jpg", "thumbnail_b"=>"http://cdn.stumble-upon.com/images/nobthumb.png", "submit_link"=>"http://www.stumbleupon.com/submit/?url=http://creativeallies.com/", "badge_link"=>"http://www.stumbleupon.com/badge/?url=http://creativeallies.com/", "info_link"=>"http://www.stumbleupon.com/url/creativeallies.com/"}))
-      Virility::Twitter.stub(:get).and_return(double("HTTParty::Response", :parsed_response => {"count"=>121, "url"=>"http://creativeallies.com/"}))
+      allow(Virility::Facebook).to receive(:get) { double("HTTParty::Response", :parsed_response => {"fql_query_response"=>{"list"=>"true", "link_stat"=>{"like_count"=>"977662", "click_count"=>"265614", "share_count"=>"3020040", "comment_count"=>"1118601", "commentsbox_count"=>"0", "total_count"=>"5116303"}}}) }
+      allow(Virility::Pinterest).to receive(:get) { double("HTTParty::Response", :parsed_response => {"count"=>1, "url"=>"http://creativeallies.com"}) }
+      allow(Virility::PlusOne).to receive(:get) { double("HTTParty::Response", :parsed_response => {"shares"=>"8"}) }
+      allow(Virility::StumbleUpon).to receive(:get) { double("HTTParty::Response", :parsed_response => {"url"=>"http://creativeallies.com/", "in_index"=>true, "publicid"=>"2UhTwK", "views"=>4731, "title"=>"Creative Allies | Create Art For Rockstars | Upload For A Chance To Win", "thumbnail"=>"http://cdn.stumble-upon.com/mthumb/388/49348388.jpg", "thumbnail_b"=>"http://cdn.stumble-upon.com/images/nobthumb.png", "submit_link"=>"http://www.stumbleupon.com/submit/?url=http://creativeallies.com/", "badge_link"=>"http://www.stumbleupon.com/badge/?url=http://creativeallies.com/", "info_link"=>"http://www.stumbleupon.com/url/creativeallies.com/"}) }
+      allow(Virility::Twitter).to receive(:get) { double("HTTParty::Response", :parsed_response => {"count"=>121, "url"=>"http://creativeallies.com/"}) }
+      allow(Virility::Linkedin).to receive(:get) { double("HTTParty::Response", :parsed_response => { "count":17, "fCnt":"17", "fCntPlusOne":"18", "url":"http:\/\/creativeallies.com" }) }
     end
-    
+
     it "Virility.counts should return a hash of counts" do
-      Virility.counts(@url).should == {:delicious=>50, :facebook=>5116303, :pinterest=>1, :plus_one=>8, :stumble_upon=>4731, :twitter=>121}
+      expect(Virility.counts(@url)).to eq({:facebook=>5116303, :linkedin => 17, :pinterest=>1, :plus_one=>8, :stumble_upon=>4731, :twitter=>121})
     end
-    
+
     it "Virility.total should return the total count" do
-      Virility.total(@url).should == 5121214
+      expect(Virility.total(@url)).to eq(5121181)
     end
-    
+
     it "Virility.poll should return all of the hashed responses" do
-      Virility.poll(@url).should == {:delicious=>{"url"=>"http://creativeallies.com/", "total_posts"=>50, "top_tags"=>{"graphic"=>1, "art"=>1, "contest"=>1, "photography"=>1, "creativity"=>1, "design"=>1, "online"=>1, "music"=>1, "contests"=>1, "freelance"=>1}, "hash"=>"f9468b2d2842d4a9685af46e1b8e9349", "title"=>"Creative Allies | Create Art For Rockstars"}, :facebook=>{"like_count"=>"977662", "click_count"=>"265614", "share_count"=>"3020040", "comment_count"=>"1118601", "commentsbox_count"=>"0", "total_count"=>"5116303"}, :pinterest=>{"count"=>1, "url"=>"http://creativeallies.com"}, :plus_one=>{"shares"=>"8"}, :stumble_upon=>{"url"=>"http://creativeallies.com/", "in_index"=>true, "publicid"=>"2UhTwK", "views"=>4731, "title"=>"Creative Allies | Create Art For Rockstars | Upload For A Chance To Win", "thumbnail"=>"http://cdn.stumble-upon.com/mthumb/388/49348388.jpg", "thumbnail_b"=>"http://cdn.stumble-upon.com/images/nobthumb.png", "submit_link"=>"http://www.stumbleupon.com/submit/?url=http://creativeallies.com/", "badge_link"=>"http://www.stumbleupon.com/badge/?url=http://creativeallies.com/", "info_link"=>"http://www.stumbleupon.com/url/creativeallies.com/"}, :twitter=>{"count"=>121, "url"=>"http://creativeallies.com/"}}
+      expect(Virility.poll(@url)).to eq({
+        :facebook=>{"like_count"=>"977662", "click_count"=>"265614", "share_count"=>"3020040", "comment_count"=>"1118601", "commentsbox_count"=>"0", "total_count"=>"5116303"},
+        :linkedin=>{ "count":17, "fCnt":"17", "fCntPlusOne":"18", "url":"http:\/\/creativeallies.com" },
+        :pinterest=>{"count"=>1, "url"=>"http://creativeallies.com"},
+        :plus_one=>{"shares"=>"8"},
+        :stumble_upon=>{"url"=>"http://creativeallies.com/", "in_index"=>true, "publicid"=>"2UhTwK", "views"=>4731, "title"=>"Creative Allies | Create Art For Rockstars | Upload For A Chance To Win", "thumbnail"=>"http://cdn.stumble-upon.com/mthumb/388/49348388.jpg", "thumbnail_b"=>"http://cdn.stumble-upon.com/images/nobthumb.png", "submit_link"=>"http://www.stumbleupon.com/submit/?url=http://creativeallies.com/", "badge_link"=>"http://www.stumbleupon.com/badge/?url=http://creativeallies.com/", "info_link"=>"http://www.stumbleupon.com/url/creativeallies.com/"},
+        :twitter=>{"count"=>121, "url"=>"http://creativeallies.com/"}
+      })
     end
-    
+
     it "Virility.url should return a Virility::Excitation object" do
-      Virility.url(@url).should be_a_kind_of Virility::Excitation
+      expect(Virility.url(@url)).to be_a_kind_of(Virility::Excitation)
     end
   end
 
   #
   # Error Proofing
   #
-  
+
   describe "Error Proofing" do
     it "should not raise an error with a bad URL" do
-      lambda { Virility.counts("http://this.is.a.crap.url") }.should_not raise_error
+      expect{ Virility.counts("http://this.is.a.crap.url") }.not_to raise_error
     end
-    
+
     it "should return 0 for all strategy counts" do
       @virility = Virility.url("http://this.is.a.crap.url")
-      @virility.total.should == 0
-      @virility.counts.should == {:delicious=>0, :facebook=>0, :pinterest=>0, :plus_one=>0, :stumble_upon=>0, :twitter=>0}
+      expect(@virility.total).to eq(0)
+      expect(@virility.counts).to eq({:facebook=>0, :linkedin=>0, :pinterest=>0, :plus_one=>0, :stumble_upon=>0, :twitter=>0})
     end
   end
 end

data/virility.gemspec

@@ -18,8 +18,9 @@ Gem::Specification.new do |gem|
   gem.test_files    = gem.files.grep(%r{^(test|spec|features)/})
   gem.require_paths = ["lib"]
   # Development
-  gem.add_development_dependency "rspec", "~> 2.6"
+  gem.add_development_dependency "rspec", "~> 3.4"
+  gem.add_development_dependency "pry", "~> 0.10"
   # Dependencies
   gem.add_dependency "httparty", "~> 0.13"
-  gem.add_dependency "multi_json", "~> 1.0"
+  gem.add_dependency "multi_json", "~> 1.11"
 end

metadata

@@ -1,57 +1,71 @@
 --- !ruby/object:Gem::Specification
 name: virility
 version: !ruby/object:Gem::Version
-  version: 0.1.4
+  version: 0.2.0
 platform: ruby
 authors:
 - Jay Sanders
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-05-01 00:00:00.000000000 Z
+date: 2016-01-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.6'
+        version: '3.4'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.6'
+        version: '3.4'
+- !ruby/object:Gem::Dependency
+  name: pry
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.10'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.10'
 - !ruby/object:Gem::Dependency
   name: httparty
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '0.13'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '0.13'
 - !ruby/object:Gem::Dependency
   name: multi_json
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '1.11'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '1.11'
 description: Virility leverages the API's of many popular social services to collect
   data about the virility of a particular URL.
 email:
@@ -60,10 +74,10 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
-- .rspec
-- .ruby-gemset
-- .ruby-version
+- ".gitignore"
+- ".rspec"
+- ".ruby-gemset"
+- ".ruby-version"
 - Gemfile
 - LICENSE.txt
 - README.md
@@ -71,7 +85,6 @@ files:
 - lib/virility.rb
 - lib/virility/exceptions.rb
 - lib/virility/excitation.rb
-- lib/virility/strategies/delicious.rb
 - lib/virility/strategies/facebook.rb
 - lib/virility/strategies/pinterest.rb
 - lib/virility/strategies/plus_one.rb
@@ -82,7 +95,6 @@ files:
 - lib/virility/version.rb
 - spec/excitation_spec.rb
 - spec/spec_helper.rb
-- spec/strategies/delicious_spec.rb
 - spec/strategies/facebook_spec.rb
 - spec/strategies/pinterest_spec.rb
 - spec/strategies/plus_one_spec.rb
@@ -100,17 +112,17 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 2.4.8
 signing_key: 
 specification_version: 4
 summary: Virility calls upon the API's of many popular social services such as Facebook,
@@ -121,7 +133,6 @@ summary: Virility calls upon the API's of many popular social services such as F
 test_files:
 - spec/excitation_spec.rb
 - spec/spec_helper.rb
-- spec/strategies/delicious_spec.rb
 - spec/strategies/facebook_spec.rb
 - spec/strategies/pinterest_spec.rb
 - spec/strategies/plus_one_spec.rb

data/lib/virility/strategies/delicious.rb

@@ -1,19 +0,0 @@
-module Virility
-  class Delicious < Strategy
-
-    parser(
-      Proc.new do |body, format|
-        MultiJson.decode(body.scan(/(\{.+\})/).flatten.first)
-      end
-    )
-
-    def census
-      self.class.get("http://feeds.delicious.com/v2/json/urlinfo/data?url=#{@url}")
-    end
-
-    def count
-      results["total_posts"] || 0
-    end
-
-  end
-end

data/spec/strategies/delicious_spec.rb

@@ -1,66 +0,0 @@
-require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
-
-describe "Virility::Delicious" do
-  before(:each) do
-    @url = "http://creativeallies.com"
-  end
-  
-  describe "poll" do
-    context "when there is not a valid result" do
-      before(:each) do
-        response = double("HTTParty::Response", :parsed_response => {"fake_return_value"=> "OICU812"})
-        Virility::Delicious.stub(:get).and_return(response)
-        @virility = Virility::Delicious.new(@url)
-      end
-
-      it_should_behave_like "no context results"
-    end
-
-    context "when there is no result" do
-      before(:each) do
-        response = double("HTTParty::Response")
-        Virility::Delicious.stub(:get).and_return(response)
-        @virility = Virility::Delicious.new(@url)
-      end
-
-      it_should_behave_like "no context results"
-    end
-
-    context "when there is a result but no specific hash value" do
-      before(:each) do
-        response = double("HTTParty::Response", :parsed_response => {})
-        Virility::Delicious.stub(:get).and_return(response)
-        @virility = Virility::Delicious.new(@url)
-      end
-
-      it_should_behave_like "no context results"
-    end
-
-    context "when there is a result but parsed_response is weird" do
-      before(:each) do
-        response = double("HTTParty::Response", :parsed_response => Object.new)
-        Virility::Delicious.stub(:get).and_return(response)
-        @virility = Virility::Delicious.new(@url)
-      end
-
-      it_should_behave_like "no context results"
-    end
-
-    context "when there is a valid result" do
-      before(:each) do
-        response = double("HTTParty::Response", :parsed_response => {"url"=>"http://creativeallies.com/", "total_posts"=>50, "top_tags"=>{"graphic"=>1, "art"=>1, "contest"=>1, "photography"=>1, "creativity"=>1, "design"=>1, "online"=>1, "music"=>1, "contests"=>1, "freelance"=>1}, "hash"=>"f9468b2d2842d4a9685af46e1b8e9349", "title"=>"Creative Allies | Create Art For Rockstars"})
-        Virility::Delicious.stub(:get).and_return(response)
-        @virility = Virility::Delicious.new(@url)
-      end
-
-      it "should not raise an error" do
-        lambda { @virility.poll }.should_not raise_error
-      end
-
-      it "should return 50 for the count" do
-        @virility.count.should == 50
-      end
-    end
-  end
-
-end