Files
sync_replay_project/apps/desktop/test/state_diff_test.dart
T
foxyzandClaude Opus 5.5 d8d487c189 Sync fixes: clean seeks, clock offset, end of file, 2.5 min read-ahead (proto 6)
- User seeks are intercepted at the player (SyncPlayer), previewed
  locally, sent once 300 ms after the drag ends; own echoes are matched
  by seekSeq and never re-applied; the landed check can't reapply a
  stale target. 1 s position poll removed.
- ping/pong clock offset (min-RTT ClockSync); positions use server time.
- Reports carry duration; buffer gate is min(4 s, remaining - 2 s);
  new finished transport instead of a HELD stall at the end.
- cache/readahead 150 s, 256 MiB demuxer cache, pins read back.
- Resumed reports put a stale ready member back in the set; hello.resume
  takes over the old member after an unclean drop (no ghost).
- Docs: roadmap rendering findings (HW path works; shaders verified).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
2026-10-07 01:20:18 +03:00

195 lines
6.7 KiB
Dart

import 'package:flutter_test/flutter_test.dart';
import 'package:sync_relay_core/core.dart';
import 'package:watch_together/src/sync/state_diff.dart';
StateBroadcast _state({
int eventId = 0,
String transport = 'held',
double position = 10.0,
int mediaId = 1,
}) {
final now = DateTime.now().toUtc();
return StateBroadcast(
eventId: eventId,
transport: transport,
position: position,
serverTime: now,
mediaId: mediaId,
members: const [],
);
}
StateBroadcast _stateWithMembers(List<MemberView> members,
{String transport = 'playing'}) {
final now = DateTime.now().toUtc();
return StateBroadcast(
eventId: 3,
transport: transport,
position: 10.0,
serverTime: now,
mediaId: 1,
members: members,
);
}
void main() {
test('initial diff seeks to server edge and sets transport', () {
final diff = StateDiff();
final actions = diff.diff(_state(), 0, DateTime.now().toUtc());
expect(actions.whereType<SeekAction>().single.position, closeTo(10.0, 1.0));
expect(actions.whereType<TransportAction>().single.playing, isFalse);
});
test('same event small drift does nothing', () {
final diff = StateDiff();
final now = DateTime.now().toUtc();
diff.diff(_state(), 0, now);
final actions = diff.diff(_state(), 10.5, now);
expect(actions, isEmpty);
});
test('ahead drift latches slowdown, resets inside 0.1', () {
final diff = StateDiff();
final now = DateTime.now().toUtc();
diff.diff(_state(), 0, now);
var actions = diff.diff(_state(), 13.0, now);
expect(actions.whereType<RateAction>().single.rate, 0.95);
actions = diff.diff(_state(), 12.0, now);
expect(actions, isEmpty);
actions = diff.diff(_state(), 10.05, now);
expect(actions.whereType<RateAction>().single.rate, 1.0);
});
test('eventId jump seeks and resets rate', () {
final diff = StateDiff();
final now = DateTime.now().toUtc();
diff.diff(_state(), 0, now);
diff.diff(_state(), 13.0, now);
final actions =
diff.diff(_state(eventId: 1, position: 90.0), 13.0, now);
expect(actions.whereType<SeekAction>().single.position, closeTo(90.0, 1.0));
expect(actions.whereType<RateAction>().single.rate, 1.0);
});
test('media swap seeks zero and pauses', () {
final diff = StateDiff();
final now = DateTime.now().toUtc();
diff.diff(_state(transport: 'playing'), 10.0, now);
final actions = diff.diff(
_state(eventId: 2, transport: 'held', position: 0, mediaId: 2),
50.0,
now);
expect(actions.whereType<SeekAction>().single.position, 0);
expect(actions.whereType<TransportAction>().single.playing, isFalse);
});
group('seek debounce while playing', () {
StateBroadcast playing({int eventId = 5, double position = 100.0}) => _state(
eventId: eventId, transport: 'playing', position: position);
test('fresh big drift is swallowed first, seeks after window', () {
final diff = StateDiff();
final t0 = DateTime.now().toUtc();
diff.diff(playing(), 100.0, t0);
var actions = diff.diff(playing(), 120.0, t0);
expect(actions.whereType<SeekAction>(), isEmpty,
reason: 'user scrub must not be yanked back before detector runs');
actions = diff.diff(
playing(), 120.0, t0.add(StateDiff.debounceWindow + const Duration(milliseconds: 100)));
expect(actions.whereType<SeekAction>().single.position,
closeTo(102.0, 3.0),
reason: 'genuine drift still converges after the window');
});
test('new eventId seeks immediately (echo of promoted seek)', () {
final diff = StateDiff();
final t0 = DateTime.now().toUtc();
diff.diff(playing(), 100.0, t0);
diff.diff(playing(), 120.0, t0);
final actions = diff.diff(playing(eventId: 6, position: 120.0), 120.0, t0);
expect(actions.whereType<SeekAction>().single.position,
closeTo(120.0, 2.0));
});
test('held transport debounces the same way (detector promotes)', () {
final diff = StateDiff();
final t0 = DateTime.now().toUtc();
diff.diff(_state(), 10.0, t0);
var actions = diff.diff(_state(), 60.0, t0);
expect(actions.whereType<SeekAction>(), isEmpty);
actions = diff.diff(_state(), 60.0,
t0.add(StateDiff.debounceWindow + const Duration(milliseconds: 100)));
expect(actions.whereType<SeekAction>().single.position,
closeTo(10.0, 3.0));
});
});
group('local play gate', () {
const ready = MemberView(
id: 'm1',
name: 'a',
ready: true,
inSet: true,
buffered: 9,
stale: false);
const catchingUp = MemberView(
id: 'm2',
name: 'b',
ready: false,
inSet: false,
buffered: 0,
stale: false);
const unreadyInSet = MemberView(
id: 'm3',
name: 'c',
ready: false,
inSet: true,
buffered: 9,
stale: false);
test('only in-set ready members may follow transport to playing', () {
final state = _stateWithMembers([ready, catchingUp, unreadyInSet]);
expect(localPlayAllowed(state, 'm1'), isTrue);
expect(localPlayAllowed(state, 'm2'), isFalse);
expect(localPlayAllowed(state, 'm3'), isFalse);
expect(localPlayAllowed(state, 'nope'), isFalse);
expect(localPlayAllowed(state, ''), isFalse);
});
});
group('finished', () {
test('stops in place: pause, no seek to the end, no drift seeks', () {
final diff = StateDiff();
final t = DateTime.now().toUtc();
diff.diff(_state(eventId: 1, transport: 'playing', position: 100), 100, t);
final end = _state(eventId: 2, transport: 'finished', position: 120);
final actions = diff.diff(end, 119.4, t);
expect(actions.whereType<SeekAction>(), isEmpty);
expect(actions.whereType<TransportAction>().single.playing, isFalse);
// Far off and later: still nothing.
expect(diff.diff(end, 50, t.add(const Duration(seconds: 10))), isEmpty);
});
test('late joiner lands just before the end, paused', () {
final diff = StateDiff();
final actions = diff.diff(
_state(eventId: 5, transport: 'finished', position: 120), 0,
DateTime.now().toUtc());
expect(actions.whereType<SeekAction>().single.position, 119);
expect(actions.whereType<TransportAction>().single.playing, isFalse);
});
});
test('serverPositionAt extrapolates on the server clock', () {
final st = StateBroadcast(
eventId: 1,
transport: 'playing',
position: 10,
serverTime: DateTime.utc(2026, 1, 1, 12),
mediaId: 1,
members: const [],
);
expect(serverPositionAt(st, DateTime.utc(2026, 1, 1, 12, 0, 2, 500)), 12.5);
});
}