Merge pull request not-kennethreitz#36 from adammck/fix_connected_com…

…mand_status_code

Fix ConnectedCommand.status_code

envoy/core.py

@@ -100,6 +100,7 @@ def __init__(self,
         self.std_in = std_in
         self.std_out = std_out
         self.std_err = std_out
+        self._status_code = None
 
     def __enter__(self):
         return self
@@ -112,11 +113,7 @@ def status_code(self):
         """The status code of the process.
         If the code is None, assume that it's still running.
         """
-        if self._status_code is not None:
-            return self._status_code
-
-        # investigate
-        return None
+        return self._status_code
 
     @property
     def pid(self):

test_envoy.py

@@ -24,5 +24,11 @@ def test_quoted_args(self):
         self.assertEqual(r.std_out.rstrip(), sentinel)
         self.assertEqual(r.status_code, 0)
 
+class ConnectedCommandTests(unittest.TestCase):
+
+    def test_status_code(self):
+        c = envoy.connect("sleep 5")
+        self.assertEqual(c.status_code, None)
+
 if __name__ == "__main__":
     unittest.main()