headless-rails 0.0.3 → 0.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: f8f528076eccde288b3cec6d1fe33dcbddbea353
-  data.tar.gz: 0ac9c6ce2bb2856dd14b8c24588d4fd546e0408d
+  metadata.gz: 3e99bfee5683052e01dcd3ff6064f12ba84f1ebc
+  data.tar.gz: a799666cdfc2a8ae4b61b315697611ab3fda75ad
 SHA512:
-  metadata.gz: 4ec9616d598293f33e8876f5649e14b5bbfaee4fa7da824bd6b0990cb701025ed26799c9171c91b476cd57e68bad63b3ece0b2cbc7b63b6737b0df041d9614f9
-  data.tar.gz: ab7267fdb39ab4699200ccc3ca0cb0943697209c2c95eb0ba5d55c7970ce9c38b6bfc5106d1452f48b54dc44e3abc43dd6ba645026e29401cadc7821964e83fb
+  metadata.gz: 79b18dbb88c26be4af7898e4703758f61ab9b301ad0ffd964c9e99a1954474e202b8f7c2ae8c7b0cabcc5e068652ae87c817fce5a5f993a9002bd31fc9921668
+  data.tar.gz: d1ba0c21be9b57b08678ea49924f6266867d6a34bbf1b7c7d255c708a59a726c01bd779ef50d512dca4b1ffe1d87ce52999844094f1ffbdc0f39a407435c7061

data/headless-rails.gemspec

@@ -26,4 +26,5 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "bundler", "~> 1.3"
   spec.add_development_dependency "rake"
   spec.add_development_dependency "rspec"
+  spec.add_development_dependency "rack-test"
 end

data/lib/headless/ajax_crawler/middleware.rb

@@ -0,0 +1,25 @@
+require "rack/request"
+require "headless/ajax_crawler/request_matcher"
+require "headless/ajax_crawler/url_extractor"
+require "headless/api_client"
+
+module Headless
+  module AjaxCrawler
+    class Middleware < Struct.new(:app)
+      def call(env)
+        request = ::Rack::Request.new(env)
+
+        if ::Headless::AjaxCrawler::RequestMatcher.call(request)
+          url = ::Headless::AjaxCrawler::UrlExtractor.call(request)
+          crawled = ::Headless::APIClient.crawl(url)
+
+          if crawled.success?
+            return [200, { 'Content-Type' => "text/html" }, [crawled.content]]
+          end
+        end
+
+        app.call(env)
+      end
+    end
+  end
+end

data/lib/headless/ajax_crawler/request_matcher.rb

@@ -1,6 +1,8 @@
+require "headless/ajax_crawler"
+
 module Headless
   module AjaxCrawler
-    class RequestMatcher< Struct.new(:request)
+    class RequestMatcher < Struct.new(:request)
 
       def self.call(request)
         new(request).match?

data/lib/headless/ajax_crawler/url_extractor.rb

@@ -1,4 +1,5 @@
-require 'rack/utils'
+require "headless/ajax_crawler"
+require "rack/utils"
 
 module Headless
   module AjaxCrawler

data/lib/headless/ajax_crawler.rb

@@ -1,5 +1,4 @@
-require 'headless/ajax_crawler/request_matcher'
-require 'headless/ajax_crawler/url_extractor'
+require 'headless/ajax_crawler/middleware'
 
 module Headless
   module AjaxCrawler

data/lib/headless/rails/version.rb

@@ -1,5 +1,5 @@
 module Headless
   module Rails
-    VERSION = "0.0.3"
+    VERSION = "0.0.4"
   end
 end

data/lib/headless/rails.rb

@@ -1,17 +1,8 @@
 require "headless/rails/version"
-require "headless/ajax_crawler"
-require "headless/api_client"
+require "headless/ajax_crawler/middleware"
 
 module Headless
   module Rails
-
-    def respond_to_ajax_crawlers
-      if ::Headless::AjaxCrawler::RequestMatcher.call(request)
-        url = ::Headless::AjaxCrawler::UrlExtractor.call(request)
-        crawled = ::Headless::APIClient.crawl(url)
-        render :text => crawled.content if crawled.success?
-      end
-    end
-
+    Middleware = Headless::AjaxCrawler::Middleware
   end
 end

data/spec/headless/ajax_crawler/middleware_spec.rb

@@ -0,0 +1,40 @@
+require 'spec_helper'
+require 'rack/test'
+
+RSpec.describe Headless::AjaxCrawler::Middleware do
+
+  include Rack::Test::Methods
+
+  def app
+    hello_world = lambda { |env|
+      headers = { 'Content-Type' => "text/html" }
+      [200, headers, ['Hello world!']]
+    }
+    Headless::AjaxCrawler::Middleware.new(hello_world)
+  end
+
+  let(:headless_response) { double(:headless_response, :success? => true, :content => :headless_content) }
+
+  context "regular url" do
+    it "renders hello world" do
+      get "/"
+      expect(last_response.body).to include("Hello world")
+    end
+  end
+
+  context "with an escaped_fragment" do
+    it "call headless and renders it" do
+      expect(Headless::APIClient).to receive(:crawl).and_return(headless_response)
+      get "/?_escaped_fragment_="
+      expect(last_response.body).to include("headless_content")
+    end
+
+    context "a POST" do
+      it "renders hello world" do
+        post "/?_escaped_fragment_="
+        expect(last_response.body).to include("Hello world")
+      end
+    end
+  end
+end
+

data/spec/headless/rails_spec.rb

@@ -4,68 +4,4 @@ RSpec.describe Headless::Rails do
   it 'should have a version number' do
     Headless::Rails::VERSION.should_not be_nil
   end
-
-  let(:controller) do
-    double(:controller,
-      :request => request,
-      :params => {},
-      :render => nil
-    )
-  end
-
-  let(:request) { request_for_uri("/user/matthewrudy?foo=bar") }
-  let(:headless_response) { double(:headless_response, :success? => true, :content => :headless_content) }
-
-  describe "respond_to_ajax_crawlers" do
-    before do
-      controller.extend(Headless::Rails)
-    end
-
-    context "with no escaped_fragment" do
-
-      let(:request) { request_for_uri("/user/matthewrudy?foo=bar") }
-
-      before do
-        controller.stub(:params).and_return({})
-      end
-
-      it "doesn't call render" do
-        expect(controller).to_not receive(:render)
-
-        controller.respond_to_ajax_crawlers
-      end
-
-      it "returns nil" do
-        value = controller.respond_to_ajax_crawlers
-        expect(value).to be_nil
-      end
-    end
-
-    context "with an escaped fragment" do
-
-      let(:request) { request_for_uri("/user/matthewrudy?foo=bar&_escaped_fragment_=/show/twitter") }
-
-      before do
-        expect(Headless::APIClient).to receive(:crawl).with("http://localhost:3000/user/matthewrudy?foo=bar#/show/twitter").and_return(headless_response)
-      end
-
-      it "extracts the fragment, fetches content from the api, and renders its" do
-        expect(controller).to receive(:render).with(:text => :headless_content)
-
-        controller.respond_to_ajax_crawlers
-      end
-
-      context "failure" do
-        before do
-          headless_response.stub(:success?).and_return(false)
-        end
-
-        it "doesn't render" do
-          expect(controller).to_not receive(:render)
-
-          controller.respond_to_ajax_crawlers
-        end
-      end
-    end
-  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: headless-rails
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.4
 platform: ruby
 authors:
 - Matthew Rudy Jacobs
@@ -94,6 +94,20 @@ dependencies:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: rack-test
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 description: ' Seamlessly integrate headlessapp.com into your Rails application '
 email:
 - MatthewRudyJacobs@gmail.com
@@ -110,12 +124,14 @@ files:
 - Rakefile
 - headless-rails.gemspec
 - lib/headless/ajax_crawler.rb
+- lib/headless/ajax_crawler/middleware.rb
 - lib/headless/ajax_crawler/request_matcher.rb
 - lib/headless/ajax_crawler/url_extractor.rb
 - lib/headless/api_client.rb
 - lib/headless/api_response.rb
 - lib/headless/rails.rb
 - lib/headless/rails/version.rb
+- spec/headless/ajax_crawler/middleware_spec.rb
 - spec/headless/ajax_crawler/request_matcher_spec.rb
 - spec/headless/ajax_crawler/url_extractor_spec.rb
 - spec/headless/ap_response_spec.rb
@@ -148,6 +164,7 @@ signing_key:
 specification_version: 4
 summary: API wrapper for the HeadlessApp.com
 test_files:
+- spec/headless/ajax_crawler/middleware_spec.rb
 - spec/headless/ajax_crawler/request_matcher_spec.rb
 - spec/headless/ajax_crawler/url_extractor_spec.rb
 - spec/headless/ap_response_spec.rb