Add regression test for RequiredAcks == RequireNone

This new test is required because all the other Writer tests use
NewWriter() to create Writers, which sets RequiredAcks to
RequireAll when 0 (None) was specified.

Author: neilcook

writer_test.go

@@ -51,6 +51,10 @@ func TestWriter(t *testing.T) {
 			scenario: "setting a non default balancer on the writer",
 			function: testWriterSetsRightBalancer,
 		},
+		{
+			scenario: "setting RequiredAcks to None in Writer doesn't cause a panic",
+			function: testWriterRequiredAcksNone,
+		},
 	}
 
 	for _, test := range tests {
@@ -83,6 +87,30 @@ func testWriterClose(t *testing.T) {
 	}
 }
 
+func testWriterRequiredAcksNone(t *testing.T) {
+	topic := makeTopic()
+	createTopic(t, topic, 1)
+	defer deleteTopic(t, topic)
+
+	w := &Writer{
+		Addr:     TCP("localhost:9092"),
+		Topic:    topic,
+		Balancer: &RoundRobin{},
+		RequiredAcks: RequireNone,
+	}
+	defer w.Close()
+
+	msg := Message{
+		Key:   []byte("ThisIsAKey"),
+		Value: []byte("Test message for required acks test")}
+
+	err := w.WriteMessages(context.Background(), msg)
+	if err != nil {
+		t.Error(err)
+		t.FailNow()
+	}
+}
+
 func testWriterSetsRightBalancer(t *testing.T) {
 	const topic = "test-writer-1"
 	balancer := &CRC32Balancer{}