Skip to content

Commit

Permalink
caching
Browse files Browse the repository at this point in the history
  • Loading branch information
THWiseman committed Nov 2, 2023
1 parent 3153345 commit f958150
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -156,7 +156,7 @@ object TesAsyncBackendJobExecutionActor {
// Log if not all input files live in the same container.
// We'll do our best anyway, but will still only be able to retrieve a token for a single container.
if(blobFiles.forall(_.container == blobFiles.headOption.map(file => file.container).getOrElse(BlobContainerName("no_container")))) {
logger.warn(s"While parsing blob inputs, found more than one container. Can only generate an environment sas token for a single blob container at once.")
logger.info(s"While parsing blob inputs, found more than one container. Can only generate an environment sas token for a single blob container at once.")

Check warning on line 159 in supportedBackends/tes/src/main/scala/cromwell/backend/impl/tes/TesAsyncBackendJobExecutionActor.scala

View check run for this annotation

Codecov / codecov/patch

supportedBackends/tes/src/main/scala/cromwell/backend/impl/tes/TesAsyncBackendJobExecutionActor.scala#L158-L159

Added lines #L158 - L159 were not covered by tests
}

// We use the first blob file in the list as a template for determining the localized sas params
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -72,11 +72,13 @@ class TesAsyncBackendJobExecutionActorSpec extends AnyFlatSpec with Matchers wit
mockFsm.blobTokenGenerator returns mockTokenGenerator

val mockPath: cromwell.core.path.Path = mock[cromwell.core.path.Path]
mockPath.normalize() returns mockPath
mockPath.normalize returns mockPath

val mockNioPath: path.NioPath = mock[path.NioPath]
val mockJavaPath: Path = mock[java.nio.file.Path]

mockNioPath.toAbsolutePath returns mockJavaPath
mockNioPath.normalize returns mockNioPath

mockBlobPath.getFilesystemManager returns mockFsm
mockBlobPath.nioPath returns mockNioPath
mockBlobPath.toAbsolutePath returns mockPath
Expand Down

0 comments on commit f958150

Please sign in to comment.