Skip to content

Commit

Permalink
test: add signal peer test
Browse files Browse the repository at this point in the history
  • Loading branch information
braginini committed Jun 3, 2021
1 parent 9b327ea commit 33a98c7
Show file tree
Hide file tree
Showing 2 changed files with 50 additions and 2 deletions.
4 changes: 2 additions & 2 deletions signal/peer/peer.go
Original file line number Diff line number Diff line change
Expand Up @@ -47,8 +47,8 @@ func (reg *Registry) Register(peer *Peer) {
reg.Peers[peer.Id] = peer
}

// DeregisterHub deregister Peer from the Registry (usually once it disconnects)
func (reg *Registry) DeregisterHub(peer *Peer) {
// Deregister deregister Peer from the Registry (usually once it disconnects)
func (reg *Registry) Deregister(peer *Peer) {
if _, ok := reg.Peers[peer.Id]; ok {
delete(reg.Peers, peer.Id)
log.Printf("deregistered peer [%s]", peer.Id)
Expand Down
48 changes: 48 additions & 0 deletions signal/peer/peer_test.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,48 @@
package peer

import (
"testing"
)

func TestRegistry_Register(t *testing.T) {
r := NewRegistry()
peer1 := NewPeer("test_peer_1", nil)
peer2 := NewPeer("test_peer_2", nil)
r.Register(peer1)
r.Register(peer2)

if len(r.Peers) != 2 {
t.Errorf("expected 2 registered peers")
}

if _, ok := r.Peers["test_peer_1"]; !ok {
t.Errorf("expected test_peer_1 not found in the registry")
}

if _, ok := r.Peers["test_peer_2"]; !ok {
t.Errorf("expected test_peer_2 not found in the registry")
}
}

func TestRegistry_Deregister(t *testing.T) {
r := NewRegistry()
peer1 := NewPeer("test_peer_1", nil)
peer2 := NewPeer("test_peer_2", nil)
r.Register(peer1)
r.Register(peer2)

r.Deregister(peer1)

if len(r.Peers) != 1 {
t.Errorf("expected 1 registered peers after deregistring")
}

if _, ok := r.Peers["test_peer_1"]; ok {
t.Errorf("expected test_peer_1 to absent in the registry after deregistering")
}

if _, ok := r.Peers["test_peer_2"]; !ok {
t.Errorf("expected test_peer_2 not found in the registry")
}

}

0 comments on commit 33a98c7

Please sign in to comment.