diff --git a/modules/mesh-delivery-watchman/internal/watch/signals.go b/modules/mesh-delivery-watchman/internal/watch/signals.go index 24361bf1..115687ed 100644 --- a/modules/mesh-delivery-watchman/internal/watch/signals.go +++ b/modules/mesh-delivery-watchman/internal/watch/signals.go @@ -712,9 +712,6 @@ func BusUserWords(user string) (string, string) { return "the node-engine on " + m, m } if m, module, ok := strings.Cut(user, "."); ok { - if module == "node-tools" { - return "the tool runner on " + m, m - } return module + " on " + m, m } return "a client of the bus", "" diff --git a/modules/mesh-delivery-watchman/internal/watch/watch_test.go b/modules/mesh-delivery-watchman/internal/watch/watch_test.go index 0a8e05ea..591d1748 100644 --- a/modules/mesh-delivery-watchman/internal/watch/watch_test.go +++ b/modules/mesh-delivery-watchman/internal/watch/watch_test.go @@ -745,3 +745,13 @@ func TestAStuckKeyWhoseKeyIsGoneIsClosedAtTheStart(t *testing.T) { }) } } + +// design 47 G §7, §428: no part's code names another module: a bus user is said by its own module's name on its machine, the tool +// runner's no differently from any module's. +func TestABusUserIsSaidByItsModuleWithoutAModulesNameInTheCode(t *testing.T) { + for user, want := range map[string]string{"ace.node-tools": "node-tools on ace", "ace.web": "web on ace", "node.ace": "the node-engine on ace"} { + if got, _ := BusUserWords(user); got != want { + t.Errorf("BusUserWords(%q) = %q, want %q", user, got, want) + } + } +}