landable 1.7.0

data/spec/controllers/landable/api_controller_spec.rb

@@ -0,0 +1,189 @@
+require 'spec_helper'
+
+describe Landable::ApiController, json: true do
+  controller(Landable::ApiController) do
+    skip_before_filter :require_author!, except: [:index]
+
+    def index
+      render nothing: true
+    end
+
+    def ok
+      render nothing: true
+    end
+
+    def responder
+      respond_with @resource
+    end
+
+    def update_responder
+      respond_with @resource
+    end
+
+    def not_found
+      raise ActiveRecord::RecordNotFound
+    end
+
+    def xml_only
+      respond_to do |format|
+        format.xml { render xml: '<lol/>' }
+      end
+    end
+
+    def record_invalid
+      Landable::Page.create!
+    end
+
+    def uuid_invalid
+      Landable::Page.find('1')
+    end
+
+    def other_pg_error
+      ActiveRecord::Base.connection.execute "LOL THIS IS NOT SQL AT ALL GUYS!"
+    end
+  end
+
+  before do
+    routes.draw do
+      get 'index' => 'anonymous#index'
+      get 'ok' => 'anonymous#ok'
+      get 'responder' => 'anonymous#responder'
+      patch 'responder' => 'anonymous#responder'
+      put 'responder' => 'anonymous#responder'
+      get 'not_found' => 'anonymous#not_found'
+      get 'xml_only'  => 'anonymous#xml_only'
+      get 'record_invalid' => 'anonymous#record_invalid'
+      get 'uuid_invalid' => 'anonymous#uuid_invalid'
+      get 'other_pg_error' => 'anonymous#other_pg_error'
+    end
+  end
+
+  describe 'access token' do
+    let(:author) { create :author }
+    let(:token)  { create :access_token, author: author }
+    let(:headers) do
+      { 'HTTP_AUTHORIZATION' => encode_basic_auth(author.username, token.id) }
+    end
+
+    def make_request(params = nil)
+      request.env.merge!(headers)
+      get :index, params
+    end
+
+    it "sets current_author when valid" do
+      make_request
+      controller.send(:current_author).should == author
+    end
+
+    it "must be in the HTTP Authorization header" do
+      headers.delete 'HTTP_AUTHORIZATION'
+      make_request access_token: token.id
+      response.status.should == 401
+    end
+
+    it "must belong to a valid Author username" do
+      headers['HTTP_AUTHORIZATION'] = encode_basic_auth('wrong-username', token.id)
+      make_request
+      response.status.should == 401
+    end
+
+    it "must be an existing token ID" do
+      headers['HTTP_AUTHORIZATION'] = encode_basic_auth(author.username, token.id.reverse)
+      make_request
+      response.status.should == 401
+    end
+
+    it "must not be expired" do
+      token.update_attributes(expires_at: 1.minute.ago)
+      make_request
+      response.status.should == 401
+    end
+  end
+
+  context 'rescues RecordNotFound' do
+    it 'returns 404 Not Found' do
+      get :not_found
+      response.status.should == 404
+    end
+  end
+
+  context 'rescues RecordInvalid' do
+    it 'returns 422 Unprocessable Entity' do
+      get :record_invalid
+      response.status.should == 422
+    end
+
+    it 'renders ActiveModel::Errors as the JSON response' do
+      get :record_invalid
+      last_json['errors'].should have_key('path')
+    end
+  end
+
+  context 'rescues UnknownFormat' do
+    it 'returns 406 Not Acceptable' do
+      request.env['HTTP_ACCEPT'] = 'text/plain'
+      get :xml_only
+      response.status.should == 406
+    end
+  end
+
+  context 'rescues PG::Errors about invalid UUIDs' do
+    it 'returns 404' do
+      get :uuid_invalid
+      response.status.should == 404
+    end
+
+    it 're-raises any other PG::Error' do
+      expect {
+        get :other_pg_error
+      }.to raise_error(PG::Error)
+    end
+  end
+
+  describe '#api_media' do
+    let(:headers) { {} }
+
+    it 'should match the request format and api version' do
+      request.env['HTTP_ACCEPT'] = 'application/xml'
+
+      get :ok
+
+      # sanity check
+      request.format.symbol.should == :xml
+
+      controller.api_media.should == {
+        format: request.format.symbol,
+        version: Landable::VERSION::STRING,
+        param: nil,
+      }
+    end
+  end
+
+  describe 'responder' do
+    before(:each) do
+      controller.instance_variable_set :@resource, resource
+    end
+
+    let(:resource) { build :author }
+
+    it 'should set X-Landable-Media-Type' do
+      get :responder
+      response.status.should == 200
+      response.headers['X-Landable-Media-Type'].should == "landable.v#{Landable::VERSION::STRING}; format=json"
+    end
+
+    context 'patch' do
+      it 'should display the resource' do
+        put :responder
+        response.body.should_not be_empty
+      end
+    end
+
+    context 'put' do
+      it 'should display the resource' do
+        put :responder
+        response.body.should_not be_empty
+      end
+    end
+  end
+end

data/spec/controllers/public/preview/page_revisions_controller_spec.rb

@@ -0,0 +1,41 @@
+require 'spec_helper'
+
+module Landable::Public::Preview
+  describe PageRevisionsController do
+    routes { Landable::Engine.routes }
+
+    describe '#show' do
+
+      let(:author) { create :author }
+      let(:page) { create :page, body: '<p>hello</p>' }
+      let(:page_revision) do
+        page.publish! author: author
+        page.revisions.first
+      end
+
+      before(:each) do
+        # establish the tested-for revision
+        page_revision
+
+        2.times do |i|
+          page.update_attributes! body: "update #{i}"
+          page.publish! author: author
+        end
+      end
+
+      def make_request
+        get :show, id: page_revision.id
+      end
+
+      it 'renders the page revision' do
+        make_request
+        response.body.should include '<p>hello</p>'
+      end
+
+      it 'is available at /-/pr/:id' do
+        assert_recognizes({controller: 'landable/public/preview/page_revisions', action: 'show', id: page_revision.id}, "/-/pr/#{page_revision.id}")
+      end
+
+    end
+  end
+end

data/spec/controllers/public/preview/pages_controller_spec.rb

@@ -0,0 +1,36 @@
+require 'spec_helper'
+
+module Landable::Public::Preview
+  describe PagesController do
+    routes { Landable::Engine.routes }
+
+    describe '#show' do
+
+      let(:page) { create :page, body: '<p>hello</p>' }
+
+      before(:each) do
+        page.publish! author: create(:author)
+        page.update_attributes! body: '<p>why hello there</p>'
+      end
+
+      def make_request
+        get :show, id: page.id
+      end
+
+      it 'renders the page in situ' do
+        make_request
+        response.body.should include '<p>why hello there</p>'
+      end
+
+      it 'renders the preview message' do
+        make_request
+        response.body.should include 'Test your unpublished changes with this page. Share this URL!'
+      end
+
+      it 'is available at /-/p/:id' do
+        assert_recognizes({controller: 'landable/public/preview/pages', action: 'show', id: page.id}, "/-/p/#{page.id}")
+      end
+
+    end
+  end
+end

data/spec/controllers/public/sitemap_controller_spec.rb

@@ -0,0 +1,25 @@
+require 'spec_helper'
+
+module Landable::Public
+  describe SitemapController do
+    routes { Landable::Engine.routes }
+
+    it 'returns a 200 status' do
+      get :index
+      response.status.should == 200
+    end
+
+    it 'returns xml' do
+      get :index
+      response.content_type.should == 'application/xml'
+    end
+
+    it 'calls generate_sitemap with appropriate arguments' do
+      Landable::Page.should_receive(:generate_sitemap).with(host: 'test.host', 
+                                                            protocol: 'https', 
+                                                            exclude_categories: ['Testing'], 
+                                                            sitemap_additional_paths: ['/terms.html']).and_call_original
+      get :index, format: :xml
+    end
+  end
+end

data/spec/decorators/page_decorator_spec.rb

@@ -0,0 +1,90 @@
+require 'spec_helper'
+
+module Landable
+  describe PageDecorator do
+    let(:page_decorator) { Landable::PageDecorator.new(page) }
+
+    describe '#title' do
+      let(:page) { create :page, title: 'title'}
+
+      it 'lists the title' do
+        page_decorator.title.should == "<title>title</title>"
+      end
+
+      context 'nil' do
+        let(:page) { create :page, title: nil }
+
+        it 'returns nil' do
+          page_decorator.title.should be_nil
+        end
+      end
+    end
+
+    describe '#path' do
+      let(:page) { create :page, path: "/landable" }
+
+      it 'lists the path' do
+        page_decorator.path.should == "/landable"
+      end
+    end
+
+    describe '#body' do
+      let(:page) { create :page, body: "Buy pounds for your pocket" }
+
+      it 'lists the body' do
+        page_decorator.body.should == "Buy pounds for your pocket"
+        page_decorator.body.should be_html_safe
+      end
+
+      context 'nil' do
+        let(:page) { create :page, head_content: nil }
+
+        it 'returns nil' do
+          page_decorator.head_content.should be_nil
+        end
+      end
+    end
+
+    describe '#head_content' do
+      let(:page) { create :page, head_content: "<head lang='en' />" }
+
+      it 'lists the head_tags seperated by a new line' do
+        page_decorator.head_content.should == "<head lang='en' />"
+        page_decorator.head_content.should be_html_safe
+      end
+
+      context 'nil' do
+        let(:page) { create :page, head_content: nil }
+
+        it 'returns nil' do
+          page_decorator.head_content.should be_nil
+        end
+      end
+    end
+
+    describe '#meta_tags' do
+      let(:page) { create :page, meta_tags: { content: 'robots', keyword: 'p2p' } }
+
+      it 'lists the meta_tags seperated by a new line' do
+        page_decorator.meta_tags.should == %Q(<meta content="robots" name="content" />\n<meta content="p2p" name="keyword" />)
+        page_decorator.meta_tags.should be_html_safe
+      end
+
+      context 'nil' do
+        let(:page) { create :page, meta_tags: nil }
+
+        it 'returns nil' do
+          page_decorator.meta_tags.should be_nil
+        end
+      end
+
+      context 'string' do
+        let(:page) { create :page, meta_tags: "I should be a hash!" }
+
+        it 'returns nil' do
+          page_decorator.meta_tags.should be_nil
+        end
+      end
+    end
+  end
+end

data/spec/dummy/README.rdoc

@@ -0,0 +1,28 @@
+== README
+
+This README would normally document whatever steps are necessary to get the
+application up and running.
+
+Things you may want to cover:
+
+* Ruby version
+
+* System dependencies
+
+* Configuration
+
+* Database creation
+
+* Database initialization
+
+* How to run the test suite
+
+* Services (job queues, cache servers, search engines, etc.)
+
+* Deployment instructions
+
+* ...
+
+
+Please feel free to use a different markup language if you do not plan to run
+<tt>rake doc:app</tt>.

data/spec/dummy/Rakefile

@@ -0,0 +1,6 @@
+# Add your own tasks in files placed in lib/tasks ending in .rake,
+# for example lib/tasks/capistrano.rake, and they will automatically be available to Rake.
+
+require File.expand_path('../config/application', __FILE__)
+
+Dummy::Application.load_tasks

data/spec/dummy/app/assets/javascripts/application.js

@@ -0,0 +1,13 @@
+// This is a manifest file that'll be compiled into application.js, which will include all the files
+// listed below.
+//
+// Any JavaScript/Coffee file within this directory, lib/assets/javascripts, vendor/assets/javascripts,
+// or vendor/assets/javascripts of plugins, if any, can be referenced here using a relative path.
+//
+// It's not advisable to add code directly here, but if you do, it'll appear at the bottom of the
+// compiled file.
+//
+// WARNING: THE FIRST BLANK LINE MARKS THE END OF WHAT'S TO BE PROCESSED, ANY BLANK LINE SHOULD
+// GO AFTER THE REQUIRES BELOW.
+//
+//= require_tree .

data/spec/dummy/app/assets/stylesheets/application.css

@@ -0,0 +1,13 @@
+/*
+ * This is a manifest file that'll be compiled into application.css, which will include all the files
+ * listed below.
+ *
+ * Any CSS and SCSS file within this directory, lib/assets/stylesheets, vendor/assets/stylesheets,
+ * or vendor/assets/stylesheets of plugins, if any, can be referenced here using a relative path.
+ *
+ * You're free to add application-wide styles to this file and they'll appear at the top of the
+ * compiled file, but it's generally better to create a new file per style scope.
+ *
+ *= require_self
+ *= require_tree .
+ */