Update tests with new Version.get_absolute_url implementation

Author: benjaoming

readthedocs/rtd_tests/tests/test_api.py

@@ -6,6 +6,7 @@
 import dateutil
 from allauth.socialaccount.models import SocialAccount
 from django.contrib.auth.models import User
+from django.core.validators import URLValidator
 from django.http import QueryDict
 from django.test import TestCase, override_settings
 from django.urls import reverse
@@ -299,12 +300,14 @@ def test_response_building(self):
                 'version_slug': version.slug,
             },
         )
+        url_validator = URLValidator(verify_exists=False, schemes=["https"])
         build = resp.data
-        self.assertEqual(build['state'], 'cloning')
-        self.assertEqual(build['error'], '')
-        self.assertEqual(build['exit_code'], 0)
-        self.assertEqual(build['success'], True)
-        self.assertEqual(build['docs_url'], dashboard_url)
+        self.assertEqual(build["state"], "cloning")
+        self.assertEqual(build["error"], "")
+        self.assertEqual(build["exit_code"], 0)
+        self.assertEqual(build["success"], True)
+        self.assertTrue(build["docs_url"].endswith(dashboard_url))
+        self.assertTrue(url_validator.validate(build["docs_url"]))
 
     @override_settings(DOCROOT="/home/docs/checkouts/readthedocs.org/user_builds")
     def test_response_finished_and_success(self):
@@ -381,6 +384,7 @@ def test_response_finished_and_fail(self):
 
         resp = client.get('/api/v2/build/{build}/'.format(build=build.pk))
         self.assertEqual(resp.status_code, 200)
+        url_validator = URLValidator(verify_exists=False, schemes=["https"])
 
         dashboard_url = reverse(
             'project_version_detail',
@@ -390,11 +394,12 @@ def test_response_finished_and_fail(self):
             },
         )
         build = resp.data
-        self.assertEqual(build['state'], 'finished')
-        self.assertEqual(build['error'], '')
-        self.assertEqual(build['exit_code'], 1)
-        self.assertEqual(build['success'], False)
-        self.assertEqual(build['docs_url'], dashboard_url)
+        self.assertEqual(build["state"], "finished")
+        self.assertEqual(build["error"], "")
+        self.assertEqual(build["exit_code"], 1)
+        self.assertEqual(build["success"], False)
+        self.assertTrue(build["docs_url"].endswith(dashboard_url))
+        self.assertTrue(url_validator.validate(build["docs_url"]))
 
     def test_make_build_without_permission(self):
         """Ensure anonymous/non-staff users cannot write the build endpoint."""

readthedocs/rtd_tests/tests/test_oauth.py

@@ -350,6 +350,8 @@ def test_send_build_status_value_error(self, session, mock_logger):
             commit_status='success',
             user_username=self.user.username,
             statuses_url='https://api.github.com/repos/pypa/pip/statuses/1234',
+            target_url=mock.ANY,
+            status="success",
         )
         mock_logger.exception.assert_called_with(
             'GitHub commit status creation failed for project.',