protector 0.1.1 → 0.2.1

data/spec/lib/adapters/sequel_spec.rb

@@ -0,0 +1,156 @@
+require 'spec_helpers/boot'
+
+if defined?(Sequel)
+  load 'spec_helpers/adapters/sequel.rb'
+
+  describe Protector::Adapters::Sequel do
+    before(:all) do
+      load 'migrations/sequel.rb'
+
+      module ProtectionCase
+        extend ActiveSupport::Concern
+
+        included do |klass|
+          protect do |x|
+            scope{ where('1=0') } if x == '-'
+            scope{ where("#{klass.table_name}__number".to_sym => 999) } if x == '+' 
+
+            can :view, :dummy_id unless x == '-'
+          end
+        end
+      end
+
+      [Dummy, Fluffy].each{|c| c.send :include, ProtectionCase}
+
+      Dummy.create string: 'zomgstring', number: 999, text: 'zomgtext'
+      Dummy.create string: 'zomgstring', number: 999, text: 'zomgtext'
+      Dummy.create string: 'zomgstring', number: 777, text: 'zomgtext'
+      Dummy.create string: 'zomgstring', number: 777, text: 'zomgtext'
+
+      [Fluffy, Bobby].each do |m|
+        m.create string: 'zomgstring', number: 999, text: 'zomgtext', dummy_id: 1
+        m.create string: 'zomgstring', number: 777, text: 'zomgtext', dummy_id: 1
+        m.create string: 'zomgstring', number: 999, text: 'zomgtext', dummy_id: 2
+        m.create string: 'zomgstring', number: 777, text: 'zomgtext', dummy_id: 2
+      end
+
+      Fluffy.all.each{|f| Loony.create fluffy_id: f.id, string: 'zomgstring' }
+    end
+
+    #
+    # Model instance
+    #
+    describe Protector::Adapters::Sequel::Model do
+      let(:dummy) do
+        Class.new Sequel::Model(:dummies)
+      end
+
+      it "includes" do
+        Dummy.ancestors.should include(Protector::Adapters::Sequel::Model)
+      end
+
+      it "scopes" do
+        scope = Dummy.restrict!('!')
+        scope.should be_a_kind_of Sequel::Dataset
+        scope.protector_subject.should == '!'
+      end
+
+      it_behaves_like "a model"
+    end
+
+    #
+    # Model scope
+    #
+    describe Protector::Adapters::Sequel::Dataset do
+      it "includes" do
+        Dummy.none.class.ancestors.should include(Protector::DSL::Base)
+      end
+
+      it "saves subject" do
+        Dummy.restrict!('!').where(number: 999).protector_subject.should == '!'
+      end
+
+      it "forwards subject" do
+        Dummy.restrict!('!').where(number: 999).first.protector_subject.should == '!'
+        Dummy.restrict!('!').where(number: 999).to_a.first.protector_subject.should == '!'
+        Dummy.restrict!('!').eager_graph(fluffies: :loony).all.first.fluffies.first.loony.protector_subject.should == '!'
+      end
+
+      context "with null relation" do
+        it "checks existence" do
+          Dummy.any?.should == true
+          Dummy.restrict!('-').any?.should == false
+        end
+
+        it "counts" do
+          Dummy.count.should == 4
+          Dummy.restrict!('-').count.should == 0
+        end
+
+        context do
+          it "fetches first" do
+            Dummy.restrict!('-').first.should == nil
+          end
+        end
+
+        it "fetches all" do
+          fetched = Dummy.restrict!('-').to_a
+
+          Dummy.count.should == 4
+          fetched.length.should == 0
+        end
+      end
+
+      context "with active relation" do
+        it "checks existence" do
+          Dummy.any?.should == true
+          Dummy.restrict!('+').any?.should == true
+        end
+
+        it "counts" do
+          Dummy.count.should == 4
+          Dummy.restrict!('+').count.should == 2
+        end
+
+        it "fetches first" do
+          Dummy.restrict!('+').first.should be_a_kind_of Dummy
+        end
+
+        it "fetches all" do
+          fetched = Dummy.restrict!('+').to_a
+
+          Dummy.count.should == 4
+          fetched.length.should == 2
+        end
+      end
+    end
+
+    #
+    # Eager loading
+    #
+    describe Protector::Adapters::Sequel::Dataset do
+      describe "eager loading" do
+
+        context "straight" do
+          it "scopes" do
+            d = Dummy.restrict!('+').eager(:fluffies)
+            d.count.should == 2
+            d.first.fluffies.length.should == 1
+          end
+        end
+
+        context "graph" do
+          it "scopes" do
+            d = Dummy.restrict!('+').eager_graph(fluffies: :loony)
+            d.count.should == 4
+            d = d.all
+            d.length.should == 2 # which is terribly sick :doh:
+            d.first.fluffies.length.should == 1
+            d.first.fluffies.first.loony.should be_a_kind_of Loony
+          end
+        end
+      end
+    end
+  end
+
+end

data/spec/spec_helpers/adapters/active_record.rb

@@ -16,10 +16,44 @@ RSpec::Matchers.define :validate do
   end
 end
 
+RSpec::Matchers.define :destroy do
+  match do |actual|
+    actual.class.transaction do
+      actual.destroy.should == actual
+      raise ActiveRecord::Rollback
+    end
+
+    actual.class.where(id: actual.id).delete_all
+
+    true
+  end
+end
+
+RSpec::Matchers.define :survive do
+  match do |actual|
+    actual.class.transaction do
+      actual.destroy.should == false
+      raise ActiveRecord::Rollback
+    end
+
+    actual.class.where(id: actual.id).delete_all
+
+    true
+  end
+end
+
 def log!
   around(:each) do |e|
     ActiveRecord::Base.logger = Logger.new(STDOUT)
     e.run
     ActiveRecord::Base.logger = nil
   end
+end
+
+def assign!(model, fields)
+  model.assign_attributes(fields)
+end
+
+def read_attribute(model, field)
+  model.read_attribute(field)
 end

data/spec/spec_helpers/adapters/sequel.rb

@@ -0,0 +1,64 @@
+RSpec::Matchers.define :invalidate do
+  match do |actual|
+    DB.transaction do
+      expect{ actual.save }.to raise_error
+      actual.errors.on(:base).should == ["Access denied"]
+      raise Sequel::Rollback
+    end
+
+    true
+  end
+end
+
+RSpec::Matchers.define :validate do
+  match do |actual|
+    DB.transaction do
+      expect{ actual.save }.to_not raise_error
+      raise Sequel::Rollback
+    end
+
+    true
+  end
+end
+
+RSpec::Matchers.define :destroy do
+  match do |actual|
+    DB.transaction do
+      expect{ actual.destroy.should }.to_not raise_error
+      raise Sequel::Rollback
+    end
+
+    actual.class.where(id: actual.id).delete
+
+    true
+  end
+end
+
+RSpec::Matchers.define :survive do
+  match do |actual|
+    DB.transaction do
+      expect{ actual.destroy.should }.to raise_error
+      raise Sequel::Rollback
+    end
+
+    actual.class.where(id: actual.id).delete
+
+    true
+  end
+end
+
+def log!
+  around(:each) do |e|
+    DB.loggers << Logger.new(STDOUT)
+    e.run
+    DB.loggers = []
+  end
+end
+
+def assign!(model, fields)
+  model.set_all(fields)
+end
+
+def read_attribute(model, field)
+  model.instance_variable_get("@values")[field]
+end

data/spec/spec_helpers/boot.rb

@@ -1,21 +1,11 @@
 Bundler.require
 
+require 'coveralls'
+Coveralls.wear! if Coveralls.should_run?
+
 require 'protector'
 require_relative 'examples/model'
 
-module ProtectionCase
-  extend ActiveSupport::Concern
-
-  included do
-    protect do |x|
-      scope{ where('1=0') } if x == '-'
-      scope{ where(number: 999) } if x == '+' 
-
-      can :view, :dummy_id unless x == '-'
-    end
-  end
-end
-
 RSpec.configure do |config|
   config.treat_symbols_as_metadata_keys_with_true_values = true
   config.run_all_when_everything_filtered = true

data/spec/spec_helpers/examples/model.rb

@@ -12,7 +12,7 @@ shared_examples_for "a model" do
       end
     end
 
-    fields = Hash[*%w(id string number text dummy_id created_at updated_at).map{|x| [x, nil]}.flatten]
+    fields = Hash[*%w(id string number text dummy_id).map{|x| [x, nil]}.flatten]
     meta   = dummy.new.restrict!('!').protector_meta
 
     meta.access[:view].should   == fields
@@ -53,7 +53,7 @@ shared_examples_for "a model" do
       d = dummy.first.restrict!('!')
       d.number.should == nil
       d[:number].should == nil
-      d.read_attribute(:number).should_not == nil
+      read_attribute(d, :number).should_not == nil
       d.string.should == 'zomgstring'
     end
   end
@@ -184,13 +184,13 @@ shared_examples_for "a model" do
 
       it "marks blocked" do
         d = dummy.first
-        d.assign_attributes(string: 'bam', number: 1)
+        assign!(d, string: 'bam', number: 1)
         d.restrict!('!').updatable?.should == false
       end
 
       it "invalidates" do
         d = dummy.first.restrict!('!')
-        d.assign_attributes(string: 'bam', number: 1)
+        assign!(d, string: 'bam', number: 1)
         d.should invalidate
       end
     end
@@ -206,25 +206,25 @@ shared_examples_for "a model" do
 
       it "marks blocked" do
         d = dummy.first
-        d.assign_attributes(string: 'bam', number: 1)
+        assign!(d, string: 'bam', number: 1)
         d.restrict!('!').updatable?.should == false
       end
 
       it "marks allowed" do
         d = dummy.first
-        d.assign_attributes(string: 'bam')
+        assign!(d, string: 'bam')
         d.restrict!('!').updatable?.should == true
       end
 
       it "invalidates" do
         d = dummy.first.restrict!('!')
-        d.assign_attributes(string: 'bam', number: 1)
+        assign!(d, string: 'bam', number: 1)
         d.should invalidate
       end
 
       it "validates" do
         d = dummy.first.restrict!('!')
-        d.assign_attributes(string: 'bam')
+        assign!(d, string: 'bam')
         d.should validate
       end
     end
@@ -240,25 +240,25 @@ shared_examples_for "a model" do
 
       it "marks blocked" do
         d = dummy.first
-        d.assign_attributes(string: 'bam')
+        assign!(d, string: 'bam')
         d.restrict!('!').updatable?.should == false
       end
 
       it "marks allowed" do
         d = dummy.first
-        d.assign_attributes(string: '12345')
+        assign!(d, string: '12345')
         d.restrict!('!').updatable?.should == true
       end
 
       it "invalidates" do
         d = dummy.first.restrict!('!')
-        d.assign_attributes(string: 'bam')
+        assign!(d, string: 'bam')
         d.should invalidate
       end
 
       it "validates" do
         d = dummy.first.restrict!('!')
-        d.assign_attributes(string: '12345')
+        assign!(d, string: '12345')
         d.should validate
       end
     end
@@ -274,25 +274,25 @@ shared_examples_for "a model" do
 
       it "marks blocked" do
         d = dummy.first
-        d.assign_attributes(number: 500)
+        assign!(d, number: 500)
         d.restrict!('!').updatable?.should == false
       end
 
       it "marks allowed" do
         d = dummy.first
-        d.assign_attributes(number: 2)
+        assign!(d, number: 2)
         d.restrict!('!').updatable?.should == true
       end
 
       it "invalidates" do
         d = dummy.first.restrict!('!')
-        d.assign_attributes(number: 500)
+        assign!(d, number: 500)
         d.should invalidate
       end
 
       it "validates" do
         d = dummy.first.restrict!('!')
-        d.assign_attributes(number: 2)
+        assign!(d, number: 2)
         d.should validate
       end
     end
@@ -323,7 +323,8 @@ shared_examples_for "a model" do
         protect do; end
       end
 
-      dummy.first.restrict!('!').destroy.should == false
+      d = dummy.create.restrict!('!')
+      d.should survive
     end
 
     it "validates" do
@@ -331,9 +332,8 @@ shared_examples_for "a model" do
         protect do; can :destroy; end
       end
 
-      d = dummy.create!.restrict!('!')
-      d.destroy.should == d
-      d.destroyed?.should == true
+      d = dummy.create.restrict!('!')
+      d.should destroy
     end
   end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: protector
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.2.1
 platform: ruby
 authors:
 - Boris Staal
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-06-02 00:00:00.000000000 Z
+date: 2013-06-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -59,22 +59,34 @@ files:
 - gemfiles/AR_3.2.gemfile.lock
 - gemfiles/AR_4.gemfile
 - gemfiles/AR_4.gemfile.lock
+- gemfiles/Mongoid.gemfile
+- gemfiles/Mongoid.gemfile.lock
+- gemfiles/Sequel.gemfile
+- gemfiles/Sequel.gemfile.lock
 - lib/protector.rb
 - lib/protector/adapters/active_record.rb
 - lib/protector/adapters/active_record/association.rb
 - lib/protector/adapters/active_record/base.rb
 - lib/protector/adapters/active_record/preloader.rb
 - lib/protector/adapters/active_record/relation.rb
+- lib/protector/adapters/sequel.rb
+- lib/protector/adapters/sequel/dataset.rb
+- lib/protector/adapters/sequel/eager_graph_loader.rb
+- lib/protector/adapters/sequel/model.rb
 - lib/protector/dsl.rb
 - lib/protector/version.rb
 - locales/en.yml
 - migrations/active_record.rb
-- perf/active_record.rb
+- migrations/sequel.rb
+- perf/active_record_perf.rb
 - perf/perf_helpers/boot.rb
+- perf/sequel_perf.rb
 - protector.gemspec
 - spec/lib/adapters/active_record_spec.rb
+- spec/lib/adapters/sequel_spec.rb
 - spec/lib/dsl_spec.rb
 - spec/spec_helpers/adapters/active_record.rb
+- spec/spec_helpers/adapters/sequel.rb
 - spec/spec_helpers/boot.rb
 - spec/spec_helpers/examples/model.rb
 homepage: https://github.com/inossidabile/protector
@@ -104,7 +116,9 @@ summary: 'Protector is a successor to the Heimdallr gem: it hits the same goals
   the Ruby way'
 test_files:
 - spec/lib/adapters/active_record_spec.rb
+- spec/lib/adapters/sequel_spec.rb
 - spec/lib/dsl_spec.rb
 - spec/spec_helpers/adapters/active_record.rb
+- spec/spec_helpers/adapters/sequel.rb
 - spec/spec_helpers/boot.rb
 - spec/spec_helpers/examples/model.rb