Sith/club/tests.py

# -*- coding:utf-8 -*
#
# Copyright 2023 © AE UTBM
# ae@utbm.fr / ae.info@utbm.fr
#
# This file is part of the website of the UTBM Student Association (AE UTBM),
# https://ae.utbm.fr.
#
# You can find the source code of the website at https://github.com/ae-utbm/sith3
#
# LICENSED UNDER THE GNU GENERAL PUBLIC LICENSE VERSION 3 (GPLv3)
# SEE : https://raw.githubusercontent.com/ae-utbm/sith3/master/LICENSE
# OR WITHIN THE LOCAL FILE "LICENSE"
#
#
from datetime import timedelta

from django.conf import settings
from django.core.cache import cache
from django.test import TestCase
from django.utils import timezone, html
from django.utils.timezone import now, localtime
from django.utils.translation import gettext as _
from django.urls import reverse
from django.core.management import call_command

from core.models import User, AnonymousUser
from club.models import Club, Membership, Mailing
from club.forms import MailingForm
from sith.settings import SITH_BAR_MANAGER, SITH_MAIN_CLUB_ID


class ClubTest(TestCase):
    """
    Set up data for test cases related to clubs and membership
    The generated dataset is the one created by the populate command,
    plus the following modifications :

    - `self.club` is a dummy club recreated for each test
    - `self.club` has two board members : skia (role 3) and comptable (role 10)
    - `self.club` has one regular member : richard
    - `self.club` has one former member : sli (who had role 2)
    - None of the `self.club` members are in the AE club.
    """

    @classmethod
    def setUpTestData(cls):
        # subscribed users - initial members
        cls.skia = User.objects.get(username="skia")
        cls.richard = User.objects.get(username="rbatsbak")
        cls.comptable = User.objects.get(username="comptable")
        cls.sli = User.objects.get(username="sli")

        # subscribed users - not initial members
        cls.krophil = User.objects.get(username="krophil")
        cls.subscriber = User.objects.get(username="subscriber")

        # old subscriber
        cls.old_subscriber = User.objects.get(username="old_subscriber")

        # not subscribed
        cls.public = User.objects.get(username="public")

        cls.ae = Club.objects.filter(pk=SITH_MAIN_CLUB_ID)[0]

    def setUp(self):
        # by default, Skia is in the AE, which creates side effect
        self.skia.memberships.all().delete()

        # create a fake club
        self.club = Club.objects.create(
            name="Fake Club",
            unix_name="fake-club",
            address="5 rue de la République, 90000 Belfort",
        )
        self.members_url = reverse(
            "club:club_members", kwargs={"club_id": self.club.id}
        )
        a_month_ago = now() - timedelta(days=30)
        yesterday = now() - timedelta(days=1)
        Membership.objects.create(
            club=self.club, user=self.skia, start_date=a_month_ago, role=3
        )
        Membership.objects.create(club=self.club, user=self.richard, role=1)
        Membership.objects.create(
            club=self.club, user=self.comptable, start_date=a_month_ago, role=10
        )

        # sli was a member but isn't anymore
        Membership.objects.create(
            club=self.club,
            user=self.sli,
            start_date=a_month_ago,
            end_date=yesterday,
            role=2,
        )
        cache.clear()


class MembershipQuerySetTest(ClubTest):
    def test_ongoing(self):
        """
        Test that the ongoing queryset method returns the memberships that
        are not ended.
        """
        current_members = self.club.members.ongoing()
        expected = [
            self.skia.memberships.get(club=self.club),
            self.comptable.memberships.get(club=self.club),
            self.richard.memberships.get(club=self.club),
        ]
        self.assertEqual(len(current_members), len(expected))
        for member in current_members:
            self.assertIn(member, expected)

    def test_board(self):
        """
        Test that the board queryset method returns the memberships
        of user in the club board
        """
        board_members = list(self.club.members.board())
        expected = [
            self.skia.memberships.get(club=self.club),
            self.comptable.memberships.get(club=self.club),
            # sli is no more member, but he was in the board
            self.sli.memberships.get(club=self.club),
        ]
        self.assertEqual(len(board_members), len(expected))
        for member in board_members:
            self.assertIn(member, expected)

    def test_ongoing_board(self):
        """
        Test that combining ongoing and board returns users
        who are currently board members of the club
        """
        members = list(self.club.members.ongoing().board())
        expected = [
            self.skia.memberships.get(club=self.club),
            self.comptable.memberships.get(club=self.club),
        ]
        self.assertEqual(len(members), len(expected))
        for member in members:
            self.assertIn(member, expected)

    def test_update_invalidate_cache(self):
        """
        Test that the `update` queryset method properly invalidate cache
        """
        mem_skia = self.skia.memberships.get(club=self.club)
        cache.set(f"membership_{mem_skia.club_id}_{mem_skia.user_id}", mem_skia)
        self.skia.memberships.update(end_date=localtime(now()).date())
        self.assertEqual(
            cache.get(f"membership_{mem_skia.club_id}_{mem_skia.user_id}"), "not_member"
        )

        mem_richard = self.richard.memberships.get(club=self.club)
        cache.set(
            f"membership_{mem_richard.club_id}_{mem_richard.user_id}", mem_richard
        )
        self.richard.memberships.update(role=5)
        new_mem = self.richard.memberships.get(club=self.club)
        self.assertNotEqual(new_mem, "not_member")
        self.assertEqual(new_mem.role, 5)

    def test_delete_invalidate_cache(self):
        """
        Test that the `delete` queryset properly invalidate cache
        """

        mem_skia = self.skia.memberships.get(club=self.club)
        mem_comptable = self.comptable.memberships.get(club=self.club)
        cache.set(f"membership_{mem_skia.club_id}_{mem_skia.user_id}", mem_skia)
        cache.set(
            f"membership_{mem_comptable.club_id}_{mem_comptable.user_id}", mem_comptable
        )

        # should delete the subscriptions of skia and comptable
        self.club.members.ongoing().board().delete()

        self.assertEqual(
            cache.get(f"membership_{mem_skia.club_id}_{mem_skia.user_id}"), "not_member"
        )
        self.assertEqual(
            cache.get(f"membership_{mem_comptable.club_id}_{mem_comptable.user_id}"),
            "not_member",
        )


class ClubModelTest(ClubTest):
    def assert_membership_just_started(self, user: User, role: int):
        """
        Assert that the given membership is active and started today
        """
        membership = user.memberships.ongoing().filter(club=self.club).first()
        self.assertIsNotNone(membership)
        self.assertEqual(localtime(now()).date(), membership.start_date)
        self.assertIsNone(membership.end_date)
        self.assertEqual(membership.role, role)
        self.assertEqual(membership.club.get_membership_for(user), membership)
        member_group = self.club.unix_name + settings.SITH_MEMBER_SUFFIX
        board_group = self.club.unix_name + settings.SITH_BOARD_SUFFIX
        self.assertTrue(user.is_in_group(name=member_group))
        self.assertTrue(user.is_in_group(name=board_group))

    def assert_membership_just_ended(self, user: User):
        """
        Assert that the given user have a membership which ended today
        """
        today = localtime(now()).date()
        self.assertIsNotNone(
            user.memberships.filter(club=self.club, end_date=today).first()
        )
        self.assertIsNone(self.club.get_membership_for(user))

    def test_access_unauthorized(self):
        """
        Test that users who never subscribed and anonymous users
        cannot see the page
        """
        response = self.client.post(self.members_url)
        self.assertEqual(response.status_code, 403)

        self.client.login(username="public", password="plop")
        response = self.client.post(self.members_url)
        self.assertEqual(response.status_code, 403)

    def test_display(self):
        """
        Test that a GET request return a page where the requested
        information are displayed.
        """
        self.client.login(username=self.skia.username, password="plop")
        response = self.client.get(self.members_url)
        self.assertEqual(response.status_code, 200)
        expected_html = (
            "<table><thead><tr>"
            "<td>Utilisateur</td><td>Rôle</td><td>Description</td>"
            "<td>Depuis</td><td>Marquer comme ancien</td>"
            "</tr></thead><tbody>"
        )
        memberships = self.club.members.ongoing().order_by("-role")
        input_id = 0
        for membership in memberships.select_related("user"):
            user = membership.user
            expected_html += (
                f"<tr><td><a href=\"{reverse('core:user_profile', args=[user.id])}\">"
                f"{user.get_display_name()}</a></td>"
                f"<td>{settings.SITH_CLUB_ROLES[membership.role]}</td>"
                f"<td>{membership.description}</td>"
                f"<td>{membership.start_date}</td><td>"
            )
            if membership.role <= 3:  # 3 is the role of skia
                expected_html += (
                    '<input type="checkbox" name="users_old" '
                    f'value="{user.id}" '
                    f'id="id_users_old_{input_id}">'
                )
                input_id += 1
            expected_html += "</td></tr>"
        expected_html += "</tbody></table>"
        self.assertInHTML(expected_html, response.content.decode())

    def test_root_add_one_club_member(self):
        """
        Test that root users can add members to clubs, one at a time
        """
        self.client.login(username="root", password="plop")
        response = self.client.post(
            self.members_url,
            {"users": self.subscriber.id, "role": 3},
        )
        self.assertRedirects(response, self.members_url)
        self.subscriber.refresh_from_db()
        self.assert_membership_just_started(self.subscriber, role=3)

    def test_root_add_multiple_club_member(self):
        """
        Test that root users can add multiple members at once to clubs
        """
        self.client.login(username="root", password="plop")
        response = self.client.post(
            self.members_url,
            {
                "users": f"|{self.subscriber.id}|{self.krophil.id}|",
                "role": 3,
            },
        )
        self.assertRedirects(response, self.members_url)
        self.subscriber.refresh_from_db()
        self.assert_membership_just_started(self.subscriber, role=3)
        self.assert_membership_just_started(self.krophil, role=3)

    def test_add_unauthorized_members(self):
        """
        Test that users who are not currently subscribed
        cannot be members of clubs.
        """
        self.client.login(username="root", password="plop")
        response = self.client.post(
            self.members_url,
            {"users": self.public.id, "role": 1},
        )
        self.assertIsNone(self.public.memberships.filter(club=self.club).first())
        self.assertTrue('<ul class="errorlist"><li>' in str(response.content))

        response = self.client.post(
            self.members_url,
            {"users": self.old_subscriber.id, "role": 1},
        )
        self.assertIsNone(self.public.memberships.filter(club=self.club).first())
        self.assertIsNone(self.club.get_membership_for(self.public))
        self.assertTrue('<ul class="errorlist"><li>' in str(response.content))

    def test_add_members_already_members(self):
        """
        Test that users who are already members of a club
        cannot be added again to this club
        """
        self.client.login(username="root", password="plop")
        current_membership = self.skia.memberships.ongoing().get(club=self.club)
        nb_memberships = self.skia.memberships.count()
        self.client.post(
            self.members_url,
            {"users": self.skia.id, "role": current_membership.role + 1},
        )
        self.skia.refresh_from_db()
        self.assertEqual(nb_memberships, self.skia.memberships.count())
        new_membership = self.skia.memberships.ongoing().get(club=self.club)
        self.assertEqual(current_membership, new_membership)
        self.assertEqual(self.club.get_membership_for(self.skia), new_membership)

    def test_add_not_existing_users(self):
        """
        Test that not existing users cannot be added in clubs.
        If one user in the request is invalid, no membership creation at all
        can take place.
        """
        self.client.login(username="root", password="plop")
        nb_memberships = self.club.members.count()
        response = self.client.post(
            self.members_url,
            {"users": [9999], "role": 1},
        )
        self.assertContains(response, '<ul class="errorlist"><li>')
        self.club.refresh_from_db()
        self.assertEqual(self.club.members.count(), nb_memberships)
        response = self.client.post(
            self.members_url,
            {
                "users": f"|{self.subscriber.id}|{9999}|",
                "start_date": "12/06/2016",
                "role": 3,
            },
        )
        self.assertContains(response, '<ul class="errorlist"><li>')
        self.club.refresh_from_db()
        self.assertEqual(self.club.members.count(), nb_memberships)

    def test_president_add_members(self):
        """
        Test that the president of the club can add members
        """
        president = self.club.members.get(role=10).user
        nb_club_membership = self.club.members.count()
        nb_subscriber_memberships = self.subscriber.memberships.count()
        self.client.login(username=president.username, password="plop")
        response = self.client.post(
            self.members_url,
            {"users": self.subscriber.id, "role": 9},
        )
        self.assertRedirects(response, self.members_url)
        self.club.refresh_from_db()
        self.subscriber.refresh_from_db()
        self.assertEqual(self.club.members.count(), nb_club_membership + 1)
        self.assertEqual(
            self.subscriber.memberships.count(), nb_subscriber_memberships + 1
        )
        self.assert_membership_just_started(self.subscriber, role=9)

    def test_add_member_greater_role(self):
        """
        Test that a member of the club member cannot create
        a membership with a greater role than its own.
        """
        self.client.login(username=self.skia.username, password="plop")
        nb_memberships = self.club.members.count()
        response = self.client.post(
            self.members_url,
            {"users": self.subscriber.id, "role": 10},
        )
        self.assertEqual(response.status_code, 200)
        self.assertInHTML(
            "<li>Vous n'avez pas la permission de faire cela</li>",
            response.content.decode(),
        )
        self.club.refresh_from_db()
        self.assertEqual(nb_memberships, self.club.members.count())
        self.assertIsNone(self.subscriber.memberships.filter(club=self.club).first())

    def test_add_member_without_role(self):
        """
        Test that trying to add members without specifying their role fails
        """
        self.client.login(username="root", password="plop")
        response = self.client.post(
            self.members_url,
            {"users": self.subscriber.id, "start_date": "12/06/2016"},
        )
        self.assertTrue(
            '<ul class="errorlist"><li>Vous devez choisir un r' in str(response.content)
        )

    def test_end_membership_self(self):
        """
        Test that a member can end its own membership
        """
        self.client.login(username="skia", password="plop")
        self.client.post(
            self.members_url,
            {"users_old": self.skia.id},
        )
        self.skia.refresh_from_db()
        self.assert_membership_just_ended(self.skia)

    def test_end_membership_lower_role(self):
        """
        Test that board members of the club can end memberships
        of users with lower roles
        """
        # remainder : skia has role 3, comptable has role 10, richard has role 1
        self.client.login(username=self.skia.username, password="plop")
        response = self.client.post(
            self.members_url,
            {"users_old": self.richard.id},
        )
        self.assertRedirects(response, self.members_url)
        self.club.refresh_from_db()
        self.assert_membership_just_ended(self.richard)

    def test_end_membership_higher_role(self):
        """
        Test that board members of the club cannot end memberships
        of users with higher roles
        """
        membership = self.comptable.memberships.filter(club=self.club).first()
        self.client.login(username=self.skia.username, password="plop")
        self.client.post(
            self.members_url,
            {"users_old": self.comptable.id},
        )
        self.club.refresh_from_db()
        new_membership = self.club.get_membership_for(self.comptable)
        self.assertIsNotNone(new_membership)
        self.assertEqual(new_membership, membership)

        membership = self.comptable.memberships.filter(club=self.club).first()
        self.assertIsNone(membership.end_date)

    def test_end_membership_as_main_club_board(self):
        """
        Test that board members of the main club can end the membership
        of anyone
        """
        # make subscriber a board member
        self.subscriber.memberships.all().delete()
        Membership.objects.create(club=self.ae, user=self.subscriber, role=3)

        nb_memberships = self.club.members.count()
        self.client.login(username=self.subscriber.username, password="plop")
        response = self.client.post(
            self.members_url,
            {"users_old": self.comptable.id},
        )
        self.assertRedirects(response, self.members_url)
        self.assert_membership_just_ended(self.comptable)
        self.assertEqual(self.club.members.ongoing().count(), nb_memberships - 1)

    def test_end_membership_as_root(self):
        """
        Test that root users can end the membership of anyone
        """
        nb_memberships = self.club.members.count()
        self.client.login(username="root", password="plop")
        response = self.client.post(
            self.members_url,
            {"users_old": [self.comptable.id]},
        )
        self.assertRedirects(response, self.members_url)
        self.assert_membership_just_ended(self.comptable)
        self.assertEqual(self.club.members.ongoing().count(), nb_memberships - 1)
        self.assertEqual(self.club.members.count(), nb_memberships)

    def test_end_membership_as_foreigner(self):
        """
        Test that users who are not in this club cannot end its memberships
        """
        nb_memberships = self.club.members.count()
        membership = self.richard.memberships.filter(club=self.club).first()
        self.client.login(username="subscriber", password="root")
        self.client.post(
            self.members_url,
            {"users_old": [self.richard.id]},
        )
        # nothing should have changed
        new_mem = self.club.get_membership_for(self.richard)
        self.assertIsNotNone(new_mem)
        self.assertEqual(self.club.members.count(), nb_memberships)
        self.assertEqual(membership, new_mem)

    def test_delete_remove_from_meta_group(self):
        """
        Test that when a club is deleted, all its members are removed from the
        associated metagroup
        """
        memberships = self.club.members.select_related("user")
        users = [membership.user for membership in memberships]
        meta_group = self.club.unix_name + settings.SITH_MEMBER_SUFFIX

        self.club.delete()
        for user in users:
            self.assertFalse(user.is_in_group(name=meta_group))

    def test_add_to_meta_group(self):
        """
        Test that when a membership begins, the user is added to the meta group
        """
        group_members = self.club.unix_name + settings.SITH_MEMBER_SUFFIX
        board_members = self.club.unix_name + settings.SITH_BOARD_SUFFIX
        self.assertFalse(self.subscriber.is_in_group(name=group_members))
        self.assertFalse(self.subscriber.is_in_group(name=board_members))
        Membership.objects.create(club=self.club, user=self.subscriber, role=3)
        self.assertTrue(self.subscriber.is_in_group(name=group_members))
        self.assertTrue(self.subscriber.is_in_group(name=board_members))

    def test_remove_from_meta_group(self):
        """
        Test that when a membership ends, the user is removed from meta group
        """
        group_members = self.club.unix_name + settings.SITH_MEMBER_SUFFIX
        board_members = self.club.unix_name + settings.SITH_BOARD_SUFFIX
        self.assertTrue(self.comptable.is_in_group(name=group_members))
        self.assertTrue(self.comptable.is_in_group(name=board_members))
        self.comptable.memberships.update(end_date=localtime(now()))
        self.assertFalse(self.comptable.is_in_group(name=group_members))
        self.assertFalse(self.comptable.is_in_group(name=board_members))

    def test_club_owner(self):
        """
        Test that a club is owned only by board members of the main club
        """
        anonymous = AnonymousUser()
        self.assertFalse(self.club.is_owned_by(anonymous))
        self.assertFalse(self.club.is_owned_by(self.subscriber))

        # make sli a board member
        self.sli.memberships.all().delete()
        Membership(club=self.ae, user=self.sli, role=3).save()
        self.assertTrue(self.club.is_owned_by(self.sli))


class MailingFormTest(TestCase):
    """Perform validation tests for MailingForm"""

    @classmethod
    def setUpTestData(cls):
        cls.skia = User.objects.filter(username="skia").first()
        cls.rbatsbak = User.objects.filter(username="rbatsbak").first()
        cls.krophil = User.objects.filter(username="krophil").first()
        cls.comunity = User.objects.filter(username="comunity").first()
        cls.bdf = Club.objects.filter(unix_name=SITH_BAR_MANAGER["unix_name"]).first()

    def setUp(self):
        Membership(
            user=self.rbatsbak,
            club=self.bdf,
            start_date=timezone.now(),
            role=settings.SITH_CLUB_ROLES_ID["Board member"],
        ).save()

    def test_mailing_list_add_no_moderation(self):
        # Test with Communication admin
        self.client.login(username="comunity", password="plop")
        self.client.post(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id}),
            {"action": MailingForm.ACTION_NEW_MAILING, "mailing_email": "foyer"},
        )
        response = self.client.get(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id})
        )
        self.assertContains(response, text="Liste de diffusion foyer@utbm.fr")

        # Test with Root
        self.client.login(username="root", password="plop")
        self.client.post(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id}),
            {"action": MailingForm.ACTION_NEW_MAILING, "mailing_email": "mde"},
        )
        response = self.client.get(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id})
        )
        self.assertContains(response, text="Liste de diffusion mde@utbm.fr")

    def test_mailing_list_add_moderation(self):
        self.client.login(username="rbatsbak", password="plop")
        self.client.post(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id}),
            {"action": MailingForm.ACTION_NEW_MAILING, "mailing_email": "mde"},
        )
        response = self.client.get(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id})
        )
        self.assertNotContains(response, text="Liste de diffusion mde@utbm.fr")
        self.assertContains(
            response, text="<p>Listes de diffusions en attente de modération</p>"
        )
        self.assertContains(response, "<li>mde@utbm.fr")

    def test_mailing_list_forbidden(self):
        # With anonymous user
        response = self.client.get(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id})
        )
        self.assertContains(response, "", status_code=403)

        # With user not in club
        self.client.login(username="krophil", password="plop")
        response = self.client.get(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id})
        )
        self.assertContains(response, "", status_code=403)

    def test_add_new_subscription_fail_not_moderated(self):
        self.client.login(username="rbatsbak", password="plop")
        self.client.post(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id}),
            {"action": MailingForm.ACTION_NEW_MAILING, "mailing_email": "mde"},
        )

        self.client.post(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id}),
            {
                "action": MailingForm.ACTION_NEW_SUBSCRIPTION,
                "subscription_users": self.skia.id,
                "subscription_mailing": Mailing.objects.get(email="mde").id,
            },
        )
        response = self.client.get(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id})
        )
        self.assertNotContains(response, "skia@git.an")

    def test_add_new_subscription_success(self):
        # Prepare mailing list
        self.client.login(username="comunity", password="plop")
        self.client.post(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id}),
            {"action": MailingForm.ACTION_NEW_MAILING, "mailing_email": "mde"},
        )

        # Add single user
        self.client.post(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id}),
            {
                "action": MailingForm.ACTION_NEW_SUBSCRIPTION,
                "subscription_users": self.skia.id,
                "subscription_mailing": Mailing.objects.get(email="mde").id,
            },
        )
        response = self.client.get(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id})
        )
        self.assertContains(response, "skia@git.an")

        # Add multiple users
        self.client.post(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id}),
            {
                "action": MailingForm.ACTION_NEW_SUBSCRIPTION,
                "subscription_users": "|%s|%s|" % (self.comunity.id, self.rbatsbak.id),
                "subscription_mailing": Mailing.objects.get(email="mde").id,
            },
        )
        response = self.client.get(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id})
        )
        self.assertContains(response, "richard@git.an")
        self.assertContains(response, "comunity@git.an")
        self.assertContains(response, "skia@git.an")

        # Add arbitrary email
        self.client.post(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id}),
            {
                "action": MailingForm.ACTION_NEW_SUBSCRIPTION,
                "subscription_email": "arbitrary@git.an",
                "subscription_mailing": Mailing.objects.get(email="mde").id,
            },
        )
        response = self.client.get(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id})
        )
        self.assertContains(response, "richard@git.an")
        self.assertContains(response, "comunity@git.an")
        self.assertContains(response, "skia@git.an")
        self.assertContains(response, "arbitrary@git.an")

        # Add user and arbitrary email
        self.client.post(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id}),
            {
                "action": MailingForm.ACTION_NEW_SUBSCRIPTION,
                "subscription_email": "more.arbitrary@git.an",
                "subscription_users": self.krophil.id,
                "subscription_mailing": Mailing.objects.get(email="mde").id,
            },
        )
        response = self.client.get(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id})
        )
        self.assertContains(response, "richard@git.an")
        self.assertContains(response, "comunity@git.an")
        self.assertContains(response, "skia@git.an")
        self.assertContains(response, "arbitrary@git.an")
        self.assertContains(response, "more.arbitrary@git.an")
        self.assertContains(response, "krophil@git.an")

    def test_add_new_subscription_fail_form_errors(self):
        # Prepare mailing list
        self.client.login(username="comunity", password="plop")
        self.client.post(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id}),
            {"action": MailingForm.ACTION_NEW_MAILING, "mailing_email": "mde"},
        )

        # Neither email or email is specified
        response = self.client.post(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id}),
            {
                "action": MailingForm.ACTION_NEW_SUBSCRIPTION,
                "subscription_mailing": Mailing.objects.get(email="mde").id,
            },
        )
        self.assertContains(
            response, text=_("You must specify at least an user or an email address")
        )

        # No mailing specified
        response = self.client.post(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id}),
            {
                "action": MailingForm.ACTION_NEW_SUBSCRIPTION,
                "subscription_users": self.krophil.id,
            },
        )
        self.assertContains(response, text=_("This field is required"))

        # One of the selected users doesn't exist
        response = self.client.post(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id}),
            {
                "action": MailingForm.ACTION_NEW_SUBSCRIPTION,
                "subscription_users": "|789|",
                "subscription_mailing": Mailing.objects.get(email="mde").id,
            },
        )
        self.assertContains(
            response, text=html.escape(_("One of the selected users doesn't exist"))
        )

        # An user has no email adress
        self.krophil.email = ""
        self.krophil.save()

        response = self.client.post(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id}),
            {
                "action": MailingForm.ACTION_NEW_SUBSCRIPTION,
                "subscription_users": self.krophil.id,
                "subscription_mailing": Mailing.objects.get(email="mde").id,
            },
        )
        self.assertContains(
            response,
            text=html.escape(
                _("One of the selected users doesn't have an email address")
            ),
        )

        self.krophil.email = "krophil@git.an"
        self.krophil.save()

        # An user is added twice

        self.client.post(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id}),
            {
                "action": MailingForm.ACTION_NEW_SUBSCRIPTION,
                "subscription_users": self.krophil.id,
                "subscription_mailing": Mailing.objects.get(email="mde").id,
            },
        )

        response = self.client.post(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id}),
            {
                "action": MailingForm.ACTION_NEW_SUBSCRIPTION,
                "subscription_users": self.krophil.id,
                "subscription_mailing": Mailing.objects.get(email="mde").id,
            },
        )
        self.assertContains(
            response,
            text=html.escape(_("This email is already suscribed in this mailing")),
        )

    def test_remove_subscription_success(self):
        # Prepare mailing list
        self.client.login(username="comunity", password="plop")
        self.client.post(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id}),
            {"action": MailingForm.ACTION_NEW_MAILING, "mailing_email": "mde"},
        )
        mde = Mailing.objects.get(email="mde")
        self.client.post(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id}),
            {
                "action": MailingForm.ACTION_NEW_SUBSCRIPTION,
                "subscription_users": "|%s|%s|%s|"
                % (self.comunity.id, self.rbatsbak.id, self.krophil.id),
                "subscription_mailing": mde.id,
            },
        )

        response = self.client.get(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id})
        )

        self.assertContains(response, "comunity@git.an")
        self.assertContains(response, "richard@git.an")
        self.assertContains(response, "krophil@git.an")

        # Delete one user
        self.client.post(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id}),
            {
                "action": MailingForm.ACTION_REMOVE_SUBSCRIPTION,
                "removal_%d" % mde.id: mde.subscriptions.get(user=self.krophil).id,
            },
        )
        response = self.client.get(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id})
        )

        self.assertContains(response, "comunity@git.an")
        self.assertContains(response, "richard@git.an")
        self.assertNotContains(response, "krophil@git.an")

        # Delete multiple users
        self.client.post(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id}),
            {
                "action": MailingForm.ACTION_REMOVE_SUBSCRIPTION,
                "removal_%d"
                % mde.id: [
                    user.id
                    for user in mde.subscriptions.filter(
                        user__in=[self.rbatsbak, self.comunity]
                    ).all()
                ],
            },
        )
        response = self.client.get(
            reverse("club:mailing", kwargs={"club_id": self.bdf.id})
        )

        self.assertNotContains(response, "comunity@git.an")
        self.assertNotContains(response, "richard@git.an")
        self.assertNotContains(response, "krophil@git.an")


class ClubSellingViewTest(TestCase):
    """
    Perform basics tests to ensure that the page is available
    """

    def setUp(self):
        self.ae = Club.objects.filter(unix_name="ae").first()

    def test_page_not_internal_error(self):
        """
        Test that the page does not return and internal error
        """
        self.client.login(username="skia", password="plop")
        response = self.client.get(
            reverse("club:club_sellings", kwargs={"club_id": self.ae.id})
        )
        self.assertFalse(response.status_code == 500)