From a404b2b6af86df967f31022f6b2a771b82a49b04 2013-09-23 21:53:41
From: Thomas Kluyver <takowl@gmail.com>
Date: 2013-09-23 21:53:41
Subject: [PATCH] Add test for exit status from system_piped

---

diff --git a/IPython/core/tests/test_interactiveshell.py b/IPython/core/tests/test_interactiveshell.py
index 954f60c..a9764d8 100644
--- a/IPython/core/tests/test_interactiveshell.py
+++ b/IPython/core/tests/test_interactiveshell.py
@@ -33,7 +33,7 @@ from StringIO import StringIO
 import nose.tools as nt
 
 # Our own
-from IPython.testing.decorators import skipif, onlyif_unicode_paths
+from IPython.testing.decorators import skipif, skip_win32, onlyif_unicode_paths
 from IPython.testing import tools as tt
 from IPython.utils import io
 
@@ -439,6 +439,13 @@ class TestSystemRaw(unittest.TestCase):
         ip.system_raw('exit 1')
         self.assertEqual(ip.user_ns['_exit_code'], 1)
 
+class TestSystemPiped(unittest.TestCase):
+    # TODO: Exit codes are currently ignored on Windows.
+    @skip_win32
+    def test_exit_code(self):
+        ip.system_piped('exit 1')
+        self.assertEqual(ip.user_ns['_exit_code'], 1)
+
 class TestModules(unittest.TestCase, tt.TempFileMixin):
     def test_extraneous_loads(self):
         """Test we're not loading modules on startup that we shouldn't.