From d56ab02c271a4ddaddfdc1e456be0ce429bbe283 Mon Sep 17 00:00:00 2001 From: Aarni Halinen Date: Mon, 29 Mar 2021 01:45:11 +0300 Subject: [PATCH] remove sahkopiikki related code --- .../commands/createsahkopiikkiuser.py | 30 ---------------- members/tests.py | 35 ------------------- 2 files changed, 65 deletions(-) delete mode 100644 members/management/commands/createsahkopiikkiuser.py diff --git a/members/management/commands/createsahkopiikkiuser.py b/members/management/commands/createsahkopiikkiuser.py deleted file mode 100644 index ff9e3e5..0000000 --- a/members/management/commands/createsahkopiikkiuser.py +++ /dev/null @@ -1,30 +0,0 @@ -from django.core.management.base import BaseCommand, CommandError -from django.contrib.auth.models import User, Permission -from rest_framework.authtoken.models import Token - - -class Command(BaseCommand): - user_name = "sahkopiikki" - password = User.objects.make_random_password() - - def handle(self, *args, **options): - if User.objects.filter(username=self.user_name).exists(): - self.stdout.write("Sahkopiikki user already exists. Skipping.") - user = User.objects.get(username=self.user_name) - token = Token.objects.get(user=user) - self.stdout.write("Token: {}".format(token)) - return - - u = User(username=self.user_name) - u.set_password(self.password) - - u.save() - - permission = Permission.objects.get(codename='check_by_email') - u.user_permissions.add(permission) - - token = Token.objects.create(user=u) - - self.stdout.write("Created sahkopiikki user '{}' with password '{}' and token '{}'.".format( - self.user_name, self.password, token - )) diff --git a/members/tests.py b/members/tests.py index 7105332..53c3759 100644 --- a/members/tests.py +++ b/members/tests.py @@ -3,7 +3,6 @@ from django.test import TestCase, Client from unittest import skip from django.contrib.auth.models import User -from members.management.commands.createsahkopiikkiuser import Command as SahkopiikkiCommand from members.models import Member, Payment, Request from rest_framework.authtoken.models import Token @@ -31,9 +30,6 @@ class MemberRegisterTestCase(TestCase): self.c = Client() self.c.login(username=username, password=password) - sc = SahkopiikkiCommand() - sc.handle() - def test_member_created(self): """Test member creation.""" exists = Member.objects.filter(first_name="Tidus").exists() @@ -78,37 +74,6 @@ class MemberRegisterTestCase(TestCase): results = response.json()['results'] self.assertEqual(len(results), 0) - def test_sahkopiikki_check_by_email_not_found(self): - """Test if sähköpiikki auth and search work""" - email = 'teppo@tulppu.fi' - wrong_email = 'asd@asd.fi' - Member.objects.create(email=email, first_name='Teppo', last_name='Tulppu') - token = Token.objects.get(user__username='sahkopiikki').key - self.c.defaults['HTTP_AUTHORIZATION'] = 'Token ' + token - - response = self.c.get('/members/check?email={}'.format(wrong_email), follow=True) - self.assertEqual(response.json()['exists'], False) - - def test_sahkopiikki_check_by_email_found(self): - """Test if sähköpiikki auth and search work""" - email = 'teppo@tulppu.fi' - Member.objects.create(email=email, first_name='Teppo', last_name='Tulppu') - token = Token.objects.get(user__username='sahkopiikki').key - self.c.defaults['HTTP_AUTHORIZATION'] = 'Token ' + token - - response = self.c.get('/members/check?email={}'.format(email), follow=True) - self.assertEqual(response.json()['exists'], True) - - def test_sahkopiikki_check_by_email_forbidden(self): - """Test if sähköpiikki auth and search work""" - email = 'teppo@tulppu.fi' - Member.objects.create(email=email, first_name='Teppo', last_name='Tulppu') - token = Token.objects.get(user__username='sahkopiikki').key - self.c.defaults['HTTP_AUTHORIZATION'] = 'Token ' + token + 'DERP' - - response = self.c.get('/members/check?email={}'.format(email), follow=True) - self.assertEqual(response.status_code, 401) - def test_export_members_excel(self): """Test if the user can download an excel file of the member register""" resp = self.c.get('/members/export_members')