chess 0.2.0 → 0.3.2

data/test/test_illegal_moves.rb

@@ -1,14 +1,12 @@
 require 'test_helper'
 
 class ChessTest < Minitest::Test
-
   def test_illegal_moves
     game = Chess::Game.new
-    %w(Qf6 Rd4 Nc3=Q).each do |move|
+    %w[Qf6 Rd4 Nc3=Q].each do |move|
       assert_raises(Chess::IllegalMoveError) do
         game << move
       end
     end
   end
-
 end

data/test/test_insufficient_material.rb

@@ -7,25 +7,25 @@ class ChessTest < Minitest::Test
     '8/2k5/8/3b4/B7/8/2K5/8 w - - 0 1',
     '8/2k5/8/8/4B3/8/2K5/8 w - - 0 1',
     '8/2k5/8/5b2/8/8/2K5/8 w - - 0 1'
-  ]
+  ].freeze
 
   ONLY_KINGS_FENS = [
     '8/2k5/8/8/8/8/2K5/8 w - - 0 1',
     '8/4k3/8/8/1K6/8/8/8 w - - 0 1'
-  ]
+  ].freeze
 
   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,8 +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_load_fen.rb

@@ -1,7 +1,6 @@
 require 'test_helper'
 
 class ChessTest < Minitest::Test
-
   def test_fen_in_progress
     g = Chess::Game.load_fen('rnbqkbnr/pp1ppppp/8/2p5/4P3/5N2/PPPP1PPP/RNBQKB1R b KQkq - 1 2')
     assert_equal 'P', g.board['e4']
@@ -38,4 +37,15 @@ class ChessTest < Minitest::Test
     assert_equal :stalemate, g.status
   end
 
+  def test_fen_castling_from
+    g = Chess::Game.load_fen('2b1kbnr/rpq1pp1p/2n3p1/8/3Q4/2P5/PP3PPP/RN2KBNR w KQk - 0 9')
+    g.move('Kd1')
+    assert_equal '2b1kbnr/rpq1pp1p/2n3p1/8/3Q4/2P5/PP3PPP/RN1K1BNR b k - 1 9', g.board.to_fen
+  end
+
+  def test_fen_castling_to
+    g = Chess::Game.load_fen('2b1kbnr/rpq1pp1p/2n3p1/8/3Q4/2P5/PP3PPP/RN2KBNR w KQk - 0 9')
+    g.move('Qh8')
+    assert_equal '2b1kbnQ/rpq1pp1p/2n3p1/8/8/2P5/PP3PPP/RN2KBNR b KQ - 0 9', g.board.to_fen
+  end
 end

data/test/test_move_generator.rb

@@ -1,20 +1,20 @@
 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 - - 0 1' => {'e1' => ['Kd1', 'Ke2', 'Kf2', 'Kf1']},
-    'r2qk3/8/2n5/2b5/8/8/p2B4/4K2R w K - 0 1' => {'e1' => ['Kd1', 'Ke2', 'Kf1']},
-    'r2qk3/8/2n5/2b5/8/8/p2B4/4K2R b K - 0 1' => {'a2' => ['a1=Q']},
-    'r2qk3/8/2n1n3/2b5/8/8/p2B4/4K2R b K - 0 1' => {'e6' => ['Nf8', 'Ng7', 'Ng5', 'Nf4', 'Ned4', 'Nc7']},
-    'r2qk3/8/2n5/2b2pP1/8/8/3B4/4K2R w K f6 0 1' => {'g5' => ['gxf6ep', 'g6']},
-    '1B1k4/r3q3/2n1n3/2b2pP1/8/1n6/3B4/4K2R b K - 0 1' => {'c6' => ['Nxb8', 'Ne5', 'Ncd4', 'Nb4', 'Nca5']},
-    '1B1k4/r3q3/2n1n3/2b2pP1/8/8/2nB4/3K3R b - - 0 1' => {'c6' => ['Nxb8', 'Ne5', 'Nc6d4', 'N6b4', 'Na5']},
-    '1B1k4/r3q3/2n1n2P/2b5/5p2/3p1RP1/3B4/3K4 w - - 0 1' => {'f3' => ['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']}
-  }
+    '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'] },
+    'r2qk3/8/2n1n3/2b5/8/8/p2B4/4K2R b K - 0 1' => { 'e6' => %w[Nf8 Ng7 Ng5 Nf4 Ned4 Nc7] },
+    'r2qk3/8/2n5/2b2pP1/8/8/3B4/4K2R w K f6 0 1' => { 'g5' => %w[gxf6ep g6] },
+    '1B1k4/r3q3/2n1n3/2b2pP1/8/1n6/3B4/4K2R b K - 0 1' => { 'c6' => %w[Nxb8 Ne5 Ncd4 Nb4 Nca5] },
+    '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'] },
+    'rnbqkbnr/1ppppppp/8/8/pP2P3/P7/2PP1PPP/RNBQKBNR w KQkq - 0 3' => { 'c2' => %w[c3 c4] }
+  }.freeze
 
   GENS.each do |fen, generators|
     define_method("test_move_generator_#{fen}") do
@@ -25,5 +25,4 @@ class ChessTest < Minitest::Test
       end
     end
   end
-
 end

data/test/test_particular_situations.rb

@@ -1,7 +1,6 @@
 require 'test_helper'
 
 class ChessTest < Minitest::Test
-
   def test_cant_castling
     game = Chess::Game.load_fen('2r1k2r/pR2p2p/2pn2Nb/P2p1p2/8/1P2P3/2PP4/2BQK3 b k - 1 3')
     assert_raises(Chess::IllegalMoveError) do
@@ -10,5 +9,4 @@ class ChessTest < Minitest::Test
     game = Chess::Game.load_fen('2r1k2r/pR2p2p/2pn3b/P2p1p2/8/1P2P3/2PP4/2BQK3 b k - 1 3')
     game << 'O-O'
   end
-
 end

data/test/test_pgn.rb

@@ -1,7 +1,6 @@
 require 'test_helper'
 
 class ChessTest < Minitest::Test
-
   TestHelper.pgns('invalid').each do |file|
     name = File.basename(file, '.pgn')
     define_method "test_invalid_pgn_#{name}" do
@@ -21,4 +20,86 @@ class ChessTest < Minitest::Test
     end
   end
 
+  def test_load_from_string
+    pgn_string = <<~PGN
+      [Event "70th ch-ITA"]
+      [Site "Siena ITA"]
+      [Round "10"]
+      [Date "2010.12.3"]
+      [White "Caruana, Fabiano"]
+      [Black "Godena, Michele"]
+      [Result "1-0"]
+      1.d4 d5 2.c4 dxc4 3.e4 e5 4.Nf3 Bb4+ 5.Nc3 exd4 6.Nxd4 Ne7 7.Bf4 Bxc3+ 8.bxc3 Ng6 9.Bg3 Qe7 10.Bxc4 Qxe4+ 11.Qe2 Qxe2+ 12.Bxe2 Na6 13.Rb1 O-O 14.O-O Re8 15.Rfe1 Nc5 16.Bxc7 Bd7 17.Bf3 Rxe1+ 18.Rxe1 Rc8 19.Bg3 b6 20.h4 Ne6 21.h5 Ne7 22.Be5 Nc6 23.Nxc6 Bxc6 24.Bg4 Re8 25.Bg3 g6 26.h6 f5 27.Bd1 f4 28.Bh4 Kf8 29.Re5 g5 30.Bh5 Rc8 31.Bxg5 Nxg5 32.Rxg5 Bd7 33.Rg7 Rc5 34.Bf3 Bf5 35.Rxa7 Rxc3 36.Bd5 Bg6 37.Ra4 Rc1+ 38.Kh2 Rc5 39.Rxf4+ Ke7 40.Bf3 Ra5 41.Rb4 b5 42.Bd5 Kf6 43.f4 Bf5 44.Bc6 Bd3 45.Rd4 Ra3 46.Bd5 Bb1 47.Rd1 Bd3 48.Bb3 Bc4 49.Bc2 Ke7 50.Bf5 Rxa2 51.Rd7+ Kf8 52.Rxh7 Bd5 53.Rd7 1-0
+    PGN
+    pgn = Chess::Pgn.new
+    pgn.load_from_string(pgn_string, check_moves: true)
+    assert_equal '70th ch-ITA', pgn.event
+    assert_equal 'Siena ITA', pgn.site
+    assert_equal '10', pgn.round
+    assert_equal '2010.12.3', pgn.date
+    assert_equal 'Caruana, Fabiano', pgn.white
+    assert_equal 'Godena, Michele', pgn.black
+    assert_equal 'Godena, Michele', pgn.black
+    assert_equal '1-0', pgn.result
+    assert_equal 'Rd7', pgn.moves.last
+  end
+
+  def test_load_from_string_without_tags
+    pgn_string = <<~PGN
+      1. e4 e5 2. Nf3 Nc6 3. Bb5 a6 {This opening is called the Ruy Lopez.}
+      4. Ba4 Nf6 5. O-O Be7 6. Re1 b5 7. Bb3 d6 8. c3 O-O 9. h3 Nb8 10. d4 Nbd7
+      11. c4 c6 12. cxb5 axb5 13. Nc3 Bb7 14. Bg5 b4 15. Nb1 h6 16. Bh4 c5 17. dxe5
+      Nxe4 18. Bxe7 Qxe7 19. exd6 Qf6 20. Nbd2 Nxd6 21. Nc4 Nxc4 22. Bxc4 Nb6
+      23. Ne5 Rae8 24. Bxf7+ Rxf7 25. Nxf7 Rxe1+ 26. Qxe1 Kxf7 27. Qe3 Qg5 28. Qxg5
+      hxg5 29. b3 Ke6 30. a3 Kd6 31. axb4 cxb4 32. Ra5 Nd5 33. f3 Bc8 34. Kf2 Bf5
+      35. Ra7 g6 36. Ra6+ Kc5 37. Ke1 Nf4 38. g3 Nxh3 39. Kd2 Kb5 40. Rd6 Kc5 41. Ra6
+      Nf2 42. g4 Bd3 43. Re6 1/2-1/2
+    PGN
+    pgn = Chess::Pgn.new
+    pgn.load_from_string(pgn_string, check_moves: true)
+    assert_nil pgn.event
+    assert_nil pgn.site
+    assert_nil pgn.round
+    assert_nil pgn.date
+    assert_nil pgn.white
+    assert_nil pgn.black
+    assert_nil pgn.black
+    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

@@ -1,15 +1,13 @@
 require 'test_helper'
 
 class ChessTest < Minitest::Test
-
   TestHelper.pgns('stalemate').each do |file|
     name = File.basename(file, '.pgn')
     define_method "test_stalemate_#{name}" do
       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

@@ -1,14 +1,12 @@
 require 'test_helper'
 
 class ChessTest < Minitest::Test
-
   TestHelper.pgns('threefold_repetition').each do |file|
     name = File.basename(file, '.pgn')
     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.2.0
+  version: 0.3.2
 platform: ruby
 authors:
 - Enrico Pilotto
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2017-08-17 00:00:00.000000000 Z
+date: 2021-01-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -16,42 +16,140 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.14'
+        version: '2'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.14'
+        version: '2'
 - !ruby/object:Gem::Dependency
-  name: rake
+  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: '12.0'
+        version: '0.4'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '12.0'
+        version: '0.4'
 - !ruby/object:Gem::Dependency
   name: minitest
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.10'
+        version: '5'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '5'
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '13'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '13'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        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
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !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: '5.10'
+        version: '0.21'
 - !ruby/object:Gem::Dependency
   name: yard
   requirement: !ruby/object:Gem::Requirement
@@ -74,12 +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
@@ -1310,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
@@ -1325,7 +1454,7 @@ homepage: https://github.com/pioz/chess
 licenses:
 - LGPLv3
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -1333,16 +1462,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '1.9'
+      version: '2.5'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: chess
-rubygems_version: 2.6.12
-signing_key: 
+rubygems_version: 3.0.2
+signing_key:
 specification_version: 4
 summary: A fast chess library to play chess with Ruby.
 test_files:
@@ -2555,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