Skip to content

Commit f66acf0

Browse files
committed
[fix] reports with non-ascii chars close codecov#56
1 parent bfe5d50 commit f66acf0

File tree

3 files changed

+7
-4
lines changed

3 files changed

+7
-4
lines changed

codecov/__init__.py

+1-4
Original file line numberDiff line numberDiff line change
@@ -583,10 +583,7 @@ def main(*argv, **kwargs):
583583
write(" --> Found %s adjustments" % (adjustments.count('\n') - adjustments.count('\n\n') - 1))
584584
reports = str(reports) + '\n# path=fixes\n' + str(adjustments) + '<<<<<< EOF'
585585

586-
try:
587-
reports.encode('ascii', 'replace')
588-
except:
589-
reports = remove_ascii('', reports)
586+
reports = remove_ascii('', reports)
590587

591588
result = ''
592589
if codecov.dump:

tests/non-ascii-report.txt

+1
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
alt-j∆apple

tests/test.py

+5
Original file line numberDiff line numberDiff line change
@@ -334,6 +334,11 @@ def test_not_jacoco(self):
334334
self.assertEqual(res[0], '# path=tests/coverage.xml')
335335
self.assertEqual(res[1], '<data>')
336336

337+
def test_non_ascii(self):
338+
res = self.run_cli(file='tests/non-ascii-report.txt', **self.defaults)
339+
res = res['reports'].split('<<<<<< network\n')[1].splitlines()
340+
self.assertEqual(res[1], 'alt-japple')
341+
337342
def test_run_coverage(self):
338343
self.skipTest('Not sure how to pull off atm')
339344
with open(self.coverage, 'w+') as f:

0 commit comments

Comments
 (0)