Fix game finished results

- Say that user won when he won :D

Author: cveneziani

app/views/games/finished.html.erb

@@ -9,7 +9,7 @@
       </div>
     </div>
     <div class="too_late">
-      <% if @game.players.include?(current_player) && current_player.won == false %>
+      <% if @game.players.include?(current_player) && current_player.won %>
         <h1>Game Finished, Congratulations! You WON!</h1>
         <p>
           You're the master of the <strong><%= @game.name %></strong> room
@@ -29,7 +29,8 @@
           <strong><%= @game.players.last.nickname %></strong> had an epic fight deep in the space.
         </p>
       <% end %>
-        <p>In total, the battle involved <strong><%= @game.ships.count %> ships</strong> and <strong><%= @game.ships.sum(:damage) %> damages</strong>.</p>
+
+      <p>In total, the battle involved <strong><%= @game.ships.count %> ships</strong> and <strong><%= @game.ships.sum(:damage) %> damages</strong>.</p>
 
       <%= render 'components/social_links' %>
     </div>

spec/requests/games_spec.rb

@@ -4,7 +4,7 @@
   let(:game) { Game.create!(name: 'Starship Battle', slug: 'starship-battle') }
 
   describe "GET show" do
-    context "url is not parametrized" do
+    context "url is not parameterized" do
       before :each do
         get URI::encode("/games/#{game.name}")
       end
@@ -107,15 +107,42 @@
     end
 
     context "when game is finished" do
+      let(:rico)  { Player.create!(nickname: "Rico") }
+      let(:zim)   { Player.create!(nickname: "Zim") }
+
       before :each do
         game.update(state: "finished")
-        Player.create!(nickname: "Zim", game: game)
-        Player.create!(nickname: "Rico", game: game, won: true)
 
+        rico.update!(game: game, won: true)
+        zim.update!(game: game, won: false)
+      end
+
+      it 'displays that game is finished' do
         get "/games/#{game.to_param}"
+        expect(response.body).to include("finished")
+      end
+
+      context 'wen signed in player won the game' do
+        before :each do
+          allow_any_instance_of(ApplicationController).to receive(:current_player).and_return(rico)
+        end
+
+        it 'displays that user won the game' do
+          get "/games/#{game.to_param}"
+          expect(response.body).to include("WON")
+        end
       end
 
-      it { expect(response.body).to include("finished") }
+      context 'when signed in player lost the game' do
+        before :each do
+          allow_any_instance_of(ApplicationController).to receive(:current_player).and_return(zim)
+        end
+
+        it 'displays that user lost the game' do
+          get "/games/#{game.to_param}"
+          expect(response.body).to include("LOSE")
+        end
+      end
     end
   end