pg_audit_log 0.6.5 → 0.6.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 39800555b4fbeb58610d4c7725ed799ff18efb63
-  data.tar.gz: 81c6d6c3c47ee07bb1444019308a381b33ab4b9b
+  metadata.gz: 7ec491dc91a88e95b3a62a99119d71b9880b1633
+  data.tar.gz: 5eb489c343b5d17208ae25a3f00ecc418d3aeecd
 SHA512:
-  metadata.gz: 2149585c888bd5d143c872d61d18ed8f459a9b0a77a2953a244b5c39d20e71a9a53a6486c047525e6a80d68ccb8507fc4cb4b613d78b00fa489bcf1e0e7a8e62
-  data.tar.gz: 3cf888d6c84efbca541ee57da5455e067744c155ebdc5bef8f669f6807951c6649761046c76e7b1a16097114efa87212a709ce6f90c64565051f4b0b321eedaf
+  metadata.gz: 83c89e0684320a261368562fbeb03167adc617814f858c59747787c3e775a57ea2d3783c8771561ead90692b3a8a7acaf30a5a3fe66ff5841aa7815424cc01e4
+  data.tar.gz: 5f29260cda979560c08e8a2cd76474363595daba8f1821f79c6148f555a5f839fd56c8b3badf7f86655376c0901cfb5860d9134760eeae5422144e5a2a4ba216

data/.gitignore

@@ -1,10 +1,9 @@
-.vimlog
-*.gem
-*~
-.DS_Store
-.bundle
-.idea
-Gemfile.lock
-doc
-pkg/*
-tags
+/.bundle/
+/.yardoc
+/Gemfile.lock
+/_yardoc/
+/coverage/
+/doc/
+/pkg/
+/spec/reports/
+/tmp/

data/.travis.yml

@@ -1,4 +1,5 @@
 language: ruby
+sudo: false
 
 addons:
   postgresql: '9.3'

data/CHANGELOG.md

@@ -1,3 +1,8 @@
+### 0.6.6
+- Bump Rails dependency to include all 4.2.x (@MrJaba)
+- Fixed migration name to be .rb file (@MrJaba)
+
+
 ### 0.6.5
 
 - Introduce changelog.

data/README.md

@@ -1,6 +1,6 @@
 # pg_audit_log
 
-[![Build Status](https://secure.travis-ci.org/Casecommons/pg_audit_log.png?branch=master)](https://travis-ci.org/Casecommons/pg_audit_log)
+[![Build Status](https://secure.travis-ci.org/Casecommons/pg_audit_log.svg?branch=master)](https://travis-ci.org/Casecommons/pg_audit_log)
 [![Code Climate](https://img.shields.io/codeclimate/github/Casecommons/pg_audit_log.svg?style=flat)](https://codeclimate.com/github/Casecommons/pg_audit_log)
 [![Gem Version](https://img.shields.io/gem/v/pg_audit_log.svg?style=flat)](https://rubygems.org/gems/pg_audit_log)
 

data/lib/generators/pg_audit_log/install_generator.rb

@@ -11,7 +11,7 @@ module PgAuditLog
 
       def install
         directory "lib/tasks"
-        migration_template "migration.rb", "db/migrate/install_pg_audit_log"
+        migration_template "migration.rb", "db/migrate/install_pg_audit_log.rb"
       end
     end
   end

data/lib/pg_audit_log/version.rb

@@ -1,3 +1,3 @@
 module PgAuditLog
-  VERSION = '0.6.5'.freeze
+  VERSION = '0.6.6'.freeze
 end

data/pg_audit_log.gemspec

@@ -19,9 +19,9 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ['lib']
 
-  spec.add_dependency 'rails', '>= 3.2', '<= 4.2'
+  spec.add_dependency 'rails', '>= 3.2', '< 4.3'
   spec.add_dependency 'pg', '>= 0.9.0'
-  spec.add_development_dependency 'rspec', '2.14.1'
+  spec.add_development_dependency 'rspec', '~> 3.0'
   spec.add_development_dependency 'rspec-rails'
   spec.add_development_dependency 'with_model', '>= 0.1.3'
 end

data/spec/configuration_spec.rb

@@ -6,6 +6,6 @@ describe "the PostgreSQL database" do
   end
 
   it "has an audit log table" do
-    ActiveRecord::Base.connection.table_exists?('audit_log').should be_true
+    expect(ActiveRecord::Base.connection.table_exists?('audit_log')).to be_truthy
   end
 end

data/spec/function_spec.rb

@@ -8,7 +8,7 @@ describe PgAuditLog::Function do
         PgAuditLog::Function.uninstall
       end
 
-      it { should be_false }
+      it { is_expected.to be_falsey }
     end
 
     context "when it is" do
@@ -16,7 +16,7 @@ describe PgAuditLog::Function do
         PgAuditLog::Function.install
       end
 
-      it { should be_true }
+      it { is_expected.to be_truthy }
     end
   end
 
@@ -26,27 +26,27 @@ describe PgAuditLog::Function do
 
     context "new style" do
       it "escapes the email" do
-        subject.should_not match('SET')
+        expect(subject).not_to match('SET')
 
-        subject.should match('FUNCTION')
-        subject.should match("'o''connell@fred.com'::varchar")
+        expect(subject).to match('FUNCTION')
+        expect(subject).to match("'o''connell@fred.com'::varchar")
       end
     end
 
     context "old style" do
       before do
         Rails = double
-        Rails.stub_chain(:configuration, :pg_audit_log_old_style_user_id).and_return(true)
+        allow(Rails).to receive_message_chain(:configuration, :pg_audit_log_old_style_user_id).and_return(true)
       end
 
       after { Object.send(:remove_const, :Rails) }
 
       it "escapes the email" do
-        subject.should match('SET')
-        subject.should match("'o''connell@fred.com'")
+        expect(subject).to match('SET')
+        expect(subject).to match("'o''connell@fred.com'")
 
-        subject.should_not match('FUNCTION')
-        subject.should_not match("'o''connell@fred.com'::varchar")
+        expect(subject).not_to match('FUNCTION')
+        expect(subject).not_to match("'o''connell@fred.com'::varchar")
       end
     end
   end

data/spec/model_spec.rb

@@ -5,19 +5,19 @@ describe PgAuditLog::Entry do
 
   describe ".delete" do
     it "blows up because deleting audit logs is not allowed" do
-      proc { PgAuditLog::Entry.delete(subject.id) }.should raise_error(PgAuditLog::Entry::CannotDeleteError)
+      expect { PgAuditLog::Entry.delete(subject.id) }.to raise_error(PgAuditLog::Entry::CannotDeleteError)
     end
   end
 
   describe ".delete_all" do
     it "blows up because deleting audit logs is not allowed" do
-      proc { PgAuditLog::Entry.delete_all }.should raise_error(PgAuditLog::Entry::CannotDeleteError)
+      expect { PgAuditLog::Entry.delete_all }.to raise_error(PgAuditLog::Entry::CannotDeleteError)
     end
   end
 
   describe "#destroy" do
     it "blows up because deleting audit logs is not allowed" do
-      proc { subject.destroy }.should raise_error(PgAuditLog::Entry::CannotDeleteError)
+      expect { subject.destroy }.to raise_error(PgAuditLog::Entry::CannotDeleteError)
     end
   end
 end

data/spec/pg_audit_log_spec.rb

@@ -42,12 +42,32 @@ describe PgAuditLog do
             AuditedModel.create!(attributes)
           end
 
-          it { should be }
-          its(:occurred_at) { should be }
-          its(:table_name) { should == AuditedModel.table_name }
-          its(:field_name) { should == 'str' }
-          its(:primary_key) { should == AuditedModel.last.id.to_s }
-          its(:operation) { should == 'INSERT' }
+          it { is_expected.to be }
+
+          describe '#occurred_at' do
+            subject { super().occurred_at }
+            it { is_expected.to be }
+          end
+
+          describe '#table_name' do
+            subject { super().table_name }
+            it { is_expected.to eq(AuditedModel.table_name) }
+          end
+
+          describe '#field_name' do
+            subject { super().field_name }
+            it { is_expected.to eq('str') }
+          end
+
+          describe '#primary_key' do
+            subject { super().primary_key }
+            it { is_expected.to eq(AuditedModel.last.id.to_s) }
+          end
+
+          describe '#operation' do
+            subject { super().operation }
+            it { is_expected.to eq('INSERT') }
+          end
         end
 
         context "when a user is present, having just been changed" do
@@ -57,15 +77,29 @@ describe PgAuditLog do
             ActiveRecord::Persistence.instance_method(:save).bind(record).call # call save without transaction
           end
 
-          its(:user_id) { should == 1 }
-          its(:user_unique_name) { should == 'my current user' }
+          describe '#user_id' do
+            subject { super().user_id }
+            it { is_expected.to eq(1) }
+          end
+
+          describe '#user_unique_name' do
+            subject { super().user_unique_name }
+            it { is_expected.to eq('my current user') }
+          end
         end
 
         context "when no user is present" do
           before { AuditedModel.create!(attributes) }
 
-          its(:user_id) { should == -1 }
-          its(:user_unique_name) { should == 'UNKNOWN' }
+          describe '#user_id' do
+            subject { super().user_id }
+            it { is_expected.to eq(-1) }
+          end
+
+          describe '#user_unique_name' do
+            subject { super().user_unique_name }
+            it { is_expected.to eq('UNKNOWN') }
+          end
         end
 
         it "captures all new values for all fields" do
@@ -74,11 +108,11 @@ describe PgAuditLog do
           attributes.each do |field_name, value|
             entry = PgAuditLog::Entry.where(:field_name => field_name).last
             if field_name == :dt
-              entry.field_value_new.should == value.strftime("%Y-%m-%d %H:%M:%S")
+              expect(entry.field_value_new).to eq(value.strftime("%Y-%m-%d %H:%M:%S"))
             else
-              entry.field_value_new.should == value.to_s
+              expect(entry.field_value_new).to eq(value.to_s)
             end
-            entry.field_value_old.should be_nil
+            expect(entry.field_value_old).to be_nil
           end
         end
       end
@@ -90,9 +124,17 @@ describe PgAuditLog do
 
         subject { PgAuditLog::Entry.where(:field_name => 'str').last }
 
-        it { should be }
-        its(:field_name) { should == 'str' }
-        its(:primary_key) { should be_nil }
+        it { is_expected.to be }
+
+        describe '#field_name' do
+          subject { super().field_name }
+          it { is_expected.to eq('str') }
+        end
+
+        describe '#primary_key' do
+          subject { super().primary_key }
+          it { is_expected.to be_nil }
+        end
       end
     end
 
@@ -111,17 +153,35 @@ describe PgAuditLog do
             ActiveRecord::Persistence.instance_method(:save).bind(@model).call # call save without transaction
           end
 
-          its(:user_id) { should == 1 }
-          its(:user_unique_name) { should == 'my current user' }
+          describe '#user_id' do
+            subject { super().user_id }
+            it { is_expected.to eq(1) }
+          end
+
+          describe '#user_unique_name' do
+            subject { super().user_unique_name }
+            it { is_expected.to eq('my current user') }
+          end
         end
 
         context "when going from a value to a another value" do
           before { @model.update_attributes!(:str => 'bar') }
           subject { PgAuditLog::Entry.where(:field_name => 'str').last }
 
-          its(:operation) { should == 'UPDATE' }
-          its(:field_value_new) { should == 'bar' }
-          its(:field_value_old) { should == 'foo' }
+          describe '#operation' do
+            subject { super().operation }
+            it { is_expected.to eq('UPDATE') }
+          end
+
+          describe '#field_value_new' do
+            subject { super().field_value_new }
+            it { is_expected.to eq('bar') }
+          end
+
+          describe '#field_value_old' do
+            subject { super().field_value_old }
+            it { is_expected.to eq('foo') }
+          end
         end
 
         context "when going from nil to a value" do
@@ -129,23 +189,37 @@ describe PgAuditLog do
           before { @model.update_attributes!(:txt => 'baz') }
           subject { PgAuditLog::Entry.where(:field_name => 'txt').last }
 
-          its(:field_value_new) { should == 'baz' }
-          its(:field_value_old) { should be_nil }
+          describe '#field_value_new' do
+            subject { super().field_value_new }
+            it { is_expected.to eq('baz') }
+          end
+
+          describe '#field_value_old' do
+            subject { super().field_value_old }
+            it { is_expected.to be_nil }
+          end
         end
 
         context "when going from a value to nil" do
           before { @model.update_attributes!(:str => nil) }
           subject { PgAuditLog::Entry.where(:field_name => 'str').last }
 
-          its(:field_value_new) { should be_nil }
-          its(:field_value_old) { should == 'foo' }
+          describe '#field_value_new' do
+            subject { super().field_value_new }
+            it { is_expected.to be_nil }
+          end
+
+          describe '#field_value_old' do
+            subject { super().field_value_old }
+            it { is_expected.to eq('foo') }
+          end
         end
 
         context "when the value does not change" do
           before { @model.update_attributes!(:str => 'foo') }
           subject { PgAuditLog::Entry.where(:field_name => 'str', :operation => 'UPDATE').last }
 
-          it { should_not be }
+          it { is_expected.not_to be }
         end
 
         context "when the value is nil and does not change" do
@@ -153,7 +227,7 @@ describe PgAuditLog do
           before { @model.update_attributes!(:txt => nil) }
           subject { PgAuditLog::Entry.where(:field_name => 'txt', :operation => 'UPDATE').last }
 
-          it { should_not be }
+          it { is_expected.not_to be }
         end
 
         context "when the value is a boolean" do
@@ -161,8 +235,15 @@ describe PgAuditLog do
             before { @model.update_attributes!(:bool => true) }
             subject { PgAuditLog::Entry.where(:field_name => 'bool', :operation => 'UPDATE').last }
 
-            its(:field_value_new) { should == 'true' }
-            its(:field_value_old) { should be_nil }
+            describe '#field_value_new' do
+              subject { super().field_value_new }
+              it { is_expected.to eq('true') }
+            end
+
+            describe '#field_value_old' do
+              subject { super().field_value_old }
+              it { is_expected.to be_nil }
+            end
           end
 
           context "going from false -> true" do
@@ -172,8 +253,15 @@ describe PgAuditLog do
             end
             subject { PgAuditLog::Entry.where(:field_name => 'bool', :operation => 'UPDATE').last }
 
-            its(:field_value_new) { should == 'true' }
-            its(:field_value_old) { should == 'false' }
+            describe '#field_value_new' do
+              subject { super().field_value_new }
+              it { is_expected.to eq('true') }
+            end
+
+            describe '#field_value_old' do
+              subject { super().field_value_old }
+              it { is_expected.to eq('false') }
+            end
           end
 
           context "going from true -> false" do
@@ -184,8 +272,15 @@ describe PgAuditLog do
             end
             subject { PgAuditLog::Entry.where(:field_name => 'bool', :operation => 'UPDATE').last }
 
-            its(:field_value_new) { should == 'false' }
-            its(:field_value_old) { should == 'true' }
+            describe '#field_value_new' do
+              subject { super().field_value_new }
+              it { is_expected.to eq('false') }
+            end
+
+            describe '#field_value_old' do
+              subject { super().field_value_old }
+              it { is_expected.to eq('true') }
+            end
           end
         end
       end
@@ -198,7 +293,10 @@ describe PgAuditLog do
 
         subject { PgAuditLog::Entry.where(:field_name => 'str').last }
 
-        its(:primary_key) { should be_nil }
+        describe '#primary_key' do
+          subject { super().primary_key }
+          it { is_expected.to be_nil }
+        end
       end
     end
 
@@ -211,17 +309,20 @@ describe PgAuditLog do
 
         subject { PgAuditLog::Entry.where(:field_name => 'str').last }
 
-        its(:operation) { should == 'DELETE' }
+        describe '#operation' do
+          subject { super().operation }
+          it { is_expected.to eq('DELETE') }
+        end
 
         it "captures all new values for all fields" do
           attributes.each do |field_name, value|
             entry = PgAuditLog::Entry.where(:field_name => field_name).last
             if field_name == :dt
-              entry.field_value_old.should == value.strftime('%Y-%m-%d %H:%M:%S')
+              expect(entry.field_value_old).to eq(value.strftime('%Y-%m-%d %H:%M:%S'))
             else
-              entry.field_value_old.should == value.to_s
+              expect(entry.field_value_old).to eq(value.to_s)
             end
-            entry.field_value_new.should be_nil
+            expect(entry.field_value_new).to be_nil
           end
         end
       end
@@ -241,7 +342,10 @@ describe PgAuditLog do
 
         subject { PgAuditLog::Entry.where(:field_name => 'str').last }
 
-        its(:primary_key) { should be_nil }
+        describe '#primary_key' do
+          subject { super().primary_key }
+          it { is_expected.to be_nil }
+        end
       end
     end
 
@@ -271,9 +375,9 @@ describe PgAuditLog do
 
     describe "when creating the table" do
       it "should automatically create the trigger" do
-        PgAuditLog::Triggers.tables_with_triggers.should_not include('test_table')
+        expect(PgAuditLog::Triggers.tables_with_triggers).not_to include('test_table')
         connection.create_table('test_table')
-        PgAuditLog::Triggers.tables_with_triggers.should include('test_table')
+        expect(PgAuditLog::Triggers.tables_with_triggers).to include('test_table')
       end
     end
 
@@ -281,7 +385,7 @@ describe PgAuditLog do
       it "should automatically drop the trigger" do
         connection.create_table('test_table')
         connection.drop_table('test_table')
-        PgAuditLog::Triggers.tables_with_triggers.should_not include('test_table')
+        expect(PgAuditLog::Triggers.tables_with_triggers).not_to include('test_table')
       end
     end
 
@@ -299,9 +403,9 @@ describe PgAuditLog do
         connection.create_table('test_table')
         connection.rename_table('test_table', new_table_name)
 
-        trigger_names.should_not include('audit_test_table')
-        trigger_names.should include("audit_#{new_table_name}")
-        PgAuditLog::Triggers.tables_with_triggers.should include(new_table_name)
+        expect(trigger_names).not_to include('audit_test_table')
+        expect(trigger_names).to include("audit_#{new_table_name}")
+        expect(PgAuditLog::Triggers.tables_with_triggers).to include(new_table_name)
 
         connection.drop_table(new_table_name) rescue nil
       end
@@ -313,9 +417,9 @@ describe PgAuditLog do
           connection.create_table('ignored_table')
           connection.rename_table('ignored_table', new_table_name)
 
-          trigger_names.should_not include('audit_ignored_table')
-          trigger_names.should_not include("audit_#{new_table_name}")
-          PgAuditLog::Triggers.tables_with_triggers.should_not include(new_table_name)
+          expect(trigger_names).not_to include('audit_ignored_table')
+          expect(trigger_names).not_to include("audit_#{new_table_name}")
+          expect(PgAuditLog::Triggers.tables_with_triggers).not_to include(new_table_name)
 
           connection.drop_table(new_table_name) rescue nil
         end
@@ -327,7 +431,7 @@ describe PgAuditLog do
     context "when creating them" do
       it "should be ignored" do
         connection.create_table('some_temp_table', :temporary => true)
-        PgAuditLog::Triggers.tables_with_triggers.should_not include('some_temp_table')
+        expect(PgAuditLog::Triggers.tables_with_triggers).not_to include('some_temp_table')
         connection.drop_table('some_temp_table')
       end
     end
@@ -336,7 +440,7 @@ describe PgAuditLog do
       it "should be ignored" do
         connection.create_table('some_temp_table', :temporary => true)
         connection.drop_table('some_temp_table')
-        PgAuditLog::Triggers.tables_with_triggers.should_not include('some_temp_table')
+        expect(PgAuditLog::Triggers.tables_with_triggers).not_to include('some_temp_table')
       end
     end
   end
@@ -349,7 +453,7 @@ describe PgAuditLog do
     context "when creating a table" do
       it "should install the function then enable the trigger on the table" do
         connection.create_table('some_more_new_table')
-        PgAuditLog::Triggers.tables_with_triggers.should include('some_more_new_table')
+        expect(PgAuditLog::Triggers.tables_with_triggers).to include('some_more_new_table')
         connection.drop_table('some_more_new_table')
       end
     end
@@ -362,9 +466,9 @@ describe PgAuditLog do
 
     context "when creating a table" do
       it "should install the entry table then enable the trigger on the table" do
-        PgAuditLog::Entry.installed?.should be_false
+        expect(PgAuditLog::Entry.installed?).to be_falsey
         connection.create_table('another_table')
-        PgAuditLog::Entry.installed?.should be_true
+        expect(PgAuditLog::Entry.installed?).to be_truthy
         connection.drop_table('another_table')
       end
     end
@@ -375,7 +479,7 @@ describe PgAuditLog do
       it "should not automatically create a trigger for it" do
         PgAuditLog::IGNORED_TABLES << 'ignored_table'
         connection.create_table('ignored_table')
-        PgAuditLog::Triggers.tables_with_triggers.should_not include('ignored_table')
+        expect(PgAuditLog::Triggers.tables_with_triggers).not_to include('ignored_table')
         connection.drop_table('ignored_table')
       end
     end
@@ -384,7 +488,7 @@ describe PgAuditLog do
       it "should not automatically create a trigger for it" do
         PgAuditLog::IGNORED_TABLES << /ignored_table/
         connection.create_table('second_ignored_table')
-        PgAuditLog::Triggers.tables_with_triggers.should_not include('second_ignored_table')
+        expect(PgAuditLog::Triggers.tables_with_triggers).not_to include('second_ignored_table')
         connection.drop_table('second_ignored_table')
       end
     end

data/spec/triggers_spec.rb

@@ -28,23 +28,23 @@ describe PgAuditLog::Triggers do
       PgAuditLog::IGNORED_TABLES << /ignore/
     end
 
-    it { should include(TableWithTriggers.table_name) }
-    it { should include(TableWithoutTriggers.table_name) }
-    it { should_not include(PgAuditLog::Entry.table_name) }
-    it { should_not include(TableToIgnore.table_name) }
+    it { is_expected.to include(TableWithTriggers.table_name) }
+    it { is_expected.to include(TableWithoutTriggers.table_name) }
+    it { is_expected.not_to include(PgAuditLog::Entry.table_name) }
+    it { is_expected.not_to include(TableToIgnore.table_name) }
   end
 
   describe ".tables_with_triggers" do
     it "should return an array of all tables that do have an audit trigger installed" do
-      PgAuditLog::Triggers.tables_with_triggers.should include(TableWithTriggers.table_name)
-      PgAuditLog::Triggers.tables_with_triggers.should_not include(TableWithoutTriggers.table_name)
+      expect(PgAuditLog::Triggers.tables_with_triggers).to include(TableWithTriggers.table_name)
+      expect(PgAuditLog::Triggers.tables_with_triggers).not_to include(TableWithoutTriggers.table_name)
     end
   end
 
   describe ".tables_without_triggers" do
     it "should return an array of all tables that do not have an audit trigger installed" do
-      PgAuditLog::Triggers.tables_without_triggers.should_not include(TableWithTriggers.table_name)
-      PgAuditLog::Triggers.tables_without_triggers.should include(TableWithoutTriggers.table_name)
+      expect(PgAuditLog::Triggers.tables_without_triggers).not_to include(TableWithTriggers.table_name)
+      expect(PgAuditLog::Triggers.tables_without_triggers).to include(TableWithoutTriggers.table_name)
     end
   end
 
@@ -55,17 +55,17 @@ describe PgAuditLog::Triggers do
 
     describe ".install" do
       it "should work" do
-        ->{
+        expect{
           PgAuditLog::Triggers.install
-        }.should_not raise_error
+        }.not_to raise_error
       end
     end
 
     describe ".uninstall" do
       it "should work" do
-        ->{
+        expect{
           PgAuditLog::Triggers.uninstall
-        }.should_not raise_error
+        }.not_to raise_error
       end
     end
 
@@ -74,24 +74,24 @@ describe PgAuditLog::Triggers do
   context "when triggers are installed" do
     describe ".install" do
       it "should work" do
-        ->{
+        expect{
           PgAuditLog::Triggers.install
-        }.should_not raise_error
+        }.not_to raise_error
       end
     end
     describe ".uninstall" do
       it "should work" do
-        ->{
+        expect{
           PgAuditLog::Triggers.uninstall
-        }.should_not raise_error
+        }.not_to raise_error
       end
     end
 
     describe ".enable" do
       it "should not blow up" do
-        ->{
+        expect{
           PgAuditLog::Triggers.enable
-        }.should_not raise_error
+        }.not_to raise_error
       end
 
       it "should fire the audit" do
@@ -104,9 +104,9 @@ describe PgAuditLog::Triggers do
 
     describe ".disable" do
       it "should not blow up" do
-        ->{
+        expect{
           PgAuditLog::Triggers.disable
-        }.should_not raise_error
+        }.not_to raise_error
       end
 
       it "should not fire the audit" do
@@ -130,17 +130,17 @@ describe PgAuditLog::Triggers do
         expect {
           TableWithTriggers.create!
         }.to change(PgAuditLog::Entry, :count)
-        PgAuditLog::Entry.last.user_id.should == -1
+        expect(PgAuditLog::Entry.last.user_id).to eq(-1)
       end
     end
 
     describe ".create_for_table" do
       context "for a table that already has a trigger" do
         it "should not blow up" do
-          PgAuditLog::Triggers.tables_with_triggers.should include(TableWithTriggers.table_name)
-          ->{
+          expect(PgAuditLog::Triggers.tables_with_triggers).to include(TableWithTriggers.table_name)
+          expect{
             PgAuditLog::Triggers.create_for_table(TableWithTriggers.table_name)
-          }.should_not raise_error
+          }.not_to raise_error
         end
       end
     end
@@ -149,9 +149,9 @@ describe PgAuditLog::Triggers do
       context "for a table that has no trigger" do
         it "should not blow up" do
           PgAuditLog::Triggers.drop_for_table(TableWithTriggers.table_name)
-          ->{
+          expect{
             PgAuditLog::Triggers.drop_for_table(TableWithTriggers.table_name)
-          }.should_not raise_error
+          }.not_to raise_error
         end
       end
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pg_audit_log
 version: !ruby/object:Gem::Version
-  version: 0.6.5
+  version: 0.6.6
 platform: ruby
 authors:
 - Case Commons, LLC
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-12-29 00:00:00.000000000 Z
+date: 2016-03-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -17,9 +17,9 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '3.2'
-    - - "<="
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '4.2'
+        version: '4.3'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -27,9 +27,9 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '3.2'
-    - - "<="
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '4.2'
+        version: '4.3'
 - !ruby/object:Gem::Dependency
   name: pg
   requirement: !ruby/object:Gem::Requirement
@@ -48,16 +48,16 @@ dependencies:
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.14.1
+        version: '3.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.14.1
+        version: '3.0'
 - !ruby/object:Gem::Dependency
   name: rspec-rails
   requirement: !ruby/object:Gem::Requirement
@@ -98,7 +98,6 @@ extra_rdoc_files: []
 files:
 - ".gitignore"
 - ".rspec"
-- ".rvmrc"
 - ".travis.yml"
 - CHANGELOG.md
 - Gemfile
@@ -146,7 +145,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.4
+rubygems_version: 2.4.5
 signing_key: 
 specification_version: 4
 summary: PostgreSQL-only database-level audit logging of all databases changes.

data/.rvmrc

@@ -1 +0,0 @@
-rvm 1.9.3-p0@pg_audit_log --create