cantango-core 0.1.3 → 0.1.4

data/lib/cantango/core.rb

@@ -13,6 +13,6 @@ SweetLoader.mode = :require
 
 module CanTango
   autoload_modules :Helpers, :Ability, :Engine, :Filter
-  autoload_modules :Loader, :Model
+  autoload_modules :Loader, :Model, :Adaptor
   autoload_modules :Scope
 end

data/lib/cantango/helpers/debug.rb

@@ -29,11 +29,13 @@ module CanTango
       end
       
       def debug?
-        @do_debug ||= self.class.debug?
+        @do_debug ||= self.class.debug? if self.class.respond_to? :debug?
+        @do_debug ||= false
       end
       
       def debug_writer
-        @debug_writer ||= self.class.debug_writer
+        @debug_writer ||= self.class.debug_writer if self.class.respond_to? :debug_writer
+        @debug_writer ||= lambda {|m| puts m }
       end
     end
   end

data/lib/cantango/loader/yaml.rb

@@ -3,6 +3,10 @@ module CanTango
     class Yaml
       attr_accessor :file_name
 
+      def initialize file_name
+        @file_name = file_name
+      end
+
       def self.inherited subclass
         subclass.extend ClassMethods
       end
@@ -14,7 +18,7 @@ module CanTango
       def yml_content
         YAML.load_file(file_name)
       rescue
-        raise "Couldn't YAML file: #{file_name}"
+        raise "Couldn't load YAML file: #{file_name}"
       end
       
       module ClassMethods

data/spec/active_record/001_create_posters.rb

@@ -0,0 +1,13 @@
+class CreatePosters < ActiveRecord::Migration
+  def self.up
+    create_table :posters do |t|
+      t.integer   :user_id
+      t.string    :body
+      t.timestamps
+    end
+  end
+
+  def self.down
+    drop_table :posters
+  end
+end

data/spec/active_record/002_create_users.rb

@@ -0,0 +1,13 @@
+class CreateUsers < ActiveRecord::Migration
+  def self.up
+    create_table :users do |t|
+      t.string  :name
+      t.string  :email
+      t.timestamps
+    end
+  end
+
+  def self.down
+    drop_table :users
+  end
+end

data/spec/cantango/ability/base_spec.rb

@@ -3,9 +3,7 @@ require 'fixtures/models'
 
 module CanTango::Ability
   class Base
-    include CanTango::Ability::Helper::Account
-    
-    def permit_rules
+    def calculate_rules
       can :edit, Project
     end
   end
@@ -13,11 +11,10 @@ end
 
 describe CanTango::Ability::Base do
   before do
-    user = User.new 'krisy', 'krisy@gmail.com'
-    @account = Account.new user
+    @user = User.new 'krisy', 'krisy@gmail.com'
   end
 
-  subject { CanTango::Ability::Base.new @account }
+  subject { CanTango::Ability::Base.new @user }
 
   specify { subject.should be_allowed_to(:edit, Project) }
 end

data/spec/cantango/ability/callbacks_spec.rb

@@ -0,0 +1,48 @@
+require 'spec_helper'
+require 'fixtures/models'
+
+module CanTango::Ability
+  class Base
+    include CanTango::Ability::Callbacks
+
+    before_execute :add_x
+    after_execute :add_y
+    
+    def execute
+      within_callbacks do
+        clear_rules!
+        calculate_rules
+      end
+    end
+    
+    protected
+    
+    def buffer
+      @buffer ||= ""
+    end
+    
+    def add_x
+      buffer << 'x'
+    end
+
+    def add_y
+      buffer << 'y'
+    end
+  end
+end
+
+describe CanTango::Ability::Callbacks do
+  before do
+    @user = User.new 'admin', 'admin@mail.ru'
+  end
+
+  subject { CanTango::Ability::Base.new @user }
+
+  describe 'handle_callbacks :before' do
+    its(:buffer) { should match /x/ }
+  end
+
+  describe 'handle_callbacks :after' do
+    its(:buffer) { should match /y/ }
+  end
+end

data/spec/cantango/ability/executor/base_spec.rb

@@ -1,27 +1,28 @@
 require 'spec_helper'
 require 'fixtures/models'
 
-module CanTango::Ability
+module CanTango::Ability::Executor
   class Base
-    def permit_rules
-      can :read, Post
+    def calculate_rules
+      puts "calculate_rules"
+      can :write, Post
     end
   end
 end
 
 describe CanTango::Ability::Executor::Base do
-  context 'non-cached only' do
-    before do
-      @user = User.new 'admin', 'admin@mail.ru'
-    end
-
-    subject { CanTango::Ability::Executor::Base.new @user }
+  before do
+    @user = User.new 'admin', 'admin@mail.ru'
+    @ability = CanTango::Ability::Base.new @user
+  end
 
-    its(:non_cached_rules)  { should_not be_empty }
+  subject do 
+    CanTango::Ability::Executor::Base.new @ability
+  end
 
-    describe 'rules contain only non-cached rules' do
-      specify { subject.rules.size.should == @abil.non_cached_rules.size }
-      specify { subject.rules.size.should == 1 }
+  describe 'should not be executable' do
+    specify do
+      lambda { subject.execute }.should raise_error
     end
   end
-end
+end

data/spec/cantango/ability/executor/custom_spec.rb

@@ -16,14 +16,14 @@ class MyExecutor < CanTango::Ability::Executor::Base
     :my_exec
   end
 
-  def permit_rules
-    ability.permit_rules
+  def calculate_rules
+    ability.calculate_rules
   end
 end
 
 module CanTango::Ability
   class Base
-    def permit_rules
+    def calculate_rules
       can :edit, Project
     end
   end

data/spec/cantango/ability/executor/modal_spec.rb

@@ -0,0 +1,35 @@
+require 'spec_helper'
+require 'fixtures/models'
+
+module CanTango::Ability::Mode
+  class NoCache
+    def calculate_rules
+      can :write, Post
+    end
+  end
+end
+
+describe CanTango::Ability::Executor::Modal do
+  before do
+    @user = User.new 'admin', 'admin@mail.ru'
+    @ability = CanTango::Ability::Base.new @user
+  end
+
+  context 'Set execution mode to :no_cache' do
+    subject do 
+      CanTango::Ability::Executor::Modal.new @ability, :no_cache
+    end
+
+    its(:rules)  { should be_empty }
+
+    describe 'rules should be calculated on execute' do
+      before do
+        subject.execute
+      end
+      
+      specify { subject.rules.should_not be_empty }
+      specify { subject.rules.size.should == 1 }
+      specify { subject.rules.first.should be_a CanCan::Rule }
+    end
+  end
+end

data/spec/cantango/ability/executor_spec.rb

@@ -0,0 +1,29 @@
+require 'spec_helper'
+require 'fixtures/models'
+
+module CanTango::Ability
+  class MyExecutor
+    include CanTango::Ability::Executor
+  end
+end
+
+describe CanTango::Ability::Executor do
+  before do
+    @user = User.new 'krisy', 'krisy@gmail.com'
+    @ability = CanTango::Ability::Base.new @user
+  end
+
+  subject { CanTango::Ability::MyExecutor.new @ability }
+
+  describe 'calculate_rules by default returns rules' do
+    specify do
+      subject.calculate_rules.should == []
+    end
+  end
+  
+  describe 'execute by default calls calculate_rules' do
+    specify do
+      subject.execute.should == []
+    end
+  end
+end

data/spec/cantango/ability/mode/base_example.rb

@@ -0,0 +1,28 @@
+require 'spec_helper'
+require 'fixtures/models'
+
+shared_examples_for CanTango::Ability::Mode::Base do
+  before do
+    @user = User.new 'admin', 'admin@mail.ru'
+    @ability = CanTango::Ability::Base.new @user
+  end
+
+  subject { mode_class.new @ability }
+
+  describe 'Does NOT execute on instantiation' do      
+    specify do
+      subject.rules.should == []
+    end
+  end
+
+  describe 'execute will calculate rules' do
+    before do
+      subject.execute
+    end
+    specify { subject.rules.should_not be_empty }
+    specify { subject.rules.size.should == 1 }
+    specify { subject.rules.first.should be_a CanCan::Rule }
+    
+    specify { subject.should be_allowed_to(:read, Post) }
+  end
+end

data/spec/cantango/ability/mode/base_spec.rb

@@ -0,0 +1,15 @@
+require 'cantango/ability/mode/base_example'
+
+module CanTango::Ability::Mode
+  class Base
+    def calculate_rules
+      can :read, Post
+    end
+  end
+end
+
+describe CanTango::Ability::Mode::Base do
+  it_should_behave_like CanTango::Ability::Mode::Base do
+    let(:mode_class) { CanTango::Ability::Mode::Base }
+  end
+end

data/spec/cantango/ability/mode/no_cache_spec.rb

@@ -0,0 +1,15 @@
+require 'cantango/ability/mode/base_example'
+
+module CanTango::Ability::Mode
+  class NoCache
+    def calculate_rules
+      can :read, Post
+    end
+  end
+end
+
+describe CanTango::Ability::Mode::NoCache do
+  it_should_behave_like CanTango::Ability::Mode::Base do
+    let(:mode_class) { CanTango::Ability::Mode::NoCache }
+  end
+end

data/spec/cantango/ability/rules_spec.rb

@@ -0,0 +1,48 @@
+require 'spec_helper'
+require 'fixtures/models'
+
+module CanTango::Ability
+  class Base
+    include CanTango::Ability::Rules
+    
+    def calculate_rules
+      can :edit, Project
+    end
+  end
+end
+
+describe CanTango::Ability::Rules do
+  before do
+    @user = User.new 'krisy', 'krisy@gmail.com'
+  end
+
+  subject { CanTango::Ability::Base.new @user }
+
+  describe 'rules' do
+    specify do 
+      subject.rules.should == []
+    end
+  end
+
+  describe 'clear_rules!' do
+    before do
+      subject.rules = [1]
+      subject.clear_rules!
+    end
+
+    specify do 
+      subject.rules.should == []
+    end
+  end
+  
+  describe 'normalize_rules!' do
+    before do
+      subject.rules = [[1,2], 3]
+      subject.normalize_rules!
+    end
+
+    specify do 
+      subject.rules.should == [1,2,3]
+    end
+  end
+end

data/spec/cantango/adaptor/active_record_spec.rb

@@ -1,10 +1,17 @@
 # This is used to wrap the CanCan ORM adapters, that enable convenient 
 # 'simple' queries using hashes similar to 'metawhere' gem
 require 'spec_helper'
+require 'migration_helper'
 
 class Poster < ActiveRecord::Base
+  belongs_to :owner, :class_name => "User", :foreign_key => "user_id"
 end
 
+class User < ActiveRecord::Base
+  has_many :posters
+end
+
+
 module CanTango::Ability
   class Base
     def permit_rules
@@ -16,8 +23,10 @@ end
 
 describe CanTango::Adaptor::ActiveRecord do
   before do
-    @user = User.new 'admin', 'admin@mail.ru'
-    @stranger = User.new 'stranger', 'stranger@mail.ru'
+    migrate!
+
+    @user = User.create :name => 'admin', :email => 'admin@mail.ru'
+    @stranger = User.create :name => 'stranger', :email => 'strange@mail.ru'
     
     @poster = Poster.create :owner => @user
     @poster.owner = @user

data/spec/cantango/loader/yaml_spec.rb

@@ -1,5 +1,13 @@
 require 'spec_helper'
 
 describe CanTango::Loader::Yaml do
-  pending 'Nothing much to spec here!'
+  subject { CanTango::Loader::Yaml.new File.join(spec_folder, 'fixtures/test.yml')  }
+  
+  describe 'Parser not implemented' do
+    specify do
+      lambda { subject.parser }.should raise_error
+    end
+  end
+
+  its(:yml_content) { should be_a Hash }
 end

data/spec/cantango/rspec/be_allowed_to_spec.rb

@@ -1,5 +1,5 @@
 require 'spec_helper'
 
 describe 'RSpec::Matchers be_allowed_to' do
-  pending 'It should work!'
+  # pending 'It works :)'
 end

data/spec/db/database.yml

@@ -0,0 +1,4 @@
+adapter: sqlite3
+database: spec/db/development.sqlite3
+pool: 5
+timeout: 5000

data/spec/fixtures/test.yml

@@ -0,0 +1,2 @@
+---
+  hello: you

data/spec/migration_helper.rb

@@ -0,0 +1,17 @@
+require 'active_record'
+require 'database_cleaner'
+
+path = File.dirname(__FILE__) + '/db/database.yml'
+dbfile = File.open(path)
+dbconfig = YAML::load(dbfile)
+ActiveRecord::Base.establish_connection(dbconfig)
+# ActiveRecord::Base.logger = Logger.new(STDERR)
+DatabaseCleaner.strategy = :truncation
+
+def migration_folder
+  File.join(File.dirname(__FILE__), 'active_record')
+end
+
+def migrate!
+  ActiveRecord::Migrator.migrate File.join(File.dirname(__FILE__), 'active_record')
+end

data/spec/spec_helper.rb

@@ -5,6 +5,10 @@ require 'cantango/core'
 class Project
 end
 
+def spec_folder
+  File.dirname(__FILE__)
+end
+
 # Requires supporting files with custom matchers and macros, etc,
 # in ./support/ and its subdirectories.
 Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each {|f| require f}