Skip to content

Commit 583986f

Browse files
author
Juliya Smith
authored
Prevent negs (#80)
1 parent 056c2b8 commit 583986f

File tree

2 files changed

+10
-2
lines changed

2 files changed

+10
-2
lines changed

src/code42cli/worker.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,8 @@ def total_errors(self):
3030

3131
@property
3232
def total_successes(self):
33-
return self._total_processed - self._total_errors
33+
val = self._total_processed - self._total_errors
34+
return val if val >= 0 else 0
3435

3536
def __str__(self):
3637
return u"{0} succeeded, {1} failed out of {2}".format(

tests/test_worker.py

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,15 @@
11
import time
22

3-
from code42cli.worker import Worker
3+
from code42cli.worker import Worker, WorkerStats
44

55

6+
class TestWorkerStats(object):
7+
def test_successes_when_should_be_negative_returns_zero(self):
8+
stats = WorkerStats(100)
9+
stats._total_errors = 101
10+
assert not stats.total_successes
11+
12+
613
class TestWorker(object):
714
def test_is_async(self):
815
worker = Worker(5, 2)

0 commit comments

Comments
 (0)