Skip to content

Commit abc92eb

Browse files
committed
refactoring after upgrading server dependencies
1 parent 2ef4857 commit abc92eb

File tree

9 files changed

+36
-24
lines changed

9 files changed

+36
-24
lines changed

build.sbt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ ThisBuild / organization := "app.softnetwork"
3131

3232
name := "resource"
3333

34-
ThisBuild / version := "0.2.3"
34+
ThisBuild / version := "0.2.4"
3535

3636
ThisBuild / scalaVersion := "2.12.15"
3737

core/src/main/scala/app/softnetwork/resource/launch/ResourceRoutes.scala

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
11
package app.softnetwork.resource.launch
22

33
import akka.actor.typed.ActorSystem
4-
import akka.http.scaladsl.server.Route
5-
import app.softnetwork.api.server.ApiRoutes
4+
import app.softnetwork.api.server.{ApiRoute, ApiRoutes}
65
import app.softnetwork.persistence.schema.SchemaProvider
76
import app.softnetwork.resource.model.GenericResource
87
import app.softnetwork.resource.service.ResourceService
@@ -16,6 +15,10 @@ trait ResourceRoutes[Resource <: GenericResource]
1615

1716
def resourceService: ActorSystem[_] => ResourceService
1817

19-
override def apiRoutes(system: ActorSystem[_]): Route = resourceService(system).route
18+
override def apiRoutes: ActorSystem[_] => List[ApiRoute] =
19+
system =>
20+
List(
21+
resourceService(system)
22+
)
2023

2124
}

core/src/main/scala/app/softnetwork/resource/service/ResourceService.scala

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,8 @@ trait ResourceService
2828
with DefaultComplete
2929
with Json4sSupport
3030
with StrictLogging
31-
with LoadResourceService {
31+
with LoadResourceService
32+
with ApiRoute {
3233
_: GenericResourceHandler with ResourceProvider =>
3334

3435
implicit def serialization: Serialization.type = jackson.Serialization

core/src/main/scala/app/softnetwork/resource/service/ResourceServiceEndpoints.scala

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
package app.softnetwork.resource.service
22

3-
import akka.http.scaladsl.server.Route
43
import akka.stream.scaladsl.{FileIO, Source}
54
import akka.util.ByteString
65
import app.softnetwork.api.server.ApiErrors
@@ -232,7 +231,6 @@ trait ResourceServiceEndpoints
232231
deleteResource
233232
)
234233

235-
lazy val route: Route = apiRoute
236234
}
237235

238236
sealed trait Upload {

project/src/main/scala/app/softnetwork/sbt/build/Versions.scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ package app.softnetwork.sbt.build
22

33
object Versions {
44

5-
val genericPersistence = "0.3.4"
5+
val genericPersistence = "0.3.5"
66

77
val scalatest = "3.1.1"
88
}

testkit/src/main/scala/app/softnetwork/resource/scalatest/ResourceEndpointsTestKit.scala

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2,18 +2,20 @@ package app.softnetwork.resource.scalatest
22

33
import akka.actor.typed.ActorSystem
44
import app.softnetwork.api.server.ApiEndpoint
5-
import app.softnetwork.resource.service.ResourceServiceEndpoints
5+
import app.softnetwork.persistence.schema.SchemaProvider
6+
import app.softnetwork.resource.launch.ResourceEndpoints
7+
import app.softnetwork.resource.model.GenericResource
68
import app.softnetwork.session.scalatest.SessionEndpointsRoutes
79
import com.softwaremill.session.CsrfCheck
810

9-
trait ResourceEndpointsTestKit extends SessionEndpointsRoutes { _: CsrfCheck =>
10-
11-
def resourceServiceEndpoints: ActorSystem[_] => ResourceServiceEndpoints
11+
trait ResourceEndpointsTestKit[Resource <: GenericResource]
12+
extends ResourceEndpoints[Resource]
13+
with SessionEndpointsRoutes { _: SchemaProvider with CsrfCheck =>
1214

1315
override def endpoints: ActorSystem[_] => List[ApiEndpoint] =
1416
system =>
1517
List(
1618
sessionServiceEndpoints(system),
17-
resourceServiceEndpoints(system)
19+
resourceEndpoints(system)
1820
)
1921
}
Lines changed: 13 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,22 @@
11
package app.softnetwork.resource.scalatest
22

33
import akka.actor.typed.ActorSystem
4-
import akka.http.scaladsl.server.Route
5-
import app.softnetwork.resource.service.ResourceService
4+
import app.softnetwork.api.server.ApiRoute
5+
import app.softnetwork.persistence.schema.SchemaProvider
6+
import app.softnetwork.resource.launch.ResourceRoutes
7+
import app.softnetwork.resource.model.GenericResource
68
import app.softnetwork.session.scalatest.SessionServiceRoutes
79
import org.scalatest.Suite
810

9-
trait ResourceRoutesTestKit extends SessionServiceRoutes { _: Suite =>
11+
trait ResourceRoutesTestKit[Resource <: GenericResource]
12+
extends ResourceRoutes[Resource]
13+
with SessionServiceRoutes { _: Suite with SchemaProvider =>
1014

11-
def resourceService: ActorSystem[_] => ResourceService
12-
13-
override def apiRoutes(system: ActorSystem[_]): Route =
14-
sessionServiceRoute(system).route ~ resourceService(system).route
15+
override def apiRoutes: ActorSystem[_] => List[ApiRoute] =
16+
system =>
17+
List(
18+
sessionServiceRoute(system),
19+
resourceService(system)
20+
)
1521

1622
}
Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package app.softnetwork.resource.scalatest
22

33
import akka.actor.typed.ActorSystem
4+
import app.softnetwork.resource.model.Resource
45
import app.softnetwork.resource.service.{
56
LocalFileSystemResourceServiceEndpoints,
67
ResourceServiceEndpoints
@@ -10,10 +11,10 @@ import org.scalatest.Suite
1011

1112
trait ResourceToLocalFileSystemEndpointsTestKit
1213
extends ResourceToLocalFileSystemRouteTestKit
13-
with ResourceEndpointsTestKit {
14+
with ResourceEndpointsTestKit[Resource] {
1415
_: Suite with CsrfCheck =>
1516

16-
def resourceServiceEndpoints: ActorSystem[_] => ResourceServiceEndpoints = system =>
17-
LocalFileSystemResourceServiceEndpoints(system, sessionEndpoints(system))
17+
def resourceEndpoints: ActorSystem[_] => ResourceServiceEndpoints =
18+
system => LocalFileSystemResourceServiceEndpoints(system, sessionEndpoints(system))
1819

1920
}

testkit/src/main/scala/app/softnetwork/resource/scalatest/ResourceToLocalFileSystemRoutesTestKit.scala

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,13 @@
11
package app.softnetwork.resource.scalatest
22

33
import akka.actor.typed.ActorSystem
4+
import app.softnetwork.resource.model.Resource
45
import app.softnetwork.resource.service.{LocalFileSystemResourceService, ResourceService}
56
import org.scalatest.Suite
67

78
trait ResourceToLocalFileSystemRoutesTestKit
89
extends ResourceToLocalFileSystemRouteTestKit
9-
with ResourceRoutesTestKit {
10+
with ResourceRoutesTestKit[Resource] {
1011
_: Suite =>
1112

1213
override def resourceService: ActorSystem[_] => ResourceService = system =>

0 commit comments

Comments
 (0)