Asim Aslam
c7657f73f4
goreleaser / goreleaser (push) Has been cancelled
* test(harness): read agent plan from the scoped store
The store-scoping change moved an agent's plan from the default table
key agent/{name}/plan to its own table (database "agent", table {name},
key "plan"). The plan-delegate harness tests still read the old key and
failed with 'not found'; read through store.Scope(mem, "agent", name)
like the agent does.
* docs: orient agents-first across README, landing, and docs overview
Lead with agents (then services and flows), surface MCP + A2A as the
interop story, and frame agents as services. Landing hero and feature
grid reordered agents-first with an A2A gateway card.
* v6: module path go-micro.dev/v6, TLS secure by default, NewService
Cut v6. Three breaking changes, bundled so the major bump is paid once:
- Module path go-micro.dev/v5 -> go-micro.dev/v6 across all imports + go.mod.
- TLS verification on by default (was off). MICRO_TLS_SECURE removed;
MICRO_TLS_INSECURE=true opts out for self-signed/dev.
- micro.NewService(name, opts...) is the canonical service constructor,
symmetric with NewAgent/NewFlow; micro.New kept as a deprecated alias;
the old name-less NewService(opts...) removed. Generators emit NewService.
Also ports the JWT auth token provider in-module (go-micro.dev/v6/auth/jwt/token
on golang-jwt/jwt/v5), dropping the v5-pinned github.com/micro/plugins/v5/auth/jwt
and the deprecated dgrijalva/jwt-go.
Docs/README/landing updated to v6 and @latest; v5->v6 migration guide added;
CHANGELOG cut as [6.0.0]. Blog posts left at their historical versions.
---------
Co-authored-by: Claude <noreply@anthropic.com>
97 lines
2.5 KiB
Go
97 lines
2.5 KiB
Go
package nats
|
|
|
|
import (
|
|
"fmt"
|
|
"testing"
|
|
|
|
natsp "github.com/nats-io/nats.go"
|
|
"go-micro.dev/v6/broker"
|
|
)
|
|
|
|
var addrTestCases = []struct {
|
|
name string
|
|
description string
|
|
addrs map[string]string // expected address : set address
|
|
}{
|
|
{
|
|
"brokerOpts",
|
|
"set broker addresses through a broker.Option in constructor",
|
|
map[string]string{
|
|
"nats://192.168.10.1:5222": "192.168.10.1:5222",
|
|
"nats://10.20.10.0:4222": "10.20.10.0:4222"},
|
|
},
|
|
{
|
|
"brokerInit",
|
|
"set broker addresses through a broker.Option in broker.Init()",
|
|
map[string]string{
|
|
"nats://192.168.10.1:5222": "192.168.10.1:5222",
|
|
"nats://10.20.10.0:4222": "10.20.10.0:4222"},
|
|
},
|
|
{
|
|
"natsOpts",
|
|
"set broker addresses through the nats.Option in constructor",
|
|
map[string]string{
|
|
"nats://192.168.10.1:5222": "192.168.10.1:5222",
|
|
"nats://10.20.10.0:4222": "10.20.10.0:4222"},
|
|
},
|
|
{
|
|
"default",
|
|
"check if default Address is set correctly",
|
|
map[string]string{
|
|
"nats://127.0.0.1:4222": "",
|
|
},
|
|
},
|
|
}
|
|
|
|
// TestInitAddrs tests issue #100. Ensures that if the addrs is set by an option in init it will be used.
|
|
func TestInitAddrs(t *testing.T) {
|
|
for _, tc := range addrTestCases {
|
|
t.Run(fmt.Sprintf("%s: %s", tc.name, tc.description), func(t *testing.T) {
|
|
var br broker.Broker
|
|
var addrs []string
|
|
|
|
for _, addr := range tc.addrs {
|
|
addrs = append(addrs, addr)
|
|
}
|
|
|
|
switch tc.name {
|
|
case "brokerOpts":
|
|
// we know that there are just two addrs in the dict
|
|
br = NewNatsBroker(broker.Addrs(addrs[0], addrs[1]))
|
|
br.Init()
|
|
case "brokerInit":
|
|
br = NewNatsBroker()
|
|
// we know that there are just two addrs in the dict
|
|
br.Init(broker.Addrs(addrs[0], addrs[1]))
|
|
case "natsOpts":
|
|
nopts := natsp.GetDefaultOptions()
|
|
nopts.Servers = addrs
|
|
br = NewNatsBroker(Options(nopts))
|
|
br.Init()
|
|
case "default":
|
|
br = NewNatsBroker()
|
|
br.Init()
|
|
}
|
|
|
|
natsBroker, ok := br.(*natsBroker)
|
|
if !ok {
|
|
t.Fatal("Expected broker to be of types *natsBroker")
|
|
}
|
|
// check if the same amount of addrs we set has actually been set, default
|
|
// have only 1 address nats://127.0.0.1:4222 (current nats code) or
|
|
// nats://localhost:4222 (older code version)
|
|
if len(natsBroker.addrs) != len(tc.addrs) && tc.name != "default" {
|
|
t.Errorf("Expected Addr count = %d, Actual Addr count = %d",
|
|
len(natsBroker.addrs), len(tc.addrs))
|
|
}
|
|
|
|
for _, addr := range natsBroker.addrs {
|
|
_, ok := tc.addrs[addr]
|
|
if !ok {
|
|
t.Errorf("Expected '%s' has not been set", addr)
|
|
}
|
|
}
|
|
})
|
|
}
|
|
}
|