client: Changed Join and Part method to use queued send instead. [BC Break: removed error return]

client.go

@@ -473,13 +473,13 @@ func (client *Client) PrivmsgOverhead(targetName string, action bool) int {
 }
 
 // Join joins one or more channels without a key.
-func (client *Client) Join(channels ...string) error {
+func (client *Client) Join(channels ...string) {
-	return client.Sendf("JOIN %s", strings.Join(channels, ","))
+	client.SendQueuedf("JOIN %s", strings.Join(channels, ","))
 }
 
 // Part parts one or more channels.
-func (client *Client) Part(channels ...string) error {
+func (client *Client) Part(channels ...string) {
-	return client.Sendf("PART %s", strings.Join(channels, ","))
+	client.SendQueuedf("PART %s", strings.Join(channels, ","))
 }
 
 // Target gets a target by kind and name

client_test.go

@@ -3,7 +3,6 @@ package irc_test
 import (
 	"context"
 	"errors"
-	"fmt"
 	"testing"
 
 	"git.aiterp.net/gisle/irc"
@@ -102,11 +101,7 @@ func TestClient(t *testing.T) {
 				return nil
 			}},
 			{Callback: func() error {
-				err := client.Join("#Test")
+				client.Join("#Test")
-				if err != nil {
-					return fmt.Errorf("Failed to join #Test: %s", err)
-				}
-
 				return nil
 			}},
 			{Kind: 'C', Data: "JOIN #Test"},
@@ -264,11 +259,7 @@ func TestClient(t *testing.T) {
 			{Kind: 'C', Data: "PRIVMSG #Test :\x01ACTION does stuff with 42 things\x01"},
 			{Kind: 'C', Data: "PRIVMSG #Test :Hello, World"},
 			{Callback: func() error {
-				err := client.Part("#Test")
+				client.Part("#Test")
-				if err != nil {
-					return fmt.Errorf("Failed to part #Test: %s", err)
-				}
-
 				return nil
 			}},
 			{Kind: 'C', Data: "PART #Test"},