changeset 1150:d6f3b3eec520

Fix race in CliTest reviewed-by: neugens review-thread: http://icedtea.classpath.org/pipermail/thermostat/2013-July/007232.html
author Jon VanAlten <jon.vanalten@redhat.com>
date Mon, 24 Jun 2013 14:15:01 -0600
parents d1f8a091dd68
children 2b3cf603245d
files integration-tests/src/test/java/com/redhat/thermostat/itest/CliTest.java
diffstat 1 files changed, 2 insertions(+), 3 deletions(-) [+]
line wrap: on
line diff
--- a/integration-tests/src/test/java/com/redhat/thermostat/itest/CliTest.java	Fri Jun 28 12:44:41 2013 -0400
+++ b/integration-tests/src/test/java/com/redhat/thermostat/itest/CliTest.java	Mon Jun 24 14:15:01 2013 -0600
@@ -37,13 +37,11 @@
 package com.redhat.thermostat.itest;
 
 import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertTrue;
 import static org.junit.Assert.fail;
 
 import java.io.IOException;
 
-import org.junit.Ignore;
 import org.junit.Test;
 
 import expectj.Spawn;
@@ -196,7 +194,7 @@
     }
 
     @Test
-    public void testUnrecognizedEventsInShell() throws IOException, TimeoutException {
+    public void testUnrecognizedEventsInShell() throws Exception {
         // test '!' events
         Spawn shell = spawnThermostat("shell");
 
@@ -204,6 +202,7 @@
         shell.send("what!?!\n");
         shell.expect(SHELL_PROMPT);
         shell.send("exit\n");
+        shell.expectClose();
 
         assertTrue(shell.getCurrentStandardErrContents().contains("!?!: event not found"));
         assertNoExceptions(shell.getCurrentStandardOutContents(), shell.getCurrentStandardErrContents());