Merge pull request #456 from opdemand/455-get-config

Get returns current Config, not latest Config. Fixes #455.

Author: Gabriel Monroy

api/tests/test_release.py

@@ -210,6 +210,25 @@ def test_release_rollback(self):
         self.assertNotEqual(release1['uuid'], release5['uuid'])
         self.assertEqual(release1['build'], release5['build'])
         self.assertEqual(release1['config'], release5['config'])
+        # check to see that the current config is actually the initial one
+        url = "/api/apps/{app_id}/config".format(**locals())
+        response = self.client.get(url)
+        self.assertEqual(response.status_code, 200)
+        self.assertEqual(response.data['values'], json.dumps({}))
+        # rollback to #3 and see that it has the correct config
+        url = "/api/apps/{app_id}/releases/rollback/".format(**locals())
+        body = {'version': 3}
+        response = self.client.post(
+            url, json.dumps(body), content_type='application/json')
+        self.assertEqual(response.status_code, 201)
+        url = "/api/apps/{app_id}/config".format(**locals())
+        response = self.client.get(url)
+        self.assertEqual(response.status_code, 200)
+        values = json.loads(response.data['values'])
+        self.assertIn('NEW_URL1', values)
+        self.assertEqual('http://localhost:8080/', values['NEW_URL1'])
+        self.assertIn('PATH', values)
+        self.assertEqual('bin:/usr/local/bin:/usr/bin:/bin', values['PATH'])
 
     def test_release_str(self):
         """Test the text representation of a release."""

api/views.py

@@ -579,6 +579,14 @@ class AppConfigViewSet(BaseAppViewSet):
     model = models.Config
     serializer_class = serializers.ConfigSerializer
 
+    def get_object(self, *args, **kwargs):
+        """Return the Config associated with the App's latest Release."""
+        app = get_object_or_404(models.App, id=self.kwargs['id'])
+        user = self.request.user
+        if user == app.owner or user in get_users_with_perms(app):
+            return app.release_set.latest().config
+        raise PermissionDenied()
+
     def post_save(self, obj, created=False):
         if created:
             models.release_signal.send(