Skip to content

Update Tracker Architecture for Multi-Peer Connections #16

Update Tracker Architecture for Multi-Peer Connections

Update Tracker Architecture for Multi-Peer Connections #16

Triggered via pull request June 21, 2024 06:44
Status Failure
Total duration 31s
Artifacts

pr.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

5 errors and 1 warning
test/announce.test.ts > announce > should add peers to swarms on announce: test/announce.test.ts#L55
AssertionError: expected [ { peerId: '0', …(4) } ] to be a superset of [ PeerContextClass{ …(2) } ] - Expected + Received Array [ - PeerContextClass { - "0": Object { + Object { "lastAccessed": 720.504278, "peerId": "0", "sendMessage": [Function anonymous], - "socketContext": [Circular], + "socketContext": PeerContextClass { + "0": [Circular], + "sendMessage": [Function anonymous], + }, "swarm": Swarm { "completedCount": 0, "infoHash": "swarm1", - }, }, - "sendMessage": [Function anonymous], }, ] ❯ test/announce.test.ts:55:60
test/announce.test.ts > announce > should send offers to peers in a swarm: lib/fast-tracker.ts#L185
TypeError: Cannot read properties of null (reading 'peerId') ❯ FastTracker.processAnnounce lib/fast-tracker.ts:185:28 ❯ FastTracker.processMessage lib/fast-tracker.ts:127:14 ❯ test/announce.test.ts:181:13
test/announce.test.ts > announce > should process answer messages: lib/fast-tracker.ts#L185
TypeError: Cannot read properties of null (reading 'peerId') ❯ FastTracker.processAnnounce lib/fast-tracker.ts:185:28 ❯ FastTracker.processMessage lib/fast-tracker.ts:127:14 ❯ test/announce.test.ts:464:13
test/simulation.test.ts > simulation > should pass random simulations: test/simulation.test.ts#L111
AssertionError: expected undefined to exist ❯ test/simulation.test.ts:111:26
build-pr
Process completed with exit code 1.
build-pr
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-node@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.