SgfParser 0.9.0 → 0.9.1

data/.document

@@ -1,5 +1,5 @@
-README.rdoc
-lib/**/*.rb
-bin/*
-features/**/*.feature
-LICENSE
+README.rdoc
+lib/**/*.rb
+bin/*
+features/**/*.feature
+LICENSE

data/.rspec

@@ -0,0 +1 @@
+--color --format doc

data/.rvmrc

@@ -0,0 +1 @@
+rvm 1.9.2@jrun

data/Gemfile

@@ -0,0 +1,12 @@
+# A sample Gemfile
+source "http://rubygems.org"
+
+
+group :development do
+  gem "jeweler"
+  gem "rcov"
+end
+
+group :test do
+  gem "rspec"
+end

data/Gemfile.lock

@@ -0,0 +1,27 @@
+GEM
+  remote: http://rubygems.org/
+  specs:
+    diff-lcs (1.1.2)
+    git (1.2.5)
+    jeweler (1.6.4)
+      bundler (~> 1.0)
+      git (>= 1.2.5)
+      rake
+    rake (0.9.2)
+    rcov (0.9.9)
+    rspec (2.6.0)
+      rspec-core (~> 2.6.0)
+      rspec-expectations (~> 2.6.0)
+      rspec-mocks (~> 2.6.0)
+    rspec-core (2.6.4)
+    rspec-expectations (2.6.0)
+      diff-lcs (~> 1.1.2)
+    rspec-mocks (2.6.0)
+
+PLATFORMS
+  ruby
+
+DEPENDENCIES
+  jeweler
+  rcov
+  rspec

data/Rakefile

@@ -18,38 +18,23 @@ rescue LoadError
   puts "Jeweler (or a dependency) not available. Install it with: gem install jeweler"
 end
 
-require 'spec/rake/spectask'
-Spec::Rake::SpecTask.new(:spec) do |spec|
-  spec.libs << 'lib' << 'spec'
-  spec.spec_files = FileList['spec/**/*_spec.rb']
+require 'rspec/core/rake_task'
+desc 'Default: run specs.'
+task :default => :spec
+
+desc "Run specs"
+RSpec::Core::RakeTask.new do |spec|
+  spec.pattern = "./spec/**/*_spec.rb"
 end
 
-Spec::Rake::SpecTask.new(:rcov) do |spec|
-  spec.libs << 'lib' << 'spec'
+RSpec::Core::RakeTask.new(:coverage) do |spec|
   spec.pattern = 'spec/**/*_spec.rb'
   spec.rcov = true
+  spec.rcov_opts = ['--exclude', 'spec']
 end
 
-task :spec => :check_dependencies
-
-# In case I ever want to go back to cucumber.
-#begin
-#  require 'cucumber/rake/task'
-#  Cucumber::Rake::Task.new(:features)
-#
-#  task :features => :check_dependencies
-#rescue LoadError
-#  task :features do
-#    abort "Cucumber is not available. In order to run features, you must: sudo gem install cucumber"
-#  end
-#end
-
-#task :default => [:spec, :features]
-
-task :default => [:spec]
-
-require 'rake/rdoctask'
-Rake::RDocTask.new do |rdoc|
+require 'rdoc/task'
+RDoc::Task.new do |rdoc|
   version = File.exist?('VERSION') ? File.read('VERSION') : ""
 
   rdoc.rdoc_dir = 'rdoc'

data/SgfParser.gemspec

@@ -1,67 +1,71 @@
-# Generated by jeweler
-# DO NOT EDIT THIS FILE DIRECTLY
-# Instead, edit Jeweler::Tasks in rakefile, and run the gemspec command
-# -*- encoding: utf-8 -*-
-
-Gem::Specification.new do |s|
-  s.name = %q{SgfParser}
-  s.version = "0.9.0"
-
-  s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
-  s.authors = ["Aldric Giacomoni"]
-  s.date = %q{2010-04-30}
-  s.description = %q{SGFParser is a library that parses and saves SGF (Smart Game Format) files.}
-  s.email = %q{aldric@trevoke.net}
-  s.extra_rdoc_files = [
-    "LICENSE",
-     "README.rdoc"
-  ]
-  s.files = [
-    ".document",
-     ".gitignore",
-     "LICENSE",
-     "README.rdoc",
-     "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_parser.rb",
-     "sample_sgf/ff4_ex.sgf",
-     "sample_sgf/redrose-tartrate.sgf",
-     "sample_usage/parsing_files.rb",
-     "spec/node_spec.rb",
-     "spec/spec.opts",
-     "spec/spec_helper.rb",
-     "spec/tree_spec.rb",
-     "tryme.rb"
-  ]
-  s.homepage = %q{http://github.com/Trevoke/SGFParser}
-  s.rdoc_options = ["--charset=UTF-8"]
-  s.require_paths = ["lib"]
-  s.rubygems_version = %q{1.3.6}
-  s.summary = %q{A library for working with SGF files.}
-  s.test_files = [
-    "spec/node_spec.rb",
-     "spec/spec_helper.rb",
-     "spec/tree_parser_spec.rb",
-     "spec/tree_spec.rb"
-  ]
-
-  if s.respond_to? :specification_version then
-    current_version = Gem::Specification::CURRENT_SPECIFICATION_VERSION
-    s.specification_version = 3
-
-    if Gem::Version.new(Gem::RubyGemsVersion) >= Gem::Version.new('1.2.0') then
-      s.add_development_dependency(%q<rspec>, [">= 1.2.9"])
-    else
-      s.add_dependency(%q<rspec>, [">= 1.2.9"])
-    end
-  else
-    s.add_dependency(%q<rspec>, [">= 1.2.9"])
-  end
-end
-
+# Generated by jeweler
+# DO NOT EDIT THIS FILE DIRECTLY
+# Instead, edit Jeweler::Tasks in Rakefile, and run 'rake gemspec'
+# -*- encoding: utf-8 -*-
+
+Gem::Specification.new do |s|
+  s.name = %q{SgfParser}
+  s.version = "0.9.1"
+
+  s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
+  s.authors = ["Aldric Giacomoni"]
+  s.date = %q{2011-07-29}
+  s.description = %q{SGFParser is a library that parses and saves SGF (Smart Game Format) files.}
+  s.email = %q{aldric@trevoke.net}
+  s.extra_rdoc_files = [
+    "LICENSE",
+    "README.rdoc"
+  ]
+  s.files = [
+    ".document",
+    ".rspec",
+    ".rvmrc",
+    "Gemfile",
+    "Gemfile.lock",
+    "LICENSE",
+    "README.rdoc",
+    "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",
+    "sample_usage/parsing_files.rb",
+    "spec/node_spec.rb",
+    "spec/spec_helper.rb",
+    "spec/tree_parser_spec.rb",
+    "spec/tree_spec.rb"
+  ]
+  s.homepage = %q{http://github.com/Trevoke/SGFParser}
+  s.require_paths = ["lib"]
+  s.rubygems_version = %q{1.6.2}
+  s.summary = %q{A library for working with SGF files.}
+
+  if s.respond_to? :specification_version then
+    s.specification_version = 3
+
+    if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
+      s.add_development_dependency(%q<jeweler>, [">= 0"])
+      s.add_development_dependency(%q<rcov>, [">= 0"])
+      s.add_development_dependency(%q<rspec>, [">= 1.2.9"])
+    else
+      s.add_dependency(%q<jeweler>, [">= 0"])
+      s.add_dependency(%q<rcov>, [">= 0"])
+      s.add_dependency(%q<rspec>, [">= 1.2.9"])
+    end
+  else
+    s.add_dependency(%q<jeweler>, [">= 0"])
+    s.add_dependency(%q<rcov>, [">= 0"])
+    s.add_dependency(%q<rspec>, [">= 1.2.9"])
+  end
+end
+

data/VERSION

@@ -1 +1 @@
-0.9.0
+0.9.1

data/lib/sgf/parser/node.rb

@@ -1,8 +1,5 @@
 module SgfParser
 
-  # Each part of the SGF Tree is a node. This holds links to the parent node,
-  # to the next node(s) in the tree, and of course, the properties of said node.
-  # Accessors : node.parent, node.children, node.properties
   class Node
 
     attr_accessor :parent, :children, :properties
@@ -19,9 +16,6 @@ module SgfParser
       @properties.merge! args[:properties] if !args[:properties].nil?
     end
 
-    # Adds one child or several children. Can be passed in as a comma-separated
-    # list or an array of node children. Will raise an error if one of the
-    # arguments is not of class Node.
     def add_children *nodes
       nodes.flatten!
       raise "Non-node child given!" if nodes.find { |node| node.class != Node }
@@ -30,8 +24,6 @@ module SgfParser
       @children.concat nodes
     end
 
-    # Adds one or more properties to the node.
-    # Argument: a hash {'property' => 'value'}
     def add_properties hash
       hash.each do |key, value|
         @properties[key] ||= ""
@@ -39,18 +31,14 @@ module SgfParser
       end
     end
 
-    # Iterates over each child of the given node
-    # node.each_child { |child| puts child.properties }
     def each_child
       @children.each { |child| yield child }
     end
 
-    # Compares one node's properties to another node's properties
     def == other_node
       @properties == other_node.properties
     end
 
-    # Making comments easier to access.
     def comments
       @properties["C"]
     end

data/lib/sgf/parser/tree.rb

@@ -1,19 +1,10 @@
 module SgfParser
 
-  # This is a placeholder for the root of the gametree(s). It's an abstraction,
-  # but it allows an easy way to save, iterate over, and compare other trees.
-  # Accessors: tree.root
   class Tree
     include Enumerable
 
     attr_accessor :root
 
-    # Create a new tree. Can also be used to load a tree from either a file or
-    # a string. Raises an error if both are provided.
-    # options: \n
-    # :filename => filename \n
-    # !!! OR !!! \n
-    # :string => string \n
     def initialize args={}
       @root = Node.new
       @sgf = ""
@@ -27,10 +18,9 @@ module SgfParser
       parse unless @sgf.empty?
     end
 
-    # Iterates over the tree, node by node, in preorder fashion.
-    # Does not support other types of iteration, but may in the future.
-    # tree.each { |node| puts "I am node. Hear me #{node.properties} !"}
     def each order=:preorder, &block
+      # I know, I know. SGF is only preorder. Well, it's implemented, ain't it?
+      # Stop complaining.
       case order
         when :preorder
           preorder @root, &block
@@ -51,10 +41,8 @@ module SgfParser
     # tree.save :filename => file_name
     def save args={}
       raise ArgumentError, "No file name provided" if args[:filename].nil?
-      # SGF files are trees stored in pre-order traversal.
       @savable_sgf = "("
       @root.children.each { |child| write_node child }
-      # write_node @root
       @savable_sgf << ")"
 
       File.open(args[:filename], 'w') { |f| f << @savable_sgf }
@@ -67,11 +55,12 @@ module SgfParser
       @savable_sgf << ";"
       unless node.properties.empty?
         properties = ""
-        node.properties.each do |k, v|
-          v_escaped = v.gsub("]", "\\]")
-          properties += "#{k.to_s}[#{v_escaped}]"
+        node.properties.each do |identity, property|
+          new_property = property[1...-1]
+          new_property.gsub!("]", "\\]") if identity == "C"
+          properties += "#{identity.to_s}[#{new_property}]"
         end
-        @savable_sgf << "#{properties}"
+        @savable_sgf << properties
       end
 
       case node.children.size
@@ -104,13 +93,13 @@ module SgfParser
           preorder(child, &block)
         end
       end
-    end # preorder      
+    end
 
     def method_missing method_name, *args
       output = @root.children[0].properties[method_name]
       super(method_name, args) if output.nil?
       output
-    end # method_missing    
+    end
 
   end
 

data/lib/sgf/parser/tree_parse.rb

@@ -3,9 +3,8 @@ require 'stringio'
 module SgfParser
   class Tree
 
-    private
+    #private
 
-    # Creates a tree (truly, a linked list) from @sgf.
     def parse
       while char = next_character
         case char
@@ -65,14 +64,29 @@ module SgfParser
 
     def get_property
       buffer = ""
-      while true
-        next_bit = @stream.sysread(1)
-        break if next_bit == "]"
-        next_bit << @stream.sysread(1) if next_bit == "\\"
-        next_bit = "]" if next_bit == "\\]"
-        buffer << next_bit
+      while char = next_character
+        case char
+          when "]" then break unless multiple_properties?
+          when "\\" then
+            char << next_character
+            char = "]" if char == "\\]"
+        end
+
+        buffer << char
+      end
+      "[#{buffer}]"
+    end
+
+    def multiple_properties?
+      multiple_properties = false
+      if char = next_character
+        char = next_character if char == "\n"
+        if char == "["
+          multiple_properties = true
+        end
+        @stream.pos -= 1
+        multiple_properties
       end
-      buffer
     end
 
     def store_character(char)

data/lib/sgf/sgfindent.rb

@@ -0,0 +1,118 @@
+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
+
+
+
+

data/sample_sgf/ff4_ex_saved.sgf

@@ -0,0 +1,45 @@
+(;FF[4]AP[Primiview:3.1]GM[1]SZ[19]GN[Gametree 1: properties]US[Arno Hollosi](;B[pd]N[Moves, comments, annotations]C[Nodename set to: "Moves, comments, annotations"];W[dp]GW[1]C[Marked as "Good for White"];B[pp]GB[2]C[Marked as "Very good for Black"];W[dc]GW[2]C[Marked as "Very good for White"];B[pj]DM[1]C[Marked as "Even position"];W[ci]UC[1]C[Marked as "Unclear position"];B[jd]TE[1]C[Marked as "Tesuji" or "Good move"];W[jp]BM[2]C[Marked as "Very bad move"];B[gd]DO[]C[Marked as "Doubtful move"];W[de]IT[]C[Marked as "Interesting move"];B[jj];C[White "Pass" move]W[];)(;AB[dd][de][df][dg][do:gq]  AW[jd][je][jf][jg][kn:lq][pn:pq]N[Setup]C[Black & white stones at the top are added as single stones.
+
+Black & white stones at the bottom are added using compressed point lists.];AE[ep][fp][kn][lo][lq][pn:pq]C[AddEmpty
+
+Black stones & stones of left white group are erased in FF[3\] way.
+
+White stones at bottom right were erased using compressed point list.];AB[pd]AW[pp]PL[B]C[Added two stones.
+
+Node marked with "Black to play".];)(;AB[dd][de][df][dg][dh][di][dj][nj][ni][nh][nf][ne][nd][ij][ii][ih][hq][gq][fq][eq][dr][ds][dq][dp][cp][bp][ap][iq][ir][is][bo][bn][an][ms][mr]AW[pd][pe][pf][pg][ph][pi][pj][fd][fe][ff][fh][fi][fj][kh][ki][kj][os][or][oq][op][pp][qp][rp][sp][ro][rn][sn][nq][mq][lq][kq][kr][ks][fs][gs][gr][er]N[Markup]C[Position set up without compressed point lists.];TR[dd][de][df][ed][ee][ef][fd:ff] MA[dh][di][dj][ej][ei][eh][fh:fj] CR[nd][ne][nf][od][oe][of][pd:pf] SQ[nh][ni][nj][oh][oi][oj][ph:pj] SL[ih][ii][ij][jj][ji][jh][kh:kj] TW[pq:ss][so][lr:ns] TB[aq:cs][er:hs][ao]C[Markup at top partially using compressed point lists (for markup on white stones); listed clockwise, starting at upper left:
+- TR (triangle)
+- CR (circle)
+- SQ (square)
+- SL (selected points)
+- MA ('X')
+
+Markup at bottom: black & white territory (using compressed point lists)];LB[dc:1][fc:2][nc:3][pc:4][dj:a][fj:b][nj:c][pj:d][gs:ABCDEFGH][gr:ABCDEFG][gq:ABCDEF][gp:ABCDE][go:ABCD][gn:ABC][gm:AB][mm:12][mn:123][mo:1234][mp:12345][mq:123456][mr:1234567][ms:12345678]C[Label (LB property)
+
+Top: 8 single char labels (1-4, a-d)
+
+Bottom: Labels up to 8 char length.];)(;B[qd]N[Style & text type]C[There are hard linebreaks & soft linebreaks.
+Soft linebreaks are linebreaks preceeded by '\\' like this one >o\
+k<. Hard line breaks are all other linebreaks.
+Soft linebreaks are converted to >nothing<, i.e. removed.
+
+Note that linebreaks are coded differently on different systems.
+
+Examples (>ok< shouldn't be split):
+
+linebreak 1 "": >o\
+k<
+linebreak 2 "": >o\
+
+linebreak 3 "": >o\
+k<
+linebreak 4 "": >o\
+Black stones left (in this byo-yomi period): 10];W[qq]WL[200]OW[2]C[White time left: 200 sec
+White stones left: 2];B[sr]BL[87.00]OB[9]C[Black time left: 87 sec
+Black stones left: 9];W[qs]WL[13.20]OW[1]C[White time left: 13.2 sec
+White stones left: 1];B[rs]C[One white stone at s2 captured];W[ps];B[pr];W[or]MN[2]C[Set move number to 2];B[os]C[Two white stones captured
+(at q1 & r1)];MN[112]W[pq]C[Set move number to 112];B[sq];W[rp];B[ps];W[ns];B[ss];W[nr];B[rr];W[sp];);FF[4]AP[Primiview:3.1]GM[1]SZ[19]C[Gametree 2: game-info
+
+Game-info properties are usually stored in the root node.
+If games are merged into a single game-tree, they are stored in the node\
+ where the game first becomes distinguishable from all other games in\
+ the tree.];B[pd](;)(;)(;W[ep];B[pp](;)(;))

data/sample_sgf/simple.sgf

@@ -0,0 +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])

data/sample_sgf/simple_saved.sgf

@@ -0,0 +1,7 @@
+(;GM[1]FF[4]CA[UTF-8]
+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/spec_helper.rb

@@ -1,9 +1,6 @@
 $LOAD_PATH.unshift(File.dirname(__FILE__))
 $LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
 require 'sgf_parser'
-require 'spec'
-require 'spec/autorun'
+require 'rspec'
+require 'rspec/autorun'
 
-Spec::Runner.configure do |config|
-  
-end

data/spec/tree_parser_spec.rb

@@ -6,8 +6,19 @@ describe "SgfParser::Tree.parse" do
     @tree = SgfParser::Tree.new :filename => 'sample_sgf/ff4_ex.sgf'
   end
 
-  it "should return true" do
-    true
+  it "should have FF in the first node" do
+    @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
+    input = StringIO.new "dd][de][ef]"
+    tree = SgfParser::Tree.new
+    tree.instance_variable_set "@stream", input
+    tree.get_property.should == "[dd][de][ef]"
   end
 
 end

data/spec/tree_spec.rb

@@ -3,20 +3,39 @@ require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
 describe "SgfParser::Tree" do
 
   before :each do
-    @tree = SgfParser::Tree.new :filename => 'sample_sgf/ff4_ex.sgf'
+
   end
 
   it "should parse properly" do
-    @tree.class.should == SgfParser::Tree
-    @tree.root.children.size.should == 2
-    @tree.root.children[0].children.size.should == 5
+    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 properly" do
-    @new = 'sample_sgf/ff4_ex_saved.sgf'
-    @tree.save :filename => @new
-    @tree2 = SgfParser::Tree.new :filename => @new
-    @tree2.should == @tree
+  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

metadata

@@ -1,12 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: SgfParser
 version: !ruby/object:Gem::Version 
-  prerelease: false
+  hash: 57
+  prerelease: 
   segments: 
   - 0
   - 9
-  - 0
-  version: 0.9.0
+  - 1
+  version: 0.9.1
 platform: ruby
 authors: 
 - Aldric Giacomoni
@@ -14,23 +15,53 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-04-30 00:00:00 -04:00
+date: 2011-07-29 00:00:00 -04:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
-  name: rspec
-  prerelease: false
+  type: :development
   requirement: &id001 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
+        version: "0"
+  name: jeweler
+  version_requirements: *id001
+  prerelease: false
+- !ruby/object:Gem::Dependency 
+  type: :development
+  requirement: &id002 !ruby/object:Gem::Requirement 
+    none: false
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
+        version: "0"
+  name: rcov
+  version_requirements: *id002
+  prerelease: false
+- !ruby/object:Gem::Dependency 
+  type: :development
+  requirement: &id003 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: 13
         segments: 
         - 1
         - 2
         - 9
         version: 1.2.9
-  type: :development
-  version_requirements: *id001
+  name: rspec
+  version_requirements: *id003
+  prerelease: false
 description: SGFParser is a library that parses and saves SGF (Smart Game Format) files.
 email: aldric@trevoke.net
 executables: []
@@ -42,7 +73,10 @@ extra_rdoc_files:
 - README.rdoc
 files: 
 - .document
-- .gitignore
+- .rspec
+- .rvmrc
+- Gemfile
+- Gemfile.lock
 - LICENSE
 - README.rdoc
 - Rakefile
@@ -53,47 +87,51 @@ files:
 - 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
 - sample_usage/parsing_files.rb
 - spec/node_spec.rb
-- spec/spec.opts
 - spec/spec_helper.rb
+- spec/tree_parser_spec.rb
 - spec/tree_spec.rb
-- tryme.rb
 has_rdoc: true
 homepage: http://github.com/Trevoke/SGFParser
 licenses: []
 
 post_install_message: 
-rdoc_options: 
-- --charset=UTF-8
+rdoc_options: []
+
 require_paths: 
 - lib
 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 
+  none: false
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
+      hash: 3
       segments: 
       - 0
       version: "0"
 requirements: []
 
 rubyforge_project: 
-rubygems_version: 1.3.6
+rubygems_version: 1.6.2
 signing_key: 
 specification_version: 3
 summary: A library for working with SGF files.
-test_files: 
-- spec/node_spec.rb
-- spec/spec_helper.rb
-- spec/tree_parser_spec.rb
-- spec/tree_spec.rb
+test_files: []
+

data/.gitignore

@@ -1,25 +0,0 @@
-## MAC OS
-.DS_Store
-
-## TEXTMATE
-*.tmproj
-tmtags
-
-## EMACS
-*~
-\#*
-.\#*
-
-## VIM
-*.swp
-
-## PROJECT::GENERAL
-coverage
-rdoc
-pkg
-
-## PROJECT::SPECIFIC
-
-
-## Rubymine
-.idea/*

data/spec/spec.opts

@@ -1 +0,0 @@
---color

data/tryme.rb

@@ -1,3 +0,0 @@
-require 'irb/completion'
-require 'lib/sgf_parser'
-sgf = SgfParser::Tree.new :filename => 'sample_sgf/ff4_ex.sgf'