small tweaks

server/peer.go

···
 package server
 
 import (
-	"log/slog"
 	"net"
 	"sync"
-
-	"github.com/google/uuid"
 )
 
 // Status represents the status of a request
···
 type peer struct {
 	conn   net.Conn
 	connMu sync.Mutex
-	name   string
 }
 
 func newPeer(conn net.Conn) peer {
 	return peer{
 		conn: conn,
-		name: uuid.New().String(),
 	}
 }
 
···
 type connOpp func(conn net.Conn) error
 
 func (p *peer) connOperation(op connOpp, from string) error {
-	slog.Info("operation running", "from", from, "peer", p.conn.RemoteAddr(), "name", p.name, "mu addr", &p.connMu)
-
 	p.connMu.Lock()
-	err := op(p.conn)
-	p.connMu.Unlock()
+	defer p.connMu.Unlock()
 
-	slog.Info("operation finished", "from", from, "peer", p.conn.RemoteAddr(), "name", p.name, "mu addr", &p.connMu)
-
-	return err
+	return op(p.conn)
 }

server/server.go

···
 		op := func(conn net.Conn) error {
 			dataLen, err := dataLength(conn)
 			if err != nil {
-				slog.Error(err.Error(), "peer", peer.addr())
+				slog.Error("failed to read data length", "error", err, "peer", peer.addr())
 				writeStatus(Error, "invalid data length of data provided", conn)
 				return nil
 			}

server/server_test.go

···
 func TestSendsDataToTopicSubscribers(t *testing.T) {
 	_ = createServer(t)
 
-	subscribers := make([]net.Conn, 0, 1)
-	for i := 0; i < 1; i++ {
+	subscribers := make([]net.Conn, 0, 10)
+	for i := 0; i < 10; i++ {
 		subscriberConn := createConnectionAndSubscribe(t, []string{topicA, topicB})
 
 		subscribers = append(subscribers, subscriberConn)