Skip to content

Commit

Permalink
[8.x](backport #4012) [Integration testing] Remove AltErr and only ex…
Browse files Browse the repository at this point in the history
…pect 400 from ES bulk API (#4030)

* [Integration testing] Remove AltErr and only expect 400 from ES bulk API (#4012)

* Remove AltErr

* Fixed error type

(cherry picked from commit 1ae9ab6)

# Conflicts:
#	internal/pkg/bulk/bulk_integration_test.go

* Update bulk_integration_test.go

Fix mergify mess

* Printing out filename for debugging

* Fixing up conflicts

* Trigger Build

---------

Co-authored-by: Shaunak Kashyap <[email protected]>
Co-authored-by: Julien Lind <[email protected]>
  • Loading branch information
3 people authored Oct 30, 2024
1 parent a3d14ee commit 457a697
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions internal/pkg/bulk/bulk_integration_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -69,8 +69,8 @@ func TestBulkCreate(t *testing.T) {
Name: "Invalid utf-8",
Index: string([]byte{0xfe, 0xfe, 0xff, 0xff}),
Err: es.ErrElastic{
Status: 500,
Type: "json_parse_exception",
Status: 400,
Type: "parse_exception",
},
},
{
Expand Down Expand Up @@ -100,7 +100,7 @@ func TestBulkCreate(t *testing.T) {
// Create
id, err := bulker.Create(ctx, test.Index, test.ID, sampleData)
if !EqualElastic(test.Err, err) {
t.Fatal(err)
t.Fatalf("expected error: %+v, got: %+v", test.Err, err)
}
if err != nil {
return
Expand Down

0 comments on commit 457a697

Please sign in to comment.