Do not show Build button when user is not project admin (readthedocs#3309)

* Do not show Build button when user is not project admin

Closes readthedocs#3303

* Add test to check for Build section

* Fix old test to make sure a public branch is listed

Author: humitos

readthedocs/rtd_tests/tests/test_privacy.py

@@ -75,8 +75,12 @@ def test_private_repo(self):
         self.client.login(username='eric', password='test')
         r = self.client.get('/projects/django-kong/')
         self.assertEqual(r.status_code, 200)
+        # Build button should appear here
+        self.assertContains(r, 'Build a version')
         r = self.client.get('/projects/django-kong/builds/')
         self.assertEqual(r.status_code, 200)
+        # Build button should appear here
+        self.assertContains(r, 'Build Version:')
         r = self.client.get('/projects/django-kong/downloads/')
         self.assertEqual(r.status_code, 200)
 
@@ -100,16 +104,24 @@ def test_public_repo(self):
         self.client.login(username='eric', password='test')
         r = self.client.get('/projects/django-kong/')
         self.assertEqual(r.status_code, 200)
+        # Build button should appear here
+        self.assertContains(r, 'Build a version')
         r = self.client.get('/projects/django-kong/builds/')
         self.assertEqual(r.status_code, 200)
+        # Build button should appear here
+        self.assertContains(r, 'Build Version:')
         r = self.client.get('/projects/django-kong/downloads/')
         self.assertEqual(r.status_code, 200)
 
         self.client.login(username='tester', password='test')
         r = self.client.get('/projects/django-kong/')
         self.assertEqual(r.status_code, 200)
+        # Build button shouldn't appear here
+        self.assertNotContains(r, 'Build a version')
         r = self.client.get('/projects/django-kong/builds/')
         self.assertEqual(r.status_code, 200)
+        # Build button shouldn't appear here
+        self.assertNotContains(r, 'Build Version:')
         r = self.client.get('/projects/django-kong/downloads/')
         self.assertEqual(r.status_code, 200)
 
@@ -138,13 +150,14 @@ def test_public_branch(self):
 
         self.client.login(username='eric', password='test')
         Version.objects.create(project=kong, identifier='test id',
-                               verbose_name='test verbose', slug='test-slug', active=True)
+                               verbose_name='test verbose', slug='test-slug',
+                               active=True, built=True)
         self.assertEqual(Version.objects.count(), 2)
         self.assertEqual(Version.objects.all()[0].privacy_level, 'public')
         r = self.client.get('/projects/django-kong/')
         self.assertContains(r, 'test-slug')
 
-        # Make sure it doesn't show up as tester
+        # Make sure it does show up as tester
         self.client.login(username='tester', password='test')
         r = self.client.get('/projects/django-kong/')
         self.assertContains(r, 'test-slug')

readthedocs/templates/builds/build_list.html

@@ -3,6 +3,7 @@
 {% load i18n %}
 
 {% load pagination_tags %}
+{% load privacy_tags %}
 {% load projects_tags %}
 
 {% block title %}Builds{% endblock %}
@@ -37,7 +38,7 @@
           <div class="module-wrapper">
 
             {% block build_versions %}
-            {% if versions %}
+            {% if versions and request.user|is_admin:project %}
             <div class="module-header">
               <div style="float:right;">
                 <form method="post" action="{% url "builds_project_list" project.slug %}">

readthedocs/templates/core/project_details.html

@@ -55,7 +55,7 @@ <h3>{% trans "Versions" %}</h3>
   </div>
 
 {% block build_versions %}
-{% if versions %}
+{% if versions and request.user|is_admin:project %}
   <div class="build_a_version">
     <h3>{% trans "Build a version" %}</h3>
     <div class="version_right">