Rename get_env_vars -> get_build_env_vars

Author: stsewd

readthedocs/projects/tasks.py

@@ -741,7 +741,7 @@ def run_build(self, record):
         :param record: whether or not record all the commands in the ``Build``
             instance
         """
-        env_vars = self.get_env_vars()
+        env_vars = self.get_build_env_vars()
 
         if settings.DOCKER_ENABLE:
             env_cls = DockerBuildEnvironment
@@ -932,7 +932,7 @@ def setup_vcs(self, environment):
         if commit:
             self.build['commit'] = commit
 
-    def get_env_vars(self):
+    def get_build_env_vars(self):
         """Get bash environment variables used for all builder commands."""
         env = self.get_rtd_env_vars()
 

readthedocs/rtd_tests/tests/test_builds.py

@@ -2,7 +2,6 @@
 import os
 from unittest import mock
 
-from allauth.socialaccount.models import SocialAccount
 from django.contrib.auth.models import User
 from django.test import TestCase
 from django.utils import timezone
@@ -21,7 +20,6 @@
 from readthedocs.doc_builder.environments import LocalBuildEnvironment
 from readthedocs.doc_builder.exceptions import DuplicatedBuildError
 from readthedocs.doc_builder.python_environments import Virtualenv
-from readthedocs.oauth.models import RemoteRepository
 from readthedocs.projects.models import EnvironmentVariable, Feature, Project
 from readthedocs.projects.tasks import UpdateDocsTaskStep
 from readthedocs.rtd_tests.tests.test_config_integration import create_load
@@ -363,7 +361,7 @@ def test_get_env_vars(self):
             ),
             'TOKEN': 'a1b2c3',
         }
-        self.assertEqual(task.get_env_vars(), env)
+        self.assertEqual(task.get_build_env_vars(), env)
 
         # mock this object to make sure that we are in a conda env
         task.config = mock.Mock(conda=True)
@@ -377,7 +375,7 @@ def test_get_env_vars(self):
                 'bin',
             ),
         })
-        self.assertEqual(task.get_env_vars(), env)
+        self.assertEqual(task.get_build_env_vars(), env)
 
 
 class BuildModelTests(TestCase):