diff options
author | Zach Smith <zach@driver.xyz> | 2018-02-22 17:03:26 -0800 |
---|---|---|
committer | Zach Smith <zach@driver.xyz> | 2018-02-22 17:41:14 -0800 |
commit | 5d6ef236eb3b5a4fad6383b87e1572685ddf2bc5 (patch) | |
tree | 6b0254ea1d5430415412a3be92696f21bb4a4670 /src/test/scala/xyz/driver/core/rest/DriverRouteTest.scala | |
parent | 82b0ce517eb1076f72199a9c3888bbccf2692c01 (diff) | |
download | driver-core-5d6ef236eb3b5a4fad6383b87e1572685ddf2bc5.tar.gz driver-core-5d6ef236eb3b5a4fad6383b87e1572685ddf2bc5.tar.bz2 driver-core-5d6ef236eb3b5a4fad6383b87e1572685ddf2bc5.zip |
Move cors directives to DriverApp and get rid of custom rejection handler
Diffstat (limited to 'src/test/scala/xyz/driver/core/rest/DriverRouteTest.scala')
-rw-r--r-- | src/test/scala/xyz/driver/core/rest/DriverRouteTest.scala | 47 |
1 files changed, 1 insertions, 46 deletions
diff --git a/src/test/scala/xyz/driver/core/rest/DriverRouteTest.scala b/src/test/scala/xyz/driver/core/rest/DriverRouteTest.scala index 60056b7..ce3daa8 100644 --- a/src/test/scala/xyz/driver/core/rest/DriverRouteTest.scala +++ b/src/test/scala/xyz/driver/core/rest/DriverRouteTest.scala @@ -1,7 +1,6 @@ package xyz.driver.core.rest -import akka.http.scaladsl.model.{HttpMethod, StatusCodes} -import akka.http.scaladsl.model.headers._ +import akka.http.scaladsl.model.StatusCodes import akka.http.scaladsl.server.Directives.{complete => akkaComplete} import akka.http.scaladsl.server.{Directives, Route} import akka.http.scaladsl.testkit.ScalatestRouteTest @@ -27,12 +26,6 @@ class DriverRouteTest extends AsyncFlatSpec with ScalatestRouteTest with Matcher """.stripMargin) } - val allowedOrigins = Set(HttpOrigin("https", Host("example.com"))) - val allowedMethods: collection.immutable.Seq[HttpMethod] = { - import akka.http.scaladsl.model.HttpMethods._ - collection.immutable.Seq(GET, PUT, POST, PATCH, DELETE, OPTIONS) - } - "DriverRoute" should "respond with 200 OK for a basic route" in { val route = new TestRoute(akkaComplete(StatusCodes.OK)) @@ -103,42 +96,4 @@ class DriverRouteTest extends AsyncFlatSpec with ScalatestRouteTest with Matcher responseAs[String] shouldBe "Database access error" } } - - it should "respond with the correct CORS headers for the swagger OPTIONS route" in { - val route = new TestRoute(get(akkaComplete(StatusCodes.OK))) - Options(s"/api-docs/swagger.json") ~> route.routeWithDefaults ~> check { - status shouldBe StatusCodes.OK - headers should contain(`Access-Control-Allow-Origin`(HttpOriginRange(allowedOrigins.toSeq: _*))) - header[`Access-Control-Allow-Methods`].get.methods should contain theSameElementsAs allowedMethods - } - } - - it should "respond with the correct CORS headers for the test route" in { - val route = new TestRoute(get(akkaComplete(StatusCodes.OK))) - Options(s"/api/v1/test") ~> route.routeWithDefaults ~> check { - status shouldBe StatusCodes.OK - headers should contain(`Access-Control-Allow-Origin`(HttpOriginRange(allowedOrigins.toSeq: _*))) - header[`Access-Control-Allow-Methods`].get.methods should contain theSameElementsAs allowedMethods - } - } - - it should "allow subdomains of allowed origin suffixes" in { - val route = new TestRoute(get(akkaComplete(StatusCodes.OK))) - Options(s"/api/v1/test") - .withHeaders(Origin(HttpOrigin("https", Host("foo.example.com")))) ~> route.routeWithDefaults ~> check { - status shouldBe StatusCodes.OK - headers should contain(`Access-Control-Allow-Origin`(HttpOrigin("https", Host("foo.example.com")))) - header[`Access-Control-Allow-Methods`].get.methods should contain theSameElementsAs allowedMethods - } - } - - it should "respond with default domains for invalid origins" in { - val route = new TestRoute(get(akkaComplete(StatusCodes.OK))) - Options(s"/api/v1/test") - .withHeaders(Origin(HttpOrigin("https", Host("invalid.foo.bar.com")))) ~> route.routeWithDefaults ~> check { - status shouldBe StatusCodes.OK - headers should contain(`Access-Control-Allow-Origin`(HttpOriginRange(allowedOrigins.toSeq: _*))) - header[`Access-Control-Allow-Methods`].get.methods should contain theSameElementsAs allowedMethods - } - } } |