Merge pull request 'Fix login test . test for package content too' (#3) from fix_login_test into master

Reviewed-on: http://git.tfm.ro/mihaim/elbot/pulls/3

go.mod

@@ -2,4 +2,7 @@ module tfm.ro/elbot
 
 go 1.16
 
-require github.com/spf13/viper v1.8.1 // indirect
+require (
+	github.com/magiconair/properties v1.8.5 // indirect
+	github.com/spf13/viper v1.8.1 // indirect
+)

sending_test.go

@@ -3,6 +3,8 @@ package main
 import (
 	"net"
 	"testing"
+
+	"github.com/magiconair/properties/assert"
 )
 
 // Test scafolding
@@ -33,11 +35,16 @@ func TestSendLogin(t *testing.T) {
 				sendLogin(client, tt.name, tt.pass)
 			}()
 
-			reply := make([]byte, 4096)
+			reply := make([]byte, tt.resultLenght)
 			length, _ := server.Read(reply)
+
+			// test length of the received packet
 			if length != tt.resultLenght {
 				t.Errorf("Expected length %v, got %v buffer: %v for %s / %s", tt.resultLenght, length, reply[0:length], tt.name, tt.pass)
 			}
+
+			// test content of the packet
+			assert.Equal(t, reply, tt.resultBuffer, "Login buffers not the same")
 		})
 	}
 }