Repository: qpid-proton
Updated Branches:
  refs/heads/master c36a1a711 -> 90dbf46cc


PROTON-1910: [go] fix faulty message test


Project: http://git-wip-us.apache.org/repos/asf/qpid-proton/repo
Commit: http://git-wip-us.apache.org/repos/asf/qpid-proton/commit/d8cebe49
Tree: http://git-wip-us.apache.org/repos/asf/qpid-proton/tree/d8cebe49
Diff: http://git-wip-us.apache.org/repos/asf/qpid-proton/diff/d8cebe49

Branch: refs/heads/master
Commit: d8cebe494cac78ba6959ada4af2b3fc8e958769d
Parents: 674e023
Author: Alan Conway <acon...@redhat.com>
Authored: Fri Oct 12 09:25:38 2018 -0400
Committer: Alan Conway <acon...@redhat.com>
Committed: Tue Oct 23 20:04:07 2018 -0400

----------------------------------------------------------------------
 go/src/qpid.apache.org/amqp/message.go      | 4 ++--
 go/src/qpid.apache.org/amqp/message_test.go | 6 +++---
 2 files changed, 5 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/qpid-proton/blob/d8cebe49/go/src/qpid.apache.org/amqp/message.go
----------------------------------------------------------------------
diff --git a/go/src/qpid.apache.org/amqp/message.go 
b/go/src/qpid.apache.org/amqp/message.go
index e96b331..0589d67 100644
--- a/go/src/qpid.apache.org/amqp/message.go
+++ b/go/src/qpid.apache.org/amqp/message.go
@@ -117,7 +117,7 @@ type Message interface {
        ReplyToGroupId() string
        SetReplyToGroupId(string)
 
-       // Property map set by the application to be carried with the message.
+       // Properties set by the application to be carried with the message.
        // Values must be simple types (not maps, lists or sequences)
        ApplicationProperties() map[string]interface{}
        SetApplicationProperties(map[string]interface{})
@@ -151,7 +151,7 @@ type Message interface {
        // Marshal a Go value into the message body, synonym for SetBody()
        Marshal(interface{})
 
-       // Unmarshal the message body into the value pointed to by v. See 
amqp.Unmarshal()
+       // Unmarshal the message body, using amqp.Unmarshal()
        Unmarshal(interface{})
 
        // Encode encodes the message as AMQP data. If buffer is non-nil and is 
large enough

http://git-wip-us.apache.org/repos/asf/qpid-proton/blob/d8cebe49/go/src/qpid.apache.org/amqp/message_test.go
----------------------------------------------------------------------
diff --git a/go/src/qpid.apache.org/amqp/message_test.go 
b/go/src/qpid.apache.org/amqp/message_test.go
index 321a9a0..9899bd5 100644
--- a/go/src/qpid.apache.org/amqp/message_test.go
+++ b/go/src/qpid.apache.org/amqp/message_test.go
@@ -39,6 +39,9 @@ func roundTrip(m Message) error {
 
 func TestDefaultMessage(t *testing.T) {
        m := NewMessage()
+       if err := roundTrip(m); err != nil {
+               t.Error(err)
+       }
        mv := reflect.ValueOf(m)
        // Check defaults
        for _, x := range []struct {
@@ -75,9 +78,6 @@ func TestDefaultMessage(t *testing.T) {
                        t.Errorf("%s: %s", x.method, err)
                }
        }
-       if err := roundTrip(m); err != nil {
-               t.Error(err)
-       }
        if err := checkEqual("Message{}", m.String()); err != nil {
                t.Error(err)
        }


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@qpid.apache.org
For additional commands, e-mail: commits-h...@qpid.apache.org

Reply via email to