keynote 0.0.1

data/lib/keynote/testing/test_unit.rb

@@ -0,0 +1,6 @@
+# encoding: UTF-8
+
+module Keynote
+  class TestCase < ::ActionView::TestCase
+  end
+end

data/lib/keynote/version.rb

@@ -0,0 +1,6 @@
+# encoding: UTF-8
+
+module Keynote
+  # @private
+  VERSION = "0.0.1"
+end

data/spec/generator_spec.rb

@@ -0,0 +1,106 @@
+# encoding: UTF-8
+
+require 'helper'
+require 'rails/generators'
+require 'fileutils'
+
+describe "generators" do
+  def invoke_generator(*args)
+    FileUtils.mkdir_p(output_path)
+
+    args.push '--quiet'
+
+    retval = Rails::Generators.invoke 'presenter', args,
+      :behavior => :invoke,
+      :destination_root => output_path
+
+    assert retval, 'Generator must succeed'
+    yield retval.drop(2)
+  ensure
+    FileUtils.rm_rf(output_path)
+  end
+
+  def output_path
+    File.expand_path('../../tmp', __FILE__)
+  end
+
+  def file_contents(path)
+    file_contents = File.read(File.join(output_path, path))
+  end
+
+  it "should generate a presenter and Test::Unit file" do
+    Rails.application.config.generators do |g|
+      g.test_framework :test_unit, :spec => true
+    end
+
+    invoke_generator 'post' do |files|
+      files.must_equal %w(
+        app/presenters/post_presenter.rb
+        test/unit/presenters/post_presenter_test.rb
+      )
+
+      file_contents('app/presenters/post_presenter.rb').
+        must_match /class PostPresenter < Keynote::Presenter/
+
+      file_contents('test/unit/presenters/post_presenter_test.rb').
+        must_match /class PostPresenterTest < Keynote::TestCase/
+    end
+  end
+
+  it "should generate a presenter and RSpec file" do
+    Rails.application.config.generators do |g|
+      g.test_framework :rspec
+    end
+
+    invoke_generator 'post' do |files|
+      files.must_equal %w(
+        app/presenters/post_presenter.rb
+        spec/presenters/post_presenter_spec.rb
+      )
+
+      file_contents('app/presenters/post_presenter.rb').
+        must_match /class PostPresenter < Keynote::Presenter/
+
+      file_contents('spec/presenters/post_presenter_spec.rb').
+        must_match /describe PostPresenter do/
+    end
+  end
+
+  it "should generate a presenter and MiniTest::Rails spec file" do
+    Rails.application.config.generators do |g|
+      g.test_framework :mini_test, :spec => true
+    end
+
+    invoke_generator 'post' do |files|
+      files.must_equal %w(
+        app/presenters/post_presenter.rb
+        test/presenters/post_presenter_test.rb
+      )
+
+      file_contents('app/presenters/post_presenter.rb').
+        must_match /class PostPresenter < Keynote::Presenter/
+
+      file_contents('test/presenters/post_presenter_test.rb').
+        must_match /describe PostPresenter do/
+    end
+  end
+
+  it "should generate a presenter and MiniTest::Rails unit file" do
+    Rails.application.config.generators do |g|
+      g.test_framework :mini_test, :spec => false
+    end
+
+    invoke_generator 'post' do |files|
+      files.must_equal %w(
+        app/presenters/post_presenter.rb
+        test/presenters/post_presenter_test.rb
+      )
+
+      file_contents('app/presenters/post_presenter.rb').
+        must_match /class PostPresenter < Keynote::Presenter/
+
+      file_contents('test/presenters/post_presenter_test.rb').
+        must_match /class PostPresenterTest < Keynote::MiniTest::TestCase/
+    end
+  end
+end

data/spec/helper.rb

@@ -0,0 +1,84 @@
+# encoding: UTF-8
+
+require 'minitest/spec'
+require 'minitest/autorun'
+require 'mocha'
+
+require 'pry'
+
+require 'rails'
+require 'action_controller/railtie'
+require 'action_mailer/railtie'
+require 'rails/test_unit/railtie'
+
+require 'keynote'
+
+## Initialize our test app (by Jose Valim: https://gist.github.com/1942658)
+
+class TestApp < Rails::Application
+  config.active_support.deprecation = :log
+  config.eager_load = false
+
+  config.secret_token = 'a' * 100
+end
+
+class HelloController < ActionController::Base
+  def world
+    render :text => "Hello world!", :layout => false
+  end
+end
+
+TestApp.initialize!
+
+# We have to define this class because it's hard-coded into the definition of
+# ActiveSupport::TestCase, which will load regardless of whether we load
+# ActiveRecord.
+module ActiveRecord
+  class Model
+  end
+end
+
+## Examples
+
+class EmptyPresenter < Keynote::Presenter
+end
+
+class NormalPresenter < Keynote::Presenter
+  presents :model
+
+  def some_bad_js
+    "<script>alert('pwnt');</script>"
+  end
+
+  def some_bad_html
+    html do
+      div { text some_bad_js }
+      div { some_bad_js }
+      div some_bad_js
+    end
+  end
+end
+
+class Normal
+end
+
+module Keynote
+  class NestedPresenter < Keynote::Presenter
+    presents :model
+
+    def generate_div
+      html do
+        div.hi! do
+          link_to '#', 'Hello'
+        end
+      end
+    end
+  end
+
+  class Nested
+  end
+end
+
+class CombinedPresenter < Keynote::Presenter
+  presents :model_1, :model_2
+end

data/spec/keynote_spec.rb

@@ -0,0 +1,98 @@
+# encoding: UTF-8
+
+require 'helper'
+
+describe Keynote do
+  describe "with a normal presenter" do
+    let(:model) { Normal.new }
+
+    it "should find and instantiate implicitly" do
+      p = Keynote.present(:view, model)
+
+      p.wont_be_nil
+      p.must_be_instance_of NormalPresenter
+
+      p.view.must_equal  :view
+      p.model.must_equal model
+    end
+
+    it "should find and instantiate explicitly" do
+      p = Keynote.present(:view, :normal, 'hello')
+
+      p.wont_be_nil
+      p.must_be_instance_of NormalPresenter
+
+      p.view.must_equal  :view
+      p.model.must_equal 'hello'
+    end
+
+    it "should integrate with Rumble" do
+      p  = Keynote.present(:view, model)
+      rx = /<div>&lt;script&gt;alert\(/
+
+      p.some_bad_html.scan(rx).count.must_equal 3
+    end
+  end
+
+  describe "with a nested presenter" do
+    let(:model) { Keynote::Nested.new }
+
+    it "should find and instantiate implicitly" do
+      p = Keynote.present(:view, model)
+
+      p.wont_be_nil
+      p.must_be_instance_of Keynote::NestedPresenter
+
+      p.view.must_equal  :view
+      p.model.must_equal model
+    end
+
+    it "should find and instantiate explicitly" do
+      p = Keynote.present(:view, :"keynote/nested", 'hello')
+
+      p.wont_be_nil
+      p.must_be_instance_of Keynote::NestedPresenter
+
+      p.view.must_equal  :view
+      p.model.must_equal 'hello'
+    end
+  end
+
+  describe "memoization" do
+    it "should memoize on the model instance(s)" do
+      model_1 = Normal.new
+      model_2 = Normal.new
+
+      presented_1 = Keynote.present(:view, model_1)
+      presented_2 = Keynote.present(:view, model_1)
+
+      presented_1.must_be :equal?, presented_2
+
+      presented_3 = Keynote.present(:view, :combined, model_1, model_2)
+      presented_4 = Keynote.present(:view, :combined, model_1, model_2)
+      presented_5 = Keynote.present(:view, :combined, model_2, model_1)
+
+      presented_3.wont_be :equal?, presented_1
+      presented_3.must_be :equal?, presented_4
+      presented_3.wont_be :equal?, presented_5
+    end
+
+    it "shouldn't memoize if there are no models" do
+      presenter_1 = Keynote.present(:view, :empty)
+      presenter_2 = Keynote.present(:view, :empty)
+
+      presenter_1.wont_be :equal?, presenter_2
+    end
+
+    it "should update the view every time" do
+      model = Normal.new
+
+      presenter_1 = Keynote.present(:view, model)
+      presenter_1.view.must_equal :view
+
+      presenter_2 = Keynote.present(:view_2, model)
+      presenter_2.must_be :equal?, presenter_1
+      presenter_2.view.must_equal :view_2
+    end
+  end
+end

data/spec/presenter_spec.rb

@@ -0,0 +1,116 @@
+# encoding: UTF-8
+
+require 'helper'
+
+TestPresenter = Keynote::Presenter
+
+describe Keynote::Presenter do
+  describe "delegation" do
+    let(:klass) do
+      Class.new(TestPresenter) do
+        presents :grizzly, :bear
+        delegate :adams, :to => :grizzly
+        delegate :man, :to => :grizzly, :prefix => true
+      end
+    end
+
+    it "should be able to use ActiveSupport Module#delegate method" do
+      mock = mock()
+      mock.expects(:adams)
+      mock.expects(:man)
+
+      klass.new(nil, mock, nil).tap do |p|
+        p.adams
+        p.grizzly_man
+      end
+    end
+  end
+
+  describe ".presents" do
+    it "should take just the view context by default" do
+      klass = Class.new(TestPresenter)
+
+      klass.new(1).instance_variable_get(:@view).must_equal 1
+    end
+
+    describe "with two parameters" do
+      let(:klass) do
+        Class.new(TestPresenter) do
+          presents :grizzly, :bear
+        end
+      end
+
+      it "should let you specify other objects to take" do
+        klass.new(1, 2, 3).instance_eval do
+          @view.must_equal 1
+          @grizzly.must_equal 2
+          @bear.must_equal 3
+        end
+      end
+
+      it "should not be callable with the wrong arity" do
+        proc { klass.new(1, 2) }.must_raise ArgumentError
+      end
+
+      it "should generate readers for the objects" do
+        klass.new(1, 2, 3).tap do |p|
+          p.view.must_equal 1
+          p.grizzly.must_equal 2
+          p.bear.must_equal 3
+        end
+      end
+    end
+  end
+
+  describe "#present" do
+    it "should pass its view context through to the new presenter" do
+      mock = mock()
+      mock.expects(:pizza)
+
+      p1 = TestPresenter.new(mock)
+      p2 = p1.present(:test)
+
+      p1.wont_equal p2
+      p2.pizza
+    end
+  end
+
+  describe "#method_missing" do
+    it "should pass unknown method calls through to the view" do
+      mock = mock()
+      mock.expects(:talking).with(:heads)
+
+      object = Class.new do
+        define_method(:talking) do |arg|
+          mock.talking(arg)
+        end
+        private :talking
+      end.new
+
+      TestPresenter.new(object).talking(:heads)
+    end
+
+    it "should respond_to? methods of the view" do
+      object = Class.new do
+        define_method(:talking) do |arg|
+        end
+        private :talking
+      end.new
+
+      TestPresenter.new(object).respond_to?(:talking).must_equal true
+    end
+
+    it "should raise unknown methods at the presenter, not the view" do
+      err = nil
+
+      begin
+        TestPresenter.new(Object.new).talking(:heads)
+      rescue NoMethodError => e
+        err = e
+      end
+
+      err.wont_be_nil
+      err.message.must_match /#<Keynote::Presenter:/
+    end
+  end
+end

data/spec/railtie_spec.rb

@@ -0,0 +1,32 @@
+# encoding: UTF-8
+
+require 'helper'
+
+describe Keynote::Railtie do
+  let(:controller) { HelloController.new }
+  let(:context)    { controller.view_context }
+
+  it "should make the present method available to controllers" do
+    controller.must_respond_to :present
+  end
+
+  it "should make the present and p methods available to views" do
+    context.must_respond_to :present
+    context.must_respond_to :p
+  end
+
+  it "should pass present call from controller to Keynote.present" do
+    context = stub
+    controller.stubs(:view_context).returns(context)
+
+    Keynote.expects(:present).with(context, :dallas, :leeloo, :multipass)
+
+    controller.present(:dallas, :leeloo, :multipass)
+  end
+
+  it "should pass present call from view to Keynote.present" do
+    Keynote.expects(:present).with(context, :dallas, :leeloo, :multipass)
+
+    context.present(:dallas, :leeloo, :multipass)
+  end
+end