* feat: add customizable signature position and separator options * fix: correct default value note for signatureSeparator and ensure reactivity * fix: correct watcher boolean conversion and add immediate ui_settings updates - Fix watchers to convert string props to boolean values for reactive refs - Add immediate event handlers for switch changes to update ui_settings in real-time - Ensure proper synchronization between switch states and user.ui_settings Co-Authored-By: cayo@fazer.ai <cayoproliveira@gmail.com> * fix: split signature content and ui_settings updates to resolve persistence bug - Use updateUISettings store action for signature_position and signature_separator - Keep updateProfile for message_signature content only - Fixes FormData serialization issue that corrupted nested ui_settings object - Add diagnostic logging to verify data flow Co-Authored-By: cayo@fazer.ai <cayoproliveira@gmail.com> * clean: remove diagnostic console logging from updateSignature method - Remove temporary console.log statements added for verification - Keep core implementation that splits signature content and ui_settings updates - Keep console.error for proper error handling with eslint-disable comment - Implementation now ready for production use Co-Authored-By: cayo@fazer.ai <cayoproliveira@gmail.com> * fix: updateUISettings call in updateSignature method * chore: move signature application to send-time and add button highlighting (#79) * fix: move signature application from editor manipulation to send-time - Remove addSignature/removeSignature/toggleSignatureInEditor from WootWriter - Remove signature logic from draft handling and canned response insertion - Apply signatures only in getMessagePayload during message sending - Add button highlighting for signature toggle when activated - Prevents signature duplication and persistence in editor content - Fixes signature position toggle bug Co-Authored-By: cayo@fazer.ai <cayoproliveira@gmail.com> * fix: escape signature separator to prevent markdown setext heading interpretation - Escape '--' separator as '\--' in appendSignature to prevent H2 heading creation - Update removeSignature to handle escaped separators correctly - Fixes signature separator being rendered as markdown instead of plain text - Refactor nested ternary to fix ESLint error Co-Authored-By: cayo@fazer.ai <cayoproliveira@gmail.com> * fix: prevent signature separator markdown interpretation in message processing - Add fix_signature_separator_markdown method to escape '--' separators - Update ensure_processed_message_content to fix separators before saving - Prevents signature separators from being interpreted as setext headings - Ensures correct message display in channels and email notifications Co-Authored-By: cayo@fazer.ai <cayoproliveira@gmail.com> * fix: update separator format to use \n--\n instead of escaping - Change separator delimiter from '\--' to '\n--\n' format - Update removeSignature function to handle new separator format correctly - Simplify message processing since separators are already properly formatted - Ensures consistent separator handling across frontend and backend Co-Authored-By: cayo@fazer.ai <cayoproliveira@gmail.com> * fix: update signature delimiter format to include extra new lines * chore: remove comment about signature application logic * refactor: remove unused method and comments related to signature separator markdown processing * chore: simplify slash command detection by using updatedMessage directly * refactor: remove signature logic from draft message handling * refactor: simplify body empty check by removing signature manipulation logic --------- Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> Co-authored-by: cayo@fazer.ai <cayoproliveira@gmail.com> * refactor: extract signature settings logic into a separate method * fix: handle nil ui_settings in signature position and separator methods * fix: update return value of findSignatureInBody to include position information * fix: update signature handling in findSignatureInBody and related methods * fix: adjust delimiter length handling in removeSignature function * test: add cases for appending, removing, and replacing signatures with various separators * test: add cases for signature position and separator handling * test: add cases for updating signature position and separator in ui_settings * fix: correct typo in comment for findSignatureInBody function * refactor: simplify translation function calls in MessageSignature component * chore: refactoring * chore: refactor * feat: switch -> select * chore: refactor and undo changes * chore: refactor and undo changes * chore: refactor * fix: remove old select component usage * chore: remove useless style --------- Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> Co-authored-by: gabrieljablonski <contact@gabrieljablonski.com>
400 lines
14 KiB
Ruby
400 lines
14 KiB
Ruby
require 'rails_helper'
|
|
|
|
RSpec.describe 'Profile API', type: :request do
|
|
let(:account) { create(:account) }
|
|
|
|
describe 'GET /api/v1/profile' do
|
|
context 'when it is an unauthenticated user' do
|
|
it 'returns unauthorized' do
|
|
get '/api/v1/profile'
|
|
|
|
expect(response).to have_http_status(:unauthorized)
|
|
end
|
|
end
|
|
|
|
context 'when it is an authenticated user' do
|
|
let(:agent) { create(:user, account: account, custom_attributes: { test: 'test' }, role: :agent) }
|
|
|
|
it 'returns current user information' do
|
|
get '/api/v1/profile',
|
|
headers: agent.create_new_auth_token,
|
|
as: :json
|
|
|
|
expect(response).to have_http_status(:success)
|
|
json_response = response.parsed_body
|
|
expect(json_response['id']).to eq(agent.id)
|
|
expect(json_response['email']).to eq(agent.email)
|
|
expect(json_response['access_token']).to eq(agent.access_token.token)
|
|
expect(json_response['custom_attributes']['test']).to eq('test')
|
|
expect(json_response['message_signature']).to be_nil
|
|
end
|
|
end
|
|
end
|
|
|
|
describe 'PUT /api/v1/profile' do
|
|
context 'when it is an unauthenticated user' do
|
|
it 'returns unauthorized' do
|
|
put '/api/v1/profile'
|
|
|
|
expect(response).to have_http_status(:unauthorized)
|
|
end
|
|
end
|
|
|
|
context 'when it is an authenticated user' do
|
|
let(:agent) { create(:user, password: 'Test123!', account: account, role: :agent) }
|
|
|
|
it 'updates the name' do
|
|
put '/api/v1/profile',
|
|
params: { profile: { name: 'test' } },
|
|
headers: agent.create_new_auth_token,
|
|
as: :json
|
|
|
|
expect(response).to have_http_status(:success)
|
|
json_response = response.parsed_body
|
|
agent.reload
|
|
expect(json_response['id']).to eq(agent.id)
|
|
expect(json_response['name']).to eq(agent.name)
|
|
expect(agent.name).to eq('test')
|
|
end
|
|
|
|
it 'updates custom attributes' do
|
|
put '/api/v1/profile',
|
|
params: { profile: { phone_number: '+123456789' } },
|
|
headers: agent.create_new_auth_token,
|
|
as: :json
|
|
|
|
expect(response).to have_http_status(:success)
|
|
agent.reload
|
|
|
|
expect(agent.custom_attributes['phone_number']).to eq('+123456789')
|
|
end
|
|
|
|
it 'updates the message_signature' do
|
|
put '/api/v1/profile',
|
|
params: { profile: { name: 'test', message_signature: 'Thanks\nMy Signature' } },
|
|
headers: agent.create_new_auth_token,
|
|
as: :json
|
|
|
|
expect(response).to have_http_status(:success)
|
|
json_response = response.parsed_body
|
|
agent.reload
|
|
expect(json_response['id']).to eq(agent.id)
|
|
expect(json_response['name']).to eq(agent.name)
|
|
expect(agent.name).to eq('test')
|
|
expect(json_response['message_signature']).to eq('Thanks\nMy Signature')
|
|
end
|
|
|
|
it 'updates the password when current password is provided' do
|
|
put '/api/v1/profile',
|
|
params: { profile: { current_password: 'Test123!', password: 'Test1234!', password_confirmation: 'Test1234!' } },
|
|
headers: agent.create_new_auth_token,
|
|
as: :json
|
|
|
|
expect(response).to have_http_status(:success)
|
|
expect(agent.reload.valid_password?('Test1234!')).to be true
|
|
end
|
|
|
|
it 'does not reset the display name if updates the password' do
|
|
display_name = agent.display_name
|
|
|
|
put '/api/v1/profile',
|
|
params: { profile: { current_password: 'Test123!', password: 'Test1234!', password_confirmation: 'Test1234!' } },
|
|
headers: agent.create_new_auth_token,
|
|
as: :json
|
|
|
|
expect(response).to have_http_status(:success)
|
|
expect(agent.reload.display_name).to eq(display_name)
|
|
end
|
|
|
|
it 'throws error when current password provided is invalid' do
|
|
put '/api/v1/profile',
|
|
params: { profile: { current_password: 'Test', password: 'test123', password_confirmation: 'test123' } },
|
|
headers: agent.create_new_auth_token,
|
|
as: :json
|
|
|
|
expect(response).to have_http_status(:unprocessable_entity)
|
|
end
|
|
|
|
it 'validate name' do
|
|
user_name = 'test' * 999
|
|
put '/api/v1/profile',
|
|
params: { profile: { name: user_name } },
|
|
headers: agent.create_new_auth_token,
|
|
as: :json
|
|
|
|
expect(response).to have_http_status(:unprocessable_entity)
|
|
json_response = response.parsed_body
|
|
expect(json_response['message']).to eq('Name is too long (maximum is 255 characters)')
|
|
end
|
|
|
|
it 'updates avatar' do
|
|
# no avatar before upload
|
|
expect(agent.avatar.attached?).to be(false)
|
|
file = fixture_file_upload(Rails.root.join('spec/assets/avatar.png'), 'image/png')
|
|
put '/api/v1/profile',
|
|
params: { profile: { avatar: file } },
|
|
headers: agent.create_new_auth_token
|
|
|
|
expect(response).to have_http_status(:success)
|
|
agent.reload
|
|
expect(agent.avatar.attached?).to be(true)
|
|
end
|
|
|
|
it 'updates the ui settings' do
|
|
put '/api/v1/profile',
|
|
params: { profile: { ui_settings: { is_contact_sidebar_open: false } } },
|
|
headers: agent.create_new_auth_token,
|
|
as: :json
|
|
|
|
expect(response).to have_http_status(:success)
|
|
json_response = response.parsed_body
|
|
expect(json_response['ui_settings']['is_contact_sidebar_open']).to be(false)
|
|
end
|
|
|
|
it 'updates signature position in ui_settings' do
|
|
put '/api/v1/profile',
|
|
params: { profile: { ui_settings: { signature_position: 'bottom' } } },
|
|
headers: agent.create_new_auth_token,
|
|
as: :json
|
|
|
|
expect(response).to have_http_status(:success)
|
|
|
|
json_response = response.parsed_body
|
|
expect(json_response['ui_settings']['signature_position']).to eq('bottom')
|
|
expect(agent.reload.ui_settings['signature_position']).to eq('bottom')
|
|
end
|
|
|
|
it 'updates signature separator in ui_settings' do
|
|
put '/api/v1/profile',
|
|
params: { profile: { ui_settings: { signature_separator: '--' } } },
|
|
headers: agent.create_new_auth_token,
|
|
as: :json
|
|
|
|
expect(response).to have_http_status(:success)
|
|
|
|
json_response = response.parsed_body
|
|
expect(json_response['ui_settings']['signature_separator']).to eq('--')
|
|
expect(agent.reload.ui_settings['signature_separator']).to eq('--')
|
|
end
|
|
|
|
it 'updates both position and separator in ui_settings' do
|
|
put '/api/v1/profile',
|
|
params: { profile: { ui_settings: { signature_position: 'bottom', signature_separator: '--' } } },
|
|
headers: agent.create_new_auth_token,
|
|
as: :json
|
|
|
|
expect(response).to have_http_status(:success)
|
|
|
|
json_response = response.parsed_body
|
|
expect(json_response['ui_settings']['signature_position']).to eq('bottom')
|
|
expect(json_response['ui_settings']['signature_separator']).to eq('--')
|
|
expect(agent.reload.ui_settings['signature_position']).to eq('bottom')
|
|
expect(agent.ui_settings['signature_separator']).to eq('--')
|
|
end
|
|
end
|
|
|
|
context 'when an authenticated user updates email' do
|
|
let(:agent) { create(:user, password: 'Test123!', account: account, role: :agent) }
|
|
|
|
it 'populates the unconfirmed email' do
|
|
new_email = Faker::Internet.email
|
|
put '/api/v1/profile',
|
|
params: { profile: { email: new_email } },
|
|
headers: agent.create_new_auth_token,
|
|
as: :json
|
|
|
|
expect(response).to have_http_status(:success)
|
|
agent.reload
|
|
|
|
expect(agent.unconfirmed_email).to eq(new_email)
|
|
end
|
|
end
|
|
end
|
|
|
|
describe 'DELETE /api/v1/profile/avatar' do
|
|
let(:agent) { create(:user, password: 'Test123!', account: account, role: :agent) }
|
|
|
|
context 'when it is an unauthenticated user' do
|
|
it 'returns unauthorized' do
|
|
delete '/api/v1/profile/avatar'
|
|
|
|
expect(response).to have_http_status(:unauthorized)
|
|
end
|
|
end
|
|
|
|
context 'when it is an authenticated user' do
|
|
before do
|
|
agent.avatar.attach(io: Rails.root.join('spec/assets/avatar.png').open, filename: 'avatar.png', content_type: 'image/png')
|
|
end
|
|
|
|
it 'deletes the agent avatar' do
|
|
delete '/api/v1/profile/avatar',
|
|
headers: agent.create_new_auth_token,
|
|
as: :json
|
|
|
|
expect(response).to have_http_status(:success)
|
|
json_response = response.parsed_body
|
|
expect(json_response['avatar_url']).to be_empty
|
|
end
|
|
end
|
|
end
|
|
|
|
describe 'POST /api/v1/profile/availability' do
|
|
context 'when it is an unauthenticated user' do
|
|
it 'returns unauthorized' do
|
|
post '/api/v1/profile/availability'
|
|
|
|
expect(response).to have_http_status(:unauthorized)
|
|
end
|
|
end
|
|
|
|
context 'when it is an authenticated user' do
|
|
let(:agent) { create(:user, password: 'Test123!', account: account, role: :agent) }
|
|
|
|
it 'updates the availability status' do
|
|
post '/api/v1/profile/availability',
|
|
params: { profile: { availability: 'busy', account_id: account.id } },
|
|
headers: agent.create_new_auth_token,
|
|
as: :json
|
|
|
|
expect(response).to have_http_status(:success)
|
|
expect(OnlineStatusTracker.get_status(account.id, agent.id)).to eq('busy')
|
|
end
|
|
|
|
it 'dispatches account presence updated event' do
|
|
freeze_time
|
|
allow(Rails.configuration.dispatcher).to receive(:dispatch).with(
|
|
Events::Types::ACCOUNT_PRESENCE_UPDATED,
|
|
Time.zone.now,
|
|
account_id: account.id,
|
|
user_id: agent.id,
|
|
status: 'online'
|
|
)
|
|
|
|
post '/api/v1/profile/availability',
|
|
params: { profile: { availability: 'online', account_id: account.id } },
|
|
headers: agent.create_new_auth_token,
|
|
as: :json
|
|
|
|
expect(response).to have_http_status(:success)
|
|
expect(Rails.configuration.dispatcher).to have_received(:dispatch)
|
|
end
|
|
end
|
|
end
|
|
|
|
describe 'POST /api/v1/profile/auto_offline' do
|
|
context 'when it is an unauthenticated user' do
|
|
it 'returns unauthorized' do
|
|
post '/api/v1/profile/auto_offline'
|
|
expect(response).to have_http_status(:unauthorized)
|
|
end
|
|
end
|
|
|
|
context 'when it is an authenticated user' do
|
|
let(:agent) { create(:user, password: 'Test123!', account: account, role: :agent) }
|
|
|
|
it 'updates the auto offline status' do
|
|
post '/api/v1/profile/auto_offline',
|
|
params: { profile: { auto_offline: false, account_id: account.id } },
|
|
headers: agent.create_new_auth_token,
|
|
as: :json
|
|
|
|
expect(response).to have_http_status(:success)
|
|
json_response = response.parsed_body
|
|
expect(json_response['accounts'].first['auto_offline']).to be(false)
|
|
end
|
|
end
|
|
end
|
|
|
|
describe 'PUT /api/v1/profile/set_active_account' do
|
|
context 'when it is an unauthenticated user' do
|
|
it 'returns unauthorized' do
|
|
put '/api/v1/profile/set_active_account'
|
|
|
|
expect(response).to have_http_status(:unauthorized)
|
|
end
|
|
end
|
|
|
|
context 'when it is an authenticated user' do
|
|
let(:agent) { create(:user, password: 'Test123!', account: account, role: :agent) }
|
|
|
|
it 'updates the last active account id' do
|
|
put '/api/v1/profile/set_active_account',
|
|
params: { profile: { account_id: account.id } },
|
|
headers: agent.create_new_auth_token,
|
|
as: :json
|
|
|
|
expect(response).to have_http_status(:success)
|
|
end
|
|
end
|
|
end
|
|
|
|
describe 'POST /api/v1/profile/resend_confirmation' do
|
|
context 'when it is an unauthenticated user' do
|
|
it 'returns unauthorized' do
|
|
post '/api/v1/profile/resend_confirmation'
|
|
|
|
expect(response).to have_http_status(:unauthorized)
|
|
end
|
|
end
|
|
|
|
context 'when it is an authenticated user' do
|
|
let(:agent) do
|
|
create(:user, password: 'Test123!', email: 'test-unconfirmed@email.com', account: account, role: :agent,
|
|
unconfirmed_email: 'test-unconfirmed@email.com')
|
|
end
|
|
|
|
it 'does not send the confirmation email if the user is already confirmed' do
|
|
expect do
|
|
post '/api/v1/profile/resend_confirmation',
|
|
headers: agent.create_new_auth_token,
|
|
as: :json
|
|
end.not_to have_enqueued_mail(Devise::Mailer, :confirmation_instructions)
|
|
|
|
expect(response).to have_http_status(:success)
|
|
end
|
|
|
|
it 'resends the confirmation email if the user is unconfirmed' do
|
|
agent.confirmed_at = nil
|
|
agent.save!
|
|
|
|
expect do
|
|
post '/api/v1/profile/resend_confirmation',
|
|
headers: agent.create_new_auth_token,
|
|
as: :json
|
|
end.to have_enqueued_mail(Devise::Mailer, :confirmation_instructions)
|
|
|
|
expect(response).to have_http_status(:success)
|
|
end
|
|
end
|
|
end
|
|
|
|
describe 'POST /api/v1/profile/reset_access_token' do
|
|
context 'when it is an unauthenticated user' do
|
|
it 'returns unauthorized' do
|
|
post '/api/v1/profile/reset_access_token'
|
|
|
|
expect(response).to have_http_status(:unauthorized)
|
|
end
|
|
end
|
|
|
|
context 'when it is an authenticated user' do
|
|
let(:agent) { create(:user, account: account, role: :agent) }
|
|
|
|
it 'regenerates the access token' do
|
|
old_token = agent.access_token.token
|
|
|
|
post '/api/v1/profile/reset_access_token',
|
|
headers: agent.create_new_auth_token,
|
|
as: :json
|
|
|
|
expect(response).to have_http_status(:success)
|
|
agent.reload
|
|
expect(agent.access_token.token).not_to eq(old_token)
|
|
json_response = response.parsed_body
|
|
expect(json_response['access_token']).to eq(agent.access_token.token)
|
|
end
|
|
end
|
|
end
|
|
end
|