model_base_generators 0.1.2 → 0.2.0

data/lib/model_base/column_attribute.rb

@@ -10,13 +10,19 @@ module ModelBase
     attr_reader :reference
     attr_reader :model, :column
 
-    def initialize(model, name, type, column: nil, reference: nil, index_type: false, attr_options: {})
+    def initialize(model, name, type, column: nil, reference: nil, index_type: false, title: false, attr_options: {})
       super(name, type, index_type, attr_options)
       @model = model
       @reference = reference
       @column = column
+      @title = !!title
     end
 
+    def title?
+      @title
+    end
+    attr_writer :title
+
     def ref_model
       unless defined?(@ref_model)
         @ref_model = reference.nil? ? nil : ModelBase::MetaModel.new(reference.class_name)
@@ -42,6 +48,61 @@ module ModelBase
       select_renderer ? :select : super
     end
 
+    LOCALIZED_TYPES = [:time, :datetime].freeze
+    def to_be_localized?
+      LOCALIZED_TYPES.include?(type)
+    end
+
+    def sample_value(idx = 1, context: nil)
+      if name == 'id'
+        idx
+      elsif name == 'email' && type == :string
+        'user1@example.com'
+      elsif title?
+        model.full_resource_name + idx.to_s
+      elsif ref_model
+        if tc = ref_model.title_column
+          tc.sample_value(idx)
+        else
+          1
+        end
+      elsif enumerized?
+        enum = model.model_class.send(name)
+        r = enum.values.first
+        context == :factory ? r.to_sym : r.text
+      else
+        @default ||= case type
+          when :integer                     then idx
+          when :float                       then idx + 0.5
+          when :decimal                     then "#{idx}.99"
+          when :datetime, :timestamp, :time then Time.now.to_s(:db)
+          when :date                        then Date.today.to_s(:db)
+          when :string                      then
+            case name
+            when 'type' then ""
+            else "#{model.full_resource_name}_#{name}_#{idx}"
+            end
+          when :text                        then "#{model.full_resource_name}_#{name}_#{idx}"
+          when :boolean                     then false
+          when :references, :belongs_to     then nil
+          else
+            ""
+        end
+      end
+    end
+
+    def sample_string(idx = 1)
+      "'%s'" % sample_value(idx)
+    end
+
+    def new_attribute_exp
+      if enumerized?
+        '%s.%s.values[%d]' % [model.name, name, 1] # 2nd value
+      else
+        "valid_parameters[:#{name}].succ"
+      end
+    end
+
     class AbstractSelectRenderer
       attr_reader :column_attr
       def initialize(column_attr)
@@ -73,7 +134,7 @@ module ModelBase
 
     class EnumerizedSelectRenderer < AbstractSelectRenderer
       def render_core(form_name, target_name, options = {})
-        "#{form_name}.select :#{column_attr.name}, #{column_attr.model.name}.#{column_attr.name}.optoins"
+        "#{form_name}.select :#{column_attr.name}, #{column_attr.model.name}.#{column_attr.name}.options"
       end
     end
   end

data/lib/model_base/config.rb

@@ -6,11 +6,22 @@ module ModelBase
 
     config_accessor(:disabled){ false }
 
-    config_accessor(:excluded_columns) do
+    base_exclusions =
       [
         /.*_checksum/,
         /.*_count/,
-      ] + %w[_id _type id created_at updated_at]
+      ] + %w[_id _type id]
+
+    config_accessor(:excluded_columns_of_show) do
+      base_exclusions
+    end
+
+    config_accessor(:excluded_columns_of_index) do
+      base_exclusions + %w[updated_at]
+    end
+
+    config_accessor(:excluded_columns_of_form) do
+      base_exclusions + %w[created_at updated_at]
     end
 
     config_accessor(:title_column_candidates) do

data/lib/model_base/generators/erb/scaffold.rb

@@ -0,0 +1,23 @@
+require "model_base"
+
+require 'rails/generators/erb/scaffold/scaffold_generator'
+
+module ModelBase
+  module Generators
+    module Erb
+
+      module Scaffold
+        def self.enable!
+          ::Erb::Generators::ScaffoldGenerator.prepend(self)
+        end
+
+        protected
+
+        def available_views
+          %w(index edit show new _form _table)
+        end
+      end
+
+    end
+  end
+end

data/lib/model_base/generators/factory_girl/model.rb

@@ -0,0 +1,37 @@
+require "model_base"
+
+require 'generators/factory_girl/model/model_generator'
+
+module ModelBase
+  module Generators
+    module FactoryGirl
+
+      module Model
+        def self.enable!
+          ::FactoryGirl::Generators::ModelGenerator.prepend(self)
+          ::FactoryGirl::Generators::ModelGenerator.extend(ClassMethod)
+          ::FactoryGirl::Generators::ModelGenerator.instance_eval do
+            source_root File.expand_path("../../../../templates/factory_girl", __FILE__)
+          end
+        end
+
+        module ClassMethod
+          # To overwrite `source_root` again, which is overwritten by factory_girl_rails
+          # https://github.com/thoughtbot/factory_girl_rails/blob/master/lib/generators/factory_girl.rb#L6-L8
+          # https://github.com/rails/rails/blob/master/railties/lib/rails/generators/base.rb#L22-L26
+          def source_root(path=nil)
+            @_source_root = path if path
+            @_source_root ||= default_source_root
+          end
+        end
+
+        private
+
+        def create_factory_file
+          template 'factory.rb', File.join(options[:dir], "#{filename}.rb")
+        end
+
+      end
+    end
+  end
+end

data/lib/model_base/meta_model.rb

@@ -30,10 +30,6 @@ module ModelBase
       full_resource_name.pluralize
     end
 
-    def columns
-      @columns ||= retrieve_columns.reject {|c| excluded?(c.name) }
-    end
-
     def model_class
       @model_class ||= name.constantize
     end
@@ -42,6 +38,11 @@ module ModelBase
       defined?(ActiveRecord) == "constant" && ActiveRecord.class == Module && model_class < ActiveRecord::Base
     end
 
+    def title_column
+      retrieve_columns unless defined?(@title_column)
+      @title_column
+    end
+
     def retrieve_columns
       raw_cols = active_record? ? model_class.columns : model_class.fields.map {|c| c[1] }
       belongs_to_refs = model_class.reflections.values.select{|ref| ref.is_a?(ActiveRecord::Reflection::BelongsToReflection) }
@@ -52,25 +53,88 @@ module ModelBase
       @title_column = nil
       ModelBase.config.title_column_candidates.each do |tcc|
         @title_column = cols.detect{|col| tcc === col.name}
-        break if @title_column
+        if @title_column
+          @title_column.title = true
+          break
+        end
       end
       @title_column.linkable = true if @title_column
       cols
     end
 
-    def excluded?(name)
-      ModelBase.config.excluded_columns.any?{|c| c === name}
+    def raw_columns
+      @raw_columns ||= retrieve_columns
     end
 
-    def title_column
-      retrieve_columns unless defined?(@title_column)
-      @title_column
+    def columns
+      @columns ||=
+        title_column ? raw_columns : [ColumnAttribute.new(self, :id, :integer, title: true)] + raw_columns
+    end
+
+    SPEC_EXCLUSED_COLS = %w[created_at updated_at]
+    def columns_for(type)
+      case type
+      when :form, :index, :show
+        columns.reject{|c| exclude_for?(type, c) }
+      when :params     then columns_for(:form).reject{|c| c.name == 'id'}
+      when :spec_index then columns_for(:index).reject{|c| SPEC_EXCLUSED_COLS.include?(c.name)}
+      when :spec_show  then columns_for(:show ).reject{|c| SPEC_EXCLUSED_COLS.include?(c.name)}
+      when :factory     then columns_for(:params).reject{|c| SPEC_EXCLUSED_COLS.include?(c.name)}
+      else
+        raise "Unknown template type: #{type.inspect}"
+      end
     end
 
-    def display_columns
-      @dispaly_columns ||=
-        title_column ? columns : [new_attribute(:id, :integer, true)] + columns
+    def exclude_for?(type, col_attr)
+      excluded_columns = ModelBase.config.send("excluded_columns_of_#{type}")
+      excluded_columns.any?{|ex| ex === col_attr.name && !col_attr.title? }
     end
 
+    def dependencies(required = true)
+      # c: c.required?
+      # r: required (the argument)
+      #
+      # | c \ r | true  | false |
+      # |-------|-------|-------|
+      # | true  | true  | true  |
+      # | false | false | true  |
+      raw_columns.select{|c| !required || c.required? }.
+        select(&:ref_model).
+        each_with_object({}){|c,d| d[c.name] = c.ref_model}
+    end
+
+    def all_dependencies(required = true)
+      dependencies.values.map{|m| [m] + m.dependencies(required).values}.flatten.uniq(&:name)
+    end
+
+    def factory_girl_options
+      dependencies.map{|attr, model| "#{attr.sub(/_id\z/, '')}: #{model.full_resource_name}"}
+    end
+
+    def factory_girl_create(extra = {})
+      factory_girl_method(:create, extra)
+    end
+
+    def factory_girl_build(extra = {})
+      factory_girl_method(:build, extra)
+    end
+
+    def factory_girl_method(name, extra)
+      extra_str = extra.blank? ? '' : ', ' << extra.map{|k,v| "#{k}: '#{v}'"}.join(', ')
+      options = factory_girl_options
+      options_str = options.empty? ? '' : ', ' <<  factory_girl_options.join(', ')
+      'FactoryGirl.%s(:%s%s%s)' % [name, full_resource_name, extra_str, options_str]
+    end
+
+    def factory_girl_let_definition
+      'let(:%s){ %s }' % [full_resource_name, factory_girl_create]
+    end
+
+    def factory_girl_let_definitions(spacer = "  ")
+      deps = all_dependencies
+      r = deps.map(&:factory_girl_let_definition)
+      r << "let(:user){ FactoryGirl.create(:user) }" unless deps.any?{|m| m.full_resource_name == 'user' }
+      r.join("\n" << spacer)
+    end
   end
 end

data/lib/model_base/version.rb

@@ -1,3 +1,3 @@
 module ModelBase
-  VERSION = "0.1.2"
+  VERSION = "0.2.0"
 end

data/lib/model_base.rb

@@ -22,6 +22,10 @@ module ModelBase
       Rails::Generators.templates_path.unshift(templates_dir)
       Rails::Generators.lookup(["rails:scaffold_controller"])
       Rails::Generators::ScaffoldControllerGenerator.source_paths.unshift(templates_dir)
+      require 'model_base/generators/erb/scaffold'
+      ::ModelBase::Generators::Erb::Scaffold.enable!
+      require 'model_base/generators/factory_girl/model'
+      ::ModelBase::Generators::FactoryGirl::Model.enable!
     end
   end
 end

data/lib/templates/controller.rb

@@ -65,7 +65,7 @@ class <%= controller_class_name %>Controller < ApplicationController
     <%- if model.columns.empty? -%>
     params[:<%= singular_table_name %>]
     <%- else -%>
-    params.require(:<%= singular_table_name %>).permit(<%= model.columns.map { |c| ":#{c.name}" }.join(', ') %>)
+    params.require(:<%= singular_table_name %>).permit(<%= model.columns_for(:params).map { |c| ":#{c.name}" }.join(', ') %>)
     <%- end -%>
   end
 end

data/lib/templates/erb/scaffold/_form.html.erb

@@ -15,7 +15,7 @@
     </div>
   <%% end %>
 
-<%- model.columns.each do |column| -%>
+<%- model.columns_for(:form).each do |column| -%>
   <div class="form-group">
     <%%= f.label :<%= column.name %>, :class => 'control-label col-lg-2' %>
     <div class="col-lg-10">

data/lib/templates/erb/scaffold/_table.html.erb

@@ -0,0 +1,39 @@
+<%%- model_class = <%= model.name.sub(/\ATemplate::/, '::Template::') %> -%>
+<table class="table table-striped">
+  <thead>
+    <tr>
+      <%- model.columns_for(:index).each do |column| -%>
+      <th><%%= model_class.human_attribute_name(:<%= column.name %>) %></th>
+      <%- end -%>
+      <th><%%=t '.actions', :default => t("helpers.actions") %></th>
+    </tr>
+  </thead>
+  <tbody>
+    <%% @<%= model.plural_full_resource_name %>.each do |<%= model.full_resource_name %>| %>
+      <tr>
+        <%- model.columns_for(:index).each do |column| -%>
+          <%- if column.linkable? -%>
+        <td><%%= link_to <%= model.full_resource_name %>.<%= column.name %>, <%= singular_controller_routing_path %>_path(<%= model.full_resource_name %>) %></td>
+          <%- elsif tcol = column.ref_model.try(:title_column) -%>
+        <td><%%= <%= model.full_resource_name %>.<%= column.reference.name %>.<%= tcol.name %> %></td>
+          <%- elsif column.enumerized? -%>
+        <td><%%= <%= model.full_resource_name %>.<%= column.name %>_text %></td>
+          <%- elsif column.to_be_localized? -%>
+        <td><%%=l <%= model.full_resource_name %>.<%= column.name %> %></td>
+          <%- else -%>
+        <td><%%= <%= model.full_resource_name %>.<%= column.name %> %></td>
+          <%- end -%>
+        <%- end -%>
+        <td>
+          <%%= link_to t('.edit', :default => t("helpers.links.edit")),
+                      edit_<%= singular_controller_routing_path %>_path(<%= model.full_resource_name %>), :class => 'btn btn-default btn-xs' %>
+          <%%= link_to t('.destroy', :default => t("helpers.links.destroy")),
+                      <%= singular_controller_routing_path %>_path(<%= model.full_resource_name %>),
+                      :method => :delete,
+                      :data => { :confirm => t('.confirm', :default => t("helpers.links.confirm", :default => 'Are you sure?')) },
+                      :class => 'btn btn-xs btn-danger' %>
+        </td>
+      </tr>
+    <%% end %>
+  </tbody>
+</table>

data/lib/templates/erb/scaffold/index.html.erb

@@ -2,44 +2,7 @@
 <div class="page-header">
   <h1><%%=t '.title', :default => model_class.model_name.human.pluralize.titleize %></h1>
 </div>
-<table class="table table-striped">
-  <thead>
-    <tr>
-      <%- model.display_columns.each do |column| -%>
-      <th><%%= model_class.human_attribute_name(:<%= column.name %>) %></th>
-      <%- end -%>
-      <th><%%= model_class.human_attribute_name(:created_at) %></th>
-      <th><%%=t '.actions', :default => t("helpers.actions") %></th>
-    </tr>
-  </thead>
-  <tbody>
-    <%% @<%= model.plural_full_resource_name %>.each do |<%= model.full_resource_name %>| %>
-      <tr>
-        <%- model.columns.each do |column| -%>
-          <%- if column.linkable? -%>
-        <td><%%= link_to <%= model.full_resource_name %>.<%= column.name %>, <%= singular_controller_routing_path %>_path(<%= model.full_resource_name %>) %></td>
-          <%- elsif tcol = column.ref_model.try(:title_column) -%>
-        <td><%%= <%= model.full_resource_name %>.<%= column.reference.name %>.<%= tcol.name %> %></td>
-          <%- elsif column.enumerized? -%>
-        <td><%%= <%= model.full_resource_name %>.<%= column.name %>_text %></td>
-          <%- else -%>
-        <td><%%= <%= model.full_resource_name %>.<%= column.name %> %></td>
-          <%- end -%>
-        <%- end -%>
-        <td><%%=l <%= model.full_resource_name %>.created_at %></td>
-        <td>
-          <%%= link_to t('.edit', :default => t("helpers.links.edit")),
-                      edit_<%= singular_controller_routing_path %>_path(<%= model.full_resource_name %>), :class => 'btn btn-default btn-xs' %>
-          <%%= link_to t('.destroy', :default => t("helpers.links.destroy")),
-                      <%= singular_controller_routing_path %>_path(<%= model.full_resource_name %>),
-                      :method => :delete,
-                      :data => { :confirm => t('.confirm', :default => t("helpers.links.confirm", :default => 'Are you sure?')) },
-                      :class => 'btn btn-xs btn-danger' %>
-        </td>
-      </tr>
-    <%% end %>
-  </tbody>
-</table>
+<%%= render '<%= plural_table_name %>/table' %>
 
 <%%= link_to t('.new', :default => t("helpers.links.new")),
             new_<%= singular_controller_routing_path %>_path,

data/lib/templates/erb/scaffold/show.html.erb

@@ -4,12 +4,14 @@
 </div>
 
 <dl class="dl-horizontal">
-<%- model.columns.each do |column| -%>
+<%- model.columns_for(:show).each do |column| -%>
   <dt><strong><%%= model_class.human_attribute_name(:<%= column.name %>) %>:</strong></dt>
   <%- if tcol = column.ref_model.try(:title_column) -%>
   <dd><%%= @<%= model.full_resource_name %>.<%= column.reference.name %>.<%= tcol.name %> %></dd>
   <%- elsif column.enumerized? -%>
   <dd><%%= @<%= model.full_resource_name %>.<%= column.name %>_text %></dd>
+  <%- elsif column.to_be_localized? -%>
+  <dd><%%=l @<%= model.full_resource_name %>.<%= column.name %> %></dd>
   <%- else -%>
   <dd><%%= @<%= model.full_resource_name %>.<%= column.name %> %></dd>
   <%- end -%>

data/lib/templates/factory_girl/factory.rb

@@ -0,0 +1,11 @@
+FactoryGirl.define do
+  factory :<%= model.full_resource_name %> do
+<%- model.columns_for(:factory).each do |col| -%>
+  <%- if col.ref_model -%>
+    association :<%= col.name.sub(/_id\z/, '') %>, factory: :<%= col.ref_model.full_resource_name %>
+  <%- else -%>
+    <%= col.name.sub(/_id\z/, '') %> <%= col.sample_value(context: :factory).inspect %>
+  <%- end -%>
+<%- end -%>
+  end
+end

data/lib/templates/rspec/scaffold/controller_spec.rb

@@ -27,21 +27,16 @@ require 'rails_helper'
 RSpec.describe <%= controller_class_name %>Controller, <%= type_metatag(:controller) %> do
 
 <%-
-  required_ref_attrs  = model.columns.select{|attr|  attr.reference && attr.required? }
-  required_data_attrs = model.columns.select{|attr| !attr.reference && attr.required? }
+  required_ref_attrs  = model.columns_for(:params).select{|attr|  attr.reference && attr.required? }
+  required_data_attrs = model.columns_for(:params).select{|attr| !attr.reference && attr.required? }
 -%>
-<%- required_ref_attrs.each do |attr| -%>
-  let(:<%= attr.reference.name %>){ FactoryGirl.create(:<%= attr.ref_model.full_resource_name %>) }
-<%- end -%>
-<%- unless required_ref_attrs.any?{|attr| attr.ref_model.name == 'User' }-%>
-  let(:user){ FactoryGirl.create(:user) }
-<%- end -%>
+  <%= model.factory_girl_let_definitions %>
   before{ devise_user_login(user) }
 
 <%-
   unless required_ref_attrs.empty?
-    extra_attributes_to_merge = ".merge(%s)" % required_ref_attrs.map{|attr| "#{attr.name}: #{attr.reference.name}.id"}.join(', ')
-    extra_attributes_for_factory = ", %s" % required_ref_attrs.map{|attr| "#{attr.reference.name}: #{attr.reference.name}"}.join(', ')
+    extra_attributes_to_merge = ".merge(%s)" % required_ref_attrs.map{|attr| "#{attr.name}: #{attr.ref_model.full_resource_name}.id"}.join(', ')
+    extra_attributes_for_factory = ", %s" % required_ref_attrs.map{|attr| "#{attr.reference.name}: #{attr.ref_model.full_resource_name}"}.join(', ')
   end
 -%>
   let(:<%= file_name %>){ FactoryGirl.create(:<%= file_name %><%= extra_attributes_for_factory %>) }
@@ -137,7 +132,7 @@ RSpec.describe <%= controller_class_name %>Controller, <%= type_metatag(:control
     context "with valid params" do
 <%- if !required_data_attrs.empty? -%>
   <%- required_data_attrs.each do |required_data_attr| -%>
-      let(:new_<%= required_data_attr.name %>){ valid_parameters[:<%= required_data_attr.name %>].succ }
+      let(:new_<%= required_data_attr.name %>){ <%= required_data_attr.new_attribute_exp %> }
   <%- end -%>
 <%- elsif !required_ref_attrs.empty? -%>
       let(:another_<%= required_ref_attrs.last.name %>){ FactoryGirl.create(:<%= required_ref_attrs.last.name %><%= extra_attributes_for_factory %>) }
@@ -170,13 +165,13 @@ RSpec.describe <%= controller_class_name %>Controller, <%= type_metatag(:control
 
       it "assigns the requested <%= file_name %> as @<%= file_name %>" do
         <%= file_name %> # To create <%= file_name %>
-        put :update, params: {:id => <%= file_name %>.to_param, :<%= file_name %> => valid_parameters}, session: valid_session
+        put :update, params: {:id => <%= file_name %>.to_param, :<%= file_name %> => new_parameters}, session: valid_session
         expect(assigns(:<%= file_name %>)).to eq(<%= file_name %>)
       end
 
       it "redirects to the <%= file_name %>" do
         <%= file_name %> # To create <%= file_name %>
-        put :update, params: {:id => <%= file_name %>.to_param, :<%= file_name %> => valid_parameters}, session: valid_session
+        put :update, params: {:id => <%= file_name %>.to_param, :<%= file_name %> => new_parameters}, session: valid_session
         expect(response).to redirect_to(<%= file_name %>)
       end
     end

data/lib/templates/rspec/scaffold/edit_spec.rb

@@ -1,16 +1,16 @@
 require 'rails_helper'
 
-<% output_attributes = model.columns.reject{|attribute| [:datetime, :timestamp, :time, :date].index(attribute.type) } -%>
 RSpec.describe "<%= ns_table_name %>/edit", <%= type_metatag(:view) %> do
+  <%= model.factory_girl_let_definitions %>
   before(:each) do
-    @<%= ns_file_name %> = assign(:<%= ns_file_name %>, FactoryGirl.create(:<%= ns_file_name %>))
+    @<%= ns_file_name %> = assign(:<%= ns_file_name %>, <%= model.factory_girl_create %>)
   end
 
   it "renders the edit <%= ns_file_name %> form" do
     render
 
     assert_select "form[action=?][method=?]", <%= ns_file_name %>_path(@<%= ns_file_name %>), "post" do
-<% for attribute in output_attributes -%>
+<% model.columns_for(:form).each do |attribute| -%>
       <%- name = attribute.respond_to?(:column_name) ? attribute.column_name : attribute.name -%>
       assert_select "<%= attribute.input_type -%>#<%= ns_file_name %>_<%= name %>[name=?]", "<%= ns_file_name %>[<%= name %>]"
 <% end -%>

data/lib/templates/rspec/scaffold/index_spec.rb

@@ -2,18 +2,28 @@ require 'rails_helper'
 
 <% output_attributes = model.columns.reject{|attribute| [:datetime, :timestamp, :time, :date].index(attribute.type) } -%>
 RSpec.describe "<%= ns_table_name %>/index", <%= type_metatag(:view) %> do
+  <%= model.factory_girl_let_definitions %>
   before(:each) do
     assign(:<%= table_name %>, [
 <% [1,2].each_with_index do |id, model_index| -%>
-      FactoryGirl.create(:<%= ns_file_name %>),
+  <%- if tc = model.title_column -%>
+      <%= model.factory_girl_create(tc.name.to_sym => tc.sample_value(model_index + 1)) %>,
+  <%- else -%>
+      <%= model.factory_girl_create %>
+  <%- end -%>
 <% end -%>
     ])
   end
 
   it "renders a list of <%= ns_table_name %>" do
     render
-<% for attribute in output_attributes -%>
-    assert_select "tr>td", :text => <%= value_for(attribute) %>.to_s, :count => 2
+<% model.columns_for(:spec_index).each do |attribute| -%>
+  <%- if attribute.ref_model || attribute.enumerized? -%>
+    assert_select "tr>td", :text => <%= attribute.sample_string %>, :count => 2
+  <%- else -%>
+    assert_select "tr>td", :text => <%= attribute.sample_string(1) %>, :count => 1
+    assert_select "tr>td", :text => <%= attribute.sample_string(2) %>, :count => 1
+  <%- end -%>
 <% end -%>
   end
 end

data/lib/templates/rspec/scaffold/new_spec.rb

@@ -2,15 +2,16 @@ require 'rails_helper'
 
 <% output_attributes = model.columns.reject{|attribute| [:datetime, :timestamp, :time, :date].index(attribute.type) } -%>
 RSpec.describe "<%= ns_table_name %>/new", <%= type_metatag(:view) %> do
+  <%= model.factory_girl_let_definitions %>
   before(:each) do
-    assign(:<%= ns_file_name %>, FactoryGirl.build(:<%= ns_file_name %>))
+    assign(:<%= ns_file_name %>, <%= model.factory_girl_build %>)
   end
 
   it "renders new <%= ns_file_name %> form" do
     render
 
     assert_select "form[action=?][method=?]", <%= index_helper %>_path, "post" do
-<% for attribute in output_attributes -%>
+<% model.columns_for(:form).each do |attribute| -%>
       <%- name = attribute.respond_to?(:column_name) ? attribute.column_name : attribute.name -%>
       assert_select "<%= attribute.input_type -%>#<%= ns_file_name %>_<%= name %>[name=?]", "<%= ns_file_name %>[<%= name %>]"
 <% end -%>

data/lib/templates/rspec/scaffold/show_spec.rb

@@ -1,15 +1,15 @@
 require 'rails_helper'
 
-<% output_attributes = model.columns.reject{|attribute| [:datetime, :timestamp, :time, :date].index(attribute.type) } -%>
 RSpec.describe "<%= ns_table_name %>/show", <%= type_metatag(:view) %> do
+  <%= model.factory_girl_let_definitions %>
   before(:each) do
-    @<%= ns_file_name %> = assign(:<%= ns_file_name %>, FactoryGirl.create(:<%= ns_file_name %>))
+    @<%= ns_file_name %> = assign(:<%= ns_file_name %>, <%= model.factory_girl_create %>)
   end
 
   it "renders attributes in <p>" do
     render
-<% for attribute in output_attributes -%>
-    expect(rendered).to match(/<%= raw_value_for(attribute) %>/)
+<% model.columns_for(:spec_show).each do |attribute| -%>
+    expect(rendered).to match(/<%= attribute.sample_value %>/)
 <% end -%>
   end
 end