[src/trunk]: src/tests/lib/libcurses/director tests/libcurses: actually compa...

[rillig <rillig%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/81a31530c24f
branches:  trunk
changeset: 951922:81a31530c24f
user:      rillig <rillig%NetBSD.org@localhost>
date:      Sun Feb 07 17:50:16 2021 +0000

description:
tests/libcurses: actually compare the stream to an absolute path

Up to now, the test command "compare /dev/null" was a no-op since the
command was only parsed but not run at all.  Now run it.

This makes the test mvwin fail.  That test will have to be fixed.

Comparing to /dev/null is certainly possible and may make sense,
comparing to /dev/zero is nonsense since the actual stream can never be
endless.  Some tests do that nevertheless, for whatever reason.

In order to have the expected test output closer to the curses commands
that cause it, it may be a good idea to add another command
'compare_str' that would work independently of an external file and at
the same time allow the expected output to be commented and explained.
This is not possible right now since the .chk files are read exactly
as-is.

diffstat:

 tests/lib/libcurses/director/testlang_parse.y |  13 +++++++------
 1 files changed, 7 insertions(+), 6 deletions(-)

diffs (40 lines):

diff -r dad7b1194fe8 -r 81a31530c24f tests/lib/libcurses/director/testlang_parse.y
--- a/tests/lib/libcurses/director/testlang_parse.y     Sun Feb 07 17:32:55 2021 +0000
+++ b/tests/lib/libcurses/director/testlang_parse.y     Sun Feb 07 17:50:16 2021 +0000
@@ -1,5 +1,5 @@
 %{
-/*     $NetBSD: testlang_parse.y,v 1.27 2021/02/07 17:32:55 rillig Exp $       */
+/*     $NetBSD: testlang_parse.y,v 1.28 2021/02/07 17:50:16 rillig Exp $       */
 
 /*-
  * Copyright 2009 Brett Lymn <blymn%NetBSD.org@localhost>
@@ -431,15 +431,17 @@
        no_input = true;
 }
 
-compare                : COMPARE PATH
-                       /* FIXME: missing action */
+compare                : COMPARE PATH {
+                       compare_streams($2, true);
+               }
                | COMPARE FILENAME {
                        compare_streams($2, true);
                }
                ;
 
-comparend      : COMPAREND PATH
-                       /* FIXME: missing action */
+comparend      : COMPAREND PATH {
+                       compare_streams($2, false);
+               }
                | COMPAREND FILENAME {
                        compare_streams($2, false);
                }
@@ -544,7 +546,6 @@
                }
                ;
 
-               /* TODO: split into 'arg' */
 args           : /* empty */
                | arg args
                ;