otelloghttp: Implement Exporter.ForceFlush (#5164)

It holds no state, already implemented.
open-telemetry · Apr 8, 2024 · d66a661 · d66a661
1 parent 30487e1
commit d66a661
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 1 deletion.
diff --git a/exporters/otlp/otlplog/otlploghttp/exporter.go b/exporters/otlp/otlplog/otlploghttp/exporter.go
@@ -48,6 +48,5 @@ func (e *Exporter) Shutdown(ctx context.Context) error {
 
 // ForceFlush does nothing. The Exporter holds no state.
 func (e *Exporter) ForceFlush(ctx context.Context) error {
-	// TODO: implement.
 	return nil
 }
diff --git a/exporters/otlp/otlplog/otlploghttp/exporter_test.go b/exporters/otlp/otlplog/otlploghttp/exporter_test.go
@@ -0,0 +1,20 @@
+// Copyright The OpenTelemetry Authors
+// SPDX-License-Identifier: Apache-2.0
+
+package otlploghttp
+
+import (
+	"context"
+	"testing"
+
+	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
+)
+
+func TestExporterForceFlush(t *testing.T) {
+	ctx := context.Background()
+	e, err := New(ctx)
+	require.NoError(t, err, "New")
+
+	assert.NoError(t, e.ForceFlush(ctx), "ForceFlush")
+}