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.
82 lines
2.3 KiB
Go
82 lines
2.3 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 sfu
|
|
|
|
import (
|
|
"testing"
|
|
"time"
|
|
|
|
"github.com/stretchr/testify/require"
|
|
|
|
pd "github.com/hanzoai/livekit/pkg/sfu/rtpextension/playoutdelay"
|
|
"github.com/hanzoai/livekit/pkg/sfu/rtpstats"
|
|
"github.com/livekit/protocol/logger"
|
|
)
|
|
|
|
func TestPlayoutDelay(t *testing.T) {
|
|
stats := rtpstats.NewRTPStatsSender(rtpstats.RTPStatsParams{}, 128)
|
|
stats.SetClockRate(90000)
|
|
|
|
c, err := NewPlayoutDelayController(100, 120, logger.GetLogger(), stats)
|
|
require.NoError(t, err)
|
|
|
|
ext := c.GetDelayExtension(100)
|
|
playoutDelayEqual(t, ext, 100, 120)
|
|
|
|
ext = c.GetDelayExtension(105)
|
|
playoutDelayEqual(t, ext, 100, 120)
|
|
|
|
// seq acked before delay changed
|
|
c.OnSeqAcked(65534)
|
|
ext = c.GetDelayExtension(105)
|
|
playoutDelayEqual(t, ext, 100, 120)
|
|
|
|
c.OnSeqAcked(90)
|
|
ext = c.GetDelayExtension(105)
|
|
playoutDelayEqual(t, ext, 100, 120)
|
|
|
|
// seq acked, no extension sent for new packet
|
|
c.OnSeqAcked(103)
|
|
ext = c.GetDelayExtension(106)
|
|
require.Nil(t, ext)
|
|
|
|
// delay on change(can't go below min), no extension sent
|
|
c.SetJitter(0)
|
|
ext = c.GetDelayExtension(107)
|
|
require.Nil(t, ext)
|
|
|
|
// delay changed, generate new extension to send
|
|
time.Sleep(200 * time.Millisecond)
|
|
c.SetJitter(50)
|
|
t.Log(c.currentDelay, c.state.Load())
|
|
ext = c.GetDelayExtension(108)
|
|
var delay pd.PlayOutDelay
|
|
require.NoError(t, delay.Unmarshal(ext))
|
|
require.Greater(t, delay.Min, uint16(100))
|
|
|
|
// can't go above max
|
|
time.Sleep(200 * time.Millisecond)
|
|
c.SetJitter(10000)
|
|
ext = c.GetDelayExtension(109)
|
|
playoutDelayEqual(t, ext, 120, 120)
|
|
}
|
|
|
|
func playoutDelayEqual(t *testing.T, data []byte, min, max uint16) {
|
|
var delay pd.PlayOutDelay
|
|
require.NoError(t, delay.Unmarshal(data))
|
|
require.Equal(t, min, delay.Min)
|
|
require.Equal(t, max, delay.Max)
|
|
}
|