hairtrigger 0.2.21 → 0.2.25

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: f20c9f46678e0f212a8dddf861f0872609c4a519
-  data.tar.gz: 3dd34ed254df8230ff6ffd32020be89d9b63c386
+SHA256:
+  metadata.gz: 0537ad012af8bad34a62b81241c8517ac38fabeae279954e3e27ecc86e89cc9a
+  data.tar.gz: fe75190bc7fb5b8d2956929f5e72d2b48b0dbc81b5bdd27eac6fd48ccedf81cb
 SHA512:
-  metadata.gz: a8db4cbe050c37d6aed248679e72770767e4b1b05812f8dc704ae2234e02ff5c7a8d38e3f4663ef23642b9c2aa3d39d18a813b73eb3f5ccc25cc24b5019170ca
-  data.tar.gz: 827a7536fcee643bc4652a6cb7b073e13ab182b1b03f69bf80b63deb63f7618c6b564d3a91df5fa48acda4059f3e8c7eed03eb06510c79e4ed45d3aa7ffa6794
+  metadata.gz: f8df4419131442b648d2ed1a8315281e5d289638a228cf112882faacbcc27ebf5ca136474d1bf34d2b84e920187b23ec42365de67d1856112aa2449413571b52
+  data.tar.gz: 97193387286d4ed659612eca3d4d4a3266965f259a95c15494a627e494904d7a48c3b40a705455400aa5fb65bb90219a1485fd5494d8f006b472e8ac690eca6e

data/LICENSE.txt

@@ -1,4 +1,4 @@
-Copyright (c) 2011-2015 Jon Jensen
+Copyright (c) 2011-2022 Jon Jensen
 
 Permission is hereby granted, free of charge, to any person obtaining
 a copy of this software and associated documentation files (the

data/README.md

@@ -1,5 +1,5 @@
 # HairTrigger
-[<img src="https://secure.travis-ci.org/jenseng/hair_trigger.svg?branch=master" />](http://travis-ci.org/jenseng/hair_trigger)
+[<img src="https://github.com/jenseng/hair_trigger/workflows/CI/badge.svg" />](https://github.com/jenseng/hair_trigger/actions?query=workflow%3ACI)
 
 HairTrigger lets you create and manage database triggers in a concise,
 db-agnostic, Rails-y way. You declare triggers right in your models in Ruby,
@@ -7,8 +7,15 @@ and a simple rake task does all the dirty work for you.
 
 ## Installation
 
-If you are using Rails 3 or beyond, just add `gem 'hairtrigger'` to your
-Gemfile. For Rails 2, it's [slightly more involved](RAILS2.md)
+HairTrigger works with Rails 5.0 onwards. Add the following line to your Gemfile:
+
+```ruby
+gem 'hairtrigger'
+```
+
+Then run `bundle install`
+
+For older versions of Rails check the last [0.2 release](https://github.com/jenseng/hair_trigger/tree/v0.2.21)
 
 ## Usage
 
@@ -94,10 +101,10 @@ Only fire the update trigger if at least one of the columns is specified in the
 Permissions/role to check when calling trigger. PostgreSQL supports `:invoker` (default) and `:definer`, MySQL supports `:definer` (default) and arbitrary users (syntax: `'user'@'host'`).
 
 #### timing(timing)
-Required (but may be satisified by `before`/`after`). Possible values are `:before`/`:after`.
+Required (but may be satisfied by `before`/`after`). Possible values are `:before`/`:after`.
 
 #### events(*events)
-Required (but may be satisified by `before`/`after`). Possible values are `:insert`/`:update`/`:delete`/`:truncate`. MySQL/SQLite only support one action per trigger, and don't support `:truncate`.
+Required (but may be satisfied by `before`/`after`). Possible values are `:insert`/`:update`/`:delete`/`:truncate`. MySQL/SQLite only support one action per trigger, and don't support `:truncate`.
 
 #### nowrap(flag = true)
 PostgreSQL-specific option to prevent the trigger action from being wrapped in a `CREATE FUNCTION`. This is useful for executing existing triggers/functions directly, but is not compatible with the `security` setting nor can it be used with pre-9.0 PostgreSQL when supplying a `where` condition.
@@ -233,6 +240,21 @@ As long as you don't delete old migrations and schema.rb prior to running
 If you have deleted all trigger migrations, you can regenerate a new
 baseline for model triggers via `rake db:generate_trigger_migration`.
 
+### Filtering
+
+It is possible to filter which triggers are dumped by setting any of these
+configuration values:
+
+```ruby
+HairTrigger::SchemaDumper::Configuration.ignore_triggers = 'exact_trigger_name'
+HairTrigger::SchemaDumper::Configuration.ignore_tables = [/partial_/, 'exact_table_name']
+HairTrigger::SchemaDumper::Configuration.allow_triggers = [/partial_/, 'exact_trigger_name']
+HairTrigger::SchemaDumper::Configuration.allow_tables = 'exact_table_name'
+```
+
+Each option can accept a single String or Regexp, or a mixed array of both.
+
+
 ## Testing
 
 To stay on top of things, it's strongly recommended that you add a test or
@@ -248,7 +270,7 @@ create.
 
 ## Warnings and Errors
 
-There are a couple classes of errors: declaration errors and generation 
+There are a couple classes of errors: declaration errors and generation
 errors/warnings.
 
 Declaration errors happen if your trigger declaration is obviously wrong, and
@@ -313,10 +335,26 @@ existing trigger if you wish to redefine it.
 * Manual `create_trigger` statements have some gotchas. See the section
   "Manual triggers and :compatibility"
 
+## Contributing
+
+Contributions welcome! I don't write much Ruby these days 😢 (and haven't used this
+gem in years 😬) but am happy to take contributions. If I'm slow to respond, don't
+hesitate to @ me repeatedly, sometimes those github notifications slip through
+the cracks. 😆.
+
+If you want to add a feature/bugfix, you can rely on Github Actions to run the
+tests, but do also run them locally (especially if you are changing supported
+railses/etc). HairTrigger uses [appraisal](https://github.com/thoughtbot/appraisal)
+to manage all that w/ automagical gemfiles. So the tl;dr when testing locally is:
+
+1. make sure you have mysql and postgres installed (homebrew or whatever)
+2. `bundle exec appraisal install` -- get all the dependencies
+3. `bundle exec appraisal rake` -- run the specs every which way
+
 ## Compatibility
 
-* Ruby 1.8.7+
-* Rails 2.3+
+* Ruby 2.3.0+
+* Rails 5.0+
 * PostgreSQL 8.0+
 * MySQL 5.0.10+
 * SQLite 3.3.8+
@@ -325,4 +363,4 @@ existing trigger if you wish to redefine it.
 
 ## Copyright
 
-Copyright (c) 2011-2017 Jon Jensen. See LICENSE.txt for further details.
+Copyright (c) 2011-2022 Jon Jensen. See LICENSE.txt for further details.

data/lib/hair_trigger/builder.rb

@@ -137,22 +137,22 @@ module HairTrigger
       methods.each do |method|
         class_eval <<-METHOD, __FILE__, __LINE__ + 1
           alias #{method}_orig #{method}
-          def #{method}(*args)
+          def #{method}(*args, &block)
             @chained_calls << :#{method}
             if @triggers || @trigger_group
               @errors << ["mysql doesn't support #{method} within a trigger group", :mysql] unless [:name, :where, :all, :of].include?(:#{method})
             end
-            set_#{method}(*args, &(block_given? ? Proc.new : nil))
+            set_#{method}(*args, &(block_given? ? block : nil))
           end
-          def set_#{method}(*args)
+          def set_#{method}(*args, &block)
             if @triggers # i.e. each time we say t.something within a trigger group block
               @chained_calls.pop # the subtrigger will get this, we don't need it
               @chained_calls = @chained_calls.uniq
               @triggers << trigger = clone
-              trigger.#{method}(*args, &(block_given? ? Proc.new : nil))
+              trigger.#{method}(*args, &(block_given? ? block : nil))
             else
-              #{method}_orig(*args)
-              maybe_execute(&Proc.new) if block_given?
+              #{method}_orig(*args, &block)
+              maybe_execute(&block) if block_given?
               self
             end
           end
@@ -335,10 +335,11 @@ module HairTrigger
         block.call(self)
         raise DeclarationError, "trigger group did not define any triggers" if @triggers.empty?
       else
-        @actions = block.call
-        (@actions.is_a?(Hash) ? @actions.values : [@actions]).each do |actions|
-          actions.sub!(/(\s*)\z/, ';\1') if actions && actions !~ /;\s*\z/
-        end
+        @actions =
+          case (actions = block.call)
+          when Hash then actions.map { |key, action| [key, ensure_semicolon(action)] }.to_h
+          else ensure_semicolon(actions)
+          end
       end
       # only the top-most block actually executes
       if !@trigger_group
@@ -350,6 +351,10 @@ module HairTrigger
       self
     end
 
+    def ensure_semicolon(action)
+      action && action !~ /;\s*\z/ ? action.sub(/(\s*)\z/, ';\1') : action
+    end
+
     def validate_names!
       subtriggers = all_triggers(false)
       named_subtriggers = subtriggers.select{ |t| t.options[:name] }

data/lib/hair_trigger/schema_dumper.rb

@@ -1,5 +1,12 @@
 module HairTrigger
   module SchemaDumper
+    module Configuration
+      mattr_accessor :allow_tables
+      mattr_accessor :allow_triggers
+      mattr_accessor :ignore_tables
+      mattr_accessor :ignore_triggers
+    end
+
     module TrailerWithTriggersSupport
       def trailer(stream)
         orig_show_warnings = Builder.show_warnings
@@ -55,7 +62,7 @@ module HairTrigger
           stream.puts "  # no candidate create_trigger statement could be found, creating an adapter-specific one"
         end
         if definition =~ /\n/
-          stream.print "  execute(<<-TRIGGERSQL)\n#{definition.rstrip}\n  TRIGGERSQL\n\n"
+          stream.print "  execute(<<-SQL)\n#{definition.rstrip}\n  SQL\n\n"
         else
           stream.print "  execute(#{definition.inspect})\n\n"
         end
@@ -67,7 +74,7 @@ module HairTrigger
     def normalize_trigger(name, definition, type)
       @adapter_name = @connection.adapter_name.downcase.to_sym
 
-      return definition unless @adapter_name == :postgresql
+      return definition unless @adapter_name == :postgresql || @adapter_name == :postgis
       # because postgres does not preserve the original CREATE TRIGGER/
       # FUNCTION statements, its decompiled reconstruction will not match
       # ours. we work around it by creating our generated trigger/function,
@@ -91,17 +98,42 @@ module HairTrigger
     end
 
     def whitelist_triggers(triggers)
-      triggers.reject do |name, source|
+      triggers = triggers.reject do |name, source|
         ActiveRecord::SchemaDumper.ignore_tables.any? { |ignored_table_name| source =~ /ON\s+#{@connection.quote_table_name(ignored_table_name)}\s/ }
       end
+
+      if Configuration.allow_tables.present?
+        triggers = triggers.select do |name, source|
+          Array(Configuration.allow_tables).any? { |allowed_table_name| source =~ /ON\s+#{@connection.quote_table_name(allowed_table_name)}\s/ }
+        end
+      end
+
+      if Configuration.allow_triggers.present?
+        triggers = triggers.select do |name, source|
+          Array(Configuration.allow_triggers).any? { |allowed_trigger_name| allowed_trigger_name === name } # Triple equals to allow regexps or strings as allowed_trigger_name
+        end
+      end
+
+      if Configuration.ignore_tables.present?
+        triggers = triggers.reject do |name, source|
+          Array(Configuration.ignore_tables).any? { |allowed_table_name| source =~ /ON\s+#{@connection.quote_table_name(allowed_table_name)}\s/ }
+        end
+      end
+
+      if Configuration.ignore_triggers.present?
+        triggers = triggers.reject do |name, source|
+          Array(Configuration.ignore_triggers).any? { |allowed_trigger_name| allowed_trigger_name === name } # Triple equals to allow regexps or strings as allowed_trigger_name
+        end
+      end
+
+      triggers
     end
 
     def self.included(base)
       base.class_eval do
         prepend TrailerWithTriggersSupport
-        class << self
-          attr_accessor :previous_schema
-        end
+
+        class_attribute :previous_schema
       end
     end
   end

data/lib/hair_trigger/version.rb

@@ -1,5 +1,5 @@
 module HairTrigger
-  VERSION = "0.2.21"
+  VERSION = "0.2.25"
 
   def VERSION.<=>(other)
     split(/\./).map(&:to_i) <=> other.split(/\./).map(&:to_i)

data/lib/hair_trigger.rb

@@ -22,7 +22,7 @@ module HairTrigger
     end
 
     def models
-      if defined?(Rails) && Rails::VERSION::MAJOR > 2
+      if defined?(Rails)
         Rails.application.eager_load!
       else
         Dir[model_path + '/*rb'].each do |model|
@@ -35,22 +35,24 @@ module HairTrigger
           end
         end
       end
-      ActiveRecord::VERSION::STRING < "3.0." ?
-        ActiveRecord::Base.send(:subclasses) :
-        ActiveRecord::Base.descendants
+      ActiveRecord::Base.descendants
     end
 
     def migrator
       version = ActiveRecord::VERSION::STRING
-      if version >= "5.2."
+      if version >= "6.0."
+        migrations = ActiveRecord::MigrationContext.new(migration_path, ActiveRecord::SchemaMigration).migrations
+      elsif version >= "5.2."
         migrations = ActiveRecord::MigrationContext.new(migration_path).migrations
-      elsif version < "4.0."
-        migrations = migration_path
       else # version >= "4.0."
         migrations = ActiveRecord::Migrator.migrations(migration_path)
       end
 
-      ActiveRecord::Migrator.new(:up, migrations)
+      if version >= "6.0."
+        ActiveRecord::Migrator.new(:up, migrations, ActiveRecord::SchemaMigration)
+      else
+        ActiveRecord::Migrator.new(:up, migrations)
+      end
     end
 
     def current_migrations(options = {})
@@ -59,8 +61,8 @@ module HairTrigger
         options[:schema_rb_first] = true
         options[:skip_pending_migrations] = true
       end
-  
-      # if we're in a db:schema:dump task (explict or kicked off by db:migrate),
+
+      # if we're in a db:schema:dump task (explicit or kicked off by db:migrate),
       # we evaluate the previous schema.rb (if it exists), and then all applied
       # migrations in order (even ones older than schema.rb). this ensures we
       # handle db:migrate:down scenarios correctly
@@ -76,7 +78,7 @@ module HairTrigger
         triggers = MigrationReader.get_triggers(migration, options)
         migrations << [migration, triggers] unless triggers.empty?
       end
-  
+
       if previous_schema = (options.has_key?(:previous_schema) ? options[:previous_schema] : File.exist?(schema_rb_path) && File.read(schema_rb_path))
         base_triggers = MigrationReader.get_triggers(previous_schema, options)
         unless base_triggers.empty?
@@ -84,9 +86,9 @@ module HairTrigger
           migrations.unshift [OpenStruct.new({:version => version}), base_triggers]
         end
       end
-      
+
       migrations = migrations.sort_by{|(migration, triggers)| migration.version} unless options[:schema_rb_first]
-  
+
       all_builders = []
       migrations.each do |(migration, triggers)|
         triggers.each do |new_trigger|
@@ -97,7 +99,7 @@ module HairTrigger
           all_builders << [migration.name, new_trigger] unless new_trigger.options[:drop]
         end
       end
-  
+
       all_builders
     end
 
@@ -108,27 +110,27 @@ module HairTrigger
     def generate_migration(silent = false)
       begin
         canonical_triggers = current_triggers
-      rescue 
+      rescue
         $stderr.puts $!
         exit 1
       end
-  
+
       migrations = current_migrations
       migration_names = migrations.map(&:first)
       existing_triggers = migrations.map(&:last)
-  
+
       up_drop_triggers = []
       up_create_triggers = []
       down_drop_triggers = []
       down_create_triggers = []
-  
+
       # see which triggers need to be dropped
       existing_triggers.each do |existing|
         next if canonical_triggers.any?{ |t| t.prepared_name == existing.prepared_name }
         up_drop_triggers.concat existing.drop_triggers
         down_create_triggers << existing
       end
-  
+
       # see which triggers need to be added/replaced
       (canonical_triggers - existing_triggers).each do |new_trigger|
         up_create_triggers << new_trigger
@@ -141,29 +143,28 @@ module HairTrigger
           down_create_triggers << existing
         end
       end
-  
+
       return if up_drop_triggers.empty? && up_create_triggers.empty?
 
       migration_name = infer_migration_name(migration_names, up_create_triggers, up_drop_triggers)
       migration_version = infer_migration_version(migration_name)
       file_name = migration_path + '/' + migration_version + "_" + migration_name.underscore + ".rb"
       FileUtils.mkdir_p migration_path
-      prefix = ActiveRecord::VERSION::STRING < "3.1." ? "self." : ""
-      File.open(file_name, "w"){ |f| f.write <<-MIGRATION }
+      File.open(file_name, "w") { |f| f.write <<-RUBY }
 # This migration was auto-generated via `rake db:generate_trigger_migration'.
 # While you can edit this file, any changes you make to the definitions here
 # will be undone by the next auto-generated trigger migration.
 
-class #{migration_name} < ActiveRecord::Migration
-  def #{prefix}up
+class #{migration_name} < ActiveRecord::Migration[#{ActiveRecord::Migration.current_version}]
+  def up
     #{(up_drop_triggers + up_create_triggers).map{ |t| t.to_ruby('    ') }.join("\n\n").lstrip}
   end
 
-  def #{prefix}down
+  def down
     #{(down_drop_triggers + down_create_triggers).map{ |t| t.to_ruby('    ') }.join("\n\n").lstrip}
   end
 end
-      MIGRATION
+      RUBY
       file_name
     end
 
@@ -225,11 +226,9 @@ end
   end
 end
 
-ActiveRecord::Base.send :extend, HairTrigger::Base
-if ActiveRecord::VERSION::STRING < "4.1."
-  ActiveRecord::Migrator.send :extend, HairTrigger::Migrator
-else
+ActiveSupport.on_load(:active_record) do
+  ActiveRecord::Base.send :extend, HairTrigger::Base
   ActiveRecord::Migration.send :include, HairTrigger::Migrator
+  ActiveRecord::ConnectionAdapters::AbstractAdapter.class_eval { include HairTrigger::Adapter }
+  ActiveRecord::SchemaDumper.class_eval { include HairTrigger::SchemaDumper }
 end
-ActiveRecord::ConnectionAdapters::AbstractAdapter.class_eval { include HairTrigger::Adapter }
-ActiveRecord::SchemaDumper.class_eval { include HairTrigger::SchemaDumper }

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: hairtrigger
 version: !ruby/object:Gem::Version
-  version: 0.2.21
+  version: 0.2.25
 platform: ruby
 authors:
 - Jon Jensen
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-08-08 00:00:00.000000000 Z
+date: 2022-01-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -16,14 +16,20 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '2.3'
+        version: '5.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '8'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '2.3'
+        version: '5.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '8'
 - !ruby/object:Gem::Dependency
   name: ruby_parser
   requirement: !ruby/object:Gem::Requirement
@@ -73,22 +79,6 @@ files:
 - lib/hair_trigger/version.rb
 - lib/hairtrigger.rb
 - lib/tasks/hair_trigger.rake
-- spec/adapter_spec.rb
-- spec/builder_spec.rb
-- spec/migrations-3.2/20110331212003_initial_tables.rb
-- spec/migrations-3.2/20110331212631_user_trigger.rb
-- spec/migrations-3.2/20110417185102_manual_user_trigger.rb
-- spec/migrations-pre-3.1/20110331212003_initial_tables.rb
-- spec/migrations-pre-3.1/20110331212631_user_trigger.rb
-- spec/migrations-pre-3.1/20110417185102_manual_user_trigger.rb
-- spec/migrations/20110331212003_initial_tables.rb
-- spec/migrations/20110331212631_user_trigger.rb
-- spec/migrations/20110417185102_manual_user_trigger.rb
-- spec/migrations_spec.rb
-- spec/models/user.rb
-- spec/models/user_group.rb
-- spec/schema_dumper_spec.rb
-- spec/spec_helper.rb
 homepage: http://github.com/jenseng/hair_trigger
 licenses:
 - MIT
@@ -101,15 +91,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 1.8.7
+      version: 2.3.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.5
+      version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.6.13
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: easy database triggers for active record

data/spec/adapter_spec.rb

@@ -1,94 +0,0 @@
-require 'spec_helper'
-
-# for this spec to work, you need to have postgres and mysql installed (in
-# addition to the gems), and you should make sure that you have set up
-# appropriate users and permissions. see database.yml for more info
-
-describe "adapter" do
-  include_context "hairtrigger utils"
-
-  describe ".triggers" do
-    before do
-      reset_tmp(:migration_glob => "*initial_tables*")
-      initialize_db
-      migrate_db
-    end
-
-    shared_examples_for "mysql" do
-      # have to stub SHOW TRIGGERS to get back a '%' host, since GRANTs
-      # and such get a little dicey for testing (local vs travis, etc.)
-      it "matches the generated trigger with a '%' grant" do
-        conn.instance_variable_get(:@config)[:host] = "somehost" # wheeeee!
-        implicit_definer = "'root'@'somehost'"
-        show_triggers_definer = "root@%"
-
-        builder = trigger.on(:users).before(:insert){ "UPDATE foos SET bar = 1" }
-        triggers = builder.generate.select{|t|t !~ /\ADROP/}
-        expect(conn).to receive(:implicit_mysql_definer).and_return(implicit_definer)
-        expect(conn).to receive(:select_rows).with("SHOW TRIGGERS").and_return([
-          ['users_before_insert_row_tr', 'INSERT', 'users', "BEGIN\n    UPDATE foos SET bar = 1;\nEND", 'BEFORE', 'NULL', 'STRICT_ALL_TABLES', show_triggers_definer]
-        ])
-
-        expect(db_triggers).to eq(triggers)
-      end
-
-      it "quotes table names" do
-        conn.execute <<-SQL
-          CREATE TRIGGER foos_tr AFTER DELETE ON users
-          FOR EACH ROW
-          BEGIN
-            UPDATE user_groups SET bob_count = bob_count - 1;
-          END
-        SQL
-
-        expect(conn.triggers["foos_tr"]).to match(/CREATE TRIGGER foos_tr AFTER DELETE ON `users`/)
-      end
-    end
-
-    context "mysql" do
-      let(:adapter) { :mysql }
-      it_behaves_like "mysql"
-    end if ADAPTERS.include? :mysql
-
-    context "mysql2" do
-      let(:adapter) { :mysql2 }
-      it_behaves_like "mysql"
-    end if ADAPTERS.include? :mysql2
-
-    context "postgresql" do
-      let(:adapter) { :postgresql }
-
-      it "quotes table names" do
-        conn.execute <<-SQL
-          CREATE FUNCTION foos_tr()
-          RETURNS TRIGGER AS $$
-          BEGIN
-            UPDATE user_groups SET bob_count = bob_count - 1;
-          END;
-          $$ LANGUAGE plpgsql;
-
-          CREATE TRIGGER foos_tr AFTER DELETE ON users
-          FOR EACH ROW EXECUTE PROCEDURE foos_tr();
-        SQL
-
-        expect(conn.triggers["foos_tr"]).to match(/CREATE TRIGGER foos_tr AFTER DELETE ON "users"/)
-      end
-    end
-
-    context "sqlite3" do
-      let(:adapter) { :sqlite3 }
-
-      it "quotes table names" do
-        conn.execute <<-SQL
-          CREATE TRIGGER foos_tr AFTER DELETE ON users
-          FOR EACH ROW
-          BEGIN
-            UPDATE user_groups SET bob_count = bob_count - 1;
-          END;
-        SQL
-
-        expect(conn.triggers["foos_tr"]).to match(/CREATE TRIGGER foos_tr AFTER DELETE ON "users"/)
-      end
-    end
-  end
-end