changeset 941:bb6bd5efc633

Fix error with HeaderPanel review-thread: http://icedtea.classpath.org/pipermail/thermostat/2013-January/005414.html reviewed-by: jerboaa
author Mario Torre <neugens.limasoftware@gmail.com>
date Tue, 29 Jan 2013 14:47:23 +0100
parents d7378ffe4fba
children 7e5cbd159365
files client/swing/src/main/java/com/redhat/thermostat/client/swing/components/HeaderPanel.java client/swing/src/test/java/com/redhat/thermostat/client/swing/components/HeaderPanelTest.java
diffstat 2 files changed, 39 insertions(+), 3 deletions(-) [+]
line wrap: on
line diff
--- a/client/swing/src/main/java/com/redhat/thermostat/client/swing/components/HeaderPanel.java	Mon Jan 28 18:51:22 2013 +0100
+++ b/client/swing/src/main/java/com/redhat/thermostat/client/swing/components/HeaderPanel.java	Tue Jan 29 14:47:23 2013 +0100
@@ -87,7 +87,7 @@
     }
     
     public HeaderPanel(String header) {
-        this(Preferences.userRoot().node(HeaderPanel.class.getName()), "");
+        this(Preferences.userRoot().node(HeaderPanel.class.getName()), header);
     }
     
     public HeaderPanel(Preferences prefs, String header) {
--- a/client/swing/src/test/java/com/redhat/thermostat/client/swing/components/HeaderPanelTest.java	Mon Jan 28 18:51:22 2013 +0100
+++ b/client/swing/src/test/java/com/redhat/thermostat/client/swing/components/HeaderPanelTest.java	Tue Jan 29 14:47:23 2013 +0100
@@ -52,6 +52,7 @@
 import org.fest.swing.edt.GuiTask;
 import org.fest.swing.fixture.FrameFixture;
 import org.fest.swing.fixture.JPanelFixture;
+
 import org.junit.After;
 import org.junit.Before;
 import org.junit.BeforeClass;
@@ -60,6 +61,7 @@
 
 import static org.junit.Assert.assertTrue;
 import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertEquals;
 
 @RunWith(CacioFESTRunner.class)
 public class HeaderPanelTest {
@@ -140,17 +142,51 @@
     @Test
     public void testContentAdded() {
         frameFixture.show();
+        final String[] results = new String[2];
         
+        GuiActionRunner.execute(new GuiTask() {
+            @Override
+            protected void executeInEDT() throws Throwable {
+                results[0] = header.getHeader();
+                header.setHeader("fluff");
+                results[1] = header.getHeader();
+            }
+        });
+        
+        assertEquals("Test Panel", results[0]);
+        assertEquals("fluff", results[1]);
+        
+        // do it again, with a new header
+        GuiActionRunner.execute(new GuiTask() {
+            @Override
+            protected void executeInEDT() throws Throwable {
+                
+                HeaderPanel header = new HeaderPanel("Test");
+                
+                results[0] = header.getHeader();
+                header.setHeader("fluff");
+                results[1] = header.getHeader();
+            }
+        });
+        
+        assertEquals("Test", results[0]);
+        assertEquals("fluff", results[1]);
+    }
+    
+    @Test
+    public void testHeaderHasText() throws InterruptedException {
+        frameFixture.show();
+
         JPanelFixture contentPanel = frameFixture.panel("contentPanel");
         contentPanel.requireVisible();
-        
+
         frameFixture.button("button1").requireVisible();
         frameFixture.button("button2").requireVisible();
         frameFixture.toggleButton("toggle1").requireVisible();
     }
     
     @Test
-    public void testshowtoolbarText() throws InterruptedException {
+    public void testShowToolbarText() throws InterruptedException {
         frameFixture.show();
         
         JPanelFixture headerPanel = frameFixture.panel("clickableArea");