console-blackjack 1.0.7 → 1.1.0

data/spec/lib/blackjack_spec.rb

@@ -1,14 +1,11 @@
 # frozen_string_literal: true
 
 RSpec.describe Blackjack do
-  let(:shoe) { build(:shoe, :new_regular) }
-  let(:blackjack) { build(:blackjack, shoe: shoe) }
+  let(:blackjack) { build(:blackjack, shoe: build(:shoe, :new_regular)) }
   let(:player_hand) { build(:player_hand, blackjack: blackjack) }
   let(:dealer_hand) { build(:dealer_hand, blackjack: blackjack) }
-  let(:ace) { build(:card, :ace) }
-  let(:seven) { build(:card, :seven) }
-  let(:six) { build(:card, :six) }
-  let(:ten) { build(:card, :ten) }
+  let(:ace) { build(:card, :ace, blackjack: blackjack) }
+  let(:ten) { build(:card, :ten, blackjack: blackjack) }
 
   describe '#current_player_hand' do
     it 'returns the current hand' do
@@ -30,7 +27,7 @@ RSpec.describe Blackjack do
 
   describe '#getc' do
     it 'get a single character from stdin' do
-      allow(STDIN).to receive(:getc).and_return('q')
+      allow($stdin).to receive(:getc).and_return('q')
       c = described_class.getc
       expect(c).to eq('q')
     end
@@ -44,13 +41,24 @@ RSpec.describe Blackjack do
   end
 
   describe '#need_to_play_dealer_hand?' do
-    it 'returns false' do
-      expect(blackjack).to_not be_need_to_play_dealer_hand
+    before do
+      blackjack.player_hands << player_hand
     end
 
-    it 'returns true' do
-      blackjack.player_hands << player_hand
-      expect(blackjack).to be_need_to_play_dealer_hand
+    context 'when busted' do
+      it 'returns false' do
+        allow(player_hand).to receive(:blackjack?).and_return(false)
+        allow(player_hand).to receive(:busted?).and_return(true)
+        expect(blackjack).to_not be_need_to_play_dealer_hand
+      end
+    end
+
+    context 'when blackjack' do
+      it 'returns false' do
+        allow(player_hand).to receive(:busted?).and_return(false)
+        allow(player_hand).to receive(:blackjack?).and_return(true)
+        expect(blackjack).to_not be_need_to_play_dealer_hand
+      end
     end
   end
 
@@ -93,37 +101,56 @@ RSpec.describe Blackjack do
 
   describe '#save_game' do
     let(:file) { instance_double('File') }
-    let(:content) { "#{blackjack.num_decks}|#{blackjack.money}|#{blackjack.current_bet}" }
 
-    it 'opens and put save file data' do
+    before do
       allow(File).to receive(:open).with(SAVE_FILE, 'w').and_yield(file)
       allow(file).to receive(:puts)
+    end
+
+    it 'opens and put save file data' do
       blackjack.save_game
-      expect(file).to have_received(:puts).with(content)
+      fields = %i[num_decks deck_type face_type money current_bet]
+      expected = fields.map { |f| blackjack.send(f) }.join('|')
+      expect(file).to have_received(:puts).with(expected)
     end
   end
 
   describe '#load_game' do
-    let(:content) { '8|2000|1000' }
+    context 'with a unreadable save file' do
+      it 'fails to load save file' do
+        allow(File).to receive(:read).with(SAVE_FILE)
+        allow(File).to receive(:readable?).with(SAVE_FILE).and_return(false)
 
-    before do
-      allow(File).to receive(:readable?).with(SAVE_FILE).and_return(true)
-      allow(File).to receive(:read).with(SAVE_FILE).and_return(content)
+        blackjack.load_game
+        expect(File).to_not have_received(:read).with(SAVE_FILE)
+      end
     end
 
-    it 'loads num_decks from save file data' do
-      blackjack.load_game
-      expect(blackjack.num_decks).to eq(8)
-    end
+    context 'with a readabale save file' do
+      before do
+        allow(File).to receive(:readable?).with(SAVE_FILE).and_return(true)
+        allow(File).to receive(:read).with(SAVE_FILE).and_return('8|1|1|2000|1000')
+      end
 
-    it 'loads money from save file data' do
-      blackjack.load_game
-      expect(blackjack.money).to eq(2000)
-    end
+      it 'loads num_decks from save file data' do
+        blackjack.load_game
+        expect(blackjack.num_decks).to eq(8)
+      end
 
-    it 'loads current_bet from save file data' do
-      blackjack.load_game
-      expect(blackjack.current_bet).to eq(1000)
+      it 'loads face_type from save file data' do
+        blackjack.load_game
+        expect(blackjack.face_type).to eq(1)
+      end
+
+      it 'loads money from save file data' do
+        blackjack.load_game
+        expect(blackjack.money).to eq(2000)
+      end
+
+      it 'loads current_bet from save file data' do
+        blackjack.load_game
+        expect(blackjack.current_bet).to eq(1000)
+      end
     end
   end
 
@@ -257,19 +284,19 @@ RSpec.describe Blackjack do
         dealer_hand.cards << ten << ten
         allow(blackjack).to receive(:build_new_hand).and_return(player_hand)
         allow(player_hand).to receive(:action?)
-        allow(shoe).to receive(:new_regular)
+        allow(blackjack.shoe).to receive(:new_regular)
       end
 
       it 'shuffles' do
-        allow(shoe).to receive(:needs_to_shuffle?).and_return(true)
+        allow(blackjack.shoe).to receive(:needs_to_shuffle?).and_return(true)
         blackjack.deal_new_hand
-        expect(shoe).to have_received(:new_regular)
+        expect(blackjack.shoe).to have_received(:new_regular)
       end
 
       it 'does not shuffle' do
-        allow(shoe).to receive(:needs_to_shuffle?).and_return(false)
+        allow(blackjack.shoe).to receive(:needs_to_shuffle?).and_return(false)
         blackjack.deal_new_hand
-        expect(shoe).to_not have_received(:new_regular)
+        expect(blackjack.shoe).to_not have_received(:new_regular)
       end
     end
 
@@ -372,7 +399,7 @@ RSpec.describe Blackjack do
   describe '#new_bet' do
     before do
       blackjack.dealer_hand = dealer_hand
-      allow(STDIN).to receive(:gets).and_return('10')
+      allow($stdin).to receive(:gets).and_return('10')
       allow(described_class).to receive(:getc).and_return('s', 'q')
       allow(blackjack).to receive(:print)
       allow(blackjack).to receive(:puts)
@@ -430,7 +457,7 @@ RSpec.describe Blackjack do
 
       it 'draws the blackjack options' do
         blackjack.draw_game_options
-        expected = ' (N) Number of Decks  (T) Deck Type  (B) Back'
+        expected = ' (N) Number of Decks  (T) Deck Type  (F) Face Type  (B) Back'
         expect(blackjack).to have_received(:puts).with(expected)
       end
 
@@ -481,6 +508,37 @@ RSpec.describe Blackjack do
       end
     end
 
+    context 'when updating the face type' do
+      before do
+        allow(described_class).to receive(:getc).and_return('f')
+        allow(blackjack).to receive(:clear)
+        allow(blackjack).to receive(:draw_hands)
+        allow(blackjack).to receive(:new_face_type)
+        allow(blackjack).to receive(:draw_bet_options)
+        allow(blackjack).to receive(:puts)
+      end
+
+      it 'clears the screen' do
+        blackjack.draw_game_options
+        expect(blackjack).to have_received(:clear).twice
+      end
+
+      it 'draws the hands' do
+        blackjack.draw_game_options
+        expect(blackjack).to have_received(:draw_hands).twice
+      end
+
+      it 'updates face type' do
+        blackjack.draw_game_options
+        expect(blackjack).to have_received(:new_face_type)
+      end
+
+      it 'draws the bet options' do
+        blackjack.draw_game_options
+        expect(blackjack).to have_received(:draw_bet_options)
+      end
+    end
+
     context 'when going back to previous menu' do
       before do
         allow(described_class).to receive(:getc).and_return('b')
@@ -526,7 +584,7 @@ RSpec.describe Blackjack do
   describe '#new_num_decks' do
     before do
       blackjack.dealer_hand = dealer_hand
-      allow(STDIN).to receive(:gets).and_return('2')
+      allow($stdin).to receive(:gets).and_return('2')
       allow(described_class).to receive(:getc).and_return('b', 'q')
       allow(blackjack).to receive(:print)
       allow(blackjack).to receive(:puts)
@@ -581,6 +639,7 @@ RSpec.describe Blackjack do
     before do
       blackjack.dealer_hand = dealer_hand
       allow(blackjack).to receive(:puts)
+      allow(blackjack).to receive(:save_game)
     end
 
     context 'when choosing a new deck type' do
@@ -593,44 +652,50 @@ RSpec.describe Blackjack do
 
       it 'builds a new regular' do
         allow(described_class).to receive(:getc).and_return('1')
-        allow(shoe).to receive(:new_regular)
+        allow(blackjack.shoe).to receive(:new_regular)
         blackjack.new_deck_type
-        expect(shoe).to have_received(:new_regular)
+        expect(blackjack.shoe).to have_received(:new_regular)
+        expect(blackjack).to have_received(:save_game)
       end
 
       it 'builds a new aces' do
         allow(described_class).to receive(:getc).and_return('2')
-        allow(shoe).to receive(:new_aces)
+        allow(blackjack.shoe).to receive(:new_aces)
         blackjack.new_deck_type
-        expect(shoe).to have_received(:new_aces)
+        expect(blackjack.shoe).to have_received(:new_aces)
+        expect(blackjack).to have_received(:save_game)
       end
 
       it 'builds a new jacks' do
         allow(described_class).to receive(:getc).and_return('3')
-        allow(shoe).to receive(:new_jacks)
+        allow(blackjack.shoe).to receive(:new_jacks)
         blackjack.new_deck_type
-        expect(shoe).to have_received(:new_jacks)
+        expect(blackjack.shoe).to have_received(:new_jacks)
+        expect(blackjack).to have_received(:save_game)
       end
 
       it 'builds a new aces_jacks' do
         allow(described_class).to receive(:getc).and_return('4')
-        allow(shoe).to receive(:new_aces_jacks)
+        allow(blackjack.shoe).to receive(:new_aces_jacks)
         blackjack.new_deck_type
-        expect(shoe).to have_received(:new_aces_jacks)
+        expect(blackjack.shoe).to have_received(:new_aces_jacks)
+        expect(blackjack).to have_received(:save_game)
       end
 
       it 'builds a new sevens' do
         allow(described_class).to receive(:getc).and_return('5')
-        allow(shoe).to receive(:new_sevens)
+        allow(blackjack.shoe).to receive(:new_sevens)
         blackjack.new_deck_type
-        expect(shoe).to have_received(:new_sevens)
+        expect(blackjack.shoe).to have_received(:new_sevens)
+        expect(blackjack).to have_received(:save_game)
       end
 
       it 'builds a new eights' do
         allow(described_class).to receive(:getc).and_return('6')
-        allow(shoe).to receive(:new_eights)
+        allow(blackjack.shoe).to receive(:new_eights)
         blackjack.new_deck_type
-        expect(shoe).to have_received(:new_eights)
+        expect(blackjack.shoe).to have_received(:new_eights)
+        expect(blackjack).to have_received(:save_game)
       end
     end
 
@@ -640,13 +705,56 @@ RSpec.describe Blackjack do
         allow(blackjack).to receive(:draw_hands)
         blackjack.new_deck_type
         expect(blackjack).to have_received(:draw_hands)
+        expect(blackjack).to have_received(:save_game).twice
+      end
+    end
+  end
+
+  describe '#new_face_type' do
+    before do
+      blackjack.dealer_hand = dealer_hand
+      allow(blackjack).to receive(:puts)
+    end
+
+    context 'when choosing a new face type' do
+      it 'draws options' do
+        allow(described_class).to receive(:getc).and_return('1')
+        blackjack.new_face_type
+        expected = ' (1) 🂡  (2) A♠'
+        expect(blackjack).to have_received(:puts).with(expected)
+      end
+
+      it 'sets regular faces' do
+        allow(described_class).to receive(:getc).and_return('1')
+        allow(blackjack).to receive(:save_game)
+        allow(blackjack).to receive(:face_type=).with(1)
+        blackjack.new_face_type
+        expect(blackjack).to have_received(:face_type=).with(1)
+        expect(blackjack).to have_received(:save_game)
+      end
+
+      it 'sets alternate faces' do
+        allow(described_class).to receive(:getc).and_return('2')
+        allow(blackjack).to receive(:save_game)
+        allow(blackjack).to receive(:face_type=).with(2)
+        blackjack.new_face_type
+        expect(blackjack).to have_received(:face_type=).with(2)
+        expect(blackjack).to have_received(:save_game)
+      end
+    end
+
+    context 'when invalid input' do
+      it 'gets the action again' do
+        allow(described_class).to receive(:getc).and_return('x', '1')
+        allow(blackjack).to receive(:draw_hands)
+        blackjack.new_face_type
+        expect(blackjack).to have_received(:draw_hands)
       end
     end
   end
 
   describe '#ask_insurance' do
     before do
-      # blackjack.dealer_hand = dealer_hand
       allow(blackjack).to receive(:puts)
       allow(blackjack).to receive(:insure_hand)
       allow(blackjack).to receive(:no_insurance)
@@ -765,7 +873,7 @@ RSpec.describe Blackjack do
 
     context 'when dealer hand is not blackjack' do
       before do
-        dealer_hand.cards << ace << seven
+        dealer_hand.cards << ace << build(:card, :seven)
         allow(blackjack).to receive(:play_dealer_hand)
       end
 
@@ -822,7 +930,7 @@ RSpec.describe Blackjack do
 
     context 'when current hand can split' do
       before do
-        player_hand.cards << six << six
+        player_hand.cards << build(:card, :six) << build(:card, :six)
         allow(described_class).to receive(:getc).and_return('s', 's')
         allow(blackjack).to receive(:draw_bet_options)
         allow(blackjack).to receive(:draw_hands)
@@ -843,7 +951,7 @@ RSpec.describe Blackjack do
 
     context 'when current hand cannot split' do
       before do
-        player_hand.cards << ace << six
+        player_hand.cards << ace << build(:card, :six)
         allow(described_class).to receive(:getc).and_return('s')
         allow(blackjack).to receive(:draw_bet_options)
         allow(blackjack).to receive(:draw_hands)

data/spec/spec_helper.rb

@@ -3,14 +3,13 @@
 require 'simplecov'
 SimpleCov.start do
   add_filter %r{/spec/}
+  enable_coverage :branch
+  primary_coverage :branch
 end
 
-require 'coveralls'
-Coveralls.wear!
-
 Dir.glob(File.join(File.dirname(__FILE__),
-                   '..' + File::SEPARATOR + 'lib', '**',
-                   '*.rb'), &method(:require))
+                   "..#{File::SEPARATOR}lib", '**',
+                   '*.rb')).each(&method(:require))
 
 require 'factory_bot'
 require 'pry'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: console-blackjack
 version: !ruby/object:Gem::Version
-  version: 1.0.7
+  version: 1.1.0
 platform: ruby
 authors:
 - Greg Donald
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-05-02 00:00:00.000000000 Z
+date: 2022-03-26 00:00:00.000000000 Z
 dependencies: []
 description: Blackjack for your console, full version.
 email: gdonald@gmail.com
@@ -23,8 +23,10 @@ files:
 - README.md
 - Rakefile
 - bin/console-blackjack
-- bj.png
 - bj.txt
+- console-blackjack-1.0.7.gem
+- console-blackjack-1.0.8.gem
+- console-blackjack-1.0.9.gem
 - console-blackjack.gemspec
 - lib/blackjack.rb
 - lib/blackjack/card.rb
@@ -52,11 +54,14 @@ files:
 - spec/lib/blackjack/shoe_spec.rb
 - spec/lib/blackjack_spec.rb
 - spec/spec_helper.rb
+- ss1.png
+- ss2.png
 homepage: https://github.com/gdonald/console-blackjack-ruby
 licenses:
 - MIT
 metadata:
   source_code_uri: https://github.com/gdonald/console-blackjack-ruby
+  rubygems_mfa_required: 'true'
 post_install_message: |2+
 
   Type `console-blackjack` to run!
@@ -68,14 +73,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '3.0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.6
+rubygems_version: 3.2.32
 signing_key:
 specification_version: 4
 summary: Console Blackjack