Improved most messages in spanish (#39)

Removing Typos

bot.py

@@ -2,7 +2,7 @@
 # -*- coding: utf-8 -*-
 #
 # Telegram bot to play UNO in group chats
-# Copyright (c) 2016 Jannes Höke <uno@jhoeke.de>
+# Copyright (c) 2016 - 2017 Jannes Höke <uno@jhoeke.de> and Karho Yau
 #
 # This program is free software: you can redistribute it and/or modify
 # it under the terms of the GNU Affero General Public License as
@@ -23,9 +23,9 @@ from datetime import datetime
 from random import randint
 
 from telegram import ParseMode, Message, Chat, InlineKeyboardMarkup, \
-    InlineKeyboardButton
+    InlineKeyboardButton, ReplyKeyboardMarkup, Emoji
 from telegram.ext import InlineQueryHandler, ChosenInlineResultHandler, \
-    CommandHandler, MessageHandler, Filters, CallbackQueryHandler
+    CommandHandler, MessageHandler, Filters, CallbackQueryHandler, RegexHandler
 from telegram.ext.dispatcher import run_async
 
 from start_bot import start_bot
@@ -36,7 +36,7 @@ from user_setting import UserSetting
 from utils import display_name
 import card as c
 from errors import (NoGameInChatError, LobbyClosedError, AlreadyJoinedError,
-                    NotEnoughPlayersError, DeckEmptyError)
+                    NotEnoughPlayersError, DeckEmptyError, PlayerLeftError)
 from utils import send_async, answer_async, error, TIMEOUT
 from shared_vars import botan, gm, updater, dispatcher
 from internationalization import _, __, user_locale, game_locales
@@ -55,7 +55,7 @@ logger = logging.getLogger(__name__)
 @user_locale
 def notify_me(bot, update):
     """Handler for /notify_me command, pm people for next game"""
-    chat_id = update.message.chat_id
+    chat_id = update.message.chat.id
     if update.message.chat.type == 'private':
         send_async(bot,
                    chat_id,
@@ -64,6 +64,11 @@ def notify_me(bot, update):
     else:
         try:
             gm.remind_dict[chat_id].add(update.message.from_user.id)
+            send_async(bot,
+                       chat_id,
+                       text=_("You will be notified "
+                          "when a new game is started in {title}.").format(
+                                title=update.message.chat.title))
         except KeyError:
             gm.remind_dict[chat_id] = {update.message.from_user.id}
 
@@ -71,7 +76,7 @@ def notify_me(bot, update):
 @user_locale
 def new_game(bot, update):
     """Handler for the /new command"""
-    chat_id = update.message.chat_id
+    chat_id = update.message.chat.id
 
     if update.message.chat.type == 'private':
         help(bot, update)
@@ -79,19 +84,20 @@ def new_game(bot, update):
     else:
 
         if update.message.chat_id in gm.remind_dict:
-            for user in gm.remind_dict[update.message.chat_id]:
+            for user in gm.remind_dict[chat_id]:
                 send_async(bot,
                            user,
-                           text=_("A new game has been started in {title}").format(
+                           text=_("A new game has been started in {title}.").format(
                                 title=update.message.chat.title))
 
-            del gm.remind_dict[update.message.chat_id]
+            del gm.remind_dict[chat_id]
 
         game = gm.new_game(update.message.chat)
         game.owner = update.message.from_user
         send_async(bot, chat_id,
-                   text=_("Created a new game! Join the game with /join "
+                   text=_("Created a new game! Wait for your friends "
                           "and start the game with /start"))
+        gm.join_game(update.message.from_user, update.message.chat)
 
         if botan:
             botan.track(update.message, 'New games')
@@ -162,7 +168,7 @@ def leave_game(bot, update):
                    reply_to_message_id=update.message.message_id)
 
     except NotEnoughPlayersError:
-        gm.end_game(chat, user)
+        gm.end_game(user, chat)
         send_async(bot, chat.id, text=__("Game ended!", multi=game.translate))
 
     else:
@@ -226,7 +232,7 @@ def status_update(bot, update):
         except NoGameInChatError:
             pass
         except NotEnoughPlayersError:
-            gm.end_game(chat, user)
+            gm.end_game(user, chat)
             send_async(bot, chat.id, text=__("Game ended!",
                                              multi=game.translate))
         else:
@@ -419,6 +425,47 @@ def disable_translations(bot, update):
         return
 
     
+@game_locales
+@user_locale
+def mode(bot, update):
+    """Handler for the /mode command"""
+    chat = update.message.chat
+    user = update.message.from_user
+    games = gm.chatid_games.get(chat.id)
+
+    if not games:
+        send_async(bot, chat.id,
+                   text=__("There is no running game in this chat."))
+        return
+
+    game = games[-1]
+    
+    if chat.type == 'private':
+        send_async(bot, chat.id,
+                   text=_("Please change the group mode in the public game group with "
+                          "the bot."))
+        return
+
+    if game.owner.id == user.id and not games.started:
+        kb = [["🎻 " + _("Original"), "🚴 " + _("Progressive UNO")]]
+        markup = ReplyKeyboardMarkup(kb, resize_keyboard=True, one_time_keyboard=True)
+        choice = send_async(bot, chat.id, text=_("Choose the game mode:"), reply_markup = markup)
+        if choice[0] == "🎻":
+            game.mode = 0
+            send_async(bot, chat.id, text=_("Original rules will be used."))
+        else if choice[0] == "🚴":
+            game.mode = 1
+            send_async(bot, chat.id, text=_("Progressive UNO rules will be used."))
+        return
+
+    else:
+        send_async(bot, chat.id,
+                   text=_("Only the game creator ({name}) can do that when the game does not start")
+                   .format(name=game.owner.first_name),
+                   reply_to_message_id=update.message.message_id)
+        return
+
+    
 @game_locales
 @user_locale
 def skip_player(bot, update):
@@ -475,7 +522,7 @@ def skip_player(bot, update):
         try:
             gm.leave_game(skipped_player.user, chat)
             send_async(bot, chat.id,
-                       text=__("{name1} was skipped four times in a row "
+                       text=__("{name1} was skipped three times in a row "
                                "and has been removed from the game.\n"
                                "Next player: {name2}", multi=game.translate)
                        .format(name1=display_name(skipped_player.user),
@@ -483,12 +530,17 @@ def skip_player(bot, update):
 
         except NotEnoughPlayersError:
             send_async(bot, chat.id,
-                       text=__("{name} was skipped four times in a row "
+                       text=__("{name} was skipped three times in a row "
                                "and has been removed from the game.\n"
                                "The game ended.", multi=game.translate)
                        .format(name=display_name(skipped_player.user)))
 
-            gm.end_game(chat.id, skipped_player.user)
+            us2 = UserSetting.get(id=skipped_player.user.id)
+            if us2 and us2.stats:
+                us2.games_played += 1
+                
+            gm.end_game(skipped_player.user, chat)
+           
 
 
 @game_locales
@@ -594,7 +646,13 @@ def process_result(bot, update):
     elif result_id == 'pass':
         game.turn()
     elif result_id in c.COLORS:
+        try:
             game.choose_color(result_id)
+        except PlayerLeftError:
+            send_async(bot, chat.id,
+                       text=__("There are errors in choosing color. "
+                               "Color is now unchanged.", multi=game.translate))
+            game.turn()
     else:
         reset_waiting_time(bot, player)
         do_play_card(bot, player, result_id)
@@ -609,10 +667,10 @@ def reset_waiting_time(bot, player):
     """Resets waiting time for a player and sends a notice to the group"""
     chat = player.game.chat
 
-    if player.waiting_time < 90:
-        player.waiting_time = 90
+    if player.waiting_time < 60:
+        player.waiting_time = 60
         send_async(bot, chat.id,
-                   text=__("Waiting time for {name} has been reset to 90 "
+                   text=__("Waiting time for {name} has been reset to 60 "
                            "seconds", multi=player.game.translate)
                    .format(name=display_name(player.user)))
 
@@ -661,7 +719,7 @@ def do_play_card(bot, player, result_id):
             if us2 and us2.stats:
                 us2.games_played += 1
 
-            gm.end_game(chat, user)
+            gm.end_game(user, chat)
 
     if botan:
         botan.track(Message(randint(1, 1000000000), user, datetime.now(),
@@ -694,9 +752,10 @@ def do_call_bluff(bot, player):
 
     if player.prev.bluffing:
         send_async(bot, chat.id,
-                   text=__("Bluff called! Giving 4 cards to {name}",
+                   text=__("Bluff called! Giving {numbers} cards to {name}",
                            multi=game.translate)
-                   .format(name=player.prev.user.first_name))
+                   .format(name=player.prev.user.first_name,
+                           numbers=game.draw_counter))
 
         try:
             player.prev.draw()
@@ -708,10 +767,11 @@ def do_call_bluff(bot, player):
     else:
         game.draw_counter += 2
         send_async(bot, chat.id,
-                   text=__("{name1} didn't bluff! Giving 6 cards to {name2}",
+                   text=__("{name1} didn't bluff! Giving {numbers} cards to {name2}",
                            multi=game.translate)
                    .format(name1=player.prev.user.first_name,
-                           name2=player.user.first_name))
+                           name2=player.user.first_name,
+                           numbers=game.draw_counter))
         try:
             player.draw()
         except DeckEmptyError:
@@ -738,6 +798,8 @@ dispatcher.add_handler(CommandHandler('disable_translations',
                                       disable_translations))
 dispatcher.add_handler(CommandHandler('skip', skip_player))
 dispatcher.add_handler(CommandHandler('notify_me', notify_me))
+dispatcher.add_handler(CommandHandler('mode', mode))
+dispatcher.add_handler(RegexHandler('^(Original|Progressive UNO)$', mode, pass_groups=True))
 simple_commands.register()
 settings.register()
 dispatcher.add_handler(MessageHandler([Filters.status_update], status_update))

errors.py

@@ -36,3 +36,7 @@ class NotEnoughPlayersError(Exception):
 
 class DeckEmptyError(Exception):
     pass
+
+
+class PlayerLeftError(Exception):
+    pass

game.py

@@ -2,7 +2,7 @@
 # -*- coding: utf-8 -*-
 #
 # Telegram bot to play UNO in group chats
-# Copyright (c) 2016 Jannes Höke <uno@jhoeke.de>
+# Copyright (c) 2016 - 2017 Jannes Höke <uno@jhoeke.de> and Karho Yau
 #
 # This program is free software: you can redistribute it and/or modify
 # it under the terms of the GNU Affero General Public License as
@@ -36,10 +36,12 @@ class Game(object):
     open = True
     translate = False
     players_won = 0
+    mode = 0
 
     def __init__(self, chat):
         self.chat = chat
         self.last_card = None
+        self.joined_before = set()
 
         while not self.last_card or self.last_card.special:
             self.deck = Deck()

game_manager.py

@@ -19,6 +19,7 @@
 
 
 import logging
+import random
 
 from game import Game
 from player import Player
@@ -77,7 +78,8 @@ class GameManager(object):
         # Don not re-add a player and remove the player from previous games in
         # this chat, if he is in one of them
         for player in players:
-            if player in game.players:
+            # Try to pervent someone win or leave then join again.
+            if player in game.players or user.id in game.joined_before: 
                 raise AlreadyJoinedError()
         else:
             try:
@@ -94,7 +96,13 @@ class GameManager(object):
 
         player = Player(game, user)
         
+        # Randomize player position.
+        game.joined_before.append(user.id)
+        if len(players) > 2:
+          players.insert(random.randrange(len(players)), player)
+        else:
           players.append(player)
+                       
         self.userid_current[user.id] = player
 
     def leave_game(self, user, chat):
@@ -135,7 +143,7 @@ class GameManager(object):
                 del self.userid_current[user.id]
                 del self.userid_players[user.id]
 
-    def end_game(self, chat, user):
+    def end_game(self, user, chat):
         """
         End a game
         """

locales/es_ES/LC_MESSAGES/unobot.po

@@ -107,29 +107,28 @@ msgstr ""
 msgid ""
 "Created a new game! Join the game with /join and start the game with /start"
 msgstr ""
-"¡Partida nueva creada! Únete a la partida con /join y comienza la partida "
-"con /start "
+"¡Nueva partida creada! Únete con /join. Cuando estéis todos listos, haz /start para empezar."
 
 #: bot.py:152
 msgid "The lobby is closed"
-msgstr "La sala esta cerrada"
+msgstr "La sala está cerrada."
 
 #: bot.py:156
 msgid "No game is running at the moment. Create a new game with /new"
 msgstr ""
-"No hay ninguna partida activa por el momento. Crea una partida con /new"
+"No hay ninguna partida en curso. Crea una partida con /new."
 
 #: bot.py:162
 msgid "You already joined the game. Start the game with /start"
-msgstr "Ya te uniste a la partida. Comienza la partida con /start"
+msgstr "Ya te uniste a esta partida. Comienza la partida con /start."
 
 #: bot.py:167
 msgid "Joined the game"
-msgstr "Te uniste a la partida"
+msgstr "Te uniste a la partida."
 
 #: bot.py:179 bot.py:191
 msgid "You are not playing in a game in this group."
-msgstr "No estas jugando una partida en este grupo."
+msgstr "No estas jugando a una partida en este grupo."
 
 #: bot.py:197 bot.py:258 bot.py:595
 msgid "Game ended!"
@@ -145,7 +144,7 @@ msgstr "Partida no encontrada."
 
 #: bot.py:223
 msgid "Back to last group"
-msgstr "De vuelta al último grupo"
+msgstr "De vuelta al último grupo."
 
 #: bot.py:227
 msgid "Please switch to the group you selected!"
@@ -163,20 +162,20 @@ msgstr ""
 #: bot.py:260
 #, python-format
 msgid "Removing {name} from the game"
-msgstr "Removiendo a {name} de la partida"
+msgstr "Eliminando a {name} de la partida."
 
 #: bot.py:273
 msgid "There is no game running in this chat. Create a new one with /new"
-msgstr "No hay ninguna partida en curso en este chat. Crea una nueva con /new"
+msgstr "No hay ninguna partida en curso en este chat. Crea una nueva con /new."
 
 #: bot.py:278
 msgid "The game has already started"
-msgstr "La partida ya ha comenzado"
+msgstr "La partida ya ha comenzado."
 
 #: bot.py:281
 msgid "At least two players must /join the game before you can start it"
 msgstr ""
-"Antes de iniciar la partida, al menos dos jugadores deben darle al /join"
+"Antes de iniciar la partida, al menos dos jugadores deben unirse usando /join."
 
 #: bot.py:297
 #, python-format
@@ -187,7 +186,7 @@ msgid ""
 msgstr ""
 "Primer Jugador: {name}\n"
 "Usa /close para evitar que la gente se una a la partida.\n"
-"Habilita multi-traducciones con /enable_translations"
+"Habilita multi-traducciones con /enable_translations."
 
 #: bot.py:321
 msgid "Please select the group you want to play in."
@@ -195,11 +194,11 @@ msgstr "Elige el grupo en el que deseas jugar."
 
 #: bot.py:335 bot.py:361
 msgid "There is no running game in this chat."
-msgstr "No hay ninguna partida ejecutándose en este chat."
+msgstr "No hay ninguna partida en curso en este grupo."
 
 #: bot.py:342
 msgid "Closed the lobby. No more players can join this game."
-msgstr "Sala cerrada. No se pueden unir más jugadores a la partida."
+msgstr "Sala cerrada. No podrán unirse más jugadores a la partida."
 
 #: bot.py:348 bot.py:373
 #, python-format
@@ -210,7 +209,7 @@ msgstr "Solo el creador de la partida ({name}) puede hacer eso."
 #, python-format
 msgid "Enabled multi-translations. Disable with /disable_translations"
 msgstr ""
-"Multi-traducciones habilitadas. Deshabilítalas con /disable_translations"
+"Multi-traducciones habilitadas. Deshabilítalas con /disable_translations."
 
 #: bot.py:377
 #, python-format
@@ -218,11 +217,11 @@ msgid ""
 "Disabled multi-translations. Enable them again with /enable_translations"
 msgstr ""
 "Multi-traducciones deshabilitadas. Habilítalas de nuevo con /"
-"enable_translations"
+"enable_translations."
 
 #: bot.py:368
 msgid "Opened the lobby. New players may /join the game."
-msgstr "Sala abierta. Nuevos jugadores darle al /join."
+msgstr "Sala abierta. Ahora pueden unirse nuevos jugadores usando /join."
 
 #: bot.py:386
 msgid "You are not playing in a game in this chat."
@@ -232,8 +231,8 @@ msgstr "No estás jugando una partida en este chat."
 #, python-format
 msgid "Please wait {time} second"
 msgid_plural "Please wait {time} seconds"
-msgstr[0] "Por favor, espere  {time} segundo"
-msgstr[1] "Por favor, espere  {time} segundos"
+msgstr[0] "Por favor, espere un segundo."
+msgstr[1] "Por favor, espere {time} segundos."
 
 #: bot.py:413
 #, python-format
@@ -256,7 +255,7 @@ msgid ""
 "{name1} was skipped four times in a row and has been removed from the game.\n"
 "Next player: {name2}"
 msgstr ""
-"{name1} fue saltado cuatro veces consecutivas y ha sido removido de la "
+"{name1} fue saltado cuatro veces consecutivas y ha sido eliminado de la "
 "partida.\n"
 "Siguiente Jugador {name2}"
 
@@ -266,7 +265,7 @@ msgid ""
 "{name} was skipped four times in a row and has been removed from the game.\n"
 "The game ended."
 msgstr ""
-"{name} fue saltado cuatro veces consecutivas y ha sido removido de la "
+"{name} fue saltado cuatro veces consecutivas y ha sido eliminado de la "
 "partida.\n"
 "Partida finalizada."
 
@@ -282,11 +281,11 @@ msgstr "Partida actual: {group}"
 #: bot.py:545
 #, python-format
 msgid "Cheat attempt by {name}"
-msgstr "Intento de trampa por {name}"
+msgstr "{name} intentó hacer trampa."
 
 #: bot.py:562
 msgid "Next player: {name}"
-msgstr "Siguiente Jugador: {name}"
+msgstr "Siguiente Jugador: {name}."
 
 #: bot.py:572
 #, python-format
@@ -309,12 +308,12 @@ msgstr "No hay más cartas en la baraja."
 #: bot.py:627
 #, python-format
 msgid "Bluff called! Giving 4 cards to {name}"
-msgstr "¡Engaño descubierto! {name} recibe 4 cartas"
+msgstr "¡Era un farol! {name} recibe 4 cartas."
 
 #: bot.py:639
 #, python-format
 msgid "{name1} didn't bluff! Giving 6 cards to {name2}"
-msgstr "¡{name1} no ha engañado!,  {name2} recibe 6 cartas"
+msgstr "¡{name1} no se había tirado un farol!, {name2} recibe 6 cartas."
 
 #: results.py:38
 msgid "Choose Color"
@@ -323,16 +322,16 @@ msgstr "Selecciona un color"
 #: results.py:56
 msgid "Last card (tap for game state):"
 msgid_plural "Cards (tap for game state):"
-msgstr[0] "Última carta (presiona para el estado de la partida):"
-msgstr[1] "Cartas (presiona para el estado de la partida):"
+msgstr[0] "Última carta (pulsa para ver el estado de la partida):"
+msgstr[1] "Cartas (pulsa para ver el estado de la partida):"
 
 #: results.py:60 results.py:123 results.py:165
 msgid "Current player: {name}"
-msgstr "Jugador actual: {name}"
+msgstr "Jugador actual: {name}."
 
 #: results.py:61 results.py:124 results.py:167
 msgid "Last card: {card}"
-msgstr "Última carta: {card}"
+msgstr "Última carta: {card}."
 
 #: results.py:62 results.py:125 results.py:168
 msgid "Player: {player_list}"
@@ -349,30 +348,30 @@ msgstr[1] "{name} ({number} cartas)"
 
 #: results.py:81
 msgid "You are not playing"
-msgstr "No estás jugando"
+msgstr "No estás jugando."
 
 #: results.py:83
 msgid ""
 "Not playing right now. Use /new to start a game or /join to join the current "
 "game in this group"
 msgstr ""
-"No estas jugando en este momento. Usa /new para comenzar una partida o /"
-"join para unirte a la partida actual en este grupo"
+"No estás jugando en este momento. Usa /new para comenzar una partida o /"
+"join si hay una partida en curso en este grupo."
 
 #: results.py:95
 msgid "The game wasn't started yet"
-msgstr "La partida no ha comenzado todavía"
+msgstr "La partida no ha comenzado todavía."
 
 #: results.py:97
 msgid "Start the game with /start"
-msgstr "Comenzar la partida con /start"
+msgstr "Comienza la partida con /start."
 
 #: results.py:108
 #, python-format
 msgid "Drawing {number} card"
 msgid_plural "Drawing {number} cards"
-msgstr[0] "Robando {number} carta"
-msgstr[1] "Robando {number} cartas"
+msgstr[0] "Robando {number} carta."
+msgstr[1] "Robando {number} cartas."
 
 #: results.py:136
 msgid "Pass"
@@ -380,7 +379,7 @@ msgstr "Pasar"
 
 #: results.py:148
 msgid "I'm calling your bluff!"
-msgstr "Me estas engañando!"
+msgstr "¡Es un farol!"
 
 #: settings.py:39
 msgid "Please edit your settings in a private chat with the bot."
@@ -404,45 +403,45 @@ msgstr "Ajustes"
 
 #: settings.py:68
 msgid "Enabled statistics!"
-msgstr "Estadísticas habilitadas!"
+msgstr "¡Estadísticas habilitadas!"
 
 #: settings.py:70
 msgid "Select locale"
-msgstr "Seleccionar región"
+msgstr "Selecciona tu idioma"
 
 #: settings.py:81
 msgid "Deleted and disabled statistics!"
-msgstr "Estadísticas borradas y deshabilitadas! "
+msgstr "¡Estadísticas borradas y deshabilitadas! "
 
 #: settings.py:94
 msgid "Set locale!"
-msgstr "Región seleccionada!"
+msgstr "¡Idioma seleccionada!"
 
 #: simple_commands.py
 msgid ""
 "You did not enable statistics. Use /settings in a private chat with the bot "
 "to enable them."
 msgstr ""
-"No has habilitado las estadísticas. Usa /settings en un chat privado con el "
+"Tus estadísticas no están habilitadas. Usa /settings en un chat privado con el "
 "bot para habilitarlas."
 
 #: simple_commands.py
 msgid "{number} game played"
 msgid_plural "{number} games played"
-msgstr[0] "{number} partida jugada"
-msgstr[1] "{number} partidas jugadas"
+msgstr[0] "{number} partida jugada."
+msgstr[1] "{number} partidas jugadas."
 
 #: simple_commands.py
 msgid "{number} first place"
 msgid_plural "{number} first places"
-msgstr[0] "{number} primero lugare"
-msgstr[1] "{number} primeros lugares"
+msgstr[0] "{number} primer lugar."
+msgstr[1] "{number} primeros lugares."
 
 #: simple_commands.py
 msgid "{number} card played"
 msgid_plural "{number} cards played"
-msgstr[0] "{number} carta jugada"
-msgstr[1] "{number} cartas jugadas"
+msgstr[0] "{number} carta jugada."
+msgstr[1] "{number} cartas jugadas."
 
 #: utils.py
 msgid "{emoji} Green"

player.py

@@ -2,7 +2,7 @@
 # -*- coding: utf-8 -*-
 #
 # Telegram bot to play UNO in group chats
-# Copyright (c) 2016 Jannes Höke <uno@jhoeke.de>
+# Copyright (c) 2016 - 2017 Jannes Höke <uno@jhoeke.de> and Karho Yau
 #
 # This program is free software: you can redistribute it and/or modify
 # it under the terms of the GNU Affero General Public License as
@@ -63,7 +63,7 @@ class Player(object):
         self.drew = False
         self.anti_cheat = 0
         self.turn_started = datetime.now()
-        self.waiting_time = 90
+        self.waiting_time = 60
 
     def leave(self):
         """Removes player from the game and closes the gap in the list"""
@@ -160,14 +160,15 @@ class Player(object):
 
         is_playable = True
         last = self.game.last_card
+        mode = self.game.mode
         self.logger.debug("Checking card " + str(card))
 
+        if mode == 0: # This mode is to apply the original rule
             if (card.color != last.color and card.value != last.value and
                 not card.special):
                 self.logger.debug("Card's color or value doesn't match")
                 is_playable = False
-        elif last.value == c.DRAW_TWO and not \
-                card.value == c.DRAW_TWO and self.game.draw_counter:
+            elif last.value == c.DRAW_TWO and self.game.draw_counter:
                 self.logger.debug("Player has to draw and can't counter")
                 is_playable = False
             elif last.special == c.DRAW_FOUR and self.game.draw_counter:
@@ -177,7 +178,32 @@ class Player(object):
                 (card.special == c.CHOOSE or card.special == c.DRAW_FOUR):
                 self.logger.debug("Can't play colorchooser on another one")
                 is_playable = False
-        elif not last.color:
+            # Prevent game being locked by choosing colors.
+            # When player is going to leave and he doesn't select a color, it causes game lock.
+            elif not last.color and (last.special != c.CHOOSE and last.special != c.DRAW_FOUR):
+                self.logger.debug("Last card has no color")
+                is_playable = False
+
+        elif mode == 1: # This mode is to apply the Progressive UNO rule.
+            if (card.color != last.color and card.value != last.value and
+                not card.special):
+                self.logger.debug("Card's color or value doesn't match")
+                is_playable = False
+            elif last.value == c.DRAW_TWO and self.game.draw_counter and not \
+                card.value == c.DRAW_TWO:
+                self.logger.debug("Player has to draw and can't counter")
+                is_playable = False
+            elif last.special == c.DRAW_FOUR and self.game.draw_counter and not \
+                card.special == c.DRAW_FOUR:
+                self.logger.debug("Player has to draw and can't counter")
+                is_playable = False
+            elif  (last.special == c.CHOOSE and (card.special == c.CHOOSE or card.special == c.DRAW_FOUR)) or \
+                (last.special == c.DRAW_FOUR and card.special == c.CHOOSE):
+                self.logger.debug("Can't play colorchooser on another one")
+                is_playable = False
+            # Prevent game being locked by choosing colors.
+            # When player is going to leave and he doesn't select a color, it causes game lock.
+            elif not last.color and (last.special != c.CHOOSE and last.special != c.DRAW_FOUR):
                 self.logger.debug("Last card has no color")
                 is_playable = False
 

test_player.py

@@ -0,0 +1,202 @@
+#!/usr/bin/env python3
+# -*- coding: utf-8 -*-
+#
+# Telegram bot to play UNO in group chats
+# Copyright (c) 2016 Jannes Höke <uno@jhoeke.de>
+#
+# This program is free software: you can redistribute it and/or modify
+# it under the terms of the GNU Affero General Public License as
+# published by the Free Software Foundation, either version 3 of the
+# License, or (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU Affero General Public License for more details.
+#
+# You should have received a copy of the GNU Affero General Public License
+# along with this program. If not, see <http://www.gnu.org/licenses/>.
+
+
+import unittest
+
+import telegram
+from game import Game
+from player import Player
+import card as c
+
+import logging
+logging.basicConfig(
+    format='%(asctime)s - %(name)s - %(levelname)s - %(message)s',
+    level=logging.DEBUG)
+logger = logging.getLogger(__name__)
+
+
+class Test(unittest.TestCase):
+
+    game = None
+
+    def setUp(self):
+        self.game = Game(None)
+
+    def test_insert(self):
+        p0 = Player(self.game, "Player 0")
+        p1 = Player(self.game, "Player 1")
+        p2 = Player(self.game, "Player 2")
+
+        self.assertEqual(p0, p2.next)
+        self.assertEqual(p1, p0.next)
+        self.assertEqual(p2, p1.next)
+
+        self.assertEqual(p0.prev, p2)
+        self.assertEqual(p1.prev, p0)
+        self.assertEqual(p2.prev, p1)
+
+    def test_reverse(self):
+        p0 = Player(self.game, "Player 0")
+        p1 = Player(self.game, "Player 1")
+        p2 = Player(self.game, "Player 2")
+        self.game.reverse()
+        p3 = Player(self.game, "Player 3")
+
+        self.assertEqual(p0, p3.next)
+        self.assertEqual(p1, p2.next)
+        self.assertEqual(p2, p0.next)
+        self.assertEqual(p3, p1.next)
+
+        self.assertEqual(p0, p2.prev)
+        self.assertEqual(p1, p3.prev)
+        self.assertEqual(p2, p1.prev)
+        self.assertEqual(p3, p0.prev)
+
+    def test_leave(self):
+        p0 = Player(self.game, "Player 0")
+        p1 = Player(self.game, "Player 1")
+        p2 = Player(self.game, "Player 2")
+
+        p1.leave()
+
+        self.assertEqual(p0, p2.next)
+        self.assertEqual(p2, p0.next)
+
+    def test_draw(self):
+        p = Player(self.game, "Player 0")
+
+        deck_before = len(self.game.deck.cards)
+        top_card = self.game.deck.cards[-1]
+
+        p.draw()
+
+        self.assertEqual(top_card, p.cards[-1])
+        self.assertEqual(deck_before, len(self.game.deck.cards) + 1)
+
+    def test_draw_two(self):
+        p = Player(self.game, "Player 0")
+
+        deck_before = len(self.game.deck.cards)
+        self.game.draw_counter = 2
+
+        p.draw()
+
+        self.assertEqual(deck_before, len(self.game.deck.cards) + 2)
+
+    def test_playable_cards_simple(self):
+        p = Player(self.game, "Player 0")
+
+        self.game.last_card = c.Card(c.RED, '5')
+
+        p.cards = [c.Card(c.RED, '0'), c.Card(c.RED, '5'), c.Card(c.BLUE, '0'),
+                   c.Card(c.GREEN, '5'), c.Card(c.GREEN, '8')]
+
+        expected = [c.Card(c.RED, '0'), c.Card(c.RED, '5'),
+                    c.Card(c.GREEN, '5')]
+
+        self.assertListEqual(p.playable_cards(), expected)
+
+    def test_playable_cards_on_draw_two(self):
+        p = Player(self.game, "Player 0")
+
+        self.game.last_card = c.Card(c.RED, c.DRAW_TWO)
+        self.game.draw_counter = 2
+
+        p.cards = [c.Card(c.RED, c.DRAW_TWO), c.Card(c.RED, '5'),
+                   c.Card(c.BLUE, '0'), c.Card(c.GREEN, '5'),
+                   c.Card(c.GREEN, c.DRAW_TWO)]
+
+        expected = [c.Card(c.RED, c.DRAW_TWO), c.Card(c.GREEN, c.DRAW_TWO)]
+
+        self.assertListEqual(p.playable_cards(), expected)
+        
+    def test_playable_cards_on_draw_two_then_four(self):
+        p = Player(self.game, "Player 0")
+
+        self.game.last_card = c.Card(c.RED, c.DRAW_TWO)
+        self.game.draw_counter = 2
+
+        p.cards = [c.Card(c.RED, c.DRAW_TWO), c.Card(c.RED, '5'),
+                   c.Card(c.BLUE, '0'), c.Card(c.GREEN, '5'),
+                   c.Card(c.GREEN, c.DRAW_TWO),
+                   c.Card(None, None, c.DRAW_FOUR)]
+
+        expected = [c.Card(c.RED, c.DRAW_TWO), c.Card(c.GREEN, c.DRAW_TWO), c.Card(None, None, c.DRAW_FOUR)]
+
+        self.assertListEqual(p.playable_cards(), expected)
+
+    def test_playable_cards_on_draw_four(self):
+        p = Player(self.game, "Player 0")
+
+        self.game.last_card = c.Card(c.RED, None, c.DRAW_FOUR)
+        self.game.draw_counter = 4
+
+        p.cards = [c.Card(c.RED, c.DRAW_TWO), c.Card(c.RED, '5'),
+                   c.Card(c.BLUE, '0'), c.Card(c.GREEN, '5'),
+                   c.Card(c.GREEN, c.DRAW_TWO),
+                   c.Card(None, None, c.DRAW_FOUR),
+                   c.Card(None, None, c.CHOOSE)]
+
+        expected = [c.Card(None, None, c.DRAW_FOUR)]
+        self.assertListEqual(p.playable_cards(), expected)
+        
+#     def test_playable_cards_on_draw_four_then_four(self):
+#         p = Player(self.game, "Player 0")
+
+#         self.game.last_card = c.Card(c.RED, None, c.DRAW_FOUR)
+#         self.game.draw_counter = 4
+
+#         p.cards = [c.Card(c.RED, c.DRAW_TWO), c.Card(c.RED, '5'),
+#                    c.Card(c.BLUE, '0'), c.Card(c.GREEN, '5'),
+#                    c.Card(c.GREEN, c.DRAW_TWO),
+#                    c.Card(None, None, c.DRAW_FOUR)]
+
+#         expected = [c.Card(None, None, c.DRAW_FOUR)]
+#         self.assertListEqual(p.playable_cards(), expected)
+
+    def test_bluffing(self):
+        p = Player(self.game, "Player 0")
+        Player(self.game, "Player 01")
+
+        self.game.last_card = c.Card(c.RED, '1')
+
+        p.cards = [c.Card(c.RED, c.DRAW_TWO), c.Card(c.RED, '5'),
+                   c.Card(c.BLUE, '0'), c.Card(c.GREEN, '5'),
+                   c.Card(c.RED, '5'), c.Card(c.GREEN, c.DRAW_TWO),
+                   c.Card(None, None, c.DRAW_FOUR),
+                   c.Card(None, None, c.CHOOSE)]
+
+        p.playable_cards()
+        self.assertTrue(p.bluffing)
+
+        p.cards = [c.Card(c.BLUE, '1'), c.Card(c.GREEN, '1'),
+                   c.Card(c.GREEN, c.DRAW_TWO),
+                   c.Card(None, None, c.DRAW_FOUR),
+                   c.Card(None, None, c.CHOOSE)]
+
+        p.playable_cards()
+
+        p.play(c.Card(None, None, c.DRAW_FOUR))
+        self.game.choose_color(c.GREEN)
+
+        self.assertFalse(self.game.current_player.prev.bluffing)
+
+if __name__ == '__main__':
+    unittest.main()