Merge pull request #505 from now/list-includes-in-correct-order

Fix mixin order of multiple arguments to include

lib/yard/handlers/ruby/mixin_handler.rb

@@ -5,7 +5,7 @@ class YARD::Handlers::Ruby::MixinHandler < YARD::Handlers::Ruby::Base
 
   process do
     errors = []
-    statement.parameters(false).each do |mixin|
+    statement.parameters(false).reverse.each do |mixin|
       begin
         process_mixin(mixin)
       rescue YARD::Parser::UndocumentableError => err

spec/handlers/examples/mixin_handler_001.rb.txt

@@ -19,6 +19,10 @@ module Y
   class << self; include A; end
 end
 
+module Z
+  include A, B
+end
+
 module ABC
   module FOO; end
   module DEF
@@ -30,4 +34,4 @@ module ABC
       include ABC::BAR
     end
   end
-end  
+end  

spec/handlers/mixin_handler_spec.rb

@@ -32,6 +32,10 @@
   it "should treat a mixed in Constant by taking its value as the real object name" do
     P(:Y).instance_mixins.should include(Registry.at('B::D'))
   end
+
+  it "should add includes in the correct order when include is given multiple arguments" do
+    P(:Z).instance_mixins.should == [P(:A), P(:B)]
+  end
 
   it "should avoid including self for unresolved mixins of the same name" do
     P("ABC::DEF::FOO").mixins.should == [P("ABC::FOO")]

spec/templates/examples/module001.txt

@@ -16,7 +16,7 @@ Defined Under Namespace:
 Includes:
 ---------
 
-    A::B::C, F, E, D, B, BaseMod
+    D, E, F, A::B::C, B, BaseMod
 
 
 Class methods: