Skip to content

Commit

Permalink
Add TODO
Browse files Browse the repository at this point in the history
  • Loading branch information
smaye81 committed Feb 6, 2024
1 parent 0122e39 commit f8cd911
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions internal/app/connectconformance/results.go
Original file line number Diff line number Diff line change
Expand Up @@ -194,9 +194,11 @@ func (r *testResults) assert(testCase string, expected, actual *conformancev1.Cl
expectedWire := expected.WireDetails
actualWire := actual.WireDetails
if expectedWire != nil && actualWire != nil {
if diff := cmp.Diff(expectedWire.ConnectErrorRaw, actualWire.ConnectErrorRaw, protocmp.Transform()); diff != "" {
errs = append(errs, fmt.Errorf("raw Connect error does not match: - wanted, + got\n%s", diff))
}
// TODO - Add comparison (and tests) for connecterrorraw and actual http trailers

// if diff := cmp.Diff(expectedWire.ConnectErrorRaw, actualWire.ConnectErrorRaw, protocmp.Transform()); diff != "" {
// errs = append(errs, fmt.Errorf("raw Connect error does not match: - wanted, + got\n%s", diff))
// }

if diff := cmp.Diff(expectedWire.ActualStatusCode, actualWire.ActualStatusCode, protocmp.Transform()); diff != "" {
errs = append(errs, fmt.Errorf("actual HTTP status code does not match: - wanted, + got\n%s", diff))
Expand Down

0 comments on commit f8cd911

Please sign in to comment.