return nil ocStatus when only message is provided

Author: chris-smith-zocdoc

translator/trace/zipkin/status_code.go

@@ -74,7 +74,7 @@ func (m *statusMapper) ocStatus() *tracepb.Status {
 		}
 	}
 
-	if s.codePtr != nil || s.message != "" {
+	if s.codePtr != nil {
 		code := int32(0)
 		if s.codePtr != nil {
 			code = *s.codePtr

translator/trace/zipkin/status_code_test.go

@@ -116,6 +116,14 @@ func TestStatusCodeMapperCases(t *testing.T) {
 			},
 		},
 
+		{
+			name:     "http: message only, nil",
+			expected: nil,
+			attributes: map[string]string{
+				"http.status_message": "something",
+			},
+		},
+
 		{
 			name:     "http: 500",
 			expected: &tracepb.Status{Code: 13, Message: "a message"},
@@ -161,8 +169,8 @@ func TestStatusCodeMapperCases(t *testing.T) {
 		},
 
 		{
-			name:     "oc: description only, status ok",
-			expected: &tracepb.Status{Code: 0, Message: "a description"},
+			name:     "oc: description only, no status",
+			expected: nil,
 			attributes: map[string]string{
 				"opencensus.status_description": "a description",
 			},