[commit: ghc] master: Testsuite: use ignore_stderr/stdout instead of ignore_output (1084d37)

git at git.haskell.org git at git.haskell.org
Wed Jun 29 16:16:57 UTC 2016


Repository : ssh://git@git.haskell.org/ghc

On branch  : master
Link       : http://ghc.haskell.org/trac/ghc/changeset/1084d3755cac6ccd90f8decc0d79c315387ae388/ghc

>---------------------------------------------------------------

commit 1084d3755cac6ccd90f8decc0d79c315387ae388
Author: Thomas Miedema <thomasmiedema at gmail.com>
Date:   Fri Jun 24 20:45:04 2016 +0200

    Testsuite: use ignore_stderr/stdout instead of ignore_output
    
    The problem with ignore_output is that it hides errors for WAY=ghci.
    GHCi always returns with exit code 0 (unless it is broken itself).
    
    For example: ghci015 must have been failing with compile errors for
    years, but we didn't notice because all output was ignored.
    
    Therefore, replace all uses of ignore_output with either ignore_stderr
    or ignore_stdout. In some cases I opted for adding the expected output.
    
    Update submodule hpc and stm.
    
    Reviewed by: simonmar
    
    Differential Revision: https://phabricator.haskell.org/D2367


>---------------------------------------------------------------

1084d3755cac6ccd90f8decc0d79c315387ae388
 libraries/base/tests/all.T                         |  8 ---
 libraries/base/tests/topHandler03.stderr           |  1 +
 libraries/hpc                                      |  2 +-
 libraries/stm                                      |  2 +-
 testsuite/driver/extra_files.py                    |  1 +
 testsuite/driver/testglobals.py                    |  6 +-
 testsuite/driver/testlib.py                        | 49 +++++++-------
 testsuite/tests/cabal/Makefile                     |  3 +-
 testsuite/tests/cabal/all.T                        | 16 ++++-
 testsuite/tests/cabal/cabal03/Makefile             |  8 +--
 testsuite/tests/cabal/cabal03/all.T                |  2 +-
 testsuite/tests/cabal/cabal03/cabal03.stderr       |  4 ++
 testsuite/tests/cabal/cabal05/all.T                |  2 +-
 testsuite/tests/cabal/cabal05/cabal05.stderr       |  5 ++
 testsuite/tests/cabal/cabal09/all.T                |  2 +-
 testsuite/tests/cabal/ghcpkg04.stderr              |  4 +-
 testsuite/tests/concurrent/should_run/all.T        |  6 +-
 testsuite/tests/dph/enumfromto/dph-enumfromto.T    |  3 +-
 testsuite/tests/driver/Makefile                    |  8 +--
 testsuite/tests/driver/all.T                       |  6 +-
 testsuite/tests/ghc-e/should_fail/Makefile         | 14 ++--
 testsuite/tests/ghc-e/should_fail/T7962.stderr     |  3 +
 .../tests/ghc-e/should_fail/T9905fail1.stderr      |  4 ++
 .../tests/ghc-e/should_fail/T9905fail2.stderr      |  3 +
 .../tests/ghc-e/should_fail/T9905fail3.stderr      |  2 +
 testsuite/tests/ghc-e/should_fail/T9930fail.stderr |  2 +
 testsuite/tests/ghc-e/should_fail/all.T            | 14 ++--
 .../tests/ghc-e/should_fail/ghc-e-fail1.stderr     |  3 +
 .../tests/ghc-e/should_fail/ghc-e-fail2.stderr     |  4 ++
 testsuite/tests/ghci/scripts/all.T                 |  8 ++-
 testsuite/tests/ghci/scripts/ghci015.hs            |  4 +-
 testsuite/tests/hpc/all.T                          |  2 +-
 testsuite/tests/perf/should_run/all.T              |  4 +-
 testsuite/tests/rts/T11223/all.T                   |  2 +-
 .../tests/{ghc-api/T7478/C.hs => rts/T9839_01.hs}  |  0
 testsuite/tests/rts/T9839_01.stderr                |  1 +
 testsuite/tests/rts/T9839_02.stderr                |  1 +
 testsuite/tests/rts/T9839_03.stderr                |  1 +
 testsuite/tests/rts/T9839_05.stderr                |  1 +
 testsuite/tests/rts/T9839_06.stderr                |  1 +
 testsuite/tests/rts/all.T                          | 74 +++++++++++++---------
 41 files changed, 174 insertions(+), 112 deletions(-)

Diff suppressed because of size. To see it, use:

    git diff-tree --root --patch-with-stat --no-color --find-copies-harder --ignore-space-at-eol --cc 1084d3755cac6ccd90f8decc0d79c315387ae388


More information about the ghc-commits mailing list