fix: empty account variable for custom_attrbute_definition running in background job (#7351)
This commit is contained in:
@@ -2,7 +2,7 @@ class Conversations::FilterService < FilterService
|
|||||||
ATTRIBUTE_MODEL = 'conversation_attribute'.freeze
|
ATTRIBUTE_MODEL = 'conversation_attribute'.freeze
|
||||||
|
|
||||||
def initialize(params, user, filter_account = nil)
|
def initialize(params, user, filter_account = nil)
|
||||||
@filter_account = filter_account
|
@account = filter_account || Current.account
|
||||||
super(params, user)
|
super(params, user)
|
||||||
end
|
end
|
||||||
|
|
||||||
@@ -54,8 +54,7 @@ class Conversations::FilterService < FilterService
|
|||||||
end
|
end
|
||||||
|
|
||||||
def base_relation
|
def base_relation
|
||||||
account = @filter_account || Current.account
|
@account.conversations.left_outer_joins(:labels)
|
||||||
account.conversations.left_outer_joins(:labels)
|
|
||||||
end
|
end
|
||||||
|
|
||||||
def current_page
|
def current_page
|
||||||
|
|||||||
@@ -370,4 +370,31 @@ describe Conversations::FilterService do
|
|||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
describe '#perform on date filter with no current account' do
|
||||||
|
before do
|
||||||
|
Current.account = nil
|
||||||
|
end
|
||||||
|
|
||||||
|
context 'with query present' do
|
||||||
|
let!(:params) { { payload: [], page: 1 } }
|
||||||
|
|
||||||
|
it 'filter by created_at' do
|
||||||
|
params[:payload] = [
|
||||||
|
{
|
||||||
|
attribute_key: 'created_at',
|
||||||
|
filter_operator: 'is_greater_than',
|
||||||
|
values: ['2022-01-20'],
|
||||||
|
query_operator: nil,
|
||||||
|
custom_attribute_type: ''
|
||||||
|
}.with_indifferent_access
|
||||||
|
]
|
||||||
|
result = filter_service.new(params, user_1, account).perform
|
||||||
|
expected_count = Conversation.where('created_at > ?', DateTime.parse('2022-01-20')).count
|
||||||
|
|
||||||
|
expect(Current.account).to be_nil
|
||||||
|
expect(result[:conversations].length).to be expected_count
|
||||||
|
end
|
||||||
|
end
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|||||||
Reference in New Issue
Block a user