Skip to content

Commit

Permalink
FMWK-556-restore-folder-validation
Browse files Browse the repository at this point in the history
- tests
  • Loading branch information
filkeith committed Oct 2, 2024
1 parent 0d07702 commit 15313fa
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 40 deletions.
16 changes: 3 additions & 13 deletions io/azure/blob/reader_writer_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -219,19 +219,9 @@ func (s *AzureSuite) TestReader_StreamFilesEmpty() {

go reader.StreamFiles(ctx, rCH, eCH)

var filesCounter int

for {
select {
case err := <-eCH:
s.Require().NoError(err)
case _, ok := <-rCH:
if !ok {
require.Equal(s.T(), 0, filesCounter)
return
}
filesCounter++
}
for err = range eCH {
s.Require().ErrorContains(err, "is empty")
return
}
}

Expand Down
16 changes: 3 additions & 13 deletions io/gcp/storage/reader_writer_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -276,19 +276,9 @@ func (s *GCPSuite) TestReader_StreamFilesEmpty() {

go reader.StreamFiles(ctx, rCH, eCH)

var filesCounter int

for {
select {
case err := <-eCH:
s.Require().NoError(err)
case _, ok := <-rCH:
if !ok {
require.Equal(s.T(), 0, filesCounter)
return
}
filesCounter++
}
for err = range eCH {
s.Require().ErrorContains(err, "is empty")
return
}
}

Expand Down
17 changes: 3 additions & 14 deletions io/local/reader_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -147,20 +147,9 @@ func (s *checkRestoreDirectoryTestSuite) TestDirectoryReader_StreamFiles_ErrEmpt
errorChan := make(chan error)
go streamingReader.StreamFiles(context.Background(), readerChan, errorChan)

var counter int
for {
select {
case _, ok := <-readerChan:
// if chan closed, we're done.
if !ok {
s.Require().Equal(2, counter)
return
}
counter++
case err = <-errorChan:
require.ErrorContains(s.T(), err, "is empty")
return
}
for err = range errorChan {
s.Require().ErrorContains(err, "is empty")
return
}
}

Expand Down

0 comments on commit 15313fa

Please sign in to comment.