settingcrazy 0.0.2

data/spec/namespaced_settings_proxy_spec.rb

@@ -0,0 +1,70 @@
+require 'spec_helper'
+
+describe SettingCrazy::NamespacedSettingsProxy do
+  context "no template" do
+    let(:model) { VendorInstance.create(:name => "VI") }
+    let(:namespace) { SettingCrazy::Namespace.new('google') }
+    subject         { SettingCrazy::NamespacedSettingsProxy.new(model, namespace) }
+
+    context "single values" do
+      before    { subject.foo = "bar"; model.save! }
+      its(:foo) { should == 'bar' }
+      its(:oth) { should be(nil) }
+      it        { subject[:foo].should == 'bar' }
+      it        { subject[:oth].should be(nil) }
+
+      it "should apply the namespace to the setting values" do
+        model.setting_values(true).first.namespace.should == 'google'
+      end
+
+      describe "update a value" do
+        before { subject.foo = "different" }
+        its(:foo) { should == "different" }
+      end
+    end
+
+    context "multiple values" do
+      before do
+        subject.foo = %w(a b c)
+        model.save!
+      end
+
+      its(:foo) { should == [ 'a', 'b', 'c' ] }
+
+      describe "update a value" do
+        before do
+          subject.foo = %w(d e f)
+        end
+        its(:foo) { should == [ 'd', 'e', 'f' ] }
+      end
+    end
+  end
+
+  describe 'one settable object with multiple namespaces' do
+    let(:scenario) { Scenario.create }
+    subject        { scenario.settings }
+
+    context 'not share setting_values' do
+      before do
+        scenario.settings.google.foo = 'bar'
+        scenario.save!
+      end
+      it { subject.google.inspect.should == '{:foo=>"bar"}' }
+      it { subject.google.foo.should     == 'bar' }
+      it { subject.yahoo.inspect.should  == '{}' }
+      it { subject.yahoo.foo.should      be(nil) }
+    end
+
+    context 'has unique values' do
+      before do
+        scenario.settings.google.foo = 'bar'
+        scenario.settings.yahoo.foo  = 'baz'
+        scenario.save!
+      end
+      it { subject.google.inspect.should == '{:foo=>"bar"}' }
+      it { subject.google.foo.should     == 'bar' }
+      it { subject.yahoo.inspect.should  == '{:foo=>"baz"}' }
+      it { subject.yahoo.foo.should      == 'baz' }
+    end
+  end
+end

data/spec/settingcrazy_spec.rb

@@ -0,0 +1,78 @@
+require 'spec_helper'
+
+describe SettingCrazy do
+  describe "class methods" do
+
+  end
+
+  describe "settings" do
+
+    context "without namespaces" do
+      let(:model) { VendorInstance.create(:name => "VI") }
+      subject     { model.settings }
+      it          { should be_a(SettingCrazy::SettingsProxy) }
+    end
+
+    context "with namespaces" do
+      let(:model)   { Scenario.create(:name => "Scenario") }
+      subject       { model.settings }
+      it            { should be_a(SettingCrazy::SettingsProxy) }
+      its(:google)  { should be_a(SettingCrazy::SettingsProxy) }
+      its(:yahoo)   { should be_a(SettingCrazy::SettingsProxy) }
+      its(:unknown) { should be_nil }
+      
+      describe "setting and getting" do
+        before do
+          model.settings.google.foo = "bar"
+          model.save!
+        end
+
+        subject   { model.settings.google }
+        its(:foo) { should == 'bar' }
+
+        it "should not be set in other namespaces" do
+          model.settings.yahoo.foo.should be_nil
+        end
+      end
+    end
+
+    context "with namespaces and templates" do
+      let(:model)   { TemplatedScenario.create(:name => "Scenario") }
+      subject       { model.settings }
+      it            { should be_a(SettingCrazy::SettingsProxy) }
+      its(:google)  { should be_a(SettingCrazy::SettingsProxy) }
+      its(:yahoo)   { should be_a(SettingCrazy::SettingsProxy) }
+      its(:unknown) { should be_nil }
+
+      it "should have a template for google" do
+        subject.google.template.should == ExampleTemplate
+      end
+      
+      describe "setting and getting" do
+        before do
+          model.settings.google.foo = "bar"
+          model.save!
+        end
+
+        subject   { model.settings.google }
+        its(:foo) { should == 'bar' }
+
+        it "should not be set in other namespaces" do
+          model.settings.yahoo.foo.should be_nil
+        end
+
+        it "should not allow settings not in the templated namespace" do
+          -> {
+            model.settings.google.not_in_template
+          }.should raise_error(ActiveRecord::UnknownAttributeError)
+        end
+
+        it "should ALLOW settings in the UNtemplated namespace" do
+          -> {
+            model.settings.yahoo.not_in_template_but_we_dont_care
+          }.should_not raise_error(ActiveRecord::UnknownAttributeError)
+        end
+      end
+    end
+  end
+end

data/spec/settings_proxy_spec.rb

@@ -0,0 +1,55 @@
+require 'spec_helper'
+
+describe SettingCrazy::SettingsProxy do
+  let(:model) { VendorInstance.create(:name => "VI") }
+
+  context "no template" do
+    subject   { SettingCrazy::SettingsProxy.new(model, nil) }
+
+    context "single values" do
+      before    { subject.foo = "bar"; model.save! }
+      its(:foo) { should == 'bar' }
+      its(:oth) { should be(nil) }
+      it        { subject[:foo].should == 'bar' }
+      it        { subject[:oth].should be(nil) }
+      it        { subject.to_hash.should == {foo: 'bar'} }
+
+      describe "update a value" do
+        before { subject.foo = "different" }
+        its(:foo) { should == "different" }
+      end
+    end
+
+    context "multiple values" do
+      before do
+        subject.foo = %w(a b c)
+        model.save!
+      end
+
+      its(:foo) { should == [ 'a', 'b', 'c' ] }
+
+      describe "update a value" do
+        before do
+          subject.foo = %w(d e f)
+        end
+        its(:foo) { should == [ 'd', 'e', 'f' ] }
+      end
+    end
+  end
+
+  context "a template is provided" do
+    let(:template) { ExampleTemplate }
+    subject   { SettingCrazy::SettingsProxy.new(model, template) }
+    before    { subject.foo = "1234"; model.save! }
+    its(:foo) { should == "1234" }
+    its(:bar) { should == "A string default" }
+    it        { subject[:foo].should == "1234" }
+    it        { subject[:bar].should == "A string default" }
+
+    it "should raise if we try to get an invalid option" do
+      -> {
+        subject.unknown
+      }.should raise_error(ActiveRecord::UnknownAttributeError)
+    end
+  end
+end

data/spec/settings_validator_spec.rb

@@ -0,0 +1,121 @@
+require 'spec_helper'
+
+describe SettingsValidator do
+  context 'no template' do
+    context 'model not saved' do
+      subject { VendorInstance.new(:name => "VI") }
+      it      { should be_valid }
+    end
+
+    context 'model saved' do
+      subject { VendorInstance.create(:name => "VI") }
+      it      { should be_valid }
+    end
+  end
+
+  context 'a template is provided' do
+    context 'model not saved' do
+      subject { TemplatedCampaign.new(:name => "TemplatedCampaign") }
+      it      { should be_valid }
+    end
+
+    context 'model saved' do
+      subject { TemplatedCampaign.create(:name => "TemplatedCampaign") }
+      it      { should_not be_valid }
+
+      context 'validates presence' do
+        context 'required_key does not exist' do
+          before { subject.valid? }
+          it     { subject.errors.messages[:required_key].should include("Setting, 'RequiredKey', is required") }
+        end
+
+        context 'required_key exists' do
+          before { subject.settings.required_key = 'true'; subject.save! }
+          it     { should be_valid }
+        end
+      end
+
+      context 'validates singleness' do
+        before { subject.settings.required_key = 'true'; subject.valid? }
+
+        context 'multiple_key' do
+          context 'has one value' do
+            before { subject.settings.multiple_key = 'foo', 'bar' }
+            it     { should be_valid }
+          end
+
+          context 'has more than one values' do
+            before { subject.settings.multiple_key = ['foo', 'bar'] }
+            it     { should be_valid }
+          end
+        end
+
+        context 'single_key' do
+          context 'assigns one value' do
+            before { subject.settings.single_key = 'foo' }
+            it     { should be_valid }
+          end
+
+          context 'assigns more than one values' do
+            before { subject.settings.single_key = ['foo', 'bar']; subject.valid? }
+            it     { should_not be_valid }
+            it     { subject.errors.messages[:single_key].should include("Cannot save multiple values for Setting, 'SingleKey'") }
+          end
+        end
+      end
+
+      context 'validates dependency' do
+        before { subject.settings.required_key = 'true'; subject.valid? }
+
+        context 'dependent key' do
+          context 'dependee value does not exist' do
+            before { subject.settings.dependent_key = 'BAR'; subject.valid? }
+            it     { should_not be_valid }
+            it     { subject.errors.messages[:dependent_key].should include("'DependentKey' can only be specified if 'DependeeKey' is set to 'bar'") }
+          end
+
+          context 'dependee value is not satisfied' do
+            before { subject.settings = {dependee_key: 'foo', dependent_key: 'BAR'}; subject.valid? }
+            it     { should_not be_valid }
+            it     { subject.errors.messages[:dependent_key].should include("'DependentKey' can only be specified if 'DependeeKey' is set to 'bar'") }
+          end
+
+          context 'dependee value is satisfied' do
+            before { subject.settings = {dependee_key: 'bar', dependent_key: 'BAR'} }
+            it     { should be_valid }
+          end
+        end
+      end
+
+      context 'validates values' do
+        before { subject.settings.required_key = 'true'; subject.valid? }
+
+        context 'single key' do
+          context 'invalid value' do
+            before { subject.settings.single_key = 'foobar'; subject.valid? }
+            it     { should_not be_valid }
+            it     { subject.errors.messages[:single_key].should include("'foobar' is not a valid setting for 'SingleKey'") }
+          end
+
+          context 'valid value' do
+            before { subject.settings.single_key = 'bar' }
+            it     { should be_valid }
+          end
+        end
+
+        context 'multiple key' do
+          context 'invalid value' do
+            before { subject.settings.multiple_key = ['foo', 'foobar']; subject.valid? }
+            it     { should_not be_valid }
+            it     { subject.errors.messages[:multiple_key].should include("'foobar' is not a valid setting for 'MultipleKey'") }
+          end
+
+          context 'valid value' do
+            before { subject.settings.multiple_key = ['foo', 'bar'] }
+            it     { should be_valid }
+          end
+        end
+      end
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,8 @@
+require File.expand_path("../../lib/settingcrazy", __FILE__)
+
+require 'support/templates'
+require 'support/models'
+
+RSpec.configure do |config|
+  # config.mock_with :rr
+end

data/spec/support/models.rb

@@ -0,0 +1,10 @@
+require 'active_record'
+require 'sqlite3'
+
+root = File.expand_path("../../../", __FILE__)
+ActiveRecord::Base.establish_connection(
+  :adapter => "sqlite3",
+  :database => "#{root}/db/safeattributes.db"
+)
+
+Dir["#{root}/spec/support/models/*.rb"].each { |f| require f }

data/spec/support/models/campaign.rb

@@ -0,0 +1,12 @@
+ActiveRecord::Base.connection.execute("DROP TABLE IF EXISTS 'campaigns'")
+ActiveRecord::Base.connection.create_table(:campaigns) do |t|
+  t.string :name
+  t.references :scenario
+end
+
+class Campaign < ActiveRecord::Base  
+  include SettingCrazy
+  belongs_to :scenario
+  attr_accessible :name
+  settings_inherit_via :scenario, :namespace => :google
+end

data/spec/support/models/case.rb

@@ -0,0 +1,11 @@
+ActiveRecord::Base.connection.execute("DROP TABLE IF EXISTS 'cases'")
+ActiveRecord::Base.connection.create_table(:cases) do |t|
+  t.string :name
+end
+
+class Case < ActiveRecord::Base  
+  include SettingCrazy
+  use_setting_template ExampleTemplate
+  attr_accessible :name
+  has_many :notes
+end

data/spec/support/models/clever_campaign.rb

@@ -0,0 +1,13 @@
+ActiveRecord::Base.connection.execute("DROP TABLE IF EXISTS 'clever_campaigns'")
+ActiveRecord::Base.connection.create_table(:clever_campaigns) do |t|
+  t.string :name
+  t.string :setting_namespace
+  t.references :scenario
+end
+
+class CleverCampaign < ActiveRecord::Base  
+  include SettingCrazy
+  belongs_to :scenario
+  attr_accessible :name, :setting_namespace
+  settings_inherit_via :scenario, :namespace => Proc.new { |clever_campaign| clever_campaign.setting_namespace }
+end

data/spec/support/models/duck.rb

@@ -0,0 +1,13 @@
+ActiveRecord::Base.connection.execute("DROP TABLE IF EXISTS 'ducks'")
+ActiveRecord::Base.connection.create_table(:ducks) do |t|
+  t.string :name
+  t.integer :quacks
+  t.references :farm
+end
+
+class Duck < ActiveRecord::Base  
+  include SettingCrazy
+  attr_accessible :name, :quacks
+  belongs_to :farm
+  settings_inherit_via :farm
+end

data/spec/support/models/farm.rb

@@ -0,0 +1,10 @@
+ActiveRecord::Base.connection.execute("DROP TABLE IF EXISTS 'farms'")
+ActiveRecord::Base.connection.create_table(:farms) do |t|
+  t.string :name
+end
+
+class Farm < ActiveRecord::Base  
+  include SettingCrazy
+  attr_accessible :name
+  has_many :ducks
+end

data/spec/support/models/note.rb

@@ -0,0 +1,12 @@
+ActiveRecord::Base.connection.execute("DROP TABLE IF EXISTS 'notes'")
+ActiveRecord::Base.connection.create_table(:notes) do |t|
+  t.string :name
+  t.references :case
+end
+
+class Note < ActiveRecord::Base  
+  include SettingCrazy
+  attr_accessible :name
+  belongs_to :case
+  settings_inherit_via :case
+end

data/spec/support/models/scenario.rb

@@ -0,0 +1,13 @@
+ActiveRecord::Base.connection.execute("DROP TABLE IF EXISTS 'scenarios'")
+ActiveRecord::Base.connection.create_table(:scenarios) do |t|
+  t.string :name
+end
+
+class Scenario < ActiveRecord::Base  
+  include SettingCrazy
+  has_many :campaigns
+  has_many :clever_campaigns
+  attr_accessible :name
+  setting_namespace :google
+  setting_namespace :yahoo
+end