# HG changeset patch # User Mario Torre # Date 1359467243 -3600 # Node ID bb6bd5efc633dbc6741aca5750779874fbfb947c # Parent d7378ffe4fba28703772a7f30e6522703becf116 Fix error with HeaderPanel review-thread: http://icedtea.classpath.org/pipermail/thermostat/2013-January/005414.html reviewed-by: jerboaa diff -r d7378ffe4fba -r bb6bd5efc633 client/swing/src/main/java/com/redhat/thermostat/client/swing/components/HeaderPanel.java --- 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) { diff -r d7378ffe4fba -r bb6bd5efc633 client/swing/src/test/java/com/redhat/thermostat/client/swing/components/HeaderPanelTest.java --- 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");