Add errors test case, check for sensitive attachments

error_struct.go

@@ -18,6 +18,8 @@ func newBerr(errorType ErrorType, errorMessage string, attachments ...Attachment
 			next = fmt.Errorf("%s: %w", next, d.Value().(error))
 		case d.Type() == AttachmentMetadataType:
 			errorMetadata[d.Key()] = d.Value()
+		case d.Sensitive() == true:
+			errorMetadata[d.Key()] = d.Value()
 		default:
 			errorDetail[d.Key()] = d.Value()
 		}

error_test.go

@@ -195,6 +195,35 @@ func TestTimeoutError(t *testing.T) {
 	testErrorFunc(t, berr.TimeoutErrorType, berr.Timeout, "request cancelled")
 }
 
+func TestErrorsFormatting(t *testing.T) {
+	const unimplementedErrorMessage = "store.GetData method not implemented"
+
+	const applicationErrorMessage = "unexpected problem querying database"
+
+	errs := berr.Errors{
+		berr.Unimplemented(unimplementedErrorMessage),
+		berr.Application(applicationErrorMessage),
+	}
+
+	errorString := errs.Error()
+
+	if !strings.Contains(errorString, unimplementedErrorMessage) {
+		t.Errorf("expected error string to contain '%s', found: %s\n", unimplementedErrorMessage, errorString)
+	}
+
+	if !strings.Contains(errorString, applicationErrorMessage) {
+		t.Errorf("expected error string to contain '%s', found: %s\n", applicationErrorMessage, errorString)
+	}
+
+	if !strings.HasPrefix(errorString, "[") {
+		t.Errorf("expected error string to start with '['\n")
+	}
+
+	if !strings.HasSuffix(errorString, "]") {
+		t.Errorf("expected error string to start with ']'\n")
+	}
+}
+
 func testErrorFunc(
 	t *testing.T,
 	errorType berr.ErrorType,