Merge pull request #101 from jfacorro/maintain.argument.order

Maintain original order in lists
inaka · May 29, 2017 · 7861a17 · 7861a17
2 parents 629c27d + 81a44ca
commit 7861a17
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/cowboy_swagger.erl b/src/cowboy_swagger.erl
@@ -184,7 +184,7 @@ normalize_list_values(List) ->
       (V, Acc) ->
         [V | Acc]
       end,
-  lists:foldl(F, [], List).
+  lists:foldr(F, [], List).
 
 %% @private
 validate_swagger_map(Map) ->

diff --git a/test/cowboy_swagger_SUITE.erl b/test/cowboy_swagger_SUITE.erl
@@ -47,12 +47,12 @@ to_json_test(_Config) ->
           <<"parameters">> := [
             #{<<"description">> := <<"bla">>,
               <<"in">> := <<"path">>,
-              <<"name">> := <<"c">>,
+              <<"name">> := <<"b">>,
               <<"required">> := false,
               <<"type">> := <<"string">>},
             #{<<"description">> := <<"bla">>,
               <<"in">> := <<"path">>,
-              <<"name">> := <<"b">>,
+              <<"name">> := <<"c">>,
               <<"required">> := false,
               <<"type">> := <<"string">>}
           ],
@@ -87,12 +87,12 @@ to_json_test(_Config) ->
           <<"parameters">> := [
             #{<<"description">> := <<"bla">>,
               <<"in">> := <<"path">>,
-              <<"name">> := <<"c">>,
+              <<"name">> := <<"b">>,
               <<"required">> := false,
               <<"type">> := <<"string">>},
             #{<<"description">> := <<"bla">>,
               <<"in">> := <<"path">>,
-              <<"name">> := <<"b">>,
+              <<"name">> := <<"c">>,
               <<"required">> := false,
               <<"type">> := <<"string">>}
           ]