finished kicking another user from a game

Author: GerwinV

lib/pickup_bot.rb

@@ -9,8 +9,6 @@ def run(message)
     begin
       handle_command(message)
     rescue => exception
-      puts 'gerwin test dit'
-      puts exception.message
       Rails.logger.error exception.message
       PickupBot::Commands::Error.run(telegram_bot, message, exception)
     end

lib/pickup_bot/commands/kick.rb

@@ -23,21 +23,22 @@ def current_game
       Game.active.find_by_chat_id(@message.chat.id)
     end
 
-    def kicked_player
-      Player.find_by(username: kick_username)
-    end
-
-    def user_attendance
-      Attendance.find_by(game: current_game, player: kicked_player )
-    end
-
     def destroy_user_attendance
       user_attendance.destroy
+
       @telegram_bot.api.send_message(
         chat_id: @message.chat.id,
         text: I18n.t("bot.kick_user")
       )
     end
 
+    def user_attendance
+       Attendance.find_by(game: current_game, player: kicked_player)
+    end
+
+    def kicked_player
+      Player.find_by(username: kick_username)
+    end
+
   end
 end

spec/lib/pickup_bot/commands/kick_game_spec.rb

@@ -7,28 +7,28 @@
   let(:pickup_bot) { PickupBot.new(bot) }
   let(:telegram_user) { Telegram::Bot::Types::User.new(user_params) }
   let(:telegram_chat) { Telegram::Bot::Types::Chat.new(chat_params) }
-  let(:other_player_username) { 'OneTrickPony' }
-  let(:message) { Telegram::Bot::Types::Message.new(message_params("/kick #{other_player_username}")) }
+  let(:message) { Telegram::Bot::Types::Message.new(message_params("/kick OneTrickPony")) }
 
   before :each do
     stub_request(:any, /api.telegram.org/).to_return(status: 200, body:"[]", :headers => {})
   end
 
   scenario "kick another user from a game" do
     game = Game.create(chat_id: fake_chat_id, required_players: 4)
-    other_player = Player.create(telegram_user_id: 12, username: other_player_username)
-    Attendance.create(game: game, player: other_player)
+    kicked_player = Player.create(telegram_user_id: 12, username: "OneTrickPony")
+    Attendance.create(game: game, player: kicked_player)
 
     pickup_bot.run(message)
 
     expect(a_request(:post, "https://api.telegram.org/botfake-token/sendMessage").
             with(body: {
-              "chat_id" => message.chat.id,
+              "chat_id" => "123",
               "text" => I18n.t(
                 "bot.kick_user"
               )
             }
           )).to have_been_made.times(1)
-  end
 
+      expect(Attendance.count).to eq(0)
+  end
 end