fix: #99 Handle nil invocation stats and raise timer for race conditions

Signed-off-by: Laurent Broudoux <laurent.broudoux@gmail.com>

Author: lbroudoux

internal/test/helper.go

@@ -81,6 +81,11 @@ func MicrocksMockingFunctionality(t *testing.T, ctx context.Context, microcksCon
 	baseApiUrl, err := microcksContainer.RestMockEndpoint(ctx, "API Pastries", "0.0.1")
 	require.NoError(t, err)
 
+	// Check it has not yet been called once.
+	callCount, err := microcksContainer.ServiceInvocationsCount(ctx, "API Pastries", "0.0.1")
+	require.NoError(t, err)
+	require.Equal(t, 0, callCount)
+
 	resp, err := http.Get(baseApiUrl + "/pastries/Millefeuille")
 	require.NoError(t, err)
 	require.Equal(t, http.StatusOK, resp.StatusCode)
@@ -99,7 +104,7 @@ func MicrocksMockingFunctionality(t *testing.T, ctx context.Context, microcksCon
 	require.NoError(t, err)
 	require.True(t, called)
 
-	callCount, err := microcksContainer.ServiceInvocationsCount(ctx, "API Pastries", "0.0.1")
+	callCount, err = microcksContainer.ServiceInvocationsCount(ctx, "API Pastries", "0.0.1")
 	require.NoError(t, err)
 	require.Equal(t, 1, callCount)
 

microcks.go

@@ -384,11 +384,17 @@ func (container *MicrocksContainer) ServiceInvocationsCountAtDate(ctx context.Co
 	day := formatDay(date)
 
 	// To avoid race condition issue while Microcks server is processing metrics asynchronously.
-	time.Sleep(100 * time.Millisecond)
+	// The wait time is lower on other language bindings (100ms). Don't know why...
+	time.Sleep(250 * time.Millisecond)
 	stats, err := c.GetInvocationStatsByServiceWithResponse(ctx, serviceName, serviceVersion, &client.GetInvocationStatsByServiceParams{
 		Day: &day,
 	})
-	return int(stats.JSON200.DailyCount), err
+
+	invocationStats := stats.JSON200
+	if invocationStats != nil {
+		return int(stats.JSON200.DailyCount), err
+	}
+	return 0, err
 }
 
 func importArtifactHook(artifactFilePath string, mainArtifact bool) testcontainers.ContainerHook {