Skip to content

Commit

Permalink
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
fmt
Browse files Browse the repository at this point in the history
lucymcnatt committed Jan 16, 2025

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
1 parent 24e2d6d commit 57a77bf
Showing 1 changed file with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
@@ -61,10 +61,10 @@ case class GcsUriDownloader(gcsUrl: String,
backoff foreach { b => Thread.sleep(b.backoffMillis) }

Check warning on line 61 in cromwell-drs-localizer/src/main/scala/drs/localizer/downloaders/GcsUriDownloader.scala

Codecov / codecov/patch

cromwell-drs-localizer/src/main/scala/drs/localizer/downloaders/GcsUriDownloader.scala#L60-L61

Added lines #L60 - L61 were not covered by tests
logger.warn(s"Attempting download retry $downloadAttempt of $downloadRetries for a GCS url")
downloadWithRetries(downloadRetries,
backoff map {
_.next
},
downloadAttempt + 1
backoff map {
_.next

Check warning on line 65 in cromwell-drs-localizer/src/main/scala/drs/localizer/downloaders/GcsUriDownloader.scala

Codecov / codecov/patch

cromwell-drs-localizer/src/main/scala/drs/localizer/downloaders/GcsUriDownloader.scala#L63-L65

Added lines #L63 - L65 were not covered by tests
},
downloadAttempt + 1

Check warning on line 67 in cromwell-drs-localizer/src/main/scala/drs/localizer/downloaders/GcsUriDownloader.scala

Codecov / codecov/patch

cromwell-drs-localizer/src/main/scala/drs/localizer/downloaders/GcsUriDownloader.scala#L67

Added line #L67 was not covered by tests
)
} else {
IO.raiseError(new RuntimeException(s"Exhausted $downloadRetries resolution retries to download GCS file"))

Check warning on line 70 in cromwell-drs-localizer/src/main/scala/drs/localizer/downloaders/GcsUriDownloader.scala

Codecov / codecov/patch

cromwell-drs-localizer/src/main/scala/drs/localizer/downloaders/GcsUriDownloader.scala#L70

Added line #L70 was not covered by tests

0 comments on commit 57a77bf

Please sign in to comment.