icss 0.1.2 → 0.1.3

data/.watchr

@@ -15,6 +15,6 @@ watch("spec/.*/*_spec\.rb") do |match|
   run_spec match[0]
 end
 
-watch("app/(.*)\.rb") do |match|
+watch("lib/icss/(.*)\.rb") do |match|
   run_spec %{spec/#{match[1]}_spec.rb}
 end

data/Gemfile

@@ -5,6 +5,8 @@ source "http://rubygems.org"
 
 gem   'gorillib',  "~> 0.1.1"
 
+gem   'activemodel', "~> 3.0.9"
+
 # Add dependencies to develop your gem here.
 # Include everything needed to run rake, tests, features, etc.
 group :development do

data/Gemfile.lock

@@ -1,9 +1,16 @@
 GEM
   remote: http://rubygems.org/
   specs:
+    activemodel (3.0.9)
+      activesupport (= 3.0.9)
+      builder (~> 2.1.2)
+      i18n (~> 0.5.0)
+    activesupport (3.0.9)
+    builder (2.1.2)
     diff-lcs (1.1.2)
     git (1.2.5)
     gorillib (0.1.1)
+    i18n (0.5.0)
     jeweler (1.5.2)
       bundler (~> 1.0.0)
       git (>= 1.2.5)
@@ -24,6 +31,7 @@ PLATFORMS
   ruby
 
 DEPENDENCIES
+  activemodel (~> 3.0.9)
   bundler (~> 1.0.0)
   gorillib (~> 0.1.1)
   jeweler (~> 1.5.2)

data/TODO.md

@@ -0,0 +1,31 @@
+# Proposed ICSS refactoring:
+
+* one ICSS <=> one Dataset <=> one Catalog Entry
+
+
+### catalog_entry
+
+Most of it moves to top-level:
+* tags:         move to top-level attribute
+* title:        disappears -- take from `protocol`
+* description:  disappears -- take from `doc`
+* owner:        stays in catalog_entry
+* price:        attach to bulk data target
+
+* messages:     only necessary to select among messages
+* packages:     only necessary to select among packages
+
+move to a new top-level section, `provenance`:
+
+* license
+* link -> becomes sources
+
+moves *to* catalog_entry:
+
+* `under_consideration` 
+* `update_frequency` (?? def. doesn't feel like a top-level thing, but this feels weird too)
+
+
+___________________________________________________________________________
+
+Make the get_data part of culture/books/comics/grand_comics_db a three-liner

data/VERSION

@@ -1 +1 @@
-0.1.2
+0.1.3

data/examples/chronic.icss.yaml

@@ -2,6 +2,9 @@
 namespace: util.time
 protocol: chronic
 
+doc: >-
+  A series of calls hooking into the Chronic ruby gem
+
 code_assets:
   - location:   code/chronic_endpoint.rb
     type:       apeyeye_endpoint

data/icss.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{icss}
-  s.version = "0.1.2"
+  s.version = "0.1.3"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Philip (flip) Kromer for Infochimps"]
-  s.date = %q{2011-07-08}
+  s.date = %q{2011-07-15}
   s.description = %q{Infochimps Stupid Schema library: an avro-compatible data description standard. ICSS completely describes a collection of data (and associated assets) in a way that is expressive, scalable and sufficient to drive remarkably complex downstream processes.}
   s.email = %q{coders@infochimps.com}
   s.extra_rdoc_files = [
@@ -26,6 +26,7 @@ Gem::Specification.new do |s|
     "LICENSE.textile",
     "README.textile",
     "Rakefile",
+    "TODO.md",
     "VERSION",
     "examples/avro_examples/BulkData.avpr",
     "examples/avro_examples/complicated.icss.yaml",
@@ -61,7 +62,8 @@ Gem::Specification.new do |s|
     "lib/icss/view_helper.rb",
     "spec/icss_spec.rb",
     "spec/protocol_spec.rb",
-    "spec/spec_helper.rb"
+    "spec/spec_helper.rb",
+    "spec/validations_spec.rb"
   ]
   s.homepage = %q{http://github.com/mrflip/icss}
   s.licenses = ["MIT"]
@@ -71,7 +73,8 @@ Gem::Specification.new do |s|
   s.test_files = [
     "spec/icss_spec.rb",
     "spec/protocol_spec.rb",
-    "spec/spec_helper.rb"
+    "spec/spec_helper.rb",
+    "spec/validations_spec.rb"
   ]
 
   if s.respond_to? :specification_version then
@@ -80,6 +83,7 @@ Gem::Specification.new do |s|
 
     if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
       s.add_runtime_dependency(%q<gorillib>, ["~> 0.1.1"])
+      s.add_runtime_dependency(%q<activemodel>, ["~> 3.0.9"])
       s.add_development_dependency(%q<rspec>, ["~> 2.3.0"])
       s.add_development_dependency(%q<yard>, ["~> 0.6.0"])
       s.add_development_dependency(%q<bundler>, ["~> 1.0.0"])
@@ -87,6 +91,7 @@ Gem::Specification.new do |s|
       s.add_development_dependency(%q<rcov>, [">= 0"])
     else
       s.add_dependency(%q<gorillib>, ["~> 0.1.1"])
+      s.add_dependency(%q<activemodel>, ["~> 3.0.9"])
       s.add_dependency(%q<rspec>, ["~> 2.3.0"])
       s.add_dependency(%q<yard>, ["~> 0.6.0"])
       s.add_dependency(%q<bundler>, ["~> 1.0.0"])
@@ -95,6 +100,7 @@ Gem::Specification.new do |s|
     end
   else
     s.add_dependency(%q<gorillib>, ["~> 0.1.1"])
+    s.add_dependency(%q<activemodel>, ["~> 3.0.9"])
     s.add_dependency(%q<rspec>, ["~> 2.3.0"])
     s.add_dependency(%q<yard>, ["~> 0.6.0"])
     s.add_dependency(%q<bundler>, ["~> 1.0.0"])

data/lib/icss.rb

@@ -10,9 +10,8 @@ require 'gorillib/metaprogramming/class_attribute'
 
 require 'gorillib/hashlike'
 require 'gorillib/receiver'
-require 'gorillib/receiver/acts_as_hash'
-require 'gorillib/receiver/acts_as_loadable'
-require 'gorillib/receiver/validations'
+require 'gorillib/receiver_model'
+
 require 'time' # ain't that always the way
 
 $: << File.dirname(__FILE__)

data/lib/icss/protocol.rb

@@ -55,11 +55,12 @@ module Icss
     include Receiver
     include Receiver::ActsAsHash
     include Receiver::ActsAsLoadable
-    include Icss::Validations
+    include Gorillib::Hashlike::TreeMerge
+    include Receiver::ActiveModelShim
 
     rcvr_accessor :protocol,    String, :required => true
     alias_method  :name, :protocol
-    rcvr_accessor :namespace,   String # must be *dotted* ("foo.bar"), not slashed ("foo/bar")
+    rcvr_accessor :namespace,   String
     rcvr_accessor :doc,         String
     #
     rcvr_accessor :types,       Array, :of => Icss::TypeFactory, :default => []
@@ -69,17 +70,21 @@ module Icss
     rcvr_accessor :code_assets, Array, :of => Icss::CodeAsset,   :default => []
     rcvr_accessor :targets,     Hash,  :of => Icss::TargetListFactory, :default => {}, :merge_as => :hash_of_arrays
     rcvr_accessor :under_consideration, Boolean
-    rcvr_accessor :update_frequency, String # must be of the form daily, weekly, monthly, quarterly, never
+    rcvr_accessor :update_frequency, String
+
+    validates_presence_of :protocol, :namespace
+    validates_format_of :protocol,  :with => /\A[A-Za-z_]\w*\z/,       :message => "must start with [A-Za-z_] and contain only [A-Za-z0-9_].",                                            :allow_blank => true
+    validates_format_of :namespace, :with => /\A([A-Za-z_]\w*\.?)+\z/, :message => "must be a dot-separated sequence of avro names (start with [A-Za-z_] and contain only [A-Za-z0-9_])", :allow_blank => true
+    validates_format_of :update_frequency, :with => /daily|weekly|monthly|quarterly|never/, :message => "must be one of daily, weekly, monthly, quarterly, never",                        :allow_blank => true
 
-    # attr_accessor :body
     after_receive do |hsh|
       # Set each message's protocol to self, and if the name wasn't given, set
       # it using the message's hash key.
       self.messages.each{|msg_name, msg| msg.protocol = self; msg.name ||= msg_name }
-      # Set all the type's parent to self (for namespace resolution)
+      # Set each type's parent to self (for namespace resolution)
       self.types.each{|type| type.parent  = self }
-      validate_name
-      validate_namespace
+      # warn if invalid
+      warn errors.inspect unless valid?
     end
 
     # String: namespace.name
@@ -87,14 +92,11 @@ module Icss
       [namespace, name].compact.join(".")
     end
 
+    # a / separated version of the name, with no / at start
     def path
       fullname.gsub('.', '/')
     end
 
-    def tree_merge! arg
-      to_hash.tree_merge!(arg)
-    end
-
     def find_message nm
       return if messages.blank?
       nm = nm.to_s.gsub("/", ".").split(".").last
@@ -107,11 +109,13 @@ module Icss
       self.namespace = namespace_and_name.join('.')
     end
 
-    def receive_targets hsh
-      self.targets = hsh.inject({}) do |target_obj_hsh, (target_name, target_info_list)|
-        target_obj_hsh[target_name] = TargetListFactory.receive(target_info_list, target_name) # returns an arry of targets
-        target_obj_hsh
+    def receive_targets tgts
+      return unless tgts.present?
+      self.targets ||= {}
+      tgts.each do |target_name, target_info_list|
+        targets[target_name] = TargetListFactory.receive(target_info_list, target_name) # array of targets
       end
+      targets
     end
 
     def to_hash()

data/spec/protocol_spec.rb

@@ -1,10 +1,116 @@
 require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
 require 'icss'
 
-
 describe Icss::Protocol do
-  before do
-    @icss = Icss::Protocol.receive_from_file(ICSS_ROOT_DIR('examples/chronic.icss.yaml'))
+  let(:simple_icss) do
+    Icss::Protocol.receive_from_file(ICSS_ROOT_DIR('examples/chronic.icss.yaml'))
+  end
+
+  it 'loads cleanly' do
+    simple_icss.name.should == 'chronic'
+    simple_icss.fullname.should == 'util.time.chronic'
+  end
+
+  describe '#fullname' do
+    it 'has namespace and name' do
+      simple_icss.fullname.should == 'util.time.chronic'
+      simple_icss.namespace = nil
+      simple_icss.fullname.should == 'chronic'
+    end
+  end
+  describe '#path' do
+    it 'is a / separated version of the name, with no / at start' do
+      simple_icss.path.should == 'util/time/chronic'
+    end
+  end
+  describe '#doc' do
+    it 'takes a doc string' do
+      simple_icss.doc.should == 'A series of calls hooking into the Chronic ruby gem'
+    end
+  end
+
+  describe 'types' do
+    it 'has an array of types' do
+      simple_icss.types.map(&:name).should == ['chronic_parse_params', 'chronic_parse_response']
+    end
+  end
+
+  describe 'messages' do
+    it 'has a hash of messages' do
+      simple_icss.messages.keys.should == ['parse']
+    end
+
+    it 'named each message for its key' do
+      simple_icss.messages['parse'].name.should == 'parse'
+      simple_icss.receive!({ :messages => { 'foo' => { :request => [] } }})
+      simple_icss.messages['foo'].name.should == 'foo'
+    end
+
+    it '#find_message' do
+      msg = simple_icss.find_message(:parse)
+      msg.should be_a(Icss::Message) ; msg.name.should == 'parse'
+      msg = simple_icss.find_message('util.time.parse')
+      msg.should be_a(Icss::Message) ; msg.name.should == 'parse'
+      msg = simple_icss.find_message('util/time/parse')
+      msg.should be_a(Icss::Message) ; msg.name.should == 'parse'
+    end
+  end
+
+  describe 'targets' do
+    it 'has a hash of targets' do
+      simple_icss.targets.keys.should == ['catalog']
+      simple_icss.targets['catalog'].first.should be_a(Icss::CatalogTarget)
+      simple_icss.targets['catalog'].first.name.should == 'util_time_chronic_parse'
+    end
+  end
+
+  # describe 'tree_merge!' do
+  #   it 'merges recursively'
+  # end
+
+  describe 'validations' do
+    it 'validates protocol name' do
+      simple_icss.should be_valid
+      simple_icss.protocol = '' ;    simple_icss.should_not be_valid ; simple_icss.errors[:protocol].should == ["can't be blank"]
+      simple_icss.protocol = '1bz' ; simple_icss.should_not be_valid ; simple_icss.errors[:protocol].should == ["must start with [A-Za-z_] and contain only [A-Za-z0-9_]."]
+    end
+
+    it 'validates namespace' do
+      simple_icss.should be_valid
+      simple_icss.namespace = '' ;    simple_icss.should_not be_valid ; simple_icss.errors[:namespace].should == ["can't be blank"]
+      simple_icss.namespace = '1bz' ; simple_icss.should_not be_valid ; simple_icss.errors[:namespace].first.should =~ /must be a dot-s/
+    end
+  end
+
+  describe '#to_hash' do
+    it 'roundtrips' do
+      hsh = simple_icss.to_hash
+      hsh.should == {
+        :namespace=>"util.time", :protocol=>"chronic",
+        :doc=>"A series of calls hooking into the Chronic ruby gem",
+        :types => [
+          {:name=>"chronic_parse_params",   :doc=>"Query API parameters for the /util/time/chronic/parse call", :type=>:record,
+            :fields=>[
+              {:name=>"time_str", :type=>:string, :doc=>"The string to parse."},
+              {:name=>"context", :type=>:symbol, :doc=>"<tt>past</tt> or <tt>future</tt> (defaults to <tt>future</tt>)\nIf your string represents a birthday, you can set <tt>context</tt> to <tt>past</tt> and if an ambiguous string is given, it will assume it is in the past. Specify <tt>future</tt> or omit to set a future context."},
+              {:name=>"now", :type=>:time, :doc=>"Time (defaults to Time.now)\nBy setting <tt>:now</tt> to a Time, all computations will be based off of that time instead of Time.now. If set to nil, Chronic will use the current time in UTC. You must supply a date that unambiguously parses with the much-less-generous ruby Time.parse()"},
+              {:name=>"ambiguous_time_range", :type=>:int, :doc=>"Integer or <tt>:none</tt> (defaults to <tt>6</tt> (6am-6pm))\nIf an Integer is given, ambiguous times (like 5:00) will be assumed to be within the range of that time in the AM to that time in the PM. For example, if you set it to <tt>7</tt>, then the parser will look for the time between 7am and 7pm. In the case of 5:00, it would assume that means 5:00pm. If <tt>:none</tt> is given, no assumption will be made, and the first matching instance of that time will be used."}
+            ]},
+          {:name=>"chronic_parse_response", :doc=>"Query API response for the /util/time/chronic/parse call",   :type=>:record,
+            :fields=>[
+              {:name=>"time", :type=>:string, :doc=>"The UTC parsed time, as a \"ISO 8601 combined date time\":http://en.wikipedia.org/wiki/ISO_8601 string."},
+              {:name=>"epoch_seconds", :type=>:int, :doc=>"The UTC parsed time, as \"epoch seconds\":http://en.wikipedia.org/wiki/Epoch_seconds integer."}
+            ]}
+        ],
+        :messages => {
+          'parse' => {:doc=>"\nChronic is a natural language date/time parser written in pure Ruby. See below\nfor the wide variety of formats Chronic will parse.", :request=>[{:name=>"chronic_parse_params", :type=>"chronic_parse_params"}], :response=>"chronic_parse_response", :samples=>[{:request=>[{"time_str"=>"one hour ago", "now"=>"2007-03-16T12:09:08Z"}], :response=>{"epoch_seconds"=>1174043348, "time"=>"2007-03-16T11:09:08Z"}, :url=>"?now=2007-03-16T12%3A09%3A08Z&time_str=one%20hour%20ago"}, {:request=>[{"time_str"=>"Yesterday", "now"=>"5:06:07T2010-08-08Z"}], :response=>{"epoch_seconds"=>1281182400, "time"=>"2010-08-07T12:00:00Z"}, :url=>"?now=5%3A06%3A07%202010-08-08&time_str=Yesterday"}, {:url=>"?time_str=5pm+on+November+4th&context=past"}]},
+        },
+        :data_assets=>[],
+        :code_assets=>[{:location=>"code/chronic_endpoint.rb"}],
+        :targets => {"catalog"=>[{:name=>"util_time_chronic_parse", :title=>"Utils - Parse Times", :description=>"An API call to parse human-readable date / time strings", :tags=>["apiawesome", "ruby", "gems", "chronic", "time", "date", "util", "parse"], :messages=>["parse"]}]},
+      }
+    end
   end
 
 end
+

data/spec/validations_spec.rb

@@ -0,0 +1,171 @@
+require 'rubygems'
+require 'rspec'
+require 'icss'
+
+def template_icss
+  return <<EOS
+---
+namespace: foo.bar
+protocol: baz
+
+data_assets:
+- name: test_data_asset
+  location: data/test_data.tsv
+  type: test_data_record
+
+messages:
+  search:
+    doc: A testable template message
+    request:
+    - name: test_request
+      type: test_request_record
+    response: test_data_record
+    samples:
+    - request:
+      - first_param: foo
+        second_param: bar
+
+targets:
+  catalog:
+  - name: test_catalog_entry
+    title: Test Icss
+    description: This is a template Icss to test the error handling of the Icss library.
+    tags:
+    - test
+    - icss
+    - template
+    messages:
+    - search
+    packages:
+    - data_assets:
+      - test_data_asset
+
+types:
+- name: test_request_record
+  doc: A template request record
+  type: record
+  fields:
+  - name: first_param
+    doc: The first test parameter
+    type: string
+  - name: second param
+    doc: The second test parameter
+    type: string
+
+- name: test_complex_data
+  doc: A template complex data type
+  type: record
+  fields:
+  - name: field_one
+    doc: A simple field for a complex data type
+    type: string
+  - name: field_two
+    doc: Another simple field for a complex data type
+    type: string
+
+- name: test_data_record
+  doc: A template data record
+  type: record
+  fields:
+  - name: simple_data
+    doc: A simple piece of data
+    type: int
+  - name: complex_data
+    doc: A complex piece of data
+    type: test_complex_data
+
+EOS
+end
+
+describe "Icss::Protocol validations" do
+  before :each do
+    @template = YAML.load(template_icss)
+  end
+
+  it "should be able to receive a correctly formatted template Icss" do
+    lambda { @icss =Icss::Protocol.receive @template }.should_not raise_error
+    @icss.errors.should be_empty
+  end
+
+  it "should contain only the keys that were included in the Icss file" do
+    @icss = Icss::Protocol.receive @template
+    @icss.keys.map { |k| k.to_s }.should == @template.keys
+  end
+
+  it "should generate an error when the namespace if formatted incorrectly" do
+    @template['namespace'] = '$bad_namespace'
+    Icss::Protocol.receive(@template).errors.keys.should == [:namespace]
+  end
+
+  it "should generate an error when the protocol if formatted incorrectly" do
+    @template['protocol'] = '$bad_protocol'
+    Icss::Protocol.receive(@template).errors.keys.should == [:protocol]
+  end
+
+  context "Catalog Target" do
+
+    it "should generate an error when an undefined data_asset is specified" do
+      @template['targets']['catalog'].first['packages'].first['data_assets'] = ['fake_data_asset']
+      Icss::Protocol.receive(@template).errors.keys.should == [:catalog]
+    end
+
+    it "should generate an error when an undefined message name is specified" do
+      @template['targets']['catalog'].first['messages'] = ['fake_message']
+      Icss::Protocol.receive(@template).errors.keys.should == [:catalog]
+    end
+
+  end
+
+  context "Data Assets" do
+
+    it "should generate an error when an asset's type is undefined" do
+      @template['data_assets'].first['type'] = 'fake_data_asset'
+      Icss::Protocol.receive(@template).errors.keys.should == [:data_assets]
+    end
+
+  end
+
+  context "Messages" do
+
+    it "should generate an error when a message's request type is undefined" do
+      @template['messages']['search']['request'].first['type'] = 'fake_request_type'
+      Icss::Protocol.receive(@template).errors.keys.should == [:messages]
+    end
+
+    it "should generate an error when a message's response type is undefined" do
+      @template['messages']['search']['response'] = 'fake_response_type'
+      Icss::Protocol.receive(@template).errors.keys.should == [:messages]
+    end
+
+    it "should generate an error when a message's sample request types do not match the request record" do
+      @template['messages']['search']['samples'].first['request'] = [{ 'foo' => 'bar' }]
+      Icss::Protocol.receive(@template).errors.keys.should == [:messages]
+    end
+
+  end
+
+  context "Types" do
+
+    it "should generate an error when an bad type definition is given for a specific type" do
+      @template['types'].push({
+          'name' => 'fake_type_record',
+          'type' => 'fake'
+        })
+      Icss::Protocol.receive(@template).errors.keys.should == [:types]
+    end
+
+    it "should generate an error when an undefined type definition is given for a specific field" do
+      @template['types'].push({
+          'name' => 'fake_type_record',
+          'type' => 'record',
+          'fields' => [{
+              'name' => 'fake_field',
+              'type' => 'fake_type'
+            }]
+        })
+      Icss::Protocol.receive(@template).errors.keys.should == [:types]
+    end
+
+  end
+
+end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: icss
 version: !ruby/object:Gem::Version 
-  hash: 31
+  hash: 29
   prerelease: false
   segments: 
   - 0
   - 1
-  - 2
-  version: 0.1.2
+  - 3
+  version: 0.1.3
 platform: ruby
 authors: 
 - Philip (flip) Kromer for Infochimps
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-07-08 00:00:00 -05:00
+date: 2011-07-15 00:00:00 -05:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -36,6 +36,22 @@ dependencies:
   version_requirements: *id001
 - !ruby/object:Gem::Dependency 
   requirement: &id002 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ~>
+      - !ruby/object:Gem::Version 
+        hash: 21
+        segments: 
+        - 3
+        - 0
+        - 9
+        version: 3.0.9
+  type: :runtime
+  name: activemodel
+  prerelease: false
+  version_requirements: *id002
+- !ruby/object:Gem::Dependency 
+  requirement: &id003 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ~>
@@ -49,9 +65,9 @@ dependencies:
   type: :development
   name: rspec
   prerelease: false
-  version_requirements: *id002
+  version_requirements: *id003
 - !ruby/object:Gem::Dependency 
-  requirement: &id003 !ruby/object:Gem::Requirement 
+  requirement: &id004 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ~>
@@ -65,9 +81,9 @@ dependencies:
   type: :development
   name: yard
   prerelease: false
-  version_requirements: *id003
+  version_requirements: *id004
 - !ruby/object:Gem::Dependency 
-  requirement: &id004 !ruby/object:Gem::Requirement 
+  requirement: &id005 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ~>
@@ -81,9 +97,9 @@ dependencies:
   type: :development
   name: bundler
   prerelease: false
-  version_requirements: *id004
+  version_requirements: *id005
 - !ruby/object:Gem::Dependency 
-  requirement: &id005 !ruby/object:Gem::Requirement 
+  requirement: &id006 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ~>
@@ -97,9 +113,9 @@ dependencies:
   type: :development
   name: jeweler
   prerelease: false
-  version_requirements: *id005
+  version_requirements: *id006
 - !ruby/object:Gem::Dependency 
-  requirement: &id006 !ruby/object:Gem::Requirement 
+  requirement: &id007 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
@@ -111,7 +127,7 @@ dependencies:
   type: :development
   name: rcov
   prerelease: false
-  version_requirements: *id006
+  version_requirements: *id007
 description: "Infochimps Stupid Schema library: an avro-compatible data description standard. ICSS completely describes a collection of data (and associated assets) in a way that is expressive, scalable and sufficient to drive remarkably complex downstream processes."
 email: coders@infochimps.com
 executables: []
@@ -131,6 +147,7 @@ files:
 - LICENSE.textile
 - README.textile
 - Rakefile
+- TODO.md
 - VERSION
 - examples/avro_examples/BulkData.avpr
 - examples/avro_examples/complicated.icss.yaml
@@ -167,6 +184,7 @@ files:
 - spec/icss_spec.rb
 - spec/protocol_spec.rb
 - spec/spec_helper.rb
+- spec/validations_spec.rb
 has_rdoc: true
 homepage: http://github.com/mrflip/icss
 licenses: 
@@ -205,3 +223,4 @@ test_files:
 - spec/icss_spec.rb
 - spec/protocol_spec.rb
 - spec/spec_helper.rb
+- spec/validations_spec.rb