Fix bug with date format when importing to inventory

This commit is contained in:
Mojca Lorber 2020-03-12 12:19:25 +01:00
parent d924faaafd
commit c716a52aae
3 changed files with 35 additions and 2 deletions

View file

@ -26,7 +26,7 @@ class RepositoryDateTimeValueBase < ApplicationRecord
end end
def self.import_from_text(text, attributes, options = {}) def self.import_from_text(text, attributes, options = {})
date_format = options.dig(:user, :settings, :date_format) || Constants::DEFAULT_DATE_FORMAT date_format = options[:user].settings[:date_format] || Constants::DEFAULT_DATE_FORMAT
new(attributes.merge(data: DateTime.strptime(text, date_format))) new(attributes.merge(data: DateTime.strptime(text, date_format)))
rescue ArgumentError rescue ArgumentError
nil nil

View file

@ -137,7 +137,7 @@ module RepositoryImportParser
cell_value = column.data_type.constantize.import_from_text( cell_value = column.data_type.constantize.import_from_text(
value, value,
cell_value_attributes, cell_value_attributes,
user: @user.as_json user: @user
) )
next if cell_value.nil? next if cell_value.nil?

View file

@ -39,4 +39,37 @@ describe RepositoryDateTimeValueBase, type: :model do
end end
end end
end end
describe 'self.import_from_text' do
let(:user) { create :user }
let(:date_string1) { '2020-01-22' }
let(:date_string2) { '01-22-2020' }
let(:date_string3) { '01/22/2020' }
let(:test_date) { Date.parse('2020-01-22') }
let(:attributes) { {} }
it 'takes date format from user settings 1' do
user.settings[:date_format] = '%Y-%m-%d'
user.save
result = RepositoryDateTimeValueBase.import_from_text(date_string1, attributes, user: user)
expect(result).to respond_to(:data)
expect(result.data).to eq(test_date)
end
it 'takes date format from user settings 2' do
user.settings[:date_format] = '%m-%d-%Y'
user.save
result = RepositoryDateTimeValueBase.import_from_text(date_string2, attributes, user: user)
expect(result).to respond_to(:data)
expect(result.data).to eq(test_date)
end
it 'takes default date format when there is no user settings' do
user.settings[:date_format] = nil
user.save
result = RepositoryDateTimeValueBase.import_from_text(date_string3, attributes, user: user)
expect(result).to respond_to(:data)
expect(result.data).to eq(test_date)
end
end
end end