framecurve 1.0.1 → 1.0.2

data/framecurve.gemspec

@@ -5,7 +5,7 @@
 
 Gem::Specification.new do |s|
   s.name = "framecurve"
-  s.version = "1.0.1"
+  s.version = "1.0.2"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Julik"]

data/lib/framecurve.rb

@@ -1,5 +1,5 @@
 module Framecurve
-  VERSION = "1.0.1"
+  VERSION = "1.0.2"
   
   # Is raised when a malformed framecurve bit has occurred in the system
   class Malformed < RuntimeError

data/lib/framecurve/validator.rb

@@ -62,6 +62,19 @@ class Framecurve::Validator
     end
   end
   
+  def verify_non_negative_source_and_destination_frames(curve)
+    curve.each_with_index do | t, i |
+      next unless t.tuple?
+      
+      line_no = i + 1
+      if t.at < 0
+        @errors.push("The tuple %d had a negative at_frame value (%d). The spec mandates positive frames." % [line_no, t.at])
+      elsif t.value < 0
+        @errors.push("The tuple %d had a negative use_frame_of_source value (%.5f). The spec mandates positive frames." % [line_no, t.value])
+      end
+    end
+  end
+  
   def verify_file_naming(curve)
     return unless curve.respond_to?(:filename) && curve.filename
     unless curve.filename =~ /\.framecurve\.txt$/

data/test/test_framecurve_validator.rb

@@ -75,4 +75,14 @@ class TestFramecurveValidator < Test::Unit::TestCase
     assert v.any_errors?
     assert_equal ["The frame sequencing is out of order (expected [1, 10] but got [10, 1]). The framecurve spec mandates that frames are recorded sequentially"], v.errors
   end
+  
+  def test_should_error_out_with_neg_source_and_dest_values
+     c = Framecurve::Curve.new( Framecurve::Tuple.new(-10, 123.4), Framecurve::Tuple.new(1, -345.67) )
+     v = Framecurve::Validator.new
+     v.validate(c)
+     assert v.any_errors?
+     errs = ["The tuple 1 had a negative at_frame value (-10). The spec mandates positive frames.",
+      "The tuple 2 had a negative use_frame_of_source value (-345.67000). The spec mandates positive frames."]
+     assert_equal errs, v.errors
+   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: framecurve
 version: !ruby/object:Gem::Version
-  version: 1.0.1
+  version: 1.0.2
   prerelease: 
 platform: ruby
 authors:
@@ -13,7 +13,7 @@ date: 2011-12-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: term-ansicolor
-  requirement: &10821110 !ruby/object:Gem::Requirement
+  requirement: &10767710 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *10821110
+  version_requirements: *10767710
 - !ruby/object:Gem::Dependency
   name: jeweler
-  requirement: &10820040 !ruby/object:Gem::Requirement
+  requirement: &10767420 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -32,10 +32,10 @@ dependencies:
         version: 1.6.4
   type: :development
   prerelease: false
-  version_requirements: *10820040
+  version_requirements: *10767420
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &10817530 !ruby/object:Gem::Requirement
+  requirement: &10767130 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +43,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *10817530
+  version_requirements: *10767130
 - !ruby/object:Gem::Dependency
   name: cli_test
-  requirement: &10816550 !ruby/object:Gem::Requirement
+  requirement: &10766770 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,7 +54,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *10816550
+  version_requirements: *10766770
 description: ! ' Parser, validation and interpolation'
 email: me@julik.nl
 executables: