graphql: make coverage recording thread safe (#739)

Author: tsiq-karold

webtau-graphql/src/main/java/org/testingisdocumenting/webtau/graphql/GraphQLCoveredQueries.java

@@ -16,21 +16,20 @@
 
 package org.testingisdocumenting.webtau.graphql;
 
-import java.util.LinkedHashMap;
-import java.util.LinkedHashSet;
 import java.util.Map;
 import java.util.Set;
+import java.util.concurrent.ConcurrentHashMap;
 import java.util.stream.Stream;
 
 public class GraphQLCoveredQueries {
     private final Map<GraphQLQuery, Set<Call>> actualCallsIdsByQuery;
 
     GraphQLCoveredQueries() {
-        actualCallsIdsByQuery = new LinkedHashMap<>();
+        actualCallsIdsByQuery = new ConcurrentHashMap<>();
     }
 
     public void add(GraphQLQuery query, String id, long elapsedTime) {
-        Set<Call> calls = actualCallsIdsByQuery.computeIfAbsent(query, k -> new LinkedHashSet<>());
+        Set<Call> calls = actualCallsIdsByQuery.computeIfAbsent(query, k -> ConcurrentHashMap.newKeySet());
 
         calls.add(new Call(id, elapsedTime));
     }

webtau-graphql/src/test/groovy/org/testingisdocumenting/webtau/graphql/GraphQLReportDataProviderTest.groovy

@@ -41,7 +41,7 @@ class GraphQLReportDataProviderTest {
     void "computes timing per query"() {
         def timeStats = reportDataProvider.provide(null)
             .find {it.getId() == "graphQLQueryTimeStatistics" }
-            .getData()
+            .getData() as Set
         def expectedStats = [
             [
                 name: 'allTasks',
@@ -67,7 +67,7 @@ class GraphQLReportDataProviderTest {
                     p99: 6
                 ]
             ]
-        ]
+        ] as Set
         timeStats.should == expectedStats
     }