Small human cleanup of test cases

Author: ericholscher

readthedocs/rtd_tests/tests/test_core_management.py

@@ -40,7 +40,7 @@ def _input(_, files=None):
             assert cmd._get_input() == '{"test": "foobar"}'
 
     @patch.object(run_docker.Command, '_get_input', _get_input)
-    .patch('readthedocs.projects.tasks.docker_build', _docker_build)
+    @patch('readthedocs.projects.tasks.docker_build', _docker_build)
     @patch('sys.stdout', new_callable=StringIO)
     def test_good_input(self, mock_output):
         '''Test docker build command'''
@@ -52,7 +52,7 @@ def test_good_input(self, mock_output):
             '{"html": [0, "DOCKER PASS", ""]}\n'
         )
 
-    .patch('readthedocs.projects.tasks.docker_build', _docker_build)
+    @patch('readthedocs.projects.tasks.docker_build', _docker_build)
     def test_bad_input(self):
         '''Test docker build command'''
         with patch.object(run_docker.Command, '_get_input') as mock_input:

readthedocs/rtd_tests/tests/test_project_symlinks.py

@@ -18,7 +18,7 @@ def wrapper(self):
         def _collect_commands(cmd):
             self.commands.append(cmd)
 
-        with.patch('readthedocs.projects.symlinks.run_on_app_servers', _collect_commands):
+        with patch('readthedocs.projects.symlinks.run_on_app_servers', _collect_commands):
             with patch('readthedocs.projects.symlinks.run_on_app_servers', _collect_commands):
                 return fn(self)
     return wrapper

readthedocs/rtd_tests/tests/test_urls.py

@@ -8,7 +8,7 @@
 class SubdomainUrlTests(TestCase):
 
     def test_sub_index(self):
-        url = reverse(core.views.redirect_project_slug,
+        url = reverse(readthedocs.core.views.redirect_project_slug,
             urlconf='readthedocs.core.subdomain_urls')
         self.assertEqual(url, '/')