Skip to content

Commit 2083904

Browse files
authored
Fix MySQL backwards compatibility test failure (#5085)
Sequel::Postgres::Dataset is only defined when the postgres adapter is loaded. In regular CI, the case_insensitive_string_monkeypatch requires sequel/adapters/postgres unconditionally, making the constant available even in MySQL runs. The backwards compatibility tests skip migrations (NO_DB_MIGRATION=true), so the monkeypatch is never loaded and the constant is undefined. Use Sequel::Dataset instead, which is always available and sufficient for the mock (it only needs .update).
1 parent 340829f commit 2083904

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

spec/unit/jobs/v3/services/update_broker_job_spec.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -448,7 +448,7 @@ module V3
448448

449449
context 'when database disconnects during state rollback' do
450450
let(:catalog_error) { StandardError.new('Catalog fetch failed') }
451-
let(:mock_dataset) { instance_double(Sequel::Postgres::Dataset) }
451+
let(:mock_dataset) { instance_double(Sequel::Dataset) }
452452

453453
before do
454454
allow_any_instance_of(VCAP::CloudController::V3::ServiceBrokerCatalogUpdater).to receive(:refresh).and_raise(catalog_error)

0 commit comments

Comments
 (0)