From fdc39a4ef8b3b9c20585f913a0e9fd7c040a5e11 Mon Sep 17 00:00:00 2001 From: Kinsley Wong Date: Tue, 7 Jan 2014 16:34:55 -0800 Subject: [PATCH] Add test-stdout-close-catch and test-stdout-stderr-reading to the list of unit test. Reviewed-by: asquare --- build.xml | 2 ++ .../simple/test-stdout-close-catch.js.patch | 21 +++++++++++++++++++ project.properties | 2 ++ 3 files changed, 25 insertions(+) create mode 100644 patches/test/simple/test-stdout-close-catch.js.patch diff --git a/build.xml b/build.xml index d954638..1345050 100644 --- a/build.xml +++ b/build.xml @@ -315,6 +315,7 @@ + @@ -404,6 +405,7 @@ + diff --git a/patches/test/simple/test-stdout-close-catch.js.patch b/patches/test/simple/test-stdout-close-catch.js.patch new file mode 100644 index 0000000..4f79f72 --- /dev/null +++ b/patches/test/simple/test-stdout-close-catch.js.patch @@ -0,0 +1,21 @@ +--- ../nodejs/test/simple/test-stdout-close-catch.js 2014-01-07 15:32:53.556005400 -0800 ++++ test/simple/test-stdout-close-catch.js 2014-01-07 16:20:47.154233600 -0800 +@@ -28,14 +28,15 @@ + + var testScript = path.join(common.fixturesDir, 'catch-stdout-error.js'); + +-var cmd = JSON.stringify(process.execPath) + ' ' + ++var cmd = process.execPath + ' ' + + JSON.stringify(testScript) + ' | ' + +- JSON.stringify(process.execPath) + ' ' + ++ process.execPath + ' ' + + '-pe "process.exit(1);"'; + + var child = child_process.exec(cmd); + var output = ''; +-var outputExpect = { 'code': 'EPIPE', ++var outputExpect = { 'message' : 'write EPIPE', ++ 'code': 'EPIPE', + 'errno': 'EPIPE', + 'syscall': 'write' }; + diff --git a/project.properties b/project.properties index b8efd7c..8e455d3 100644 --- a/project.properties +++ b/project.properties @@ -521,7 +521,9 @@ source.test.simple.list = \ test-stdin-resume-pause.js \ test-stdio-readable-writable.js \ test-stdout-cannot-be-closed-child-process-pipe.js \ + test-stdout-close-catch.js \ test-stdout-close-unref.js \ + test-stdout-stderr-reading.js \ test-stdout-to-file.js \ test-stream-big-push.js \ test-stream-pipe-after-end.js \