chess 0.3.1 → 0.3.2

data/lib/chess/utf8_notation.rb

@@ -1,4 +1,6 @@
 module Chess
+  # rubocop:disable Style/AsciiComments
+
   # With this module is possible call the method {#to_utf8} on a string. This
   # method convert the chess piece identifier character into UTF8 chess
   # character.
@@ -34,6 +36,7 @@ module Chess
       self.gsub(/[PRNBQKprnbqk]/, UTF8_MAP)
     end
   end
+  # rubocop:enable Style/AsciiComments
 end
 
 String.prepend(Chess::UTF8Notation)

data/lib/chess/version.rb

@@ -1,5 +1,5 @@
 # The Chess library module.
 module Chess
   # The library version.
-  VERSION = '0.3.1'.freeze
+  VERSION = '0.3.2'.freeze
 end

data/test/test_big_pgn_collection.rb

@@ -11,7 +11,7 @@ class ChessTest < Minitest::Test
       define_method "test_big_pgn_#{filename}" do
         pgn = Chess::Pgn.new(path)
         game = Chess::Game.new(pgn.moves)
-        assert(game.checkmate?) if pgn.moves.last =~ /\#$/
+        assert(game.checkmate?) if pgn.moves.last.match?(/\#$/)
       end
     end
   end

data/test/test_checkmate.rb

@@ -9,11 +9,11 @@ class ChessTest < Minitest::Test
       game = Chess::Game.new(pgn.moves)
       assert(game.board.checkmate?)
       if file.include?('white_won')
-        assert_equal(game.result, '1-0')
-        assert_equal(game.active_player, :black)
+        assert_equal('1-0', game.result)
+        assert_equal(:black, game.active_player)
       elsif file.include?('black_won')
-        assert_equal(game.result, '0-1')
-        assert_equal(game.active_player, :white)
+        assert_equal('0-1', game.result)
+        assert_equal(:white, game.active_player)
       end
     end
   end

data/test/test_errors.rb

@@ -0,0 +1,22 @@
+require 'test_helper'
+
+class ChessTest < Minitest::Test
+  def test_bad_notation_error
+    game = Chess::Game.new
+    assert_raises(Chess::BadNotationError) do
+      game << 'gg'
+    end
+  end
+
+  def test_invalid_pgn_format_error
+    assert_raises(Chess::InvalidPgnFormatError) do
+      Chess::Pgn.new('test/pgn_collection/invalid/0001.pgn')
+    end
+  end
+
+  def test_invalid_fen_format_error
+    assert_raises(Chess::InvalidFenFormatError) do
+      Chess::Game.load_fen('invalid')
+    end
+  end
+end

data/test/test_fifty_rule_move.rb

@@ -3,10 +3,10 @@ require 'test_helper'
 class ChessTest < Minitest::Test
   TestHelper.pgns('fifty_move_rule').each do |file|
     name = File.basename(file, '.pgn')
-    define_method "test_fifty_rule_move_#{name}" do
+    define_method "test_fifty_move_rule_#{name}" do
       pgn = Chess::Pgn.new(file)
       game = Chess::Game.new(pgn.moves)
-      assert(game.board.fifty_rule_move?)
+      assert game.board.fifty_move_rule?
     end
   end
 end

data/test/test_game.rb

@@ -0,0 +1,82 @@
+require 'test_helper'
+
+class ChessTest < Minitest::Test
+  def test_moves
+    game = Chess::Game.new
+    game.moves = %w[e4 e5]
+    assert_equal %w[e4 e5], game.moves
+  end
+
+  def test_active_player
+    game = Chess::Game.new
+    game << 'a3'
+    assert_equal :black, game.active_player
+  end
+
+  def test_inactive_player
+    game = Chess::Game.new
+    game.moves = %w[a3 f5]
+    assert_equal :black, game.inactive_player
+  end
+
+  def test_status_white_won_resign
+    game = Chess::Game.new
+    game.resign(:black)
+    assert_equal :white_won_resign, game.status
+    assert game.over?
+    assert '1-0', game.result
+  end
+
+  def test_status_black_won_resign
+    game = Chess::Game.new
+    game << 'e4'
+    game.resign(:white)
+    assert_equal :black_won_resign, game.status
+    assert game.over?
+    assert '0-1', game.result
+  end
+
+  def test_status_insufficient_material
+    pgn = TestHelper.pick_pgn('insufficient_material/0001.pgn')
+    game = Chess::Game.new(pgn.moves)
+    assert_equal :insufficient_material, game.status
+  end
+
+  def test_status_fifty_move_rule
+    pgn = TestHelper.pick_pgn('fifty_move_rule/0001.pgn')
+    game = Chess::Game.new(pgn.moves)
+    game.draw
+    assert_equal :fifty_move_rule, game.status
+  end
+
+  def test_status_threefold_repetition
+    pgn = TestHelper.pick_pgn('threefold_repetition/0001.pgn')
+    game = Chess::Game.new(pgn.moves)
+    game.draw
+    assert_equal :threefold_repetition, game.status
+  end
+
+  def test_pgn
+    pgn = TestHelper.pick_pgn('valid/0001.pgn')
+    game = Chess::Game.new(pgn.moves)
+    expected_pgn = <<~PGN
+      [Event ""]
+      [Site ""]
+      [Date "??"]
+      [Round ""]
+      [White ""]
+      [Black ""]
+      [Result "*"]
+
+      1. e4 c5 2. Nf3 d6 3. d4 cxd4 4. Qxd4 Nc6 5. Bb5 Bd7 6. Bxc6 Bxc6 7. Bg5 Nf6
+      8. Bxf6 gxf6 9. Nc3 e6 10. O-O-O Be7 11. Rhe1 Rg8 12. Qe3 Rxg2 13. Rg1 Rg6 14.
+      Nd4 Qb6 15. h4 O-O-O 16. h5 Rg5 17. Nd5 Bxd5 18. exd5 e5 19. Qh3+ Kb8 20. Nf5
+      Bf8 21. Rxg5 fxg5 22. Ne3 Qb4 23. c4 g4 24. Qxg4 Bh6 25. Kb1 Rc8 26. Rc1 Qd2
+      27. Qf5 Bf4 28. Qc2 Qxc2+ 29. Rxc2 Rg8 30. b4 Rg5 31. c5 e4 32. c6 Rxh5 33.
+      Rc4 f5 34. b5 Rg5 35. Rc3 Be5 36. Rc1 Bd4 37. Nc4 Bc5 38. Ne5 dxe5 39. Rxc5
+      Rg6 40. a4 Rd6 41. a5 b6 42. axb6 axb6 43. Rc2 Rxd5 44. Rb2 h5 45. Ka2 h4 46.
+      Ka3 h3 47. Ka4 Rd1 *
+    PGN
+    assert_equal expected_pgn, game.pgn.to_s
+  end
+end

data/test/test_helper.rb

@@ -1,5 +1,15 @@
+require 'simplecov'
+SimpleCov.start do
+  add_filter 'lib/chess/gnuchess.rb'
+end
+if ENV['CODECOV'] == 'true'
+  require 'codecov'
+  SimpleCov.formatter = SimpleCov::Formatter::Codecov
+end
+
 require 'chess'
 require 'minitest/autorun'
+require 'byebug'
 
 module TestHelper
   PGN_COLLECTION = 'test/pgn_collection'.freeze
@@ -8,4 +18,9 @@ module TestHelper
   def self.pgns(path, prefix = PGN_COLLECTION)
     Dir[File.join(prefix, path, '**/*.pgn')]
   end
+
+  def self.pick_pgn(path, prefix = PGN_COLLECTION)
+    file = File.join(prefix, path)
+    return Chess::Pgn.new(file)
+  end
 end

data/test/test_insufficient_material.rb

@@ -17,15 +17,15 @@ class ChessTest < Minitest::Test
   FENS.each_with_index do |fen, i|
     define_method("test_insufficient_material_by_fen_#{i}") do
       game = Chess::Game.load_fen(fen)
-      assert(game.board.insufficient_material?)
+      assert game.board.insufficient_material?
     end
   end
 
   ONLY_KINGS_FENS.each_with_index do |fen, i|
     define_method("test_only_kings_by_fen_#{i}") do
       game = Chess::Game.load_fen(fen)
-      assert(game.board.insufficient_material?)
-      assert(game.board.only_kings?)
+      assert game.board.insufficient_material?
+      assert game.board.only_kings?
     end
   end
 
@@ -34,7 +34,7 @@ class ChessTest < Minitest::Test
     define_method "test_insufficient_material_#{name}" do
       pgn = Chess::Pgn.new(file)
       game = Chess::Game.new(pgn.moves)
-      assert(game.board.insufficient_material?)
+      assert game.board.insufficient_material?
     end
   end
 end

data/test/test_move_generator.rb

@@ -2,7 +2,7 @@ require 'test_helper'
 
 class ChessTest < Minitest::Test
   GENS = {
-    'r2qk3/8/2n5/8/8/8/p2B4/4K2R w K - 0 1' => { 'e1' => ['Kd1', 'Ke2', 'Kf2', 'Kf1', 'O-O'] },
+    'r2qk3/8/2n5/8/8/8/p2B4/4K2R w K - 0 1' => { 'e1' => %w[Kd1 Ke2 Kf2 Kf1 O-O] },
     'r2qk3/8/2n5/8/8/8/p2B4/4K2R w - - 0 1' => { 'e1' => %w[Kd1 Ke2 Kf2 Kf1] },
     'r2qk3/8/2n5/2b5/8/8/p2B4/4K2R w K - 0 1' => { 'e1' => %w[Kd1 Ke2 Kf1] },
     'r2qk3/8/2n5/2b5/8/8/p2B4/4K2R b K - 0 1' => { 'a2' => ['a1=Q'] },
@@ -12,7 +12,8 @@ class ChessTest < Minitest::Test
     '1B1k4/r3q3/2n1n3/2b2pP1/8/8/2nB4/3K3R b - - 0 1' => { 'c6' => %w[Nxb8 Ne5 Nc6d4 N6b4 Na5] },
     '1B1k4/r3q3/2n1n2P/2b5/5p2/3p1RP1/3B4/3K4 w - - 0 1' => { 'f3' => %w[Rxd3 Re3 Rxf4 Rf2 Rf1] },
     'k7/1Q6/2P5/8/8/8/8/3K4 b - - 0 1' => { 'a8' => [] },
-    'k7/6P1/8/8/8/3r4/3Q4/3K4 w - - 0 1' => { 'd2' => ['Qxd3'] }
+    'k7/6P1/8/8/8/3r4/3Q4/3K4 w - - 0 1' => { 'd2' => ['Qxd3'] },
+    'rnbqkbnr/1ppppppp/8/8/pP2P3/P7/2PP1PPP/RNBQKBNR w KQkq - 0 3' => { 'c2' => %w[c3 c4] }
   }.freeze
 
   GENS.each do |fen, generators|

data/test/test_pgn.rb

@@ -67,4 +67,39 @@ class ChessTest < Minitest::Test
     assert_nil pgn.result
     assert_equal 'Re6', pgn.moves.last
   end
+
+  def test_write_pgn
+    tempfile = Tempfile.new('test_pgn')
+    game = Chess::Game.new
+    game.moves = %w[e4 e5 d3]
+    pgn = game.pgn
+    pgn.event = 'Ruby Chess Tournament'
+    pgn.site = 'Ruby Chess test suite'
+    pgn.date = '1984.10.21'
+    pgn.round = '2'
+    pgn.white = 'Pioz'
+    pgn.black = 'Elizabeth Harmon'
+    pgn.write(tempfile.path)
+
+    loaded_pgn = Chess::Pgn.new
+    loaded_pgn.load(tempfile.path)
+    tempfile.delete
+    assert_equal 'Ruby Chess Tournament', loaded_pgn.event
+    assert_equal 'Ruby Chess test suite', loaded_pgn.site
+    assert_equal '1984.10.21', loaded_pgn.date
+    assert_equal '2', loaded_pgn.round
+    assert_equal 'Pioz', loaded_pgn.white
+    assert_equal 'Elizabeth Harmon', loaded_pgn.black
+    assert_equal '*', loaded_pgn.result
+    assert_equal %w[e4 e5 d3], loaded_pgn.moves
+  end
+
+  def test_set_date
+    pgn = Chess::Pgn.new
+    pgn.date = Time.parse('21-10-1984')
+    assert_equal '1984.10.21', pgn.date
+
+    pgn.date = '1984.10.21'
+    assert_equal '1984.10.21', pgn.date
+  end
 end

data/test/test_pgn_collection.rb

@@ -8,8 +8,8 @@ class ChessTest < Minitest::Test
       game = Chess::Game.new
       pgn.moves.each do |m|
         game.move(m)
-        assert(game.board.check?) if m =~ /\+$/
-        assert(game.board.checkmate?) if m =~ /\#$/
+        assert(game.board.check?) if m.match?(/\+$/)
+        assert(game.board.checkmate?) if m.match?(/\#$/)
       end
     end
     define_method "test_pgn_result_#{name}" do

data/test/test_stalemate.rb

@@ -7,7 +7,7 @@ class ChessTest < Minitest::Test
       pgn = Chess::Pgn.new(file)
       game = Chess::Game.new(pgn.moves)
       assert(game.board.stalemate?)
-      assert_equal(game.result, '1/2-1/2')
+      assert_equal('1/2-1/2', game.result)
     end
   end
 end

data/test/test_threefold_repetition.rb

@@ -6,7 +6,7 @@ class ChessTest < Minitest::Test
     define_method "test_threefold_repetition_#{name}" do
       pgn = Chess::Pgn.new(file)
       game = Chess::Game.new(pgn.moves)
-      assert(game.threefold_repetition?)
+      assert game.threefold_repetition?
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: chess
 version: !ruby/object:Gem::Version
-  version: 0.3.1
+  version: 0.3.2
 platform: ruby
 authors:
 - Enrico Pilotto
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-06-16 00:00:00.000000000 Z
+date: 2021-01-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -24,6 +24,34 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '2'
+- !ruby/object:Gem::Dependency
+  name: byebug
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '11'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '11'
+- !ruby/object:Gem::Dependency
+  name: codecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.4'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.4'
 - !ruby/object:Gem::Dependency
   name: minitest
   requirement: !ruby/object:Gem::Requirement
@@ -44,28 +72,42 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '12'
+        version: '13'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '12'
+        version: '13'
 - !ruby/object:Gem::Dependency
   name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.71'
+        version: '1'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.71'
+        version: '1'
+- !ruby/object:Gem::Dependency
+  name: rubocop-minitest
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.10'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.10'
 - !ruby/object:Gem::Dependency
   name: rubocop-performance
   requirement: !ruby/object:Gem::Requirement
@@ -80,6 +122,34 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1'
+- !ruby/object:Gem::Dependency
+  name: rubocop-rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.5'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.5'
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.21'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.21'
 - !ruby/object:Gem::Dependency
   name: yard
   requirement: !ruby/object:Gem::Requirement
@@ -102,13 +172,41 @@ extensions:
 - ext/extconf.rb
 extra_rdoc_files: []
 files:
+- ".github/FUNDING.yml"
+- ".github/workflows/ruby.yml"
 - ".gitignore"
 - ".rubocop.yml"
 - Gemfile
+- Gemfile.lock
 - LICENSE
 - README.md
 - Rakefile
 - chess.gemspec
+- docs/Chess.html
+- docs/Chess/BadNotationError.html
+- docs/Chess/Board.html
+- docs/Chess/CGame.html
+- docs/Chess/Game.html
+- docs/Chess/Gnuchess.html
+- docs/Chess/IllegalMoveError.html
+- docs/Chess/InvalidFenFormatError.html
+- docs/Chess/InvalidPgnFormatError.html
+- docs/Chess/Pgn.html
+- docs/Chess/UTF8Notation.html
+- docs/_index.html
+- docs/class_list.html
+- docs/css/common.css
+- docs/css/full_list.css
+- docs/css/style.css
+- docs/file.README.html
+- docs/file_list.html
+- docs/frames.html
+- docs/index.html
+- docs/js/app.js
+- docs/js/full_list.js
+- docs/js/jquery.js
+- docs/method_list.html
+- docs/top-level-namespace.html
 - ext/MakefileC
 - ext/bitboard.c
 - ext/bitboard.h
@@ -1339,7 +1437,9 @@ files:
 - test/pgn_collection/valid/1000.pgn
 - test/test_big_pgn_collection.rb
 - test/test_checkmate.rb
+- test/test_errors.rb
 - test/test_fifty_rule_move.rb
+- test/test_game.rb
 - test/test_helper.rb
 - test/test_illegal_moves.rb
 - test/test_insufficient_material.rb
@@ -1354,7 +1454,7 @@ homepage: https://github.com/pioz/chess
 licenses:
 - LGPLv3
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -1362,7 +1462,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.0'
+      version: '2.5'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
@@ -1370,7 +1470,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubygems_version: 3.0.2
-signing_key: 
+signing_key:
 specification_version: 4
 summary: A fast chess library to play chess with Ruby.
 test_files:
@@ -2583,7 +2683,9 @@ test_files:
 - test/pgn_collection/valid/1000.pgn
 - test/test_big_pgn_collection.rb
 - test/test_checkmate.rb
+- test/test_errors.rb
 - test/test_fifty_rule_move.rb
+- test/test_game.rb
 - test/test_helper.rb
 - test/test_illegal_moves.rb
 - test/test_insufficient_material.rb