Round 2 T2, T5: mesh-wireguard and route-proxy end G with their merge-base test counts; roles in the test

The two tests G added fold into existing ones as subtests (design 47 G §626); route-proxy's provider test names
a role, not a node (T5).

Builds: design 47 G §626, A-306; review round 2 findings T2, T5
Claude-Session: https://claude.ai/code/session_01NcbgfGKaxw984XyAPfLm4h

modules/mesh-wireguard/go/overlay/seat_test.go

@@ -73,11 +73,9 @@ func TestEveryProvisionTheModuleOffersSaysItsCredentialForm(t *testing.T) {
 			t.Errorf("%s offers credential %+v: the form is none", o.Name, o.Credential)
 		}
 	}
-}
-
 	// Design 47 G §411, §339, §341: no artifact of this repository's manifests keeps a `series` (one version per module), and
 	// the generated TypeScript contract is `index.js` with its `index.d.ts`, with no source for a build of its own.
-func TestNoArtifactKeepsASeriesAndTheTypeScriptContractIsJavaScript(t *testing.T) {
+	t.Run("NoArtifactKeepsASeriesAndTheTypeScriptContractIsJavaScript", func(t *testing.T) {
 		for _, path := range []string{"../../module.json", "../../../mesh-connectivity-contracts/module.json"} {
 			file, err := os.ReadFile(path)
 			if err != nil {
@@ -103,4 +101,5 @@ func TestNoArtifactKeepsASeriesAndTheTypeScriptContractIsJavaScript(t *testing.T
 				t.Errorf("%s: present %v, want %v (§341)", file, err == nil, want)
 			}
 		}
+	})
 }

modules/route-proxy/cmd/route-provider/provider_test.go

@@ -57,8 +57,8 @@ func TestAConsumerOfARouteIsToldItsNamesFromItsPlanEntry(t *testing.T) {
 		"grafana": {"name": "grafana.example.org"},
 	} {
 		plan, err := meshprovisionerclient.EncodePlan(meshprovisionerclient.Plan{Node: "g14", Module: module, Plan: catalogue.Plan{
-			Provisions: []catalogue.PlanProvision{{Provision: ProvisionName, Provider: "novox", ProviderModule: ModuleName,
+			Provisions: []catalogue.PlanProvision{{Provision: ProvisionName, Provider: "anchor", ProviderModule: ModuleName,
-				At: "10.7.0.1", From: "novox", As: "mesh_g14_" + module, Names: names}}}})
+				At: "10.7.0.1", From: "anchor", As: "mesh_g14_" + module, Names: names}}}})
 		if err != nil {
 			t.Fatal(err)
 		}
@@ -67,7 +67,7 @@ func TestAConsumerOfARouteIsToldItsNamesFromItsPlanEntry(t *testing.T) {
 			t.Fatal(err)
 		}
 		asked, err := json.Marshal(meshprovisionerclient.Provision{V: meshprovisionerclient.Version, Provision: ProvisionName,
-			Consumer: meshprovisionerclient.End{Node: "g14", Module: module}, Provider: meshprovisionerclient.End{Node: "novox", Module: ModuleName},
+			Consumer: meshprovisionerclient.End{Node: "g14", Module: module}, Provider: meshprovisionerclient.End{Node: "anchor", Module: ModuleName},
 			State: meshprovisionerclient.Asked})
 		if err != nil {
 			t.Fatal(err)
@@ -76,7 +76,7 @@ func TestAConsumerOfARouteIsToldItsNamesFromItsPlanEntry(t *testing.T) {
 			t.Fatal(err)
 		}
 	}
-	p := newProvider("novox", theProvisioner, bus, t.Logf)
+	p := newProvider("anchor", theProvisioner, bus, t.Logf)
 	out, err := p.Pass(context.Background())
 	if err != nil {
 		t.Fatal(err)

modules/route-proxy/cmd/route-proxy/names_test.go

@@ -30,10 +30,8 @@ func TestTheReadIsTheManifests(t *testing.T) {
 	if !slices.Contains(m.Invokes, "seat:mesh-module-assigner.assignments") {
 		t.Errorf("the proxy asks who holds a seat, its manifest invokes %v", m.Invokes)
 	}
-}
-
 	// A read in the seat form is the bus's `<holder>.<state>` of the holder on record, whoever it is.
-func TestHolderReadResolvesTheSeat(t *testing.T) {
+	t.Run("HolderReadResolvesTheSeat", func(t *testing.T) {
 		call := func(holder string) func(string, any) (json.RawMessage, error) {
 			return func(string, any) (json.RawMessage, error) {
 				return json.RawMessage(`{"seats":[{"seat":"mesh-provisioner","holders":[{"module":"` + holder + `"}]}]}`), nil
@@ -50,4 +48,5 @@ func TestHolderReadResolvesTheSeat(t *testing.T) {
 		if _, err := holderRead(call("x"), "mesh-provisioner.plans"); err == nil {
 			t.Errorf("a read naming a module was taken as a seat's")
 		}
+	})
 }