Skip to content

Commit

Permalink
serverutils: move TracerI into TestTenantInterface
Browse files Browse the repository at this point in the history
Tracer method is already in TestTenantInterface.

Release note: None
  • Loading branch information
yuzefovich committed Jul 18, 2023
1 parent bd6ded9 commit 609230c
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 3 deletions.
5 changes: 5 additions & 0 deletions pkg/server/testserver.go
Expand Up @@ -856,6 +856,11 @@ func (t *TestTenant) Tracer() *tracing.Tracer {
return t.sql.ambientCtx.Tracer
}

// TracerI is part of the serverutils.TestTenantInterface.
func (t *TestTenant) TracerI() interface{} {
return t.Tracer
}

// SettingsWatcher is part of the serverutils.TestTenantInterface.
func (t *TestTenant) SettingsWatcher() interface{} {
return t.sql.settingsWatcher
Expand Down
3 changes: 0 additions & 3 deletions pkg/testutils/serverutils/test_server_shim.go
Expand Up @@ -149,9 +149,6 @@ type TestServerInterface interface {
// insql.InternalDB as an interface{}.
InternalDB() interface{}

// TracerI returns a *tracing.Tracer as an interface{}.
TracerI() interface{}

// GossipI returns the gossip used by the TestServer.
// The real return type is *gossip.Gossip.
GossipI() interface{}
Expand Down
3 changes: 3 additions & 0 deletions pkg/testutils/serverutils/test_tenant_shim.go
Expand Up @@ -218,6 +218,9 @@ type TestTenantInterface interface {
// Tracer returns a reference to the tenant's Tracer.
Tracer() *tracing.Tracer

// TracerI is the same as Tracer but returns an interface{}.
TracerI() interface{}

// MigrationServer returns the tenant's migration server, which is used in
// upgrade testing.
MigrationServer() interface{}
Expand Down

0 comments on commit 609230c

Please sign in to comment.