expect value object

Author: pjurewicz

ruby_event_store-active_record/lib/ruby_event_store/active_record/generators/foreign_key_on_event_id_migration_generator.rb

@@ -3,8 +3,7 @@
 module RubyEventStore
   module ActiveRecord
     class ForeignKeyOnEventIdMigrationGenerator
-      def call(database_adapter_name, migration_path)
-        database_adapter = DatabaseAdapter.from_string(database_adapter_name)
+      def call(database_adapter, migration_path)
         each_migration(database_adapter) do |migration_name|
           path = build_path(migration_path, migration_name)
           write_to_file(path, migration_code(database_adapter, migration_name))

ruby_event_store-active_record/spec/foreign_key_on_event_id_migration_generator_spec.rb

@@ -19,7 +19,8 @@ module ActiveRecord
 
       before { allow(Time).to receive(:now).and_return(Time.new(2022, 11, 30, 21, 37, 00)) }
 
-      %w[mysql2 postgresql sqlite].each do |adapter|
+      [DatabaseAdapter::MySQL, DatabaseAdapter::SQLite, DatabaseAdapter::PostgreSQL].each do |adapter_class|
+        adapter = adapter_class.new
         specify "it is created within specified directory" do
           migration_generator(adapter, @dir)
 
@@ -51,13 +52,14 @@ module ActiveRecord
 
       context "when postgresql adapter is used" do
         specify "should do migration in two steps" do
-          migration_generator('postgresql', @dir)
+          migration_generator(DatabaseAdapter::PostgreSQL.new, @dir)
           expect(second_step_migration_exists?(@dir)).to be_truthy
           expect(generated_files_count(@dir)).to eq(2)
         end
       end
 
-      %w[mysql2 sqlite].each do |adapter|
+      [DatabaseAdapter::MySQL, DatabaseAdapter::SQLite].each do |adapter_class|
+        adapter = adapter_class.new
         context "when #{adapter} adapter is used" do
           specify "should do migration in single step" do
             migration_generator(adapter, @dir)
@@ -66,10 +68,6 @@ module ActiveRecord
         end
       end
 
-      specify "unsupported adapter raises error" do
-        expect { migration_generator('unsupported', @dir) }.to raise_error(UnsupportedAdapter)
-      end
-
       private
 
       def migration_generator(adapter, dir)