12345678910111213141516171819202122232425262728293031323334353637383940414243444546474849505152535455565758596061626364656667686970717273747576777879808182838485868788899091929394959697989910010110210310410510610710810911011111211311411511611711811912012112212312412512612712812913013113213313413513613713813914014114214314414514614714814915015115215315415515615715815916016116216316416516616716816917017117217317417517617717817918018118218318418518618718818919019119219319419519619719819920020120220320420520620720820921021121221321421521621721821922022122222322422522622722822923023123223323423523623723823924024124224324424524624724824925025125225325425525625725825926026126226326426526626726826927027127227327427527627727827928028128228328428528628728828929029129229329429529629729829930030130230330430530630730830931031131231331431531631731831932032132232332432532632732832933033133233333433533633733833934034134234334434534634734834935035135235335435535635735835936036136236336436536636736836937037137237337437537637737837938038138238338438538638738838939039139239339439539639739839940040140240340440540640740840941041141241341441541641741841942042142242342442542642742842943043143243343443543643743843944044144244344444544644744844945045145245345445545645745845946046146246346446546646746846947047147247347447547647747847948048148248348448548648748848949049149249349449549649749849950050150250350450550650750850951051151251351451551651751851952052152252352452552652752852953053153253353453553653753853954054154254354454554654754854955055155255355455555655755855956056156256356456556656756856957057157257357457557657757857958058158258358458558658758858959059159259359459559659759859960060160260360460560660760860961061161261361461561661761861962062162262362462562662762862963063163263363463563663763863964064164264364464564664764864965065165265365465565665765865966066166266366466566666766866967067167267367467567667767867968068168268368468568668768868969069169269369469569669769869970070170270370470570670770870971071171271371471571671771871972072172272372472572672772872973073173273373473573673773873974074174274374474574674774874975075175275375475575675775875976076176276376476576676776876977077177277377477577677777877978078178278378478578678778878979079179279379479579679779879980080180280380480580680780880981081181281381481581681781881982082182282382482582682782882983083183283383483583683783883984084184284384484584684784884985085185285385485585685785885986086186286386486586686786886987087187287387487587687787887988088188288388488588688788888989089189289389489589689789889990090190290390490590690790890991091191291391491591691791891992092192292392492592692792892993093193293393493593693793893994094194294394494594694794894995095195295395495595695795895996096196296396496596696796896997097197297397497597697797897998098198298398498598698798898999099199299399499599699799899910001001100210031004100510061007100810091010101110121013101410151016101710181019102010211022102310241025102610271028102910301031103210331034103510361037103810391040104110421043104410451046104710481049105010511052105310541055105610571058105910601061106210631064106510661067106810691070107110721073107410751076107710781079108010811082108310841085108610871088108910901091109210931094109510961097109810991100110111021103110411051106110711081109111011111112111311141115111611171118111911201121112211231124112511261127112811291130113111321133113411351136113711381139114011411142114311441145114611471148114911501151115211531154115511561157115811591160116111621163116411651166116711681169117011711172117311741175117611771178117911801181118211831184118511861187118811891190119111921193119411951196119711981199120012011202120312041205120612071208120912101211121212131214121512161217121812191220122112221223122412251226122712281229123012311232123312341235123612371238123912401241124212431244124512461247124812491250125112521253125412551256125712581259126012611262126312641265126612671268126912701271127212731274127512761277127812791280128112821283128412851286128712881289129012911292129312941295129612971298129913001301130213031304130513061307130813091310131113121313131413151316131713181319132013211322132313241325132613271328132913301331133213331334133513361337133813391340134113421343134413451346134713481349135013511352135313541355135613571358135913601361136213631364136513661367136813691370137113721373137413751376137713781379138013811382138313841385138613871388138913901391139213931394139513961397139813991400140114021403140414051406140714081409141014111412141314141415141614171418141914201421142214231424142514261427142814291430143114321433143414351436143714381439144014411442144314441445144614471448144914501451145214531454145514561457145814591460146114621463146414651466146714681469147014711472147314741475147614771478147914801481148214831484148514861487148814891490149114921493149414951496149714981499150015011502150315041505150615071508150915101511151215131514151515161517151815191520152115221523152415251526152715281529153015311532153315341535153615371538153915401541154215431544154515461547154815491550155115521553155415551556155715581559156015611562156315641565156615671568156915701571157215731574157515761577157815791580158115821583158415851586158715881589159015911592159315941595159615971598159916001601160216031604160516061607160816091610161116121613161416151616161716181619162016211622162316241625162616271628162916301631163216331634163516361637163816391640164116421643164416451646164716481649165016511652165316541655165616571658165916601661166216631664166516661667166816691670167116721673167416751676167716781679168016811682168316841685168616871688168916901691169216931694169516961697169816991700170117021703170417051706170717081709171017111712171317141715171617171718171917201721172217231724172517261727172817291730173117321733173417351736173717381739174017411742174317441745174617471748174917501751175217531754175517561757175817591760176117621763176417651766176717681769177017711772177317741775177617771778177917801781178217831784178517861787178817891790179117921793179417951796179717981799180018011802180318041805180618071808180918101811181218131814181518161817181818191820182118221823182418251826182718281829183018311832183318341835183618371838183918401841184218431844184518461847184818491850185118521853185418551856185718581859186018611862186318641865186618671868186918701871187218731874187518761877187818791880188118821883188418851886188718881889189018911892189318941895189618971898189919001901190219031904190519061907190819091910191119121913191419151916191719181919192019211922192319241925192619271928192919301931193219331934193519361937193819391940194119421943194419451946194719481949195019511952195319541955195619571958195919601961196219631964196519661967196819691970197119721973197419751976197719781979198019811982198319841985198619871988198919901991199219931994199519961997199819992000200120022003200420052006200720082009201020112012201320142015201620172018201920202021202220232024202520262027202820292030203120322033203420352036203720382039204020412042204320442045204620472048204920502051205220532054205520562057205820592060206120622063206420652066206720682069207020712072207320742075207620772078207920802081208220832084208520862087208820892090209120922093209420952096209720982099210021012102210321042105210621072108210921102111211221132114211521162117211821192120212121222123212421252126212721282129213021312132213321342135213621372138213921402141214221432144214521462147214821492150215121522153215421552156215721582159216021612162216321642165216621672168216921702171217221732174217521762177217821792180218121822183218421852186218721882189219021912192219321942195219621972198219922002201220222032204220522062207220822092210221122122213221422152216221722182219222022212222222322242225222622272228222922302231223222332234223522362237223822392240224122422243224422452246224722482249225022512252225322542255225622572258225922602261226222632264226522662267226822692270227122722273227422752276227722782279228022812282228322842285228622872288228922902291229222932294229522962297229822992300230123022303230423052306230723082309231023112312231323142315231623172318231923202321232223232324232523262327232823292330233123322333233423352336233723382339234023412342234323442345234623472348234923502351235223532354235523562357235823592360236123622363236423652366236723682369237023712372237323742375237623772378237923802381238223832384238523862387238823892390239123922393239423952396239723982399240024012402240324042405240624072408240924102411241224132414241524162417241824192420242124222423242424252426242724282429243024312432243324342435243624372438243924402441244224432444244524462447244824492450245124522453245424552456245724582459246024612462246324642465246624672468246924702471247224732474247524762477247824792480248124822483248424852486248724882489249024912492249324942495249624972498249925002501250225032504250525062507250825092510251125122513251425152516251725182519252025212522252325242525252625272528252925302531253225332534253525362537253825392540254125422543254425452546 |
- # Copyright (C) 2012-2022 Zammad Foundation, https://zammad-foundation.org/
- require 'rails_helper'
- require 'system/examples/core_workflow_examples'
- RSpec.describe 'Ticket zoom', type: :system do
- describe 'owner auto-assignment', authenticated_as: :authenticate do
- let!(:ticket) { create(:ticket, group: Group.find_by(name: 'Users'), state: Ticket::State.find_by(name: 'new')) }
- let!(:session_user) { User.find_by(login: 'admin@example.com') }
- context 'for agent disabled' do
- def authenticate
- Setting.set('ticket_auto_assignment', false)
- Setting.set('ticket_auto_assignment_selector', { condition: { 'ticket.state_id' => { operator: 'is', value: Ticket::State.by_category(:work_on).pluck(:id) } } })
- Setting.set('ticket_auto_assignment_user_ids_ignore', [])
- true
- end
- it 'do not assign ticket to current session user' do
- visit "#ticket/zoom/#{ticket.id}"
- within(:active_content) do
- expect(page).to have_css('select[name=owner_id]')
- expect(page).to have_select('owner_id',
- selected: '-',
- options: ['-', 'Agent 1 Test', 'Test Admin Agent'])
- end
- end
- end
- context 'for agent enabled' do
- def authenticate
- Setting.set('ticket_auto_assignment', true)
- Setting.set('ticket_auto_assignment_selector', { condition: { 'ticket.state_id' => { operator: 'is', value: Ticket::State.by_category(:work_on).pluck(:id) } } })
- Setting.set('ticket_auto_assignment_user_ids_ignore', setting_user_ids_ignore) if defined?(setting_user_ids_ignore)
- true
- end
- context 'with empty "ticket_auto_assignment_user_ids_ignore"' do
- it 'assigns ticket to current session user' do
- visit "#ticket/zoom/#{ticket.id}"
- within(:active_content) do
- expect(page).to have_css('.content.active select[name=owner_id]')
- expect(page).to have_select('owner_id',
- selected: session_user.fullname,
- options: ['-', 'Agent 1 Test', 'Test Admin Agent'])
- end
- end
- end
- context 'with "ticket_auto_assignment_user_ids_ignore" (as integer)' do
- let(:setting_user_ids_ignore) { session_user.id }
- it 'assigns ticket not to current session user' do
- visit "#ticket/zoom/#{ticket.id}"
- within(:active_content) do
- expect(page).to have_css('select[name=owner_id]')
- expect(page).to have_select('owner_id',
- selected: '-',
- options: ['-', 'Agent 1 Test', 'Test Admin Agent'])
- end
- end
- end
- context 'with "ticket_auto_assignment_user_ids_ignore" (as string)' do
- let(:setting_user_ids_ignore) { session_user.id.to_s }
- it 'assigns ticket not to current session user' do
- visit "#ticket/zoom/#{ticket.id}"
- within(:active_content) do
- expect(page).to have_css('select[name=owner_id]')
- expect(page).to have_select('owner_id',
- selected: '-',
- options: ['-', 'Agent 1 Test', 'Test Admin Agent'])
- end
- end
- end
- context 'with "ticket_auto_assignment_user_ids_ignore" (as [integer])' do
- let(:setting_user_ids_ignore) { [session_user.id] }
- it 'assigns ticket not to current session user' do
- visit "#ticket/zoom/#{ticket.id}"
- within(:active_content) do
- expect(page).to have_css('select[name=owner_id]')
- expect(page).to have_select('owner_id',
- selected: '-',
- options: ['-', 'Agent 1 Test', 'Test Admin Agent'])
- end
- end
- end
- context 'with "ticket_auto_assignment_user_ids_ignore" (as [string])' do
- let(:setting_user_ids_ignore) { [session_user.id.to_s] }
- it 'assigns ticket not to current session user' do
- visit "#ticket/zoom/#{ticket.id}"
- within(:active_content) do
- expect(page).to have_css('select[name=owner_id]')
- expect(page).to have_select('owner_id',
- selected: '-',
- options: ['-', 'Agent 1 Test', 'Test Admin Agent'])
- end
- end
- end
- context 'with "ticket_auto_assignment_user_ids_ignore" and other user ids' do
- let(:setting_user_ids_ignore) { [99_999, 999_999] }
- it 'assigns ticket to current session user' do
- visit "#ticket/zoom/#{ticket.id}"
- within(:active_content) do
- expect(page).to have_css('select[name=owner_id]')
- expect(page).to have_select('owner_id',
- selected: session_user.fullname,
- options: ['-', 'Agent 1 Test', 'Test Admin Agent'])
- end
- end
- end
- end
- end
- context 'when ticket has an attachment' do
- let(:group) { Group.find_by(name: 'Users') }
- let(:ticket) { create(:ticket, group: group) }
- let(:article) { create(:ticket_article, ticket: ticket) }
- let(:attachment_name) { 'some_file.txt' }
- before do
- Store.add(
- object: 'Ticket::Article',
- o_id: article.id,
- data: 'some content',
- filename: attachment_name,
- preferences: {
- 'Content-Type' => 'text/plain',
- },
- created_by_id: 1,
- )
- end
- context 'article was already forwarded once' do
- before do
- visit "#ticket/zoom/#{ticket.id}"
- within(:active_content) do
- find('a[data-type=emailForward]').click
- click('.js-reset')
- have_no_css('.js-reset')
- end
- end
- it 'adds attachments when forwarding multiple times' do
- within(:active_content) do
- find('a[data-type=emailForward]').click
- end
- within('.js-writeArea') do
- expect(page).to have_text attachment_name
- end
- end
- end
- end
- context 'replying' do
- context 'Group without signature' do
- let(:ticket) { create(:ticket) }
- let(:current_user) { create(:agent, password: 'test', groups: [ticket.group]) }
- before do
- # initial article to reply to
- create(:ticket_article, ticket: ticket)
- end
- it 'ensures that text input opens on multiple replies', authenticated_as: :current_user do
- visit "ticket/zoom/#{ticket.id}"
- 2.times do |article_offset|
- articles_existing = 1
- articles_expected = articles_existing + (article_offset + 1)
- all('a[data-type=emailReply]').last.click
- # wait till input box expands completely
- find('.attachmentPlaceholder-label').in_fixed_position
- expect(page).to have_no_css('.attachmentPlaceholder-hint')
- find('.articleNewEdit-body').send_keys('Some reply')
- click '.js-submit'
- expect(page).to have_css('.ticket-article-item', count: articles_expected)
- end
- end
- end
- context 'to inbound phone call', current_user_id: -> { agent.id }, authenticated_as: -> { agent } do
- let(:agent) { create(:agent, groups: [Group.first]) }
- let(:customer) { create(:customer) }
- let(:ticket) { create(:ticket, customer: customer, group: agent.groups.first) }
- let!(:article) { create(:ticket_article, :inbound_phone, ticket: ticket) }
- before do
- create(:customer, active: false)
- end
- it 'goes to customer email' do
- visit "ticket/zoom/#{ticket.id}"
- within :active_ticket_article, article do
- click '.js-ArticleAction[data-type=emailReply]'
- end
- within :active_content do
- within '.article-new' do
- expect(find('[name=to]', visible: :all).value).to eq customer.email
- end
- end
- end
- it 'check active and inactive user in TO-field' do
- visit "ticket/zoom/#{ticket.id}"
- within :active_ticket_article, article do
- click '.js-ArticleAction[data-type=emailReply]'
- end
- within :active_content do
- within '.article-new' do
- find('[name=to] ~ .ui-autocomplete-input').fill_in with: '**'
- end
- end
- expect(page).to have_css('ul.ui-autocomplete > li.ui-menu-item', minimum: 2)
- expect(page).to have_css('ul.ui-autocomplete > li.ui-menu-item.is-inactive', count: 1)
- end
- end
- context 'to outbound phone call', current_user_id: -> { agent.id }, authenticated_as: -> { agent } do
- let(:agent) { create(:agent, groups: [Group.first]) }
- let(:customer) { create(:customer) }
- let(:ticket) { create(:ticket, customer: customer, group: agent.groups.first) }
- let!(:article) { create(:ticket_article, :outbound_phone, ticket: ticket) }
- it 'goes to customer email' do
- visit "ticket/zoom/#{ticket.id}"
- within :active_ticket_article, article do
- click '.js-ArticleAction[data-type=emailReply]'
- end
- within :active_content do
- within '.article-new' do
- expect(find('[name=to]', visible: :all).value).to eq customer.email
- end
- end
- end
- end
- context 'scrollPageHeader disappears when answering via email #3736' do
- let(:ticket) do
- ticket = create(:ticket, group: Group.first)
- create_list(:ticket_article, 15, ticket: ticket)
- ticket
- end
- before do
- visit "ticket/zoom/#{ticket.id}"
- end
- it 'does reset the scrollPageHeader on rerender of the ticket' do
- select User.find_by(email: 'admin@example.com').fullname, from: 'Owner'
- find('.js-textarea').send_keys('test 1234')
- find('.js-submit').click
- expect(page).to have_selector('div.scrollPageHeader .js-ticketTitleContainer')
- end
- end
- end
- describe 'delete article', authenticated_as: :authenticate do
- let(:group) { Group.first }
- let(:admin) { create :admin, groups: [group] }
- let(:agent) { create :agent, groups: [group] }
- let(:other_agent) { create :agent, groups: [group] }
- let(:customer) { create :customer }
- let(:article) { send(item) }
- def authenticate
- Setting.set('ui_ticket_zoom_article_delete_timeframe', setting_delete_timeframe) if defined?(setting_delete_timeframe)
- article
- user
- end
- def article_communication
- create_ticket_article(sender_name: 'Agent', internal: false, type_name: 'email', updated_by: customer)
- end
- def article_note_self
- create_ticket_article(sender_name: 'Agent', internal: true, type_name: 'note', updated_by: user)
- end
- def article_note_other
- create_ticket_article(sender_name: 'Agent', internal: true, type_name: 'note', updated_by: other_agent)
- end
- def article_note_customer
- create_ticket_article(sender_name: 'Customer', internal: false, type_name: 'note', updated_by: customer)
- end
- def article_note_communication_self
- create(:ticket_article_type, name: 'note_communication', communication: true)
- create_ticket_article(sender_name: 'Agent', internal: true, type_name: 'note_communication', updated_by: user)
- end
- def article_note_communication_other
- create(:ticket_article_type, name: 'note_communication', communication: true)
- create_ticket_article(sender_name: 'Agent', internal: true, type_name: 'note_communication', updated_by: other_agent)
- end
- def create_ticket_article(sender_name:, internal:, type_name:, updated_by:)
- UserInfo.current_user_id = updated_by.id
- ticket = create :ticket, group: group, customer: customer
- create(:ticket_article,
- sender_name: sender_name, internal: internal, type_name: type_name, ticket: ticket,
- body: "to be deleted #{offset} #{item}",
- created_at: offset.ago, updated_at: offset.ago)
- end
- context 'going through full stack' do
- context 'as admin' do
- let(:user) { admin }
- let(:item) { 'article_note_self' }
- let(:offset) { 0.minutes }
- it 'succeeds' do
- ensure_websocket do
- visit "ticket/zoom/#{article.ticket.id}"
- end
- within :active_ticket_article, article do
- click '.js-ArticleAction[data-type=delete]'
- end
- in_modal do
- click '.js-submit'
- end
- wait.until_disappears { find :active_ticket_article, article, wait: false }
- end
- end
- end
- context 'verifying permissions matrix' do
- shared_examples 'according to permission matrix' do |item:, expects_visible:, offset:, description:|
- context "looking at #{description} #{item}" do
- let(:item) { item }
- let(:offset) { offset }
- let(:matcher) { expects_visible ? :have_css : :have_no_css }
- it expects_visible ? 'delete button is visible' : 'delete button is not visible' do
- visit "ticket/zoom/#{article.ticket.id}"
- wait.until_exists { find("#article-#{article.id}") }
- within :active_ticket_article, article do
- expect(page).to send(matcher, '.js-ArticleAction[data-type=delete]', wait: 0)
- end
- end
- end
- end
- shared_examples 'deleting ticket article' do |item:, now:, later:, much_later:|
- include_examples 'according to permission matrix', item: item, expects_visible: now, offset: 0.minutes, description: 'just created'
- include_examples 'according to permission matrix', item: item, expects_visible: later, offset: 6.minutes, description: 'few minutes old'
- include_examples 'according to permission matrix', item: item, expects_visible: much_later, offset: 11.minutes, description: 'very old'
- end
- context 'as admin' do
- let(:user) { admin }
- include_examples 'deleting ticket article',
- item: 'article_communication',
- now: false, later: false, much_later: false
- include_examples 'deleting ticket article',
- item: 'article_note_self',
- now: true, later: true, much_later: false
- include_examples 'deleting ticket article',
- item: 'article_note_other',
- now: false, later: false, much_later: false
- include_examples 'deleting ticket article',
- item: 'article_note_customer',
- now: false, later: false, much_later: false
- include_examples 'deleting ticket article',
- item: 'article_note_communication_self',
- now: true, later: true, much_later: false
- include_examples 'deleting ticket article',
- item: 'article_note_communication_other',
- now: false, later: false, much_later: false
- end
- context 'as agent' do
- let(:user) { agent }
- include_examples 'deleting ticket article',
- item: 'article_communication',
- now: false, later: false, much_later: false
- include_examples 'deleting ticket article',
- item: 'article_note_self',
- now: true, later: true, much_later: false
- include_examples 'deleting ticket article',
- item: 'article_note_other',
- now: false, later: false, much_later: false
- include_examples 'deleting ticket article',
- item: 'article_note_customer',
- now: false, later: false, much_later: false
- include_examples 'deleting ticket article',
- item: 'article_note_communication_self',
- now: true, later: true, much_later: false
- include_examples 'deleting ticket article',
- item: 'article_note_communication_other',
- now: false, later: false, much_later: false
- end
- context 'as customer' do
- let(:user) { customer }
- include_examples 'deleting ticket article',
- item: 'article_communication',
- now: false, later: false, much_later: false
- include_examples 'deleting ticket article',
- item: 'article_note_customer',
- now: false, later: false, much_later: false
- end
- context 'with custom offset' do
- let(:setting_delete_timeframe) { 6_000 }
- context 'as admin' do
- let(:user) { admin }
- include_examples 'according to permission matrix', item: 'article_note_self', expects_visible: true, offset: 5000.seconds, description: 'outside of delete timeframe'
- include_examples 'according to permission matrix', item: 'article_note_self', expects_visible: false, offset: 8000.seconds, description: 'outside of delete timeframe'
- end
- context 'as agent' do
- let(:user) { agent }
- include_examples 'according to permission matrix', item: 'article_note_self', expects_visible: true, offset: 5000.seconds, description: 'outside of delete timeframe'
- include_examples 'according to permission matrix', item: 'article_note_self', expects_visible: false, offset: 8000.seconds, description: 'outside of delete timeframe'
- end
- end
- context 'with timeframe as 0' do
- let(:setting_delete_timeframe) { 0 }
- context 'as agent' do
- let(:user) { agent }
- include_examples 'according to permission matrix', item: 'article_note_self', expects_visible: true, offset: 99.days, description: 'long after'
- end
- end
- end
- context 'button is hidden on the go' do
- let(:setting_delete_timeframe) { 10 }
- let(:user) { agent }
- let(:item) { 'article_note_self' }
- let!(:article) { send(item) }
- let(:offset) { 0.seconds }
- it 'successfully' do
- visit "ticket/zoom/#{article.ticket.id}"
- within :active_ticket_article, article do
- find '.js-ArticleAction[data-type=delete]' # make sure delete button did show up
- expect(page).to have_no_css('.js-ArticleAction[data-type=delete]')
- end
- end
- end
- end
- context 'S/MIME active', authenticated_as: :authenticate do
- let(:system_email_address) { 'smime1@example.com' }
- let(:email_address) { create(:email_address, email: system_email_address) }
- let(:group) { create(:group, email_address: email_address) }
- let(:agent_groups) { [group] }
- let(:agent) { create(:agent, groups: agent_groups) }
- let(:sender_email_address) { 'smime2@example.com' }
- let(:customer) { create(:customer, email: sender_email_address) }
- let!(:ticket) { create(:ticket, group: group, owner: agent, customer: customer) }
- def authenticate
- Setting.set('smime_integration', true)
- agent
- end
- context 'received mail' do
- context 'article meta information' do
- context 'success' do
- it 'shows encryption/sign information' do
- create(:ticket_article, preferences: {
- security: {
- type: 'S/MIME',
- encryption: {
- success: true,
- comment: 'COMMENT_ENCRYPT_SUCCESS',
- },
- sign: {
- success: true,
- comment: 'COMMENT_SIGN_SUCCESS',
- },
- }
- }, ticket: ticket)
- visit "#ticket/zoom/#{ticket.id}"
- expect(page).to have_css('svg.icon-lock')
- expect(page).to have_css('svg.icon-signed')
- open_article_meta
- expect(page).to have_css('span', text: 'Encrypted')
- expect(page).to have_css('span', text: 'Signed')
- expect(page).to have_css('span[title=COMMENT_ENCRYPT_SUCCESS]')
- expect(page).to have_css('span[title=COMMENT_SIGN_SUCCESS]')
- end
- end
- context 'error' do
- it 'shows create information about encryption/sign failed' do
- create(:ticket_article, preferences: {
- security: {
- type: 'S/MIME',
- encryption: {
- success: false,
- comment: 'Encryption failed because XXX',
- },
- sign: {
- success: false,
- comment: 'Sign failed because XXX',
- },
- }
- }, ticket: ticket)
- visit "#ticket/zoom/#{ticket.id}"
- expect(page).to have_css('svg.icon-not-signed')
- open_article_meta
- expect(page).to have_css('div.alert.alert--warning', text: 'Encryption failed because XXX')
- expect(page).to have_css('div.alert.alert--warning', text: 'Sign failed because XXX')
- end
- end
- end
- context 'certificate not present at time of arrival' do
- let(:mail) do
- smime1 = create(:smime_certificate, :with_private, fixture: system_email_address)
- smime2 = create(:smime_certificate, :with_private, fixture: sender_email_address)
- mail = Channel::EmailBuild.build(
- from: sender_email_address,
- to: system_email_address,
- body: 'somebody with some text',
- content_type: 'text/plain',
- security: {
- type: 'S/MIME',
- sign: {
- success: true,
- },
- encryption: {
- success: true,
- },
- },
- )
- smime1.destroy
- smime2.destroy
- mail
- end
- it 'does retry successfully' do
- parsed_mail = Channel::EmailParser.new.parse(mail.to_s)
- ticket, article, _user, _mail = Channel::EmailParser.new.process({ group_id: group.id }, parsed_mail['raw'])
- expect(Ticket::Article.find(article.id).body).to eq('no visible content')
- create(:smime_certificate, fixture: sender_email_address)
- create(:smime_certificate, :with_private, fixture: system_email_address)
- visit "#ticket/zoom/#{ticket.id}"
- expect(page).to have_no_css('.article-content', text: 'somebody with some text')
- click '.js-securityRetryProcess'
- expect(page).to have_css('.article-content', text: 'somebody with some text')
- end
- it 'does fail on retry (S/MIME function buttons no longer working in tickets #3957)' do
- parsed_mail = Channel::EmailParser.new.parse(mail.to_s)
- ticket, article, _user, _mail = Channel::EmailParser.new.process({ group_id: group.id }, parsed_mail['raw'])
- expect(Ticket::Article.find(article.id).body).to eq('no visible content')
- visit "#ticket/zoom/#{ticket.id}"
- expect(page).to have_no_css('.article-content', text: 'somebody with some text')
- click '.js-securityRetryProcess'
- expect(page).to have_css('#notify', text: 'Decryption failed! Private key for decryption could not be found.')
- end
- end
- end
- context 'replying', authenticated_as: :setup_and_authenticate do
- def setup_and_authenticate
- create(:ticket_article, ticket: ticket, from: customer.email)
- create(:smime_certificate, :with_private, fixture: system_email_address)
- create(:smime_certificate, fixture: sender_email_address)
- authenticate
- end
- it 'plain' do
- visit "#ticket/zoom/#{ticket.id}"
- all('a[data-type=emailReply]').last.click
- find('.articleNewEdit-body').send_keys('Test')
- expect(page).to have_css('.js-securityEncrypt.btn--active')
- expect(page).to have_css('.js-securitySign.btn--active')
- click '.js-securityEncrypt'
- click '.js-securitySign'
- click '.js-submit'
- expect(page).to have_css('.ticket-article-item', count: 2)
- expect(Ticket::Article.last.preferences['security']['encryption']['success']).to be nil
- expect(Ticket::Article.last.preferences['security']['sign']['success']).to be nil
- end
- it 'signed' do
- visit "#ticket/zoom/#{ticket.id}"
- all('a[data-type=emailReply]').last.click
- find('.articleNewEdit-body').send_keys('Test')
- expect(page).to have_css('.js-securityEncrypt.btn--active')
- expect(page).to have_css('.js-securitySign.btn--active')
- click '.js-securityEncrypt'
- click '.js-submit'
- expect(page).to have_css('.ticket-article-item', count: 2)
- expect(Ticket::Article.last.preferences['security']['encryption']['success']).to be nil
- expect(Ticket::Article.last.preferences['security']['sign']['success']).to be true
- end
- it 'encrypted' do
- visit "#ticket/zoom/#{ticket.id}"
- all('a[data-type=emailReply]').last.click
- find('.articleNewEdit-body').send_keys('Test')
- expect(page).to have_css('.js-securityEncrypt.btn--active')
- expect(page).to have_css('.js-securitySign.btn--active')
- click '.js-securitySign'
- click '.js-submit'
- expect(page).to have_css('.ticket-article-item', count: 2)
- expect(Ticket::Article.last.preferences['security']['encryption']['success']).to be true
- expect(Ticket::Article.last.preferences['security']['sign']['success']).to be nil
- end
- it 'signed and encrypted' do
- visit "#ticket/zoom/#{ticket.id}"
- all('a[data-type=emailReply]').last.click
- find('.articleNewEdit-body').send_keys('Test')
- expect(page).to have_css('.js-securityEncrypt.btn--active')
- expect(page).to have_css('.js-securitySign.btn--active')
- click '.js-submit'
- expect(page).to have_css('.ticket-article-item', count: 2)
- expect(Ticket::Article.last.preferences['security']['encryption']['success']).to be true
- expect(Ticket::Article.last.preferences['security']['sign']['success']).to be true
- end
- end
- context 'Group default behavior' do
- let(:smime_config) { {} }
- def authenticate
- Setting.set('smime_integration', true)
- Setting.set('smime_config', smime_config)
- create(:ticket_article, ticket: ticket, from: customer.email)
- create(:smime_certificate, :with_private, fixture: system_email_address)
- create(:smime_certificate, fixture: sender_email_address)
- agent
- end
- shared_examples 'security defaults example' do |sign:, encrypt:|
- it "security defaults sign: #{sign}, encrypt: #{encrypt}" do
- within(:active_content) do
- encrypt_button = find('.js-securityEncrypt')
- sign_button = find('.js-securitySign')
- active_button_class = '.btn--active'
- expect(encrypt_button.matches_css?(active_button_class)).to be(encrypt)
- expect(sign_button.matches_css?(active_button_class)).to be(sign)
- end
- end
- end
- shared_examples 'security defaults' do |sign:, encrypt:|
- before do
- visit "#ticket/zoom/#{ticket.id}"
- within(:active_content) do
- all('a[data-type=emailReply]').last.click
- find('.articleNewEdit-body').send_keys('Test')
- end
- end
- include_examples 'security defaults example', sign: sign, encrypt: encrypt
- end
- shared_examples 'security defaults group change' do |sign:, encrypt:|
- before do
- visit "#ticket/zoom/#{ticket.id}"
- within(:active_content) do
- all('a[data-type=emailReply]').last.click
- find('.articleNewEdit-body').send_keys('Test')
- select new_group.name, from: 'group_id'
- end
- end
- include_examples 'security defaults example', sign: sign, encrypt: encrypt
- end
- context 'not configured' do
- it_behaves_like 'security defaults', sign: true, encrypt: true
- end
- context 'configuration present' do
- let(:smime_config) do
- {
- 'group_id' => group_defaults
- }
- end
- let(:group_defaults) do
- {
- 'default_encryption' => {
- group.id.to_s => default_encryption,
- },
- 'default_sign' => {
- group.id.to_s => default_sign,
- }
- }
- end
- let(:default_sign) { true }
- let(:default_encryption) { true }
- shared_examples 'sign and encrypt variations' do |check_examples_name|
- it_behaves_like check_examples_name, sign: true, encrypt: true
- context 'no value' do
- let(:group_defaults) { {} }
- it_behaves_like check_examples_name, sign: true, encrypt: true
- end
- context 'signing disabled' do
- let(:default_sign) { false }
- it_behaves_like check_examples_name, sign: false, encrypt: true
- end
- context 'encryption disabled' do
- let(:default_encryption) { false }
- it_behaves_like check_examples_name, sign: true, encrypt: false
- end
- end
- context 'same Group' do
- it_behaves_like 'sign and encrypt variations', 'security defaults'
- end
- context 'Group change' do
- let(:new_group) { create(:group, email_address: email_address) }
- let(:agent_groups) { [group, new_group] }
- let(:group_defaults) do
- {
- 'default_encryption' => {
- new_group.id.to_s => default_encryption,
- },
- 'default_sign' => {
- new_group.id.to_s => default_sign,
- }
- }
- end
- it_behaves_like 'sign and encrypt variations', 'security defaults group change'
- end
- end
- end
- end
- describe 'linking Knowledge Base answer' do
- include_context 'basic Knowledge Base'
- let(:ticket) { create :ticket, group: Group.find_by(name: 'Users') }
- let(:answer) { published_answer }
- let(:translation) { answer.translations.first }
- shared_examples 'verify linking' do
- it 'allows to look up an answer' do
- visit "#ticket/zoom/#{ticket.id}"
- within :active_content do
- within '.link_kb_answers' do
- find('.js-add').click
- find('.js-input').send_keys translation.title
- find(%(li[data-value="#{translation.id}"])).click
- expect(find('.link_kb_answers ol')).to have_text translation.title
- end
- end
- end
- end
- context 'with ES', searchindex: true, authenticated_as: :authenticate do
- def authenticate
- configure_elasticsearch(required: true, rebuild: true) do
- answer
- end
- true
- end
- include_examples 'verify linking'
- end
- context 'without ES', authenticated_as: :authenticate do
- def authenticate
- answer
- true
- end
- include_examples 'verify linking'
- end
- end
- describe 'forwarding article with an image' do
- let(:ticket_article_body) do
- filename = 'squares.png'
- file = File.binread(Rails.root.join("spec/fixtures/image/#{filename}"))
- ext = File.extname(filename)[1...]
- base64 = Base64.encode64(file).delete("\n")
- "<img style='width: 1004px; max-width: 100%;' src=\\\"data:image/#{ext};base64,#{base64}\\\"><br>"
- end
- def current_ticket
- Ticket.find current_url.split('/').last
- end
- def create_ticket
- visit '#ticket/create'
- within :active_content do
- find('[data-type=email-out]').click
- find('[name=title]').fill_in with: 'Title'
- find('[name=customer_id_completion]').fill_in with: 'customer@example.com'
- find('[name=group_id]').select 'Users'
- find(:richtext).execute_script "this.innerHTML = \"#{ticket_article_body}\""
- find('.js-submit').click
- end
- end
- def forward
- within :active_content do
- # binding.pry
- wait.until_exists { find('.textBubble-content .richtext-content') }
- click '.js-ArticleAction[data-type=emailForward]'
- fill_in 'To', with: 'customer@example.com'
- find('.js-submit').click
- end
- end
- def images_identical?(image_a, image_b)
- return false if image_a.height != image_b.height
- return false if image_a.width != image_b.width
- image_a.height.times do |y|
- image_a.row(y).each_with_index do |pixel, x|
- return false if pixel != image_b[x, y]
- end
- end
- true
- end
- it 'keeps image intact' do
- create_ticket
- forward
- images = current_ticket.articles.map do |article|
- ChunkyPNG::Image.from_string article.attachments.first.content
- end
- expect(images_identical?(images.first, images.second)).to be(true)
- end
- end
- # https://github.com/zammad/zammad/issues/3335
- context 'ticket state sort order maintained when locale is de-de', authenticated_as: :user do
- context 'when existing ticket is open' do
- let(:user) { create(:customer, preferences: { locale: 'de-de' }) }
- let(:ticket) { create(:ticket, customer: user) }
- it 'shows ticket state dropdown options in sorted translated alphabetically order' do
- visit "ticket/zoom/#{ticket.id}"
- within :active_content, '.tabsSidebar' do
- expect(all('select[name=state_id] option').map(&:text)).to eq(%w[geschlossen neu offen])
- end
- end
- end
- context 'when a new ticket is created' do
- let(:user) { create(:agent, preferences: { locale: 'de-de' }, groups: [permitted_group]) }
- let(:permitted_group) { create(:group) }
- it 'shows ticket state dropdown options in sorted order' do
- visit 'ticket/create'
- expect(all('select[name=state_id] option').map(&:text)).to eq ['-', 'geschlossen', 'neu', 'offen', 'warten auf Erinnerung', 'warten auf Schließen']
- end
- end
- end
- context 'object manager attribute permission view' do
- let!(:group_users) { Group.find_by(name: 'Users') }
- shared_examples 'shows attributes and values for agent view and editable' do
- it 'shows attributes and values for agent view and editable', authenticated_as: :current_user do
- visit "ticket/zoom/#{ticket.id}"
- refresh # refresh to have assets generated for ticket
- expect(page).to have_select('state_id', options: ['new', 'open', 'pending reminder', 'pending close', 'closed'])
- expect(page).to have_select('priority_id')
- expect(page).to have_select('owner_id')
- expect(page).to have_css('div.tabsSidebar-tab[data-tab=customer]')
- end
- end
- shared_examples 'shows attributes and values for agent view but disabled' do
- it 'shows attributes and values for agent view but disabled', authenticated_as: :current_user do
- visit "ticket/zoom/#{ticket.id}"
- refresh # refresh to have assets generated for ticket
- expect(page).to have_css('select[name=state_id][disabled]')
- expect(page).to have_css('select[name=priority_id][disabled]')
- expect(page).to have_css('select[name=owner_id][disabled]')
- expect(page).to have_css('div.tabsSidebar-tab[data-tab=customer]')
- end
- end
- shared_examples 'shows attributes and values for customer view' do
- it 'shows attributes and values for customer view', authenticated_as: :current_user do
- visit "ticket/zoom/#{ticket.id}"
- refresh # refresh to have assets generated for ticket
- expect(page).to have_select('state_id', options: %w[new open closed])
- expect(page).to have_no_select('priority_id')
- expect(page).to have_no_select('owner_id')
- expect(page).to have_no_css('div.tabsSidebar-tab[data-tab=customer]')
- end
- end
- context 'as customer' do
- let!(:current_user) { create(:customer) }
- let(:ticket) { create(:ticket, customer: current_user) }
- include_examples 'shows attributes and values for customer view'
- end
- context 'as agent with full permissions' do
- let(:current_user) { create(:agent, groups: [ group_users ]) }
- let(:ticket) { create(:ticket, group: group_users) }
- include_examples 'shows attributes and values for agent view and editable'
- end
- context 'as agent with change permissions' do
- let!(:current_user) { create(:agent) }
- let(:ticket) { create(:ticket, group: group_users) }
- before do
- current_user.group_names_access_map = {
- group_users.name => %w[read change],
- }
- end
- include_examples 'shows attributes and values for agent view and editable'
- end
- context 'as agent with read permissions' do
- let!(:current_user) { create(:agent) }
- let(:ticket) { create(:ticket, group: group_users) }
- before do
- current_user.group_names_access_map = {
- group_users.name => 'read',
- }
- end
- include_examples 'shows attributes and values for agent view but disabled'
- end
- context 'as agent+customer with full permissions' do
- let!(:current_user) { create(:agent_and_customer, groups: [ group_users ]) }
- context 'normal ticket' do
- let(:ticket) { create(:ticket, group: group_users) }
- include_examples 'shows attributes and values for agent view and editable'
- end
- context 'ticket where current_user is also customer' do
- let(:ticket) { create(:ticket, customer: current_user, group: group_users) }
- include_examples 'shows attributes and values for agent view and editable'
- end
- end
- context 'as agent+customer with change permissions' do
- let!(:current_user) { create(:agent_and_customer) }
- before do
- current_user.group_names_access_map = {
- group_users.name => %w[read change],
- }
- end
- context 'normal ticket' do
- let(:ticket) { create(:ticket, group: group_users) }
- include_examples 'shows attributes and values for agent view and editable'
- end
- context 'ticket where current_user is also customer' do
- let(:ticket) { create(:ticket, customer: current_user, group: group_users) }
- include_examples 'shows attributes and values for agent view and editable'
- end
- end
- context 'as agent+customer with read permissions' do
- let!(:current_user) { create(:agent_and_customer) }
- before do
- current_user.group_names_access_map = {
- group_users.name => 'read',
- }
- end
- context 'normal ticket' do
- let(:ticket) { create(:ticket, group: group_users) }
- include_examples 'shows attributes and values for agent view but disabled'
- end
- context 'ticket where current_user is also customer' do
- let(:ticket) { create(:ticket, customer: current_user, group: group_users) }
- include_examples 'shows attributes and values for agent view but disabled'
- end
- end
- context 'as agent+customer but only customer for the ticket (no agent access)' do
- let!(:current_user) { create(:agent_and_customer) }
- let(:ticket) { create(:ticket, customer: current_user) }
- include_examples 'shows attributes and values for customer view'
- end
- end
- describe 'note visibility', authenticated_as: :customer do
- context 'when logged in as a customer' do
- let(:customer) { create(:customer) }
- let(:ticket) { create(:ticket, customer: customer) }
- let!(:ticket_article) { create(:ticket_article, ticket: ticket) }
- let!(:ticket_note) { create(:ticket_article, ticket: ticket, internal: true, type_name: 'note') }
- it 'previously created private note is not visible' do
- visit "ticket/zoom/#{ticket_article.ticket.id}"
- expect(page).to have_no_selector(:active_ticket_article, ticket_note)
- end
- it 'previously created private note shows up via WS push' do
- visit "ticket/zoom/#{ticket_article.ticket.id}"
- # make sure ticket is done loading and change will be pushed via WS
- find(:active_ticket_article, ticket_article)
- ticket_note.update!(internal: false)
- expect(page).to have_selector(:active_ticket_article, ticket_note)
- end
- end
- end
- # https://github.com/zammad/zammad/issues/3012
- describe 'article type selection' do
- context 'when logged in as a customer', authenticated_as: :customer do
- let(:customer) { create(:customer) }
- let(:ticket) { create(:ticket, customer: customer) }
- it 'hides button for single choice' do
- visit "ticket/zoom/#{ticket.id}"
- find('.articleNewEdit-body').send_keys('Some reply')
- expect(page).to have_no_selector('.js-selectedArticleType')
- end
- end
- context 'when logged in as an agent' do
- let(:ticket) { create(:ticket, group: Group.find_by(name: 'Users')) }
- it 'shows button for multiple choices' do
- visit "ticket/zoom/#{ticket.id}"
- find('.articleNewEdit-body').send_keys('Some reply')
- expect(page).to have_selector('.js-selectedArticleType')
- end
- end
- end
- # https://github.com/zammad/zammad/issues/3260
- describe 'next in overview macro changes URL', authenticated_as: :authenticate do
- let(:next_ticket) { create(:ticket, title: 'next Ticket', group: Group.first) }
- let(:macro) { create(:macro, name: 'next macro', ux_flow_next_up: 'next_from_overview') }
- def authenticate
- next_ticket && macro
- true
- end
- it 'to next Ticket ID' do
- visit 'ticket/view/all_unassigned'
- click_on 'Welcome to Zammad!'
- click '.js-openDropdownMacro'
- find(:macro, macro.id).click
- wait(5, interval: 1).until_constant { current_url }
- expect(current_url).to include("ticket/zoom/#{next_ticket.id}")
- end
- end
- # https://github.com/zammad/zammad/issues/3279
- describe 'previous/next clickability when at last or first ticket' do
- let(:ticket_a) { create(:ticket, title: 'ticket a', group: Group.first) }
- let(:ticket_b) { create(:ticket, title: 'ticket b', group: Group.first) }
- before do
- ticket_a && ticket_b
- visit 'ticket/view/all_unassigned'
- end
- it 'previous is not clickable for the first item' do
- open_nth_item(0)
- expect { click '.pagination .previous' }.not_to change { page.find('.content.active')[:id] }
- end
- it 'next is clickable for the first item' do
- open_nth_item(0)
- expect { click '.pagination .next' }.to change { page.find('.content.active')[:id] }
- end
- it 'previous is clickable for the middle item' do
- open_nth_item(1)
- expect { click '.pagination .previous' }.to change { page.find('.content.active')[:id] }
- end
- it 'next is clickable for the middle item' do
- open_nth_item(1)
- expect { click '.pagination .next' }.to change { page.find('.content.active')[:id] }
- end
- it 'previous is clickable for the last item' do
- open_nth_item(2)
- expect { click '.pagination .previous' }.to change { page.find('.content.active')[:id] }
- end
- it 'next is not clickable for the last item' do
- open_nth_item(2)
- expect { click '.pagination .next' }.not_to change { page.find('.content.active')[:id] }
- end
- def open_nth_item(nth)
- within :active_content do
- find_all('.table tr.item a[href^="#ticket/zoom"]')[nth].click
- end
- await_empty_ajax_queue
- end
- end
- # https://github.com/zammad/zammad/issues/3267
- describe 'previous/next buttons are added when open ticket is opened from overview' do
- let(:ticket_a) { create(:ticket, title: 'ticket a', group: Group.first) }
- let(:ticket_b) { create(:ticket, title: 'ticket b', group: Group.first) }
- # prepare an opened ticket and go to overview
- before do
- ticket_a && ticket_b
- visit "ticket/zoom/#{ticket_a.id}"
- visit 'ticket/view/all_unassigned'
- end
- it 'adds previous/next buttons to existing ticket' do
- within :active_content do
- click_on ticket_a.title
- expect(page).to have_css('.pagination-counter')
- end
- end
- it 'keeps previous/next buttons when navigating to overview ticket from elsewhere' do
- within :active_content do
- click_on ticket_a.title
- visit 'dashboard'
- visit "ticket/zoom/#{ticket_a.id}"
- expect(page).to have_css('.pagination-counter')
- end
- end
- end
- # https://github.com/zammad/zammad/issues/2942
- describe 'attachments are lost in specific conditions' do
- let(:ticket) { create(:ticket, group: Group.first) }
- it 'attachment is retained when forwarding a fresh article' do
- ensure_websocket do
- visit "ticket/zoom/#{ticket.id}"
- end
- # add an article, forcing reset of form_id
- # click in the upper most upper left corner of the article create textbox
- # (that works for both Firefox and Chrome)
- # to avoid clicking on attachment upload
- find('.js-writeArea').click({ x: 5, y: 5 })
- # wait for propagateOpenTextarea to be completed
- find('.attachmentPlaceholder-label').in_fixed_position
- expect(page).to have_no_css('.attachmentPlaceholder-hint')
- # write article content
- find('.articleNewEdit-body').send_keys('Some reply')
- click '.js-submit'
- # wait for article to be added to the page
- expect(page).to have_css('.ticket-article-item', count: 1)
- # create a on-the-fly article with attachment that will get pushed to open browser
- article1 = create(:ticket_article, ticket: ticket)
- Store.add(
- object: 'Ticket::Article',
- o_id: article1.id,
- data: 'some content',
- filename: 'some_file.txt',
- preferences: {
- 'Content-Type' => 'text/plain',
- },
- created_by_id: 1,
- )
- # wait for article to be added to the page
- expect(page).to have_css('.ticket-article-item', count: 2)
- # click on forward of created article
- within :active_ticket_article, article1 do
- find('a[data-type=emailForward]').click
- end
- # wait for propagateOpenTextarea to be completed
- find('.attachmentPlaceholder-label').in_fixed_position
- expect(page).to have_no_css('.attachmentPlaceholder-hint')
- # fill forward information and create article
- fill_in 'To', with: 'forward@example.org'
- find('.articleNewEdit-body').send_keys('Forwarding with the attachment')
- click '.js-submit'
- # wait for article to be added to the page
- expect(page).to have_css('.ticket-article-item', count: 3)
- # check if attachment was forwarded successfully
- within :active_ticket_article, ticket.reload.articles.last do
- within '.attachments--list' do
- expect(page).to have_text('some_file.txt')
- end
- end
- end
- end
- describe 'mentions' do
- context 'when logged in as agent' do
- let(:ticket) { create(:ticket, group: Group.find_by(name: 'Users')) }
- let!(:other_agent) { create(:agent, groups: [Group.find_by(name: 'Users')]) }
- it 'can subscribe and unsubscribe' do
- ensure_websocket do
- visit "ticket/zoom/#{ticket.id}"
- click '.js-subscriptions .js-subscribe input'
- expect(page).to have_selector('.js-subscriptions .js-unsubscribe input')
- expect(page).to have_selector('.js-subscriptions span.avatar')
- click '.js-subscriptions .js-unsubscribe input'
- expect(page).to have_selector('.js-subscriptions .js-subscribe input')
- expect(page).to have_no_selector('.js-subscriptions span.avatar')
- create(:mention, mentionable: ticket, user: other_agent)
- expect(page).to have_selector('.js-subscriptions span.avatar')
- # check history for mention entries
- click 'h2.sidebar-header-headline.js-headline'
- click 'li[data-type=ticket-history] a'
- expect(page).to have_text('created Mention')
- expect(page).to have_text('removed Mention')
- end
- end
- end
- end
- # https://github.com/zammad/zammad/issues/2671
- describe 'Pending time field in ticket sidebar', authenticated_as: :customer do
- let(:customer) { create(:customer) }
- let(:ticket) { create(:ticket, customer: customer, pending_time: 1.day.from_now, state: Ticket::State.lookup(name: 'pending reminder')) }
- it 'not shown to customer' do
- visit "ticket/zoom/#{ticket.id}"
- within :active_content do
- expect(page).to have_no_css('.controls[data-name=pending_time]')
- end
- end
- end
- describe 'Pending time field in ticket sidebar as agent' do
- before do
- ticket.update(pending_time: 1.day.from_now, state: Ticket::State.lookup(name: 'pending reminder'))
- visit "ticket/zoom/#{ticket.id}"
- sleep 3 # wait for popover killer to pass
- end
- let(:ticket) { Ticket.first }
- let(:elem) { find('.js-timepicker') }
- # has to run asynchronously to keep both Firefox and Safari
- # https://github.com/zammad/zammad/issues/3414
- # https://github.com/zammad/zammad/issues/2887
- context 'when clicking timepicker component' do
- it 'in the first half, hours selected' do
- within :active_content do
- elem.click({ x: 10, y: 20 })
- expect(elem).to have_selection(0..2)
- end
- end
- it 'in the second half, minutes selected' do
- within :active_content do
- elem.click({ x: 35, y: 20 })
- expect(elem).to have_selection(3..5)
- end
- end
- end
- matcher :have_selection do
- match { starts_at == expected.begin && ends_at == expected.end }
- def starts_at
- actual.evaluate_script 'this.selectionStart'
- end
- def ends_at
- actual.evaluate_script 'this.selectionEnd'
- end
- end
- end
- describe 'Article ID URL / link' do
- let(:ticket) { create(:ticket, group: Group.first) }
- let!(:article) { create(:'ticket/article', ticket: ticket) }
- it 'shows Article direct link' do
- ensure_websocket do
- visit "ticket/zoom/#{ticket.id}"
- end
- url = "#{Setting.get('http_type')}://#{Setting.get('fqdn')}/#ticket/zoom/#{ticket.id}/#{article.id}"
- within :active_ticket_article, article do
- expect(page).to have_css(%(a[href="#{url}"]))
- end
- end
- context 'when multiple Articles are present' do
- let(:article_count) { 20 }
- let(:article_top) { ticket.articles.second }
- let(:article_middle) { ticket.articles[ article_count / 2 ] }
- let(:article_bottom) { ticket.articles.last }
- before do
- article_count.times do
- create(:'ticket/article', ticket: ticket, body: SecureRandom.uuid)
- end
- visit "ticket/zoom/#{ticket.id}"
- end
- def wait_for_scroll
- wait(5, interval: 0.2).until_constant do
- find('.ticketZoom').native.location.y
- end
- end
- def check_shown(top: false, middle: false, bottom: false)
- wait_for_scroll
- expect(page).to have_css("div#article-content-#{article_top.id} .richtext-content", obscured: !top)
- .and(have_css("div#article-content-#{article_middle.id} .richtext-content", obscured: !middle, wait: 0))
- .and(have_css("div#article-content-#{article_bottom.id} .richtext-content", obscured: !bottom, wait: 0))
- end
- it 'scrolls to top article ID' do
- visit "ticket/zoom/#{ticket.id}/#{article_top.id}"
- check_shown(top: true)
- end
- it 'scrolls to middle article ID' do
- visit "ticket/zoom/#{ticket.id}/#{article_middle.id}"
- check_shown(middle: true)
- end
- it 'scrolls to bottom article ID' do
- visit "ticket/zoom/#{ticket.id}/#{article_top.id}"
- wait_for_scroll
- visit "ticket/zoom/#{ticket.id}/#{article_bottom.id}"
- check_shown(bottom: true)
- end
- end
- context 'when long articles are present' do
- it 'will properly show the "See more" link if you switch between the ticket and the dashboard on new articles' do
- ensure_websocket do
- # prerender ticket
- visit "ticket/zoom/#{ticket.id}"
- # ticket tab becomes background
- visit 'dashboard'
- end
- # create a new article
- article_id = create(:'ticket/article', ticket: ticket, body: "#{SecureRandom.uuid} #{"lorem ipsum\n" * 200}")
- wait(30).until { has_css?('div.tasks a.is-modified') }
- visit "ticket/zoom/#{ticket.id}"
- within :active_content do
- expect(find("div#article-content-#{article_id.id}")).to have_text('See more')
- end
- end
- end
- end
- describe 'Macros', authenticated_as: :authenticate do
- let(:macro_body) { 'macro <b>body</b>' }
- let(:macro) { create :macro, perform: { 'article.note' => { 'body' => macro_body, 'internal' => 'true', 'subject' => 'macro note' } } }
- let!(:ticket) { create(:ticket, group: Group.find_by(name: 'Users')) }
- def authenticate
- macro
- true
- end
- it 'does html macro by default' do
- visit "ticket/zoom/#{ticket.id}"
- find('.js-openDropdownMacro').click
- find(:macro, macro.id).click
- expect(ticket.reload.articles.last.body).to eq(macro_body)
- expect(ticket.reload.articles.last.content_type).to eq('text/html')
- end
- end
- describe 'object manager attributes maxlength', authenticated_as: :authenticate, db_strategy: :reset do
- let(:ticket) { create(:ticket, group: Group.find_by(name: 'Users')) }
- def authenticate
- ticket
- create :object_manager_attribute_text, name: 'maxtest', display: 'maxtest', screens: attributes_for(:required_screen), data_option: {
- 'type' => 'text',
- 'maxlength' => 3,
- 'null' => true,
- 'translate' => false,
- 'default' => '',
- 'options' => {},
- 'relation' => '',
- }
- ObjectManager::Attribute.migration_execute
- true
- end
- it 'checks ticket zoom' do
- visit "ticket/zoom/#{ticket.id}"
- within(:active_content) do
- fill_in 'maxtest', with: 'hellu'
- expect(page.find_field('maxtest').value).to eq('hel')
- end
- end
- end
- describe 'GitLab Integration', :integration, authenticated_as: :authenticate, required_envs: %w[GITLAB_ENDPOINT GITLAB_APITOKEN] do
- let!(:ticket) { create(:ticket, group: Group.find_by(name: 'Users')) }
- def authenticate
- Setting.set('gitlab_integration', true)
- Setting.set('gitlab_config', {
- api_token: ENV['GITLAB_APITOKEN'],
- endpoint: ENV['GITLAB_ENDPOINT'],
- })
- true
- end
- it 'creates links and removes them' do
- visit "#ticket/zoom/#{ticket.id}"
- within(:active_content) do
- # switch to GitLab sidebar
- click('.tabsSidebar-tab[data-tab=gitlab]')
- click('.sidebar-header-headline.js-headline')
- # add issue
- click_on 'Link issue'
- fill_in 'link', with: ENV['GITLAB_ISSUE_LINK']
- click_on 'Submit'
- # verify issue
- content = find('.sidebar-git-issue-content')
- expect(content).to have_text('#1 Example issue')
- expect(content).to have_text('critical')
- expect(content).to have_text('special')
- expect(content).to have_text('important milestone')
- expect(content).to have_text('zammad-robot')
- expect(ticket.reload.preferences[:gitlab][:issue_links][0]).to eq(ENV['GITLAB_ISSUE_LINK'])
- # check sidebar counter increased to 1
- expect(find('.tabsSidebar-tab[data-tab=gitlab] .js-tabCounter')).to have_text('1')
- # delete issue
- click(".sidebar-git-issue-delete span[data-issue-id='#{ENV['GITLAB_ISSUE_LINK']}']")
- content = find('.sidebar[data-tab=gitlab] .sidebar-content')
- expect(content).to have_text('No linked issues')
- expect(ticket.reload.preferences[:gitlab][:issue_links][0]).to be nil
- # check that counter got removed
- expect(page).to have_no_selector('.tabsSidebar-tab[data-tab=gitlab] .js-tabCounter')
- end
- end
- end
- describe 'GitHub Integration', :integration, authenticated_as: :authenticate, required_envs: %w[GITHUB_ENDPOINT GITHUB_APITOKEN] do
- let!(:ticket) { create(:ticket, group: Group.find_by(name: 'Users')) }
- def authenticate
- Setting.set('github_integration', true)
- Setting.set('github_config', {
- api_token: ENV['GITHUB_APITOKEN'],
- endpoint: ENV['GITHUB_ENDPOINT'],
- })
- true
- end
- it 'creates links and removes them' do
- visit "#ticket/zoom/#{ticket.id}"
- within(:active_content) do
- # switch to GitHub sidebar
- click('.tabsSidebar-tab[data-tab=github]')
- click('.sidebar-header-headline.js-headline')
- # add issue
- click_on 'Link issue'
- fill_in 'link', with: ENV['GITHUB_ISSUE_LINK']
- click_on 'Submit'
- # verify issue
- content = find('.sidebar-git-issue-content')
- expect(content).to have_text('#1575 GitHub integration')
- expect(content).to have_text('feature backlog')
- expect(content).to have_text('integration')
- expect(content).to have_text('4.0')
- expect(content).to have_text('Thorsten')
- expect(ticket.reload.preferences[:github][:issue_links][0]).to eq(ENV['GITHUB_ISSUE_LINK'])
- # check sidebar counter increased to 1
- expect(find('.tabsSidebar-tab[data-tab=github] .js-tabCounter')).to have_text('1')
- # delete issue
- click(".sidebar-git-issue-delete span[data-issue-id='#{ENV['GITHUB_ISSUE_LINK']}']")
- content = find('.sidebar[data-tab=github] .sidebar-content')
- expect(content).to have_text('No linked issues')
- expect(ticket.reload.preferences[:github][:issue_links][0]).to be nil
- # check that counter got removed
- expect(page).to have_no_selector('.tabsSidebar-tab[data-tab=github] .js-tabCounter')
- end
- end
- end
- describe 'Core Workflow' do
- include_examples 'core workflow' do
- let(:ticket) { create(:ticket, group: Group.find_by(name: 'Users')) }
- let(:object_name) { 'Ticket' }
- let(:before_it) do
- lambda {
- ensure_websocket(check_if_pinged: false) do
- visit "#ticket/zoom/#{ticket.id}"
- end
- }
- end
- end
- end
- context 'Sidebar - Open & Closed Tickets', searchindex: true, authenticated_as: :authenticate do
- let(:customer) { create(:customer, :with_org) }
- let(:ticket_open) { create(:ticket, group: Group.find_by(name: 'Users'), customer: customer, title: SecureRandom.uuid) }
- let(:ticket_closed) { create(:ticket, group: Group.find_by(name: 'Users'), customer: customer, state: Ticket::State.find_by(name: 'closed'), title: SecureRandom.uuid) }
- def authenticate
- ticket_open
- ticket_closed
- configure_elasticsearch(required: true, rebuild: true)
- Scheduler.worker(true)
- true
- end
- it 'does show open and closed tickets in advanced search url' do
- visit "#ticket/zoom/#{ticket_open.id}"
- click '.tabsSidebar-tab[data-tab=customer]'
- click '.user-tickets[data-type=open]'
- expect(page).to have_text(ticket_open.title)
- visit "#ticket/zoom/#{ticket_open.id}"
- click '.user-tickets[data-type=closed]'
- expect(page).to have_text(ticket_closed.title)
- end
- end
- context 'Sidebar - Organization' do
- let(:organization) { create(:organization) }
- context 'members section' do
- let(:customers) { create_list(:customer, 50, organization: organization) }
- let(:ticket) { create(:ticket, group: Group.find_by(name: 'Users'), customer: customers.first) }
- let(:members) { organization.members.order(id: :asc) }
- before do
- visit "#ticket/zoom/#{ticket.id}"
- click '.tabsSidebar-tab[data-tab=organization]'
- end
- it 'shows first 10 members and loads more on demand' do
- expect(page).to have_text(members[9].fullname)
- expect(page).to have_no_text(members[10].fullname)
- click '.js-showMoreMembers'
- expect(page).to have_text(members[10].fullname)
- end
- end
- end
- describe 'merging', authenticated_as: :user do
- before do
- merged_into_trigger && received_merge_trigger && update_trigger
- visit "ticket/zoom/#{ticket.id}"
- visit "ticket/zoom/#{target_ticket.id}"
- ensure_websocket do
- visit 'dashboard'
- end
- end
- let(:merged_into_trigger) { create(:trigger, :conditionable, condition_ticket_action: :merged_into) }
- let(:received_merge_trigger) { create(:trigger, :conditionable, condition_ticket_action: :received_merge) }
- let(:update_trigger) { create(:trigger, :conditionable, condition_ticket_action: :update) }
- let(:ticket) { create(:ticket) }
- let(:target_ticket) { create(:ticket) }
- let(:user) { create(:agent, :preferencable, notification_group_ids: [ticket, target_ticket].map(&:group_id), groups: [ticket, target_ticket].map(&:group)) }
- context 'when merging ticket' do
- before do
- ticket.merge_to(ticket_id: target_ticket.id, user_id: 1)
- end
- it 'pulses source ticket' do
- expect(page).to have_css("#navigation a.is-modified[data-key=\"Ticket-#{ticket.id}\"]")
- end
- it 'pulses target ticket' do
- expect(page).to have_css("#navigation a.is-modified[data-key=\"Ticket-#{target_ticket.id}\"]")
- end
- end
- context 'when merging and looking at online notifications', :performs_jobs do
- before do
- perform_enqueued_jobs do
- ticket.merge_to(ticket_id: target_ticket.id, user_id: 1)
- end
- find('.js-toggleNotifications').click
- end
- it 'shows online notification for source ticket' do
- expect(page).to have_text("Ticket #{ticket.title} was merged into another ticket")
- end
- it 'shows online notification for target ticket' do
- expect(page).to have_text("Another ticket was merged into ticket #{ticket.title}")
- end
- end
- end
- describe 'Tab behaviour - Define default "stay on tab" / "close tab" behavior #257', authenticated_as: :authenticate do
- def authenticate
- Setting.set('ticket_secondary_action', 'closeTabOnTicketClose')
- true
- end
- let!(:ticket) { create(:ticket, group: Group.find_by(name: 'Users')) }
- before do
- visit "ticket/zoom/#{ticket.id}"
- end
- it 'does show the default of the system' do
- expect(page).to have_text('Close tab on ticket close')
- end
- it 'does save state for the user preferences' do
- click '.js-attributeBar .dropup div'
- click 'span[data-type=stayOnTab]'
- refresh
- expect(page).to have_text('Stay on tab')
- expect(User.find_by(email: 'admin@example.com').preferences[:secondaryAction]).to eq('stayOnTab')
- end
- context 'Tab behaviour - Close tab on ticket close' do
- it 'does not close the tab without any action' do
- click '.js-submit'
- expect(current_url).to include('ticket/zoom')
- end
- it 'does close the tab on ticket close' do
- select 'closed', from: 'State'
- click '.js-submit'
- expect(current_url).not_to include('ticket/zoom')
- end
- end
- context 'Tab behaviour - Stay on tab' do
- def authenticate
- Setting.set('ticket_secondary_action', 'stayOnTab')
- true
- end
- it 'does not close the tab without any action' do
- click '.js-submit'
- expect(current_url).to include('ticket/zoom')
- end
- it 'does not close the tab on ticket close' do
- select 'closed', from: 'State'
- click '.js-submit'
- expect(current_url).to include('ticket/zoom')
- end
- end
- context 'Tab behaviour - Close tab' do
- def authenticate
- Setting.set('ticket_secondary_action', 'closeTab')
- true
- end
- it 'does close the tab without any action' do
- click '.js-submit'
- expect(current_url).not_to include('ticket/zoom')
- end
- it 'does close the tab on ticket close' do
- select 'closed', from: 'State'
- click '.js-submit'
- expect(current_url).not_to include('ticket/zoom')
- end
- end
- context 'Tab behaviour - Next in overview' do
- let(:ticket1) { create(:ticket, title: SecureRandom.uuid, group: Group.find_by(name: 'Users')) }
- let(:ticket2) { create(:ticket, title: SecureRandom.uuid, group: Group.find_by(name: 'Users')) }
- let(:ticket3) { create(:ticket, title: SecureRandom.uuid, group: Group.find_by(name: 'Users')) }
- def authenticate
- Setting.set('ticket_secondary_action', 'closeNextInOverview')
- ticket1
- ticket2
- ticket3
- true
- end
- before do
- visit 'ticket/view/all_open'
- end
- it 'does change the tab without any action' do
- click_on ticket1.title
- expect(current_url).to include("ticket/zoom/#{ticket1.id}")
- click '.js-submit'
- expect(current_url).to include("ticket/zoom/#{ticket2.id}")
- click '.js-submit'
- expect(current_url).to include("ticket/zoom/#{ticket3.id}")
- end
- it 'does show default stay on tab if secondary action is not given' do
- click_on ticket1.title
- refresh
- expect(page).to have_text('Stay on tab')
- end
- end
- context 'On ticket switch' do
- let(:ticket1) { create(:ticket, title: SecureRandom.uuid, group: Group.find_by(name: 'Users')) }
- let(:ticket2) { create(:ticket, title: SecureRandom.uuid, group: Group.find_by(name: 'Users')) }
- before do
- visit "ticket/zoom/#{ticket1.id}"
- visit "ticket/zoom/#{ticket2.id}"
- end
- it 'does setup the last behaviour' do
- click '.js-attributeBar .dropup div'
- click 'span[data-type=stayOnTab]'
- wait.until do
- User.find_by(email: 'admin@example.com').preferences['secondaryAction'] == 'stayOnTab'
- end
- visit "ticket/zoom/#{ticket1.id}"
- expect(page).to have_text('Stay on tab')
- end
- end
- end
- describe 'Core Workflow: Show hidden attributes on group selection (ticket edit) #3739', authenticated_as: :authenticate, db_strategy: :reset do
- let!(:ticket) { create(:ticket, group: Group.find_by(name: 'Users')) }
- let(:field_name) { SecureRandom.uuid }
- let(:field) do
- create :object_manager_attribute_text, name: field_name, display: field_name, screens: {
- 'edit' => {
- 'ticket.agent' => {
- 'shown' => false,
- 'required' => false,
- }
- }
- }
- ObjectManager::Attribute.migration_execute
- end
- before do
- visit "#ticket/zoom/#{ticket.id}"
- end
- context 'when field visible' do
- let(:workflow) do
- create(:core_workflow,
- object: 'Ticket',
- perform: { "ticket.#{field_name}" => { 'operator' => 'show', 'show' => 'true' } })
- end
- def authenticate
- field
- workflow
- true
- end
- it 'does show up the field' do
- expect(page).to have_css("div[data-attribute-name='#{field_name}']")
- end
- end
- context 'when field hidden' do
- def authenticate
- field
- true
- end
- it 'does not show the field' do
- expect(page).to have_css("div[data-attribute-name='#{field_name}'].is-hidden", visible: :hidden)
- end
- end
- end
- describe 'Notes on existing ticks are discarded by editing profile settings #3088' do
- let(:ticket) { create(:ticket, group: Group.find_by(name: 'Users')) }
- before do
- visit "#ticket/zoom/#{ticket.id}"
- end
- def upload_and_set_text
- page.find('.js-textarea').send_keys("Hello\nThis\nis\nimportant!\nyo\nhoho\ntest test test test")
- page.find('input#fileUpload_1', visible: :all).set(Rails.root.join('test/data/mail/mail001.box'))
- expect(page).to have_text('mail001.box')
- wait_for_upload_present
- end
- def wait_for_upload_present
- wait.until { Taskbar.find_by(key: "Ticket-#{ticket.id}").attributes_with_association_ids['attachments'].present? }
- end
- def wait_for_upload_blank
- wait.until { Taskbar.find_by(key: "Ticket-#{ticket.id}").attributes_with_association_ids['attachments'].blank? }
- end
- def switch_language_german
- visit '#profile/language'
- # Suppress the modal dialog that invites to contributions for translations that are < 90% as this breaks the tests for de-de.
- page.evaluate_script "App.LocalStorage.set('translation_support_no', true, App.Session.get('id'))"
- page.find('.js-input').click
- page.find('.js-input').set('Deutsch')
- page.find('.js-input').send_keys(:enter)
- click_on 'Submit'
- visit "#ticket/zoom/#{ticket.id}"
- expect(page).to have_text(Translation.translate('de-de', 'select attachment…'))
- end
- def expect_upload_and_text
- expect(page.find('.article-new')).to have_text('mail001.box')
- expect(page.find('.article-new')).to have_text("Hello\nThis\nis\nimportant!\nyo\nhoho\ntest test test test")
- end
- def expect_no_upload_and_text
- expect(page.find('.article-new')).to have_no_text('mail001.box')
- expect(page.find('.article-new')).to have_no_text("Hello\nThis\nis\nimportant!\nyo\nhoho\ntest test test test")
- end
- it 'does show up the attachments after a reload of the page' do
- upload_and_set_text
- expect_upload_and_text
- refresh
- expect_upload_and_text
- end
- it 'does show up the attachments after updating language (ui:rerender event)' do
- upload_and_set_text
- expect_upload_and_text
- switch_language_german
- expect_upload_and_text
- end
- it 'does remove attachments and text on reset' do
- upload_and_set_text
- expect_upload_and_text
- page.find('.js-reset').click
- wait_for_upload_blank
- expect_no_upload_and_text
- refresh
- expect_no_upload_and_text
- end
- context 'when rerendering (#3831)' do
- def rerender
- page.evaluate_script("App.Event.trigger('ui:rerender')")
- end
- it 'does loose attachments after rerender' do
- upload_and_set_text
- expect_upload_and_text
- rerender
- expect_upload_and_text
- end
- it 'does not readd the attachments after reset' do
- upload_and_set_text
- expect_upload_and_text
- page.find('.js-reset').click
- wait_for_upload_blank
- expect_no_upload_and_text
- rerender
- expect_no_upload_and_text
- end
- it 'does not readd the attachments after submit' do
- upload_and_set_text
- expect_upload_and_text
- page.find('.js-submit').click
- wait_for_upload_blank
- expect_no_upload_and_text
- rerender
- expect_no_upload_and_text
- end
- it 'does not show the ticket as changed after the upload removal' do
- page.find('input#fileUpload_1', visible: :all).set(Rails.root.join('test/data/mail/mail001.box'))
- expect(page.find('.article-new')).to have_text('mail001.box')
- wait_for_upload_present
- begin
- page.evaluate_script("$('div.attachment-delete.js-delete:last').trigger('click')") # not interactable
- rescue # Lint/SuppressedException
- # because its not interactable it also
- # returns this weird exception for the jquery
- # even tho it worked fine
- end
- expect(page).to have_no_selector('.js-reset')
- end
- end
- end
- describe 'Unable to close tickets in certran cases if core workflow is used #3710', authenticated_as: :authenticate, db_strategy: :reset do
- let!(:ticket) { create(:ticket, group: Group.find_by(name: 'Users')) }
- let(:field_name) { SecureRandom.uuid }
- let(:field) do
- create :object_manager_attribute_text, name: field_name, display: field_name, screens: {
- 'edit' => {
- 'ticket.agent' => {
- 'shown' => false,
- 'required' => false,
- }
- }
- }
- ObjectManager::Attribute.migration_execute
- end
- let(:workflow) do
- create(:core_workflow,
- object: 'Ticket',
- perform: { "ticket.#{field_name}" => { 'operator' => 'set_mandatory', 'set_mandatory' => 'true' } })
- end
- def authenticate
- field
- workflow
- true
- end
- before do
- visit "#ticket/zoom/#{ticket.id}"
- end
- it 'does save the ticket because the field is mandatory but hidden' do
- admin = User.find_by(email: 'admin@example.com')
- select admin.fullname, from: 'Owner'
- find('.js-submit').click
- expect(ticket.reload.owner_id).to eq(admin.id)
- end
- end
- describe "escaped 'Set fixed' workflows don't refresh set values on active ticket sessions #3757", authenticated_as: :authenticate, db_strategy: :reset do
- let(:field_name) { SecureRandom.uuid }
- let(:ticket) { create(:ticket, group: Group.find_by(name: 'Users'), field_name => false) }
- def authenticate
- workflow
- create :object_manager_attribute_boolean, name: field_name, display: field_name, screens: attributes_for(:required_screen)
- ObjectManager::Attribute.migration_execute
- ticket
- true
- end
- before do
- visit "#ticket/zoom/#{ticket.id}"
- end
- context 'when operator set_fixed_to' do
- let(:workflow) do
- create(:core_workflow,
- object: 'Ticket',
- perform: { "ticket.#{field_name}" => { 'operator' => 'set_fixed_to', 'set_fixed_to' => ['false'] } })
- end
- context 'when saved value is removed by set_fixed_to operator' do
- it 'does show up the saved value if it would not be possible because of the restriction' do
- expect(page.find("select[name='#{field_name}']").value).to eq('false')
- ticket.update(field_name => true)
- wait.until { page.find("select[name='#{field_name}']").value == 'true' }
- expect(page.find("select[name='#{field_name}']").value).to eq('true')
- end
- end
- end
- context 'when operator remove_option' do
- let(:workflow) do
- create(:core_workflow,
- object: 'Ticket',
- perform: { "ticket.#{field_name}" => { 'operator' => 'remove_option', 'remove_option' => ['true'] } })
- end
- context 'when saved value is removed by set_fixed_to operator' do
- it 'does show up the saved value if it would not be possible because of the restriction' do
- expect(page.find("select[name='#{field_name}']").value).to eq('false')
- ticket.update(field_name => true)
- wait.until { page.find("select[name='#{field_name}']").value == 'true' }
- expect(page.find("select[name='#{field_name}']").value).to eq('true')
- end
- end
- end
- end
- context 'Basic sidebar handling because of regressions in #3757' do
- let(:ticket) { create(:ticket, group: Group.find_by(name: 'Users')) }
- before do
- visit "#ticket/zoom/#{ticket.id}"
- end
- it 'does show up the new priority' do
- high_prio = Ticket::Priority.find_by(name: '3 high')
- ticket.update(priority: high_prio)
- wait.until { page.find("select[name='priority_id']").value == high_prio.id.to_s }
- expect(page.find("select[name='priority_id']").value).to eq(high_prio.id.to_s)
- end
- it 'does show up the new group (different case because it will also trigger a full rerender because of potential permission changes)' do
- group = Group.find_by(name: 'some group1')
- ticket.update(group: group)
- wait.until { page.find("select[name='group_id']").value == group.id.to_s }
- expect(page.find("select[name='group_id']").value).to eq(group.id.to_s)
- end
- it 'does show up the new state and pending time' do
- pending_state = Ticket::State.find_by(name: 'pending reminder')
- ticket.update(state: pending_state, pending_time: 1.day.from_now)
- wait.until { page.find("select[name='state_id']").value == pending_state.id.to_s }
- expect(page.find("select[name='state_id']").value).to eq(pending_state.id.to_s)
- expect(page).to have_selector("div[data-name='pending_time']")
- end
- it 'does merge attributes with remote priority (ajax) and local state (user)' do
- select 'closed', from: 'State'
- high_prio = Ticket::Priority.find_by(name: '3 high')
- closed_state = Ticket::State.find_by(name: 'closed')
- ticket.update(priority: high_prio)
- wait.until { page.find("select[name='priority_id']").value == high_prio.id.to_s }
- expect(page.find("select[name='priority_id']").value).to eq(high_prio.id.to_s)
- expect(page.find("select[name='state_id']").value).to eq(closed_state.id.to_s)
- end
- context 'when 2 users are in 2 different tickets' do
- let(:ticket2) { create(:ticket, group: Group.find_by(name: 'Users')) }
- let(:agent2) { create(:agent, password: 'test', groups: [Group.find_by(name: 'Users')]) }
- before do
- using_session(:second_browser) do
- login(
- username: agent2.login,
- password: 'test',
- )
- visit "#ticket/zoom/#{ticket.id}"
- visit "#ticket/zoom/#{ticket2.id}"
- end
- end
- it 'does not make any changes to the second browser ticket' do
- closed_state = Ticket::State.find_by(name: 'closed')
- select 'closed', from: 'State'
- find('.js-submit').click
- using_session(:second_browser) do
- sleep 3
- expect(page.find("select[name='state_id']").value).not_to eq(closed_state.id.to_s)
- end
- end
- end
- end
- context 'Article box opening on tickets with no changes #3789' do
- let(:ticket) { create(:ticket, group: Group.find_by(name: 'Users')) }
- before do
- visit "#ticket/zoom/#{ticket.id}"
- end
- it 'does not expand the article box without changes' do
- refresh
- sleep 3
- expect(page).to have_no_selector('form.article-add.is-open')
- end
- it 'does open and close by usage' do
- find('.js-textarea').send_keys(' ')
- expect(page).to have_selector('form.article-add.is-open')
- find('input#global-search').click
- expect(page).to have_no_selector('form.article-add.is-open')
- end
- it 'does open automatically when body is given from sidebar' do
- find('.js-textarea').send_keys('test')
- wait.until { Taskbar.find_by(key: "Ticket-#{ticket.id}").state.dig('article', 'body').present? }
- refresh
- expect(page).to have_selector('form.article-add.is-open')
- end
- it 'does open automatically when attachment is given from sidebar' do
- page.find('input#fileUpload_1', visible: :all).set(Rails.root.join('test/data/mail/mail001.box'))
- wait.until { Taskbar.find_by(key: "Ticket-#{ticket.id}").attributes_with_association_ids['attachments'].present? }
- refresh
- expect(page).to have_selector('form.article-add.is-open')
- end
- end
- context 'Owner should get cleared if not listed in changed group #3818', authenticated_as: :authenticate do
- let(:group1) { create(:group) }
- let(:group2) { create(:group) }
- let(:agent1) { create(:agent) }
- let(:agent2) { create(:agent) }
- let(:ticket) { create(:ticket, group: group1, owner: agent1) }
- def authenticate
- agent1.group_names_access_map = {
- group1.name => 'full',
- group2.name => %w[read change overview]
- }
- agent2.group_names_access_map = {
- group1.name => 'full',
- group2.name => 'full',
- }
- agent1
- end
- before do
- visit "#ticket/zoom/#{ticket.id}"
- end
- it 'does clear agent1 on select of group 2' do
- select group2.name, from: 'Group'
- wait.until { page.find('select[name=owner_id]').value != agent1.id.to_s }
- expect(page.find('select[name=owner_id]').value).to eq('')
- expect(page.all('select[name=owner_id] option').map(&:value)).not_to include(agent1.id.to_s)
- expect(page.all('select[name=owner_id] option').map(&:value)).to include(agent2.id.to_s)
- end
- end
- describe 'Not displayed fields should not impact the edit screen #3819', authenticated_as: :authenticate, db_strategy: :reset do
- let(:field_name) { SecureRandom.uuid }
- let(:ticket) { create(:ticket, group: Group.find_by(name: 'Users')) }
- def authenticate
- create :object_manager_attribute_boolean, default: nil, screens: {
- edit: {
- 'ticket.agent' => {
- shown: false,
- required: false,
- }
- }
- }
- ObjectManager::Attribute.migration_execute
- ticket
- true
- end
- before do
- visit "#ticket/zoom/#{ticket.id}"
- end
- it 'does not show any changes for the field because it has no value and because it is not shown it should also not show the ticket as changed' do
- sleep 3
- expect(page).to have_no_selector('.js-reset')
- end
- end
- describe 'Changing ticket status from "new" to any other status always results in uncommited status "closed" #3880', authenticated_as: :authenticate do
- let(:ticket) { create(:ticket, group: Group.find_by(name: 'Users')) }
- let(:workflow) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'ticket.priority_id': {
- operator: 'is',
- value: [ Ticket::Priority.find_by(name: '3 high').id.to_s ],
- },
- },
- perform: { 'ticket.state_id' => { operator: 'remove_option', remove_option: [ Ticket::State.find_by(name: 'pending reminder').id.to_s ] } })
- end
- def authenticate
- workflow
- true
- end
- before do
- visit "#ticket/zoom/#{ticket.id}"
- end
- it 'does switch back to the saved value in the ticket instead of the first value of the dropdown' do
- page.select 'pending reminder', from: 'state_id'
- page.select '3 high', from: 'priority_id'
- expect(page).to have_select('state_id', selected: 'new')
- end
- end
- describe 'Multiselect displaying and saving', authenticated_as: :authenticate, db_strategy: :reset do
- let(:field_name) { SecureRandom.uuid }
- let(:ticket) { create(:ticket, group: Group.find_by(name: 'Users'), field_name => %w[key_2 key_3]) }
- def authenticate
- create :object_manager_attribute_multiselect, name: field_name, display: field_name, screens: {
- 'edit' => {
- 'ticket.agent' => {
- 'shown' => true,
- 'required' => false,
- }
- }
- }
- ObjectManager::Attribute.migration_execute
- ticket
- true
- end
- before do
- visit "#ticket/zoom/#{ticket.id}"
- end
- def multiselect_value
- page.find("select[name='#{field_name}']").value
- end
- def multiselect_set(values)
- multiselect_unset_all
- values = Array(values)
- values.each do |value|
- page.find("select[name='#{field_name}']").select(value)
- end
- end
- def multiselect_unset_all
- values = page.all("select[name='#{field_name}'] option").map(&:text)
- values.each do |value|
- page.find("select[name='#{field_name}']").unselect(value)
- end
- end
- it 'does show values properly and can save values also' do
- # check ticket state rendering
- wait.until { multiselect_value == %w[key_2 key_3] }
- expect(multiselect_value).to eq(%w[key_2 key_3])
- # save 2 values
- multiselect_set(%w[value_1 value_2])
- click '.js-submit'
- wait.until { ticket.reload[field_name] == %w[key_1 key_2] }
- # save 1 value
- multiselect_set(['value_1'])
- click '.js-submit'
- wait.until { ticket.reload[field_name] == ['key_1'] }
- # unset all values
- multiselect_unset_all
- click '.js-submit'
- wait.until { ticket.reload[field_name].nil? }
- end
- end
- describe 'Add confirmation dialog on visibility change of an article or in article creation #3924', authenticated_as: :authenticate do
- let(:ticket) { create(:ticket, group: Group.find_by(name: 'Users')) }
- let(:article) { create(:ticket_article, ticket: ticket) }
- before do
- visit "#ticket/zoom/#{article.ticket.id}"
- end
- context 'when dialog is disabled' do
- def authenticate
- true
- end
- it 'does set the article internal and external for existing articles' do
- expect { page.find('.js-ArticleAction[data-type=internal]').click }.to change { article.reload.internal }.to(true)
- expect { page.find('.js-ArticleAction[data-type=public]').click }.to change { article.reload.internal }.to(false)
- end
- it 'does set the article internal and external for new article' do
- page.find('.js-writeArea').click({ x: 5, y: 5 })
- expect(page).to have_css('.article-new .icon-internal')
- expect(page).to have_no_css('.article-new .icon-public')
- page.find('.article-new .icon-internal').click
- expect(page).to have_no_css('.article-new .icon-internal')
- expect(page).to have_css('.article-new .icon-public')
- page.find('.article-new .icon-public').click
- expect(page).to have_css('.article-new .icon-internal')
- expect(page).to have_no_css('.article-new .icon-public')
- end
- end
- context 'when dialog is enabled' do
- def authenticate
- Setting.set('ui_ticket_zoom_article_visibility_confirmation_dialog', true)
- true
- end
- it 'does set the article internal and external for existing articles' do
- expect { page.find('.js-ArticleAction[data-type=internal]').click }.to change { article.reload.internal }.to(true)
- page.find('.js-ArticleAction[data-type=public]').click
- expect(page).to have_css('.modal-dialog')
- expect { find('.modal-dialog button[type=submit]').click }.to change { article.reload.internal }.to(false)
- end
- it 'does set the article internal and external for new article' do
- page.find('.js-writeArea').click({ x: 5, y: 5 })
- expect(page).to have_css('.article-new .icon-internal')
- expect(page).to have_no_css('.article-new .icon-public')
- page.find('.article-new .icon-internal').click
- expect(page).to have_css('.modal-dialog')
- find('.modal-dialog button[type=submit]').click
- expect(page).to have_no_css('.article-new .icon-internal')
- expect(page).to have_css('.article-new .icon-public')
- page.find('.article-new .icon-public').click
- expect(page).to have_css('.article-new .icon-internal')
- expect(page).to have_no_css('.article-new .icon-public')
- end
- end
- end
- describe 'Show which escalation type escalated in ticket zoom #3928', authenticated_as: :authenticate do
- let(:sla) { create(:sla, first_response_time: 1, update_time: 1, solution_time: 1) }
- let(:ticket) { create(:ticket, group: Group.find_by(name: 'Users')) }
- def authenticate
- sla
- true
- end
- before do
- visit "#ticket/zoom/#{ticket.id}"
- end
- it 'does show the extended escalation information' do
- sleep 4 # wait for popup killer
- page.find('.escalation-popover').hover
- expect(page).to have_text('FIRST RESPONSE TIME')
- expect(page).to have_text('UPDATE TIME')
- expect(page).to have_text('SOLUTION TIME')
- end
- end
- end
|