From: Bruno Haible <bruno@clisp.org>
Date: Sun, 21 Mar 2010 23:13:13 +0000 (+0100)
Subject: Tests for openpty: Make stricter.
X-Git-Tag: v0.1~4454
X-Git-Url: http://erislabs.org.uk/gitweb/?a=commitdiff_plain;h=4ed661693e3d571b72963ed2e96b88a5feb74bf7;p=gnulib.git

Tests for openpty: Make stricter.
---

diff --git a/ChangeLog b/ChangeLog
index fde459d9b..fc94be5b5 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,11 @@
 2010-03-21  Bruno Haible  <bruno@clisp.org>
 
+	Tests for openpty: Make stricter.
+	* tests/test-openpty.c (main): Add test of canonical processing and
+	erase.
+	* modules/openpty-tests (test_openpty_LDADD): Verify that PTY_LIB is
+	defined.
+
 	openpty: Provide replacement on AIX, HP-UX, IRIX, Solaris.
 	* lib/openpty.c (openpty): New replacement function.
 	* lib/pty.in.h: Include <termios.h>.
diff --git a/modules/openpty-tests b/modules/openpty-tests
index 8b730e68b..8c4d37c13 100644
--- a/modules/openpty-tests
+++ b/modules/openpty-tests
@@ -5,4 +5,4 @@ tests/test-openpty.c
 Makefile.am:
 TESTS += test-openpty
 check_PROGRAMS += test-openpty
-test_openpty_LDADD = $(LDADD) $(PTY_LIB)
+test_openpty_LDADD = $(LDADD) @PTY_LIB@
diff --git a/tests/test-openpty.c b/tests/test-openpty.c
index 2af25b49c..4012e160f 100644
--- a/tests/test-openpty.c
+++ b/tests/test-openpty.c
@@ -14,7 +14,8 @@
    You should have received a copy of the GNU General Public License
    along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
 
-/* Written by Simon Josefsson <simon@josefsson.org>, 2009.  */
+/* Written by Simon Josefsson <simon@josefsson.org>, 2009
+   and Bruno Haible <bruno@clisp.org>, 2010.  */
 
 #include <config.h>
 
@@ -25,20 +26,80 @@ SIGNATURE_CHECK (openpty, int, (int *, int *, char *, struct termios const *,
                                 struct winsize const *));
 
 #include <stdio.h>
+#include <string.h>
+#include <termios.h>
+#include <unistd.h>
 
 int
 main ()
 {
-  int res;
-  int amaster;
-  int aslave;
+  {
+    int master;
+    int slave;
 
-  res = openpty (&amaster, &aslave, NULL, NULL, NULL);
-  if (res != 0)
+    /* Open a pseudo-terminal, as a master-slave pair.  */
     {
-      printf ("openpty returned %d\n", res);
-      return 1;
+      int res = openpty (&master, &slave, NULL, NULL, NULL);
+      if (res != 0)
+        {
+          fprintf (stderr, "openpty returned %d\n", res);
+          return 1;
+        }
     }
 
+    /* Set the terminal characteristics.
+       On Linux or MacOS X, they can be set on either the master or the slave;
+       the effect is the same.  But on Solaris, they have to be set on the
+       master; tcgetattr on the slave fails.  */
+    {
+      int tcfd = slave; /* You can try  tcfd = master;  here.  */
+      struct termios attributes;
+
+      if (tcgetattr (tcfd, &attributes) < 0)
+        {
+          fprintf (stderr, "tcgetattr failed\n");
+          return 1;
+        }
+      /* Enable canonical processing, including erase.  */
+      attributes.c_lflag |= ECHO | ICANON | ECHOE;
+      attributes.c_cc[VERASE] = '\177';
+      if (tcsetattr (tcfd, TCSANOW, &attributes) < 0)
+        {
+          fprintf (stderr, "tcsetattr failed\n");
+          return 1;
+        }
+    }
+
+    /* Write into the master side.  */
+    {
+      static const char input[] = "Hello worst\177\177ld!\n";
+
+      if (write (master, input, strlen (input)) < (int) strlen (input))
+        {
+          fprintf (stderr, "write failed\n");
+          return 1;
+        }
+    }
+
+    /* Read from the slave side.  */
+    {
+      char buf[100];
+      int res = read (slave, buf, sizeof (buf));
+      static const char expected[] = "Hello world!\n";
+
+      if (res < 0)
+        {
+          fprintf (stderr, "read failed\n");
+          return 1;
+        }
+      if (!(res == strlen (expected)
+            && memcmp (buf, expected, strlen (expected)) == 0))
+        {
+          fprintf (stderr, "read result unexpected\n");
+          return 1;
+        }
+    }
+  }
+
   return 0;
 }