ID-808 Implement PUT /api/termsOfService/v1/user/self/accept

broadinstitute · Nov 9, 2023 · f226999 · f226999
1 parent e08c499
commit f226999
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 7 deletions.
diff --git a/src/main/scala/org/broadinstitute/dsde/workbench/sam/api/TermsOfServiceRoutes.scala b/src/main/scala/org/broadinstitute/dsde/workbench/sam/api/TermsOfServiceRoutes.scala
@@ -80,13 +80,9 @@ trait TermsOfServiceRoutes extends SamUserDirectives {
                 complete {
                   tosService.getTermsOfServiceDetailsForUser(samUser.id, samRequestContext)
                 }
-              }
-            } ~
-            pathPrefix("accept") { // api/termsOfService/v1/user/accept
-              pathEndOrSingleSlash {
-                put {
-                  complete(StatusCodes.NotImplemented)
-                }
+              } ~
+              put {
+                complete(tosService.acceptTosStatus(samUser.id, samRequestContext).map(_ => StatusCodes.NoContent))
               }
             } ~
             pathPrefix("reject") { // api/termsOfService/v1/user/reject

diff --git a/src/test/scala/org/broadinstitute/dsde/workbench/sam/api/TermsOfServiceRouteSpec.scala b/src/test/scala/org/broadinstitute/dsde/workbench/sam/api/TermsOfServiceRouteSpec.scala
@@ -152,4 +152,14 @@ class TermsOfServiceRouteSpec extends AnyFunSpec with Matchers with ScalatestRou
     }
   }
 
+  it("should return 200 when tos accepted") {
+    val samRoutes = TestSamRoutes(Map.empty)
+    eventually {
+      Put("/api/termsOfService/v1/user/self") ~> samRoutes.route ~> check {
+        status shouldEqual StatusCodes.NoContent
+        responseAs[String] shouldBe ""
+      }
+    }
+  }
+
 }