active_attr 0.2.2 → 0.3.0

data/spec/unit/active_attr/block_initialization_spec.rb

@@ -0,0 +1,39 @@
+require "spec_helper"
+require "active_attr/block_initialization"
+
+module ActiveAttr
+  describe BlockInitialization do
+    subject { model_class.new("arg") {} }
+
+    let :model_class do
+      Class.new do
+        include InitializationVerifier
+        include BlockInitialization
+
+        def initialize(arg)
+          super
+        end
+      end
+    end
+
+    describe "#initialize" do
+      it "invokes the superclass initializer" do
+        should be_initialized
+      end
+
+      it "doesn't raise when not passed a block" do
+        expect { model_class.new("arg") }.not_to raise_error
+      end
+
+      it "yields the new instance" do
+        yielded_instance = nil
+
+        returned_instance = model_class.new("arg") do |yielded|
+          yielded_instance = yielded
+        end
+
+        returned_instance.should equal yielded_instance
+      end
+    end
+  end
+end

data/spec/unit/active_attr/dangerous_attribute_error_spec.rb

@@ -0,0 +1,9 @@
+require "spec_helper"
+require "active_attr/dangerous_attribute_error"
+
+module ActiveAttr
+  describe DangerousAttributeError do
+    it { should be_a_kind_of ScriptError }
+    it { should be_a_kind_of Error }
+  end
+end

data/spec/unit/active_attr/logger_spec.rb

@@ -0,0 +1,186 @@
+require "spec_helper"
+require "active_attr/logger"
+
+module ActiveAttr
+  describe Logger do
+    subject { model_class.new }
+    let(:logger) { double "logger" }
+    let(:child_class) { Class.new(parent_class) }
+    let(:parent_class) { model_class }
+
+    let :model_class do
+      Class.new do
+        include Logger
+      end
+    end
+
+    context "when the logger is not set" do
+      it "#{described_class}.logger is nil" do
+        described_class.logger.should be_nil
+      end
+
+      it "#{described_class}.logger? is false" do
+        described_class.logger?.should == false
+      end
+
+      it ".logger is nil" do
+        model_class.logger.should be_nil
+      end
+
+      it ".logger? is false" do
+        model_class.logger?.should == false
+      end
+
+      it "#logger is nil" do
+        subject.logger.should be_nil
+      end
+
+      it "#logger? is false" do
+        subject.logger?.should == false
+      end
+    end
+
+    context "when the logger is set on the module" do
+      before { described_class.logger = logger }
+      after { described_class.logger = nil }
+
+      it "#{described_class}.logger is the logger" do
+        described_class.logger.should == logger
+      end
+
+      it "#{described_class}.logger? is true" do
+        described_class.logger?.should == true
+      end
+
+      it ".logger is the logger" do
+        model_class.logger.should == logger
+      end
+
+      it ".logger? is true" do
+        model_class.logger?.should == true
+      end
+
+      it "#logger is the logger" do
+        subject.logger.should == logger
+      end
+
+      it "#logger? is true" do
+        subject.logger?.should == true
+      end
+    end
+
+    context "when the logger is set on the class" do
+      before { model_class.logger = logger }
+
+      it "#{described_class}.logger is nil" do
+        described_class.logger.should be_nil
+      end
+
+      it "#{described_class}.logger? is false" do
+        described_class.logger?.should == false
+      end
+
+      it ".logger is the logger" do
+        model_class.logger.should == logger
+      end
+
+      it ".logger? is true" do
+        model_class.logger?.should == true
+      end
+
+      it "#logger is the logger" do
+        subject.logger.should == logger
+      end
+
+      it "#logger? is true" do
+        subject.logger?.should == true
+      end
+    end
+
+    context "when the logger is set on a parent class" do
+      before { parent_class.logger = logger }
+      subject { child_class.new }
+
+      it "#{described_class}.logger is nil" do
+        described_class.logger.should be_nil
+      end
+
+      it "#{described_class}.logger? is false" do
+        described_class.logger?.should == false
+      end
+
+      it ".logger is the logger" do
+        child_class.logger.should == logger
+      end
+
+      it ".logger? is true" do
+        child_class.logger?.should == true
+      end
+
+      it "#logger is the logger" do
+        subject.logger.should == logger
+      end
+
+      it "#logger? is true" do
+        subject.logger?.should == true
+      end
+    end
+
+    context "when the logger is set on a child class" do
+      before { child_class.logger = logger }
+      subject { parent_class.new }
+
+      it "#{described_class}.logger is nil" do
+        described_class.logger.should be_nil
+      end
+
+      it "#{described_class}.logger? is false" do
+        described_class.logger?.should == false
+      end
+
+      it ".logger is nil" do
+        parent_class.logger.should be_nil
+      end
+
+      it ".logger? is false" do
+        parent_class.logger?.should == false
+      end
+
+      it "#logger is nil" do
+        subject.logger.should be_nil
+      end
+
+      it "#logger? is false" do
+        subject.logger?.should == false
+      end
+    end
+
+    context "when the logger is set on the instance" do
+      before { subject.logger = logger }
+
+      it "#{described_class}.logger is nil" do
+        described_class.logger.should be_nil
+      end
+
+      it "#{described_class}.logger? is false" do
+        described_class.logger?.should == false
+      end
+
+      it ".logger is nil" do
+        model_class.logger.should be_nil
+      end
+
+      it ".logger? is false" do
+        model_class.logger?.should == false
+      end
+
+      it "#logger is the logger" do
+        subject.logger.should == logger
+      end
+
+      it "#logger? is true" do
+        subject.logger?.should == true
+      end
+    end
+  end
+end

data/spec/unit/active_attr/mass_assignment_security_spec.rb

@@ -0,0 +1,62 @@
+require "spec_helper"
+require "active_attr/mass_assignment_security"
+
+module ActiveAttr
+  describe MassAssignmentSecurity, :mass_assignment do
+    subject { model_class }
+
+    before do
+      model_class.class_eval do
+        include MassAssignmentSecurity
+        attr_accessor :age
+      end
+    end
+
+    shared_examples "secure mass assignment method", :secure_mass_assignment_method => true do
+      include_examples "mass assignment method"
+
+      it "ignores assigning a protected attribute" do
+        person = mass_assign_attributes(:age => 21)
+        person.age.should be_nil
+      end
+    end
+
+    shared_examples "secure mass assignment method with options", :secure_mass_assignment_method_with_options => true do
+      it "supports role-based mass assignment security" do
+        person = mass_assign_attributes_with_options({ :age => 21 }, :as => :admin)
+        person.age.should == 21
+      end
+
+      it "skips security if passed the :without_protection option" do
+        person = mass_assign_attributes_with_options({ :age => 21 }, :without_protection => true)
+        person.age.should == 21
+      end
+    end
+
+    context "white listing attributes" do
+      before do
+        model_class.class_eval do
+          attr_accessible :first_name, :last_name, :name
+          attr_accessible :age, :as => :admin
+        end
+      end
+
+      describe "#assign_attributes", :assign_attributes, :secure_mass_assignment_method, :secure_mass_assignment_method_with_options
+      describe "#attributes=", :attributes=, :secure_mass_assignment_method
+      describe "#initialize", :initialize, :secure_mass_assignment_method, :secure_mass_assignment_method_with_options
+    end
+
+    context "black listing attributes" do
+      before do
+        model_class.class_eval do
+          attr_protected :age
+          attr_protected :as => :admin
+        end
+      end
+
+      describe "#assign_attributes", :assign_attributes, :secure_mass_assignment_method, :secure_mass_assignment_method_with_options
+      describe "#attributes=", :attributes=, :secure_mass_assignment_method
+      describe "#initialize", :initialize, :secure_mass_assignment_method, :secure_mass_assignment_method_with_options
+    end
+  end
+end

data/spec/unit/active_attr/query_attributes_spec.rb

@@ -0,0 +1,225 @@
+require "spec_helper"
+require "active_attr/query_attributes"
+require "bigdecimal"
+
+module ActiveAttr
+  describe QueryAttributes do
+    subject { model_class.new }
+
+    let :model_class do
+      Class.new do
+        include QueryAttributes
+
+        attribute :value
+        attribute :overridden
+
+        def overridden?
+          super
+        end
+
+        def true?
+          true
+        end
+
+        def false?
+          false
+        end
+      end
+    end
+
+    describe ".attribute" do
+      it "defines an attribute predicate method that calls #attribute?" do
+        subject.should_receive(:attribute?).with("value")
+        subject.value?
+      end
+
+      it "defines an attribute reader that can be called via super" do
+        subject.should_receive(:attribute?).with("overridden")
+        subject.overridden?
+      end
+    end
+
+    describe "#query_attribute" do
+      it "raises ArgumentError when called with two arguments" do
+        expect { subject.query_attribute(:a, :b) }.to raise_error ArgumentError
+      end
+
+      it "does not raise when called with a single argument" do
+        expect { subject.query_attribute(:a) }.not_to raise_error ArgumentError
+      end
+
+      it "raises ArgumentError when called with no arguments" do
+        expect { subject.query_attribute }.to raise_error ArgumentError
+      end
+
+      it "calls the predicate method if defined" do
+        subject.query_attribute(:true).should == true
+        subject.query_attribute(:false).should == false
+      end
+
+      it "raises when getting an undefined attribute" do
+        expect { subject.query_attribute(:initials) }.to raise_error UnknownAttributeError, "unknown attribute: initials"
+      end
+
+      it "is false when the attribute is false" do
+        subject.value = false
+        subject.value?.should == false
+      end
+
+      it "is true when the attribute is true" do
+        subject.value = true
+        subject.value?.should == true
+      end
+
+      it "is false when the attribute is nil" do
+        subject.value = nil
+        subject.value?.should == false
+      end
+
+      it "is true when the attribute is an Object" do
+        subject.value = Object.new
+        subject.value?.should == true
+      end
+
+      it "is false when the attribute is an empty string" do
+        subject.value = ""
+        subject.value?.should == false
+      end
+
+      it "is true when the attribute is a non-empty string" do
+        subject.value = "Chris"
+        subject.value?.should == true
+      end
+
+      it "is false when the attribute is 0" do
+        subject.value = 0
+        subject.value?.should == false
+      end
+
+      it "is true when the attribute is 1" do
+        subject.value = 1
+        subject.value?.should == true
+      end
+
+      it "is false when the attribute is 0.0" do
+        subject.value = 0.0
+        subject.value?.should == false
+      end
+
+      it "is true when the attribute is 0.1" do
+        subject.value = 0.1
+        subject.value?.should == true
+      end
+
+      it "is false when the attribute is a zero BigDecimal" do
+        subject.value = BigDecimal.new("0.0")
+        subject.value?.should == false
+      end
+
+      it "is true when the attribute is a non-zero BigDecimal" do
+        subject.value = BigDecimal.new("0.1")
+        subject.value?.should == true
+      end
+
+      it "is true when the attribute is -1" do
+        subject.value = -1
+        subject.value?.should == true
+      end
+
+      it "is false when the attribute is -0.0" do
+        subject.value = -0.0
+        subject.value?.should == false
+      end
+
+      it "is true when the attribute is -0.1" do
+        subject.value = -0.1
+        subject.value?.should == true
+      end
+
+      it "is false when the attribute is a negative zero BigDecimal" do
+        subject.value = BigDecimal.new("-0.0")
+        subject.value?.should == false
+      end
+
+      it "is true when the attribute is a negative BigDecimal" do
+        subject.value = BigDecimal.new("-0.1")
+        subject.value?.should == true
+      end
+
+      it "is false when the attribute is '0'" do
+        subject.value = "0"
+        subject.value?.should == false
+      end
+
+      it "is true when the attribute is '1'" do
+        subject.value = "1"
+        subject.value?.should == true
+      end
+
+      it "is false when the attribute is '0.0'" do
+        subject.value = "0.0"
+        subject.value?.should == false
+      end
+
+      it "is true when the attribute is '0.1'" do
+        subject.value = "0.1"
+        subject.value?.should == true
+      end
+
+      it "is true when the attribute is '-1'" do
+        subject.value = "-1"
+        subject.value?.should == true
+      end
+
+      it "is false when the attribute is '-0.0'" do
+        subject.value = "-0.0"
+        subject.value?.should == false
+      end
+
+      it "is true when the attribute is '-0.1'" do
+        subject.value = "-0.1"
+        subject.value?.should == true
+      end
+
+      it "is true when the attribute is 'true'" do
+        subject.value = "true"
+        subject.value?.should == true
+      end
+
+      it "is false when the attribute is 'false'" do
+        subject.value = "false"
+        subject.value?.should == false
+      end
+
+      it "is true when the attribute is 't'" do
+        subject.value = "t"
+        subject.value?.should == true
+      end
+
+      it "is false when the attribute is 'f'" do
+        subject.value = "f"
+        subject.value?.should == false
+      end
+
+      it "is true when the attribute is 'T'" do
+        subject.value = "T"
+        subject.value?.should == true
+      end
+
+      it "is false when the attribute is 'F'" do
+        subject.value = "F"
+        subject.value?.should == false
+      end
+
+      it "is true when the attribute is 'TRUE'" do
+        subject.value = "TRUE"
+        subject.value?.should == true
+      end
+
+      it "is false when the attribute is 'FALSE" do
+        subject.value = "FALSE"
+        subject.value?.should == false
+      end
+    end
+  end
+end