Some checks failed
Lock Threads / action (push) Has been cancelled
Mark stale issues and pull requests / stale (push) Has been cancelled
Publish Chatwoot EE docker images / build (linux/amd64, ubuntu-latest) (push) Has been cancelled
Publish Chatwoot EE docker images / build (linux/arm64, ubuntu-22.04-arm) (push) Has been cancelled
Publish Chatwoot EE docker images / merge (push) Has been cancelled
Publish Chatwoot CE docker images / build (linux/amd64, ubuntu-latest) (push) Has been cancelled
Publish Chatwoot CE docker images / build (linux/arm64, ubuntu-22.04-arm) (push) Has been cancelled
Publish Chatwoot CE docker images / merge (push) Has been cancelled
Run Chatwoot CE spec / lint-backend (push) Has been cancelled
Run Chatwoot CE spec / lint-frontend (push) Has been cancelled
Run Chatwoot CE spec / frontend-tests (push) Has been cancelled
Run Chatwoot CE spec / backend-tests (0, 16) (push) Has been cancelled
Run Chatwoot CE spec / backend-tests (1, 16) (push) Has been cancelled
Run Chatwoot CE spec / backend-tests (10, 16) (push) Has been cancelled
Run Chatwoot CE spec / backend-tests (11, 16) (push) Has been cancelled
Run Chatwoot CE spec / backend-tests (12, 16) (push) Has been cancelled
Run Chatwoot CE spec / backend-tests (13, 16) (push) Has been cancelled
Run Chatwoot CE spec / backend-tests (14, 16) (push) Has been cancelled
Run Chatwoot CE spec / backend-tests (15, 16) (push) Has been cancelled
Run Chatwoot CE spec / backend-tests (2, 16) (push) Has been cancelled
Run Chatwoot CE spec / backend-tests (3, 16) (push) Has been cancelled
Run Chatwoot CE spec / backend-tests (4, 16) (push) Has been cancelled
Run Chatwoot CE spec / backend-tests (5, 16) (push) Has been cancelled
Run Chatwoot CE spec / backend-tests (6, 16) (push) Has been cancelled
Run Chatwoot CE spec / backend-tests (7, 16) (push) Has been cancelled
Run Chatwoot CE spec / backend-tests (8, 16) (push) Has been cancelled
Run Chatwoot CE spec / backend-tests (9, 16) (push) Has been cancelled
Run Linux nightly installer / nightly (push) Has been cancelled
- Add Logistics component with progress tracking - Add OrderDetail component for order information - Support data-driven steps and actions - Add blue color scale to widget SCSS - Fix node overflow and progress bar rendering issues - Add English translations for dashboard components Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
120 lines
4.9 KiB
Ruby
120 lines
4.9 KiB
Ruby
require 'rails_helper'
|
|
|
|
RSpec.describe Avatar::AvatarFromUrlJob do
|
|
let(:file) { fixture_file_upload(Rails.root.join('spec/assets/avatar.png'), 'image/png') }
|
|
let(:valid_url) { 'https://example.com/avatar.png' }
|
|
|
|
it 'enqueues the job' do
|
|
contact = create(:contact)
|
|
expect { described_class.perform_later(contact, 'https://example.com/avatar.png') }
|
|
.to have_enqueued_job(described_class).on_queue('purgable')
|
|
end
|
|
|
|
context 'with rate-limited avatarable (Contact)' do
|
|
let(:avatarable) { create(:contact) }
|
|
|
|
it 'attaches and updates sync attributes' do
|
|
expect(Down).to receive(:download).with(valid_url, max_size: Avatar::AvatarFromUrlJob::MAX_DOWNLOAD_SIZE).and_return(file)
|
|
described_class.perform_now(avatarable, valid_url)
|
|
avatarable.reload
|
|
expect(avatarable.avatar).to be_attached
|
|
expect(avatarable.additional_attributes['avatar_url_hash']).to eq(Digest::SHA256.hexdigest(valid_url))
|
|
expect(avatarable.additional_attributes['last_avatar_sync_at']).to be_present
|
|
end
|
|
|
|
it 'returns early when rate limited' do
|
|
ts = 30.seconds.ago.iso8601
|
|
avatarable.update(additional_attributes: { 'last_avatar_sync_at' => ts })
|
|
expect(Down).not_to receive(:download)
|
|
described_class.perform_now(avatarable, valid_url)
|
|
avatarable.reload
|
|
expect(avatarable.avatar).not_to be_attached
|
|
expect(avatarable.additional_attributes['last_avatar_sync_at']).to be_present
|
|
expect(Time.zone.parse(avatarable.additional_attributes['last_avatar_sync_at']))
|
|
.to be > Time.zone.parse(ts)
|
|
expect(avatarable.additional_attributes['avatar_url_hash']).to eq(Digest::SHA256.hexdigest(valid_url))
|
|
end
|
|
|
|
it 'returns early when hash unchanged' do
|
|
avatarable.update(additional_attributes: { 'avatar_url_hash' => Digest::SHA256.hexdigest(valid_url) })
|
|
expect(Down).not_to receive(:download)
|
|
described_class.perform_now(avatarable, valid_url)
|
|
expect(avatarable.avatar).not_to be_attached
|
|
avatarable.reload
|
|
expect(avatarable.additional_attributes['last_avatar_sync_at']).to be_present
|
|
expect(avatarable.additional_attributes['avatar_url_hash']).to eq(Digest::SHA256.hexdigest(valid_url))
|
|
end
|
|
|
|
it 'updates sync attributes even when URL is invalid' do
|
|
invalid_url = 'invalid_url'
|
|
expect(Down).not_to receive(:download)
|
|
described_class.perform_now(avatarable, invalid_url)
|
|
avatarable.reload
|
|
expect(avatarable.avatar).not_to be_attached
|
|
expect(avatarable.additional_attributes['last_avatar_sync_at']).to be_present
|
|
expect(avatarable.additional_attributes['avatar_url_hash']).to eq(Digest::SHA256.hexdigest(invalid_url))
|
|
end
|
|
|
|
it 'updates sync attributes when file download is valid but content type is unsupported' do
|
|
temp_file = Tempfile.new(['invalid', '.xml'])
|
|
temp_file.write('<invalid>content</invalid>')
|
|
temp_file.rewind
|
|
|
|
uploaded = ActionDispatch::Http::UploadedFile.new(
|
|
tempfile: temp_file,
|
|
filename: 'invalid.xml',
|
|
type: 'application/xml'
|
|
)
|
|
|
|
expect(Down).to receive(:download).with(valid_url, max_size: Avatar::AvatarFromUrlJob::MAX_DOWNLOAD_SIZE).and_return(uploaded)
|
|
|
|
described_class.perform_now(avatarable, valid_url)
|
|
avatarable.reload
|
|
|
|
expect(avatarable.avatar).not_to be_attached
|
|
expect(avatarable.additional_attributes['last_avatar_sync_at']).to be_present
|
|
expect(avatarable.additional_attributes['avatar_url_hash']).to eq(Digest::SHA256.hexdigest(valid_url))
|
|
|
|
temp_file.close
|
|
temp_file.unlink
|
|
end
|
|
end
|
|
|
|
context 'with regular avatarable' do
|
|
let(:avatarable) { create(:agent_bot) }
|
|
|
|
it 'downloads and attaches avatar' do
|
|
expect(Down).to receive(:download).with(valid_url, max_size: Avatar::AvatarFromUrlJob::MAX_DOWNLOAD_SIZE).and_return(file)
|
|
described_class.perform_now(avatarable, valid_url)
|
|
expect(avatarable.avatar).to be_attached
|
|
end
|
|
end
|
|
|
|
# ref: https://github.com/chatwoot/chatwoot/issues/10449
|
|
it 'does not raise error when downloaded file has no filename (invalid content)' do
|
|
contact = create(:contact)
|
|
temp_file = Tempfile.new(['invalid', '.xml'])
|
|
temp_file.write('<invalid>content</invalid>')
|
|
temp_file.rewind
|
|
|
|
expect(Down).to receive(:download).with(valid_url, max_size: Avatar::AvatarFromUrlJob::MAX_DOWNLOAD_SIZE)
|
|
.and_return(ActionDispatch::Http::UploadedFile.new(tempfile: temp_file, type: 'application/xml'))
|
|
|
|
expect { described_class.perform_now(contact, valid_url) }.not_to raise_error
|
|
|
|
temp_file.close
|
|
temp_file.unlink
|
|
end
|
|
|
|
it 'skips sync attribute updates when URL is nil' do
|
|
contact = create(:contact)
|
|
expect(Down).not_to receive(:download)
|
|
|
|
expect { described_class.perform_now(contact, nil) }.not_to raise_error
|
|
|
|
contact.reload
|
|
expect(contact.additional_attributes['last_avatar_sync_at']).to be_nil
|
|
expect(contact.additional_attributes['avatar_url_hash']).to be_nil
|
|
end
|
|
end
|