BlackJack Assignment submission.

[codeblahblah]

@jwo After changing the Card's to_s to show "QH" instead of queen-hearts"....
Should we change the Game#status to make @player_hand.cards and @delaer_hand.cards more readable?

[jwo]

Initial code looks fine, no issues there.

Should we change the Game#status to make @player_hand.cards and @delaer_hand.cards more readable?

I think it should return data; if you want to have a method, or a class, that formats it nicely, I think that's a good addition to the project.

[codeblahblah]

@jwo Okay. Will add that to the wishlist.
I'm stuck here : "If a player busts (goes over 21), the game should #standfor the player."
Should I be looking aroung Game#hit ?

[jwo]

Try on the player-hand.. if it's over 21 and hits, that's an exception and it shouldn't grab a card.

[codeblahblah]

@jwo I did this:

  def hit
   if @player_hand.value > 21
     stand
     puts "You bust! Dealer wins."
    else
      @player_hand.hit!(@deck)
    end
  end

I only got this far for the test case as I do not understand how you to make method call on a mock object which will change the properties thereof.

it "should stand for a player if he/she busts" do
      deck = double(:deck, :cards => [Card.new(:spades, 10), Card.new(:clubs, 9), Card.new(:clubs, 8), Card.new(:diamonds, 9), Card.new(:hearts, 6)])
      dealer_hand = Hand.new
      player_hand = Hand.new
      game = double(:game,
        :deck => deck,
        :player_hand => player_hand,
        :dealer_hand => dealer_hand )
      2.times { player_hand.hit!(deck) }
      2.times { dealer_hand.hit!(deck) }
      game.stub(:hit)
      player_hand.hit!(deck)
    end

[jwo]

Use a real Game object, with a mock card object.

At some point, the objects must be real.

[codeblahblah]

@jwo Understood. However, the Game object comes with a randomised Deck object which I cannot change/set to suit this specific scenario?

[jwo]

although that's the idea, set the game's deck to something where you have the cards predetermined.

On Tue, Feb 25, 2014 at 10:16 AM, drammopo [email protected]
wrote:

@jwo Understood. However, the Game object comes with a randomised Deck object which I cannot change/set to suit this specific scenario?

Reply to this email directly or view it on GitHub:
#40 (comment)

[codeblahblah]

@jwo There's no setter method - only @deck = Deck.new...should I create a backdoor just for testing?

[jwo]

With the following class:

class Deck

  attr_reader :cards
  def initialize
    @cards = 1..52 #build cards
  end
end

You could do:

describe Deck do
  it "has cards" do
    the_cards = [double, double]
    deck = Deck.new
    deck.stub(:cards) { the_cards}
    expect(deck.cards).to eq(the_cards)
  end
end

So, you're stubbing the #cards method on deck and forcing it to return a known state.

This only works if you are diligent in only using attr_reader methods and not @cards variables

[jwo]

(then you would pass in your Deck to Game, and you know what happens)