Files
livekit/pkg/utils/incrementaldispatcher_test.go
zeekay e9292da479 refactor: own the module path, and move the KV client to hanzokv/go
This fork lived at github.com/hanzoai/livekit but still declared
`module github.com/livekit/livekit-server`, so all 505 self-references pointed
at upstream. A fork that does not own its module path is the exact thing the
owned-module-paths rule exists to prevent: every import line in every consumer
carries someone else's brand, and `go get` resolves to their code, not ours.

  module + 505 self-imports  github.com/livekit/livekit-server -> github.com/hanzoai/livekit
  KV client                  github.com/redis/go-redis/v9      -> github.com/hanzokv/go/v9
  message bus                github.com/livekit/psrpc          -> github.com/hanzoai/psrpc

psrpc had to be forked (hanzoai/psrpc, Apache-2.0) because its bus is typed
against the KV client — the same shape redsync had for hanzoai/git.

DELIBERATELY NOT FORKED: github.com/livekit/{protocol,mediatransportutil,
mageutil} keep their addresses. Only `protocol` still reaches go-redis, through
a single file (redis/redis.go) out of 305. Forking an 11.5MB core wire-protocol
library that this server must track in lockstep — and that upstream pushed
yesterday — to change one file trades a tidy dependency graph for permanent
divergence on the fastest-moving dependency here. So one indirect
github.com/redis/go-redis entry remains, via livekit/protocol/redis, and that
is the correct place for it to live.
2026-07-26 08:14:23 -07:00

98 lines
2.1 KiB
Go

/*
* Copyright 2024 LiveKit, Inc
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
package utils_test
import (
"fmt"
"sync"
"testing"
"time"
"go.uber.org/atomic"
"github.com/stretchr/testify/require"
"github.com/hanzoai/livekit/pkg/testutils"
"github.com/hanzoai/livekit/pkg/utils"
)
func TestForEach(t *testing.T) {
producer := utils.NewIncrementalDispatcher[int]()
go func() {
defer producer.Done()
producer.Add(1)
producer.Add(2)
producer.Add(3)
}()
sum := 0
producer.ForEach(func(item int) {
sum += item
})
require.Equal(t, 6, sum)
}
func TestConcurrentConsumption(t *testing.T) {
producer := utils.NewIncrementalDispatcher[int]()
numConsumers := 100
sums := make([]atomic.Int32, numConsumers)
var wg sync.WaitGroup
for i := range numConsumers {
wg.Add(1)
i := i
go func() {
defer wg.Done()
producer.ForEach(func(item int) {
sums[i].Add(int32(item))
})
}()
}
// Add items
expectedSum := 0
for i := range 20 {
expectedSum += i
producer.Add(i)
}
for i := range numConsumers {
testutils.WithTimeout(t, func() string {
if sums[i].Load() != int32(expectedSum) {
return fmt.Sprintf("consumer %d did not consume all the items. expected %d, actual: %d",
i, expectedSum, sums[i].Load())
}
return ""
}, time.Second)
}
// keep adding and ensure it's consumed
for i := 20; i < 30; i++ {
expectedSum += i
producer.Add(i)
}
// wait for all consumers to finish
producer.Done()
wg.Wait()
for i := range numConsumers {
require.Equal(t, int32(expectedSum), sums[i].Load(), "consumer %d did not match", i)
}
}