From edae2c7469e0b2821d73db2c0f7d5df83278922c Mon Sep 17 00:00:00 2001 From: Bogdan Drutu Date: Fri, 24 May 2024 00:40:05 +0300 Subject: [PATCH] [chore] test that implementation implements interface without allocation (#10214) --- otelcol/configprovider.go | 4 ++-- service/internal/graph/graph_test.go | 2 +- service/internal/graph/nodes.go | 10 +++++----- 3 files changed, 8 insertions(+), 8 deletions(-) diff --git a/otelcol/configprovider.go b/otelcol/configprovider.go index 477541fd4bd..6360b536509 100644 --- a/otelcol/configprovider.go +++ b/otelcol/configprovider.go @@ -67,8 +67,8 @@ type configProvider struct { mapResolver *confmap.Resolver } -var _ ConfigProvider = &configProvider{} -var _ ConfmapProvider = &configProvider{} +var _ ConfigProvider = (*configProvider)(nil) +var _ ConfmapProvider = (*configProvider)(nil) // ConfigProviderSettings are the settings to configure the behavior of the ConfigProvider. type ConfigProviderSettings struct { diff --git a/service/internal/graph/graph_test.go b/service/internal/graph/graph_test.go index eddd3098153..343f752b888 100644 --- a/service/internal/graph/graph_test.go +++ b/service/internal/graph/graph_test.go @@ -34,7 +34,7 @@ import ( "go.opentelemetry.io/collector/service/pipelines" ) -var _ component.Component = &testNode{} +var _ component.Component = (*testNode)(nil) type testNode struct { id component.ID diff --git a/service/internal/graph/nodes.go b/service/internal/graph/nodes.go index 13726d1c760..238bbf3b287 100644 --- a/service/internal/graph/nodes.go +++ b/service/internal/graph/nodes.go @@ -104,7 +104,7 @@ func (n *receiverNode) buildComponent(ctx context.Context, return nil } -var _ consumerNode = &processorNode{} +var _ consumerNode = (*processorNode)(nil) // Every processor instance is unique to one pipeline. // Therefore, nodeID is derived from "pipeline ID" and "component ID". @@ -152,7 +152,7 @@ func (n *processorNode) buildComponent(ctx context.Context, return nil } -var _ consumerNode = &exporterNode{} +var _ consumerNode = (*exporterNode)(nil) // An exporter instance can be shared by multiple pipelines of the same type. // Therefore, nodeID is derived from "pipeline type" and "component ID". @@ -200,7 +200,7 @@ func (n *exporterNode) buildComponent( return nil } -var _ consumerNode = &connectorNode{} +var _ consumerNode = (*connectorNode)(nil) // A connector instance connects one pipeline type to one other pipeline type. // Therefore, nodeID is derived from "exporter pipeline type", "receiver pipeline type", and "component ID". @@ -347,7 +347,7 @@ func (n *connectorNode) buildComponent( return nil } -var _ consumerNode = &capabilitiesNode{} +var _ consumerNode = (*capabilitiesNode)(nil) // Every pipeline has a "virtual" capabilities node immediately after the receiver(s). // There are two purposes for this node: @@ -374,7 +374,7 @@ func (n *capabilitiesNode) getConsumer() baseConsumer { return n } -var _ consumerNode = &fanOutNode{} +var _ consumerNode = (*fanOutNode)(nil) // Each pipeline has one fan-out node before exporters. // Therefore, nodeID is derived from "pipeline ID".