cia 0.5.7 → 0.5.8

data/lib/cia/attribute_change.rb

@@ -11,7 +11,7 @@ module CIA
     validates_presence_of :event, :attribute_name
 
     if ActiveRecord::VERSION::MAJOR > 2
-      scope :previous, :order => "id desc"
+      scope :previous, lambda { order("id desc") }
     else
       named_scope :previous, :order => "id desc"
     end
@@ -19,7 +19,8 @@ module CIA
     delegate :created_at, :to => :event
 
     def self.on_attribute(attribute)
-      scoped(:conditions => {:attribute_name => attribute})
+      conditions = {:attribute_name => attribute}
+      ActiveRecord::VERSION::MAJOR == 2 ? scoped(:conditions => conditions) : where(conditions)
     end
 
     def self.max_value_size

data/lib/cia/event.rb

@@ -10,7 +10,8 @@ module CIA
     validates_presence_of :action
 
     def self.previous
-      scoped(:order => "created_at desc")
+      order = "created_at desc"
+      ActiveRecord::VERSION::MAJOR == 2 ? scoped(:order => order) : order(order)
     end
 
     def attribute_change_hash

data/lib/cia/version.rb

@@ -1,3 +1,3 @@
 module CIA
-  VERSION = '0.5.7'
+  VERSION = '0.5.8'
 end

metadata

@@ -1,27 +1,126 @@
 --- !ruby/object:Gem::Specification
 name: cia
 version: !ruby/object:Gem::Version
-  version: 0.5.7
+  version: 0.5.8
+  prerelease: 
 platform: ruby
 authors:
 - Michael Grosser
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-12-05 00:00:00.000000000 Z
+date: 2014-09-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: json
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: bump
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '2'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '2'
+- !ruby/object:Gem::Dependency
+  name: wwtd
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: activerecord
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: sqlite3
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 description: 
@@ -30,51 +129,42 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- .travis.yml
-- Appraisals
-- Gemfile
-- Gemfile.lock
-- MIGRATION.rb
-- Rakefile
-- Readme.md
-- cia.gemspec
-- gemfiles/rails2.gemfile
-- gemfiles/rails2.gemfile.lock
-- gemfiles/rails3.gemfile
-- gemfiles/rails3.gemfile.lock
 - lib/cia.rb
 - lib/cia/attribute_change.rb
 - lib/cia/auditable.rb
 - lib/cia/event.rb
 - lib/cia/source_validation.rb
 - lib/cia/version.rb
-- spec/cia/attribute_change_spec.rb
-- spec/cia/event_spec.rb
-- spec/cia_spec.rb
-- spec/spec_helper.rb
-homepage: http://github.com/grosser/cia
+homepage: https://github.com/grosser/cia
 licenses:
 - MIT
-metadata: {}
 post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
+  none: false
   requirements:
-  - - '>='
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
+      segments:
+      - 0
+      hash: 3646470647389073675
 required_rubygems_version: !ruby/object:Gem::Requirement
+  none: false
   requirements:
-  - - '>='
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
+      segments:
+      - 0
+      hash: 3646470647389073675
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.0.3
+rubygems_version: 1.8.23
 signing_key: 
-specification_version: 4
+specification_version: 3
 summary: Audit model events like update/create/delete + attribute changes + group
   them by transaction, in normalized table layout for easy query access.
 test_files: []

checksums.yaml

@@ -1,7 +0,0 @@
----
-SHA1:
-  metadata.gz: 095757bdffb2cf6dd9966a37562bd4a61958fd20
-  data.tar.gz: d01a2b7a29c0791224934dcf4a012a20cc60c512
-SHA512:
-  metadata.gz: 377d5103d09854f5dac67ae4fcc9e84ee3bc94ec51519a19b5dfa97b3482bb15198dda28a543121eb5a44ff6a765b947c95f045982e080b90aa978c46a6d6a55
-  data.tar.gz: 4d940a36f27b7108e609509ae6b76d8eba745b33f59cb634ac5b1ec0530a85d056ab4857a63e133b1fb5f0f4d884c096705a4c5b255844c1c5b6a743785b64d6

data/.travis.yml

@@ -1,12 +0,0 @@
-bundler_args: ""
-rvm:
-  - 1.9.3
-  - 2.0.0
-gemfile:
- - gemfiles/rails2.gemfile
- - gemfiles/rails3.gemfile
-matrix:
-  exclude:
-    - rvm: 2.0.0
-      gemfile: gemfiles/rails2.gemfile
-script: bundle exec rake spec

data/Appraisals

@@ -1,8 +0,0 @@
-appraise "rails2" do
-  gem 'activerecord', "2.3.14"
-  gem 'after_commit'
-end
-
-appraise "rails3" do
-  gem 'activerecord', "3.2.3"
-end

data/Gemfile

@@ -1,10 +0,0 @@
-source "https://rubygems.org"
-gemspec
-
-gem 'bump'
-gem 'rake'
-gem 'rspec', '~>2'
-gem 'appraisal'
-gem 'wwtd'
-gem 'activerecord'
-gem 'sqlite3'

data/Gemfile.lock

@@ -1,55 +0,0 @@
-PATH
-  remote: .
-  specs:
-    cia (0.5.7)
-      json
-
-GEM
-  remote: https://rubygems.org/
-  specs:
-    activemodel (3.2.11)
-      activesupport (= 3.2.11)
-      builder (~> 3.0.0)
-    activerecord (3.2.11)
-      activemodel (= 3.2.11)
-      activesupport (= 3.2.11)
-      arel (~> 3.0.2)
-      tzinfo (~> 0.3.29)
-    activesupport (3.2.11)
-      i18n (~> 0.6)
-      multi_json (~> 1.0)
-    appraisal (0.5.1)
-      bundler
-      rake
-    arel (3.0.2)
-    builder (3.0.4)
-    bump (0.3.9)
-    diff-lcs (1.1.3)
-    i18n (0.6.1)
-    json (1.8.1)
-    multi_json (1.5.0)
-    rake (10.0.3)
-    rspec (2.12.0)
-      rspec-core (~> 2.12.0)
-      rspec-expectations (~> 2.12.0)
-      rspec-mocks (~> 2.12.0)
-    rspec-core (2.12.2)
-    rspec-expectations (2.12.1)
-      diff-lcs (~> 1.1.3)
-    rspec-mocks (2.12.1)
-    sqlite3 (1.3.6)
-    tzinfo (0.3.35)
-    wwtd (0.4.4)
-
-PLATFORMS
-  ruby
-
-DEPENDENCIES
-  activerecord
-  appraisal
-  bump
-  cia!
-  rake
-  rspec (~> 2)
-  sqlite3
-  wwtd

data/MIGRATION.rb

@@ -1,20 +0,0 @@
-create_table :cia_events do |t|
-  t.integer :actor_id
-  t.string :actor_type
-  t.string :source_type, :action, :null => false
-  t.integer :source_id, :null => false
-  t.string :ip_address
-  t.string :message
-  t.string :source_display_name
-  t.timestamp :created_at#, :null => false
-end
-
-create_table :cia_attribute_changes do |t|
-  t.integer :cia_event_id, :source_id, :null => false
-  t.string :attribute_name, :source_type, :null => false
-  t.string :old_value, :new_value
-end
-
-# DOWN
-# drop_table :cia_events
-# drop_table :cia_attribute_changes

data/Rakefile

@@ -1,11 +0,0 @@
-require 'bundler/setup'
-require 'appraisal'
-require 'bundler/gem_tasks'
-require 'bump/tasks'
-require 'wwtd/tasks'
-
-task :default => ["appraisal:gemfiles", :wwtd]
-
-task :spec do
-  sh "rspec spec/"
-end

data/Readme.md

@@ -1,106 +0,0 @@
-Central Internal Auditing
-============================
-
-Audit model actions like update/create/destroy/<custom> + attribute changes.
-
- - normalized and queryable through table layout
- - actors and subjects are polymorphic
- - works on ActiveRecord 2 and 3
-
-Table layout:
-
-   Event (actor/ip/time/updated subject + message)
-    -> has many attribute changes (changed password from foo to bar on subject)
-
-
-Install
-=======
-    gem install cia
-Or
-
-    rails plugin install git://github.com/grosser/cia.git
-
-`rails g migration add_cia` + paste [Migration](https://raw.github.com/grosser/cia/master/MIGRATION.rb)
-
-
-Usage
-=====
-
-```Ruby
-class User < ActiveRecord::Base
-  include CIA::Auditable
-  audited_attributes :email, :crypted_password
-end
-
-class ApplicationController < ActionController::Base
-  around_filter :scope_auditing
-
-  def scope_auditing
-    CIA.audit :actor => current_user, :ip_address => request.remote_ip do
-      yield
-    end
-  end
-end
-
-# quick access
-User.last.cia_events
-changes = User.last.cia_attribute_changes
-last_passwords = changes.where(:attribute_name => "crypted_password").map(&:new_value)
-
-# exceptions (raised by default)
-CIA.exception_handler = lambda{|e| raise e unless Rails.env.production? }
-
-# conditional auditing
-class User < ActiveRecord::Base
-  audited_attributes :email, :if => :interesting?
-
-  def interesting?
-    ...
-  end
-end
-
-# adding an actor e.g. for user creation
-CIA.current_actor = @user
-
-# custom changes
-class User < ActiveRecord::Base
-  def cia_changes
-    changes.merge("this" => ["always", "changes"])
-  end
-end
-
-# using after_commit, useful if the CIA::Event is stored in a different database then the audited class
-class User < ActiveRecord::Base
-  include CIA::Auditable
-  audited_attributes :email, :crypted_password, :callback => :after_commit
-end
-
-# passing arbitrary attributes into the .audit method
-CIA.non_recordable_attributes = [:my_pretty_audit_property]
-CIA.audit(:actor => current_user, :my_pretty_audit_property => "12345") do
-  ...
-end
-
-# storing complex objects in old/new and reducing it's size if it's to big (serialized via json)
-value = CIA::AttributeChange.serialize_for_storage(["some", "complex"*1000, "object"]){|too_big| too_big.delete_at(1); too_big }
-CIA::AttributeChange.create!(:old_value => value)
-
-# add something to current transaction or start a new audit
-CIA.audit :bar => :baz, :foo => :bang do
-  CIA.amend_audit :foo => :bar do
-    puts CIA.current_transaction
-  end
-end
--> {:foo => :bar, :bar => :baz}
-```
-
-
-# TODO
- - reuse AR3+ previous_changes in a nice way
-
-Author
-======
-[Michael Grosser](http://grosser.it)<br/>
-michael@grosser.it<br/>
-License: MIT<br/>
-[![Build Status](https://secure.travis-ci.org/grosser/cia.png)](http://travis-ci.org/grosser/cia)

data/cia.gemspec

@@ -1,13 +0,0 @@
-$LOAD_PATH.unshift File.expand_path('../lib', __FILE__)
-name = "cia"
-require "#{name}/version"
-
-Gem::Specification.new name, CIA::VERSION do |s|
-  s.summary = "Audit model events like update/create/delete + attribute changes + group them by transaction, in normalized table layout for easy query access."
-  s.authors = ["Michael Grosser"]
-  s.email = "michael@grosser.it"
-  s.homepage = "http://github.com/grosser/#{name}"
-  s.files = `git ls-files`.split("\n")
-  s.license = 'MIT'
-  s.add_runtime_dependency "json"
-end

data/gemfiles/rails2.gemfile

@@ -1,14 +0,0 @@
-# This file was generated by Appraisal
-
-source "https://rubygems.org"
-
-gem "bump"
-gem "rake"
-gem "rspec", "~>2"
-gem "appraisal"
-gem "wwtd"
-gem "sqlite3"
-gem "activerecord", "2.3.14"
-gem "after_commit"
-
-gemspec :path=>"../"

data/gemfiles/rails2.gemfile.lock

@@ -1,45 +0,0 @@
-PATH
-  remote: ../
-  specs:
-    cia (0.5.6)
-      json
-
-GEM
-  remote: https://rubygems.org/
-  specs:
-    activerecord (2.3.14)
-      activesupport (= 2.3.14)
-    activesupport (2.3.14)
-    after_commit (1.0.10)
-      activerecord (>= 1.15.6, < 3.0.0)
-    appraisal (0.4.1)
-      bundler
-      rake
-    bump (0.3.9)
-    diff-lcs (1.1.3)
-    json (1.8.1)
-    rake (0.9.2.2)
-    rspec (2.10.0)
-      rspec-core (~> 2.10.0)
-      rspec-expectations (~> 2.10.0)
-      rspec-mocks (~> 2.10.0)
-    rspec-core (2.10.1)
-    rspec-expectations (2.10.0)
-      diff-lcs (~> 1.1.3)
-    rspec-mocks (2.10.1)
-    sqlite3 (1.3.6)
-    wwtd (0.4.4)
-
-PLATFORMS
-  ruby
-
-DEPENDENCIES
-  activerecord (= 2.3.14)
-  after_commit
-  appraisal
-  bump
-  cia!
-  rake
-  rspec (~> 2)
-  sqlite3
-  wwtd

data/gemfiles/rails3.gemfile

@@ -1,13 +0,0 @@
-# This file was generated by Appraisal
-
-source "https://rubygems.org"
-
-gem "bump"
-gem "rake"
-gem "rspec", "~>2"
-gem "appraisal"
-gem "wwtd"
-gem "sqlite3"
-gem "activerecord", "3.2.3"
-
-gemspec :path=>"../"

data/gemfiles/rails3.gemfile.lock

@@ -1,55 +0,0 @@
-PATH
-  remote: ../
-  specs:
-    cia (0.5.6)
-      json
-
-GEM
-  remote: https://rubygems.org/
-  specs:
-    activemodel (3.2.3)
-      activesupport (= 3.2.3)
-      builder (~> 3.0.0)
-    activerecord (3.2.3)
-      activemodel (= 3.2.3)
-      activesupport (= 3.2.3)
-      arel (~> 3.0.2)
-      tzinfo (~> 0.3.29)
-    activesupport (3.2.3)
-      i18n (~> 0.6)
-      multi_json (~> 1.0)
-    appraisal (0.5.1)
-      bundler
-      rake
-    arel (3.0.2)
-    builder (3.0.4)
-    bump (0.3.9)
-    diff-lcs (1.1.3)
-    i18n (0.6.1)
-    json (1.8.1)
-    multi_json (1.5.0)
-    rake (10.0.3)
-    rspec (2.12.0)
-      rspec-core (~> 2.12.0)
-      rspec-expectations (~> 2.12.0)
-      rspec-mocks (~> 2.12.0)
-    rspec-core (2.12.2)
-    rspec-expectations (2.12.1)
-      diff-lcs (~> 1.1.3)
-    rspec-mocks (2.12.1)
-    sqlite3 (1.3.6)
-    tzinfo (0.3.35)
-    wwtd (0.4.4)
-
-PLATFORMS
-  ruby
-
-DEPENDENCIES
-  activerecord (= 3.2.3)
-  appraisal
-  bump
-  cia!
-  rake
-  rspec (~> 2)
-  sqlite3
-  wwtd

data/spec/cia/attribute_change_spec.rb

@@ -1,100 +0,0 @@
-# encoding: utf-8
-require 'spec_helper'
-
-describe CIA::AttributeChange do
-  it "stores times as db format" do
-    t = Time.now
-    create_change(:old_value => t).reload.old_value.sub(/\.\d+$/,'').should == t.to_s(:db)
-  end
-
-  it "stores dates as db format" do
-    create_change(:old_value => Date.new(2012)).reload.old_value.should == "2012-01-01"
-  end
-
-  it "stores booleans as db format" do
-    create_change(:old_value => false).reload.old_value.should == "f"
-    create_change(:old_value => true).reload.old_value.should == "t"
-  end
-
-  it "stores nil as nil" do
-    create_change(:old_value => nil).reload.old_value.should == nil
-  end
-
-  it "delegates create_at to event" do
-    t = Time.now
-    event = CIA::Event.new(:created_at => t)
-    change = CIA::AttributeChange.new(:event => event)
-    change.created_at.should == event.created_at
-  end
-
-  describe ".previous" do
-    it "finds by id desc" do
-      CIA::AttributeChange.delete_all
-      a = create_change
-      b = create_change
-      CIA::AttributeChange.previous.should == [b,a]
-    end
-  end
-
-  describe ".on_attribute" do
-    it "finds with attribute" do
-      a = create_change :attribute_name => :xxx
-      b = create_change :attribute_name => :yyy
-      CIA::AttributeChange.on_attribute(:xxx).all.should == [a]
-    end
-  end
-
-  describe "enforcing presence of source" do
-    it "requires a source when associated event requires a source" do
-      event = CIA::Event.new { |event| event.id = 1 }
-      event.stub(:source_must_be_present? => true)
-      change = CIA::AttributeChange.new(:event => event, :attribute_name => 'awesomeness')
-
-      change.valid?.should be_false
-      change.errors.full_messages.should =~ ["Source can't be blank"]
-    end
-
-    it "does not require a source when associated event does not" do
-      event = CIA::Event.new { |event| event.id = 1 }
-      event.stub(:source_must_be_present? => false)
-      change = CIA::AttributeChange.new(:event => event, :attribute_name => 'awesomeness',
-                                        :source_type => 'ObscureType', :source_id => 101)
-
-      change.valid?.should be_true
-    end
-  end
-
-  describe ".max_value_size" do
-    it "is the width of the old/new column" do
-      CIA::AttributeChange.max_value_size.should == 255
-    end
-  end
-
-  describe ".serialize_for_storage" do
-    it "stores as json" do
-      CIA::AttributeChange.serialize_for_storage([["xxx"]]){}.should == '[["xxx"]]'
-    end
-
-    it "calls the block to remove an item" do
-      CIA::AttributeChange.serialize_for_storage([["xxx"], ["x"*300], ["yyy"]]){ |array| array.delete_at(1); array  }.should == '[["xxx"],["yyy"]]'
-    end
-
-    it "blows up if block fails to reduce size to prevent loops" do
-      expect{
-        CIA::AttributeChange.serialize_for_storage([["xxx"], ["x"*300], ["yyy"]]){ |array| array  }
-      }.to raise_error
-    end
-
-    it "takes multibyte into account" do
-      called = false
-      CIA::AttributeChange.serialize_for_storage(["å" * 200]){ |array| called = true; "x" }
-      called.should == true
-    end
-
-    it "does not go crazy on multibytes" do
-      called = false
-      CIA::AttributeChange.serialize_for_storage(["å" * 100]){ |array| called = true; "x" }
-      called.should == false
-    end
-  end
-end

data/spec/cia/event_spec.rb

@@ -1,57 +0,0 @@
-require 'spec_helper'
-
-describe CIA::Event do
-  it "has many attribute_changes" do
-    change = create_change
-    change.event.attribute_changes.should == [change]
-    change.event.destroy
-    expect{ change.reload }.to raise_error(ActiveRecord::RecordNotFound)
-  end
-
-  context "attribute_change_hash" do
-    it "is empty for empty changes" do
-      create_event.attribute_change_hash.should == {}
-    end
-
-    it "contains all changes" do
-      change = create_change(:old_value => "a", :new_value => "b")
-      change = create_change(:attribute_name => "foo", :old_value => "b", :new_value => nil, :event => change.event)
-      change.event.attribute_change_hash.should == {"bar" => ["a", "b"], "foo" => ["b", nil]}
-    end
-  end
-
-  context ".previous" do
-    it "is sorted id desc" do
-      events = [create_event(:created_at => 3.days.ago), create_event(:created_at => 2.days.ago), create_event(:created_at => 1.day.ago)].map(&:id)
-      CIA::Event.previous.map(&:id).should == events.reverse
-    end
-  end
-
-  context "validations" do
-    let(:source_attributes){ {:source => nil, :source_id => 99999, :source_type => "Car"} }
-
-    it "validates source" do
-      expect{
-        create_event(source_attributes)
-      }.to raise_error
-    end
-
-    it "does not validates source when action is destroy" do
-      create_event(source_attributes.merge(:action => "destroy"))
-    end
-
-    it "does not validates source when updating" do
-      create_event.update_attributes!(:source_id => 9999)
-    end
-
-    it "does not validates source when source_display_name is present" do
-      create_event(:source => nil, :source_id => -111, :source_type => 'FakeTypeHere', :source_display_name => 'abc')
-    end
-
-    it "validates source when source_display_name is blank" do
-      expect{
-        create_event(:source => nil, :source_id => -111, :source_type => 'FakeTypeHere', :source_display_name => '')
-      }.to raise_error
-    end
-  end
-end

data/spec/cia_spec.rb

@@ -1,439 +0,0 @@
-require 'spec_helper'
-
-describe CIA do
-  it "has a VERSION" do
-    CIA::VERSION.should =~ /^[\.\da-z]+$/
-  end
-
-  describe ".audit" do
-    it "has no transaction when it starts" do
-      CIA.current_transaction.should == nil
-    end
-
-    it "starts a new transaction" do
-      result = 1
-      CIA.audit({:a => 1}) do
-        result = CIA.current_transaction
-      end
-      result.should == {:a => 1}
-    end
-
-    it "stops the transaction after the block" do
-      CIA.audit({}){}
-      CIA.current_transaction.should == nil
-    end
-
-    it "returns the block content" do
-      CIA.audit({}){ 1 }.should == 1
-    end
-
-    it "is threadsafe" do
-      Thread.new do
-        CIA.audit({}) do
-          sleep 0.04
-        end
-      end
-      sleep 0.01
-      CIA.current_transaction.should == nil
-      sleep 0.04 # so next tests dont fail
-    end
-
-    it "can stack" do
-      states = []
-      CIA.audit(:a => 1) do
-        states << CIA.current_transaction
-        CIA.audit(:b => 1) do
-          states << CIA.current_transaction
-        end
-        states << CIA.current_transaction
-      end
-      states << CIA.current_transaction
-      states.should == [{:a => 1}, {:b => 1}, {:a => 1}, nil]
-    end
-  end
-
-  describe ".amend_audit" do
-    it "opens a new transaction when none exists" do
-      t = nil
-      CIA.amend_audit(:actor => 111){ t = CIA.current_transaction }
-      t.should == {:actor => 111}
-    end
-
-    it "amends a running transaction" do
-      t = nil
-      CIA.amend_audit(:actor => 222, :ip_address => 123) do
-        CIA.amend_audit(:actor => 111) { t = CIA.current_transaction }
-      end
-      t.should == {:actor => 111, :ip_address => 123}
-    end
-
-    it "returns to old state after transaction" do
-      CIA.amend_audit(:actor => 222, :ip_address => 123) do
-        CIA.amend_audit(:actor => 111) {  }
-      end
-      CIA.current_transaction.should == nil
-
-      CIA.amend_audit(:actor => 111) {  }
-      CIA.current_transaction.should == nil
-    end
-  end
-
-  describe ".record" do
-    let(:object) { Car.new }
-
-    around do |example|
-      CIA.audit :actor => User.create! do
-        example.call
-      end
-    end
-
-    it "tracks create" do
-      expect{
-        object.save!
-      }.to change{ CIA::Event.count }.by(+1)
-      CIA::Event.last.action.should == "create"
-    end
-
-    it "tracks delete" do
-      object.save!
-      expect{
-        object.destroy
-      }.to change{ CIA::Event.count }.by(+1)
-      CIA::Event.last.action.should == "destroy"
-    end
-
-    it "tracks update" do
-      object.save!
-      expect{
-        object.update_attributes(:wheels => 3)
-      }.to change{ CIA::Event.count }.by(+1)
-      CIA::Event.last.action.should == "update"
-    end
-
-    it "does not track failed changes" do
-      car = Car.create!(:wheels => 1).id
-      expect{
-        expect{ FailCar.new(:wheels => 4).save  }.to raise_error(FailCar::Oops)
-        car = FailCar.find(car)
-        expect{ car.update_attributes(:wheels => 2) }.to raise_error(FailCar::Oops)
-        expect{ car.destroy }.to raise_error(FailCar::Oops)
-      }.to_not change{ CIA::Event.count }
-    end
-
-    it "is rolled back if auditing fails" do
-      CIA.should_receive(:record).and_raise("XXX")
-      expect{
-        expect{
-          CIA.audit{ object.save! }
-        }.to raise_error("XXX")
-      }.to_not change{ object.class.count }
-    end
-
-    it "is ok with non-attribute methods passed into .audit if they are set as non-recordable" do
-      CIA.non_recordable_attributes = [:foo]
-      expect {
-        CIA.audit(:actor => User.create!, :foo => 'bar') {
-          object.save!
-        }
-      }.to change{ CIA::Event.count }.by(+1)
-    end
-
-    context "nested classes with multiple audited_attributes" do
-      let(:object){ NestedCar.new }
-
-      it "has the exclusive sub-classes attributes of the nested class" do
-        object.class.audited_attributes.should == ["drivers"]
-      end
-
-      it "does not record twice for nested classes" do
-        expect{
-          CIA.audit{ object.save! }
-        }.to change{ CIA::Event.count }.by(+1)
-      end
-
-      it "does not record twice for super classes" do
-        expect{
-          CIA.audit{ Car.new.save! }
-        }.to change{ CIA::Event.count }.by(+1)
-      end
-    end
-
-    context "nested classes with 1 audited_attributes" do
-      let(:object){ InheritedCar.new }
-
-      it "has the super-classes attributes" do
-        object.class.audited_attributes.should == ["wheels"]
-      end
-
-      it "does not record twice for nested classes" do
-        expect{
-          CIA.audit{ object.save! }
-        }.to change{ CIA::Event.count }.by(+1)
-      end
-
-      it "does not record twice for super classes" do
-        expect{
-          CIA.audit{ Car.new.save! }
-        }.to change{ CIA::Event.count }.by(+1)
-      end
-    end
-
-    context "custom changes" do
-      let(:object) { CarWithCustomChanges.new }
-
-      it "tracks custom changes" do
-        object.save!
-        expect{
-          object.update_attributes(:wheels => 3)
-        }.to change{ CIA::Event.count }.by(+1)
-        CIA::Event.last.action.should == "update"
-        CIA::Event.last.attribute_change_hash.should == {
-          "wheels" => [nil, "3"],
-          "foo" => ["bar", "baz"]
-        }
-      end
-    end
-
-    context ":if" do
-      let(:object) { CarWithIf.new }
-
-      it "tracks if :if is true" do
-        expect{
-          object.tested = true
-          object.save!
-        }.to change{ CIA::Event.count }.by(+1)
-        CIA::Event.last.action.should == "create"
-      end
-
-      it "does not track if :if is false" do
-        expect{
-          object.save!
-        }.to_not change{ CIA::Event.count }
-        CIA::Event.last.should == nil
-      end
-    end
-
-    context ":unless" do
-      let(:object) { CarWithUnless.new }
-
-      it "tracks if :unless is false" do
-        expect{
-          object.save!
-        }.to change{ CIA::Event.count }.by(+1)
-        CIA::Event.last.action.should == "create"
-      end
-
-      it "does not track if :unless is true" do
-        expect{
-          object.tested = true
-          object.save!
-        }.to_not change{ CIA::Event.count }
-        CIA::Event.last.should == nil
-      end
-    end
-
-    context "events" do
-      def parse_event_changes(event)
-        event.attribute_changes.map { |c| [c.attribute_name, c.old_value, c.new_value] }
-      end
-
-      def no_audit_created!
-        event = nil
-        expect{
-          event = yield
-        }.to_not change{ CIA::Event.count }
-
-        event.should == nil
-      end
-
-      it "records attributes in transaction" do
-        event = nil
-        CIA.audit :actor => User.create!, :ip_address => "1.2.3.4" do
-          event = CIA.record(:destroy, Car.create!)
-        end
-        event.ip_address.should == "1.2.3.4"
-      end
-
-      it "records attribute creations" do
-        source = Car.create!
-        source.wheels = 4
-        event = CIA.record(:update, source).reload
-
-        parse_event_changes(event).should == [["wheels", nil, "4"]]
-      end
-
-      it "can act on attributes in before_save" do
-        x = nil
-        CIA.current_transaction[:hacked_before_save_action] = lambda{|event| x = event.attribute_changes.size }
-        source = Car.create!
-        source.wheels = 4
-        CIA.record(:update, source)
-        x.should == 1
-      end
-
-      it "records multiple attributes" do
-        source = CarWith3Attributes.create!
-        source.wheels = 4
-        source.drivers = 2
-        source.color = "red"
-        event = CIA.record(:update, source).reload
-        parse_event_changes(event).should =~ [["wheels", nil, "4"], ["drivers", nil, "2"], ["color", nil, "red"]]
-      end
-
-      it "records attribute changes" do
-        source = Car.create!(:wheels => 2)
-        source.wheels = 4
-        event = CIA.record(:update, source).reload
-        parse_event_changes(event).should == [["wheels", "2", "4"]]
-      end
-
-      it "records attribute deletions" do
-        source = Car.create!(:wheels => 2)
-        source.wheels = nil
-        event = CIA.record(:update, source).reload
-        parse_event_changes(event).should == [["wheels", "2", nil]]
-      end
-
-      it "does not record unaudited attribute changes" do
-        source = Car.create!
-        source.drivers = 2
-        no_audit_created!{ CIA.record(:update, source) }
-      end
-
-      it "records audit_message as message even if there are no changes" do
-        source = CarWithAMessage.create!
-        source.audit_message = "Foo"
-        event = CIA.record(:update, source)
-
-        event.message.should == "Foo"
-        parse_event_changes(event).should == []
-      end
-
-      it "does not record after saving with an audit_message" do
-        source = CarWithAMessage.create!
-        source.audit_message = "Foo"
-        CIA.record(:update, source)
-
-        no_audit_created!{ CIA.record(:update, source) }
-      end
-
-      it "does not record if it's empty and there are no changes" do
-        source = CarWithAMessage.create!
-        source.audit_message = "   "
-        no_audit_created!{ CIA.record(:update, source) }
-      end
-
-      it "record non-updates even without changes" do
-        source = Car.create!
-        event = CIA.record(:create, source)
-        parse_event_changes(event).should == []
-      end
-    end
-
-    context "exception_handler" do
-      before do
-        $stderr.stub(:puts)
-        CIA.stub(:current_transaction).and_raise(StandardError.new("foo"))
-      end
-
-      def capture_exception
-        begin
-          old = CIA.exception_handler
-          ex = nil
-          CIA.exception_handler = lambda{|e| ex = e }
-          yield
-          ex
-        ensure
-          CIA.exception_handler = old
-        end
-      end
-
-      it "raises exceptions by the transaction" do
-        ex = nil
-        begin
-          object.save!
-        rescue Object => e
-          ex = e
-        end
-        ex.inspect.should == '#<StandardError: foo>'
-      end
-
-      it "can capture exception via handler" do
-        ex = capture_exception do
-          object.save!
-        end
-        ex.inspect.should == '#<StandardError: foo>'
-      end
-    end
-
-    context "with after_commit" do
-      let(:object){ CarWithTransactions.new(:wheels => 1) }
-
-      it "still tracks" do
-        expect{
-          CIA.audit{ object.save! }
-        }.to change{ CIA::Event.count }.by(+1)
-        CIA::Event.last.attribute_change_hash.should == {"wheels" => [nil, "1"]}
-      end
-
-      it "unsets temp-changes after the save" do
-        object.save!
-
-        # does not re-track old changes
-        expect{
-          CIA.audit{ object.update_attributes(:drivers => 2) }
-        }.to change{ CIA::Event.count }.by(+1)
-        CIA::Event.last.attribute_change_hash.should == {"drivers" => [nil, "2"]}
-
-        # empty changes
-        expect{
-          CIA.audit{ object.update_attributes(:drivers => 2) }
-        }.to_not change{ CIA::Event.count }
-      end
-
-      it "is not rolled back if auditing fails" do
-        CIA.should_receive(:record).and_raise("XXX")
-        begin
-          expect{
-            CIA.audit{ object.save! }
-          }.to change{ object.class.count }.by(+1)
-        rescue RuntimeError => e
-          # errors from after_commit are never raised in rails 3+
-          raise e if ActiveRecord::VERSION::MAJOR != 2 || e.message != "XXX"
-        end
-      end
-    end
-  end
-
-  context ".current_actor" do
-    it "is nil when nothing is set" do
-      CIA.current_actor.should == nil
-    end
-
-    it "is nil when no actor is set" do
-      CIA.audit do
-        CIA.current_actor.should == nil
-      end
-    end
-
-    it "is the current :actor" do
-      CIA.audit :actor => 111 do
-        CIA.current_actor.should == 111
-      end
-    end
-  end
-
-  context ".current_actor=" do
-    it "does nothing if no transaction is running" do
-      CIA.current_actor = 111
-      CIA.current_transaction.should == nil
-    end
-
-    it "sets when transaction is started" do
-      CIA.audit :actor => 222 do
-        CIA.current_actor = 111
-        CIA.current_transaction.should == {:actor => 111}
-      end
-    end
-  end
-end

data/spec/spec_helper.rb

@@ -1,120 +0,0 @@
-require 'cia'
-require 'after_commit' if ActiveRecord::VERSION::MAJOR == 2
-
-RSpec.configure do |config|
-  config.before do
-    CIA::Event.delete_all
-    CIA::AttributeChange.delete_all
-    CIA.non_recordable_attributes = nil
-  end
-end
-
-ActiveRecord::Base.establish_connection(
-  :adapter => "sqlite3",
-  :database => ":memory:"
-)
-
-ActiveRecord::Schema.verbose = false
-ActiveRecord::Schema.define(:version => 1) do
-  eval(File.read(File.expand_path('../../MIGRATION.rb', __FILE__)))
-
-  create_table :cars do |t|
-    t.integer :wheels
-    t.integer :drivers
-    t.string :color
-  end
-
-  create_table :users do |t|
-    t.string :email
-  end
-end
-
-class User < ActiveRecord::Base
-end
-
-class Car < ActiveRecord::Base
-  include CIA::Auditable
-  audit_attribute :wheels
-end
-
-class CarWithAMessage < ActiveRecord::Base
-  self.table_name = "cars"
-  include CIA::Auditable
-  audit_attribute :wheels
-  attr_accessor :audit_message
-end
-
-class CarWith3Attributes < ActiveRecord::Base
-  self.table_name = "cars"
-  include CIA::Auditable
-  audit_attribute :wheels, :color, :drivers
-end
-
-class CarWithIf < ActiveRecord::Base
-  self.table_name = "cars"
-  include CIA::Auditable
-  audit_attribute :wheels, :if => :tested
-  attr_accessor :tested
-end
-
-class CarWithUnless < ActiveRecord::Base
-  self.table_name = "cars"
-  include CIA::Auditable
-  audit_attribute :wheels, :unless => :tested
-  attr_accessor :tested
-end
-
-class CarWithCustomChanges < ActiveRecord::Base
-  self.table_name = "cars"
-  include CIA::Auditable
-  audit_attribute :wheels, :foo
-
-  def cia_changes
-    changes.merge("foo" => ["bar", "baz"])
-  end
-end
-
-class FailCar < ActiveRecord::Base
-  self.table_name = "cars"
-  include CIA::Auditable
-  audit_attribute :wheels
-
-  class Oops < Exception
-  end
-
-  after_update { |x| raise Oops }
-  after_create { |x| raise Oops }
-  after_destroy { |x| raise Oops }
-end
-
-class CarWithTransactions < ActiveRecord::Base
-  self.table_name = "cars"
-  include CIA::Auditable
-  audit_attribute :wheels, :drivers, :callback => :after_commit
-end
-
-class NestedCar < Car
-  audit_attribute :drivers
-end
-
-class InheritedCar < Car
-end
-
-def create_event(options={})
-  CIA::Event.create!({:source => Car.create!, :actor => User.create!, :action => "update"}.merge(options))
-end
-
-def create_change(options={})
-  event = options.delete(:event) || create_event
-  CIA::AttributeChange.create!({:event => event, :source => event.source, :attribute_name => "bar"}.merge(options))
-end
-
-# simulate a hacked cia event
-CIA::Event.class_eval do
-  before_save :hacked_before_save
-  attr_accessor :hacked_before_save_action
-
-  def hacked_before_save
-    hacked_before_save_action.call(self) if hacked_before_save_action
-  end
-end