draper_new 3.0.0

data/spec/dummy/spec/shared_examples/decoratable.rb

@@ -0,0 +1,24 @@
+shared_examples_for "a decoratable model" do
+  describe ".decorate" do
+    it "applies a collection decorator to a scope" do
+      described_class.create
+      decorated = described_class.limit(1).decorate
+
+      expect(decorated.size).to eq(1)
+      expect(decorated).to be_decorated
+    end
+  end
+
+  describe "#==" do
+    it "is true for other instances' decorators" do
+      pending "Mongoid < 3.1 overrides `#==`" if defined?(Mongoid) && Mongoid::VERSION.to_f < 3.1 && described_class < Mongoid::Document
+
+      described_class.create
+      one = described_class.first
+      other = described_class.first
+
+      expect(one).not_to be other
+      expect(one == other.decorate).to be_truthy
+    end
+  end
+end

data/spec/dummy/spec/spec_helper.rb

@@ -0,0 +1,8 @@
+ENV['RAILS_ENV'] ||= 'test'
+require File.expand_path('../../config/environment', __FILE__)
+require 'rspec/rails'
+
+RSpec.configure do |config|
+  config.expect_with(:rspec) {|c| c.syntax = :expect}
+  config.order = :random
+end

data/spec/dummy/test/decorators/minitest/devise_test.rb

@@ -0,0 +1,64 @@
+require 'minitest_helper'
+
+if defined?(Devise)
+  describe "A decorator test" do
+    it "can sign in a real user" do
+      user = User.new
+      sign_in user
+
+      assert_same user, helper.current_user
+    end
+
+    it "can sign in a mock user" do
+      user = Object.new
+      sign_in :user, user
+
+      assert_same user, helper.current_user
+    end
+
+    it "can sign in a real admin" do
+      admin = Admin.new
+      sign_in admin
+
+      assert_same admin, helper.current_admin
+    end
+
+    it "can sign in a mock admin" do
+      admin = Object.new
+      sign_in :admin, admin
+
+      assert_same admin, helper.current_admin
+    end
+
+    it "can sign out a real user" do
+      user = User.new
+      sign_in user
+      sign_out user
+
+      assert helper.current_user.nil?
+    end
+
+    it "can sign out a mock user" do
+      user = Object.new
+      sign_in :user, user
+      sign_out :user
+
+      assert helper.current_user.nil?
+    end
+
+    it "can sign out without a user" do
+      sign_out :user
+
+      assert helper.current_user.nil?
+    end
+
+    it "is backwards-compatible" do
+      user = Object.new
+      ActiveSupport::Deprecation.silence do
+        sign_in user
+      end
+
+      assert_same user, helper.current_user
+    end
+  end
+end

data/spec/dummy/test/decorators/minitest/helpers_test.rb

@@ -0,0 +1,21 @@
+require 'minitest_helper'
+
+describe "A decorator test" do
+  it "can access helpers through `helper`" do
+    assert_equal "<p>Help!</p>", helper.content_tag(:p, "Help!")
+  end
+
+  it "can access helpers through `helpers`" do
+    assert_equal "<p>Help!</p>", helpers.content_tag(:p, "Help!")
+  end
+
+  it "can access helpers through `h`" do
+    assert_equal "<p>Help!</p>", h.content_tag(:p, "Help!")
+  end
+
+  it "gets the same helper object as a decorator" do
+    decorator = Draper::Decorator.new(Object.new)
+
+    assert_same decorator.helpers, helpers
+  end
+end

data/spec/dummy/test/decorators/minitest/spec_type_test.rb

@@ -0,0 +1,52 @@
+require 'minitest_helper'
+
+def it_is_a_decorator_test
+  it "is a decorator test" do
+    assert_kind_of Draper::TestCase, self
+  end
+end
+
+def it_is_not_a_decorator_test
+  it "is not a decorator test" do
+    refute_kind_of Draper::TestCase, self
+  end
+end
+
+ProductDecorator = Class.new(Draper::Decorator)
+ProductsDecorator = Class.new(Draper::CollectionDecorator)
+
+describe ProductDecorator do
+  it_is_a_decorator_test
+end
+
+describe ProductsDecorator do
+  it_is_a_decorator_test
+end
+
+describe "ProductDecorator" do
+  it_is_a_decorator_test
+end
+
+describe "AnyDecorator" do
+  it_is_a_decorator_test
+end
+
+describe "Any decorator" do
+  it_is_a_decorator_test
+end
+
+describe "AnyDecoratorTest" do
+  it_is_a_decorator_test
+end
+
+describe "Any decorator test" do
+  it_is_a_decorator_test
+end
+
+describe Object do
+  it_is_not_a_decorator_test
+end
+
+describe "Nope" do
+  it_is_not_a_decorator_test
+end

data/spec/dummy/test/decorators/minitest/view_context_test.rb

@@ -0,0 +1,24 @@
+require 'minitest_helper'
+
+def it_does_not_leak_view_context
+  2.times do
+    it "has an independent view context" do
+      refute_equal :leaked, Draper::ViewContext.current
+      Draper::ViewContext.current = :leaked
+    end
+  end
+end
+
+describe "A decorator test" do
+  it_does_not_leak_view_context
+end
+
+describe "A controller test" do
+  tests Class.new(ActionController::Base)
+
+  it_does_not_leak_view_context
+end
+
+describe "A mailer test" do
+  it_does_not_leak_view_context
+end

data/spec/dummy/test/decorators/test_unit/devise_test.rb

@@ -0,0 +1,64 @@
+require 'test_helper'
+
+if defined?(Devise)
+  class DeviseTest < Draper::TestCase
+    def test_sign_in_a_real_user
+      user = User.new
+      sign_in user
+
+      assert_same user, helper.current_user
+    end
+
+    def test_sign_in_a_mock_user
+      user = Object.new
+      sign_in :user, user
+
+      assert_same user, helper.current_user
+    end
+
+    def test_sign_in_a_real_admin
+      admin = Admin.new
+      sign_in admin
+
+      assert_same admin, helper.current_admin
+    end
+
+    def test_sign_in_a_mock_admin
+      admin = Object.new
+      sign_in :admin, admin
+
+      assert_same admin, helper.current_admin
+    end
+
+    def test_sign_out_a_real_user
+      user = User.new
+      sign_in user
+      sign_out user
+
+      assert helper.current_user.nil?
+    end
+
+    def test_sign_out_a_mock_user
+      user = Object.new
+      sign_in :user, user
+      sign_out :user
+
+      assert helper.current_user.nil?
+    end
+
+    def test_sign_out_without_a_user
+      sign_out :user
+
+      assert helper.current_user.nil?
+    end
+
+    def test_backwards_compatibility
+      user = Object.new
+      ActiveSupport::Deprecation.silence do
+        sign_in user
+      end
+
+      assert_same user, helper.current_user
+    end
+  end
+end

data/spec/dummy/test/decorators/test_unit/helpers_test.rb

@@ -0,0 +1,21 @@
+require 'test_helper'
+
+class HelpersTest < Draper::TestCase
+  def test_access_helpers_through_helper
+    assert_equal "<p>Help!</p>", helper.content_tag(:p, "Help!")
+  end
+
+  def test_access_helpers_through_helpers
+    assert_equal "<p>Help!</p>", helpers.content_tag(:p, "Help!")
+  end
+
+  def test_access_helpers_through_h
+    assert_equal "<p>Help!</p>", h.content_tag(:p, "Help!")
+  end
+
+  def test_same_helper_object_as_decorators
+    decorator = Draper::Decorator.new(Object.new)
+
+    assert_same decorator.helpers, helpers
+  end
+end

data/spec/dummy/test/decorators/test_unit/view_context_test.rb

@@ -0,0 +1,24 @@
+require 'test_helper'
+
+def it_does_not_leak_view_context
+  2.times do |n|
+    define_method("test_has_independent_view_context_#{n}") do
+      refute_equal :leaked, Draper::ViewContext.current
+      Draper::ViewContext.current = :leaked
+    end
+  end
+end
+
+class DecoratorTest < Draper::TestCase
+  it_does_not_leak_view_context
+end
+
+class ControllerTest < ActionController::TestCase
+  tests Class.new(ActionController::Base)
+
+  it_does_not_leak_view_context
+end
+
+class MailerTest < ActionMailer::TestCase
+  it_does_not_leak_view_context
+end

data/spec/dummy/test/minitest_helper.rb

@@ -0,0 +1,2 @@
+require 'test_helper'
+require 'minitest/rails'

data/spec/dummy/test/test_helper.rb

@@ -0,0 +1,3 @@
+ENV['RAILS_ENV'] ||= 'test'
+require File.expand_path('../../config/environment', __FILE__)
+require 'rails/test_help'

data/spec/generators/controller/controller_generator_spec.rb

@@ -0,0 +1,22 @@
+require 'spec_helper'
+require 'rails'
+require 'ammeter/init'
+require 'generators/controller_override'
+require 'generators/rails/decorator_generator'
+
+describe Rails::Generators::ControllerGenerator do
+  destination File.expand_path("../tmp", __FILE__)
+
+  before { prepare_destination }
+  after(:all) { FileUtils.rm_rf destination_root }
+
+  describe "the generated decorator" do
+    subject { file("app/decorators/your_model_decorator.rb") }
+
+    describe "naming" do
+      before { run_generator %w(YourModels) }
+
+      it { should contain "class YourModelDecorator" }
+    end
+  end
+end

data/spec/generators/decorator/decorator_generator_spec.rb

@@ -0,0 +1,92 @@
+require 'spec_helper'
+require 'rails'
+require 'ammeter/init'
+require 'generators/rails/decorator_generator'
+
+describe Rails::Generators::DecoratorGenerator do
+  destination File.expand_path("../tmp", __FILE__)
+
+  before { prepare_destination }
+  after(:all) { FileUtils.rm_rf destination_root }
+
+  describe "the generated decorator" do
+    subject { file("app/decorators/your_model_decorator.rb") }
+
+    describe "naming" do
+      before { run_generator %w(YourModel) }
+
+      it { should contain "class YourModelDecorator" }
+    end
+
+    describe "namespacing" do
+      subject { file("app/decorators/namespace/your_model_decorator.rb") }
+      before { run_generator %w(Namespace::YourModel) }
+
+      it { should contain "class Namespace::YourModelDecorator" }
+    end
+
+    describe "inheritance" do
+      context "by default" do
+        before { run_generator %w(YourModel) }
+
+        it { should contain "class YourModelDecorator < Draper::Decorator" }
+      end
+
+      context "with the --parent option" do
+        before { run_generator %w(YourModel --parent=FooDecorator) }
+
+        it { should contain "class YourModelDecorator < FooDecorator" }
+      end
+
+      context "with an ApplicationDecorator" do
+        before do
+          allow_any_instance_of(Object).to receive(:require).with("application_decorator").and_return(
+            stub_const "ApplicationDecorator", Class.new
+          )
+        end
+
+        before { run_generator %w(YourModel) }
+
+        it { should contain "class YourModelDecorator < ApplicationDecorator" }
+      end
+    end
+  end
+
+  context "with -t=rspec" do
+    describe "the generated spec" do
+      subject { file("spec/decorators/your_model_decorator_spec.rb") }
+
+      describe "naming" do
+        before { run_generator %w(YourModel -t=rspec) }
+
+        it { should contain "describe YourModelDecorator" }
+      end
+
+      describe "namespacing" do
+        subject { file("spec/decorators/namespace/your_model_decorator_spec.rb") }
+        before { run_generator %w(Namespace::YourModel -t=rspec) }
+
+        it { should contain "describe Namespace::YourModelDecorator" }
+      end
+    end
+  end
+
+  context "with -t=test_unit" do
+    describe "the generated test" do
+      subject { file("test/decorators/your_model_decorator_test.rb") }
+
+      describe "naming" do
+        before { run_generator %w(YourModel -t=test_unit) }
+
+        it { should contain "class YourModelDecoratorTest < Draper::TestCase" }
+      end
+
+      describe "namespacing" do
+        subject { file("test/decorators/namespace/your_model_decorator_test.rb") }
+        before { run_generator %w(Namespace::YourModel -t=test_unit) }
+
+        it { should contain "class Namespace::YourModelDecoratorTest < Draper::TestCase" }
+      end
+    end
+  end
+end

data/spec/integration/integration_spec.rb

@@ -0,0 +1,66 @@
+require 'spec_helper'
+require 'support/dummy_app'
+require 'support/matchers/have_text'
+
+app = DummyApp.new(ENV["RAILS_ENV"])
+spec_types = {
+  view: ["/posts/1", "PostsController"],
+  mailer: ["/posts/1/mail", "PostMailer"]
+}
+
+app.start_server do
+  spec_types.each do |type, (path, controller)|
+    page = app.get(path)
+
+    describe "in a #{type}" do
+      it "runs in the correct environment" do
+        expect(page).to have_text(app.environment).in("#environment")
+      end
+
+      it "uses the correct view context controller" do
+        expect(page).to have_text(controller).in("#controller")
+      end
+
+      it "can use built-in helpers" do
+        expect(page).to have_text("Once upon a...").in("#truncated")
+      end
+
+      it "can use built-in private helpers" do
+        # Nokogiri unescapes text!
+        expect(page).to have_text("<script>danger</script>").in("#html_escaped")
+      end
+
+      it "can use user-defined helpers from app/helpers" do
+        expect(page).to have_text("Hello, world!").in("#hello_world")
+      end
+
+      it "can use user-defined helpers from the controller" do
+        expect(page).to have_text("Goodnight, moon!").in("#goodnight_moon")
+      end
+
+      it "can be passed to path helpers" do
+        expect(page).to have_text("/en/posts/1").in("#path_with_decorator")
+      end
+
+      it "can use path helpers with a model" do
+        expect(page).to have_text("/en/posts/1").in("#path_with_model")
+      end
+
+      it "can use path helpers with an id" do
+        expect(page).to have_text("/en/posts/1").in("#path_with_id")
+      end
+
+      it "can be passed to url helpers" do
+        expect(page).to have_text("http://www.example.com:12345/en/posts/1").in("#url_with_decorator")
+      end
+
+      it "can use url helpers with a model" do
+        expect(page).to have_text("http://www.example.com:12345/en/posts/1").in("#url_with_model")
+      end
+
+      it "can use url helpers with an id" do
+        expect(page).to have_text("http://www.example.com:12345/en/posts/1").in("#url_with_id")
+      end
+    end
+  end
+end