lucid_works 0.7.18 → 0.9.4

data/lib/lucid_works.rb

@@ -9,13 +9,13 @@ require 'active_support/inflector'
 require 'restclient'
 require 'json'
 require 'rsolr'
-require 'rsolr-ext'
 require 'nokogiri'
 
 require 'lucid_works/utils'
 require 'lucid_works/patch_restclient'
 require 'lucid_works/patch_time'
 require 'lucid_works/simple_naming'
+require 'lucid_works/inflections'
 
 require 'lucid_works/exceptions'
 require 'lucid_works/associations'
@@ -43,6 +43,7 @@ require 'lucid_works/activity/status'
 require 'lucid_works/activity/history'
 require 'lucid_works/crawler'
 require 'lucid_works/datasource'
+require 'lucid_works/datasource/mapping'
 require 'lucid_works/datasource/index'
 require 'lucid_works/datasource/status'
 require 'lucid_works/datasource/history'
@@ -51,17 +52,23 @@ require 'lucid_works/datasource/crawldata'
 require 'lucid_works/datasource/job'
 require 'lucid_works/datasource_type'
 require 'lucid_works/datasource_property'
+require 'lucid_works/dynamicfield'
 require 'lucid_works/field'
 require 'lucid_works/fieldtype'
 require 'lucid_works/jdbcdriver'
 require 'lucid_works/role'
+require 'lucid_works/cache'
 require 'lucid_works/synonym'
+require 'lucid_works/elevation'
 require 'lucid_works/logs'
 require 'lucid_works/logs/query'
 require 'lucid_works/logs/query/summary'
 require 'lucid_works/logs/index'
 require 'lucid_works/logs/index/summary'
+require 'lucid_works/server/crawlers_status'
 require 'lucid_works/version'
+require 'lucid_works/cluster'
+require 'lucid_works/request_handler'
 
 I18n.load_path += Dir.glob(File.join(File.dirname(__FILE__), '..', 'config', 'locales', '*.yml'))
 I18n.reload! if Rails.env == 'development' rescue nil

data/lucid_works.gemspec

@@ -6,10 +6,10 @@ Gem::Specification.new do |s|
   s.name        = "lucid_works"
   s.version     = LucidWorks::VERSION
   s.platform    = Gem::Platform::RUBY
-  s.authors     = ["Sam Pierson"]
-  s.email       = ["sam.pierson@lucidimagination.com"]
-  s.homepage    = ""
-  s.summary     = %q{Ruby wrapper for the LucidWorks REST API}
+  s.authors     = ["Lucid Imagination"]
+  s.email       = ["support@lucidimagination.com"]
+  s.homepage    = "http://github.com/lucidimagination/lucidworks-ruby"
+  s.summary     = %q{LucidWorks REST API client}
   s.description = %q{Ruby wrapper for the LucidWorks REST API}
 
   s.rubyforge_project = "lucid_works"
@@ -19,11 +19,10 @@ Gem::Specification.new do |s|
   s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
   s.require_paths = ["lib"]
 
-  s.add_runtime_dependency 'activesupport', '>= 3'
-  s.add_runtime_dependency 'activemodel',   '>= 3'
-  s.add_runtime_dependency 'rest-client',   '>= 1.6.1'
+  s.add_runtime_dependency 'activesupport', '>= 4'
+  s.add_runtime_dependency 'activemodel', '>= 4'
+  s.add_runtime_dependency 'rest-client', '1.6.7'
   s.add_runtime_dependency 'json'
-  s.add_runtime_dependency 'rsolr'
-  s.add_runtime_dependency 'rsolr-ext'
+  s.add_runtime_dependency 'rsolr', '1.0.9'
   s.add_runtime_dependency 'nokogiri'
 end

data/spec/fixtures/zookeeper/clusterstate.json

@@ -0,0 +1,30 @@
+{
+  "znode":{
+    "path":"/clusterstate.json",
+    "prop":{
+      "version":15,
+      "aversion":0,
+      "children_count":0,
+      "ctime":"Wed Aug 22 16:38:45 PDT 2012 (1345678725057)",
+      "cversion":0,
+      "czxid":12,
+      "dataLength":1803,
+      "ephemeralOwner":0,
+      "mtime":"Wed Aug 22 16:39:07 PDT 2012 (1345678747682)",
+      "mzxid":149,
+      "pzxid":12
+    },
+  "data":"{\n  \"collection1\":{\n    \"shards\":{\n    \"shard1\":{\n      \"range\":\"80000000-ffffffff\",\n      \"replicas\":{\n        \"myserver:8888_solr_collection1\":{\n          \"shard\":\"shard1\",\n          \"roles\":null,\n          \"state\":\"active\",\n          \"core\":\"collection1\",\n          \"collection\":\"collection1\",\n          \"node_name\":\"myserver:8888_solr\",\n          \"base_url\":\"http://myserver:8888/solr\",\n          \"leader\":\"true\"},\n        \"myserver:14888_solr_collection1\":{\n          \"shard\":\"shard1\",\n          \"roles\":null,\n          \"state\":\"recovering\",\n          \"core\":\"collection1\",\n          \"collection\":\"collection1\",\n          \"node_name\":\"myserver:14888_solr\",\n          \"base_url\":\"http://myserver:14888/solr\"}}},\n    \"shard2\":{\n      \"range\":\"0-7fffffff\",\n      \"replicas\":{\"myserver:13888_solr_collection1\":{\n          \"shard\":\"shard2\",\n          \"roles\":null,\n          \"state\":\"active\",\n          \"core\":\"collection1\",\n          \"collection\":\"collection1\",\n          \"node_name\":\"myserver:13888_solr\",\n          \"base_url\":\"http://myserver:13888/solr\",\n          \"leader\":\"true\"}}}}},\n  \"LucidWorksLogs\":{\n    \"shards\":{\n    \"shard1\":{\n      \"range\":\"80000000-ffffffff\",\n      \"replicas\":{\n        \"myserver:8888_solr_LucidWorksLogs\":{\n          \"shard\":\"shard1\",\n          \"roles\":null,\n          \"state\":\"active\",\n          \"core\":\"LucidWorksLogs\",\n          \"collection\":\"LucidWorksLogs\",\n          \"node_name\":\"myserver:8888_solr\",\n          \"base_url\":\"http://myserver:8888/solr\",\n          \"leader\":\"true\"},\n        \"myserver:14888_solr_LucidWorksLogs\":{\n          \"shard\":\"shard1\",\n          \"roles\":null,\n          \"state\":\"recovering\",\n          \"core\":\"LucidWorksLogs\",\n          \"collection\":\"LucidWorksLogs\",\n          \"node_name\":\"myserver:14888_solr\",\n          \"base_url\":\"http://myserver:14888/solr\"}}},\n    \"shard2\":{\n      \"range\":\"0-7fffffff\",\n      \"replicas\":{\"myserver:13888_solr_LucidWorksLogs\":{\n          \"shard\":\"shard2\",\n          \"roles\":null,\n          \"state\":\"active\",\n          \"core\":\"LucidWorksLogs\",\n          \"collection\":\"LucidWorksLogs\",\n          \"node_name\":\"myserver:13888_solr\",\n          \"base_url\":\"http://myserver:13888/solr\",\n          \"leader\":\"true\"}}}}}}"
+  },
+  "tree":[
+    {
+      "data":{
+        "title":"/clusterstate.json",
+        "attr":{
+          "href":"zookeeper?detail=true&path=%2Fclusterstate.json"
+        }
+      }
+    }
+  ]
+}
+          

data/spec/fixtures/zookeeper/clusterstate_broken_shard.json

@@ -0,0 +1,29 @@
+{
+  "znode":{
+    "path":"/clusterstate.json",
+    "prop":{
+      "version":16,
+      "aversion":0,
+      "children_count":0,
+      "ctime":"Sun Jul 15 22:58:05 UTC 2012 (1342393085065)",
+      "cversion":0,
+      "czxid":12,
+      "dataLength":1250,
+      "ephemeralOwner":0,
+      "mtime":"Wed Jul 18 16:16:21 UTC 2012 (1342628181385)",
+      "mzxid":307,
+      "pzxid":12
+    },
+    "data":"{\n  \"collection1\":{\n    \"shards\":{\n    \"shard1\":{\n      \"range\":\"80000000-ffffffff\",\n      \"replicas\":{\n        \"myserver:8888_solr_collection1\":{\n        \"shard\":\"shard1\",\n        \"leader\":\"true\",\n        \"state\":\"active\",\n        \"core\":\"collection1\",\n        \"collection\":\"collection1\",\n        \"node_name\":\"myserver:8888_solr\",\n        \"base_url\":\"http://myserver:8888/solr\"}}}},\n    \"shard2\":{}},\n  \"LucidWorksLogs\":{\n    \"shards\":{\n    \"shard1\":{\n      \"range\":\"80000000-ffffffff\",\n      \"replicas\":{\n        \"myserver:8888_solr_LucidWorksLogs\":{\n        \"shard\":\"shard1\",\n        \"leader\":\"true\",\n        \"state\":\"active\",\n        \"core\":\"LucidWorksLogs\",\n        \"collection\":\"LucidWorksLogs\",\n        \"node_name\":\"myserver:8888_solr\",\n        \"base_url\":\"http://myserver:8888/solr\"}}},\n    \"shard2\":{\n      \"range\":\"80000000-ffffffff\",\n      \"replicas\":{\n        \"myserver:13888_solr_LucidWorksLogs\":{\n        \"shard\":\"shard2\",\n        \"leader\":\"true\",\n        \"state\":\"active\",\n        \"core\":\"LucidWorksLogs\",\n        \"collection\":\"LucidWorksLogs\",\n        \"node_name\":\"myserver:13888_solr\",\n        \"base_url\":\"http://myserver:13888/solr\"}}}}}}"
+  },
+  "tree":[
+    {
+      "data":{
+        "title":"/clusterstate.json",
+        "attr":{
+          "href":"zookeeper?detail=true&path=%2Fclusterstate.json"
+        }
+      }
+    }
+  ]
+}

data/spec/fixtures/zookeeper/live_nodes.json

@@ -0,0 +1,28 @@
+{"znode":{
+    "path":"/live_nodes","prop":{
+      "version":0,
+      "aversion":0,
+      "children_count":3,
+      "ctime":"Wed Aug 22 16:38:44 PDT 2012 (1345678724992)",
+      "cversion":3,
+      "czxid":2,
+      "dataLength":0,
+      "ephemeralOwner":0,
+      "mtime":"Wed Aug 22 16:38:44 PDT 2012 (1345678724992)",
+      "mzxid":2,
+      "pzxid":132}},"tree":[{"data":{
+        "title":"/live_nodes","attr":{
+          "href":"zookeeper?detail=true&path=%2Flive_nodes"}},
+      "children":[{"data":{
+            "title":"myserver:13888_solr","attr":{
+              "href":"zookeeper?detail=true&path=%2Flive_nodes%2Fmyserver%3A13888_solr"}},
+          "ephemeral":true,
+          "version":0},{"data":{
+            "title":"myserver:14888_solr","attr":{
+              "href":"zookeeper?detail=true&path=%2Flive_nodes%2Fmyserver%3A14888_solr"}},
+          "ephemeral":true,
+          "version":0},{"data":{
+            "title":"myserver:8888_solr","attr":{
+              "href":"zookeeper?detail=true&path=%2Flive_nodes%2Fmyserver%3A8888_solr"}},
+          "ephemeral":true,
+          "version":0}]}]}

data/spec/fixtures/zookeeper/live_nodes_no_children.json

@@ -0,0 +1,26 @@
+{
+  "znode":{
+    "path":"/live_nodes",
+    "prop":{
+      "version":0,
+      "aversion":0,
+      "children_count":1,
+      "ctime":"Sun Jul 15 22:58:05 UTC 2012 (1342393085027)",
+      "cversion":17,
+      "czxid":2,
+      "dataLength":0,
+      "ephemeralOwner":0,
+      "mtime":"Sun Jul 15 22:58:05 UTC 2012 (1342393085027)",
+      "mzxid":2,
+      "pzxid":298
+    }
+  },
+  "tree":[{
+    "data":{
+      "title":"/live_nodes",
+      "attr":{
+        "href":"zookeeper?detail=true&path=%2Flive_nodes"
+      }
+    }
+  }]
+}

data/spec/fixtures/zookeeper/live_nodes_one_child.json

@@ -0,0 +1,36 @@
+{
+  "znode":{
+    "path":"/live_nodes",
+    "prop":{
+      "version":0,
+      "aversion":0,
+      "children_count":1,
+      "ctime":"Sun Jul 15 22:58:05 UTC 2012 (1342393085027)",
+      "cversion":17,
+      "czxid":2,
+      "dataLength":0,
+      "ephemeralOwner":0,
+      "mtime":"Sun Jul 15 22:58:05 UTC 2012 (1342393085027)",
+      "mzxid":2,
+      "pzxid":298
+    }
+  },
+  "tree":[{
+    "data":{
+      "title":"/live_nodes",
+      "attr":{
+        "href":"zookeeper?detail=true&path=%2Flive_nodes"
+      }
+    },
+    "children":[{
+      "data":{
+        "title":"myserver:8888_solr",
+        "attr":{
+          "href":"zookeeper?detail=true&path=%2Flive_nodes%2Fmyserver%3A8888_solr"
+        }
+      },
+      "ephemeral":true,
+      "version":0
+    }]
+  }]
+}

data/spec/lib/lucid_works/base_spec.rb

@@ -40,37 +40,36 @@ describe LucidWorks::Base do
   it_should_behave_like "ActiveModel"
   
   describe "naming support" do
-    context "for a namespaced model" do
-      class LucidWorks::NamespacedModel < LucidWorks::Base
+    context "for a regular model" do
+      module LucidWorks
+        class Foo < Base
+        end
       end
       
-      it "demodulizes the model's singular name" do
-        LucidWorks::NamespacedModel.model_name.singular.should == "namespaced_model"
-      end
-
-      it "demodulizes the model's plural name" do
-        LucidWorks::NamespacedModel.model_name.plural.should == "namespaced_models"
-      end
-
-      it "demodulizes the model's collection name" do
-        LucidWorks::NamespacedModel.model_name.collection.should == "namespaced_models"
+      it "demodulizes the model's name" do
+        LucidWorks::Foo.model_name.should == "Foo"
       end
     end
 
-    context "for a top-level model" do
-      class TopLevelModel < LucidWorks::Base
+    context "for a nested model" do
+      module LucidWorks
+        class Foo < Base
+          class Bar < Base
+          end
+        end
       end
       
-      it "returns the model's singular name" do
-        TopLevelModel.model_name.singular.should == "top_level_model"
+      it "demodulizes the model's name" do
+        LucidWorks::Foo::Bar.model_name.should == "Foo::Bar"
       end
+    end
 
-      it "returns the model's plural name" do
-        TopLevelModel.model_name.plural.should == "top_level_models"
+    context "for a top-level model" do
+      class TopLevel < LucidWorks::Base
       end
 
-      it "returns the model's collection name" do
-        TopLevelModel.model_name.plural.should == "top_level_models"
+      it "doesn't change the model's name" do
+        TopLevel.model_name.should == "TopLevel"
       end
     end
   end
@@ -419,18 +418,28 @@ describe LucidWorks::Base do
           before do
             RestClient.stub(:post) {
               e = RestClient::UnprocessableEntity.new
-              e.response = '{"errors":[{"message":"name is a required key","key":"name"},{"message":"name must consist of only A-Z a-z 0-9 - _","key":"name"}],"http_status_name":"Unprocessable Entity","http_status_code":422}'
+              # e.response = '{"errors":[{"message":"name is a required key","key":"name"},{"message":"name must consist of only A-Z a-z 0-9 - _","key":"name"}],"http_status_name":"Unprocessable Entity","http_status_code":422}'
+              e.response = '{"errors":[' + 
+                '{"message":"some message","key":"name"},' + 
+                '{"message":"some message","key":"class"}' + 
+                '],"http_status_name":"Unprocessable Entity","http_status_code":422}'
               raise e
             }
-            @widget = Widget.new(:name => '', :size => '', :parent => @server)
+            @widget = Widget.new(:name => '', :size => '', :_class => '', :parent => @server)
           end
           
-          it "should set errors on the model" do
+          it "should attach errors to simple attributes" do
             @widget.save
             @widget.errors.should_not be_empty
             @widget.errors['name'].should_not be_empty
           end
 
+          it "should attach errors to mapped (reserved names) attributes" do
+            @widget.save
+            @widget.errors.should_not be_empty
+            @widget.errors['_class'].should_not be_empty
+          end
+
           it "should not set persisted" do
             @widget.save
             @widget.should_not be_persisted
@@ -629,7 +638,7 @@ describe LucidWorks::Base do
 
           it "should attach errors to the model" do
             @widget.destroy
-            @widget.errors.should == {:name => ["Field Type currently in use: text_en"]}
+            @widget.errors[:name].should == ["Field Type currently in use: text_en"]
           end
 
           it "should return false" do

data/spec/lib/lucid_works/cache_spec.rb

@@ -0,0 +1,44 @@
+require 'spec_helper'
+
+describe LucidWorks::Cache do
+  before :all do
+    @server = connect_to_live_server
+    @server.reset_collections!
+    @collection = @server.collection('collection1')
+  end
+
+  it "is connected to the rest api" do
+    LucidWorks::Cache.new(:parent => @collection).should be_a(LucidWorks::Base)
+  end
+
+  describe "validations" do
+    it "requires 'size' to be present" do
+      field = LucidWorks::Cache.new(:parent => @collection)
+      field.should_not be_valid
+      field.errors[:size].should == ["can't be blank"]
+    end
+
+    [:initial_size, :size].each do |attribute|
+      it "requires #{attribute} to be >= 0" do
+        field = LucidWorks::Cache.new(:parent => @collection, attribute => -1)
+        field.should_not be_valid
+        field.errors[attribute].should == ["is not a valid value"]
+      end
+
+      it "requires #{attribute} to be an integer" do
+        field = LucidWorks::Cache.new(:parent => @collection, attribute => 1.2)
+        field.should_not be_valid
+        field.errors[attribute].should == ["is not a valid value"]
+      end
+    end
+  end
+  
+  it "exposes cache details" do
+    cache = @collection.caches.find('document_cache')
+    cache.name.should == 'document_cache'
+    cache._class.should == 'solr.LRUCache'
+    cache.initial_size.should == '512'
+    cache.size.should == '512'
+    cache.autowarm_count.should == '0'
+  end
+end

data/spec/lib/lucid_works/cluster_spec.rb

@@ -0,0 +1,109 @@
+require 'spec_helper'
+
+describe LucidWorks::Cluster do
+  before(:each) do
+    RestClient.stub(:get).with('http://myserver:8888/solr/zookeeper?detail=true&path=/clusterstate.json') do
+      File.read(fixture_path('zookeeper/clusterstate.json'))
+    end
+    RestClient.stub(:get).with('http://myserver:8888/solr/zookeeper?detail=true&path=/live_nodes/') do
+      File.read(fixture_path('zookeeper/live_nodes.json'))
+    end
+  end
+  
+  let(:cluster) { LucidWorks::Cluster.new('http://myserver:8888/solr') }
+  
+  [8888, 13888, 14888].each do |port|
+    let("node_on_port_#{port}") { cluster.nodes.find {|node| node.port == port } }
+  end
+  
+  context "with multiple nodes in different states" do
+    it "builds a list of nodes" do
+      cluster.should have(3).nodes
+    end
+    
+    it "indicates the state of each node" do
+      node_on_port_8888.state.should == 'live'
+      node_on_port_13888.state.should == 'live'
+      node_on_port_14888.state.should == 'recovering'
+    end
+  end
+  
+  context "without any children under /live_nodes" do
+    before(:each) do
+      RestClient.stub(:get).with('http://myserver:8888/solr/zookeeper?detail=true&path=/live_nodes/') do
+        File.read(fixture_path('zookeeper/live_nodes_no_children.json'))
+      end
+    end
+    
+    it "builds a list of nodes" do
+      cluster.should have(3).nodes
+    end
+    
+    it "indicates the state of each node" do
+      cluster.nodes.all? {|node| node.state == 'down' }.should be_true
+    end
+  end
+  
+  context "with a broken shard" do
+    before(:each) do
+      RestClient.stub(:get).with('http://myserver:8888/solr/zookeeper?detail=true&path=/clusterstate.json') do
+        File.read(fixture_path('zookeeper/clusterstate_broken_shard.json'))
+      end
+    end
+    
+    it "builds a list of nodes" do
+      cluster.should have(2).nodes # the clusterstate fixture used here only has 2 nodes
+    end
+    
+    it "only attaches valid shards to nodes" do
+      cluster.nodes.second.should have(1).shard
+    end
+  end
+  
+  it "extracts the address and port from the base URL" do
+    cluster.nodes.first.host.should == 'myserver'
+    cluster.nodes.first.port.should == 8888
+  end
+  
+  it "exposes the access URLs" do
+    node_on_port_8888.solr_url.should == 'http://myserver:8888/solr'
+    node_on_port_13888.api_url.should == 'http://myserver:13888/api'
+  end
+  
+  describe "a shard" do
+    let(:a_shard) { node_on_port_8888.shards.first }
+    
+    it "belong to nodes" do
+      node_on_port_8888.should have(2).shards
+      node_on_port_8888.shards.first.node.should be(node_on_port_8888)
+    end
+    
+    it "have an id" do
+      a_shard.id.should == 'collection1_shard1'
+    end
+    
+    it "have a collection" do
+      a_shard.collection.should == 'collection1'
+    end
+    
+    it "have a name" do
+      a_shard.name.should == 'shard1'
+    end
+
+    it "can be a leader" do
+      node_on_port_8888.shards.first.should be_leader
+      node_on_port_14888.shards.first.should_not be_leader
+    end
+  end
+  
+  it "is serializable" do
+    cluster.nodes.to_json.should == "[" + 
+      %Q({"id":"myserver_8888","host":"myserver","port":8888,"state":"live","solr_url":"http://myserver:8888/solr","api_url":"http://myserver:8888/api",) + 
+        %Q("shards":[{"name":"shard1","collection":"collection1","leader":true},{"name":"shard1","collection":"LucidWorksLogs","leader":true}]},) + 
+      %Q({"id":"myserver_14888","host":"myserver","port":14888,"state":"recovering","solr_url":"http://myserver:14888/solr","api_url":"http://myserver:14888/api",) + 
+        %Q("shards":[{"name":"shard1","collection":"collection1","leader":false},{"name":"shard1","collection":"LucidWorksLogs","leader":false}]},) +
+      %Q({"id":"myserver_13888","host":"myserver","port":13888,"state":"live","solr_url":"http://myserver:13888/solr","api_url":"http://myserver:13888/api",) + 
+        %Q("shards":[{"name":"shard2","collection":"collection1","leader":true},{"name":"shard2","collection":"LucidWorksLogs","leader":true}]}) +
+    "]"
+  end
+end

data/spec/lib/lucid_works/collection/activity_spec.rb

@@ -31,6 +31,17 @@ describe LucidWorks::Activity do
     end
 
     describe '#frequency' do
+      it "assumes an 'every' frequency when period ranges from 1 to 59 minutes" do
+        @activity.period = 0
+        @activity.frequency.should_not == 'every'
+        @activity.period = 1 * 60
+        @activity.frequency.should == 'every'
+        @activity.period = 59 * 60
+        @activity.frequency.should == 'every'
+        @activity.period = 60 * 60
+        @activity.frequency.should_not == 'every'
+      end
+
       it "should return 'weekly' when period == num of seconds in a week" do
         @activity.period = 604800
         @activity.frequency.should == 'weekly'
@@ -184,6 +195,24 @@ describe LucidWorks::Activity do
         end
       end
 
+      describe "with an 'every' frequency" do
+        let(:attributes) { {"frequency"=>"every", "period" => "1800"} }
+
+        it "ceils start_time based on the period" do
+          Timecop.freeze Time.iso8601("2012-11-07T10:12:33Z") do
+            @activity.schedule = attributes
+            @activity.start_time.should == Time.iso8601("2012-11-07T10:30:00Z")
+          end
+        end
+
+        it "sets the period" do
+          Timecop.freeze Time.iso8601("2012-11-07T10:12:33Z") do
+            @activity.schedule = attributes
+            @activity.period.should == 1800
+          end
+        end
+      end
+      
       context 'supplied start time has elapsed w/rt current interval' do
         describe 'with hourly frequency' do
           it "should advance the start time" do

data/spec/lib/lucid_works/collection/prime_activities_spec.rb

@@ -41,7 +41,7 @@ describe "LucidWorks::Collection#prime_activities" do
       it "should not create duplicates for existing types" do
         lambda {
           @collection.prime_activities
-        }.should change(@collection, :activities_count).by (LucidWorks::Activity::TYPES.size - 1)
+        }.should change(@collection, :activities_count).by(LucidWorks::Activity::TYPES.size - 1)
       end
     end
 

data/spec/lib/lucid_works/collection/settings_spec.rb

@@ -0,0 +1,31 @@
+require 'spec_helper'
+
+describe LucidWorks::Collection::Settings do
+  before :all do
+    @server = connect_to_live_server
+    @server.reset_collections!
+    @collection = @server.collection('collection1')
+  end
+
+  it "is connected to the rest api" do
+    LucidWorks::Collection::Settings.new(:parent => @collection).should be_a(LucidWorks::Base)
+  end
+
+  describe "validations" do
+    subject { LucidWorks::Collection::Settings.new(:parent => @collection) }
+    
+    [:update_handler_autocommit_max_docs].each  do |attribute|
+      it "requires #{attribute} to be an integer" do
+        resource = LucidWorks::Collection::Settings.new(:parent => @collection, attribute => 1.2)
+        resource.should_not be_valid
+        resource.errors[attribute].should == ["is not a valid value"]
+      end
+
+      it "requires #{attribute} to be > 0" do
+        resource = LucidWorks::Collection::Settings.new(:parent => @collection, attribute => 0)
+        resource.should_not be_valid
+        resource.errors[attribute].should == ["is not a valid value"]
+      end
+    end
+  end
+end