Add causal chain to the stack string

errors.go

@@ -158,13 +158,28 @@ func (p *Error) StackTrace() []uintptr {
 	return out
 }
 
-// StackString formats the stack as a beautiful string with newlines
+// StackString formats the stacks from the terror chain as a string. If we
+// encounter more than one terror in the chain with a stack frame, we'll print
+// each one, separated by three hyphens on their own line.
 func (p *Error) StackString() string {
-	stackStr := ""
-	for _, frame := range p.StackFrames {
-		stackStr = fmt.Sprintf("%s\n  %s:%d in %s", stackStr, frame.Filename, frame.Line, frame.Method)
+	var buffer strings.Builder
+	terr := p
+	for terr != nil {
+		if buffer.Len() != 0 && len(terr.StackFrames) > 0 {
+			fmt.Fprintf(&buffer, "\n---")
+		}
+		for _, frame := range terr.StackFrames {
+			fmt.Fprintf(&buffer, "\n  %s:%d in %s", frame.Filename, frame.Line, frame.Method)
+		}
+
+		if tcause, ok := terr.cause.(*Error); ok {
+			terr = tcause
+		} else {
+			break
+		}
 	}
-	return stackStr
+
+	return buffer.String()
 }
 
 // VerboseString returns the error message, stack trace and params

errors_test.go

@@ -583,3 +583,15 @@ func TestSetIsUnexpected(t *testing.T) {
 	err.SetIsUnexpected(false)
 	assert.False(t, *err.IsUnexpected)
 }
+
+func failyFunction() error {
+	return InternalService("halp", "I'm in trouble", nil)
+}
+
+func TestStackStringChasesCausalChain(t *testing.T) {
+	err := Augment(failyFunction(), "something may be up", nil)
+	terr := err.(*Error)
+	ss := terr.StackString()
+	t.Log(ss)
+	assert.Contains(t, ss, "failyFunction")
+}