Merge pull request #39 from cesarsalgado/master

Added public GetError method to the packet.

packet.go

@@ -19,6 +19,10 @@ type Packet interface {
 
 	// IsError returns a boolean defining whether the Packet contains an error.
 	IsError() bool
+
+	// Returns the packet's error or nil if there is none.
+	GetError() error
+
 	// IsOk returns a boolean defining whether Packet was success
 	IsOk() bool
 }
@@ -54,6 +58,14 @@ func (p *packet) getStatusCode() int {
 	return p.cmd >> 24 & 127
 }
 
+func (p *packet) GetError() error {
+	if p.IsError() {
+		return p.getError()
+	} else {
+		return nil
+	}
+}
+
 func (p *packet) getError() error {
 	errDescMap := map[int]string{
 		2:   "Invalid expression",

packet_test.go

@@ -14,6 +14,22 @@ func TestErrorPacketIsError(t *testing.T) {
 	}
 }
 
+func TestErrorPacketGetErrorNonNil(t *testing.T) {
+	pkt := newErrorPacket(errors.New("test error"))
+	err := pkt.GetError()
+	if err == nil {
+		t.Error("GetError should return a non nil error when the packet has an error")
+	}
+}
+
+func TestErrorPacketGetErrorNil(t *testing.T) {
+	pkt := newErrorPacket(nil)
+	err := pkt.GetError()
+	if err != nil {
+		t.Error("GetError should return nil when the packet has no error")
+	}
+}
+
 func TestErrorPacketResultObject(t *testing.T) {
 	testError := errors.New("test error")
 	pkt := newErrorPacket(testError)