bldr 0.7.0 → 1.0.0

data/.gitignore

@@ -1,3 +1,5 @@
 .rspec
 Gemfile.lock
 pkg
+/log
+coverage

data/Gemfile

@@ -1,10 +1,12 @@
-source :rubygems
+source 'https://rubygems.org'
 
 gemspec
 
+gem 'coveralls', require: false
 gem "rake"
 
 group :test do
+  gem 'rails', '~>3.2'
   gem "rspec", ">= 2.6.0"
   gem 'debugger', :platform => :mri_19
 end

data/HISTORY.md

@@ -1,7 +1,8 @@
-## 0.7.0 (2013-xx-xx)
-* Support for inherited instance variables
+## 0.7.0 (2013-02-24)
+* Full support for instance variables
 * Breaking change: Node#attribute block syntax no longer inherits
   context from current_object. See 941608e7 for more
+* Block variables for `object` and `collection` methods
 * Breaking change: Drop ruby 1.8 support
 
 ## 0.6.1 (2013-02-18) -- yanked

data/README.md

@@ -3,11 +3,67 @@
 
 # Bldr
 
-Bldr is a minimalist templating library that provides a simple DSL for generating
-json documents from ruby objects. It currently supports Sinatra out of
-the box -- Rails 3 support is planned for the near future.
+Bldr is a minimalist templating DSL that provides a simple syntax for generating
+json documents from ruby objects. Bldr supports Sinatra and Rails
+3.2.
 
-If you would like to contribute, pull requests with specs are warmly accepted :)
+Bldr enables to quickly generate json documents from ruby with a
+simple and intuitive DSL.
+
+```ruby
+# app/views/posts/index.json.bldr
+
+collection @posts do |post|
+  attributes :title, :body, :created_at, :slug
+  
+  object :author => post.author do |author|
+    attribute(:name) { author.display_name }
+  end
+
+  collection :comments => post.comments do |comment|
+    attribute :spamminess
+    attribute :created_at
+    attribute(:body) do
+      xss_filter(comment.body)
+    end
+  end
+end
+```
+
+This would output the following json document:
+
+```json
+[
+  {
+    "title": "Some title",
+    "body": "blah blah",
+    "slug": "some-title",
+    "created_at": "2013-04-11T15:46:17-07:00",
+    "author": {
+      "name": "Joe Author"
+    },
+    "comments": [
+      {
+        "spamminess": 1.0,
+        "created_at": "2013-04-11T15:46:17-07:00",
+        "body": "a comment"
+      }
+    ]
+  }
+]
+```
+
+## Usage
+
+Bldr is a very concise DSL, containing only four core methods:
+
+* `object`
+* `collection`
+* `attribute`
+* `attributes`
+
+These four methods provide a great deal of power and flexibility in describing
+json responses.
 
 ## Why
 
@@ -15,18 +71,29 @@ If you're building an API, `Model#to_json` just doesn't cut it. Besides the JSON
 representation of your models arguably being a presentation concern, trying
 to cram all of this logic into an `#as_json` method quickly turns into pure chaos.
 
-There are other json templating libraries available -- [rabl](http://github.com/nesquena/rabl) being the most popular -- but I wasn't satisfied with any of the DSL's, so I created Bldr.
+There are other json templating libraries available such as
+[rabl](https://github.com/nesquena/rabl) or [json_builder](https://github.com/dewski/json_builder).
+Bldr is in the same vein as these libraries, but with a simpler synxtax.
 
-## Features
+## Usage & Examples
 
-* Simple json templating DSL
-* Uses Tilt's built-in rendering and template caching for better performance
-* Partials
+See [Examples on the wiki](https://github.com/ajsharp/bldr/wiki/Documentation-&-Examples)
+for documentation and usage examples.
 
 ## Installation
 
-There are two ways to use Bldr in your Sinatra app, depending on whether
-you are using Sinatra's classic or module application style:
+In your gemfile:
+
+```ruby
+gem 'bldr'
+```
+
+## Configuration
+
+No additional configuration is required for rails applications.
+
+For sinatra apps, dependening on whether you're using a modular or classic
+application style, do one of the following:
 
 ```ruby
 
@@ -48,10 +115,6 @@ class MyApp < Sinatra::Base
 end
 ```
 
-## Usage
-
-See the [Documentation & Examples](https://github.com/ajsharp/bldr/wiki/Documentation-&-Examples) page on the wiki.
-
 ## Deprecations & Breaking Changes
 
 ### 0.7.0: current_object deprecation
@@ -123,16 +186,24 @@ See [941608e](https://github.com/ajsharp/bldr/commit/d0bfbd8) and [d0bfbd8](http
 
 ## Editor Syntax Support
 
-To get proper syntax highlighting in vim, add this line to your .vimrc:
+### Vim
+
+Add this line to your .vimrc:
 
 ```
 au BufRead,BufNewFile *.bldr set filetype=ruby
 ```
 
+### Emacs
+
+Add this to your `~/.emacs.d/init.el`:
+
+```
+(add-to-list 'auto-mode-alist '("\\.bldr$" . ruby-mode))
+```
+
 ## TODO
 
-* Rails 3 support.  An attempt for this was made for this but was reverted in e1cfd7fcbe130b316d95773d8c73ece4e247200e.  Feel free to take a shot.
-* Replace current_object with a block param for collection methods
 * XML support
 
 ## Acknowledgements

data/lib/action_view/template/handlers/bldr.rb

@@ -0,0 +1,24 @@
+module ActionView::Template::Handlers
+  class Bldr
+
+    # @param [ActionView::Template] template the template instance
+    # @return [String] the rendered ruby code string to render the template
+    def self.call(template, opts = {})
+      source = if template.source.empty?
+        File.read(template.identifier)
+      else
+        template.source
+      end
+
+      %{
+        node = ::Bldr::Node.new(nil, parent: self, root: true) {
+          #{source}
+        }
+        MultiJson.encode node.result
+      }
+    end
+
+  end
+end
+
+ActionView::Template.register_template_handler :bldr, ActionView::Template::Handlers::Bldr

data/lib/bldr.rb

@@ -5,6 +5,7 @@ end
 
 require 'multi_json'
 require 'bldr/node'
+require 'bldr/railtie' if defined?(Rails)
 
 if defined?(Tilt)
   require 'bldr/template'

data/lib/bldr/node.rb

@@ -1,12 +1,22 @@
+require 'forwardable'
 
 module Bldr
 
   class Node
+    extend Forwardable
 
+    # These do not get copied into child nodes. All other instance variables do.
     PROTECTED_IVARS = [:@current_object, :@result, :@parent, :@opts, :@views, :@locals]
 
+    # List of bldr public api method. So we don't overwrite them when we do
+    # crazy ruby metaprogramming when we build nodes.
+    API_METHODS = [:object, :collection, :attribute, :attributes]
+
     attr_reader :current_object, :result, :parent, :opts, :views, :locals
 
+    # @!attribute [r] request params from a rails or sinatra controller
+    attr_reader :params
+
     # Initialize a new Node instance.
     #
     # @example Building a simple node
@@ -20,6 +30,9 @@ module Bldr
     #
     # @param [Object] value an object to serialize.
     # @param [Hash] opts
+    # @option opts [Object] :parent The parent object is used to copy instance variables
+    #   into each node in the node tree.
+    # @option opts [Boolean] :root indicates whether this is the root node or not
     # @option [Object] opts :parent used to copy instance variables into self
     def initialize(value = nil, opts = {}, &block)
       @current_object = value
@@ -27,10 +40,27 @@ module Bldr
       @parent         = opts[:parent]
       @views          = opts[:views]
       @locals         = opts[:locals]
-      # Storage hash for all descendant nodes
-      @result         = {}
+      @result         = {} # Storage hash for all descendant nodes
 
-      copy_instance_variables_from(opts[:parent]) if opts[:parent]
+      # opts[:parent] will only get set to an ActionView::Base instance
+      # when rails renders a bldr template. This logic doesn't belong here,
+      # and there's a concept to be extracted here.
+      #
+      # The upshot of initializing the root node like this is that all child
+      # nodes will have access to rails helper methods. This is necessary
+      # due to the way bldr makes judicious use of instance_eval.
+      #
+      # @todo refactor this
+      if opts[:root] && @parent
+        # assign @parent to @view so it will be copied down to each child nod
+        # @parent is in PROTECTED_IVARS and won't be copied. This effectively
+        # gives all child nodes access to helper methods passed into the parent
+        @view = @parent
+      end
+
+      copy_instance_variables(@parent) if @parent
+      delegate_helpers if @view
+      assign_params if @parent
 
       if block_given?
         if value && block.arity > 0
@@ -94,12 +124,11 @@ module Bldr
           value = nil
         end
 
-        # Short circuit here if the object passed in pointed
-        # at a nil value. There's some debate about how this
-        # should behave by default -- should it build the keyspace,
-        # pointing a null value, or should it leave the key out.
-        # With this implementation, it leaves the keyspace out.
-        return nil if value.nil? and keyed_object?(base)
+        # handle nil objects
+        if value.nil? && keyed_object?(base)
+          merge_result!(key, nil)
+          return self
+        end
 
         node  = Node.new(value, opts.merge(:parent => self), &block)
         merge_result!(key, node.result)
@@ -160,7 +189,7 @@ module Bldr
       else
         @result = massage_value(vals)
       end
-        
+
       self
     end
 
@@ -173,28 +202,7 @@ module Bldr
     #
     # @example Attribute aliasing
     #   object :person => dude do
-    #     attributes :surname => :last_name
-    #   end
-    #
-    # @example Dynamic attributes (explicit object context)
-    #   object :person => employee do
-    #     collection :colleagues => employee.colleagues do |colleague|
-    #       attribute :isBoss do |colleague|
-    #         employee.works_with?(colleague) && colleague.admin?
-    #       end
-    #     end
-    #   end
-    #
-    # @example Dynamic attributes (implicit object context)
-    #   object :person => dude do
-    #     collection :colleagues => employee.colleagues do |colleague|
-    #       attribute :rank do
-    #         # method called on colleague
-    #         if admin? && superior_to?(employee)
-    #           "High Up"
-    #         end
-    #       end
-    #     end
+    #     attributes :surname => :last_name # invokes dude.last_name
     #   end
     #
     # @return [Nil]
@@ -213,25 +221,61 @@ module Bldr
       self
     end
 
-    def attribute(*args,&block)
+    # @example Dynamic attributes
+    #   object :person => employee do
+    #     collection :colleagues => employee.colleagues do |colleague|
+    #       attribute :isBoss do
+    #         employee.works_with?(colleague) && colleague.admin?
+    #       end
+    #     end
+    #   end
+    #
+    def attribute(*args, &block)
       if block_given?
-        raise(ArgumentError, "You may only pass one argument to #attribute when using the block syntax.") if args.size > 1
-        raise(ArgumentError, "You cannot use a block of arity > 0 if current_object is not present.") if block.arity > 0 and @current_object.nil?
+        # e.g. attribute(:one, :two) { "value" }
+        if args.size > 1
+          raise(ArgumentError, "You may only pass one argument to #attribute when using the block syntax.")
+        end
+
+        # e.g.
+        # object do
+        #   attribute { 'value' }
+        # end
+        if block.arity > 0 && @current_object.nil?
+          raise(ArgumentError, "You cannot use a block of arity > 0 if current_object is not present.")
+        end
+
         if block.arity > 0
+          # object(person: @person) do
+          #   attribute(:name) { |person| person.name }
+          # end
           merge_result! args.first, block.call(@current_object)
         else
+          # object(person: @person) do
+          #   attribute(:name) # i.e. @person.name
+          # end
           merge_result! args.first, block.call
         end
       else
         case args.size
-        when 1 # inferred object
+        when 1
+          # object do
+          #   attribute(:name)
+          # end
           raise(ArgumentError, "#attribute can't be used when there is no current_object.") if @current_object.nil?
           if args[0].is_a?(Hash)
+            # object(person: @person) do
+            #   attribute :key => :display_name # i.e. @person.display_name
+            # end
             merge_result!(args[0].keys.first, @current_object.send(args[0].values.first))
           else
+            # object(person: @person) do
+            #   attribute :name
+            # end
             merge_result!(args[0], @current_object.send(args[0]))
           end
-        when 2 # static property
+        when 2
+          # attribute :name, @person.name
           merge_result!(args[0], args[1])
         else
           raise(ArgumentError, "You cannot pass more than two arguments to #attribute.")
@@ -246,7 +290,7 @@ module Bldr
     #   object :person => dude do
     #     template "path/to/template"
     #   end
-    # 
+    #
     # @example Using locals
     #   object :person => dude do
     #     template "path/to/template", :locals => {:foo => 'bar'}
@@ -269,13 +313,32 @@ module Bldr
     #   current scope.
     #
     # @param [Object] object The object to copy instance variables from.
-    def copy_instance_variables_from(object)
+    def copy_instance_variables(object)
       ivar_names = (object.instance_variables - PROTECTED_IVARS).map(&:to_s)
       ivar_names.map do |name|
         instance_variable_set(name, object.instance_variable_get(name))
       end
     end
 
+    # Delegate helper methods on the @view to @view
+    def delegate_helpers
+      # ActionView::Base instances carry a method called helpers,
+      # which is a module that contains helper methods available in a rails
+      # controller.
+      @_helpers = @view.helpers if @view.respond_to?(:helpers)
+
+      # Delegate all helper methods, minus those with the same name as any
+      # bldr api methods to @view via this object's metaclass
+      if @_helpers
+        (class << self; self; end).def_delegators :@view, *(@_helpers.instance_methods - API_METHODS)
+      end
+    end
+
+    # Assigns the params attribute from the parent.
+    def assign_params
+      @params = @parent.params if @parent.respond_to?(:params)
+    end
+
     # Determines if an object was passed in with a key pointing to it, or if
     # it was passed in as the "root" of the current object. Essentially, this
     # checks if `obj` quacks like a hash.
@@ -285,7 +348,7 @@ module Bldr
     def keyed_object?(obj)
       obj.respond_to?(:keys)
     end
-    
+
     def find_template(template)
       path = []
       path << views if views
@@ -312,6 +375,6 @@ module Bldr
         val
       end
     end
-    
+
   end
 end

data/lib/bldr/railtie.rb

@@ -0,0 +1,9 @@
+module Bldr
+  class Railtie < Rails::Railtie
+    initializer 'bldr.initialize' do |app|
+      ActiveSupport.on_load(:action_view) do
+        require 'action_view/template/handlers/bldr'
+      end
+    end
+  end
+end

data/lib/bldr/version.rb

@@ -1,4 +1,4 @@
 
 module Bldr
-  VERSION = '0.7.0'
+  VERSION = '1.0.0'
 end

data/spec/fixtures/templates/rails/people/index.json.bldr

@@ -0,0 +1,4 @@
+
+collection @people do
+  attribute :name
+end

data/spec/fixtures/templates/rails/people/show.json.bldr

@@ -0,0 +1,7 @@
+
+attribute(:id) { params[:id] }
+attribute(:name) { @person.name }
+
+if params[:use_boss_helper]
+  attribute(:boss) { boss?(@person) }
+end

data/spec/functional/helpers_spec.rb

@@ -0,0 +1,57 @@
+require 'spec_helper'
+
+module Bldr
+  describe Node, 'delegating helper methods' do
+    before do
+      # mock version of a module of methods that would be attached
+      # to an ActionView::Base instance
+      helpers = Module.new do
+        def my_helper
+          'my helper'
+        end
+
+        def helper_with_args_and_block(one, two)
+          yield if block_given?
+        end
+      end
+
+      # set up a mock ActionView::Base instance
+      mock_action_view = Struct.new(:helpers)
+      @view = mock_action_view.new(helpers)
+      @view.extend(helpers)
+
+      @node = Node.new(nil, root: true, parent: @view)
+    end
+
+    it 'delegates the methods to the parent object' do
+      @view.should_receive(:my_helper)
+      @node.my_helper
+    end
+
+    it 'gives access to helper methods to child nodes' do
+      node = Node.new(nil, root: true, parent: @view) do
+        object(:foo => Object.new) do
+          attribute(:bar) { my_helper }
+        end
+      end
+      node.result.should == {foo: {bar: 'my helper'}}
+    end
+
+    it 'assigns opts[:parent] as a @view instance variable' do
+      @node.instance_variable_get(:@view).should == @view
+    end
+
+    it 'delegates arguments and blocks to the parent' do
+      lam = lambda { }
+      @view.should_receive(:helper_with_args_and_block).with(1, 2, lam)
+      @node.helper_with_args_and_block(1, 2, lam)
+    end
+
+    it 'defines helper methods on a per-instance basis' do
+      @node.methods.should include :my_helper
+
+      new_parent = Struct.new(nil).new
+      Node.new(nil, root: true, parent: new_parent).methods.should_not include :my_helper
+    end
+  end
+end

data/spec/functional/params_spec.rb

@@ -0,0 +1,26 @@
+require 'spec_helper'
+
+module Bldr
+  describe Node, 'access to the params hash' do
+    let(:params) { {foo: 'bar'} }
+    let(:parent) { Struct.new(:params).new(params) }
+
+    it 'has access in the root node' do
+      Node.new(nil, parent: parent) do
+        attribute(:foo) { params[:foo] }
+      end.result.should == {
+        foo: 'bar'
+      }
+    end
+
+    it 'has access in child nodes' do
+      Node.new(nil, parent: parent) do
+        object(:foo) do
+          attribute(:baz) { params[:foo] }
+        end
+      end.result.should == {
+        foo: {baz: 'bar'}
+      }
+    end
+  end
+end

data/spec/integration/rails_32_spec.rb

@@ -0,0 +1,84 @@
+require 'spec_helper'
+require 'rails'
+require 'action_controller/railtie'
+require 'bldr/railtie'
+
+describe 'a template for a rails controller' do
+  TestRailsApp = Class.new(Rails::Application) do
+    routes.append do
+      resources :people
+    end
+
+    config.secret_token = "secret"
+    config.session_store = :disabled
+    config.active_support.deprecation = nil
+    config.middleware.delete 'ActionDispatch::Session::CookieStore'
+  end
+
+  class PeopleController < ActionController::Base
+    helper_method :boss?
+
+    # GET /people
+    def index
+      @people = [Person.new('Dave Chappelle'), Person.new('Chris Rock')]
+      render 'spec/fixtures/templates/rails/people/index', handlers: [:bldr], formats: [:json]
+    end
+
+    # GET /people/:id
+    def show
+      @person = Person.new('Dave Chappelle')
+      render 'spec/fixtures/templates/rails/people/show', handlers: [:bldr], formats: [:json]
+    end
+
+    private
+    def boss?(person)
+      person.name == 'Dave Chappelle'
+    end
+  end
+
+  TestRailsApp.initialize!
+
+  def app
+    TestRailsApp.app
+  end
+
+  def get(url)
+    Rack::MockRequest.new(app).get(url)
+  end
+
+  def decode(d)
+    MultiJson.decode(d)
+  end
+
+  it 'returns 200' do
+    get('/people').status.should == 200
+  end
+
+  it 'returns a json response body' do
+    decode(get('/people').body).should == [
+      {'name' => 'Dave Chappelle'},
+      {'name' => 'Chris Rock'}
+    ]
+  end
+
+  it 'returns json content type' do
+    get('/people').content_type.should =~ %r{application/json}
+  end
+
+  it 'has access to controller helper methods' do
+    response = get('/people/123?use_boss_helper=true')
+    response.status.should == 200
+    decode(response.body).should == {
+      'id'   => '123',
+      'name' => 'Dave Chappelle',
+      'boss' => true
+    }
+  end
+
+  it 'has access to params' do
+    decode(get('/people/123').body).should == {
+      'id' => '123',
+      'name' => 'Dave Chappelle'
+    }
+  end
+end

data/spec/integration/sinatra_spec.rb

@@ -11,6 +11,19 @@ describe "Using Bldr with a sinatra app" do
     disable :show_exceptions
     enable  :raise_errors
 
+    get '/people/:id' do
+      @alex = Person.new("alex", 25)
+
+      tpl = %{
+object(:person => @alex) do
+  attribute(:id) { params['id'] }
+  attribute :name
+end
+      }
+      status 200
+      bldr tpl
+    end
+
     get '/' do
       alex =  Person.new("alex", 25)
       tpl = <<-RUBY
@@ -67,6 +80,13 @@ describe "Using Bldr with a sinatra app" do
     end
   end
 
+  it 'has access to the params hash in bldr templates' do
+    response = Rack::MockRequest.new(TestApp).get('/people/123')
+    decode(response.body).should == {
+      'person' => { 'id' => '123', 'name' => 'alex'}
+    }
+  end
+
   it 'passes ivars through to the template' do
     response = Rack::MockRequest.new(TestApp).get('/ivar')
     decode(response.body).should == {'person' => {'name' => 'bert', 'age' => 99}}

data/spec/spec_helper.rb

@@ -1,4 +1,10 @@
 require 'rubygems'
+
+if ENV['TRAVIS']
+  require 'coveralls'
+  Coveralls.wear!
+end
+
 require 'rspec'
 
 require 'yajl'

data/spec/unit/node_spec.rb

@@ -163,6 +163,19 @@ module Bldr
   end
 
   describe Node, "#object" do
+    it 'renders the object structure for a nil object' do
+      node = Node.new do
+        object :person => nil do
+          attributes :name
+        end
+        attribute(:foo) { "bar" }
+      end
+      node.result.should == {
+        person: nil,
+        foo: 'bar'
+      }
+    end
+
     it 'is passes block the block variable to the block' do
       denver = Person.new('John Denver')
       node = Node.new do
@@ -231,7 +244,6 @@ module Bldr
         end
       end
 
-
       describe "#attributes" do
         describe "when an object key is passed a null value" do
           subject {
@@ -250,13 +262,14 @@ module Bldr
             }.not_to raise_error(ArgumentError, ERROR_MESSAGES[:attributes_inferred_missing])
           end
 
-          its(:result) { should == {} }
+          its(:result) { should == {:person => nil} }
         end
 
         it "renders each argument against the inferred object" do
           node = wrap { attributes(:name, :age) }
           node.result.should == {:person => {:name => 'alex', :age => 25}}
         end
+
         it "renders nil attributes" do
           node = node_wrap do
             object :person => Person.new('alex') do
@@ -266,9 +279,7 @@ module Bldr
 
           node.result.should == {:person => {:name => 'alex', :age => nil}}
         end
-
       end
-
     end
 
     describe "embedded objects" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: bldr
 version: !ruby/object:Gem::Version
-  version: 0.7.0
+  version: 1.0.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-02-25 00:00:00.000000000 Z
+date: 2013-05-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: multi_json
@@ -124,8 +124,10 @@ files:
 - README.md
 - Rakefile
 - bldr.gemspec
+- lib/action_view/template/handlers/bldr.rb
 - lib/bldr.rb
 - lib/bldr/node.rb
+- lib/bldr/railtie.rb
 - lib/bldr/template.rb
 - lib/bldr/version.rb
 - lib/sinatra/bldr.rb
@@ -140,8 +142,13 @@ files:
 - spec/fixtures/root_partial.bldr
 - spec/fixtures/root_template.json.bldr
 - spec/fixtures/some/include.json.bldr
+- spec/fixtures/templates/rails/people/index.json.bldr
+- spec/fixtures/templates/rails/people/show.json.bldr
 - spec/functional/handlers_spec.rb
+- spec/functional/helpers_spec.rb
+- spec/functional/params_spec.rb
 - spec/functional/tilt_template_spec.rb
+- spec/integration/rails_32_spec.rb
 - spec/integration/sinatra_spec.rb
 - spec/models/comment.rb
 - spec/models/person.rb
@@ -170,7 +177,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: bldr
-rubygems_version: 1.8.24
+rubygems_version: 1.8.25
 signing_key: 
 specification_version: 3
 summary: Templating library with a simple, minimalist DSL.
@@ -183,8 +190,13 @@ test_files:
 - spec/fixtures/root_partial.bldr
 - spec/fixtures/root_template.json.bldr
 - spec/fixtures/some/include.json.bldr
+- spec/fixtures/templates/rails/people/index.json.bldr
+- spec/fixtures/templates/rails/people/show.json.bldr
 - spec/functional/handlers_spec.rb
+- spec/functional/helpers_spec.rb
+- spec/functional/params_spec.rb
 - spec/functional/tilt_template_spec.rb
+- spec/integration/rails_32_spec.rb
 - spec/integration/sinatra_spec.rb
 - spec/models/comment.rb
 - spec/models/person.rb
@@ -193,3 +205,4 @@ test_files:
 - spec/spec_helper.rb
 - spec/unit/bldr_spec.rb
 - spec/unit/node_spec.rb
+has_rdoc: