Implemented user redirection to memberkit for webinars

renzon · renzon · commit 301fe2a810ab · 2021-06-24T18:32:02.000-03:00
close #3845
diff --git a/pythonpro/cohorts/tests/test_webinar_detail.py b/pythonpro/cohorts/tests/test_webinar_detail.py
@@ -1,65 +1,56 @@
 import pytest
 from django.core.files.uploadedfile import SimpleUploadedFile
 from django.urls import reverse
+from django.utils import timezone
 from model_bakery import baker
 
 from pythonpro.cohorts.models import Webinar
 from pythonpro.cohorts.tests.conftest import img_path
-from pythonpro.django_assertions import dj_assert_contains, dj_assert_not_contains
+from pythonpro.memberkit.models import Subscription
 
 
 @pytest.fixture
 def webinar(cohort) -> Webinar:
     image = SimpleUploadedFile(name='renzo-nuccitelli.jpeg', content=open(img_path, 'rb').read(),
                                content_type='image/png')
-    return baker.make(Webinar, cohort=cohort, image=image, vimeo_id='1')
+    return baker.make(Webinar, cohort=cohort, image=image, vimeo_id='1', memberkit_url='https://plataforma.dev.pro.br')
 
 
 @pytest.fixture
-def resp(client_with_level_three_roles, webinar: Webinar):
-    return client_with_level_three_roles.get(reverse('cohorts:webinar', kwargs={'slug': webinar.slug}))
+def resp(client_with_user, webinar: Webinar):
+    return client_with_user.get(reverse('cohorts:webinar', kwargs={'slug': webinar.slug}))
 
 
-def test_logged_user(resp):
-    assert resp.status_code == 200
+def test_logged_user(resp, webinar):
+    assert resp.status_code == 302
+    assert resp.url == reverse('checkout:bootcamp_lp')
 
 
 def test_link_unavailable_for_non_users(client):
     resp = client.get(reverse('cohorts:webinar', kwargs={'slug': 'foo'}))
     assert resp.status_code == 302
 
 
-@pytest.mark.parametrize('property_name', 'speaker speaker_title title description vimeo_id discourse_topic_id'.split())
-def test_basic_contents(resp, webinar, property_name):
-    dj_assert_contains(resp, getattr(webinar, property_name))
-
-
-@pytest.fixture
-def resp_video_not_recorded(client_with_level_three_roles, webinar: Webinar):
-    webinar.vimeo_id = ''
-    webinar.save()
-    return client_with_level_three_roles.get(reverse('cohorts:webinar', kwargs={'slug': webinar.slug}))
-
-
-def test_pending_webinar_msg(resp_video_not_recorded):
-    dj_assert_contains(
-        resp_video_not_recorded,
-        'Ainda não temos máquina do tempo, esse webinário ainda não foi gravado'
+def test_redirect_user_not_migrated_to_memberkit(client_with_user, webinar, logged_user):
+    baker.make(
+        Subscription,
+        subscriber=logged_user,
+        activated_at=None,
+        memberkit_user_id=None
     )
-
-
-def test_vimeo_player_not_present(resp_video_not_recorded):
-    dj_assert_not_contains(
-        resp_video_not_recorded,
-        'src="https://player.vimeo.com/video/"'
+    resp = client_with_user.get(reverse('cohorts:webinar', kwargs={'slug': webinar.slug}))
+    assert resp.status_code == 301
+    assert resp.url == reverse('migrate_to_memberkit')
+
+
+def test_python_birds_migrated_user(client_with_user, webinar, logged_user):
+    baker.make(
+        Subscription,
+        status=Subscription.Status.ACTIVE,
+        subscriber=logged_user,
+        activated_at=timezone.now(),
+        memberkit_user_id=1
     )
-
-
-@pytest.fixture
-def resp_not_level_three(client_with_not_level_three_roles, webinar: Webinar, logged_user):
-    return client_with_not_level_three_roles.get(reverse('cohorts:webinar', kwargs={'slug': webinar.slug}))
-
-
-def test_webinar_landing_for_client(cohort, resp_not_level_three):
-    assert resp_not_level_three.status_code == 302
-    assert resp_not_level_three.url == reverse('checkout:bootcamp_lp')
+    resp = client_with_user.get(reverse('cohorts:webinar', kwargs={'slug': webinar.slug}))
+    assert resp.status_code == 301
+    assert resp.url == webinar.memberkit_url
diff --git a/pythonpro/cohorts/views.py b/pythonpro/cohorts/views.py
@@ -5,6 +5,7 @@
 
 from pythonpro.cohorts import facade
 from pythonpro.core.roles import access_cohorts
+from pythonpro.memberkit import facade as memberkit_facade
 
 
 @login_required
@@ -19,9 +20,13 @@ def webinars(request):
 
 @login_required
 def webinar(request, slug):
-    if not has_permission(request.user, access_cohorts):
-        return redirect(reverse('checkout:bootcamp_lp'), permanent=False)
-    return render(request, 'cohorts/webinar_detail.html', {'webinar': facade.find_webinar(slug=slug)})
+    user = request.user
+    if memberkit_facade.has_memberkit_account(user):
+        webinar = facade.find_webinar(slug=slug)
+        return redirect(webinar.memberkit_url, permanent=True)
+    if memberkit_facade.has_any_subscription(user):
+        return redirect(reverse('migrate_to_memberkit'), permanent=True)
+    return redirect(reverse('checkout:bootcamp_lp'), permanent=False)
 
 
 @login_required
