Merge pull request #8 from hmrc/API-1445

API-1445 - Signed out page for api revocation

Author: haroonzone

app/config/frontendAppConfig.scala

@@ -45,5 +45,5 @@ object FrontendAppConfig extends AppConfig with ServicesConfig {
   override lazy val reportAProblemPartialUrl = s"$contactHost/contact/problem_reports_ajax?service=$contactFormServiceIdentifier"
   override lazy val reportAProblemNonJSUrl = s"$contactHost/contact/problem_reports_nonjs?service=$contactFormServiceIdentifier"
   override lazy val signInUrl = s"$caFrontendHost/gg/sign-in?continue=$loginCallbackBaseUrl/applications-permissions-withdrawal/applications"
-  override lazy val signOutUrl = s"$caFrontendHost/gg/sign-out"
+  override lazy val signOutUrl = s"$caFrontendHost/gg/sign-out?continue=$loginCallbackBaseUrl/applications-permissions-withdrawal/loggedout"
 }

app/controllers/Revocation.scala

@@ -36,6 +36,10 @@ trait Revocation extends FrontendController with Authentication {
     Future.successful(Ok(views.html.revocation.start()))
   }
 
+  val loggedOut = Action.async { implicit request =>
+    Future.successful(Ok(views.html.revocation.loggedOut()))
+  }
+
   val listAuthorizedApplications = authenticated.async { implicit user => implicit request =>
     delegatedAuthorityConnector.fetchApplicationAuthorities()
       .map(applications => Ok(views.html.revocation.authorizedApplications(applications)))

app/views/revocation/loggedOut.scala.html

@@ -0,0 +1,25 @@
+@()(implicit request: Request[_])
+
+@body = {
+
+    <div class="content__body">
+
+        <header>
+            <h1>You’re signed out</h1>
+        </header>
+        <ul class="action-list">
+            <ul>
+                <li>
+                    <a href="@controllers.routes.Revocation.listAuthorizedApplications" target="_self" data-sso="false">Sign in to your account</a>
+                </li>
+                <li>
+                    <a href="https://www.gov.uk" target="_blank" data-sso="false" rel="external">Explore GOV.UK<span class="visuallyhidden">link opens in a new window</span></a>
+                </li>
+            </ul>
+        </ul>
+    </div>
+}
+
+@main_template(title = "Signed Out", headerNavLinks = None) {
+    @body
+}

conf/app.routes

@@ -4,4 +4,5 @@ GET        /                                           controllers.Revocation.st
 GET        /applications                               controllers.Revocation.listAuthorizedApplications
 GET        /application/:id/withdraw-permission        controllers.Revocation.withdrawPage(id: java.util.UUID)
 POST       /application/:id/withdraw-permission        controllers.Revocation.withdrawAction(id: java.util.UUID)
-GET        /application/:id/permission-withdrawn       controllers.Revocation.withdrawConfirmationPage(id: java.util.UUID)
+GET        /application/:id/permission-withdrawn       controllers.Revocation.withdrawConfirmationPage(id: java.util.UUID)
+GET        /loggedout                                  controllers.Revocation.loggedOut

conf/application.conf

@@ -31,9 +31,6 @@ microservice {
         }
     }
 
-    #dev only
-    contact-frontend.host = "http://localhost:9250"
-
     services {
         auth {
           host = localhost

test/unit/controllers/RevocationSpec.scala

@@ -61,6 +61,15 @@ class RevocationSpec extends UnitSpec with WithFakeApplication with MockitoSugar
     }
   }
 
+  "Logged Out" should {
+    "return 200" in {
+
+      val result = underTest.loggedOut(loggedOutRequest)
+
+      status(result) shouldBe Status.OK
+    }
+  }
+
   "listAuthorizedApplications" should {
     "return 200 when the user is logged in" in {