Merge lp:~jml/python-fixtures/misc-fixes into lp:~python-fixtures/python-fixtures/trunk

[Jonathan Lange]

 * Work with testtools trunk, post API change requested by fixtures
 * Work with Python 3, provided that https://code.launchpad.net/~jml/testtools/reraise/+merge/69375 has been merged into aforementioned testtools trunk

[Robert Collins (lifeless)]

Cool.

Needs a NEWS entry and a README update about the dependency version,

[Jonathan Lange (jml)]

Done. Should probably wait until https://code.launchpad.net/~jml/testtools/reraise/+merge/69375 is reviewed & merged before landing this.

[lp:~jml/python-fixtures/misc-fixes]

32. By Jonathan Lange on 2011-07-27

NEWS & README.

[Jonathan Lange (jml)]

https://code.launchpad.net/~jml/testtools/reraise/+merge/69375 merged.

=== modified file '.bzrignore'
--- .bzrignore	2010-08-15 09:01:23 +0000
+++ .bzrignore	2011-07-27 00:08:32 +0000
@@ -4,3 +4,4 @@
 MANIFEST
 dist
 .testrepository
+__pycache__

=== modified file 'NEWS'
--- NEWS	2011-06-23 08:21:10 +0000
+++ NEWS	2011-07-27 00:08:32 +0000
@@ -6,6 +6,13 @@
 IN DEVELOPMENT
 ~~~~~~~~~~~~~~
 
+CHANGES:
+
+* Python 3 now supported. (Jonathan Lange, #816665)
+
+* Updated to match the changed ``gather_details`` API in testtools. See #801027.
+  (Jonathan Lange)
+
 0.3.6
 ~~~~~
 

=== modified file 'README'
--- README	2011-06-23 05:54:36 +0000
+++ README	2011-07-27 00:08:32 +0000
@@ -28,7 +28,7 @@
 * Python 2.4+
   This is the base language fixtures is written in and for.
 
-* testtools <https://launchpad.net/testtools> 0.9.8 or newer.
+* testtools <https://launchpad.net/testtools> 0.9.12 or newer.
   testtools provides helpful glue functions for the details API used to report
   information about a fixture (whether its used in a testing or production
   environment).
@@ -99,7 +99,7 @@
   ...     shutil.rmtree(fixture)
   >>> fixture = fixtures.FunctionFixture(setup_function, teardown_function)
   >>> fixture.setUp()
-  >>> print os.path.isdir(fixture.fn_result)
+  >>> print (os.path.isdir(fixture.fn_result))
   True
   >>> fixture.cleanUp()
 
@@ -107,7 +107,7 @@
 
   >>> fixture = fixtures.FunctionFixture(tempfile.mkdtemp, shutil.rmtree)
   >>> fixture.setUp()
-  >>> print os.path.isdir(fixture.fn_result)
+  >>> print (os.path.isdir(fixture.fn_result))
   True
   >>> fixture.cleanUp()
 
@@ -159,14 +159,14 @@
   ...         self.assertEqual(42, fixture.frobnozzle)
   >>> result = unittest.TestResult()
   >>> _ = NoddyTest('test_example').run(result)
-  >>> print result.wasSuccessful()
+  >>> print (result.wasSuccessful())
   True
 
 Fixtures implement the context protocol, so you can also use a fixture as a
 context manager::
 
   >>> with fixtures.FunctionFixture(setup_function, teardown_function) as fixture:
-  ...    print os.path.isdir(fixture.fn_result)
+  ...    print (os.path.isdir(fixture.fn_result))
   True
 
 When multiple cleanups error, fixture.cleanUp() will raise a wrapper exception
@@ -185,8 +185,8 @@
   ...    fixture.cleanUp()
   ... except MultipleExceptions:
   ...    exc_info = sys.exc_info()
-  >>> print exc_info[1].args[0][0]
-  <type 'exceptions.ZeroDivisionError'>
+  >>> print (exc_info[1].args[0][0].__name__)
+  ZeroDivisionError
 
 Shared Dependencies
 +++++++++++++++++++
@@ -298,8 +298,8 @@
 Pretend to run an external command rather than needing it to be present to run
 tests.
 
-  >>> from StringIO import StringIO
-  >>> fixture = fixtures.PopenFixture(lambda _:{'stdout': StringIO('foobar')})
+  >>> from testtools.compat import BytesIO
+  >>> fixture = fixtures.PopenFixture(lambda _:{'stdout': BytesIO('foobar')})
 
 PythonPackage
 +++++++++++++

=== modified file 'lib/fixtures/_fixtures/pythonpackage.py'
--- lib/fixtures/_fixtures/pythonpackage.py	2010-11-07 01:48:22 +0000
+++ lib/fixtures/_fixtures/pythonpackage.py	2011-07-27 00:08:32 +0000
@@ -55,7 +55,7 @@
         os.mkdir(root)
         init_seen = not self.init
         for modulename, contents in self.modulelist:
-            stream = file(os.path.join(root, modulename), 'wb')
+            stream = open(os.path.join(root, modulename), 'wb')
             try:
                 stream.write(contents)
             finally:
@@ -63,4 +63,4 @@
             if modulename == '__init__.py':
                 init_seen = True
         if not init_seen:
-            file(os.path.join(root, '__init__.py'), 'wb').close()
+            open(os.path.join(root, '__init__.py'), 'wb').close()

=== modified file 'lib/fixtures/fixture.py'
--- lib/fixtures/fixture.py	2011-06-23 07:56:42 +0000
+++ lib/fixtures/fixture.py	2011-07-27 00:08:32 +0000
@@ -23,6 +23,7 @@
 import itertools
 import sys
 
+from testtools.compat import reraise
 from testtools.helpers import try_import
 
 class MultipleExceptions(Exception):
@@ -115,7 +116,7 @@
         if result and raise_first:
             if 1 == len(result):
                 error = result[0]
-                raise error[0], error[1], error[2]
+                reraise(error[0], error[1], error[2])
             else:
                 raise MultipleExceptions(*result)
         if not raise_first:
@@ -199,7 +200,7 @@
             # The child failed to come up, capture any details it has (copying
             # the content, it may go away anytime).
             if gather_details is not None:
-                gather_details(fixture, self)
+                gather_details(fixture.getDetails(), self._details)
             raise
         else:
             self.addCleanup(fixture.cleanUp)

=== modified file 'lib/fixtures/testcase.py'
--- lib/fixtures/testcase.py	2011-06-23 07:56:42 +0000
+++ lib/fixtures/testcase.py	2011-07-27 00:08:32 +0000
@@ -45,7 +45,7 @@
         except:
             if use_details:
                 # Capture the details now, in case the fixture goes away.
-                gather_details(fixture, self)
+                gather_details(fixture.getDetails(), self.getDetails())
             raise
         else:
             self.addCleanup(fixture.cleanUp)

=== modified file 'lib/fixtures/tests/_fixtures/test_popen.py'
--- lib/fixtures/tests/_fixtures/test_popen.py	2010-10-15 02:41:18 +0000
+++ lib/fixtures/tests/_fixtures/test_popen.py	2011-07-27 00:08:32 +0000
@@ -13,10 +13,13 @@
 # license you chose for the specific language governing permissions and
 # limitations under that license.
 
-import StringIO
 import subprocess
 
 import testtools
+from testtools.compat import (
+    _b,
+    BytesIO,
+    )
 
 import fixtures
 from fixtures import PopenFixture, TestWithFixtures
@@ -63,6 +66,6 @@
         self.assertEqual(0, proc.returncode)
 
     def test_communicate_with_out(self):
-        proc = FakeProcess({}, {'stdout': StringIO.StringIO('foo')})
-        self.assertEqual(('foo', ''), proc.communicate())
+        proc = FakeProcess({}, {'stdout': BytesIO(_b('foo'))})
+        self.assertEqual((_b('foo'), ''), proc.communicate())
         self.assertEqual(0, proc.returncode)

=== modified file 'lib/fixtures/tests/_fixtures/test_pythonpackage.py'
--- lib/fixtures/tests/_fixtures/test_pythonpackage.py	2010-11-07 01:48:22 +0000
+++ lib/fixtures/tests/_fixtures/test_pythonpackage.py	2011-07-27 00:08:32 +0000
@@ -16,6 +16,7 @@
 import os.path
 
 import testtools
+from testtools.compat import _b
 
 import fixtures
 from fixtures import PythonPackage, TempDir, TestWithFixtures
@@ -32,18 +33,18 @@
             fixture.cleanUp()
    
     def test_writes_package(self):
-        fixture = PythonPackage('foo', [('bar.py', 'woo')])
+        fixture = PythonPackage('foo', [('bar.py', _b('woo'))])
         fixture.setUp()
         try:
-            self.assertEqual('', file(os.path.join(fixture.base, 'foo',
+            self.assertEqual('', open(os.path.join(fixture.base, 'foo',
                 '__init__.py')).read())
-            self.assertEqual('woo', file(os.path.join(fixture.base, 'foo',
+            self.assertEqual('woo', open(os.path.join(fixture.base, 'foo',
                 'bar.py')).read())
         finally:
             fixture.cleanUp()
 
     def test_no__init__(self):
-        fixture = PythonPackage('foo', [('bar.py', 'woo')], init=False)
+        fixture = PythonPackage('foo', [('bar.py', _b('woo'))], init=False)
         fixture.setUp()
         try:
             self.assertFalse(os.path.exists(os.path.join(fixture.base, 'foo',