add ci conf and subscribe hook for the AsyncTopic
This commit is contained in:
1
.gitignore
vendored
Normal file
1
.gitignore
vendored
Normal file
@@ -0,0 +1 @@
|
|||||||
|
*.out
|
||||||
13
.gitlab-ci.yml
Normal file
13
.gitlab-ci.yml
Normal file
@@ -0,0 +1,13 @@
|
|||||||
|
image: golang:latest
|
||||||
|
|
||||||
|
stages:
|
||||||
|
- test
|
||||||
|
|
||||||
|
format:
|
||||||
|
stage: test
|
||||||
|
script:
|
||||||
|
- go fmt gitlab.com/naterciom/gubgub
|
||||||
|
- go vet gitlab.com/naterciom/gubgub
|
||||||
|
- go test -race gitlab.com/naterciom/gubgub
|
||||||
|
|
||||||
|
|
||||||
18
async.go
18
async.go
@@ -20,7 +20,8 @@ type AsyncTopic[T any] struct {
|
|||||||
// NewAsyncTopic creates a Topic that publishes messages asynchronously.
|
// NewAsyncTopic creates a Topic that publishes messages asynchronously.
|
||||||
func NewAsyncTopic[T any](ctx context.Context, opts ...AsyncTopicOption) *AsyncTopic[T] {
|
func NewAsyncTopic[T any](ctx context.Context, opts ...AsyncTopicOption) *AsyncTopic[T] {
|
||||||
options := AsyncTopicOptions{
|
options := AsyncTopicOptions{
|
||||||
onClose: func() {},
|
onClose: func() {},
|
||||||
|
onSubscribe: func(count int) {},
|
||||||
}
|
}
|
||||||
|
|
||||||
for _, opt := range opts {
|
for _, opt := range opts {
|
||||||
@@ -52,6 +53,7 @@ func (t *AsyncTopic[T]) run(ctx context.Context) {
|
|||||||
|
|
||||||
case newCallback := <-t.subscribeCh:
|
case newCallback := <-t.subscribeCh:
|
||||||
subscribers = append(subscribers, newCallback)
|
subscribers = append(subscribers, newCallback)
|
||||||
|
t.options.onSubscribe(len(subscribers))
|
||||||
|
|
||||||
case msg := <-t.publishCh:
|
case msg := <-t.publishCh:
|
||||||
keepers := make([]Subscriber[T], 0, len(subscribers))
|
keepers := make([]Subscriber[T], 0, len(subscribers))
|
||||||
@@ -88,7 +90,6 @@ func (t *AsyncTopic[T]) Subscribe(fn Subscriber[T]) error {
|
|||||||
|
|
||||||
if t.closed {
|
if t.closed {
|
||||||
return fmt.Errorf("async topic subscribe: %w", ErrTopicClosed)
|
return fmt.Errorf("async topic subscribe: %w", ErrTopicClosed)
|
||||||
// return nil
|
|
||||||
}
|
}
|
||||||
|
|
||||||
t.subscribeCh <- fn
|
t.subscribeCh <- fn
|
||||||
@@ -104,13 +105,20 @@ func (t *AsyncTopic[T]) close() {
|
|||||||
}
|
}
|
||||||
|
|
||||||
type AsyncTopicOptions struct {
|
type AsyncTopicOptions struct {
|
||||||
onClose func()
|
onClose func()
|
||||||
|
onSubscribe func(count int)
|
||||||
}
|
}
|
||||||
|
|
||||||
type AsyncTopicOption func(*AsyncTopicOptions)
|
type AsyncTopicOption func(*AsyncTopicOptions)
|
||||||
|
|
||||||
func WithOnClose(fn func()) AsyncTopicOption {
|
func WithOnClose(fn func()) AsyncTopicOption {
|
||||||
return func(a *AsyncTopicOptions) {
|
return func(opts *AsyncTopicOptions) {
|
||||||
a.onClose = fn
|
opts.onClose = fn
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func WithOnSubscribe(fn func(count int)) AsyncTopicOption {
|
||||||
|
return func(opts *AsyncTopicOptions) {
|
||||||
|
opts.onSubscribe = fn
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -107,6 +107,34 @@ func TestAsyncTopic_WithOnClose(t *testing.T) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func TestAsyncTopic_WithOnSubscribe(t *testing.T) {
|
||||||
|
const totalSub = 10
|
||||||
|
|
||||||
|
ctx, cancel := context.WithTimeout(context.Background(), time.Second)
|
||||||
|
defer cancel()
|
||||||
|
|
||||||
|
feedback := make(chan int, totalSub)
|
||||||
|
defer close(feedback)
|
||||||
|
|
||||||
|
topic := NewAsyncTopic[int](ctx, WithOnSubscribe(func(count int) { feedback <- count }))
|
||||||
|
|
||||||
|
for range totalSub {
|
||||||
|
topic.Subscribe(func(i int) bool { return true })
|
||||||
|
}
|
||||||
|
|
||||||
|
count := 0
|
||||||
|
for count < totalSub {
|
||||||
|
select {
|
||||||
|
case c := <-feedback:
|
||||||
|
count++
|
||||||
|
assert.Equal(t, count, c, "expected %d but got %d instead", count, c)
|
||||||
|
|
||||||
|
case <-time.After(time.Second):
|
||||||
|
t.Fatalf("expected %d feedback items by now but only got %d", totalSub, count)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
func TestAsyncTopic_SubscribeClosedTopicError(t *testing.T) {
|
func TestAsyncTopic_SubscribeClosedTopicError(t *testing.T) {
|
||||||
ctx, cancel := context.WithCancel(context.Background())
|
ctx, cancel := context.WithCancel(context.Background())
|
||||||
defer cancel()
|
defer cancel()
|
||||||
|
|||||||
Reference in New Issue
Block a user