onebox 1.0.1 → 1.1.0

data/spec/lib/onebox/engine/{amazon_spec.rb → amazon_onebox_spec.rb}

@@ -1,12 +1,15 @@
 require "spec_helper"
 
 describe Onebox::Engine::AmazonOnebox do
-  let(:link) { "http://amazon.com" }
-  before do
-    fake(link, response("amazon.response"))
+  before(:all) do
+    @link = "http://www.amazon.com/Knit-Noro-Accessories-Colorful-Little/dp/193609620X"
+    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/{bliptv_spec.rb → bliptv_onebox_spec.rb}

@@ -1,12 +1,15 @@
 require "spec_helper"
 
 describe Onebox::Engine::BliptvOnebox do
-  let(:link) { "http://blip.tv" }
-  before do
-    fake(link, response("bliptv.response"))
+  before(:all) do
+    @link = "http://blip.tv"
+    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/{clikthrough_spec.rb → clikthrough_onebox_spec.rb}

@@ -1,12 +1,15 @@
 require "spec_helper"
 
 describe Onebox::Engine::ClikThroughOnebox do
-  let(:link) { "http://www.clickthough.com"}
-  before do
-    fake(link, response("clikthrough.response"))
+  before(:all) do
+    @link = "http://www.clikthrough.com/theater/video/49/en-US"
+    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/{college_humor_spec.rb → college_humor_onebox_spec.rb}

@@ -1,12 +1,15 @@
 require "spec_helper"
 
 describe Onebox::Engine::CollegeHumorOnebox do
-  let(:link) { "http://collegehumor.com" }
-  before do
-    fake(link, response("collegehumor.response"))
+  before(:all) do
+    @link = "http://collegehumor.com"
+    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/{dailymotion_spec.rb → dailymotion_onebox_spec.rb}

@@ -1,12 +1,15 @@
 require "spec_helper"
 
 describe Onebox::Engine::DailymotionOnebox do
-  let(:link) { "http://dailymotion.com" }
-  before do
-    fake(link, response("dailymotion.response"))
+  before(:all) do
+    @link = "http://dailymotion.com"
+    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/{dotsub_spec.rb → dotsub_onebox_spec.rb}

@@ -1,12 +1,15 @@
 require "spec_helper"
 
 describe Onebox::Engine::DotsubOnebox do
-  let(:link) { "http://dotsub.com/view/665bd0d5-a9f4-4a07-9d9e-b31ba926ca78" }
-  before do
-    fake(link, response("dotsub.response"))
+  before(:all) do
+    @link = "http://dotsub.com/view/665bd0d5-a9f4-4a07-9d9e-b31ba926ca78"
+    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/example_onebox_spec.rb

@@ -0,0 +1,21 @@
+require "spec_helper"
+
+describe Onebox::Engine::ExampleOnebox do
+  before(:all) do
+    @link = "http://example.com"
+    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 template if given valid data" do
+      expect(html).to include("Example Domain 1")
+    end
+  end
+end

data/spec/lib/onebox/engine/{flickr_spec.rb → flickr_onebox_spec.rb}

@@ -1,12 +1,15 @@
 require "spec_helper"
 
 describe Onebox::Engine::FlickrOnebox do
-  let(:link) { "http://flickr.com" }
-  before do
-    fake(link, response("flickr.response"))
+  before(:all) do
+    @link = "http://flickr.com"
+    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/{funny_or_die_spec.rb → funny_or_die_onebox_spec.rb}

@@ -1,12 +1,15 @@
 require "spec_helper"
 
 describe Onebox::Engine::FunnyOrDieOnebox do
-  let(:link) { "http://funnyordie.com" }
-  before do
-    fake(link, response("funnyordie.response"))
+  before(:all) do
+    @link = "http://funnyordie.com"
+    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/github_blob_onebox_spec.rb

@@ -0,0 +1,30 @@
+require "spec_helper"
+
+describe Onebox::Engine::GithubBlobOnebox do
+  let(:link) { "https://github.com/discourse/discourse/blob/master/lib/oneboxer/github_blob_onebox.rb" }
+  before do
+    fake(link, response("githubblob"))
+  end
+
+  it_behaves_like "an engine"
+
+  describe "#to_html" do
+    let(:html) { described_class.new(link).to_html }
+
+    it "has file name" do
+      expect(html).to include("github_blob_onebox.rb")
+    end
+
+    it "has number of lines" do
+      expect(html).to include("50")
+    end
+
+    it "has blob contents" do
+      expect(html).to include("module Oneboxer")
+    end
+
+    it "has URL" do
+      expect(html).to include(link)
+    end
+  end
+end

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

@@ -0,0 +1,61 @@
+require "spec_helper"
+
+describe Onebox::Engine::GithubCommitOnebox do
+  before(:all) do
+    @link = "https://github.com/discourse/discourse/commit/803d023e2307309f8b776ab3b8b7e38ba91c0919"
+    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 repo owner" do
+      expect(html).to include("discourse")
+    end
+
+    it "has repo name" do
+      expect(html).to include("discourse")
+    end
+
+    it "has commit sha" do
+      expect(html).to include("803d023e2307309f8b776ab3b8b7e38ba91c0919")
+    end
+
+    it "has commit author gravatar" do
+      expect(html).to include("2F7d3010c11d08cf990b7614d2c2ca9098.png")
+    end
+
+    it "has commit message" do
+      expect(html).to include("Fixed GitHub auth")
+    end
+
+    it "has commit author" do
+      expect(html).to include("SamSaffron")
+    end
+
+    it "has commit time and date" do
+      expect(html).to include("2013-08-02T02:16:44Z")
+    end
+
+    it "has number of files changed" do
+      expect(html).to include("1 changed file")
+    end
+
+    it "has number of additions" do
+      expect(html).to include("18 additions")
+    end
+
+    it "has number of deletions" do
+      expect(html).to include("2 deletions")
+    end
+
+    it "has URL" do
+      expect(html).to include(link)
+    end
+  end
+end

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

@@ -0,0 +1,29 @@
+require "spec_helper"
+
+describe Onebox::Engine::GithubGistOnebox do
+  before(:all) do
+    @link = "https://gist.github.com/anikalindtner/153044e9bea3331cc103"
+    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 raw file contents" do
+      expect(html).to include("# Create a new blog post on GitHub")
+    end
+
+    it "has author" do
+      expect(html).to include("anikalindtner")
+    end
+
+    it "has URL" do
+      expect(html).to include(link)
+    end
+  end
+end

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

@@ -0,0 +1,58 @@
+require "spec_helper"
+
+describe Onebox::Engine::GithubPullRequestOnebox do
+  let(:link) { "https://github.com/discourse/discourse/pull/1253/" }
+  before do
+    fake(link, response("githubpullrequest"))
+  end
+
+  it_behaves_like "an engine"
+
+  describe "#to_html" do
+    let(:html) { described_class.new(link).to_html }
+
+    it "has pull request author" do
+      expect(html).to include("jamesaanderson")
+    end
+
+    it "has pull request title" do
+      expect(html).to include("Add audio onebox")
+    end
+
+    it "has repo name" do
+      expect(html).to include("discourse")
+    end
+
+    it "has commit author gravatar" do
+      expect(html).to include("b3e9977094ce189bbb493cf7f9adea21")
+    end
+
+    it "has commit description" do
+      expect(html).to include("8168")
+    end
+
+    it "has commit time and date" do
+      expect(html).to include("2013-07-26T02:05:53Z")
+    end
+
+    it "has number of commits" do
+      expect(html).to include("1")
+    end
+
+    it "has number of files changed" do
+      expect(html).to include("4")
+    end
+
+    it "has number of additions" do
+      expect(html).to include("19")
+    end
+
+    it "has number of deletions" do
+      expect(html).to include("1")
+    end
+
+    it "has URL" do
+      expect(html).to include(link)
+    end
+  end
+end

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

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

data/spec/lib/onebox/engine/{hulu_spec.rb → hulu_onebox_spec.rb}

@@ -1,12 +1,15 @@
 require "spec_helper"
 
 describe Onebox::Engine::HuluOnebox do
-  let(:link) { "http://hulu.com" }
-  before do
-    fake(link, response("hulu.response"))
+  before(:all) do
+    @link = "http://www.hulu.com/watch/515146"
+    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/imgur_image_onebox_spec.rb

@@ -0,0 +1,29 @@
+require "spec_helper"
+
+describe Onebox::Engine::ImgurImageOnebox do
+  before(:all) do
+    @link = "http://imgur.com/gallery/twoDTCU"
+    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 the image title" do
+      expect(html).to include("My dog likes to hug me")
+    end
+
+    it "returns the image" do
+      expect(html).to include("twoDTCU.jpg")
+    end
+
+    it "returns URL" do
+      expect(html).to include(link)
+    end
+  end
+end