chore: drop coverage 4.0.x support

Author: TheKevJames

coveralls/api.py

@@ -268,11 +268,7 @@ def get_coverage(self):
         config_file = self.config.get('config_file', True)
         workman = coverage.coverage(config_file=config_file)
         workman.load()
-
-        if hasattr(workman, '_harvest_data'):
-            workman._harvest_data()  # pylint: disable=W0212
-        else:
-            workman.get_data()
+        workman.get_data()
 
         return CoverallReporter(workman, workman.config).coverage
 

coveralls/git.py

@@ -18,10 +18,7 @@ def run_command(*args):
             'command return code {}, STDOUT: "{}"\nSTDERR: "{}"'.format(
                 cmd.returncode, stdout, stderr))
 
-    try:
-        return stdout.decode().strip()
-    except UnicodeDecodeError:
-        return stdout.decode('utf-8').strip()
+    return stdout.decode().strip()
 
 
 def gitlog(fmt):

coveralls/reporter.py

@@ -5,8 +5,6 @@
 from coverage.misc import NoSource
 from coverage.misc import NotPython
 
-from .exception import CoverallsException
-
 
 log = logging.getLogger('coveralls.reporter')
 
@@ -112,19 +110,7 @@ def report(self, cov, conf, morfs=None):
         except ImportError:  # coverage >= 5.0
             return self.report5(cov)
 
-        units = None
-        if hasattr(self.reporter, 'find_code_units'):
-            self.reporter.find_code_units(morfs)
-        else:
-            units = self.reporter.find_file_reporters(morfs)
-
-        if units is None:
-            if hasattr(self.reporter, 'code_units'):
-                units = self.reporter.code_units
-            else:
-                units = self.reporter.file_reporters
-
-        for cu in units:
+        for cu in self.reporter.find_file_reporters(morfs):
             try:
                 _fn = self.reporter.coverage._analyze  # pylint: disable=W0212
                 analyzed = _fn(cu)
@@ -138,17 +124,6 @@ def report(self, cov, conf, morfs=None):
                 if (cu.should_be_python()
                         and not self.reporter.config.ignore_errors):
                     log.warning('Source file is not python %s', cu.filename)
-            except KeyError:
-                version = [int(x) for x in __version__.split('.')]
-                cov40 = version[0] == 4 and version[1] < 1
-                if cov40:
-                    raise CoverallsException(
-                        'Old (<4.1) versions of coverage.py do not work '
-                        'consistently on new versions of Python. Please '
-                        'upgrade your coverage.py.'
-                    )
-
-                raise
 
         return self.coverage
 
@@ -207,27 +182,15 @@ def get_arcs(analysis):
 
     def parse_file(self, cu, analysis):
         """Generate data for single file."""
-        if hasattr(analysis, 'parser'):
-            filename = cu.file_locator.relative_filename(cu.filename)
-            source_lines = analysis.parser.lines
-            with cu.source_file() as source_file:
-                source = source_file.read()
-        else:
-            if hasattr(cu, 'relative_filename'):
-                filename = cu.relative_filename()
-            else:
-                filename = analysis.coverage.file_locator.relative_filename(
-                    cu.filename)
+        filename = cu.relative_filename()
+        # ensure results are properly merged between platforms
+        posix_filename = filename.replace(os.path.sep, '/')
 
-            token_lines = analysis.file_reporter.source_token_lines()
-            source_lines = list(enumerate(token_lines))
-            source = analysis.file_reporter.source()
+        source = analysis.file_reporter.source()
 
+        token_lines = analysis.file_reporter.source_token_lines()
         coverage_lines = [self.get_hits(i, analysis)
-                          for i in range(1, len(source_lines) + 1)]
-
-        # ensure results are properly merged between platforms
-        posix_filename = filename.replace(os.path.sep, '/')
+                          for i, _ in enumerate(token_lines, 1)]
 
         results = {
             'name': posix_filename,

setup.py

@@ -33,7 +33,7 @@
         ],
     },
     install_requires=[
-        'coverage>=4.0,<6.0',
+        'coverage>=4.1,<6.0',
         'docopt>=0.6.1',
         'requests>=1.0.0',
     ],

tests/conftest.py

@@ -0,0 +1,12 @@
+import os
+
+import pytest
+
+
+@pytest.fixture(scope='session', autouse=True)
+def nuke_coverage():
+    for folder in ('.', './example', './nonunicode'):
+        try:
+            os.remove('{}/.coverage'.format(folder))
+        except FileNotFoundError:
+            pass

tox.ini

@@ -1,5 +1,5 @@
 [tox]
-envlist = py{34,35}-cov{4,41}-{default,pyyaml},py{36,37,38,py3}-cov{41,5}-{default,pyyaml}
+envlist = py{34,35}-cov{41}-{default,pyyaml},py{36,37,38,py3}-cov{41,5}-{default,pyyaml}
 
 [gh-actions]
 python =
@@ -15,7 +15,6 @@ deps =
     mock
     pytest
     pyyaml: PyYAML>=3.10,<5.3
-    cov4: coverage>=4.0,<4.1
     cov41: coverage>=4.1,<5.0
     cov5: coverage>=5.0,<6.0
 commands =