bourgeois 0.1.1 → 0.1.2

data/.travis.yml

@@ -4,4 +4,8 @@ rvm:
   - 2.0.0
   - 1.9.3
 
+gemfile:
+  - gemfiles/Gemfile.activemodel-4.0
+  - gemfiles/Gemfile.activemodel-3.2.x
+
 script: "echo 'DO IT' && bundle exec rake spec"

data/README.md

@@ -1,6 +1,7 @@
 # Bourgeois
 
 [![Gem Version](https://badge.fury.io/rb/bourgeois.png)](https://rubygems.org/gems/bourgeois)
+[![Code Climate](https://codeclimate.com/github/mirego/bourgeois.png)](https://codeclimate.com/github/mirego/bourgeois)
 [![Build Status](https://travis-ci.org/mirego/bourgeois.png?branch=master)](https://travis-ci.org/mirego/bourgeois)
 
 Bourgeois is a Ruby library that makes using presenters a very simple thing.
@@ -41,7 +42,18 @@ Then, you can use the `present` helper in your views to wrap `ActiveModel` (and
 <% end %>
 ```
 
-Methods that aren’t in the presenter (`first_name` and `last_name`) are delegated to the presented object. You can also use the `view` method in the presenter to get the original view it was called in.
+Methods that aren’t in the presenter (`first_name` and `last_name`) are delegated to the presented object. You can also use the `view` method in the presenter to get the original view it was called in:
+
+```ruby
+# app/presenters/user_presenter.rb
+
+class UserPresenter < Bourgeois::Presenter
+  def birthdate
+    # To get the original `birthdate` value, you can either use `super` or `object.birthdate`
+    super.presence || view.content_tag(:em, 'Unknown')
+  end
+end
+```
 
 ## Inspiration
 

data/gemfiles/Gemfile.activemodel-3.2.x

@@ -0,0 +1,6 @@
+source 'https://rubygems.org'
+
+gemspec path: '../'
+
+gem 'activemodel', '~> 3.2.0'
+gem 'actionpack', '~> 3.2.0'

data/gemfiles/Gemfile.activemodel-4.0

@@ -0,0 +1,6 @@
+source 'https://rubygems.org'
+
+gemspec path: '../'
+
+gem 'activemodel', '~> 4.0.0'
+gem 'actionpack', '~> 4.0.0'

data/lib/bourgeois/errors.rb

@@ -0,0 +1,11 @@
+module Bourgeois
+  class UnknownPresenter < StandardError
+    def initialize(klass)
+      @klass = klass
+    end
+
+    def to_s
+      "unknown presenter class #{@klass}"
+    end
+  end
+end

data/lib/bourgeois/presenter.rb

@@ -33,6 +33,11 @@ module Bourgeois
       @view
     end
 
+    # Return the original delegated object
+    def object
+      @object
+    end
+
     # Return the original object class based on the presenter class name
     # We would be able to use `@object.class` but we need this in class methods
     def self.klass

data/lib/bourgeois/railtie.rb

@@ -4,9 +4,7 @@ require 'rails'
 module Bourgeois
   class Railtie < Rails::Railtie
     initializer 'bourgeois.action_view' do |app|
-      ActiveSupport.on_load :action_view do
-        ActionView::Base.send(:include, ViewHelper)
-      end
+      ActiveSupport.on_load :action_view, {}, &Bourgeois.inject_into_action_view
     end
   end
 end

data/lib/bourgeois/version.rb

@@ -1,3 +1,3 @@
 module Bourgeois
-  VERSION = "0.1.1"
+  VERSION = "0.1.2"
 end

data/lib/bourgeois/view_helper.rb

@@ -1,6 +1,6 @@
 module Bourgeois
   module ViewHelper
-    # Wrap a resource or a collection into its related delegator
+    # Wrap a resource or a collection into its related presenter
     #
     # @example
     #   present User.new(name: 'Remi') do |user|
@@ -10,7 +10,15 @@ module Bourgeois
     def present(object, klass = nil, &blk)
       return object.map { |o| present(o, klass, &blk) } if object.respond_to?(:to_a)
 
-      klass ||= "#{object.class}Presenter".constantize
+      if klass.blank?
+        begin
+          klass_name = "#{object.class}Presenter"
+          klass = klass_name.constantize
+        rescue ::NameError
+          raise UnknownPresenter.new(klass_name)
+        end
+      end
+
       presenter = klass.new(object, self)
       yield presenter if block_given?
 

data/lib/bourgeois.rb

@@ -4,10 +4,16 @@ require 'delegate'
 require 'active_model'
 require 'action_view'
 
+require 'bourgeois/errors'
 require 'bourgeois/presenter'
 require 'bourgeois/view_helper'
 
 module Bourgeois
+  def self.inject_into_action_view
+    @inject_into_action_view ||= Proc.new do
+      ActionView::Base.send(:include, ViewHelper)
+    end
+  end
 end
 
 require 'bourgeois/railtie' if defined?(Rails) && Rails::VERSION::MAJOR >= 3

data/spec/bourgeois/presenter_spec.rb

@@ -1,7 +1,7 @@
 require 'spec_helper'
 
 describe Bourgeois::Presenter do
-  let(:user) { User.new first_name: 'Patrick', last_name: 'Bourgeois' }
+  let(:user) { User.new first_name: 'Patrick', last_name: 'Bourgeois', birthdate: '1962-06-16' }
   let(:view) { ActionView::Base.new }
   let(:presenter) { UserPresenter.new(user, view) }
 
@@ -11,6 +11,10 @@ describe Bourgeois::Presenter do
         def formatted_name
           "#{first_name} #{last_name}".strip
         end
+
+        def birthdate
+          super.presence || 'Unknown'
+        end
       end
 
       class User < OpenStruct
@@ -20,6 +24,7 @@ describe Bourgeois::Presenter do
     it { expect(presenter.formatted_name).to eql 'Patrick Bourgeois' }
     it { expect(presenter.first_name).to eql 'Patrick' }
     it { expect(presenter.last_name).to eql 'Bourgeois' }
+    it { expect(presenter.birthdate).to eql '1962-06-16' }
   end
 
   describe :InstanceMethods do
@@ -39,6 +44,10 @@ describe Bourgeois::Presenter do
             def local_name
               view.t('users.attributes.local_name')
             end
+
+            def first_name_in_bold
+              view.content_tag :strong, first_name
+            end
           end
 
           class ActionView::Base
@@ -51,6 +60,7 @@ describe Bourgeois::Presenter do
         end
 
         it { expect(presenter.local_name).to eql 'Fancy translated string from users.attributes.local_name' }
+        it { expect(presenter.first_name_in_bold).to eql '<strong>Patrick</strong>' }
       end
 
       context 'with blank view' do
@@ -65,6 +75,20 @@ describe Bourgeois::Presenter do
       end
     end
 
+    describe :object do
+      before do
+        class UserPresenter < Bourgeois::Presenter
+          def birthdate
+            object.birthdate.gsub(/-/, '/')
+          end
+        end
+
+        class User < OpenStruct; end
+      end
+
+      it { expect(presenter.birthdate).to eql '1962/06/16' }
+    end
+
     describe :inspect do
       before do
         class UserPresenter < Bourgeois::Presenter; end

data/spec/bourgeois/view_helper_spec.rb

@@ -0,0 +1,56 @@
+require 'spec_helper'
+
+describe Bourgeois::ViewHelper do
+  describe :present do
+    let(:view) { ActionView::Base.new }
+
+    before do
+      class UserPresenter < Bourgeois::Presenter
+        def formatted_name
+          "#{first_name} #{last_name}".strip
+        end
+      end
+
+      class User < OpenStruct; end
+    end
+
+    context 'on a single resource' do
+      let(:user) { User.new first_name: 'Patrick', last_name: 'Bourgeois' }
+
+      context 'without a block' do
+        it { expect(view.present(user).formatted_name).to eql 'Patrick Bourgeois' }
+      end
+
+      context 'with a block' do
+        specify do
+          view.present(user) do |u|
+            expect(u.formatted_name).to eql 'Patrick Bourgeois'
+          end
+        end
+      end
+    end
+
+    context 'on a collection of resources' do
+      let(:user1) { User.new first_name: 'Patrick', last_name: 'Bourgeois' }
+      let(:user2) { User.new first_name: 'Francois', last_name: 'Jean' }
+      let(:user3) { User.new first_name: 'Alain', last_name: 'Lapointe' }
+      let(:users) { [user1, user2, user3] }
+
+      specify do
+        output = []
+        view.present(users) { |u| output << u.formatted_name }
+
+        expect(output).to eql ['Patrick Bourgeois', 'Francois Jean', 'Alain Lapointe']
+      end
+    end
+
+    context 'on a resource without a defined presenter class' do
+      before do
+        class Project < OpenStruct; end
+      end
+
+      let(:project) { Project.new name: 'Les B.B.' }
+      it { expect { view.present(project) }.to raise_error(Bourgeois::UnknownPresenter, 'unknown presenter class ProjectPresenter') }
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -4,5 +4,5 @@ require 'ostruct'
 require 'rspec'
 require 'bourgeois'
 
-RSpec.configure do |config|
-end
+# Inject our helper into ActionView
+ActionView::Base.class_eval(&Bourgeois.inject_into_action_view)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: bourgeois
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-07-25 00:00:00.000000000 Z
+date: 2013-08-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activemodel
@@ -107,12 +107,16 @@ files:
 - README.md
 - Rakefile
 - bourgeois.gemspec
+- gemfiles/Gemfile.activemodel-3.2.x
+- gemfiles/Gemfile.activemodel-4.0
 - lib/bourgeois.rb
+- lib/bourgeois/errors.rb
 - lib/bourgeois/presenter.rb
 - lib/bourgeois/railtie.rb
 - lib/bourgeois/version.rb
 - lib/bourgeois/view_helper.rb
 - spec/bourgeois/presenter_spec.rb
+- spec/bourgeois/view_helper_spec.rb
 - spec/spec_helper.rb
 homepage: https://github.com/mirego/bourgeois
 licenses:
@@ -129,7 +133,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -4422150060977082943
+      hash: 1321347131962644702
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -138,7 +142,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -4422150060977082943
+      hash: 1321347131962644702
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.23
@@ -147,4 +151,5 @@ specification_version: 3
 summary: Bourgeois is a Ruby library that makes using presenters a very simple thing.
 test_files:
 - spec/bourgeois/presenter_spec.rb
+- spec/bourgeois/view_helper_spec.rb
 - spec/spec_helper.rb