Broadcast the update of player's nickname

Author: cveneziani

app/controllers/players_controller.rb

@@ -5,7 +5,15 @@ def update
     return render json: { error: "Cannot update another user" }, status: 401 if @player != current_player
 
     @player.update(player_params)
-    render json: { status: :ok }, status: 200
+
+    payload = {
+      code:      "successful_player_nickname_update",
+      player_id: @player.id,
+      nickname:  @player.nickname
+    }
+
+    ActionCable.server.broadcast "game_#{current_player.game_id}", payload
+    head 200
   end
 
   private

app/views/games/show.html.erb

@@ -20,7 +20,7 @@
         <li class="life_bar"><%= render "games/hud/life" %></li>
       </ul>
       <div id="current_player" data-player-id="<%= current_player.id %>" data-player-nickname="<%= current_player.nickname %>"></div>
-      <%= form_for(current_player, url: player_path(current_player), class: "player") do |f|%>
+      <%= form_for(current_player, url: player_path(current_player), data: { remote: true }, class: "player") do |f| %>
         <%= f.text_field :nickname, id: 'current_player_nickname' %>
       <% end %>
       <ul id="life_opponent">

spec/requests/players_spec.rb

@@ -7,11 +7,26 @@
     context 'update current_player' do
       before :each do
         allow_any_instance_of(ApplicationController).to receive(:current_player).and_return(player)
+      end
+
+      it "returns 200 HTTP status" do
         put "/players/#{player.id}", params: { player: { nickname: "Zim" }}
+        expect(response).to have_http_status(200)
       end
 
-      it { expect(response).to have_http_status(200) }
-      it { expect(player.reload.nickname).to eq("Zim") }
+      it "updates player nickname" do
+        put "/players/#{player.id}", params: { player: { nickname: "Zim" }}
+        expect(player.reload.nickname).to eq("Zim")
+      end
+
+      it 'broadcasts a player nickname updated payload' do
+        allow(ActionCable.server).to receive(:broadcast)
+        put "/players/#{player.id}", params: { player: { nickname: "Zim" }}
+        expect(ActionCable.server).to have_received(:broadcast).with(
+          anything,
+          code: 'successful_player_nickname_update', player_id: player.id, nickname: 'Zim'
+        )
+      end
     end
 
     context 'update another player' do