Skip to content

Commit

Permalink
Merge branch 'develop' into aws-ecr-remote-hashing
Browse files Browse the repository at this point in the history
  • Loading branch information
sberss authored Jun 4, 2024
2 parents 6c6a075 + 1ca1bb0 commit fbf415b
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@ case class BulkAccessUrlDownloader(resolvedUrls: List[ResolvedDrsUrl]) extends D
Files.deleteIfExists(getmManifestPath)

def generateGetmCommand(pathToMainfestJson: Path): String =
s"""timeout 24h getm --manifest ${pathToMainfestJson.toString} -vv"""
s"""timeout 24h getm --manifest ${pathToMainfestJson.toString}"""
def runGetm: IO[GetmResult] =
generateJsonManifest(resolvedUrls).flatMap { manifestPath =>
val script = generateGetmCommand(manifestPath)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ class BulkAccessUrlDownloaderSpec extends AnyFlatSpec with CromwellTimeoutSpec w
it should "properly construct the invocation command" in {
val downloader = BulkAccessUrlDownloader(oneElement)
val filepath: Path = downloader.generateJsonManifest(threeElements).unsafeRunSync()
val expected = s"""timeout 24h getm --manifest ${filepath.toString} -vv"""
val expected = s"""timeout 24h getm --manifest ${filepath.toString}"""
downloader.generateGetmCommand(filepath) shouldBe expected
}

Expand Down

0 comments on commit fbf415b

Please sign in to comment.