Merge pull request ruby-grape#238 from tim-vandecasteele/master

Also parse params when no description is given

Author: dblock

lib/grape/validations.rb

@@ -172,13 +172,12 @@ def params(&block)
       end
 
       def document_attribute(names, opts)
-        if @last_description
-          @last_description[:params] ||= {}
+        @last_description ||= {}
+        @last_description[:params] ||= {}
 
-          Array(names).each do |name|
-            @last_description[:params][name[:name].to_s] ||= {}
-            @last_description[:params][name[:name].to_s].merge!(opts).merge!({:full_name => name[:full_name]})
-          end
+        Array(names).each do |name|
+          @last_description[:params][name[:name].to_s] ||= {}
+          @last_description[:params][name[:name].to_s].merge!(opts).merge!({:full_name => name[:full_name]})
         end
       end
 

spec/grape/api_spec.rb

@@ -1073,6 +1073,17 @@ class CommunicationError < RuntimeError; end
         { :description => "nesting", :params => { "root_param" => { :required => true, :desc => "root param", :full_name=>"root_param" }, "nested_param" => { :required => true, :desc => "nested param", :full_name=>"nested[nested_param]" } } }
       ]
     end
+    it "should parse parameters when no description is given" do
+      subject.params do
+        requires :one_param, :desc => "one param"
+      end
+      subject.get "method" do ; end
+      subject.routes.map { |route|
+        { :description => route.route_description, :params => route.route_params }
+      }.should eq [
+        { :description => nil, :params => { "one_param" => { :required => true, :desc => "one param", :full_name=>"one_param" } } }
+      ]
+    end
     it "should not symbolize params" do
       subject.desc "Reverses a string.", { :params =>
         { "s" => { :desc => "string to reverse", :type => "string" }}