From 3ae308426a9abfed733f13a5e1c0f8d57e65133a Mon Sep 17 00:00:00 2001 From: jochen Date: Sun, 1 Nov 2026 02:36:23 +0100 Subject: [PATCH] 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 --- modules/mesh-delivery-indexer/judge/rules_test.go | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/modules/mesh-delivery-indexer/judge/rules_test.go b/modules/mesh-delivery-indexer/judge/rules_test.go index 370e79a..dede722 100644 --- a/modules/mesh-delivery-indexer/judge/rules_test.go +++ b/modules/mesh-delivery-indexer/judge/rules_test.go @@ -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 /mesh/, and the definition names it by id. m := catalogue.Manifest{Module: "umami",