From 10e7a466bc936737b81005cb50d90c8bc8ca49bd Mon Sep 17 00:00:00 2001
From: Artur Signell <artur@vaadin.com>
Date: Wed, 24 Jun 2015 23:28:22 +0300
Subject: Pass critical notification details to the client (#18342)

Change-Id: I3c4eace624453eb854a32fef5fe44d253b164f62
---
 .../tests/application/CriticalNotifications.java   | 28 ++++++++++++++++++----
 1 file changed, 23 insertions(+), 5 deletions(-)

(limited to 'uitest/src')

diff --git a/uitest/src/com/vaadin/tests/application/CriticalNotifications.java b/uitest/src/com/vaadin/tests/application/CriticalNotifications.java
index bde3fbaf6f..14b4cb62e9 100644
--- a/uitest/src/com/vaadin/tests/application/CriticalNotifications.java
+++ b/uitest/src/com/vaadin/tests/application/CriticalNotifications.java
@@ -26,16 +26,21 @@ import com.vaadin.tests.components.AbstractTestUI;
 import com.vaadin.ui.Button;
 import com.vaadin.ui.Button.ClickEvent;
 import com.vaadin.ui.Button.ClickListener;
+import com.vaadin.ui.CheckBox;
 
 public class CriticalNotifications extends AbstractTestUI {
 
     private SystemMessages systemMessages;
+    private CheckBox includeDetails;
 
     @Override
     protected void setup(VaadinRequest request) {
         systemMessages = VaadinService.getCurrent().getSystemMessages(
                 getLocale(), request);
 
+        includeDetails = new CheckBox("Include details");
+        addComponent(includeDetails);
+
         Button sessionExpired = new Button("Session expired");
         addComponent(sessionExpired);
         sessionExpired.addClickListener(new ClickListener() {
@@ -43,7 +48,8 @@ public class CriticalNotifications extends AbstractTestUI {
             public void buttonClick(ClickEvent event) {
                 showCriticalNotification(
                         systemMessages.getSessionExpiredCaption(),
-                        systemMessages.getSessionExpiredMessage(), null,
+                        systemMessages.getSessionExpiredMessage(),
+                        getDetailsMessage(),
                         systemMessages.getSessionExpiredURL());
 
             }
@@ -56,7 +62,8 @@ public class CriticalNotifications extends AbstractTestUI {
             public void buttonClick(ClickEvent event) {
                 showCriticalNotification(
                         systemMessages.getAuthenticationErrorCaption(),
-                        systemMessages.getAuthenticationErrorMessage(), null,
+                        systemMessages.getAuthenticationErrorMessage(),
+                        getDetailsMessage(),
                         systemMessages.getAuthenticationErrorURL());
 
             }
@@ -69,7 +76,8 @@ public class CriticalNotifications extends AbstractTestUI {
             public void buttonClick(ClickEvent event) {
                 showCriticalNotification(
                         systemMessages.getCommunicationErrorCaption(),
-                        systemMessages.getCommunicationErrorMessage(), null,
+                        systemMessages.getCommunicationErrorMessage(),
+                        getDetailsMessage(),
                         systemMessages.getCommunicationErrorURL());
 
             }
@@ -82,7 +90,8 @@ public class CriticalNotifications extends AbstractTestUI {
             public void buttonClick(ClickEvent event) {
                 showCriticalNotification(
                         systemMessages.getInternalErrorCaption(),
-                        systemMessages.getInternalErrorMessage(), null,
+                        systemMessages.getInternalErrorMessage(),
+                        getDetailsMessage(),
                         systemMessages.getInternalErrorURL());
 
             }
@@ -95,7 +104,8 @@ public class CriticalNotifications extends AbstractTestUI {
             public void buttonClick(ClickEvent event) {
                 showCriticalNotification(
                         systemMessages.getCookiesDisabledCaption(),
-                        systemMessages.getCookiesDisabledMessage(), null,
+                        systemMessages.getCookiesDisabledMessage(),
+                        getDetailsMessage(),
                         systemMessages.getCookiesDisabledURL());
 
             }
@@ -112,6 +122,14 @@ public class CriticalNotifications extends AbstractTestUI {
         });
     }
 
+    protected String getDetailsMessage() {
+        if (includeDetails.getValue()) {
+            return "Some details for the error";
+        } else {
+            return null;
+        }
+    }
+
     protected void showCriticalNotification(String caption, String message,
             String details, String url) {
         VaadinService service = VaadinService.getCurrent();
-- 
cgit v1.2.3