Merge four of the judge's moved tests into one, so the move adds no test to the budget

Issue 1585: three tests are split between mesh-sdk (their composition half) and the judge (their judging half).

Claude-Session: https://claude.ai/code/session_01NcbgfJ9Gum47zkFKWRaovL
This commit is contained in:
2026-11-01 02:36:23 +01:00
parent 8a98271680
commit 3ae308426a
@@ -600,6 +600,9 @@ func TestADeliveringSeatIsOnlyHeldByAModuleThatProvides(t *testing.T) {
}
func TestADirectoryBeneathAPlacedOneIsPlacedWithIt(t *testing.T) {
t.Run("anEnvironmentItemIsTypedAndNamesWhatItNeeds", anEnvironmentItemIsTypedAndNamesWhatItNeeds)
t.Run("theMeshsDirectoryForAModuleIsAPlace", theMeshsDirectoryForAModuleIsAPlace)
t.Run("anIdentityServedAsADNSLabelIsBoundedToOne", anIdentityServedAsADNSLabelIsBoundedToOne)
// An adopted layout keeps a subdirectory the predecessor made under the mesh's directory
// (issue 174: a forge's runtime state, a manager's output). Stated as beneath the placed one,
// it moves with it — a node's root moves both, and the definition names no host path.
@@ -1830,7 +1833,7 @@ func TestAnEndpointNameIsHeldToItsShape(t *testing.T) {
}
}
func TestAnEnvironmentItemIsTypedAndNamesWhatItNeeds(t *testing.T) {
func anEnvironmentItemIsTypedAndNamesWhatItNeeds(t *testing.T) {
item := func(attrs map[string]string) []string {
return environmentItemProblems("x", catalogue.Item{Collection: "environment", Attributes: attrs})
}
@@ -1859,7 +1862,7 @@ func TestAnEnvironmentItemIsTypedAndNamesWhatItNeeds(t *testing.T) {
}
// An identity served as a DNS label is never truncated, so the offer's bound must keep it in one.
func TestAnIdentityServedAsADNSLabelIsBoundedToOne(t *testing.T) {
func anIdentityServedAsADNSLabelIsBoundedToOne(t *testing.T) {
serves := map[string]map[string]any{"bucket": {"name": "${consumer:as:dns}"}}
wide := catalogue.Manifest{Module: "wide", Serves: serves, Provides: []catalogue.Offer{{Name: "bucket", Scope: catalogue.ScopeMesh,
Identity: &catalogue.OfferIdentity{Max: 255, In: "a client id"}}}}
@@ -4613,7 +4616,7 @@ func TestTheMachinesReachIsAProvisionsOnlyReachAndIsNodeScoped(t *testing.T) {
}
}
func TestTheMeshsDirectoryForAModuleIsAPlace(t *testing.T) {
func theMeshsDirectoryForAModuleIsAPlace(t *testing.T) {
// novox/hq issue 174. What the mesh writes for a module — its bus credential, its bindings —
// is the mesh's plumbing under <root>/mesh/<module>, and the definition names it by id.
m := catalogue.Manifest{Module: "umami",