onebox 1.0.1 → 1.1.0

data/spec/lib/onebox/engine/itunes_onebox_spec.rb

@@ -0,0 +1,29 @@
+require "spec_helper"
+
+describe Onebox::Engine::ItunesOnebox do
+  before(:all) do
+    @link = "https://itunes.apple.com/us/app/minecraft-pocket-edition/id479516143?mt=8"
+    fake(@link, response("itunes"))
+  end
+  before(:each) { Onebox.defaults.cache.clear }
+
+  let(:link) { @link }
+
+  it_behaves_like "an engine"
+
+  describe "#to_html" do
+    let(:html) { described_class.new(link).to_html }
+
+    it "returns the product title" do
+      expect(html).to include("Minecraft – Pocket Edition")
+    end
+
+    it "returns the product image" do
+      expect(html).to include("bxerxqln.png")
+    end
+
+    it "returns the product description" do
+      expect(html).to include("Get Minecraft – Pocket Edition on the App Store.")
+    end
+  end
+end

data/spec/lib/onebox/engine/json_spec.rb

@@ -0,0 +1,27 @@
+require "spec_helper"
+
+describe Onebox::Engine::JSON do
+  before(:all) do
+    @link = "http://gist.github.com"
+    fake(@link, response("githubgist"))
+  end
+  before(:each) { Onebox.defaults.cache.clear }
+
+  let(:link) { @link }
+
+  describe "#raw" do
+    class OneboxEngineJSON
+      include Onebox::Engine
+      include Onebox::Engine::JSON
+
+      def initialize(link)
+        @url = link
+      end
+    end
+
+    it "returns a hash" do
+      object = OneboxEngineJSON.new("http://gist.github.com").send(:raw)
+      expect(object).to be_a(Hash)
+    end
+  end
+end

data/spec/lib/onebox/engine/kinomap_onebox_spec.rb

@@ -0,0 +1,37 @@
+require "spec_helper"
+
+describe Onebox::Engine::KinomapOnebox do
+  before(:all) do
+    @link = "http://www.kinomap.com/watch/52wjcu"
+    fake(@link, response(described_class.template_name))
+  end
+  before(:each) { Onebox.defaults.cache.clear }
+
+  let(:link) { @link }
+
+  it_behaves_like "an engine"
+
+  describe "#to_html" do
+    let(:html) { described_class.new(link).to_html }
+
+    it "returns video title" do
+      expect(html).to include("Enduro Jeunes Touquet - Nicolas Letévé")
+    end
+
+    it "returns video photo" do
+      expect(html).to include("52wjcu_320x240.jpg")
+    end
+
+    it "returns video description" do
+      expect(html).to include("A partir du parc moto,")
+    end
+
+    it "returns video URL" do
+      expect(html).to include("http://v2.kinomap.com/embed/52wjcu")
+    end
+
+    it "returns URL" do
+      expect(html).to include(link)
+    end
+  end
+end

data/spec/lib/onebox/engine/{nfb_spec.rb → nfb_onebox_spec.rb}

@@ -1,12 +1,15 @@
 require "spec_helper"
 
 describe Onebox::Engine::NFBOnebox do
-  let(:link) { "http://nfb.ca" }
-  before do
-    fake(link, response("nfb.response"))
+  before(:all) do
+    @link = "http://www.nfb.ca/film/overdose"
+    fake(@link, response(described_class.template_name))
   end
+  before(:each) { Onebox.defaults.cache.clear }
 
-  it_behaves_like "engines"
+  let(:link) { @link }
+
+  it_behaves_like "an engine"
 
   describe "#to_html" do
     let(:html) { described_class.new(link).to_html }

data/spec/lib/onebox/engine/open_graph_spec.rb

@@ -0,0 +1,27 @@
+require "spec_helper"
+
+describe Onebox::Engine::OpenGraph do
+  before(:all) do
+    @link = "http://flickr.com"
+    fake(@link, response("flickr"))
+  end
+  before(:each) { Onebox.defaults.cache.clear }
+
+  let(:link) { @link }
+
+  describe "#raw" do
+    class OneboxEngineOpenGraph
+      include Onebox::Engine
+      include Onebox::Engine::OpenGraph
+
+      def initialize(link)
+        @url = link
+      end
+    end
+
+    it "returns a OpenGraph object that has a metadata method" do
+      object = OneboxEngineOpenGraph.new("http://flickr.com").send(:raw)
+      expect(object).to respond_to(:metadata)
+    end
+  end
+end

data/spec/lib/onebox/engine/{qik_spec.rb → qik_onebox_spec.rb}

@@ -1,12 +1,15 @@
 require "spec_helper"
 
 describe Onebox::Engine::QikOnebox do
-  let(:link) { "http://qik.com" }
-  before do
-    fake(link, response("qik.response"))
+  before(:all) do
+    @link = "http://qik.com/video/13430626"
+    fake(@link, response(described_class.template_name))
   end
+  before(:each) { Onebox.defaults.cache.clear }
 
-  it_behaves_like "engines"
+  let(:link) { @link }
+
+  it_behaves_like "an engine"
 
   describe "#to_html" do
     let(:html) { described_class.new(link).to_html }

data/spec/lib/onebox/engine/{revision3_spec.rb → revision3_onebox_spec.rb}

@@ -1,12 +1,15 @@
 require "spec_helper"
 
 describe Onebox::Engine::Revision3Onebox do
-  let(:link) { "http://revision3.com" }
-  before do
-    fake(link, response("revision3.response"))
+  before(:all) do
+    @link = "http://revision3.com/discoverysharks/blue-sharks"
+    fake(@link, response(described_class.template_name))
   end
+  before(:each) { Onebox.defaults.cache.clear }
 
-  it_behaves_like "engines"
+  let(:link) { @link }
+
+  it_behaves_like "an engine"
 
   describe "#to_html" do
     let(:html) { described_class.new(link).to_html }

data/spec/lib/onebox/engine/{slideshare_spec.rb → slideshare_onebox_spec.rb}

@@ -1,12 +1,15 @@
 require "spec_helper"
 
 describe Onebox::Engine::SlideshareOnebox do
-  let(:link) { "http://slideshare.net" }
-  before do
-    fake(link, response("slideshare.response"))
+  before(:all) do
+    @link = "http://www.slideshare.net/TravelWorldPassport/12-local-traditions"
+    fake(@link, response(described_class.template_name))
   end
+  before(:each) { Onebox.defaults.cache.clear }
 
-  it_behaves_like "engines"
+  let(:link) { @link }
+
+  it_behaves_like "an engine"
 
   describe "#to_html" do
     let(:html) { described_class.new(link).to_html }

data/spec/lib/onebox/engine/{sound_cloud_spec.rb → sound_cloud_onebox_spec.rb}

@@ -1,12 +1,15 @@
 require "spec_helper"
 
 describe Onebox::Engine::SoundCloudOnebox do
-  let(:link) { "http://soundcloud.com" }
-  before do
-    fake(link, response("soundcloud.response"))
+  before(:all) do
+    @link = "https://soundcloud.com/rac/penguin-prison-worse-it-gets-rac-mix"
+    fake(@link, response(described_class.template_name))
   end
+  before(:each) { Onebox.defaults.cache.clear }
 
-  it_behaves_like "engines"
+  let(:link) { @link }
+
+  it_behaves_like "an engine"
 
   describe "#to_html" do
     let(:html) { described_class.new(link).to_html }

data/spec/lib/onebox/engine/spotify_onebox_spec.rb

@@ -0,0 +1,36 @@
+require "spec_helper"
+
+describe Onebox::Engine::SpotifyOnebox do
+  before(:all) do
+    @link = "http://open.spotify.com/album/3eEtlM70GU40OyHMotY15N"
+    fake(@link, response(described_class.template_name))
+  end
+  before(:each) { Onebox.defaults.cache.clear }
+
+  let(:link) { @link }
+
+  it_behaves_like "an engine"
+
+  describe "#to_html" do
+    let(:html) { described_class.new(link).to_html }
+
+    it "returns album title" do
+      title = "Keep Moving Forward EP"
+      expect(html).to include(title)
+    end
+
+    it "returns album description" do
+      description = "Keep Moving Forward EP, an album by Bubble on Spotify"
+      expect(html).to include(description)
+    end
+
+    it "returns album image" do
+      image = "http://o.scdn.co/image/d2c3de070317af4aae4e03daa976690431f1849d"
+      expect(html).to include(image)
+    end
+
+    it "returns URL" do
+      expect(html).to include(link)
+    end
+  end
+end

data/spec/lib/onebox/engine/{stack_exchange_spec.rb → stack_exchange_onebox_spec.rb}

@@ -1,12 +1,15 @@
 require "spec_helper"
 
 describe Onebox::Engine::StackExchangeOnebox do
-  let(:link) { "http://stackexchange.com" }
-  before do
-    fake(link, response("stackexchange.response"))
+  before(:all) do
+    @link = "http://stackoverflow.com/questions/17992553/concept-behind-these-four-lines-of-tricky-c-code"
+    fake(@link, response(described_class.template_name))
   end
+  before(:each) { Onebox.defaults.cache.clear }
 
-  it_behaves_like "engines"
+  let(:link) { @link }
+
+  it_behaves_like "an engine"
 
   describe "#to_html" do
     let(:html) { described_class.new(link).to_html }

data/spec/lib/onebox/engine/{ted_spec.rb → ted_onebox_spec.rb}

@@ -1,12 +1,15 @@
 require "spec_helper"
 
 describe Onebox::Engine::TedOnebox do
-  let(:link) { "http://ted.com" }
-  before do
-    fake(link, response("ted.response"))
+  before(:all) do
+    @link = "http://www.ted.com/talks/eli_beer_the_fastest_ambulance_a_motorcycle.html"
+    fake(@link, response(described_class.template_name))
   end
+  before(:each) { Onebox.defaults.cache.clear }
 
-  it_behaves_like "engines"
+  let(:link) { @link }
+
+  it_behaves_like "an engine"
 
   describe "#to_html" do
     let(:html) { described_class.new(link).to_html }

data/spec/lib/onebox/engine/twitter_onebox_spec.rb

@@ -0,0 +1,47 @@
+require "spec_helper"
+
+describe Onebox::Engine::TwitterOnebox do
+  before(:all) do
+    @link = "https://twitter.com/toastergrrl/status/363116819147538433"
+    fake(@link, response(described_class.template_name))
+  end
+  before(:each) { Onebox.defaults.cache.clear }
+
+  let(:link) { @link }
+
+  it_behaves_like "an engine"
+
+  describe "#to_html" do
+    let(:html) { described_class.new(link).to_html }
+
+    it "has tweet text" do
+      expect(html).to include("I'm a sucker for pledges.")
+    end
+
+    it "has tweet time and date" do
+      expect(html).to include("6:59 PM - 1 Aug 13")
+    end
+
+    it "has user name" do
+      expect(html).to include("@toastergrrl")
+    end
+
+    it "has user avatar" do
+      expect(html).to include("39b969d32a10b2437563e246708c8f9d_normal.jpeg")
+    end
+
+    it "has tweet favorite count" do
+      pending
+      expect(html).to include("")
+    end
+
+    it "has retweet count" do
+      pending
+      expect(html).to include("")
+    end
+
+    it "has URL" do
+      expect(html).to include(link)
+    end
+  end
+end

data/spec/lib/onebox/engine/{viddler_spec.rb → viddler_onebox_spec.rb}

@@ -1,12 +1,15 @@
 require "spec_helper"
 
 describe Onebox::Engine::ViddlerOnebox do
-  let(:link) { "http://viddler.com" }
-  before do
-    fake(link, response("viddler.response"))
+  before(:all) do
+    @link = "http://www.viddler.com/v/7164f749"
+    fake(@link, response(described_class.template_name))
   end
+  before(:each) { Onebox.defaults.cache.clear }
 
-  it_behaves_like "engines"
+  let(:link) { @link }
+
+  it_behaves_like "an engine"
 
   describe "#to_html" do
     let(:html) { described_class.new(link).to_html }

data/spec/lib/onebox/engine/{vimeo_spec.rb → vimeo_onebox_spec.rb}

@@ -1,12 +1,15 @@
 require "spec_helper"
 
 describe Onebox::Engine::VimeoOnebox do
-  let(:link) { "http://vimeo.com" }
-  before do
-    fake(link, response("vimeo.response"))
+  before(:all) do
+    @link = "http://vimeo.com/70437049"
+    fake(@link, response(described_class.template_name))
   end
+  before(:each) { Onebox.defaults.cache.clear }
 
-  it_behaves_like "engines"
+  let(:link) { @link }
+
+  it_behaves_like "an engine"
 
   describe "#to_html" do
     let(:html) { described_class.new(link).to_html }

data/spec/lib/onebox/engine/{wikipedia_spec.rb → wikipedia_onebox_spec.rb}

@@ -1,12 +1,15 @@
 require "spec_helper"
 
 describe Onebox::Engine::WikipediaOnebox do
-  let(:link) { "http://wikipedia.com" }
-  before do
-    fake(link, response("wikipedia.response"))
+  before(:all) do
+    @link = "http://en.wikipedia.org/wiki/Kevin_Bacon"
+    fake(@link, response(described_class.template_name))
   end
+  before(:each) { Onebox.defaults.cache.clear }
 
-  it_behaves_like "engines"
+  let(:link) { @link }
+
+  it_behaves_like "an engine"
 
   describe "#to_html" do
     let(:html) { described_class.new(link).to_html }

data/spec/lib/onebox/engine/{yfrog_spec.rb → yfrog_onebox_spec.rb}

@@ -1,12 +1,15 @@
 require "spec_helper"
 
 describe Onebox::Engine::YfrogOnebox do
-  let(:link) { "http://yfrog.com" }
-  before do
-    fake(link, response("yfrog.response"))
+  before(:each) { Onebox.defaults.cache.clear }
+  before(:all) do
+    @link = "http://twitter.yfrog.com/h0jjdobj?sa=0"
+    fake(@link, response(described_class.template_name))
   end
 
-  it_behaves_like "engines"
+  let(:link) { @link }
+
+  it_behaves_like "an engine"
 
   describe "#to_html" do
     let(:html) { described_class.new(link).to_html }

data/spec/lib/onebox/engine_spec.rb

@@ -11,8 +11,10 @@ class OneboxEngineExample
     { key: "value" }
   end
 
-  def template
-    %|<div class="onebox"><a href="{{url}}"></a></div>|
+  def view
+    @view.tap do |layout|
+      layout.view.template = %|<div class="onebox"><a href="{{url}}"></a></div>|
+    end
   end
 end
 
@@ -30,7 +32,7 @@ describe Onebox::Engine do
   end
 
   describe "#record" do
-    class OneboxEngineBar
+    class OneboxEngineRecord
       include Onebox::Engine
 
       def data
@@ -40,49 +42,58 @@ describe Onebox::Engine do
 
     it "returns cached value for given url if its url is already in cache" do
       cache = { "http://example.com" => "old content" }
-      result = OneboxEngineBar.new("http://example.com", cache).send(:record)
+      result = OneboxEngineRecord.new("http://example.com", cache).send(:record)
       expect(result).to eq("old content")
     end
 
     it "stores cache value for given url if cache key doesn't exist" do
       cache = { "http://example.com1" => "old content" }
-      result = OneboxEngineBar.new("http://example.com", cache).send(:record)
+      result = OneboxEngineRecord.new("http://example.com", cache).send(:record)
       expect(result).to eq("new content")
     end
   end
 
-  describe "#template_path" do
-    it "returns file path for onebox template" do
-      class OneboxEngineVoo
-        include Onebox::Engine
-      end
-      result = OneboxEngineVoo.new("http://amazon.com").send(:template_path)
-      expect(result).to eq("/home/jzeta/Projects/rgsoc/onebox/templates/enginevoo.handlebars")
-    end
-  end
-
   describe ".===" do
+    class OneboxEngineTripleEqual
+      include Onebox::Engine
+      @@matcher = /example/
+    end
     it "returns true if argument matches the matcher" do
-      class OneboxEngineFoo
-        include Onebox::Engine
-        @@matcher = /example/
-      end
-      result = OneboxEngineFoo === "http://www.example.com/product/5?var=foo&bar=5"
+      result = OneboxEngineTripleEqual === "http://www.example.com/product/5?var=foo&bar=5"
       expect(result).to eq(true)
     end
   end
 
   describe ".matches" do
+    class OneboxEngineMatches
+      include Onebox::Engine
+
+      matches do
+        find "foo.com"
+      end
+    end
+
     it "sets @@matcher to a regular expression" do
-      class OneboxEngineFar
-        include Onebox::Engine
+      regex = OneboxEngineMatches.class_variable_get(:@@matcher)
+      expect(regex).to be_a(Regexp)
+    end
+  end
 
-        matches do
-          find "foo.com"
-        end
+  describe ".template_name" do
+    module ScopeForTemplateName
+      class TemplateNameOnebox
+        include Onebox::Engine
       end
-      regex = OneboxEngineFar.class_variable_get(:@@matcher)
-      expect(regex).to eq(/(?:foo\.com)/i)
+    end
+
+    let(:template_name) { ScopeForTemplateName::TemplateNameOnebox.template_name }
+
+    it "should not include the scope" do
+      expect(template_name).not_to include("ScopeForTemplateName", "scopefortemplatename")
+    end
+
+    it "should not include the word Onebox" do
+      expect(template_name).not_to include("onebox", "Onebox")
     end
   end
 end