Remove param in test

Author: jonkafton

authentication/views.py

@@ -4,7 +4,7 @@
 
 from django.contrib.auth import logout
 from django.shortcuts import redirect
-from django.utils.http import url_has_allowed_host_and_scheme, urlencode
+from django.utils.http import url_has_allowed_host_and_scheme
 from django.views import View
 
 from main import settings
@@ -80,8 +80,7 @@ def get(
                 not profile.completed_onboarding
                 and request.GET.get("skip_onboarding", "0") == "0"
             ):
-                params = urlencode({"next": redirect_url})
-                redirect_url = f"{settings.MITOL_NEW_USER_LOGIN_URL}?{params}"
+                redirect_url = settings.MITOL_NEW_USER_LOGIN_URL
                 profile.completed_onboarding = True
                 profile.save()
         return redirect(redirect_url)

authentication/views_test.py

@@ -120,15 +120,14 @@ def test_custom_login_view_authenticated_user_with_onboarding(mocker):
     request.user = MagicMock(is_anonymous=False)
     request.user.profile = MagicMock(completed_onboarding=False)
     mocker.patch("authentication.views.get_redirect_url", return_value="/dashboard")
-    mocker.patch("authentication.views.urlencode", return_value="next=/dashboard")
     mocker.patch(
         "authentication.views.settings.MITOL_NEW_USER_LOGIN_URL", "/onboarding"
     )
 
     response = CustomLoginView().get(request)
 
     assert response.status_code == 302
-    assert response.url == "/onboarding?next=/dashboard"
+    assert response.url == "/onboarding"
 
 
 def test_custom_login_view_authenticated_user_skip_onboarding(mocker):