Profile: redirect to /accounts/edit/ view on successful edit

Closes #10404

Author: humitos

readthedocs/profiles/views.py

@@ -61,8 +61,7 @@ def get_object(self):
 
     def get_success_url(self):
         return reverse(
-            'profiles_profile_detail',
-            kwargs={'username': self.request.user.username},
+            "profiles_profile_edit",
         )
 
 

readthedocs/rtd_tests/tests/test_profile_views.py

@@ -36,6 +36,7 @@ def test_edit_profile(self):
             },
         )
         self.assertTrue(resp.status_code, 200)
+        self.assertEqual(resp["Location"], "/accounts/edit/")
 
         self.user.refresh_from_db()
         self.user.profile.refresh_from_db()
@@ -195,32 +196,32 @@ def test_list_security_logs(self):
         resp = self.client.get(reverse('profiles_security_log'))
         self.assertEqual(resp.status_code, 200)
         auditlogs = resp.context_data['object_list']
-        self.assertQuerysetEqual(auditlogs, queryset)
+        self.assertQuerySetEqual(auditlogs, queryset)
 
         # Show logs filtered by project.
         resp = self.client.get(reverse('profiles_security_log') + '?project=project')
         self.assertEqual(resp.status_code, 200)
         auditlogs = resp.context_data["object_list"]
-        self.assertQuerysetEqual(auditlogs, queryset.filter(log_project_slug="project"))
+        self.assertQuerySetEqual(auditlogs, queryset.filter(log_project_slug="project"))
 
         # Show logs filtered by IP.
         ip = "10.10.10.2"
         resp = self.client.get(reverse("profiles_security_log") + f"?ip={ip}")
         self.assertEqual(resp.status_code, 200)
         auditlogs = resp.context_data['object_list']
-        self.assertQuerysetEqual(auditlogs, queryset.filter(ip=ip))
+        self.assertQuerySetEqual(auditlogs, queryset.filter(ip=ip))
 
         # Show logs filtered by action.
         resp = self.client.get(reverse('profiles_security_log') + '?action=authentication')
         self.assertEqual(resp.status_code, 200)
         auditlogs = resp.context_data['object_list']
-        self.assertQuerysetEqual(auditlogs, queryset.filter(action=AuditLog.AUTHN))
+        self.assertQuerySetEqual(auditlogs, queryset.filter(action=AuditLog.AUTHN))
 
         # Show logs filtered by action.
         resp = self.client.get(reverse('profiles_security_log') + '?action=authentication-failure')
         self.assertEqual(resp.status_code, 200)
         auditlogs = resp.context_data["object_list"]
-        self.assertQuerysetEqual(
+        self.assertQuerySetEqual(
             auditlogs, queryset.filter(action=AuditLog.AUTHN_FAILURE)
         )
 
@@ -235,7 +236,7 @@ def test_list_security_logs(self):
         resp = self.client.get(reverse('profiles_security_log') + '?action=invalid')
         self.assertEqual(resp.status_code, 200)
         auditlogs = resp.context_data['object_list']
-        self.assertQuerysetEqual(auditlogs, queryset)
+        self.assertQuerySetEqual(auditlogs, queryset)
 
 
 @override_settings(