removed env variable and added as input

build.sbt

@@ -49,9 +49,7 @@ lazy val fileManager = project
   .settings(
     name := "interop-commons-file-manager",
     sharedSettings,
-    libraryDependencies ++= Dependencies.Jars.fileDependencies,
-    Test / fork := true,
-    Test / javaOptions += "-Dconfig.file=src/test/resources/application-test.conf"
+    libraryDependencies ++= Dependencies.Jars.fileDependencies
   )
   .dependsOn(utils)
   .setupBuildInfo

file-manager/src/main/scala/it/pagopa/interop/commons/files/StorageConfiguration.scala

@@ -5,7 +5,5 @@ import com.typesafe.config.{Config, ConfigFactory}
 object StorageConfiguration {
   val config: Config = ConfigFactory.load()
 
-  val maxConcurrency: Int        = config.getInt("interop-commons.storage.max-concurrency")
-  val getUrlDurationMinutes: Int = config.getInt("get-url-duration-minutes")
-  val putUrlDurationMinutes: Int = config.getInt("put-url-duration-minutes")
+  val maxConcurrency: Int = config.getInt("interop-commons.storage.max-concurrency")
 }

file-manager/src/main/scala/it/pagopa/interop/commons/files/service/FileManager.scala

@@ -42,9 +42,9 @@ trait FileManager {
 
   def close(): Unit
 
-  def generateGetPresignedUrl(bucketName: String, path: String, fileName: String): Try[String]
+  def generateGetPresignedUrl(bucketName: String, path: String, fileName: String, duration: Int): Try[String]
 
-  def generatePutPresignedUrl(bucketName: String, path: String, fileName: String): Try[String]
+  def generatePutPresignedUrl(bucketName: String, path: String, fileName: String, duration: Int): Try[String]
 }
 
 object FileManager {

file-manager/src/main/scala/it/pagopa/interop/commons/files/service/impl/FileManagerImpl.scala

@@ -86,12 +86,22 @@ final class FileManagerImpl(blockingExecutionContext: ExecutionContextExecutor)
     pathCreated.resolve(fileName.stripMargin('/'))
   }
 
-  override def generateGetPresignedUrl(bucketName: String, path: String, fileName: String): Try[String] = {
+  override def generateGetPresignedUrl(
+    bucketName: String,
+    path: String,
+    fileName: String,
+    duration: Int
+  ): Try[String] = {
     val destPath: String = createPath(path, "", fileName).toAbsolutePath.toString
     Try(destPath)
   }
 
-  override def generatePutPresignedUrl(bucketName: String, path: String, fileName: String): Try[String] = {
+  override def generatePutPresignedUrl(
+    bucketName: String,
+    path: String,
+    fileName: String,
+    duration: Int
+  ): Try[String] = {
     val destPath: String = createPath(path, "", fileName).toAbsolutePath.toString
     Try(destPath)
   }

file-manager/src/main/scala/it/pagopa/interop/commons/files/service/impl/S3ManagerImpl.scala

@@ -167,8 +167,13 @@ final class S3ManagerImpl(blockingExecutionContext: ExecutionContextExecutor)(
 
   def calcContentMd5(byteArray: Array[Byte]): String = new String(Base64.encodeBase64(DigestUtils.md5(byteArray)))
 
-  override def generateGetPresignedUrl(bucketName: String, path: String, fileName: String): Try[String] = {
-    val key: String                        = s3Key(path, "", fileName)
+  override def generateGetPresignedUrl(
+    bucketName: String,
+    path: String,
+    fileName: String,
+    duration: Int
+  ): Try[String] = {
+    val key: String = s3Key(path, "", fileName)
 
     Using(S3Presigner.create()) { s3Presigner =>
       val objectRequest: GetObjectRequest = GetObjectRequest
@@ -179,7 +184,7 @@ final class S3ManagerImpl(blockingExecutionContext: ExecutionContextExecutor)(
 
       val presignRequest: GetObjectPresignRequest = GetObjectPresignRequest
         .builder()
-        .signatureDuration(Duration.ofMinutes(StorageConfiguration.getUrlDurationMinutes.toLong))
+        .signatureDuration(Duration.ofMinutes(duration.toLong))
         .getObjectRequest(objectRequest)
         .build()
 
@@ -188,8 +193,13 @@ final class S3ManagerImpl(blockingExecutionContext: ExecutionContextExecutor)(
     }
   }
 
-  override def generatePutPresignedUrl(bucketName: String, path: String, fileName: String): Try[String] = {
-    val key: String                        = s3Key(path, "", fileName)
+  override def generatePutPresignedUrl(
+    bucketName: String,
+    path: String,
+    fileName: String,
+    duration: Int
+  ): Try[String] = {
+    val key: String = s3Key(path, "", fileName)
 
     Using(S3Presigner.create()) { s3Presigner =>
       val objectRequest: PutObjectRequest = PutObjectRequest
@@ -200,7 +210,7 @@ final class S3ManagerImpl(blockingExecutionContext: ExecutionContextExecutor)(
 
       val presignRequest: PutObjectPresignRequest = PutObjectPresignRequest
         .builder()
-        .signatureDuration(Duration.ofMinutes(StorageConfiguration.putUrlDurationMinutes.toLong))
+        .signatureDuration(Duration.ofMinutes(duration.toLong))
         .putObjectRequest(objectRequest)
         .build()
 

file-manager/src/test/scala/it/pagopa/interop/commons/files/S3FileManagerTest.scala

@@ -65,8 +65,8 @@ class S3FileManagerTest
 
     "have a single file with just one is written" in {
       val filesAndContent: Future[(List[String], String)] = for {
-        _       <- fileManager.storeBytes("testBucket", "testFolder", "testFile")("testFile".getBytes())
-        files   <- fileManager.listFiles("testBucket")("/testFolder")
+        _ <- fileManager.storeBytes("testBucket", "testFolder", "testFile")("testFile".getBytes())
+        files <- fileManager.listFiles("testBucket")("/testFolder")
         content <- fileManager.getFile("testBucket")("/testFolder/testFile").map(new String(_))
       } yield (files, content)
 
@@ -77,10 +77,10 @@ class S3FileManagerTest
 
     "have two files, one of whom in the root dir" in {
       val filesAndContent: Future[(List[String], String, String)] = for {
-        _           <- fileManager.storeBytes("testBucket", "", "rootFile")("rootFile".getBytes())
-        _           <- fileManager.storeBytes("testBucket", "testFolder", "testFile")("testFile".getBytes())
-        files       <- fileManager.listFiles("testBucket")("")
-        content     <- fileManager.getFile("testBucket")("/testFolder/testFile").map(new String(_))
+        _ <- fileManager.storeBytes("testBucket", "", "rootFile")("rootFile".getBytes())
+        _ <- fileManager.storeBytes("testBucket", "testFolder", "testFile")("testFile".getBytes())
+        files <- fileManager.listFiles("testBucket")("")
+        content <- fileManager.getFile("testBucket")("/testFolder/testFile").map(new String(_))
         rootContent <- fileManager.getFile("testBucket")("/rootFile").map(new String(_))
       } yield (files, content, rootContent)
 
@@ -91,27 +91,27 @@ class S3FileManagerTest
     }
     "get all the files" in {
       val filesF: Future[Map[String, Array[Byte]]] = for {
-        _     <- fileManager.storeBytes("testBucket", "", "rootFile")("rootFile".getBytes())
-        _     <- fileManager.storeBytes("testBucket", "testFolder", "testFile")("testFile".getBytes())
-        _     <- fileManager.storeBytes("testBucket", "testFolder/nestedFolder", "nestedFile")("nestedFile".getBytes())
+        _ <- fileManager.storeBytes("testBucket", "", "rootFile")("rootFile".getBytes())
+        _ <- fileManager.storeBytes("testBucket", "testFolder", "testFile")("testFile".getBytes())
+        _ <- fileManager.storeBytes("testBucket", "testFolder/nestedFolder", "nestedFile")("nestedFile".getBytes())
         files <- fileManager.getAllFiles("testBucket")("")
       } yield files
 
-      val files      = filesF.futureValue
+      val files = filesF.futureValue
       val contentMap = files.map { case (k, v) => (k, new String(v)) }
-      val expected   = Map(
-        "rootFile"                           -> "rootFile",
-        "testFolder/testFile"                -> "testFile",
+      val expected = Map(
+        "rootFile" -> "rootFile",
+        "testFolder/testFile" -> "testFile",
         "testFolder/nestedFolder/nestedFile" -> "nestedFile"
       )
       assert(contentMap == expected)
     }
     "Get presigned url" in {
-      val url = fileManager.generateGetPresignedUrl("testBucket", "testFolder", "testFile").get
+      val url = fileManager.generateGetPresignedUrl("testBucket", "testFolder", "testFile", 5).get
       assert(url.nonEmpty)
     }
     "Put presigned url" in {
-      val url = fileManager.generatePutPresignedUrl("testBucket", "testFolder", "testFile").get
+      val url = fileManager.generatePutPresignedUrl("testBucket", "testFolder", "testFile", 1).get
       assert(url.nonEmpty)
     }
   }