Return only the latest four Jenkins builds (closes #15).

textbook · Dec 13, 2016 · d8da547 · d8da547
1 parent f732577
commit d8da547
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/flash_services/__init__.py b/flash_services/__init__.py
@@ -15,7 +15,7 @@
 from .travis import TravisOS, TravisPro
 
 __author__ = 'Jonathan Sharpe'
-__version__ = '0.5.0'
+__version__ = '0.5.1'
 
 blueprint = Blueprint(
     'services',

diff --git a/flash_services/jenkins.py b/flash_services/jenkins.py
@@ -73,7 +73,7 @@ def format_data(cls, data):
         builds = [cls.format_build(build) for build in data['builds']]
         estimate_time(builds)
         return dict(
-            builds=builds,
+            builds=builds[:4],
             health=health_summary(builds),
             name=data['name'],
         )

diff --git a/tests/test_jenkins_service.py b/tests/test_jenkins_service.py
@@ -150,10 +150,13 @@ def test_estimated_formatting(_, __, service):
         dict(duration=10000, description=None, timestamp=1481387964313, result='SUCCESS'),
         dict(duration=10000, description=None, timestamp=1481387964313, result='SUCCESS'),
         dict(duration=10000, description=None, timestamp=1481387964313, result='SUCCESS'),
+        dict(duration=10000, description=None, timestamp=1481387964313, result='SUCCESS'),
+        dict(duration=10000, description=None, timestamp=1481387964313, result='SUCCESS'),
     ])
 
     result = service.format_data(response)
 
+    assert len(result['builds']) == 4
     assert result['builds'][0] == dict(
         author='<no author>',
         duration=5,