RubyGems - draper - Versions diffs - 0.18.0 → 1.0.0.beta1 - Mend

draper 0.18.0 → 1.0.0.beta1

Files changed (123) hide show

data/.gitignore +3 -0
data/.travis.yml +5 -0
data/CHANGELOG.markdown +8 -0
data/Gemfile +4 -0
data/Rakefile +50 -31
data/Readme.markdown +42 -48
data/draper.gemspec +2 -1
data/lib/draper.rb +42 -7
data/lib/draper/collection_decorator.rb +88 -0
data/lib/draper/decoratable.rb +44 -0
data/lib/draper/decorated_association.rb +55 -0
data/lib/draper/decorator.rb +208 -0
data/lib/draper/finders.rb +44 -0
data/lib/draper/helper_proxy.rb +16 -0
data/lib/draper/railtie.rb +17 -21
data/lib/draper/security.rb +48 -0
data/lib/draper/tasks/tu.rake +5 -0
data/lib/draper/test/minitest_integration.rb +1 -1
data/lib/draper/test/test_unit_integration.rb +9 -0
data/lib/draper/version.rb +1 -1
data/lib/draper/view_context.rb +6 -13
data/lib/draper/view_helpers.rb +36 -0
data/lib/generators/decorator/decorator_generator.rb +1 -1
data/lib/generators/decorator/templates/decorator.rb +0 -1
data/spec/draper/collection_decorator_spec.rb +240 -0
data/spec/draper/decoratable_spec.rb +164 -0
data/spec/draper/decorated_association_spec.rb +130 -0
data/spec/draper/decorator_spec.rb +497 -0
data/spec/draper/finders_spec.rb +156 -0
data/spec/draper/helper_proxy_spec.rb +12 -0
data/spec/draper/security_spec.rb +158 -0
data/spec/draper/view_helpers_spec.rb +41 -0
data/spec/dummy/.rspec +2 -0
data/spec/dummy/README.rdoc +261 -0
data/spec/dummy/Rakefile +7 -0
data/spec/dummy/app/controllers/application_controller.rb +4 -0
data/spec/dummy/app/controllers/localized_urls.rb +5 -0
data/spec/dummy/app/controllers/posts_controller.rb +17 -0
data/spec/dummy/app/decorators/post_decorator.rb +25 -0
data/spec/dummy/app/helpers/application_helper.rb +2 -0
data/spec/dummy/app/mailers/application_mailer.rb +3 -0
data/spec/dummy/app/mailers/post_mailer.rb +9 -0
data/spec/dummy/app/models/post.rb +3 -0
data/spec/dummy/app/views/layouts/application.html.erb +11 -0
data/spec/dummy/app/views/post_mailer/decorated_email.html.erb +1 -0
data/spec/dummy/app/views/posts/_post.html.erb +19 -0
data/spec/dummy/app/views/posts/show.html.erb +1 -0
data/spec/dummy/config.ru +4 -0
data/spec/dummy/config/application.rb +64 -0
data/spec/dummy/config/boot.rb +10 -0
data/spec/dummy/config/database.yml +25 -0
data/spec/dummy/config/environment.rb +5 -0
data/spec/dummy/config/environments/development.rb +34 -0
data/spec/dummy/config/environments/production.rb +55 -0
data/spec/dummy/config/environments/test.rb +32 -0
data/spec/dummy/config/initializers/backtrace_silencers.rb +7 -0
data/spec/dummy/config/initializers/inflections.rb +15 -0
data/spec/dummy/config/initializers/mime_types.rb +5 -0
data/spec/dummy/config/initializers/secret_token.rb +7 -0
data/spec/dummy/config/initializers/session_store.rb +8 -0
data/spec/dummy/config/initializers/wrap_parameters.rb +14 -0
data/spec/dummy/config/locales/en.yml +5 -0
data/spec/dummy/config/routes.rb +7 -0
data/spec/dummy/db/migrate/20121019115657_create_posts.rb +8 -0
data/spec/dummy/db/schema.rb +21 -0
data/spec/dummy/db/seeds.rb +2 -0
data/spec/dummy/lib/tasks/spec.rake +5 -0
data/spec/dummy/public/404.html +26 -0
data/spec/dummy/public/422.html +26 -0
data/spec/dummy/public/500.html +25 -0
data/spec/dummy/public/favicon.ico +0 -0
data/spec/dummy/script/rails +6 -0
data/spec/dummy/spec/decorators/post_decorator_spec.rb +23 -0
data/spec/dummy/spec/mailers/post_mailer_spec.rb +29 -0
data/spec/dummy/spec/spec_helper.rb +9 -0
data/spec/generators/decorator/decorator_generator_spec.rb +3 -4
data/spec/integration/integration_spec.rb +33 -0
data/{performance → spec/performance}/active_record.rb +0 -0
data/{performance/bechmark.rb → spec/performance/benchmark.rb} +0 -0
data/{performance → spec/performance}/decorators.rb +2 -4
data/{performance → spec/performance}/models.rb +0 -0
data/spec/spec_helper.rb +20 -41
data/spec/support/action_controller.rb +12 -0
data/spec/support/active_model.rb +7 -0
data/spec/support/{samples/active_record.rb → active_record.rb} +5 -0
data/spec/support/{samples → decorators}/decorator_with_application_helper.rb +1 -1
data/spec/support/decorators/namespaced_product_decorator.rb +5 -0
data/spec/support/decorators/non_active_model_product_decorator.rb +2 -0
data/spec/support/decorators/product_decorator.rb +19 -0
data/spec/support/{samples → decorators}/products_decorator.rb +1 -1
data/spec/support/decorators/some_thing_decorator.rb +2 -0
data/spec/support/{samples → decorators}/specific_product_decorator.rb +0 -2
data/spec/support/{samples → decorators}/widget_decorator.rb +0 -0
data/spec/support/dummy_app.rb +84 -0
data/spec/support/matchers/have_text.rb +50 -0
data/spec/support/{samples → models}/namespaced_product.rb +1 -3
data/spec/support/{samples → models}/non_active_model_product.rb +1 -0
data/spec/support/{samples → models}/product.rb +13 -2
data/spec/support/models/some_thing.rb +5 -0
data/spec/support/models/uninferrable_decorator_model.rb +3 -0
data/spec/support/{samples → models}/widget.rb +0 -0
metadata +185 -68
data/lib/draper/active_model_support.rb +0 -27
data/lib/draper/base.rb +0 -312
data/lib/draper/decorated_enumerable_proxy.rb +0 -90
data/lib/draper/model_support.rb +0 -25
data/lib/draper/rspec_integration.rb +0 -2
data/lib/draper/system.rb +0 -18
data/spec/draper/base_spec.rb +0 -873
data/spec/draper/decorated_enumerable_proxy_spec.rb +0 -45
data/spec/draper/model_support_spec.rb +0 -48
data/spec/support/samples/decorator.rb +0 -5
data/spec/support/samples/decorator_with_allows.rb +0 -3
data/spec/support/samples/decorator_with_denies.rb +0 -3
data/spec/support/samples/decorator_with_denies_all.rb +0 -3
data/spec/support/samples/decorator_with_multiple_allows.rb +0 -4
data/spec/support/samples/decorator_with_special_methods.rb +0 -13
data/spec/support/samples/enumerable_proxy.rb +0 -3
data/spec/support/samples/namespaced_product_decorator.rb +0 -7
data/spec/support/samples/product_decorator.rb +0 -7
data/spec/support/samples/sequel_product.rb +0 -13
data/spec/support/samples/some_thing.rb +0 -2
data/spec/support/samples/some_thing_decorator.rb +0 -3

data/spec/dummy/config/locales/en.yml ADDED Viewed

@@ -0,0 +1,5 @@
+# Sample localization file for English. Add more files in this directory for other locales.
+# See https://github.com/svenfuchs/rails-i18n/tree/master/rails%2Flocale for starting points.
+en:
+  hello: "Hello world"

data/spec/dummy/config/routes.rb ADDED Viewed

@@ -0,0 +1,7 @@
+Dummy::Application.routes.draw do
+  scope "(:locale)", locale: /en|zh/ do
+    resources :posts, only: [:show] do
+      get "mail", on: :member
+    end
+  end
+end

data/spec/dummy/db/migrate/20121019115657_create_posts.rb ADDED Viewed

@@ -0,0 +1,8 @@
+class CreatePosts < ActiveRecord::Migration
+  def change
+    create_table :posts do |t|
+      t.timestamps
+    end
+  end
+end

data/spec/dummy/db/schema.rb ADDED Viewed

@@ -0,0 +1,21 @@
+# encoding: UTF-8
+# This file is auto-generated from the current state of the database. Instead
+# of editing this file, please use the migrations feature of Active Record to
+# incrementally modify your database, and then regenerate this schema definition.
+#
+# Note that this schema.rb definition is the authoritative source for your
+# database schema. If you need to create the application database on another
+# system, you should be using db:schema:load, not running all the migrations
+# from scratch. The latter is a flawed and unsustainable approach (the more migrations
+# you'll amass, the slower it'll run and the greater likelihood for issues).
+#
+# It's strongly recommended to check this file into your version control system.
+ActiveRecord::Schema.define(:version => 20121019115657) do
+  create_table "posts", :force => true do |t|
+    t.datetime "created_at", :null => false
+    t.datetime "updated_at", :null => false
+  end
+end

data/spec/dummy/db/seeds.rb ADDED Viewed

	@@ -0,0 +1,2 @@
1	+ Post.delete_all
2	+ Post.create id: 1

data/spec/dummy/lib/tasks/spec.rake ADDED Viewed

@@ -0,0 +1,5 @@
+require 'rspec/core/rake_task'
+RSpec::Core::RakeTask.new
+task "default" => "spec"

data/spec/dummy/public/404.html ADDED Viewed

@@ -0,0 +1,26 @@
+<!DOCTYPE html>
+<html>
+<head>
+  <title>The page you were looking for doesn't exist (404)</title>
+  <style type="text/css">
+    body { background-color: #fff; color: #666; text-align: center; font-family: arial, sans-serif; }
+    div.dialog {
+      width: 25em;
+      padding: 0 4em;
+      margin: 4em auto 0 auto;
+      border: 1px solid #ccc;
+      border-right-color: #999;
+      border-bottom-color: #999;
+    }
+    h1 { font-size: 100%; color: #f00; line-height: 1.5em; }
+  </style>
+</head>
+<body>
+  <!-- This file lives in public/404.html -->
+  <div class="dialog">
+    <h1>The page you were looking for doesn't exist.</h1>
+    <p>You may have mistyped the address or the page may have moved.</p>
+  </div>
+</body>
+</html>

data/spec/dummy/public/422.html ADDED Viewed

@@ -0,0 +1,26 @@
+<!DOCTYPE html>
+<html>
+<head>
+  <title>The change you wanted was rejected (422)</title>
+  <style type="text/css">
+    body { background-color: #fff; color: #666; text-align: center; font-family: arial, sans-serif; }
+    div.dialog {
+      width: 25em;
+      padding: 0 4em;
+      margin: 4em auto 0 auto;
+      border: 1px solid #ccc;
+      border-right-color: #999;
+      border-bottom-color: #999;
+    }
+    h1 { font-size: 100%; color: #f00; line-height: 1.5em; }
+  </style>
+</head>
+<body>
+  <!-- This file lives in public/422.html -->
+  <div class="dialog">
+    <h1>The change you wanted was rejected.</h1>
+    <p>Maybe you tried to change something you didn't have access to.</p>
+  </div>
+</body>
+</html>

data/spec/dummy/public/500.html ADDED Viewed

@@ -0,0 +1,25 @@
+<!DOCTYPE html>
+<html>
+<head>
+  <title>We're sorry, but something went wrong (500)</title>
+  <style type="text/css">
+    body { background-color: #fff; color: #666; text-align: center; font-family: arial, sans-serif; }
+    div.dialog {
+      width: 25em;
+      padding: 0 4em;
+      margin: 4em auto 0 auto;
+      border: 1px solid #ccc;
+      border-right-color: #999;
+      border-bottom-color: #999;
+    }
+    h1 { font-size: 100%; color: #f00; line-height: 1.5em; }
+  </style>
+</head>
+<body>
+  <!-- This file lives in public/500.html -->
+  <div class="dialog">
+    <h1>We're sorry, but something went wrong.</h1>
+  </div>
+</body>
+</html>

data/spec/dummy/public/favicon.ico ADDED Viewed

File without changes

data/spec/dummy/script/rails ADDED Viewed

@@ -0,0 +1,6 @@
+#!/usr/bin/env ruby
+# This command will automatically be run when you run "rails" with Rails 3 gems installed from the root of your application.
+APP_PATH = File.expand_path('../../config/application',  __FILE__)
+require File.expand_path('../../config/boot',  __FILE__)
+require 'rails/commands'

data/spec/dummy/spec/decorators/post_decorator_spec.rb ADDED Viewed

@@ -0,0 +1,23 @@
+require 'spec_helper'
+describe PostDecorator do
+  subject { PostDecorator.new(source) }
+  let(:source) { Post.create }
+  it "can use path helpers with its model" do
+    subject.path_with_model.should == "/en/posts/#{source.id}"
+  end
+  it "can use path helpers with its id" do
+    subject.path_with_id.should == "/en/posts/#{source.id}"
+  end
+  it "can use url helpers with its model" do
+    subject.url_with_model.should == "http://www.example.com/en/posts/#{source.id}"
+  end
+  it "can use url helpers with its id" do
+    subject.url_with_id.should == "http://www.example.com/en/posts/#{source.id}"
+  end
+end

data/spec/dummy/spec/mailers/post_mailer_spec.rb ADDED Viewed

@@ -0,0 +1,29 @@
+require 'spec_helper'
+describe PostMailer do
+  describe "#decorated_email" do
+    subject { Capybara.string(email.body.to_s) }
+    let(:email) { PostMailer.decorated_email(post).deliver }
+    let(:post) { Post.create }
+    it "decorates" do
+      subject.should have_content "Today"
+    end
+    it "can use path helpers with a model" do
+      subject.should have_css "#path_with_model", text: "/en/posts/#{post.id}"
+    end
+    it "can use path helpers with an id" do
+      subject.should have_css "#path_with_id", text: "/en/posts/#{post.id}"
+    end
+    it "can use url helpers with a model" do
+      subject.should have_css "#url_with_model", text: "http://www.example.com/en/posts/#{post.id}"
+    end
+    it "can use url helpers with an id" do
+      subject.should have_css "#url_with_id", text: "http://www.example.com/en/posts/#{post.id}"
+    end
+  end
+end

data/spec/dummy/spec/spec_helper.rb ADDED Viewed

@@ -0,0 +1,9 @@
+ENV['RAILS_ENV'] ||= 'test'
+require File.expand_path('../../config/environment', __FILE__)
+require 'rspec/rails'
+require 'draper/test/rspec_integration'
+RSpec.configure do |config|
+  config.treat_symbols_as_metadata_keys_with_true_values = true
+  config.order = :random
+end

data/spec/generators/decorator/decorator_generator_spec.rb CHANGED Viewed

@@ -15,8 +15,7 @@ describe Rails::Generators::DecoratorGenerator do
     describe 'app/decorators/your_model_decorator.rb' do
       subject { file('app/decorators/your_model_decorator.rb') }
       it { should exist }
-      it { should contain "class YourModelDecorator < Draper::Base" }
-      it { should contain "decorates :your_model" }
+      it { should contain "class YourModelDecorator < Draper::Decorator" }
     end
   end
@@ -31,12 +30,12 @@ describe Rails::Generators::DecoratorGenerator do
   end
   context 'parent decorator' do
-    describe 'decorator inhereted from Draper::Base' do
+    describe 'decorator inhereted from Draper::Decorator' do
       before { run_generator ["YourModel"] }
       subject { file('app/decorators/your_model_decorator.rb') }
       it { should exist }
-      it { should contain "class YourModelDecorator < Draper::Base" }
+      it { should contain "class YourModelDecorator < Draper::Decorator" }
     end
     describe "decorator inhereted from ApplicationDecorator if it's present" do

data/spec/integration/integration_spec.rb ADDED Viewed

@@ -0,0 +1,33 @@
+require 'spec_helper'
+require 'support/dummy_app'
+require 'support/matchers/have_text'
+app = DummyApp.new(ENV["RAILS_ENV"])
+app.start_server do
+  {view: "/posts/1", mailer: "/posts/1/mail"}.each do |type, path|
+    page = app.get(path)
+    describe "in a #{type}" do
+      it "runs in the correct environment" do
+        page.should have_text(app.environment).in("#environment")
+      end
+      it "can use path helpers with a model" do
+        page.should have_text("/en/posts/1").in("#path_with_model")
+      end
+      it "can use path helpers with an id" do
+        page.should have_text("/en/posts/1").in("#path_with_id")
+      end
+      it "can use url helpers with a model" do
+        page.should have_text("http://www.example.com/en/posts/1").in("#url_with_model")
+      end
+      it "can use url helpers with an id" do
+        page.should have_text("http://www.example.com/en/posts/1").in("#url_with_id")
+      end
+    end
+  end
+end

data/{performance → spec/performance}/active_record.rb RENAMED Viewed

File without changes

data/{performance/bechmark.rb → spec/performance/benchmark.rb} RENAMED Viewed

File without changes

data/{performance → spec/performance}/decorators.rb RENAMED Viewed

@@ -1,6 +1,5 @@
 require "./performance/models"
-class ProductDecorator < Draper::Base
-  decorates :product
+class ProductDecorator < Draper::Decorator
   def awesome_title
     "Awesome Title"
@@ -21,8 +20,7 @@ class ProductDecorator < Draper::Base
 end
-class FastProductDecorator < Draper::Base
-  decorates :product
+class FastProductDecorator < Draper::Decorator
   def awesome_title
     "Awesome Title"

data/{performance → spec/performance}/models.rb RENAMED Viewed

File without changes

data/spec/spec_helper.rb CHANGED Viewed

@@ -3,51 +3,30 @@ require 'ammeter/init'
 require 'rails'
 require 'action_view'
-Bundler.require
-require './spec/support/samples/active_record'
-require './spec/support/samples/decorator'
-require './spec/support/samples/decorator_with_allows'
-require './spec/support/samples/decorator_with_multiple_allows'
-require './spec/support/samples/decorator_with_application_helper'
-require './spec/support/samples/decorator_with_denies'
-require './spec/support/samples/decorator_with_denies_all'
-require './spec/support/samples/decorator_with_special_methods'
-require './spec/support/samples/enumerable_proxy'
-require './spec/support/samples/namespaced_product'
-require './spec/support/samples/namespaced_product_decorator'
-require './spec/support/samples/non_active_model_product'
-require './spec/support/samples/product'
-require './spec/support/samples/product_decorator'
-require './spec/support/samples/products_decorator'
-require './spec/support/samples/sequel_product'
-require './spec/support/samples/specific_product_decorator'
-require './spec/support/samples/some_thing'
-require './spec/support/samples/some_thing_decorator'
-require './spec/support/samples/widget'
-require './spec/support/samples/widget_decorator'
 require 'action_controller'
 require 'action_controller/test_case'
-module ActionController
-  class Base
-    Draper::System.setup_action_controller(self)
-  end
-end
-module ActiveRecord
-  class Relation
-  end
-end
-class ApplicationController < ActionController::Base
-  def hello_world
-    "Hello, World!"
-  end
-  helper_method :hello_world
-end
+Bundler.require
+require './spec/support/active_model'
+require './spec/support/active_record'
+require './spec/support/action_controller'
+require './spec/support/models/product'
+require './spec/support/models/namespaced_product'
+require './spec/support/models/non_active_model_product'
+require './spec/support/models/widget'
+require './spec/support/models/some_thing'
+require './spec/support/models/uninferrable_decorator_model'
+require './spec/support/decorators/product_decorator'
+require './spec/support/decorators/namespaced_product_decorator'
+require './spec/support/decorators/non_active_model_product_decorator'
+require './spec/support/decorators/widget_decorator'
+require './spec/support/decorators/specific_product_decorator'
+require './spec/support/decorators/products_decorator'
+require './spec/support/decorators/some_thing_decorator'
+require './spec/support/decorators/decorator_with_application_helper'
 class << Rails
   undef application # Avoid silly Rails bug: https://github.com/rails/rails/pull/6429

data/spec/support/action_controller.rb ADDED Viewed

@@ -0,0 +1,12 @@
+module ActionController
+  class Base
+    Draper.setup_action_controller(self)
+  end
+end
+class ApplicationController < ActionController::Base
+  def hello_world
+    "Hello, World!"
+  end
+  helper_method :hello_world
+end

data/spec/support/active_model.rb ADDED Viewed

@@ -0,0 +1,7 @@
+module ActiveModel
+  module Serialization
+    def serializable_hash
+      {overridable: send(:overridable)}
+    end
+  end
+end

data/spec/support/{samples/active_record.rb → active_record.rb} RENAMED Viewed

@@ -15,3 +15,8 @@ module ActiveRecord
   end
 end
+module ActiveRecord
+  class Relation
+  end
+end

data/spec/support/{samples → decorators}/decorator_with_application_helper.rb RENAMED Viewed

@@ -1,4 +1,4 @@
-class DecoratorWithApplicationHelper < Draper::Base
+class DecoratorWithApplicationHelper < Draper::Decorator
   def uses_hello_world
     h.hello_world
   end

data/spec/support/decorators/namespaced_product_decorator.rb ADDED Viewed

@@ -0,0 +1,5 @@
+module Namespace
+  class ProductDecorator < Draper::Decorator
+    has_finders
+  end
+end

data/spec/support/decorators/non_active_model_product_decorator.rb ADDED Viewed

	@@ -0,0 +1,2 @@
1	+ class NonActiveModelProductDecorator
2	+ end

data/spec/support/decorators/product_decorator.rb ADDED Viewed

@@ -0,0 +1,19 @@
+class ProductDecorator < Draper::Decorator
+  has_finders
+  def awesome_title
+    "Awesome Title"
+  end
+  def overridable
+    :overridden
+  end
+  def self.my_class_method
+  end
+  private
+  def awesome_private_title
+  end
+end