add Build.push test coverage

Author: Gabriel Monroy

api/models.py

@@ -459,8 +459,9 @@ def publish(self):
         return data
 
     def converge(self):
-        self.publish()
+        databag = self.publish()
         self.formation.converge()
+        return databag
 
     def calculate(self):
         """Return a representation for configuration management"""
@@ -709,8 +710,8 @@ def push(cls, push):
            len(app.formation.container_set.filter(type='web')) < 1:
             # scale an initial web containers
             Container.objects.scale(app, {'web': 1})
-        # publish the app, triggering a formation converge
-        return app.publish()
+        # publish and converge the application
+        return app.converge()
 
 
 @python_2_unicode_compatible

api/tests/build.py

@@ -11,6 +11,8 @@
 
 from django.test import TestCase
 
+from api.models import Build
+
 # pylint: disable=R0904
 
 
@@ -93,3 +95,14 @@ def test_build(self):
         self.assertEqual(self.client.put(url).status_code, 405)
         self.assertEqual(self.client.patch(url).status_code, 405)
         self.assertEqual(self.client.delete(url).status_code, 405)
+
+    def test_build_push(self):
+        url = '/api/apps'
+        body = {'formation': 'autotest'}
+        response = self.client.post(url, json.dumps(body), content_type='application/json')
+        self.assertEqual(response.status_code, 201)
+        app_id = response.data['id']
+        # simulate a git push calling Build.push()
+        push = {'username': 'autotest', 'app': app_id}
+        databag = Build.push(push)
+        self.assertIn('release', databag)