Skip to content

Commit

Permalink
Deprecate extensiontest.NewNopSettings
Browse files Browse the repository at this point in the history
  • Loading branch information
sincejune committed Feb 10, 2025
1 parent 398ceed commit 2dcfecd
Show file tree
Hide file tree
Showing 40 changed files with 77 additions and 77 deletions.
2 changes: 1 addition & 1 deletion exporter/fileexporter/encoding_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ func TestEncoding(t *testing.T) {
ef := otlpencodingextension.NewFactory()
efCfg := ef.CreateDefaultConfig().(*otlpencodingextension.Config)
efCfg.Protocol = "otlp_json"
ext, err := ef.Create(context.Background(), extensiontest.NewNopSettings(), efCfg)
ext, err := ef.Create(context.Background(), extensiontest.NewNopSettingsWithType(extensiontest.NopType), efCfg)
require.NoError(t, err)
require.NoError(t, ext.Start(context.Background(), componenttest.NewNopHost()))

Expand Down
2 changes: 1 addition & 1 deletion extension/asapauthextension/factory_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ func TestCreate(t *testing.T) {
cfg.PrivateKey = testcase.settings.PrivateKey

// validate extension creation
ext, err := createExtension(context.Background(), extensiontest.NewNopSettings(), cfg)
ext, err := createExtension(context.Background(), extensiontest.NewNopSettingsWithType(extensiontest.NopType), cfg)
if testcase.shouldError {
assert.Error(t, err)
} else {
Expand Down
2 changes: 1 addition & 1 deletion extension/awsproxy/factory_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ func TestFactory_Create(t *testing.T) {
t.Setenv("AWS_SECRET_ACCESS_KEY", "fakeSecretAccessKey")

ctx := context.Background()
cs := extensiontest.NewNopSettings()
cs := extensiontest.NewNopSettingsWithType(extensiontest.NopType)
ext, err := createExtension(ctx, cs, cfg)
assert.NoError(t, err)
assert.NotNil(t, ext)
Expand Down
2 changes: 1 addition & 1 deletion extension/basicauthextension/factory_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ func TestCreateExtension_ValidConfig(t *testing.T) {
},
}

ext, err := createExtension(context.Background(), extensiontest.NewNopSettings(), cfg)
ext, err := createExtension(context.Background(), extensiontest.NewNopSettingsWithType(extensiontest.NopType), cfg)
assert.NoError(t, err)
assert.NotNil(t, ext)
}
Expand Down
2 changes: 1 addition & 1 deletion extension/bearertokenauthextension/factory_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ func TestFactory_CreateDefaultConfig(t *testing.T) {
func TestFactory_Create(t *testing.T) {
cfg := createDefaultConfig().(*Config)
cfg.BearerToken = "somerandometoken"
ext, err := createExtension(context.Background(), extensiontest.NewNopSettings(), cfg)
ext, err := createExtension(context.Background(), extensiontest.NewNopSettingsWithType(extensiontest.NopType), cfg)
require.NoError(t, err)
require.NotNil(t, ext)
}
2 changes: 1 addition & 1 deletion extension/cgroupruntimeextension/extension_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ func TestExtension(t *testing.T) {
allCalls++
return func() { allCalls++ }, _err
}
settings := extensiontest.NewNopSettings()
settings := extensiontest.NewNopSettingsWithType(extensiontest.NopType)
cg := newCgroupRuntime(test.config, settings.Logger, setterMock, func(_ float64) (undoFunc, error) { return setterMock() })
ctx := context.Background()

Expand Down
4 changes: 2 additions & 2 deletions extension/encoding/jaegerencodingextension/extension_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ func TestExtension_Start(t *testing.T) {
name: "jaegerProtobuf",
getExtension: func() (extension.Extension, error) {
factory := NewFactory()
return factory.Create(context.Background(), extensiontest.NewNopSettings(), factory.CreateDefaultConfig())
return factory.Create(context.Background(), extensiontest.NewNopSettingsWithType(extensiontest.NopType), factory.CreateDefaultConfig())
},
},
{
Expand All @@ -32,7 +32,7 @@ func TestExtension_Start(t *testing.T) {
factory := NewFactory()
cfg := factory.CreateDefaultConfig()
cfg.(*Config).Protocol = "xyz"
return factory.Create(context.Background(), extensiontest.NewNopSettings(), cfg)
return factory.Create(context.Background(), extensiontest.NewNopSettingsWithType(extensiontest.NopType), cfg)
},
expectedErr: "unsupported protocol: \"xyz\"",
},
Expand Down
4 changes: 2 additions & 2 deletions extension/encoding/otlpencodingextension/extension_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ func TestExtension_Start(t *testing.T) {
factory := NewFactory()
cfg := factory.CreateDefaultConfig()
cfg.(*Config).Protocol = "otlp_json"
return factory.Create(context.Background(), extensiontest.NewNopSettings(), cfg)
return factory.Create(context.Background(), extensiontest.NewNopSettingsWithType(extensiontest.NopType), cfg)
},
},

Expand All @@ -40,7 +40,7 @@ func TestExtension_Start(t *testing.T) {
factory := NewFactory()
cfg := factory.CreateDefaultConfig()
cfg.(*Config).Protocol = "otlp_proto"
return factory.Create(context.Background(), extensiontest.NewNopSettings(), cfg)
return factory.Create(context.Background(), extensiontest.NewNopSettingsWithType(extensiontest.NopType), cfg)
},
},
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ func TestExtension_Start(t *testing.T) {
getExtension: func() (extension.Extension, error) {
factory := NewFactory()
cfg := factory.CreateDefaultConfig()
return factory.Create(context.Background(), extensiontest.NewNopSettings(), cfg)
return factory.Create(context.Background(), extensiontest.NewNopSettingsWithType(extensiontest.NopType), cfg)
},
},
}
Expand Down
6 changes: 3 additions & 3 deletions extension/encoding/textencodingextension/extension_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ func TestExtension_Start(t *testing.T) {
name: "text",
getExtension: func() (extension.Extension, error) {
factory := NewFactory()
return factory.Create(context.Background(), extensiontest.NewNopSettings(), factory.CreateDefaultConfig())
return factory.Create(context.Background(), extensiontest.NewNopSettingsWithType(extensiontest.NopType), factory.CreateDefaultConfig())
},
},
{
Expand All @@ -32,7 +32,7 @@ func TestExtension_Start(t *testing.T) {
factory := NewFactory()
cfg := factory.CreateDefaultConfig()
cfg.(*Config).Encoding = "gbk"
return factory.Create(context.Background(), extensiontest.NewNopSettings(), cfg)
return factory.Create(context.Background(), extensiontest.NewNopSettingsWithType(extensiontest.NopType), cfg)
},
},
{
Expand All @@ -41,7 +41,7 @@ func TestExtension_Start(t *testing.T) {
factory := NewFactory()
cfg := factory.CreateDefaultConfig()
cfg.(*Config).Encoding = "blabla"
return factory.Create(context.Background(), extensiontest.NewNopSettings(), cfg)
return factory.Create(context.Background(), extensiontest.NewNopSettingsWithType(extensiontest.NopType), cfg)
},
expectedErr: "unsupported encoding 'blabla'",
},
Expand Down
8 changes: 4 additions & 4 deletions extension/encoding/zipkinencodingextension/extension_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ func TestExtension_Start(t *testing.T) {
factory := NewFactory()
cfg := factory.CreateDefaultConfig()
cfg.(*Config).Protocol = "zipkin_json"
return factory.Create(context.Background(), extensiontest.NewNopSettings(), cfg)
return factory.Create(context.Background(), extensiontest.NewNopSettingsWithType(extensiontest.NopType), cfg)
},
},
{
Expand All @@ -34,7 +34,7 @@ func TestExtension_Start(t *testing.T) {
factory := NewFactory()
cfg := factory.CreateDefaultConfig()
cfg.(*Config).Protocol = "zipkin_proto"
return factory.Create(context.Background(), extensiontest.NewNopSettings(), cfg)
return factory.Create(context.Background(), extensiontest.NewNopSettingsWithType(extensiontest.NopType), cfg)
},
},
{
Expand All @@ -44,7 +44,7 @@ func TestExtension_Start(t *testing.T) {
cfg := factory.CreateDefaultConfig()
cfg.(*Config).Protocol = "zipkin_thrift"
cfg.(*Config).Version = "v1"
return factory.Create(context.Background(), extensiontest.NewNopSettings(), cfg)
return factory.Create(context.Background(), extensiontest.NewNopSettingsWithType(extensiontest.NopType), cfg)
},
},
{
Expand All @@ -54,7 +54,7 @@ func TestExtension_Start(t *testing.T) {
cfg := factory.CreateDefaultConfig()
cfg.(*Config).Protocol = "zipkin_thrift"
cfg.(*Config).Version = "v2"
return factory.Create(context.Background(), extensiontest.NewNopSettings(), cfg)
return factory.Create(context.Background(), extensiontest.NewNopSettingsWithType(extensiontest.NopType), cfg)
},
expectedErr: "unsupported version: \"v2\"",
},
Expand Down
2 changes: 1 addition & 1 deletion extension/headerssetterextension/factory_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ func TestCreateDefaultConfig(t *testing.T) {
func TestCreate(t *testing.T) {
cfg := createDefaultConfig().(*Config)

ext, err := createExtension(context.Background(), extensiontest.NewNopSettings(), cfg)
ext, err := createExtension(context.Background(), extensiontest.NewNopSettingsWithType(extensiontest.NopType), cfg)
assert.NoError(t, err)
assert.NotNil(t, ext)
}
Expand Down
4 changes: 2 additions & 2 deletions extension/healthcheckextension/factory_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ func TestFactory_CreateDefaultConfig(t *testing.T) {
}, cfg)

assert.NoError(t, componenttest.CheckConfigStruct(cfg))
ext, err := createExtension(context.Background(), extensiontest.NewNopSettings(), cfg)
ext, err := createExtension(context.Background(), extensiontest.NewNopSettingsWithType(extensiontest.NopType), cfg)
require.NoError(t, err)
require.NotNil(t, ext)
}
Expand All @@ -36,7 +36,7 @@ func TestFactory_Create(t *testing.T) {
cfg := createDefaultConfig().(*Config)
cfg.Endpoint = testutil.GetAvailableLocalAddress(t)

ext, err := createExtension(context.Background(), extensiontest.NewNopSettings(), cfg)
ext, err := createExtension(context.Background(), extensiontest.NewNopSettingsWithType(extensiontest.NopType), cfg)
require.NoError(t, err)
require.NotNil(t, ext)
}
2 changes: 1 addition & 1 deletion extension/healthcheckextension/integration_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ func Test_SimpleHealthCheck(t *testing.T) {
port := testutil.GetAvailablePort(t)
cfg := f.CreateDefaultConfig().(*Config)
cfg.Endpoint = fmt.Sprintf("localhost:%d", port)
e, err := f.Create(context.Background(), extensiontest.NewNopSettings(), cfg)
e, err := f.Create(context.Background(), extensiontest.NewNopSettingsWithType(extensiontest.NopType), cfg)
require.NoError(t, err)
err = e.Start(context.Background(), componenttest.NewNopHost())
require.NoError(t, err)
Expand Down
4 changes: 2 additions & 2 deletions extension/healthcheckv2extension/extension_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ func TestComponentStatus(t *testing.T) {
cfg.HTTPConfig.Endpoint = testutil.GetAvailableLocalAddress(t)
cfg.GRPCConfig.NetAddr.Endpoint = testutil.GetAvailableLocalAddress(t)
cfg.UseV2 = true
ext := newExtension(context.Background(), *cfg, extensiontest.NewNopSettings())
ext := newExtension(context.Background(), *cfg, extensiontest.NewNopSettingsWithType(extensiontest.NopType))

// Status before Start will be StatusNone
st, ok := ext.aggregator.AggregateStatus(status.ScopeAll, status.Concise)
Expand Down Expand Up @@ -109,7 +109,7 @@ func TestNotifyConfig(t *testing.T) {
cfg.HTTPConfig.Config.Enabled = true
cfg.HTTPConfig.Config.Path = "/config"

ext := newExtension(context.Background(), *cfg, extensiontest.NewNopSettings())
ext := newExtension(context.Background(), *cfg, extensiontest.NewNopSettingsWithType(extensiontest.NopType))

require.NoError(t, ext.Start(context.Background(), componenttest.NewNopHost()))
t.Cleanup(func() { require.NoError(t, ext.Shutdown(context.Background())) })
Expand Down
4 changes: 2 additions & 2 deletions extension/healthcheckv2extension/factory_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ func TestCreateDefaultConfig(t *testing.T) {
assert.NoError(t, componenttest.CheckConfigStruct(cfg))
ctx, cancel := context.WithCancel(context.Background())
defer cancel()
ext, err := createExtension(ctx, extensiontest.NewNopSettings(), cfg)
ext, err := createExtension(ctx, extensiontest.NewNopSettingsWithType(extensiontest.NopType), cfg)
require.NoError(t, err)
require.NotNil(t, ext)
}
Expand All @@ -65,7 +65,7 @@ func TestCreate(t *testing.T) {
cfg.Endpoint = testutil.GetAvailableLocalAddress(t)
ctx, cancel := context.WithCancel(context.Background())
defer cancel()
ext, err := createExtension(ctx, extensiontest.NewNopSettings(), cfg)
ext, err := createExtension(ctx, extensiontest.NewNopSettingsWithType(extensiontest.NopType), cfg)
require.NoError(t, err)
require.NotNil(t, ext)
}
2 changes: 1 addition & 1 deletion extension/httpforwarderextension/factory_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ func TestFactory(t *testing.T) {
t.Run(test.name, func(t *testing.T) {
e, err := f.Create(
context.Background(),
extensiontest.NewNopSettings(),
extensiontest.NewNopSettingsWithType(extensiontest.NopType),
test.config,
)
if test.wantErr {
Expand Down
2 changes: 1 addition & 1 deletion extension/jaegerremotesampling/factory_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ func TestCreateDefaultConfig(t *testing.T) {
func TestCreate(t *testing.T) {
cfg := createDefaultConfig().(*Config)

ext, err := createExtension(context.Background(), extensiontest.NewNopSettings(), cfg)
ext, err := createExtension(context.Background(), extensiontest.NewNopSettingsWithType(extensiontest.NopType), cfg)
assert.NoError(t, err)
assert.NotNil(t, ext)
}
Expand Down
2 changes: 1 addition & 1 deletion extension/oauth2clientauthextension/factory_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ func TestCreate(t *testing.T) {
cfg.ClientSecret = testcase.settings.ClientSecret
cfg.TokenURL = testcase.settings.TokenURL
cfg.Scopes = testcase.settings.Scopes
ext, err := createExtension(context.Background(), extensiontest.NewNopSettings(), cfg)
ext, err := createExtension(context.Background(), extensiontest.NewNopSettingsWithType(extensiontest.NopType), cfg)
if testcase.shouldError {
assert.Error(t, err)
assert.Nil(t, ext)
Expand Down
2 changes: 1 addition & 1 deletion extension/observer/cfgardenobserver/factory_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ func TestValidConfig(t *testing.T) {
func TestCreateCFGardenObserver(t *testing.T) {
cfGardenObserver, err := createExtension(
context.Background(),
extensiontest.NewNopSettings(),
extensiontest.NewNopSettingsWithType(extensiontest.NopType),
&Config{},
)
require.NoError(t, err)
Expand Down
2 changes: 1 addition & 1 deletion extension/observer/dockerobserver/factory_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ func TestValidConfig(t *testing.T) {
func TestCreate(t *testing.T) {
dockerObserver, err := createExtension(
context.Background(),
extensiontest.NewNopSettings(),
extensiontest.NewNopSettingsWithType(extensiontest.NopType),
&Config{},
)
require.NoError(t, err)
Expand Down
4 changes: 2 additions & 2 deletions extension/observer/ecsobserver/extension_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ func TestExtensionStartStop(t *testing.T) {
sdCfg := cfg.(*Config)
sdCfg.RefreshInterval = refreshInterval
sdCfg.ResultFile = output
cs := extensiontest.NewNopSettings()
cs := extensiontest.NewNopSettingsWithType(extensiontest.NopType)
ext, err := createExtensionWithFetcher(cs, sdCfg, f)
require.NoError(t, err)
return ext
Expand All @@ -59,7 +59,7 @@ func TestExtensionStartStop(t *testing.T) {
sdCfg := cfg.(*Config)
sdCfg.RefreshInterval = 100 * time.Millisecond
sdCfg.ResultFile = "testdata/ut_ext_critical_error.actual.yaml"
cs := extensiontest.NewNopSettings()
cs := extensiontest.NewNopSettingsWithType(extensiontest.NopType)
statusEventChan := make(chan *componentstatus.Event)
ext, err := createExtensionWithFetcher(cs, sdCfg, f)
require.NoError(t, err)
Expand Down
2 changes: 1 addition & 1 deletion extension/observer/ecstaskobserver/factory_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ func TestFactoryCreatedExtensionIsEndpointsLister(t *testing.T) {
cfg := etoFactory.CreateDefaultConfig()
cfg.(*Config).Endpoint = "http://localhost:1234/mock/endpoint"

eto, err := etoFactory.Create(context.Background(), extensiontest.NewNopSettings(), cfg)
eto, err := etoFactory.Create(context.Background(), extensiontest.NewNopSettingsWithType(extensiontest.NopType), cfg)
require.NoError(t, err)
require.NotNil(t, eto)
require.Implements(t, (*observer.EndpointsLister)(nil), eto)
Expand Down
2 changes: 1 addition & 1 deletion extension/observer/hostobserver/factory_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ func TestValidConfig(t *testing.T) {
func TestCreateMetrics(t *testing.T) {
hostObserver, err := createExtension(
context.Background(),
extensiontest.NewNopSettings(),
extensiontest.NewNopSettingsWithType(extensiontest.NopType),
&Config{},
)
require.NoError(t, err)
Expand Down
10 changes: 5 additions & 5 deletions extension/observer/k8sobserver/extension_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ func TestNewExtension(t *testing.T) {
config := factory.CreateDefaultConfig().(*Config)
mockServiceHost(t, config)

ext, err := newObserver(config, extensiontest.NewNopSettings())
ext, err := newObserver(config, extensiontest.NewNopSettingsWithType(extensiontest.NopType))
require.NoError(t, err)
require.NotNil(t, ext)
}
Expand All @@ -46,7 +46,7 @@ func TestExtensionObserveServices(t *testing.T) {
config.ObservePods = false // avoid causing data race when multiple test cases running in the same process using podListerWatcher
mockServiceHost(t, config)

set := extensiontest.NewNopSettings()
set := extensiontest.NewNopSettingsWithType(extensiontest.NopType)
set.ID = component.NewID(metadata.Type)
ext, err := newObserver(config, set)
require.NoError(t, err)
Expand Down Expand Up @@ -135,7 +135,7 @@ func TestExtensionObservePods(t *testing.T) {
config := factory.CreateDefaultConfig().(*Config)
mockServiceHost(t, config)

set := extensiontest.NewNopSettings()
set := extensiontest.NewNopSettingsWithType(extensiontest.NopType)
set.ID = component.NewID(metadata.Type)
ext, err := newObserver(config, set)
require.NoError(t, err)
Expand Down Expand Up @@ -219,7 +219,7 @@ func TestExtensionObserveNodes(t *testing.T) {
config.ObservePods = false // avoid causing data race when multiple test cases running in the same process using podListerWatcher
mockServiceHost(t, config)

set := extensiontest.NewNopSettings()
set := extensiontest.NewNopSettingsWithType(extensiontest.NopType)
set.ID = component.NewID(metadata.Type)
ext, err := newObserver(config, set)
require.NoError(t, err)
Expand Down Expand Up @@ -320,7 +320,7 @@ func TestExtensionObserveIngresses(t *testing.T) {
config.ObserveIngresses = true
mockServiceHost(t, config)

set := extensiontest.NewNopSettings()
set := extensiontest.NewNopSettingsWithType(extensiontest.NopType)
set.ID = component.NewID(metadata.Type)
ext, err := newObserver(config, set)
require.NoError(t, err)
Expand Down
2 changes: 1 addition & 1 deletion extension/observer/k8sobserver/factory_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ func TestFactory_Create(t *testing.T) {
factory := NewFactory()
cfg := factory.CreateDefaultConfig().(*Config)

ext, err := factory.Create(context.Background(), extensiontest.NewNopSettings(), cfg)
ext, err := factory.Create(context.Background(), extensiontest.NewNopSettingsWithType(extensiontest.NopType), cfg)
require.Error(t, err)
require.Nil(t, ext)
}
2 changes: 1 addition & 1 deletion extension/oidcauthextension/factory_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ func TestCreate(t *testing.T) {
cfg.Audience = "collector"
cfg.IssuerURL = "https://auth.example.com"

ext, err := createExtension(context.Background(), extensiontest.NewNopSettings(), cfg)
ext, err := createExtension(context.Background(), extensiontest.NewNopSettingsWithType(extensiontest.NopType), cfg)
assert.NoError(t, err)
assert.NotNil(t, ext)
}
Expand Down
4 changes: 2 additions & 2 deletions extension/opampextension/factory_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -18,15 +18,15 @@ func TestFactory_CreateDefaultConfig(t *testing.T) {
assert.Equal(t, createDefaultConfig().(*Config), cfg)

assert.NoError(t, componenttest.CheckConfigStruct(cfg))
ext, err := createExtension(context.Background(), extensiontest.NewNopSettings(), cfg)
ext, err := createExtension(context.Background(), extensiontest.NewNopSettingsWithType(extensiontest.NopType), cfg)
require.NoError(t, err)
require.NotNil(t, ext)
require.NoError(t, ext.Shutdown(context.Background()))
}

func TestFactory_Create(t *testing.T) {
cfg := NewFactory().CreateDefaultConfig()
ext, err := createExtension(context.Background(), extensiontest.NewNopSettings(), cfg)
ext, err := createExtension(context.Background(), extensiontest.NewNopSettingsWithType(extensiontest.NopType), cfg)
require.NoError(t, err)
require.NotNil(t, ext)
require.NoError(t, ext.Shutdown(context.Background()))
Expand Down
Loading

0 comments on commit 2dcfecd

Please sign in to comment.