x/ref: clean up use of test.InitWithParams as per v.io/i/599
Change-Id: I7b5d2a781abcb2efec297dfcd0a49de1794ebc40
diff --git a/lib/security/audit/principal_test.go b/lib/security/audit/principal_test.go
index cec8999..751fe23 100644
--- a/lib/security/audit/principal_test.go
+++ b/lib/security/audit/principal_test.go
@@ -25,7 +25,7 @@
)
func TestAuditingPrincipal(t *testing.T) {
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
+ ctx, shutdown := test.V23InitAnon()
defer shutdown()
var (
thirdPartyCaveat, discharge = newThirdPartyCaveatAndDischarge(t)
@@ -129,7 +129,7 @@
}
func TestUnauditedMethodsOnPrincipal(t *testing.T) {
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
+ ctx, shutdown := test.V23InitAnon()
defer shutdown()
var (
auditor = new(mockAuditor)
diff --git a/runtime/internal/naming/namespace/cache_test.go b/runtime/internal/naming/namespace/cache_test.go
index 17fc42d..b3606f8 100644
--- a/runtime/internal/naming/namespace/cache_test.go
+++ b/runtime/internal/naming/namespace/cache_test.go
@@ -27,8 +27,8 @@
// TestCache tests the cache directly rather than via the namespace methods.
func TestCache(t *testing.T) {
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
- defer shutdown()
+ ctx, cancel := test.TestContext()
+ defer cancel()
preload := []struct {
name string
suffix string
@@ -69,8 +69,8 @@
}
func TestCacheLimit(t *testing.T) {
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
- defer shutdown()
+ ctx, cancel := test.TestContext()
+ defer cancel()
c := newTTLCache().(*ttlCache)
e := &naming.MountEntry{Servers: []naming.MountedServer{naming.MountedServer{Server: "the rain in spain", Deadline: future(3000)}}}
for i := 0; i < maxCacheEntries; i++ {
@@ -87,8 +87,8 @@
}
func TestCacheTTL(t *testing.T) {
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
- defer shutdown()
+ ctx, cancel := test.TestContext()
+ defer cancel()
before := vdltime.Deadline{time.Now()}
c := newTTLCache().(*ttlCache)
// Fill cache.
@@ -113,8 +113,8 @@
}
func TestFlushCacheEntry(t *testing.T) {
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
- defer shutdown()
+ ctx, cancel := test.TestContext()
+ defer cancel()
preload := []struct {
name string
server string
@@ -166,8 +166,8 @@
}
func TestCacheDisableEnable(t *testing.T) {
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
- defer shutdown()
+ ctx, cancel := test.TestContext()
+ defer cancel()
ns, _ := New()
// Default should be working resolution cache.
diff --git a/runtime/internal/rpc/stream/vif/set_test.go b/runtime/internal/rpc/stream/vif/set_test.go
index 66925dc..5ecf5d0 100644
--- a/runtime/internal/rpc/stream/vif/set_test.go
+++ b/runtime/internal/rpc/stream/vif/set_test.go
@@ -105,7 +105,7 @@
}
func TestSetBasic(t *testing.T) {
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
+ ctx, shutdown := test.V23InitAnon()
defer shutdown()
sockdir, err := ioutil.TempDir("", "TestSetBasic")
if err != nil {
@@ -178,7 +178,7 @@
}
func TestSetWithPipes(t *testing.T) {
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
+ ctx, shutdown := test.V23InitAnon()
defer shutdown()
c1, s1 := net.Pipe()
c2, s2 := net.Pipe()
@@ -227,7 +227,7 @@
}
func TestSetWithUnixSocket(t *testing.T) {
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
+ ctx, shutdown := test.V23InitAnon()
defer shutdown()
dir, err := ioutil.TempDir("", "TestSetWithUnixSocket")
if err != nil {
@@ -290,7 +290,7 @@
}
func TestSetInsertDelete(t *testing.T) {
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
+ ctx, shutdown := test.V23InitAnon()
defer shutdown()
c1, s1 := net.Pipe()
vf1, _, err := newVIF(ctx, c1, s1)
@@ -313,7 +313,7 @@
}
func TestBlockingFind(t *testing.T) {
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
+ ctx, shutdown := test.V23InitAnon()
defer shutdown()
network, address := "tcp", "127.0.0.1:1234"
set := vif.NewSet()
diff --git a/runtime/internal/rpc/stream/vif/vcmap_test.go b/runtime/internal/rpc/stream/vif/vcmap_test.go
index bc29468..350dd40 100644
--- a/runtime/internal/rpc/stream/vif/vcmap_test.go
+++ b/runtime/internal/rpc/stream/vif/vcmap_test.go
@@ -13,8 +13,8 @@
)
func TestVCMap(t *testing.T) {
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
- defer shutdown()
+ ctx, cancel := test.TestContext()
+ defer cancel()
m := newVCMap()
vc12 := vc.InternalNew(ctx, vc.Params{VCI: 12})
@@ -46,8 +46,8 @@
}
func TestVCMapFreeze(t *testing.T) {
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
- defer shutdown()
+ ctx, cancel := test.TestContext()
+ defer cancel()
m := newVCMap()
vc1 := vc.InternalNew(ctx, vc.Params{VCI: 1})
vc2 := vc.InternalNew(ctx, vc.Params{VCI: 2})
@@ -68,8 +68,8 @@
}
func TestVCMapDelete(t *testing.T) {
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
- defer shutdown()
+ ctx, cancel := test.TestContext()
+ defer cancel()
m := newVCMap()
vc1 := vc.InternalNew(ctx, vc.Params{VCI: 1})
diff --git a/runtime/internal/rpc/stream/vif/vif_test.go b/runtime/internal/rpc/stream/vif/vif_test.go
index 2c5c2f7..908a711 100644
--- a/runtime/internal/rpc/stream/vif/vif_test.go
+++ b/runtime/internal/rpc/stream/vif/vif_test.go
@@ -37,7 +37,7 @@
//go:generate v23 test generate
func TestSingleFlowCreatedAtClient(t *testing.T) {
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
+ ctx, shutdown := test.V23InitAnon()
defer shutdown()
pclient := testutil.NewPrincipal("client")
pserver := testutil.NewPrincipal("server")
@@ -65,7 +65,7 @@
}
func TestSingleFlowCreatedAtServer(t *testing.T) {
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
+ ctx, shutdown := test.V23InitAnon()
defer shutdown()
pclient := testutil.NewPrincipal("client")
pserver := testutil.NewPrincipal("server")
@@ -96,7 +96,7 @@
func testMultipleVCsAndMultipleFlows(t *testing.T, gomaxprocs int) {
testutil.InitRandGenerator(t.Logf)
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
+ ctx, shutdown := test.V23InitAnon()
defer shutdown()
// This test dials multiple VCs from the client to the server.
// On each VC, it creates multiple flows, writes to them and verifies
@@ -255,7 +255,7 @@
}
func TestClose(t *testing.T) {
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
+ ctx, shutdown := test.V23InitAnon()
defer shutdown()
pclient := testutil.NewPrincipal("client")
pserver := testutil.NewPrincipal("server")
@@ -294,7 +294,7 @@
}
func TestOnClose(t *testing.T) {
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
+ ctx, shutdown := test.V23InitAnon()
defer shutdown()
pclient := testutil.NewPrincipal("client")
pserver := testutil.NewPrincipal("server")
@@ -335,7 +335,7 @@
const (
waitTime = 5 * time.Millisecond
)
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
+ ctx, shutdown := test.V23InitAnon()
defer shutdown()
pclient := testutil.NewPrincipal("client")
pserver := testutil.NewPrincipal("server")
@@ -424,7 +424,7 @@
// connection of the other side to be closed especially in race testing.
waitTime = 150 * time.Millisecond
)
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
+ ctx, shutdown := test.V23InitAnon()
defer shutdown()
pclient := testutil.NewPrincipal("client")
pserver := testutil.NewPrincipal("server")
@@ -485,7 +485,7 @@
idleTime = 10 * time.Millisecond
waitTime = idleTime * 2
)
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
+ ctx, shutdown := test.V23InitAnon()
defer shutdown()
pclient := testutil.NewPrincipal("client")
pserver := testutil.NewPrincipal("server")
@@ -603,7 +603,7 @@
func TestIdleTimeoutServer(t *testing.T) { testIdleTimeout(t, true) }
func TestShutdownVCs(t *testing.T) {
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
+ ctx, shutdown := test.V23InitAnon()
defer shutdown()
pclient := testutil.NewPrincipal("client")
pserver := testutil.NewPrincipal("server")
@@ -670,7 +670,7 @@
}
func (tc *versionTestCase) Run(t *testing.T) {
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
+ ctx, shutdown := test.V23InitAnon()
defer shutdown()
pclient := testutil.NewPrincipal("client")
pserver := testutil.NewPrincipal("server")
@@ -730,7 +730,7 @@
}
func TestNetworkFailure(t *testing.T) {
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
+ ctx, shutdown := test.V23InitAnon()
defer shutdown()
pclient := testutil.NewPrincipal("client")
pserver := testutil.NewPrincipal("server")
@@ -764,7 +764,7 @@
}
func TestPreAuthentication(t *testing.T) {
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
+ ctx, shutdown := test.V23InitAnon()
defer shutdown()
pclient := testutil.NewPrincipal("client")
pserver := testutil.NewPrincipal("server")
diff --git a/runtime/internal/rpc/testutil_test.go b/runtime/internal/rpc/testutil_test.go
index 0dca147..105638d 100644
--- a/runtime/internal/rpc/testutil_test.go
+++ b/runtime/internal/rpc/testutil_test.go
@@ -79,7 +79,7 @@
}
func initForTest() (*context.T, v23.Shutdown) {
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
+ ctx, shutdown := test.V23InitAnon()
ctx, err := ivtrace.Init(ctx, flags.VtraceFlags{})
if err != nil {
panic(err)
diff --git a/services/identity/internal/auditor/blessing_auditor_test.go b/services/identity/internal/auditor/blessing_auditor_test.go
index 84919c7..5ce9400 100644
--- a/services/identity/internal/auditor/blessing_auditor_test.go
+++ b/services/identity/internal/auditor/blessing_auditor_test.go
@@ -16,8 +16,8 @@
)
func TestBlessingAuditor(t *testing.T) {
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
- defer shutdown()
+ ctx, cancel := test.TestContext()
+ defer cancel()
auditor, reader := NewMockBlessingAuditor()
p, err := vsecurity.NewPrincipal()
diff --git a/services/identity/internal/auditor/sql_database_test.go b/services/identity/internal/auditor/sql_database_test.go
index 281e8f4..e85b744 100644
--- a/services/identity/internal/auditor/sql_database_test.go
+++ b/services/identity/internal/auditor/sql_database_test.go
@@ -16,8 +16,8 @@
)
func TestSQLDatabaseQuery(t *testing.T) {
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
- defer shutdown()
+ ctx, cancel := test.TestContext()
+ defer cancel()
db, err := sqlmock.New()
if err != nil {
t.Fatalf("failed to create new mock database stub: %v", err)
diff --git a/services/mounttable/mounttablelib/mounttable_test.go b/services/mounttable/mounttablelib/mounttable_test.go
index 9d9df27..9afb038 100644
--- a/services/mounttable/mounttablelib/mounttable_test.go
+++ b/services/mounttable/mounttablelib/mounttable_test.go
@@ -648,7 +648,7 @@
}
func TestBadAccessLists(t *testing.T) {
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
+ ctx, shutdown := test.TestContext()
defer shutdown()
_, err := mounttablelib.NewMountTableDispatcher(ctx, "testdata/invalid.perms", "", "mounttable")
if err == nil {
diff --git a/services/wspr/internal/lib/signature_manager_test.go b/services/wspr/internal/lib/signature_manager_test.go
index 80fb562..689df80 100644
--- a/services/wspr/internal/lib/signature_manager_test.go
+++ b/services/wspr/internal/lib/signature_manager_test.go
@@ -21,8 +21,8 @@
name = "/vanadium/name"
)
-func initContext(t *testing.T) (*context.T, clientWithTimesCalled, v23.Shutdown) {
- ctx, shutdown := test.V23InitWithParams(test.InitParams{})
+func initRuntime(t *testing.T) (*context.T, clientWithTimesCalled, v23.Shutdown) {
+ ctx, shutdown := test.V23InitAnon()
initialSig := []signature.Interface{
{
Methods: []signature.Method{
@@ -43,7 +43,7 @@
}
func TestFetching(t *testing.T) {
- ctx, _, shutdown := initContext(t)
+ ctx, _, shutdown := initRuntime(t)
defer shutdown()
sm := NewSignatureManager()
@@ -69,7 +69,7 @@
}
func TestThatCachedAfterFetching(t *testing.T) {
- ctx, _, shutdown := initContext(t)
+ ctx, _, shutdown := initRuntime(t)
defer shutdown()
sm := NewSignatureManager().(*signatureManager)
@@ -85,7 +85,7 @@
}
func TestThatCacheIsUsed(t *testing.T) {
- ctx, client, shutdown := initContext(t)
+ ctx, client, shutdown := initRuntime(t)
defer shutdown()
// call twice
@@ -101,7 +101,7 @@
}
func TestThatLastAccessedGetUpdated(t *testing.T) {
- ctx, _, shutdown := initContext(t)
+ ctx, _, shutdown := initRuntime(t)
defer shutdown()
sm := NewSignatureManager().(*signatureManager)
@@ -121,7 +121,7 @@
}
func TestThatTTLExpires(t *testing.T) {
- ctx, client, shutdown := initContext(t)
+ ctx, client, shutdown := initRuntime(t)
defer shutdown()
sm := NewSignatureManager().(*signatureManager)
@@ -140,7 +140,7 @@
}
func TestConcurrency(t *testing.T) {
- ctx, client, shutdown := initContext(t)
+ ctx, client, shutdown := initRuntime(t)
defer shutdown()
sm := NewSignatureManager().(*signatureManager)
diff --git a/test/init.go b/test/init.go
index b8b6cc2..c627309 100644
--- a/test/init.go
+++ b/test/init.go
@@ -72,22 +72,22 @@
// using the modules package.
func V23Init() (*context.T, v23.Shutdown) {
moduleProcess := os.Getenv("V23_SHELL_HELPER_PROCESS_ENTRY_POINT") != ""
- return V23InitWithParams(InitParams{
+ return initWithParams(initParams{
CreatePrincipal: !moduleProcess,
CreateMounttable: !moduleProcess,
})
}
-// Params contains parameters for tests that need to control what happens during
+// initParams contains parameters for tests that need to control what happens during
// init carefully.
-type InitParams struct {
+type initParams struct {
CreateMounttable bool // CreateMounttable creates a new mounttable.
CreatePrincipal bool // CreatePrincipal creates a new principal with self-signed blessing.
}
-// V23InitWithParams initializes the runtime and returns a new context and shutdown function.
+// initWithParams initializes the runtime and returns a new context and shutdown function.
// Specific aspects of initialization can be controlled via the params struct.
-func V23InitWithParams(params InitParams) (*context.T, v23.Shutdown) {
+func initWithParams(params initParams) (*context.T, v23.Shutdown) {
ctx, shutdown := v23.Init()
if params.CreatePrincipal {
var err error
@@ -110,3 +110,19 @@
}
return ctx, shutdown
}
+
+// TestContext returns a *contect.T suitable for use in tests with logging
+// configured to use loggler.Global(), but nothing else. In particular it does
+// not call v23.Init and hence any of the v23 functions that
+func TestContext() (*context.T, context.CancelFunc) {
+ ctx, cancel := context.RootContext()
+ return context.WithLogger(ctx, logger.Global()), cancel
+}
+
+// V23InitEmpty initializes a runtime but with no principal.
+func V23InitAnon() (*context.T, v23.Shutdown) {
+ return initWithParams(initParams{
+ CreatePrincipal: false,
+ CreateMounttable: false,
+ })
+}