raggedstaff/openfoodnetwork
1
0
Fork 0
mirror of https://github.com/openfoodfoundation/openfoodnetwork synced 2026-03-12 03:50:22 +00:00
Code Issues Packages Projects Releases Wiki Activity

Enabling non-privileged postgres users to run specs

Browse Source 
Addressing issue #245.

A combination of fixtures and foreign key constraints requires the postgres
user to be superuser. Otherwise an attempt to disable constraints fails.
This got fixed in Rails 4 and this patch brings the same behaviour back to
Rails 3. It will allow us to run the specs with a nosuperuser postgres user.

See:
 - https://github.com/matthuhiggins/foreigner/issues/61
 - 9bb27f7ffe
This commit is contained in:
Maikel Linke
2015-04-16 11:41:54 +10:00
parent a6498c2f96
commit b70edd5424
2 changed files with 34 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
spec/spec_helper.rb
View File

@@ -48,6 +48,9 @@ Capybara.default_max_wait_time = 30
require "paperclip/matchers"
#Fix fixtures with foreign keys, fixed in Rails4
require_relative "support/active_record_postgresql_referential_integrity_patch"
# Override setting in Spree engine: Spree::Core::MailSettings
ActionMailer::Base.default_url_options[:host] = 'test.host'

31
spec/support/active_record_postgresql_referential_integrity_patch.rb Normal file
View File

@@ -0,0 +1,31 @@
# A combination of fixtures and foreign key constraints requires the postgres
# user to be superuser. Otherwise an attempt to disable constraints fails.
# This got fixed in Rails 4 and this patch brings the same behaviour back to
# Rails 3. It will allow us to run the specs with a nosuperuser postgres user.
#
# See:
# - https://github.com/matthuhiggins/foreigner/issues/61
# - https://github.com/garysweaver/rails/commit/9bb27f7ffe3eb732df737e477cd8fc25e007f77b
if Rails::VERSION::MAJOR < 4
class ActiveRecord::ConnectionAdapters::PostgreSQLAdapter
def disable_referential_integrity #:nodoc:
if supports_disable_referential_integrity? then
begin
execute(tables.collect { |name| "ALTER TABLE #{quote_table_name(name)} DISABLE TRIGGER ALL" }.join(";"))
rescue
execute(tables.collect { |name| "ALTER TABLE #{quote_table_name(name)} DISABLE TRIGGER USER" }.join(";"))
end
end
yield
ensure
if supports_disable_referential_integrity? then
begin
execute(tables.collect { |name| "ALTER TABLE #{quote_table_name(name)} ENABLE TRIGGER ALL" }.join(";"))
rescue
execute(tables.collect { |name| "ALTER TABLE #{quote_table_name(name)} ENABLE TRIGGER USER" }.join(";"))
end
end
end
end
end