Skip to content

Commit

Permalink
TestCase#create_fixtures no longer takes a block
Browse files Browse the repository at this point in the history
  • Loading branch information
byroot committed Apr 17, 2024
1 parent 57cbc2e commit 65cb5f8
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion activerecord/test/cases/fixtures_test.rb
Expand Up @@ -374,7 +374,7 @@ def test_multiple_clean_fixtures
end

def test_create_symbol_fixtures
fixtures = ActiveRecord::FixtureSet.create_fixtures(FIXTURES_ROOT, :collections, collections: Course) { Course.connection }
fixtures = ActiveRecord::FixtureSet.create_fixtures(FIXTURES_ROOT, :collections, collections: Course)

assert Course.find_by_name("Collection"), "course is not in the database"
assert fixtures.detect { |f| f.name == "collections" }, "no fixtures named 'collections' in #{fixtures.map(&:name).inspect}"
Expand Down
4 changes: 2 additions & 2 deletions activerecord/test/cases/multiple_db_test.rb
Expand Up @@ -9,8 +9,8 @@ class MultipleDbTest < ActiveRecord::TestCase
self.use_transactional_tests = false

def setup
@courses = create_fixtures("courses") { Course.retrieve_connection }
@colleges = create_fixtures("colleges") { College.retrieve_connection }
@courses = create_fixtures("courses")
@colleges = create_fixtures("colleges")
@entrants = create_fixtures("entrants")
end

Expand Down
4 changes: 2 additions & 2 deletions activerecord/test/cases/test_case.rb
Expand Up @@ -83,8 +83,8 @@ def check_connection_leaks
end
end

def create_fixtures(*fixture_set_names, &block)
ActiveRecord::FixtureSet.create_fixtures(ActiveRecord::TestCase.fixture_paths, fixture_set_names, fixture_class_names, &block)
def create_fixtures(*fixture_set_names)
ActiveRecord::FixtureSet.create_fixtures(ActiveRecord::TestCase.fixture_paths, fixture_set_names, fixture_class_names)
end

def capture_sql(include_schema: false)
Expand Down

0 comments on commit 65cb5f8

Please sign in to comment.