Refactored and corrected client's network data reading

[goircd.git] / client_test.go

diff --git a/client_test.go b/client_test.go
index 22cc401ea920bacc0e36c245145359be47d851d1..3735e1f71034f8c3ebcd03f7efb63c522eff4c1a 100644 (file)
--- a/client_test.go
+++ b/client_test.go
@@ -1,6 +1,6 @@
 /*
 goircd -- minimalistic simple Internet Relay Chat (IRC) server
-Copyright (C) 2014 Sergey Matveev <stargrave@stargrave.org>
+Copyright (C) 2014-2015 Sergey Matveev <stargrave@stargrave.org>
 
 This program is free software: you can redistribute it and/or modify
 it under the terms of the GNU General Public License as published by
@@ -15,6 +15,7 @@ GNU General Public License for more details.
 You should have received a copy of the GNU General Public License
 along with this program.  If not, see <http://www.gnu.org/licenses/>.
 */
+
 package main
 
 import (
@@ -46,10 +47,10 @@ func (conn *TestingConn) Read(b []byte) (n int, err error) {
        if msg == "" {
                return 0, conn
        }
-       for n, bt := range []byte(msg + CRLF) {
+       for n, bt := range append([]byte(msg), CRLF...) {
                b[n] = bt
        }
-       return len(msg), nil
+       return len(msg)+2, nil
 }
 
 type MyAddr struct{}
@@ -96,30 +97,27 @@ func (conn TestingConn) SetWriteDeadline(t time.Time) error {
 func TestNewClient(t *testing.T) {
        conn := NewTestingConn()
        sink := make(chan ClientEvent)
-       client := NewClient("foohost", conn)
+       host := "foohost"
+       client := NewClient(&host, conn)
        go client.Processor(sink)
 
        event := <-sink
-       if event.event_type != EVENT_NEW {
+       if event.eventType != EventNew {
                t.Fatal("no NEW event", event)
        }
        conn.inbound <- "foo"
        event = <-sink
-       ts1 := client.timestamp
-       if (event.event_type != EVENT_MSG) || (event.text != "foo") {
+       if (event.eventType != EventMsg) || (event.text != "foo") {
                t.Fatal("no first MSG", event)
        }
        conn.inbound <- "bar"
        event = <-sink
-       if (event.event_type != EVENT_MSG) || (event.text != "bar") {
+       if (event.eventType != EventMsg) || (event.text != "bar") {
                t.Fatal("no second MSG", event)
        }
        conn.inbound <- ""
-       if client.timestamp.Before(ts1) || client.timestamp.Equal(ts1) {
-               t.Fatal("timestamp updating")
-       }
        event = <-sink
-       if event.event_type != EVENT_DEL {
+       if event.eventType != EventDel {
                t.Fatal("no client termination", event)
        }
 }
@@ -127,7 +125,8 @@ func TestNewClient(t *testing.T) {
 // Test replies formatting
 func TestClientReplies(t *testing.T) {
        conn := NewTestingConn()
-       client := NewClient("foohost", conn)
+       host := "foohost"
+       client := NewClient(&host, conn)
        client.nickname = "мойник"
 
        client.Reply("hello")