SgfParser 0.9.1 → 1.0.0

data/{sample_sgf → spec/data}/simple.sgf

@@ -1,11 +1,11 @@
-(;GM[1]FF[4]CA[UTF-8]
-SZ[19]RU[Japanese]KM[0.50]TM[1290]OT[3x30 byo-yomi]
-PB[tartrate]PW[redrose]BR[7d]WR[1p]DT[2003-06-18]PC[The Kiseido Go Server (KGS) at http://kgs.kiseido.com/]C[tartrate [7d?\]: hi
-tartrate [7d?\]: thx
-]RE[B+Resign]
-;B[jj]BL[1280.503]CR[jj]C[redrose [1p\]: hi
-Jerk [7d?\]: wow
-Anark [11k\]: game of the day :)
-Jerk [7d?\]:  really big match
-]
-;W[ih]WL[1277.190]CR[ih];AB[aa][bb][cc])
+(;GM[1]FF[4]CA[UTF-8]
+SZ[19]RU[Japanese]KM[0.50]TM[1290]OT[3x30 byo-yomi]
+PB[tartrate]PW[redrose]BR[7d]WR[1p]DT[2003-06-18]PC[The Kiseido Go Server (KGS) at http://kgs.kiseido.com/]C[tartrate [7d?\]: hi
+tartrate [7d?\]: thx
+]RE[B+Resign]
+;B[jj]BL[1280.503]CR[jj]C[redrose [1p\]: hi
+Jerk [7d?\]: wow
+Anark [11k\]: game of the day :)
+Jerk [7d?\]:  really big match
+]
+;W[ih]WL[1277.190]CR[ih];AB[aa][bb][cc])

data/spec/data/simple_saved.sgf

@@ -0,0 +1,7 @@
+(;GM[1]FF[4]CA[UTF-8]SZ[19]RU[Japanese]KM[0.50]TM[1290]OT[3x30 byo-yomi]PB[tartrate]PW[redrose]BR[7d]WR[1p]DT[2003-06-18]PC[The Kiseido Go Server (KGS) at http://kgs.kiseido.com/]C[tartrate [7d?\]: hi
+tartrate [7d?\]: thx
+]RE[B+Resign];B[jj]BL[1280.503]CR[jj]C[redrose [1p\]: hi
+Jerk [7d?\]: wow
+Anark [11k\]: game of the day :)
+Jerk [7d?\]:  really big match
+];W[ih]WL[1277.190]CR[ih];))

data/spec/node_spec.rb

@@ -1,34 +1,64 @@
-require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
-
-describe "SgfParser::Node" do
-  before :each do
-    @node = SgfParser::Node.new
-  end
-
-  it "should be a valid node" do
-    @node.class.should == SgfParser::Node
-    @node.properties.should == {}
-    @node.parent.should == nil
-    @node.children.should == []
-  end
-
-  it "should store properties" do
-    @node.add_properties "PB" => "Dosaku"
-    @node.properties.should == {"PB" => "Dosaku"}
-  end
-
-  it "should link to a parent" do
-    parent = SgfParser::Node.new
-    @node.parent = parent
-    @node.parent.should == parent
-  end
-
-  it "should link to children" do
-    child1 = SgfParser::Node.new
-    child2 = SgfParser::Node.new
-    child3 = SgfParser::Node.new
-    @node.add_children child1, child2, child3
-    @node.children.should == [child1, child2, child3]
-  end
-
+require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
+
+describe "SgfParser::Node" do
+
+  before :each do
+    @node = SGF::Node.new
+  end
+
+  it "should be a valid node" do
+    @node.class.should == SGF::Node
+    @node.properties.should == {}
+    @node.parent.should == nil
+    @node.children.should == []
+  end
+
+  it "should store properties" do
+    @node.add_properties "PB" => "Dosaku"
+    @node.properties.should == {"PB" => "Dosaku"}
+  end
+
+  it "should link to a parent" do
+    parent = SGF::Node.new
+    @node.parent = parent
+    @node.parent.should == parent
+  end
+
+  it "should link to children" do
+    child1 = SGF::Node.new
+    child2 = SGF::Node.new
+    child3 = SGF::Node.new
+    @node.add_children child1, child2, child3
+    @node.children.should == [child1, child2, child3]
+  end
+
+  it "should link to children, who should get new parents" do
+    child1 = SGF::Node.new
+    child2 = SGF::Node.new
+    child3 = SGF::Node.new
+    @node.add_children child1, child2, child3
+    @node.children.each { |child| child.parent.should == @node }
+  end
+
+  it "should allow properties to be added to" do
+    @node.add_properties "TC" => "Hello,"
+    @node.add_properties "TC" => " world!"
+    @node.properties["TC"].should == "Hello, world!"
+  end
+
+  it "should give you the properties based on method given" do
+    @node.add_properties "PW" => "The Tick"
+    @node.add_properties "PB" => "Batmanuel"
+    @node.pw.should == "The Tick"
+    @node.pb.should == "Batmanuel"
+  end
+
+  it "should allow you to change a property completely" do
+    @node.add_properties "RE" => "This is made up"
+    @node.properties["RE"] = "This is also made up"
+    @node.re.should == "This is also made up"
+    @node.re = "And that too"
+    @node.re.should == "And that too"
+  end
+
 end

data/spec/parser_spec.rb

@@ -0,0 +1,20 @@
+require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
+
+describe "SgfParser::Tree.parse" do
+
+  it "should have FF in the first node" do
+    tree = SGF::Parser.new('spec/data/ff4_ex.sgf').parse
+    tree.root.children[0].properties.keys.should include("FF")
+  end
+
+  it "should give an error if FF is missing from the first node" do
+    pending "To be coded later"
+  end
+
+  it "should parse properly the AW property" do
+    sgf = "dd][de][ef]"
+    parser = SGF::Parser.new sgf
+    parser.get_property.should == "[dd][de][ef]"
+  end
+
+end

data/spec/spec_helper.rb

@@ -1,6 +1,6 @@
-$LOAD_PATH.unshift(File.dirname(__FILE__))
-$LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
-require 'sgf_parser'
-require 'rspec'
-require 'rspec/autorun'
-
+$LOAD_PATH.unshift(File.dirname(__FILE__))
+$LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
+require 'sgf'
+require 'rspec'
+require 'rspec/autorun'
+

data/spec/tree_spec.rb

@@ -1,41 +1,29 @@
-require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
-
-describe "SgfParser::Tree" do
-
-  before :each do
-
-  end
-
-  it "should parse properly" do
-    tree = SgfParser::Tree.new :filename => 'sample_sgf/ff4_ex.sgf'
-    tree.class.should == SgfParser::Tree
-    tree.root.children.size.should == 2
-    tree.root.children[0].children.size.should == 5
-  end
-
-  it "should save a simple tree properly" do
-    tree = SgfParser::Tree.new :filename => 'sample_sgf/simple.sgf'
-    new = 'sample_sgf/simple_saved.sgf'
-    tree.save :filename => new
-    tree2 = SgfParser::Tree.new :filename => new
-    tree.should == tree2
-  end
-
-  it "should save the sample SGF properly" do
-    tree = SgfParser::Tree.new :filename => 'sample_sgf/ff4_ex.sgf'
-    new = 'sample_sgf/ff4_ex_saved.sgf'
-    tree.save :filename => new
-    tree2 = SgfParser::Tree.new :filename => new
-    tree_children = []
-    tree2_children = []
-    tree.each { |node| tree_children << node }
-    tree2.each { |node| tree2_children << node }
-    tree_children.size.should == tree2_children.size
-    tree_children.each_with_index do |node, index|
-      node.properties.should == tree2_children[index].properties
-    end
-    tree_children.should == tree2_children
-    tree2.should == tree
-  end
-
-end
+require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
+
+describe "SgfParser::Tree" do
+
+
+  it "should load a file properly" do
+    tree = SGF::Parser.new('spec/data/ff4_ex.sgf').parse
+    tree.class.should == SGF::Tree
+    tree.root.children.size.should == 2
+    tree.root.children[0].children.size.should == 5
+  end
+
+  it "should save a simple tree properly" do
+    tree = SGF::Parser.new('spec/data/simple.sgf').parse
+    new_file = 'spec/data/simple_saved.sgf'
+    tree.save :filename => new_file
+    tree2 = SGF::Parser.new(new_file).parse
+    tree.should == tree2
+  end
+
+  it "should save the sample SGF properly" do
+    tree = SGF::Parser.new('spec/data/ff4_ex.sgf').parse
+    new_file = 'spec/data/ff4_ex_saved.sgf'
+    tree.save :filename => new_file
+    tree2 = SGF::Parser.new(new_file).parse
+    tree2.should == tree
+  end
+
+end

metadata

@@ -1,77 +1,70 @@
---- !ruby/object:Gem::Specification 
+--- !ruby/object:Gem::Specification
 name: SgfParser
-version: !ruby/object:Gem::Version 
-  hash: 57
+version: !ruby/object:Gem::Version
+  version: 1.0.0
   prerelease: 
-  segments: 
-  - 0
-  - 9
-  - 1
-  version: 0.9.1
 platform: ruby
-authors: 
+authors:
 - Aldric Giacomoni
 autorequire: 
 bindir: bin
 cert_chain: []
-
-date: 2011-07-29 00:00:00 -04:00
+date: 2011-08-01 00:00:00.000000000 -04:00
 default_executable: 
-dependencies: 
-- !ruby/object:Gem::Dependency 
-  type: :development
-  requirement: &id001 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+dependencies:
+- !ruby/object:Gem::Dependency
   name: jeweler
-  version_requirements: *id001
-  prerelease: false
-- !ruby/object:Gem::Dependency 
-  type: :development
-  requirement: &id002 !ruby/object:Gem::Requirement 
+  requirement: &76820790 !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: *76820790
+- !ruby/object:Gem::Dependency
   name: rcov
-  version_requirements: *id002
+  requirement: &76820540 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
   prerelease: false
-- !ruby/object:Gem::Dependency 
+  version_requirements: *76820540
+- !ruby/object:Gem::Dependency
+  name: rdoc
+  requirement: &76820300 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-  requirement: &id003 !ruby/object:Gem::Requirement 
+  prerelease: false
+  version_requirements: *76820300
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: &76820030 !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 13
-        segments: 
-        - 1
-        - 2
-        - 9
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
         version: 1.2.9
-  name: rspec
-  version_requirements: *id003
+  type: :development
   prerelease: false
-description: SGFParser is a library that parses and saves SGF (Smart Game Format) files.
+  version_requirements: *76820030
+description: SGFParser is a library that parses and saves SGF (Smart Game Format)
+  files.
 email: aldric@trevoke.net
 executables: []
-
 extensions: []
-
-extra_rdoc_files: 
+extra_rdoc_files:
 - LICENSE
 - README.rdoc
-files: 
+files:
 - .document
 - .rspec
 - .rvmrc
@@ -82,56 +75,45 @@ files:
 - Rakefile
 - SgfParser.gemspec
 - VERSION
-- lib/sgf/parser/node.rb
-- lib/sgf/parser/properties.rb
-- lib/sgf/parser/tree.rb
-- lib/sgf/parser/tree_parse.rb
-- lib/sgf/sgf_indent.rb
-- lib/sgf/sgfindent.rb
-- lib/sgf_parser.rb
-- sample_sgf/ff4_ex.sgf
-- sample_sgf/ff4_ex_saved.sgf
-- sample_sgf/redrose-tartrate.sgf
-- sample_sgf/simple.sgf
-- sample_sgf/simple_saved.sgf
+- lib/sgf.rb
+- lib/sgf/indenter.rb
+- lib/sgf/node.rb
+- lib/sgf/parser.rb
+- lib/sgf/properties.rb
+- lib/sgf/tree.rb
 - sample_usage/parsing_files.rb
+- spec/data/ff4_ex.sgf
+- spec/data/ff4_ex_saved.sgf
+- spec/data/redrose-tartrate.sgf
+- spec/data/simple.sgf
+- spec/data/simple_saved.sgf
 - spec/node_spec.rb
+- spec/parser_spec.rb
 - spec/spec_helper.rb
-- spec/tree_parser_spec.rb
 - spec/tree_spec.rb
 has_rdoc: true
 homepage: http://github.com/Trevoke/SGFParser
 licenses: []
-
 post_install_message: 
 rdoc_options: []
-
-require_paths: 
+require_paths:
 - lib
-required_ruby_version: !ruby/object:Gem::Requirement 
+required_ruby_version: !ruby/object:Gem::Requirement
   none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
-      - 0
-      version: "0"
-required_rubygems_version: !ruby/object:Gem::Requirement 
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
-      - 0
-      version: "0"
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
 requirements: []
-
 rubyforge_project: 
 rubygems_version: 1.6.2
 signing_key: 
 specification_version: 3
 summary: A library for working with SGF files.
 test_files: []
-

data/lib/sgf/sgfindent.rb

@@ -1,118 +0,0 @@
-module SgfParser
-
-  # This indents an SGF file to make it more readable. It outputs to the screen
-  # by default, but can be given a file as output.
-  # Usage: SgfParser::Indenter.new infile, outfile
-  class Indenter
-
-    def initialize file, out=$stdout
-      sgf = ""
-      File.open(file) { |f| sgf = f.read }
-      @new_string = ""
-      sgf.gsub! "\\\\n\\\\r", ""
-      sgf.gsub! "\\\\r\\\\n", ""
-      sgf.gsub! "\\\\r", ""
-      sgf.gsub! "\\\\n", ""
-      #sgf.gsub! "\n", ""
-
-      end_of_a_series = false
-      identprop = false # We are not in the middle of an identprop value.
-      # An identprop is an identity property - a value.
-
-      sgf_array = sgf.split(//)
-      iterator = 0
-      array_length = sgf_array.size
-      indent = 0
-
-      while iterator < array_length - 1
-        char = sgf_array[iterator]
-
-        case char
-          when '(' # Opening a new branch
-            if !identprop
-              @new_string << "\n"
-              indent += 2
-              #tabulate indent
-              @new_string << " " * indent
-            end
-            @new_string << char
-
-          when ')' # Closing a branch
-            @new_string << char
-            if !identprop
-              @new_string << "\n"
-              indent -= 2
-              @new_string << " " * indent
-              #tabulate indent
-            end
-
-          when ';' # Opening a new node
-            if !identprop
-              @new_string << "\n"
-              @new_string << " " * indent
-              #tabulate indent
-            end
-            @new_string << char
-
-          when '[' # Open comment?
-            if !identprop #If we're not inside a comment, then now we are.
-              identprop = true
-              end_of_a_series = false
-            end
-            @new_string << char
-
-          when ']' # Close comment
-            end_of_a_series = true # Maybe end of a series of comments.
-            identprop = false # That's our cue to close a comment.
-            @new_string << char
-
-          when "\\" # If we're inside a comment, then maybe we're about to escape a ].
-            # This is the whole reason we need this ugly charade of a loop.
-            if identprop
-              if sgf_array[iterator + 1] == "]"
-                @new_string << "\\]"
-                iterator += 1
-              else
-                @new_string << "\\"
-              end
-            else
-              #This should never happen - a backslash outside a comment ?!
-              #But let's not have it be told that I'm not prepared.
-              @new_string << "\\"
-            end
-
-          when "\n"
-            @new_string << "\n"
-            @new_string << " " * indent
-            #tabulate indent
-
-          else
-            # Well, I guess it's "just" a character after all.
-            if end_of_a_series
-              end_of_a_series = false
-            end
-            @new_string << char
-        end
-
-        iterator += 1
-      end
-
-      if out == $stdout
-        $stdout << @new_string
-      else
-        File.open(out, 'w') { |file| file << @new_string }
-      end
-      
-    end
-
-    private
-
-    def tabulate indent
-      indent.times { print " " }
-    end
-  end
-end
-
-
-
-