published 0.5.15 - fix datetime precision #95

lib/active_record/connection_adapters/clickhouse/oid/date_time.rb

@@ -9,9 +9,8 @@ class DateTime < Type::DateTime # :nodoc:
           def serialize(value)
             value = super
             return unless value
-            return value.strftime('%Y-%m-%d %H:%M:%S') unless value.acts_like?(:time)
 
-            value.to_time.strftime('%Y-%m-%d %H:%M:%S')
+            value.strftime('%Y-%m-%d %H:%M:%S' + (@precision.present? && @precision > 0 ? ".%#{@precision}N" : ''))
           end
 
           def type_cast_from_database(value)

lib/active_record/connection_adapters/clickhouse_adapter.rb

@@ -191,7 +191,7 @@ def initialize_type_map(m) # :nodoc:
         super
         register_class_with_limit m, %r(String), Type::String
         register_class_with_limit m, 'Date',  Clickhouse::OID::Date
-        register_class_with_limit m, 'DateTime',  Clickhouse::OID::DateTime
+        register_class_with_precision m, %r(datetime)i,  Clickhouse::OID::DateTime
 
         register_class_with_limit m, %r(Int8), Type::Integer
         register_class_with_limit m, %r(Int16), Type::Integer

lib/clickhouse-activerecord/version.rb

@@ -1,3 +1,3 @@
 module ClickhouseActiverecord
-  VERSION = '0.5.14'
+  VERSION = '0.5.15'
 end

spec/cases/model_spec.rb

@@ -61,6 +61,32 @@
     end
   end
 
+  context 'DateTime64' do
+
+    let!(:model) do
+      Class.new(ActiveRecord::Base) do
+        self.table_name = 'some'
+      end
+    end
+
+    before do
+      migrations_dir = File.join(FIXTURES_PATH, 'migrations', 'dsl_table_with_datetime_creation')
+      quietly { ClickhouseActiverecord::MigrationContext.new(migrations_dir, model.connection.schema_migration).up }
+    end
+
+    describe '#create' do
+      it 'create a new record' do
+        time = DateTime.parse('2023-07-21 08:00:00.123')
+        model.create!(datetime: time, datetime64: time)
+        row = model.first
+        expect(row.datetime).to_not eq(row.datetime64)
+        expect(row.datetime.strftime('%Y-%m-%d %H:%M:%S')).to eq('2023-07-21 08:00:00')
+        expect(row.datetime64.strftime('%Y-%m-%d %H:%M:%S.%3N')).to eq('2023-07-21 08:00:00.123')
+      end
+    end
+
+  end
+
   context 'array' do
 
     let!(:model) do

spec/fixtures/migrations/dsl_table_with_datetime_creation/1_create_some_table.rb

@@ -2,7 +2,7 @@
 
 class CreateSomeTable < ActiveRecord::Migration[5.0]
   def up
-    create_table :some, id: false do |t|
+    create_table :some, id: false, force: true do |t|
       t.datetime :datetime, null: false
       t.datetime :datetime64, precision: 3, null: true
     end