Merge pull request #13 from DeloitteDigitalAPAC/mm/fix-enumerate-args

Fix enumeration of arguments
DeloitteOptimalReality · Feb 26, 2022 · 81ac5b2 · 81ac5b2 · mmiller-max · Feb 26, 2022
2 parents 3157bb8 + 5ffd5de
commit 81ac5b2
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 2 deletions.
diff --git a/Project.toml b/Project.toml
@@ -1,6 +1,6 @@
 name = "GraphQLClient"
 uuid = "09d831e3-9c21-47a9-bfd8-076871817219"
-version = "0.7.3"
+version = "0.7.4"
 
 [deps]
 GraphQLParser = "0ae10fbf-af58-4883-b66b-ff0ac82d20dd"

diff --git a/src/args.jl b/src/args.jl
@@ -84,7 +84,7 @@ function get_query_args_str(args::AbstractDict)
                 end
                 if enumerate > 0
                     name = fieldname * "__" * string(enumerate)
-                    str *= string(enumerate)
+                    str *= "__" *string(enumerate)
                 end
 
                 # Add to tracker

diff --git a/test/args.jl b/test/args.jl
@@ -57,6 +57,8 @@ end
     ])
     str, arg_names = GraphQLClient.get_query_args_str(args)
     @test length(unique(vcat(collect(values(arg_names["vec"][1])), collect(values(arg_names["vec"][2]))))) == 4 # four unique names
+    @test all(key -> occursin(key, str), keys(arg_names["vec"][1]))  # Check all arg_names are in str
+    @test all(key -> occursin(key, str), keys(arg_names["vec"][2]))  # Check all arg_names are in str
 
     # Different fields with same keys
     args = Dict(