chess 0.3.2 → 0.3.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6c0769e70cd986cfb2f23d8f8eb2551cd72745e35dc5cde6832bbab0d35015d5
-  data.tar.gz: 9dd9e4fa0b620f8905e46d33ed6969a09a896e07018d3857dec3926e6da2571b
+  metadata.gz: 36668063cb71d331dbb172611791d43e0b5d465c0cd9e87baf80bc35cf11e551
+  data.tar.gz: 516ebc6fb472885ec871c6fbd955f489a6afec26c52250a9612210e61f6ce883
 SHA512:
-  metadata.gz: d6bb94f4bc3a86cfb593c0e2c233bf75887c1c10be1f968d9dcc4e169e5add14dd18624c9d2932284e6f4fcacdf43fabb65457575fa29a7d308f7366d6f6d227
-  data.tar.gz: e68c4d62784204158e9829c85265fb354357b30af59cfdc790eeea498fb93e88c136712e6b61979017488a0a14836a2a07ac7c90dc091ad2bf5dcc739c9d5845
+  metadata.gz: d3e811ed0ac4ad177ef1991cd38bbabf84b2139a8d12ea18b0f251621ed744cc54afe9252b489874a6f121551899a5382cffff67158281ca6ce64b5fe0ce2b61
+  data.tar.gz: cad20b859cc7c41b4dfba6b92e0de8bc14317b86f0e4ffeb9f656819a2f507160534eeca6c0944a251964bf2d8ec20fc199548c50d293cf4f7a1c6b64797ae3c

data/.github/workflows/ruby.yml

@@ -18,7 +18,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        ruby-version: ['2.5', '2.6', '2.7', '3.0']
+        ruby-version: ['2.6', '2.7', '3.0', '3.1', '3.2']
 
     steps:
       - uses: actions/checkout@v2
@@ -28,7 +28,7 @@ jobs:
           ruby-version: ${{ matrix.ruby-version }}
           bundler-cache: true # runs 'bundle install' and caches installed gems automatically
       - name: Compile extension
-        working-directory: ./ext
+        working-directory: ./ext/chess
         run: |
           ruby extconf.rb
           make
@@ -45,7 +45,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        ruby-version: ['2.5', '2.6', '2.7', '3.0']
+        ruby-version: ['2.6', '2.7', '3.0', '3.1', '3.2']
 
     steps:
       - uses: actions/checkout@v2

data/.gitignore

@@ -1,11 +1,12 @@
 *.gem
 .bundle
 .byebug_history
+.ruby-version
 .yardoc/
 coverage/
-ext/chess
-ext/Makefile
-ext/*.bundle
-ext/*.o
+ext/chess/Makefile
+ext/chess/*.bundle
+ext/chess/*.o
+lib/chess/chess.bundle
 main.rb
 pkg/*

data/.rubocop.yml

@@ -29,6 +29,9 @@ Metrics/MethodLength:
 Metrics/PerceivedComplexity:
   Max: 20
 
+Minitest/AssertPredicate:
+  Enabled: false
+
 Minitest/MultipleAssertions:
   Enabled: false
 

data/Gemfile.lock

@@ -1,51 +1,55 @@
 PATH
   remote: .
   specs:
-    chess (0.3.2)
+    chess (0.3.4)
 
 GEM
   remote: http://rubygems.org/
   specs:
     ast (2.4.2)
     byebug (11.1.3)
-    codecov (0.4.2)
+    codecov (0.6.0)
       simplecov (>= 0.15, < 0.22)
-    docile (1.3.5)
-    minitest (5.14.3)
-    parallel (1.20.1)
-    parser (3.0.0.0)
+    docile (1.4.0)
+    json (2.6.2)
+    minitest (5.16.2)
+    parallel (1.22.1)
+    parser (3.1.2.0)
       ast (~> 2.4.1)
-    rainbow (3.0.0)
-    rake (13.0.3)
-    regexp_parser (2.0.3)
-    rexml (3.2.4)
-    rubocop (1.8.1)
+    rainbow (3.1.1)
+    rake (13.0.6)
+    regexp_parser (2.5.0)
+    rexml (3.2.5)
+    rubocop (1.31.2)
+      json (~> 2.3)
       parallel (~> 1.10)
-      parser (>= 3.0.0.0)
+      parser (>= 3.1.0.0)
       rainbow (>= 2.2.2, < 4.0)
       regexp_parser (>= 1.8, < 3.0)
-      rexml
-      rubocop-ast (>= 1.2.0, < 2.0)
+      rexml (>= 3.2.5, < 4.0)
+      rubocop-ast (>= 1.18.0, < 2.0)
       ruby-progressbar (~> 1.7)
       unicode-display_width (>= 1.4.0, < 3.0)
-    rubocop-ast (1.4.1)
-      parser (>= 2.7.1.5)
-    rubocop-minitest (0.10.3)
-      rubocop (>= 0.87, < 2.0)
-    rubocop-performance (1.9.2)
-      rubocop (>= 0.90.0, < 2.0)
+    rubocop-ast (1.19.1)
+      parser (>= 3.1.1.0)
+    rubocop-minitest (0.20.1)
+      rubocop (>= 0.90, < 2.0)
+    rubocop-performance (1.14.2)
+      rubocop (>= 1.7.0, < 2.0)
       rubocop-ast (>= 0.4.0)
-    rubocop-rake (0.5.1)
-      rubocop
+    rubocop-rake (0.6.0)
+      rubocop (~> 1.0)
     ruby-progressbar (1.11.0)
     simplecov (0.21.2)
       docile (~> 1.1)
       simplecov-html (~> 0.11)
       simplecov_json_formatter (~> 0.1)
     simplecov-html (0.12.3)
-    simplecov_json_formatter (0.1.2)
-    unicode-display_width (2.0.0)
-    yard (0.9.26)
+    simplecov_json_formatter (0.1.4)
+    unicode-display_width (2.2.0)
+    webrick (1.7.0)
+    yard (0.9.28)
+      webrick (~> 1.7.0)
 
 PLATFORMS
   ruby
@@ -56,15 +60,15 @@ DEPENDENCIES
   bundler (~> 2)
   byebug (~> 11)
   chess!
-  codecov (~> 0.4)
+  codecov (~> 0)
   minitest (~> 5)
   rake (~> 13)
   rubocop (~> 1)
-  rubocop-minitest (~> 0.10)
+  rubocop-minitest (~> 0)
   rubocop-performance (~> 1)
-  rubocop-rake (~> 0.5)
-  simplecov (~> 0.21)
-  yard (~> 0.9)
+  rubocop-rake (~> 0)
+  simplecov (~> 0)
+  yard (>= 0.9.20)
 
 BUNDLED WITH
-   2.2.7
+   2.3.8

data/README.md

@@ -34,7 +34,7 @@ end
 puts g.status
 ```
 
-**Documentation** is available **[here](http://pioz.github.com/chess)**.
+**Documentation** is available **[here](http://pioz.github.io/chess)**.
 
 ## Questions or problems?
 

data/Rakefile

@@ -2,7 +2,7 @@ require 'bundler/gem_tasks'
 
 require 'yard'
 YARD::Rake::YardocTask.new do |t|
-  t.files = ['lib/**/*.rb', 'ext/*.c']
+  t.files = ['lib/**/*.rb', 'ext/**/*.c']
   t.options << '-rREADME.md'
   t.options << '--title=Chess'
   t.options << '-mmarkdown'
@@ -12,7 +12,7 @@ end
 require 'rake/testtask'
 task default: :test
 Rake::TestTask.new do |test|
-  test.libs << 'test'
+  test.libs += %w[ext lib test]
   test.warning = true
-  test.test_files = FileList["test/test_#{ENV['T'] || '*'}.rb"]
+  test.test_files = FileList["test/test_#{ENV.fetch('T', '*')}.rb"]
 end

data/chess.gemspec

@@ -1,5 +1,4 @@
-$LOAD_PATH.push File.expand_path('lib', __dir__)
-require 'chess/version'
+require_relative 'lib/chess/version'
 
 Gem::Specification.new do |s|
   s.name        = 'chess'
@@ -12,21 +11,22 @@ Gem::Specification.new do |s|
   s.license     = 'LGPLv3'
 
   s.files         = `git ls-files`.split("\n")
-  s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
   s.executables   = `git ls-files -- bin/*`.split("\n").map { |f| File.basename(f) }
-  s.extensions    = ['ext/extconf.rb']
+  s.extensions    = ['ext/chess/extconf.rb']
   s.require_paths = ['lib']
 
+  s.metadata['rubygems_mfa_required'] = 'true'
+
   s.required_ruby_version = '>= 2.5'
   s.add_development_dependency 'bundler', '~> 2'
   s.add_development_dependency 'byebug', '~> 11'
-  s.add_development_dependency 'codecov', '~> 0.4'
+  s.add_development_dependency 'codecov', '~> 0'
   s.add_development_dependency 'minitest', '~> 5'
   s.add_development_dependency 'rake', '~> 13'
   s.add_development_dependency 'rubocop', '~> 1'
-  s.add_development_dependency 'rubocop-minitest', '~> 0.10'
+  s.add_development_dependency 'rubocop-minitest', '~> 0'
   s.add_development_dependency 'rubocop-performance', '~> 1'
-  s.add_development_dependency 'rubocop-rake', '~> 0.5'
-  s.add_development_dependency 'simplecov', '~> 0.21'
-  s.add_development_dependency 'yard', '~> 0.9'
+  s.add_development_dependency 'rubocop-rake', '~> 0'
+  s.add_development_dependency 'simplecov', '~> 0'
+  s.add_development_dependency 'yard', '>= 0.9.20'
 end

data/ext/{chess.c → chess/chess.c}

@@ -716,6 +716,7 @@ Init_chess ()
    * This ensures a fast library.
    */
   board_klass = rb_define_class_under (chess, "Board", rb_cObject);
+  rb_undef_alloc_func (board_klass);
   rb_define_method (board_klass, "placement", board_placement, 0);
   rb_define_method (board_klass, "[]", board_get_piece, 1);
   rb_define_method (board_klass, "check?", board_king_in_check, 0);

data/lib/chess/game.rb

@@ -1,5 +1,3 @@
-require File.join(File.dirname(__FILE__), '../../ext/chess')
-
 module Chess
   # This class rappresents a chess game.
   class Game < CGame
@@ -155,16 +153,32 @@ module Chess
           promotion: match[4]       # Promote with
         }
         expand[:from] = match[2] if match[2] && match[2].size == 2
+
+        # Support UCI protocol (Lichess)
+        case expand[:from]
+        when 'e1'
+          expand[:to] = 'g1' if expand[:to] == 'h1' # UCI protocol (Lichess) white king short castling
+          expand[:to] = 'c1' if expand[:to] == 'a1' # UCI protocol (Lichess) white king long castling
+        when 'e8'
+          expand[:to] = 'g8' if expand[:to] == 'h8' # UCI protocol (Lichess) black king short castling
+          expand[:to] = 'c8' if expand[:to] == 'a8' # UCI protocol (Lichess) black king long castling
+        end
+
         return expand
-      elsif SHORT_CASTLING_REGEXP.match?(notation)
+      end
+
+      # Castling notation
+      if SHORT_CASTLING_REGEXP.match?(notation)
         return { name: 'K', dis: nil, from: 'e8', to: 'g8', promotion: nil } if self.board.active_color # black king short castling
 
         return { name: 'K', dis: nil, from: 'e1', to: 'g1', promotion: nil } # white king short castling
-      elsif LONG_CASTLING_REGEXP.match?(notation)
+      end
+      if LONG_CASTLING_REGEXP.match?(notation)
         return { name: 'K', dis: nil, from: 'e8', to: 'c8', promotion: nil } if self.board.active_color # black king long castling
 
         return { name: 'K', dis: nil, from: 'e1', to: 'c1', promotion: nil } # white king long castling
       end
+
       raise BadNotationError.new(notation)
     end
   end

data/lib/chess/pgn.rb

@@ -49,7 +49,7 @@ module Chess
     # @raise [InvalidPgnFormatError]
     # @raise [IllegalMoveError]
     def load(filename, check_moves: false)
-      str = File.open(filename, 'r').read
+      str = File.read(filename)
       load_from_string(str, check_moves: check_moves)
     end
 
@@ -88,7 +88,7 @@ module Chess
       s << "\n"
       m = ''
       @moves.each_with_index do |move, i|
-        m << "#{i / 2 + 1}. " if i.even?
+        m << "#{(i / 2) + 1}. " if i.even?
         m << "#{move} "
       end
       m << @result unless @result.nil?
@@ -98,7 +98,7 @@ module Chess
     # Write PGN to file.
     # @param [String] filename The path of the PGN file.
     def write(filename)
-      File.open(filename, 'w') { |f| f.write(self.to_s) }
+      File.write(filename, self.to_s)
     end
 
     # # @!visibility private

data/lib/chess/utf8_notation.rb

@@ -1,6 +1,4 @@
 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.
@@ -36,7 +34,6 @@ 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.2'.freeze
+  VERSION = '0.3.4'.freeze
 end

data/lib/chess.rb

@@ -1,5 +1,13 @@
-require 'chess/exceptions'
-require 'chess/game'
-require 'chess/gnuchess'
-require 'chess/pgn'
-require 'chess/version'
+require 'chess/chess' # this is the compiled ruby extension (chess.so or chess.bundle)
+require_relative 'chess/exceptions'
+require_relative 'chess/game'
+require_relative 'chess/gnuchess'
+require_relative 'chess/pgn'
+require_relative 'chess/version'
+
+# While development to require 'chess' from irb run with `irb -I lib:ext`
+# $ irb -I lib:ext
+# irb(main):001:0> require 'chess'
+# => true
+# irb(main):002:0> Chess::Game.new
+# => #<Chess::Game:0x0000000000000001>

data/test/test_game.rb

@@ -24,7 +24,7 @@ class ChessTest < Minitest::Test
     game.resign(:black)
     assert_equal :white_won_resign, game.status
     assert game.over?
-    assert '1-0', game.result
+    assert_equal '1-0', game.result
   end
 
   def test_status_black_won_resign
@@ -33,7 +33,7 @@ class ChessTest < Minitest::Test
     game.resign(:white)
     assert_equal :black_won_resign, game.status
     assert game.over?
-    assert '0-1', game.result
+    assert_equal '0-1', game.result
   end
 
   def test_status_insufficient_material

data/test/test_uci_castling.rb

@@ -0,0 +1,34 @@
+require 'test_helper'
+
+class UCICastrlingTest < Minitest::Test
+  def test_uci_white_short_castling
+    game = Chess::Game.new
+    game.moves = %w[e4 c5 c3 d5 exd5 Qxd5 d4 Nf6 Nf3 e6 Be2 Be7 e1h1 Nc6]
+    assert_equal '*', game.result
+  end
+
+  def test_uci_white_long_castling
+    game = Chess::Game.new
+    game.moves = %w[e4 c5 Nf3 Nc6 d4 cxd4 Nxd4 Nf6 Nc3 d6 Bg5 Qb6 Nb3 e6 Qd2 Be7 e1a1]
+    assert_equal '*', game.result
+  end
+
+  def test_uci_black_short_castling
+    game = Chess::Game.new
+    game.moves = %w[d4 d5 c4 dxc4 e4 e5 Nf3 Bb4+ Nc3 exd4 Nxd4 Ne7 Bf4 Bxc3+ bxc3 Ng6 Bg3 Qe7 Bxc4 Qxe4+ Qe2 Qxe2+ Bxe2 Na6 Rb1 e8h8]
+    assert_equal '*', game.result
+  end
+
+  def test_uci_black_long_castling
+    game = Chess::Game.new
+    game.moves = %w[e4 c5 Nf3 d6 d4 cxd4 Qxd4 Nc6 Bb5 Bd7 Bxc6 Bxc6 Bg5 Nf6 Bxf6 gxf6 Nc3 e6 e1c1 Be7 Rhe1 Rg8 Qe3 Rxg2 Rg1 Rg6 Nd4 Qb6 h4 e8a8]
+    assert_equal '*', game.result
+  end
+
+  def test_uci_invalid_white_long_castling
+    game = Chess::Game.new
+    assert_raises(Chess::IllegalMoveError) do
+      game.moves = %w[e4 c5 Nf3 Nc6 d4 cxd4 Nxd4 Nf6 Nc3 d6 Bg5 Qb6 Nb3 e6 Qd2 Be7 e8a8]
+    end
+  end
+end