Drop Python 2 support (#64)

Python 2 is long obsolete, and cannot be CI tested in any reasonable way. Make official the current defacto situation.

Author: cjwatson

INSTALL.txt

@@ -4,9 +4,7 @@ Installing Test Repository
 Run time dependencies
 ~~~~~~~~~~~~~~~~~~~~~
 
-* Either:
- + Python 2: version 2.7 or newer
- + Python 3: version 3.4 or newer
+* Python 3: version 3.4 or newer
 
 * subunit (0.0.18 or newer).
 

NEWS

@@ -2,6 +2,14 @@
 testrepository release notes
 ############################
 
+0.0.22
+++++++
+
+CHANGES
+-------
+
+* Drop Python 2 support. (Colin Watson)
+
 0.0.21
 ++++++
 

pyproject.toml

@@ -10,7 +10,6 @@ classifiers = [
     "License :: OSI Approved :: Apache Software License",
     "License :: OSI Approved :: BSD License",
     "Operating System :: OS Independent",
-    "Programming Language :: Python :: 2",
     "Programming Language :: Python :: 3",
     "Programming Language :: Python",
     "Topic :: Software Development :: Quality Assurance",

testrepository/arguments/__init__.py

@@ -31,8 +31,6 @@
 
 import sys
 
-from testtools.compat import reraise
-
 
 class AbstractArgument(object):
     """A argument that a command may need.
@@ -101,7 +99,7 @@ def parse(self, argv):
                 break
         if count < self.minimum_count:
             if error is not None:
-                reraise(error[0], error[1], error[2])
+                raise error[1].with_traceback(error[2])
             raise ValueError('not enough arguments present/matched in %s' % argv)
         del argv[:count]
         return result

testrepository/commands/__init__.py

@@ -38,7 +38,6 @@
 import sys
 
 import subunit
-from testtools.compat import _u
 
 from testrepository.repository import file
 
@@ -204,8 +203,8 @@ def get_command_parser(cmd):
     parser = OptionParser()
     for option in cmd.options:
         parser.add_option(option)
-    usage = _u('%%prog %(cmd)s [options] %(args)s\n\n%(help)s') % {
-        'args': _u(' ').join(map(lambda x:x.summary(), cmd.args)),
+    usage = '%%prog %(cmd)s [options] %(args)s\n\n%(help)s' % {
+        'args': ' '.join(map(lambda x:x.summary(), cmd.args)),
         'cmd': getattr(cmd, 'name', cmd),
         'help': getdoc(cmd),
         }

testrepository/commands/list_tests.py

@@ -17,7 +17,6 @@
 from io import BytesIO
 
 from testtools import TestResult
-from testtools.compat import _b
 
 from testrepository.arguments.doubledash import DoubledashArgument
 from testrepository.arguments.string import StringArgument

testrepository/commands/run.py

@@ -24,7 +24,6 @@
 from testtools import (
     TestByTestResult,
     )
-from testtools.compat import _b
 
 from testrepository.arguments.doubledash import DoubledashArgument
 from testrepository.arguments.string import StringArgument
@@ -36,7 +35,7 @@
 from testrepository.testlist import parse_list
 
 
-LINEFEED = _b('\n')[0]
+LINEFEED = b'\n'[0]
 
 
 class ReturnCodeToSubunit(object):
@@ -74,7 +73,7 @@ def _append_return_code_as_test(self):
                 # line. V2 needs to start on any fresh utf8 character border
                 # - which is not guaranteed in an arbitrary stream endpoint, so
                 # injecting a \n gives us such a guarantee.
-                self.source.write(_b('\n'))
+                self.source.write(b'\n')
             stream = subunit.StreamResultToBytes(self.source)
             stream.status(test_id='process-returncode', test_status='fail',
                 file_name='traceback', mime_type='text/plain;charset=utf8',
@@ -84,7 +83,7 @@ def _append_return_code_as_test(self):
 
     def read(self, count=-1):
         if count == 0:
-            return _b('')
+            return b''
         result = self.source.read(count)
         if result:
             self.lastoutput = result[-1]

testrepository/repository/file.py

@@ -28,7 +28,6 @@
 import subunit.v2
 from subunit import TestProtocolClient
 import testtools
-from testtools.compat import _b
 
 from testrepository.repository import (
     AbstractRepository,
@@ -123,7 +122,7 @@ def get_failing(self):
         except IOError:
             err = sys.exc_info()[1]
             if err.errno == errno.ENOENT:
-                run_subunit_content = _b('')
+                run_subunit_content = b''
             else:
                 raise
         return _DiskRun(None, run_subunit_content)

testrepository/testlist.py

@@ -19,22 +19,19 @@
 from subunit import ByteStreamToStreamResult
 from testtools.testresult.doubles import StreamResult
 
-from testtools.compat import _b, _u
-
 
 def write_list(stream, test_ids):
     """Write test_ids out to stream.
 
     :param stream: A file-like object.
     :param test_ids: An iterable of test ids.
     """
-    # May need utf8 explicitly?
-    stream.write(_b('\n'.join(list(test_ids) + [''])))
+    stream.write(('\n'.join(list(test_ids) + [''])).encode('utf8'))
 
 
 def parse_list(list_bytes):
     """Parse list_bytes into a list of test ids."""
-    return [id.strip() for id in list_bytes.decode('utf8').split(_u('\n'))
+    return [id.strip() for id in list_bytes.decode('utf8').split('\n')
             if id.strip()]
 
 

testrepository/tests/commands/test_failing.py

@@ -19,7 +19,6 @@
 
 from subunit.v2 import ByteStreamToStreamResult
 import testtools
-from testtools.compat import _b
 from testtools.matchers import (
     DocTestMatches,
     Equals,
@@ -110,7 +109,7 @@ def test_with_subunit_no_failures_exit_0(self):
         self.assertEqual(0, cmd.execute())
         self.assertEqual(1, len(ui.outputs))
         self.assertEqual('stream', ui.outputs[0][0])
-        self.assertThat(ui.outputs[0][1], Equals(_b('')))
+        self.assertThat(ui.outputs[0][1], Equals(b''))
 
     def test_with_list_shows_list_of_tests(self):
         ui, cmd = self.get_test_ui_and_cmd(options=[('list', True)])