fix(RoutingISM): SetDomain (#153)

Author: mbreithecker

CHANGELOG.md

@@ -14,6 +14,12 @@
 
 An '!' indicates a state machine breaking change.
 
+## Unreleased
+
+### Bug Fixes
+
+- [#153](https://github.com/bcp-innovations/hyperlane-cosmos/pull/153) Fix SetDomain for RoutingISM.
+
 ## [v1.0.1](https://github.com/bcp-innovations/hyperlane-cosmos/releases/tag/v1.0.1) - 2025-06-05
 
 ### Improvements

x/core/01_interchain_security/keeper/msg_server_test.go

@@ -929,7 +929,7 @@ var _ = Describe("msg_server.go", Ordered, func() {
 		Expect(ism.Id.String()).To(Equal(routingIsm.Id.String()))
 		Expect(ism.Routes).To(HaveLen(1))
 		Expect(ism.Routes[0].Domain).To(Equal(uint32(1337)))
-		Expect(ism.Routes[0].Ism.String()).To(Equal(noopIsmId.String()))
+		Expect(ism.Routes[0].Ism.String()).To(Equal(routingIsm.Id.String()))
 	})
 
 	It("RemoveRoutingIsmDomain (invalid) on non routing ism", func() {

x/core/01_interchain_security/types/routing_ism.go

@@ -50,7 +50,8 @@ func (m *RoutingISM) RemoveDomain(domainId uint32) bool {
 
 // SetDomain adds/sets a domain to the routing ISM.
 func (m *RoutingISM) SetDomain(newRoute Route) {
-	for _, route := range m.Routes {
+	for i := range m.Routes {
+		route := &m.Routes[i]
 		if newRoute.Domain == route.Domain {
 			route.Ism = newRoute.Ism
 			return