Skip to content

Commit

Permalink
Empty bucket listing error fix (#3)
Browse files Browse the repository at this point in the history
* ls: change error code for no object found case

* fix main case for bucket path (tests, other List cases and README remaining)

* add bucket case for all cases

* rm comment lines

* add empty bucket test

* add changes to changelog

* add tests for cmds that use ls
  • Loading branch information
tarikozyurtt authored Jul 5, 2024
1 parent ec32627 commit 6374ff2
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
1 change: 1 addition & 0 deletions e2e/du_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -312,6 +312,7 @@ func TestDiskUsageEmptyBucket(t *testing.T) {
cmd := s5cmd("du", "s3://"+bucket)
result := icmd.RunCmd(cmd)


result.Assert(t, icmd.Success)

assertLines(t, result.Stdout(), map[int]compareFunc{
Expand Down
2 changes: 0 additions & 2 deletions e2e/ls_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -799,6 +799,4 @@ func TestEmptyBucket(t *testing.T) {
result := icmd.RunCmd(cmd)

result.Assert(t, icmd.Success)

assertLines(t, result.Stdout(), nil)
}

0 comments on commit 6374ff2

Please sign in to comment.