public inbox for frysk-cvs@sourceware.org
help / color / mirror / Atom feed
From: cagney@sourceware.org
To: frysk-cvs@sourceware.org
Subject: [SCM]  master: Cleanup TestFhd.java.
Date: Fri, 25 Jan 2008 00:02:00 -0000	[thread overview]
Message-ID: <20080125000232.17811.qmail@sourceware.org> (raw)

The branch, master has been updated
       via  8611edff5032e4510277be420ceeb7f8ea496c47 (commit)
      from  2e243654a18792a02b13c4ffa90261c5aa1867d7 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email.

- Log -----------------------------------------------------------------
commit 8611edff5032e4510277be420ceeb7f8ea496c47
Author: Andrew Cagney <cagney@redhat.com>
Date:   Thu Jan 24 19:01:55 2008 -0500

    Cleanup TestFhd.java.
    
    frysk-core/frysk/bindir/ChangeLog
    2008-01-24  Andrew Cagney  <cagney@redhat.com>
    
    	* TestFhd.java: Do not call Expect.close(); do not hard-wire a
    	timeout.

-----------------------------------------------------------------------

Summary of changes:
 frysk-core/frysk/bindir/ChangeLog    |    5 +++++
 frysk-core/frysk/bindir/TestFhd.java |   33 +++++++++++++++------------------
 2 files changed, 20 insertions(+), 18 deletions(-)

First 500 lines of diff:
diff --git a/frysk-core/frysk/bindir/ChangeLog b/frysk-core/frysk/bindir/ChangeLog
index 426ffe9..179ded3 100644
--- a/frysk-core/frysk/bindir/ChangeLog
+++ b/frysk-core/frysk/bindir/ChangeLog
@@ -1,3 +1,8 @@
+2008-01-24  Andrew Cagney  <cagney@redhat.com>
+
+	* TestFhd.java: Do not call Expect.close(); do not hard-wire a
+	timeout.
+
 2008-01-24  Stan Cox  <scox@redhat.com>
 	
 	* fhpd.java (main): Pass sysroot to request via -sysroot option.
diff --git a/frysk-core/frysk/bindir/TestFhd.java b/frysk-core/frysk/bindir/TestFhd.java
index 0749af7..1b02a95 100755
--- a/frysk-core/frysk/bindir/TestFhd.java
+++ b/frysk-core/frysk/bindir/TestFhd.java
@@ -50,35 +50,32 @@ import frysk.expunit.Expect;
  */
 
 public class TestFhd extends TestLib {
-    String prompt = "\\(fhpd\\) ";
-  
+    private final String prompt = "\\(fhpd\\) ";
+
     public void testHpdPid () {
 	child = new Expect(Config.getPkgLibFile("hpd-c"));
 	e = new Expect(new String[] { 
-			   Config.getBinFile("fhpd").getPath(), 
-			   child.getPid().toString() 
-		       });
-	e.expect(5, "Attached to process.*\n" + prompt);
-	e.close();
+		Config.getBinFile("fhpd").getPath(), 
+		child.getPid().toString() 
+	    });
+	e.expect("Attached to process.*\n" + prompt);
     }
   
     public void testHpdCommand () {
 	e = new Expect(new String[] { 
-			   Config.getBinFile("fhpd").getPath(), 
-			   Config.getPkgLibFile("hpd-c").getPath() 
-		       });
-	e.expect(5, "Loaded executable file.*" + prompt);
-	e.close();
+		Config.getBinFile("fhpd").getPath(), 
+		Config.getPkgLibFile("hpd-c").getPath() 
+	    });
+	e.expect("Loaded executable file.*" + prompt);
     }
   
     public void testHpdCore ()  {
 	e = new Expect(new String[] {
-			   Config.getBinFile("fhpd").getPath(),
-			   Config.getPkgDataFile("test-core-x86").getPath(),
-			   "-noexe"
-		       });
-	e.expect(5, "Attached to core file.*");
-	e.close();
+		Config.getBinFile("fhpd").getPath(),
+		Config.getPkgDataFile("test-core-x86").getPath(),
+		"-noexe"
+	    });
+	e.expect("Attached to core file.*");
     }
     
 }


hooks/post-receive
--
frysk system monitor/debugger


                 reply	other threads:[~2008-01-25  0:02 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20080125000232.17811.qmail@sourceware.org \
    --to=cagney@sourceware.org \
    --cc=frysk-cvs@sourceware.org \
    --cc=frysk@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).