mirror of
https://github.com/hanzoai/livekit.git
synced 2026-10-30 18:29:34 +00:00
zeekay
e9292da479
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.
223 lines
6.1 KiB
Go
223 lines
6.1 KiB
Go
// Copyright 2023 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 streamtracker
|
|
|
|
import (
|
|
"fmt"
|
|
"sync"
|
|
"testing"
|
|
"time"
|
|
|
|
"github.com/stretchr/testify/require"
|
|
"go.uber.org/atomic"
|
|
|
|
"github.com/hanzoai/livekit/pkg/testutils"
|
|
"github.com/livekit/protocol/logger"
|
|
)
|
|
|
|
func newStreamTrackerPacket(samplesRequired uint32, cyclesRequired uint32, cycleDuration time.Duration) *StreamTracker {
|
|
stp := NewStreamTrackerPacket(StreamTrackerPacketParams{
|
|
Config: StreamTrackerPacketConfig{
|
|
SamplesRequired: samplesRequired,
|
|
CyclesRequired: cyclesRequired,
|
|
CycleDuration: cycleDuration,
|
|
},
|
|
Logger: logger.GetLogger(),
|
|
})
|
|
|
|
return NewStreamTracker(StreamTrackerParams{
|
|
StreamTrackerImpl: stp,
|
|
BitrateReportInterval: 1 * time.Second,
|
|
Logger: logger.GetLogger(),
|
|
})
|
|
}
|
|
|
|
func TestStreamTracker(t *testing.T) {
|
|
t.Run("flips to active on first observe", func(t *testing.T) {
|
|
callbackCalled := atomic.NewBool(false)
|
|
tracker := newStreamTrackerPacket(5, 60, 500*time.Millisecond)
|
|
tracker.Start()
|
|
tracker.OnStatusChanged(func(status StreamStatus) {
|
|
callbackCalled.Store(true)
|
|
})
|
|
require.Equal(t, StreamStatusStopped, tracker.Status())
|
|
|
|
// observe first packet
|
|
tracker.Observe(0, 20, 10, false, 0, nil)
|
|
|
|
testutils.WithTimeout(t, func() string {
|
|
if callbackCalled.Load() {
|
|
return ""
|
|
}
|
|
|
|
return "first packet didn't activate stream"
|
|
})
|
|
|
|
require.Equal(t, StreamStatusActive, tracker.Status())
|
|
require.True(t, callbackCalled.Load())
|
|
|
|
tracker.Stop()
|
|
})
|
|
|
|
t.Run("flips to inactive immediately", func(t *testing.T) {
|
|
tracker := newStreamTrackerPacket(5, 60, 500*time.Millisecond)
|
|
tracker.Start()
|
|
require.Equal(t, StreamStatusStopped, tracker.Status())
|
|
|
|
callbackStatusMu := sync.RWMutex{}
|
|
callbackStatusMu.Lock()
|
|
callbackStatus := StreamStatusStopped
|
|
callbackStatusMu.Unlock()
|
|
tracker.OnStatusChanged(func(status StreamStatus) {
|
|
callbackStatusMu.Lock()
|
|
callbackStatus = status
|
|
callbackStatusMu.Unlock()
|
|
})
|
|
|
|
tracker.Observe(0, 20, 10, false, 0, nil)
|
|
testutils.WithTimeout(t, func() string {
|
|
callbackStatusMu.RLock()
|
|
defer callbackStatusMu.RUnlock()
|
|
|
|
if callbackStatus == StreamStatusActive {
|
|
return ""
|
|
}
|
|
|
|
return "first packet did not activate stream"
|
|
})
|
|
require.Equal(t, StreamStatusActive, tracker.Status())
|
|
|
|
// run a single iteration
|
|
tracker.updateStatus()
|
|
|
|
testutils.WithTimeout(t, func() string {
|
|
callbackStatusMu.RLock()
|
|
defer callbackStatusMu.RUnlock()
|
|
|
|
if callbackStatus == StreamStatusStopped {
|
|
return ""
|
|
}
|
|
|
|
return "inactive cycle did not declare stream stopped"
|
|
})
|
|
require.Equal(t, StreamStatusStopped, tracker.Status())
|
|
require.Equal(t, StreamStatusStopped, callbackStatus)
|
|
|
|
tracker.Stop()
|
|
})
|
|
|
|
t.Run("flips back to active after iterations", func(t *testing.T) {
|
|
tracker := newStreamTrackerPacket(1, 2, 500*time.Millisecond)
|
|
tracker.Start()
|
|
require.Equal(t, StreamStatusStopped, tracker.Status())
|
|
|
|
tracker.Observe(0, 20, 10, false, 0, nil)
|
|
testutils.WithTimeout(t, func() string {
|
|
if tracker.Status() == StreamStatusActive {
|
|
return ""
|
|
}
|
|
|
|
return "first packet did not activate stream"
|
|
})
|
|
|
|
tracker.setStatusLocked(StreamStatusStopped)
|
|
|
|
tracker.Observe(0, 20, 10, false, 0, nil)
|
|
tracker.updateStatus()
|
|
require.Equal(t, StreamStatusStopped, tracker.Status())
|
|
|
|
tracker.Observe(0, 20, 10, false, 0, nil)
|
|
tracker.updateStatus()
|
|
require.Equal(t, StreamStatusActive, tracker.Status())
|
|
|
|
tracker.Stop()
|
|
})
|
|
|
|
t.Run("changes to inactive when paused", func(t *testing.T) {
|
|
tracker := newStreamTrackerPacket(5, 60, 500*time.Millisecond)
|
|
tracker.Start()
|
|
tracker.Observe(0, 20, 10, false, 0, nil)
|
|
testutils.WithTimeout(t, func() string {
|
|
if tracker.Status() == StreamStatusActive {
|
|
return ""
|
|
}
|
|
|
|
return "first packet did not activate stream"
|
|
})
|
|
|
|
tracker.SetPaused(true)
|
|
tracker.updateStatus()
|
|
require.Equal(t, StreamStatusStopped, tracker.Status())
|
|
|
|
tracker.Stop()
|
|
})
|
|
|
|
t.Run("flips back to active on first observe after reset", func(t *testing.T) {
|
|
callbackCalled := atomic.NewUint32(0)
|
|
tracker := newStreamTrackerPacket(5, 60, 500*time.Millisecond)
|
|
tracker.Start()
|
|
tracker.OnStatusChanged(func(status StreamStatus) {
|
|
callbackCalled.Inc()
|
|
})
|
|
require.Equal(t, StreamStatusStopped, tracker.Status())
|
|
|
|
// observe first packet
|
|
tracker.Observe(0, 20, 10, false, 0, nil)
|
|
|
|
testutils.WithTimeout(t, func() string {
|
|
if callbackCalled.Load() == 1 {
|
|
return ""
|
|
}
|
|
|
|
return fmt.Sprintf("expected onStatusChanged to be called once, actual: %d", callbackCalled.Load())
|
|
})
|
|
|
|
require.Equal(t, StreamStatusActive, tracker.Status())
|
|
require.Equal(t, uint32(1), callbackCalled.Load())
|
|
|
|
// observe a few more
|
|
tracker.Observe(0, 20, 10, false, 0, nil)
|
|
tracker.Observe(0, 20, 10, false, 0, nil)
|
|
tracker.Observe(0, 20, 10, false, 0, nil)
|
|
tracker.Observe(0, 20, 10, false, 0, nil)
|
|
tracker.updateStatus()
|
|
|
|
// should still be active
|
|
require.Equal(t, StreamStatusActive, tracker.Status())
|
|
require.Equal(t, uint32(1), callbackCalled.Load())
|
|
|
|
// Reset. The first packet after reset should flip state again
|
|
tracker.Reset()
|
|
require.Equal(t, StreamStatusStopped, tracker.Status())
|
|
require.Equal(t, uint32(2), callbackCalled.Load())
|
|
|
|
// first packet after reset
|
|
tracker.Observe(0, 20, 10, false, 0, nil)
|
|
|
|
testutils.WithTimeout(t, func() string {
|
|
if callbackCalled.Load() == 3 {
|
|
return ""
|
|
}
|
|
|
|
return fmt.Sprintf("expected onStatusChanged to be called thrice, actual %d", callbackCalled.Load())
|
|
})
|
|
|
|
require.Equal(t, StreamStatusActive, tracker.Status())
|
|
require.Equal(t, uint32(3), callbackCalled.Load())
|
|
|
|
tracker.Stop()
|
|
})
|
|
}
|