simple_show 0.1.1 → 1.0.0

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: bda67008a5f7214b1c1ebcfcce7bcc358ff62920
+  data.tar.gz: 60b9f8ece19da67fea73d5d891f247baf038dbc8
+SHA512:
+  metadata.gz: 41ceb31d5349c3226378646a39aeb827de32c98aa88bda2ab27bc81c111b9f07a7d37caa1602fd66e112468e3e800647c750fac6e2d5ae80f917d0e12816df7c
+  data.tar.gz: c495af096bf0c3c4a98088cda34fed2fe3ca3292df0a0ed5e369dbcf787d718105cc779ba2adafaa8de82dba9e0aed9dbe3d43b99e11ea83bec05f6518d3bc00

data/.gitignore

@@ -2,3 +2,4 @@
 .bundle
 Gemfile.lock
 pkg/*
+*.swp

data/Gemfile

@@ -1,10 +1,11 @@
-source "http://rubygems.org"
+source 'http://rubygems.org'
 
 group :test do
-  gem 'rake', '0.8.7'
+  gem 'rake', '10.4.2'
   gem 'rails', '3.1.3'
   gem 'sqlite3'
   gem 'nokogiri'
+  gem 'test-unit'
 end
 
 # Specify your gem's dependencies in simple_show.gemspec

data/README.rdoc

@@ -39,10 +39,15 @@ In our show action we can then do this:
       <%= s.show :cash_in_wallet, :to_currency => true  %>
       <%= s.show :height, :if => :tall %>
       <%= s.show :weight, :unless => :sensitive %>
+      <%= s.show :title, :unless_attr => :nil? %>
+      <%= s.show_collection :friends do |friend| %>
+        <%= friend.name %> - <%= friend.age %>
+      <% end %>
     <% end %>
 
 Which would generate (using the defaults) output like this, assuming that
-@johndor.tall returns false and @johndoe.sensitive returns true:
+@johndoe.tall returns false, @johndoe.sensitive returns true, and
+@johndoe.title is nil:
 
     <div class="simple_show person" id="person_123">
       <div class="show">
@@ -73,6 +78,13 @@ Which would generate (using the defaults) output like this, assuming that
         <label>Cash in wallet:</label>
         <span class="value">$123.45</span>
       </div>
+      <div class="show">
+        <label>Friends:</label>
+        <ul class="show_collection">
+          <li class="show_collection_item">Jim - 32</li>
+          <li class="show_collection_item">Tina - 23</li>
+        </ul>
+      </div>
       <br clear="all">
     </div>
 
@@ -84,7 +96,6 @@ if you don't want that BR tag then set SimpleShow.clear_on_close to false.
 
 == TODO:
 
-- Add support for associations.
 - Add options for booleans.
 
 == License:

data/Rakefile

@@ -3,7 +3,7 @@ Bundler::GemHelper.install_tasks
 
 require 'rake/testtask'
 Rake::TestTask.new(:test) do |test|
-  test.libs << 'lib' 
+  test.libs << 'lib'
   test.libs << 'test'
   test.pattern = 'test/**/test_*.rb'
   test.verbose = true

data/lib/generators/simple_show/install_generator.rb

@@ -1,13 +1,12 @@
 module SimpleShow
   module Generators
     class InstallGenerator < Rails::Generators::Base
-      desc "Copy SimpleShow configuration"
+      desc 'Copy SimpleShow configuration'
       source_root File.expand_path('../templates', __FILE__)
 
       def copy_config
         directory 'config'
       end
-
     end
   end
 end

data/lib/simple_show.rb

@@ -3,7 +3,6 @@ require 'simple_show/action_view_extensions/form_helper'
 require 'simple_show/base'
 
 module SimpleShow
-
   mattr_accessor :show_class
   @@show_class = :simple_show
 
@@ -37,18 +36,30 @@ module SimpleShow
   mattr_accessor :value_suffix
   @@value_suffix = nil
 
+  mattr_accessor :collection_tag
+  @@collection_tag = :ul
+
+  mattr_accessor :collection_class
+  @@collection_class = :show_collection
+
+  mattr_accessor :collection_item_tag
+  @@collection_item_tag = :li
+
+  mattr_accessor :collection_item_class
+  @@collection_item_class = :show_collection_item
+
   mattr_accessor :clear_on_close
   @@clear_on_close = true
 
   mattr_accessor :helpers
   @@helpers = {
-    :to_currency => :number_to_currency,
-    :to_human => :number_to_human,
-    :to_human_size => :number_to_human_size,
-    :to_percentage => :number_to_percentage,
-    :to_phone => :number_to_phone,
-    :with_delimiter => :number_with_delimiter,
-    :with_precision => :number_with_precision
+    to_currency: :number_to_currency,
+    to_human: :number_to_human,
+    to_human_size: :number_to_human_size,
+    to_percentage: :number_to_percentage,
+    to_phone: :number_to_phone,
+    with_delimiter: :number_with_delimiter,
+    with_precision: :number_with_precision
   }
 
   # Default way to setup SimpleShow. Run rails generate simple_show:install
@@ -56,5 +67,4 @@ module SimpleShow
   def self.setup
     yield self
   end
-
 end

data/lib/simple_show/action_view_extensions/form_helper.rb

@@ -2,20 +2,19 @@ module SimpleShow
   module ActionViewExtensions
     module FormHelper
       def simple_show_for(record, options = {}, &block)
-        raise ArgumentError, "Missing block" unless block_given?
+        fail ArgumentError, 'Missing block' unless block_given?
 
         options[:html] ||= {}
         options[:html][:id] ||= dom_id(record)
         options[:html][:class] = "#{SimpleShow.show_class} #{dom_class(record)} #{options[:html][:class]}".strip
 
         output = capture(SimpleShow::Base.new(self, record, options), &block)
-        output.concat content_tag(:div, '', :class => 'clear') if SimpleShow.clear_on_close
+        output.concat content_tag(:div, '', class: 'clear') if SimpleShow.clear_on_close
 
-        content_tag(:div, output, :id => options[:html][:id], :class => options[:html][:class])
+        content_tag(:div, output, id: options[:html][:id], class: options[:html][:class])
       end
     end
   end
 end
 
 ActionView::Base.send :include, SimpleShow::ActionViewExtensions::FormHelper
-

data/lib/simple_show/base.rb

@@ -7,42 +7,43 @@ module SimpleShow
     end
 
     def show(attr, options = {}, &block)
-      if options.key? :if
-        case options[:if]
-        when Symbol
-          options[:if] = @record.send(options[:if])
-        when Proc
-          options[:if] = options[:if].call
-        end
-        return nil if options[:if] == false
-      end
-      if options.key? :unless
-        case options[:unless]
-        when Symbol
-          options[:unless] = @record.send(options[:unless])
-        when Proc
-          options[:unless] = options[:unless].call
-        end
-        return nil unless options[:unless] == false
-      end
+      return nil if skip?(attr, options)
 
       output = label(attr, options, &block)
       output += value(attr, options, &block)
 
-      if SimpleShow.wrapper_tag.nil?
-        output
-      else
-        @binding.content_tag(SimpleShow.wrapper_tag, output, :class => SimpleShow.wrapper_class)
+      wrap_output(SimpleShow.wrapper_tag, SimpleShow.wrapper_class, output)
+    end
+
+    def show_collection(attr, options = {}, &block)
+      return nil if skip?(attr, options)
+
+      output = label(attr, options, &block)
+
+      value = wrap_output(SimpleShow.collection_tag, SimpleShow.collection_class) do
+        @record.send(attr).map do |attr_item|
+          wrap_output(SimpleShow.collection_item_tag, SimpleShow.collection_item_class) do
+            block_given? ? @binding.capture(attr_item, &block) : attr_item
+          end
+        end.join.html_safe
       end
+
+      output += value
+
+      wrap_output(SimpleShow.wrapper_tag, SimpleShow.wrapper_class, output)
     end
 
     def label(attr, options = {}, &block)
-      @binding.content_tag(SimpleShow.label_tag, :class => SimpleShow.label_class) do
-        [SimpleShow.label_prefix, options[:label] || @record.class.human_attribute_name(attr), SimpleShow.label_suffix].compact.join
-      end
+      return nil if skip?(attr, options)
+
+      output = [SimpleShow.label_prefix, options[:label] || @record.class.human_attribute_name(attr), SimpleShow.label_suffix].compact.join
+
+      wrap_output(SimpleShow.label_tag, SimpleShow.label_class, output)
     end
 
     def value(attr, options = {}, &block)
+      return nil if skip?(attr, options)
+
       if block_given?
         value = yield(@record)
       else
@@ -63,15 +64,66 @@ module SimpleShow
         end
       end
 
+      output = [SimpleShow.value_prefix, value, SimpleShow.value_suffix].compact.join.html_safe
       field_type = @record.class.columns_hash[attr.to_s].instance_variable_get(:@type)
-      @binding.content_tag(SimpleShow.value_tag, :class => [SimpleShow.value_class, field_type].compact) do
-        [SimpleShow.value_prefix, value, SimpleShow.value_suffix].compact.join.html_safe
-      end
+      html_class = [SimpleShow.value_class, field_type].compact
+
+      wrap_output(SimpleShow.value_tag, html_class, output)
     end
 
     def object
       @record
     end
 
+    private
+
+    def wrap_output(tag, html_class, output=nil, &block)
+      output = yield if block_given?
+      if tag.nil?
+        output
+      else
+        @binding.content_tag(tag, output, :class => html_class)
+      end
+    end
+
+    def skip?(attr, options)
+      if options.key? :if
+        case options[:if]
+        when Symbol
+          options[:if] = @record.send(options[:if])
+        when Proc
+          options[:if] = options[:if].call
+        end
+        return true if options[:if] == false
+      end
+
+      if options.key? :unless
+        case options[:unless]
+        when Symbol
+          options[:unless] = @record.send(options[:unless])
+        when Proc
+          options[:unless] = options[:unless].call
+        end
+        return true unless options[:unless] == false
+      end
+
+      if options.key? :if_attr
+        case options[:if_attr]
+        when Symbol
+          options[:if_attr] = @record.send(attr).send(options[:if_attr])
+        end
+        return true if options[:if_attr] == false
+      end
+
+      if options.key? :unless_attr
+        case options[:unless_attr]
+        when Symbol
+          options[:unless_attr] = @record.send(attr).send(options[:unless_attr])
+        end
+        return true unless options[:unless_attr] == false
+      end
+
+      return false
+    end
   end
 end

data/lib/simple_show/version.rb

@@ -1,3 +1,3 @@
 module SimpleShow
-  VERSION = "0.1.1"
+  VERSION = '1.0.0'
 end

data/simple_show.gemspec

@@ -1,23 +1,23 @@
 # -*- encoding: utf-8 -*-
-$:.push File.expand_path("../lib", __FILE__)
-require "simple_show/version"
+$LOAD_PATH.push File.expand_path('../lib', __FILE__)
+require 'simple_show/version'
 
 Gem::Specification.new do |s|
-  s.name        = "simple_show"
+  s.name        = 'simple_show'
   s.version     = SimpleShow::VERSION
   s.platform    = Gem::Platform::RUBY
-  s.authors     = ["Philip Hallstrom"]
-  s.email       = ["philip@pjkh.com"]
-  s.homepage    = ""
-  s.summary     = %q{SimpleShow is to #show what SimpleForm is to #edit}
-  s.description = %q{SimpleShow is to #show what SimpleForm is to #edit}
-  s.license     = "MIT"
+  s.authors     = ['Philip Hallstrom']
+  s.email       = ['philip@pjkh.com']
+  s.homepage    = ''
+  s.summary     = 'SimpleShow is to #show what SimpleForm is to #edit'
+  s.description = 'SimpleShow is to #show what SimpleForm is to #edit'
+  s.license     = 'MIT'
 
-  s.rubyforge_project = "simple_show"
+  s.rubyforge_project = 'simple_show'
 
   s.files         = `git ls-files`.split("\n")
   s.files         = s.files.delete_if { |path| path =~ /\.rvmrc/ }
   s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
-  s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
-  s.require_paths = ["lib"]
+  s.executables   = `git ls-files -- bin/*`.split("\n").map { |f| File.basename(f) }
+  s.require_paths = ['lib']
 end

data/test/skip_helper.rb

@@ -0,0 +1,128 @@
+def test_skip(method, field1, field2, label1, label2)
+  first_label = field1.to_s.titleize
+
+  test ':if => true/false' do
+    doc = Nokogiri::HTML.fragment(
+      simple_show_for(@philip) do |s|
+        o  = ActiveSupport::SafeBuffer.new
+        o += s.send(method, field1, if: true)
+        o += s.send(method, field2, if: false)
+        o
+      end
+    )
+    assert_equal [label1], doc.css('label').map(&:text)
+  end
+
+  test ':if => :record_method' do
+    def @philip.method_returning_true
+      true
+    end
+    def @philip.method_returning_false
+      false
+    end
+    doc = Nokogiri::HTML.fragment(
+      simple_show_for(@philip) do |s|
+        o  = ActiveSupport::SafeBuffer.new
+        o += s.send(method, field1, if: :method_returning_true)
+        o += s.send(method, field2, if: :method_returning_false)
+        o
+      end
+    )
+    assert_equal [label1], doc.css('label').map(&:text)
+  end
+
+  test ':if => {lambda}' do
+    doc = Nokogiri::HTML.fragment(
+      simple_show_for(@philip) do |s|
+        o  = ActiveSupport::SafeBuffer.new
+        o += s.send(method, field1, if: -> { true })
+        o += s.send(method, field2, if: -> { false })
+        o
+      end
+    )
+    assert_equal [label1], doc.css('label').map(&:text)
+  end
+
+  test ':unless => true/false' do
+    doc = Nokogiri::HTML.fragment(
+      simple_show_for(@philip) do |s|
+        o  = ActiveSupport::SafeBuffer.new
+        o += s.send(method, field1, unless: true)
+        o += s.send(method, field2, unless: false)
+        o
+      end
+    )
+    assert_equal [label2], doc.css('label').map(&:text)
+  end
+
+  test ':unless => :record_method' do
+    def @philip.method_returning_true
+      true
+    end
+    def @philip.method_returning_false
+      false
+    end
+    doc = Nokogiri::HTML.fragment(
+      simple_show_for(@philip) do |s|
+        o  = ActiveSupport::SafeBuffer.new
+        o += s.send(method, field1, unless: :method_returning_true)
+        o += s.send(method, field2, unless: :method_returning_false)
+        o
+      end
+    )
+    assert_equal [label2], doc.css('label').map(&:text)
+  end
+
+  test ':unless => {lambda}' do
+    doc = Nokogiri::HTML.fragment(
+      simple_show_for(@philip) do |s|
+        o  = ActiveSupport::SafeBuffer.new
+        o += s.send(method, field1, unless: -> { true })
+        o += s.send(method, field2, unless: -> { false })
+        o
+      end
+    )
+    assert_equal [label2], doc.css('label').map(&:text)
+  end
+
+  test ':if_attr => :record_method' do
+    field1_attr = @philip.send(field1)
+    def field1_attr.method_returning_true
+      true
+    end
+    field2_attr = @philip.send(field2)
+    def field2_attr.method_returning_false
+      false
+    end
+    doc = Nokogiri::HTML.fragment(
+      simple_show_for(@philip) do |s|
+        o  = ActiveSupport::SafeBuffer.new
+        o += s.send(method, field1, if_attr: :method_returning_true)
+        o += s.send(method, field2, if_attr: :method_returning_false)
+        o
+      end
+    )
+    assert_equal [label1], doc.css('label').map(&:text)
+  end
+
+  test ':unless_attr => :record_method' do
+    field1_attr = @philip.send(field1)
+    def field1_attr.method_returning_true
+      true
+    end
+    field2_attr = @philip.send(field2)
+    def field2_attr.method_returning_false
+      false
+    end
+    doc = Nokogiri::HTML.fragment(
+      simple_show_for(@philip) do |s|
+        o  = ActiveSupport::SafeBuffer.new
+        o += s.send(method, field1, unless_attr: :method_returning_true)
+        o += s.send(method, field2, unless_attr: :method_returning_false)
+        o
+      end
+    )
+    assert_equal [label2], doc.css('label').map(&:text)
+  end
+end
+

data/test/test_helper.rb

@@ -16,17 +16,32 @@ require 'simple_show'
 
 ################################################################################
 
-ActiveRecord::Base.establish_connection(:adapter => "sqlite3", :database => ":memory:")
+ActiveRecord::Base.establish_connection(adapter: 'sqlite3', database: ':memory:')
 ActiveRecord::Migration.verbose = false
 
-ActiveRecord::Schema.define(:version => 1) do
+ActiveRecord::Schema.define(version: 1) do
   create_table :golfers do |t|
     t.column :name, :string
     t.column :phone, :string
     t.column :email, :string
     t.column :born_on, :date
     t.column :is_left_handed, :boolean
-    t.column :handicap, :decimal, :scale => 3, :precision => 1
+    t.column :handicap, :decimal, scale: 3, precision: 1
+    t.timestamps
+  end
+
+  create_table :rounds do |t|
+    t.references :golfer
+
+    t.column :name, :string
+    t.column :score, :integer
+    t.timestamps
+  end
+
+  create_table :clubs do |t|
+    t.references :golfer
+
+    t.column :name, :string
     t.timestamps
   end
 end
@@ -38,6 +53,24 @@ end
 ################################################################################
 
 class Golfer < ActiveRecord::Base
+  has_many :clubs
+  has_many :rounds
+end
+
+class Round < ActiveRecord::Base
+  belongs_to :golfer
+
+  def to_s
+    name
+  end
+end
+
+class Club < ActiveRecord::Base
+  belongs_to :golfer
+
+  def to_s
+    name
+  end
 end
 
 ################################################################################
@@ -46,14 +79,13 @@ module ActionView
   module Helpers
     module TagHelper
       def piglatin(str)
-        str[1..-1] + str[0,1] + 'ay'
+        str[1..-1] + str[0, 1] + 'ay'
       end
     end
   end
 end
-    
-################################################################################
 
+################################################################################
 
 class SimpleShowTestCase < ActiveSupport::TestCase
   include SimpleShow::ActionViewExtensions::FormHelper
@@ -65,13 +97,22 @@ class SimpleShowTestCase < ActiveSupport::TestCase
 
   def setup
     @philip = Golfer.create!(
-      :name           => 'Philip Hallstrom',
-      :phone          => '3604801209',
-      :email          => 'philip@pjkh.com',
-      :born_on        => Date.civil(1974, 5, 24),
-      :is_left_handed => true,
-      :handicap       => 6.5
+      name: 'Philip Hallstrom',
+      phone: '3604801209',
+      email: 'philip@pjkh.com',
+      born_on: Date.civil(1974, 5, 24),
+      is_left_handed: true,
+      handicap: 6.5
+    )
+    @round1 = Round.create!(
+      name: 'Round 1',
+      score: 72,
+      golfer: @philip
+    )
+    @round2 = Round.create!(
+      name: 'Round 2',
+      score: 68,
+      golfer: @philip
     )
   end
 end
-

data/test/test_simple_show.rb

@@ -1,7 +1,6 @@
 require 'test_helper'
 
 class SimpleShowTest < SimpleShowTestCase
-
   test 'setup block yields self' do
     SimpleShow.setup do |config|
       assert_equal SimpleShow, config
@@ -9,18 +8,21 @@ class SimpleShowTest < SimpleShowTestCase
   end
 
   test 'default configuration' do
-    assert_equal :simple_show , SimpleShow.show_class
-    assert_equal :div         , SimpleShow.wrapper_tag
-    assert_equal :show        , SimpleShow.wrapper_class
-    assert_equal :label       , SimpleShow.label_tag
-    assert_equal nil          , SimpleShow.label_class
-    assert_equal nil          , SimpleShow.label_prefix
-    assert_equal ':'          , SimpleShow.label_suffix
-    assert_equal :span        , SimpleShow.value_tag
-    assert_equal :value       , SimpleShow.value_class
-    assert_equal nil          , SimpleShow.value_prefix
-    assert_equal nil          , SimpleShow.value_suffix
-    assert_equal true         , SimpleShow.clear_on_close
+    assert_equal :simple_show, SimpleShow.show_class
+    assert_equal :div, SimpleShow.wrapper_tag
+    assert_equal :show, SimpleShow.wrapper_class
+    assert_equal :label, SimpleShow.label_tag
+    assert_equal nil, SimpleShow.label_class
+    assert_equal nil, SimpleShow.label_prefix
+    assert_equal ':', SimpleShow.label_suffix
+    assert_equal :span, SimpleShow.value_tag
+    assert_equal :value, SimpleShow.value_class
+    assert_equal nil, SimpleShow.value_prefix
+    assert_equal nil, SimpleShow.value_suffix
+    assert_equal :ul, SimpleShow.collection_tag
+    assert_equal :show_collection, SimpleShow.collection_class
+    assert_equal :li, SimpleShow.collection_item_tag
+    assert_equal :show_collection_item, SimpleShow.collection_item_class
+    assert_equal true, SimpleShow.clear_on_close
   end
-
 end

data/test/test_simple_show_for.rb

@@ -1,40 +1,38 @@
 require 'test_helper'
 
 class SimpleShowForTest < SimpleShowTestCase
-
   test 'simple_show_for raises error without a block' do
     assert_raise ArgumentError do
-      simple_show_for 
+      simple_show_for
     end
   end
 
   test 'html wrapper includes the correct id and class' do
-    doc = Nokogiri::HTML.fragment(simple_show_for(@philip) do |s| 
+    doc = Nokogiri::HTML.fragment(simple_show_for(@philip) do |_s|
       ''
     end)
-    assert_equal %w[simple_show golfer], doc.at('div').attr('class').split(/\s+/)
+    assert_equal %w(simple_show golfer), doc.at('div').attr('class').split(/\s+/)
     assert_equal "golfer_#{@philip.id}", doc.at('div').attr('id')
   end
 
   test 'last tag is a BR that clears all when enabled' do
-    clear_on_close = SimpleShow::clear_on_close
-    SimpleShow::clear_on_close = true
-    doc = Nokogiri::HTML.fragment(simple_show_for(@philip) do |s| 
+    clear_on_close = SimpleShow.clear_on_close
+    SimpleShow.clear_on_close = true
+    doc = Nokogiri::HTML.fragment(simple_show_for(@philip) do |_s|
       ''
     end)
-    SimpleShow::clear_on_close = clear_on_close
+    SimpleShow.clear_on_close = clear_on_close
     assert_equal 'div', doc.at('div').elements.last.name
     assert_equal 'clear', doc.at('div').elements.last.attr('class')
   end
 
   test 'last tag is not a div/clear when disabled' do
-    clear_on_close = SimpleShow::clear_on_close
-    SimpleShow::clear_on_close = false
-    doc = Nokogiri::HTML.fragment(simple_show_for(@philip) do |s| 
+    clear_on_close = SimpleShow.clear_on_close
+    SimpleShow.clear_on_close = false
+    doc = Nokogiri::HTML.fragment(simple_show_for(@philip) do |_s|
       ''
     end)
     assert_not_equal 'clear', doc.at('div').elements.last.try(:attr, 'class')
-    SimpleShow::clear_on_close = clear_on_close
+    SimpleShow.clear_on_close = clear_on_close
   end
-
 end

data/test/test_simple_show_label.rb

@@ -1,18 +1,16 @@
 require 'test_helper'
 
 class SimpleShowLabelTest < SimpleShowTestCase
-
   test 'labels and label overrides' do
     doc = Nokogiri::HTML.fragment(
-      simple_show_for(@philip) do |s| 
+      simple_show_for(@philip) do |s|
         o  = ActiveSupport::SafeBuffer.new
         o += s.show :name
-        o += s.show :phone, :label => 'Cell Phone'
+        o += s.show :phone, label: 'Cell Phone'
         o
       end
     )
     assert_equal 'Name:', doc.css('label').first.text
     assert_equal 'Cell Phone:', doc.css('label').last.text
   end
-
 end

data/test/test_simple_show_show.rb

@@ -1,83 +1,7 @@
 require 'test_helper'
+require 'skip_helper'
 
 class SimpleShowShowTest < SimpleShowTestCase
-
-  test ':if => true/false' do
-    doc = Nokogiri::HTML.fragment(
-      simple_show_for(@philip) do |s| 
-        o  = ActiveSupport::SafeBuffer.new
-        o += s.show :name, :if => true
-        o += s.show :phone, :if => false
-        o
-      end
-    )
-    assert_equal ['Name:'], doc.css('label').map(&:text)
-  end
-
-  test ':if => :record_method' do
-    def @philip.method_returning_true; true; end
-    def @philip.method_returning_false; false; end
-    doc = Nokogiri::HTML.fragment(
-      simple_show_for(@philip) do |s| 
-        o  = ActiveSupport::SafeBuffer.new
-        o += s.show :name, :if => :method_returning_true
-        o += s.show :phone, :if => :method_returning_false
-        o
-      end
-    )
-    assert_equal ['Name:'], doc.css('label').map(&:text)
-  end
-
-  test ':if => {lambda}' do
-    doc = Nokogiri::HTML.fragment(
-      simple_show_for(@philip) do |s| 
-        o  = ActiveSupport::SafeBuffer.new
-        o += s.show :name, :if => lambda { true }
-        o += s.show :phone, :if => lambda { false }
-        o
-      end
-    )
-    assert_equal ['Name:'], doc.css('label').map(&:text)
-  end
-
-  test ':unless => true/false' do
-    doc = Nokogiri::HTML.fragment(
-      simple_show_for(@philip) do |s| 
-        o  = ActiveSupport::SafeBuffer.new
-        o += s.show :name, :unless => true
-        o += s.show :phone, :unless => false
-        o
-      end
-    )
-    assert_equal ['Phone:'], doc.css('label').map(&:text)
-  end
-
-  test ':unless => :record_method' do
-    def @philip.method_returning_true; true; end
-    def @philip.method_returning_false; false; end
-    doc = Nokogiri::HTML.fragment(
-      simple_show_for(@philip) do |s| 
-        o  = ActiveSupport::SafeBuffer.new
-        o += s.show :name, :unless => :method_returning_true
-        o += s.show :phone, :unless => :method_returning_false
-        o
-      end
-    )
-    assert_equal ['Phone:'], doc.css('label').map(&:text)
-  end
-
-  test ':unless => {lambda}' do
-    doc = Nokogiri::HTML.fragment(
-      simple_show_for(@philip) do |s| 
-        o  = ActiveSupport::SafeBuffer.new
-        o += s.show :name, :unless => lambda { true }
-        o += s.show :phone, :unless => lambda { false }
-        o
-      end
-    )
-    assert_equal ['Phone:'], doc.css('label').map(&:text)
-  end
-
-
-
+  test_skip(:show, :name, :phone, 'Name:', 'Phone:')
 end
+

data/test/test_simple_show_show_collection.rb

@@ -0,0 +1,36 @@
+require 'test_helper'
+require 'skip_helper'
+
+class SimpleShowShowCollectionTest < SimpleShowTestCase
+  test_skip(:show_collection, :rounds, :clubs, 'Rounds:', 'Clubs:')
+
+  test 'default display works' do
+    doc = Nokogiri::HTML.fragment(
+      simple_show_for(@philip) do |s|
+        o  = ActiveSupport::SafeBuffer.new
+        o += s.show :name
+        o += s.show_collection :rounds
+        o
+      end
+    )
+    assert_equal 'Rounds:', doc.css('label').last.text
+    assert_equal [@round1, @round2].map(&:to_s).sort, doc.css('.show_collection_item').map(&:text).sort
+  end
+
+  test 'block display works' do
+    doc = Nokogiri::HTML.fragment(
+      simple_show_for(@philip) do |s|
+        o  = ActiveSupport::SafeBuffer.new
+        o += s.show :name
+        o += s.show_collection :rounds do |round|
+          nested_o  = ActiveSupport::SafeBuffer.new
+          nested_o += "#{round} - #{round.score}"
+          nested_o
+        end
+        o
+      end
+    )
+    assert_equal 'Rounds:', doc.css('label').last.text
+    assert_equal [@round1, @round2].map{|r| "#{r} - #{r.score}" }.sort, doc.css('.show_collection_item').map(&:text).sort
+  end
+end

data/test/test_simple_show_value.rb

@@ -1,31 +1,31 @@
 require 'test_helper'
 
 class SimpleShowValueTest < SimpleShowTestCase
-
   def setup
     super
     SimpleShow.setup do |config|
       config.helpers[:to_piglatin] = :piglatin
     end
     @doc = Nokogiri::HTML.fragment(
-      simple_show_for(@philip) do |s| 
+      simple_show_for(@philip) do |s|
         o  = ActiveSupport::SafeBuffer.new
         o += s.show :name
-        o += s.show(:email) {|o| o.email.upcase }
-        o += s.show :phone, :value => '867-5309'
-        o += s.show :born_on, :format => '%B %d'
-        o += s.show :born_on, :format => :mmddyy
-        o += s.show(:html) {|o| '<b>html</b>' }
-        o += s.show :handicap, :format => '%.3f'
-        o += s.show :name, :format => '%20s'
-        o += s.show :to_currency, :value => '12345.67', :to_currency => true
-        o += s.show :with_delimeter, :value => '12345.67', :with_delimiter => true
-        o += s.show :with_delimeter, :value => '12345.67', :with_delimiter => {:delimiter => ' '}
-        o += s.show :custom_helper, :value => 'piglatin', :to_piglatin => true
+        o += s.show(:email) { |o| o.email.upcase }
+        o += s.show :phone, value: '867-5309'
+        o += s.show :born_on, format: '%B %d'
+        o += s.show :born_on, format: :mmddyy
+        o += s.show(:html) { |_o| '<b>html</b>' }
+        o += s.show :handicap, format: '%.3f'
+        o += s.show :name, format: '%20s'
+        o += s.show :to_currency, value: '12345.67', to_currency: true
+        o += s.show :with_delimeter, value: '12345.67', with_delimiter: true
+        o += s.show :with_delimeter, value: '12345.67', with_delimiter: { delimiter: ' ' }
+        o += s.show :custom_helper, value: 'piglatin', to_piglatin: true
         o
       end
     )
   end
+
   test 'css class for data types' do
     assert @doc.css('span.value')[0].attr('class').split(/\s+/).include?('string') # :name
     assert @doc.css('span.value')[6].attr('class').split(/\s+/).include?('decimal') # :handicap
@@ -64,5 +64,4 @@ class SimpleShowValueTest < SimpleShowTestCase
   test 'custom helpers' do
     assert_equal 'iglatinpay', @doc.css('span.value')[11].text
   end
-
 end

metadata

@@ -1,34 +1,23 @@
---- !ruby/object:Gem::Specification 
+--- !ruby/object:Gem::Specification
 name: simple_show
-version: !ruby/object:Gem::Version 
-  hash: 25
-  prerelease: 
-  segments: 
-  - 0
-  - 1
-  - 1
-  version: 0.1.1
+version: !ruby/object:Gem::Version
+  version: 1.0.0
 platform: ruby
-authors: 
+authors:
 - Philip Hallstrom
 autorequire: 
 bindir: bin
 cert_chain: []
-
-date: 2013-09-30 00:00:00 Z
+date: 2015-05-04 00:00:00.000000000 Z
 dependencies: []
-
-description: "SimpleShow is to #show what SimpleForm is to #edit"
-email: 
+description: 'SimpleShow is to #show what SimpleForm is to #edit'
+email:
 - philip@pjkh.com
 executables: []
-
 extensions: []
-
 extra_rdoc_files: []
-
-files: 
-- .gitignore
+files:
+- ".gitignore"
 - Gemfile
 - README.rdoc
 - Rakefile
@@ -40,49 +29,44 @@ files:
 - lib/simple_show/base.rb
 - lib/simple_show/version.rb
 - simple_show.gemspec
+- test/skip_helper.rb
 - test/test_helper.rb
 - test/test_simple_show.rb
 - test/test_simple_show_for.rb
 - test/test_simple_show_label.rb
 - test/test_simple_show_show.rb
+- test/test_simple_show_show_collection.rb
 - test/test_simple_show_value.rb
-homepage: ""
-licenses: 
+homepage: ''
+licenses:
 - MIT
+metadata: {}
 post_install_message: 
 rdoc_options: []
-
-require_paths: 
+require_paths:
 - lib
-required_ruby_version: !ruby/object:Gem::Requirement 
-  none: false
-  requirements: 
+required_ruby_version: !ruby/object:Gem::Requirement
+  requirements:
   - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
-      - 0
-      version: "0"
-required_rubygems_version: !ruby/object:Gem::Requirement 
-  none: false
-  requirements: 
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements:
   - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
-      - 0
-      version: "0"
+    - !ruby/object:Gem::Version
+      version: '0'
 requirements: []
-
 rubyforge_project: simple_show
-rubygems_version: 1.8.24
+rubygems_version: 2.4.3
 signing_key: 
-specification_version: 3
-summary: "SimpleShow is to #show what SimpleForm is to #edit"
-test_files: 
+specification_version: 4
+summary: 'SimpleShow is to #show what SimpleForm is to #edit'
+test_files:
+- test/skip_helper.rb
 - test/test_helper.rb
 - test/test_simple_show.rb
 - test/test_simple_show_for.rb
 - test/test_simple_show_label.rb
 - test/test_simple_show_show.rb
+- test/test_simple_show_show_collection.rb
 - test/test_simple_show_value.rb