123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675676677678679680681682683684685686687688689690691692693694695696697698699700701702703704705706707708709710711712713714715716717718719720721722723724725726727728729730731732733734735736737738739740741742743744745746747748749750751752753754755756757758759760761762763764765766767768769770771772773774775776777778779780781782783784785786787788789790791792793794795796797798799800801802803804805806807808809810811812813814815816817818819820821822823824825826827828829830831832833834835836837838839840841842843844845846847848849850851852853854855856857858859860861862863864865866867868869870871872873874875876877878879880881882883884885886887888889890891892893894895896897898899900901902903904905906907908909910911912913914915916917918919920921922923924925926927928929930931932933934935936937938939940941942943944945946947948949950951952953954955956957958959960961962963964965966967968969970971972973974975976977978979980981982983984985986987988989990991992993994995996997998999100010011002100310041005100610071008100910101011101210131014101510161017101810191020102110221023102410251026102710281029103010311032103310341035103610371038103910401041104210431044104510461047104810491050105110521053105410551056105710581059106010611062106310641065106610671068106910701071107210731074107510761077107810791080108110821083108410851086108710881089109010911092109310941095109610971098109911001101110211031104110511061107110811091110111111121113111411151116111711181119112011211122112311241125112611271128112911301131113211331134113511361137113811391140114111421143114411451146114711481149115011511152115311541155115611571158115911601161116211631164116511661167116811691170117111721173117411751176117711781179118011811182118311841185118611871188118911901191119211931194119511961197119811991200120112021203120412051206120712081209121012111212121312141215121612171218121912201221122212231224122512261227122812291230123112321233123412351236123712381239124012411242124312441245124612471248124912501251125212531254125512561257125812591260126112621263126412651266126712681269127012711272127312741275127612771278127912801281128212831284128512861287128812891290129112921293129412951296129712981299130013011302130313041305130613071308130913101311131213131314131513161317131813191320132113221323132413251326132713281329133013311332133313341335133613371338133913401341134213431344134513461347134813491350135113521353135413551356135713581359136013611362136313641365136613671368136913701371137213731374137513761377137813791380138113821383138413851386138713881389139013911392139313941395139613971398139914001401140214031404140514061407140814091410141114121413141414151416141714181419142014211422142314241425142614271428142914301431143214331434143514361437143814391440144114421443144414451446144714481449145014511452145314541455145614571458145914601461146214631464146514661467146814691470147114721473147414751476147714781479148014811482148314841485148614871488148914901491149214931494149514961497149814991500150115021503150415051506150715081509151015111512151315141515151615171518151915201521152215231524152515261527152815291530153115321533153415351536153715381539154015411542154315441545154615471548154915501551155215531554155515561557155815591560156115621563156415651566156715681569157015711572157315741575157615771578157915801581158215831584158515861587158815891590159115921593159415951596159715981599160016011602160316041605160616071608160916101611161216131614161516161617161816191620162116221623162416251626162716281629163016311632163316341635163616371638163916401641164216431644164516461647164816491650165116521653165416551656165716581659166016611662166316641665166616671668166916701671167216731674167516761677167816791680168116821683168416851686168716881689169016911692169316941695169616971698169917001701170217031704170517061707170817091710171117121713171417151716171717181719172017211722172317241725172617271728172917301731173217331734173517361737173817391740174117421743174417451746174717481749175017511752175317541755175617571758175917601761176217631764176517661767176817691770177117721773177417751776177717781779178017811782178317841785178617871788178917901791179217931794179517961797179817991800180118021803180418051806180718081809181018111812181318141815181618171818181918201821182218231824182518261827182818291830183118321833183418351836183718381839184018411842184318441845184618471848184918501851185218531854185518561857185818591860186118621863186418651866186718681869187018711872187318741875187618771878187918801881188218831884188518861887188818891890189118921893189418951896189718981899190019011902190319041905190619071908190919101911191219131914191519161917191819191920192119221923192419251926 |
- # Copyright (C) 2012-2023 Zammad Foundation, https://zammad-foundation.org/
- require 'rails_helper'
- RSpec.describe CoreWorkflow, mariadb: true, type: :model do
- let(:group) { create(:group) }
- let!(:ticket) { create(:ticket, state: Ticket::State.find_by(name: 'pending reminder'), pending_time: 5.days.from_now, group: group) }
- let!(:base_payload) do
- {
- 'event' => 'core_workflow',
- 'request_id' => 'default',
- 'class_name' => 'Ticket',
- 'screen' => 'create_middle',
- 'params' => {},
- }
- end
- let(:payload) { base_payload }
- let!(:action_user) { create(:agent, groups: [ticket.group]) }
- let(:result) { described_class.perform(payload: payload, user: action_user) }
- describe '.perform - No assets' do
- let(:result) { described_class.perform(payload: payload, user: action_user, assets: false) }
- it 'does not contain assets' do
- expect(result[:assets]).to be_blank
- end
- end
- describe '.perform - Default - Group' do
- let!(:group_change) { create(:group) }
- let!(:group_create) { create(:group) }
- describe 'for agent with full permissions on screen create_middle' do
- let(:action_user) { create(:agent) }
- before do
- action_user.group_names_access_map = {
- group_create.name => ['full'],
- group_change.name => ['change'],
- }
- end
- it 'does show group_create for agent with all permissions' do
- expect(result[:restrict_values]['group_id']).to include(group_create.id.to_s)
- end
- it 'does not show group_change for agent with all permissions' do
- expect(result[:restrict_values]['group_id']).not_to include(group_change.id.to_s)
- end
- end
- describe 'for agent with full permissions on screen edit' do
- let(:payload) do
- base_payload.merge('screen' => 'edit')
- end
- let(:action_user) { create(:agent) }
- before do
- action_user.group_names_access_map = {
- group_create.name => ['full'],
- group_change.name => ['change'],
- }
- end
- it 'does show group_create for agent with all permissions' do
- expect(result[:restrict_values]['group_id']).to include(group_create.id.to_s)
- end
- it 'does show group_change for agent with all permissions' do
- expect(result[:restrict_values]['group_id']).to include(group_change.id.to_s)
- end
- end
- describe 'for agent with change permissions on screen create_middle' do
- let(:action_user) { create(:agent) }
- before do
- action_user.group_names_access_map = {
- group_create.name => ['change'],
- group_change.name => ['change'],
- }
- end
- it 'does not show group_create for agent with change permissions' do
- expect(result[:restrict_values]['group_id']).not_to include(group_create.id.to_s)
- end
- it 'does not show group_change for agent with change permissions' do
- expect(result[:restrict_values]['group_id']).not_to include(group_change.id.to_s)
- end
- end
- describe 'for agent with change permissions on screen edit' do
- let(:payload) do
- base_payload.merge('screen' => 'edit')
- end
- let(:action_user) { create(:agent) }
- before do
- action_user.group_names_access_map = {
- group_create.name => ['change'],
- group_change.name => ['change'],
- }
- end
- it 'does show group_create for agent with change permissions' do
- expect(result[:restrict_values]['group_id']).to include(group_create.id.to_s)
- end
- it 'does show group_change for agent with change permissions' do
- expect(result[:restrict_values]['group_id']).to include(group_change.id.to_s)
- end
- end
- describe 'for customer on screen create_middle' do
- let(:action_user) { create(:customer) }
- it 'does show group_create for customer' do
- expect(result[:restrict_values]['group_id']).to include(group_create.id.to_s)
- end
- it 'does show group_change for customer' do
- expect(result[:restrict_values]['group_id']).to include(group_change.id.to_s)
- end
- end
- describe 'for customer on screen edit' do
- let(:payload) do
- base_payload.merge('screen' => 'edit')
- end
- let(:action_user) { create(:customer) }
- it 'does show group_create for customer' do
- expect(result[:restrict_values]['group_id']).to include(group_create.id.to_s)
- end
- it 'does show group_change for customer' do
- expect(result[:restrict_values]['group_id']).to include(group_change.id.to_s)
- end
- end
- end
- describe '.perform - Default - Owner' do
- before do
- another_group = create(:group)
- action_user.group_names_access_map = {
- ticket.group.name => ['full'],
- another_group.name => ['full'],
- }
- end
- it 'does not show any owners for no group' do
- expect(result[:restrict_values]['owner_id']).to eq([''])
- end
- describe 'on group' do
- let(:payload) do
- base_payload.merge('params' => { 'group_id' => ticket.group.id })
- end
- it 'does show ticket agent' do
- expect(result[:restrict_values]['owner_id']).to eq(['', action_user.id.to_s])
- end
- end
- describe 'on group save' do
- let(:payload) do
- base_payload.merge('request_id' => 'ChecksCoreWorkflow.validate_workflows', 'params' => { 'group_id' => ticket.group.id })
- end
- it 'does show ticket agent and system user' do
- expect(result[:restrict_values]['owner_id']).to eq(['', '1', action_user.id.to_s])
- end
- end
- end
- describe '.perform - Default - Bulk Owner' do
- let(:payload) do
- base_payload.merge('screen' => 'overview_bulk')
- end
- it 'does not show any owners for no group' do
- expect(result[:restrict_values]['owner_id']).to eq([''])
- end
- describe 'on ticket ids' do
- let(:payload) do
- base_payload.merge('screen' => 'overview_bulk', 'params' => { 'ticket_ids' => ticket.id.to_s })
- end
- it 'does show ticket agent' do
- expect(result[:restrict_values]['owner_id']).to eq(['', action_user.id.to_s])
- end
- end
- describe 'on ticket ids with no group overlap' do
- let(:ticket2) { create(:ticket) }
- let(:payload) do
- base_payload.merge('screen' => 'overview_bulk', 'params' => { 'ticket_ids' => "#{ticket.id},#{ticket2.id}" })
- end
- it 'does not show ticket agent' do
- expect(result[:restrict_values]['owner_id']).to eq([''])
- end
- end
- describe 'on ticket ids with group overlap' do
- let(:ticket2) { create(:ticket, group: ticket.group) }
- let(:payload) do
- base_payload.merge('screen' => 'overview_bulk', 'params' => { 'ticket_ids' => "#{ticket.id},#{ticket2.id}" })
- end
- it 'does show ticket agent' do
- expect(result[:restrict_values]['owner_id']).to eq(['', action_user.id.to_s])
- end
- end
- describe 'Ticket owner selection is not updated if owner selection should be empty #3809' do
- let(:group_no_owners) { create(:group) }
- let(:ticket2) { create(:ticket, group: group_no_owners) }
- let(:payload) do
- base_payload.merge('screen' => 'overview_bulk', 'params' => { 'ticket_ids' => ticket2.id.to_s })
- end
- before do
- action_user.group_names_access_map = {
- group_no_owners.name => %w[create read change overview],
- }
- end
- it 'does not show any owners for group with no full permitted users' do
- expect(result[:restrict_values]['owner_id']).to eq([''])
- end
- end
- end
- describe '.perform - Default - State' do
- it 'does show state type new for create_middle' do
- expect(result[:restrict_values]['state_id']).to include(Ticket::State.find_by(name: 'new').id.to_s)
- end
- describe 'on edit' do
- let(:payload) do
- base_payload.merge('screen' => 'edit')
- end
- it 'does not show state type new' do
- expect(result[:restrict_values]['state_id']).not_to include(Ticket::State.find_by(name: 'new').id.to_s)
- end
- end
- it 'does show empty value for create_middle' do
- expect(result[:restrict_values]['state_id']).to include('')
- end
- context 'with customer user' do
- let(:action_user) { create(:customer) }
- it 'does not show empty value for create_middle' do
- expect(result[:restrict_values]['state_id']).not_to include('')
- end
- end
- end
- describe '.perform - Default - Priority' do
- let(:prio_invalid) { create(:ticket_priority, active: false) }
- it 'does show valid priority' do
- expect(result[:restrict_values]['priority_id']).to include(Ticket::Priority.find_by(name: '3 high').id.to_s)
- end
- it 'does not show invalid priority' do
- expect(result[:restrict_values]['priority_id']).not_to include(prio_invalid.id.to_s)
- end
- end
- describe '.perform - Default - Customer setting customer_ticket_create_group_ids' do
- let(:action_user) { create(:customer) }
- let!(:group1) { create(:group) }
- let!(:group2) { create(:group) }
- let!(:group3) { create(:group) }
- it 'does show group 1' do
- expect(result[:restrict_values]['group_id']).to include(group1.id.to_s)
- end
- context 'with customer_ticket_create_group_ids set' do
- before do
- Setting.set('customer_ticket_create_group_ids', [group2.id.to_s, group3.id.to_s])
- end
- it 'does not show group 1' do
- expect(result[:restrict_values]['group_id']).not_to include(group1.id.to_s)
- end
- end
- end
- describe '.perform - Default - #3721 - Fields are falsey displayed as mandatory if they contain historic screen values', db_strategy: :reset do
- let(:field_name) { SecureRandom.uuid }
- let(:screens) do
- {
- create_middle: {
- 'ticket.agent' => {
- shown: true,
- },
- },
- }
- end
- before do
- create(:object_manager_attribute_text, object_name: 'Ticket', name: field_name, display: field_name, screens: screens)
- ObjectManager::Attribute.migration_execute
- end
- it 'does show the field as optional because it has no required value' do
- expect(result[:mandatory][field_name]).to be(false)
- end
- end
- describe '.perform - Default - Restrict values for multiselect fields', db_strategy: :reset do
- let(:field_name) { SecureRandom.uuid }
- before do
- create(:object_manager_attribute_multiselect, name: field_name, display: field_name)
- ObjectManager::Attribute.migration_execute
- end
- context 'without saved values' do
- it 'does return the correct list of selectable values' do
- expect(result[:restrict_values][field_name]).to eq(['', 'key_1', 'key_2', 'key_3'])
- end
- end
- context 'with saved values' do
- let(:payload) do
- base_payload.merge('params' => {
- 'id' => ticket.id,
- })
- end
- before do
- ticket.reload.update(field_name.to_sym => %w[key_2 key_3])
- end
- it 'does return the correct list of selectable values' do
- expect(result[:restrict_values][field_name]).to eq(['', 'key_1', 'key_2', 'key_3'])
- end
- end
- end
- describe '.perform - Custom - Pending Time' do
- it 'does not show pending time for non pending state' do
- expect(result[:visibility]['pending_time']).to eq('remove')
- end
- describe 'for ticket id with no state change' do
- let(:payload) do
- base_payload.merge('params' => {
- 'id' => ticket.id,
- })
- end
- it 'does show pending time for pending ticket' do
- expect(result[:visibility]['pending_time']).to eq('show')
- end
- end
- describe 'for ticket id with state change' do
- let(:payload) do
- base_payload.merge('params' => {
- 'id' => ticket.id,
- 'state_id' => Ticket::State.find_by(name: 'open').id.to_s,
- })
- end
- it 'does not show pending time for pending ticket' do
- expect(result[:visibility]['pending_time']).to eq('remove')
- end
- end
- end
- describe '.perform - Custom - Admin SLA' do
- let(:payload) do
- base_payload.merge(
- 'screen' => 'edit',
- 'class_name' => 'Sla',
- )
- end
- it 'does set first_response_time_in_text optional' do
- expect(result[:mandatory]['first_response_time_in_text']).to be(false)
- end
- it 'does set update_time_in_text optional' do
- expect(result[:mandatory]['update_time_in_text']).to be(false)
- end
- it 'does set solution_time_in_text optional' do
- expect(result[:mandatory]['solution_time_in_text']).to be(false)
- end
- describe 'on first_response_time_enabled' do
- let(:payload) do
- base_payload.merge(
- 'screen' => 'edit',
- 'class_name' => 'Sla',
- 'params' => { 'first_response_time_enabled' => 'true' }
- )
- end
- it 'does set first_response_time_in_text mandatory' do
- expect(result[:mandatory]['first_response_time_in_text']).to be(true)
- end
- it 'does set update_time_in_text optional' do
- expect(result[:mandatory]['update_time_in_text']).to be(false)
- end
- it 'does set solution_time_in_text optional' do
- expect(result[:mandatory]['solution_time_in_text']).to be(false)
- end
- end
- describe 'on update_time_enabled' do
- let(:payload) do
- base_payload.merge(
- 'screen' => 'edit',
- 'class_name' => 'Sla',
- 'params' => { 'update_time_enabled' => 'true', 'update_type' => 'update' }
- )
- end
- it 'does set first_response_time_in_text optional' do
- expect(result[:mandatory]['first_response_time_in_text']).to be(false)
- end
- it 'does set update_time_in_text mandatory' do
- expect(result[:mandatory]['update_time_in_text']).to be(true)
- end
- it 'does set solution_time_in_text optional' do
- expect(result[:mandatory]['solution_time_in_text']).to be(false)
- end
- end
- describe 'on solution_time_enabled' do
- let(:payload) do
- base_payload.merge(
- 'screen' => 'edit',
- 'class_name' => 'Sla',
- 'params' => { 'solution_time_enabled' => 'true' }
- )
- end
- it 'does set first_response_time_in_text optional' do
- expect(result[:mandatory]['first_response_time_in_text']).to be(false)
- end
- it 'does set update_time_in_text optional' do
- expect(result[:mandatory]['update_time_in_text']).to be(false)
- end
- it 'does set solution_time_in_text mandatory' do
- expect(result[:mandatory]['solution_time_in_text']).to be(true)
- end
- end
- end
- describe '.perform - Custom - Admin CoreWorkflow' do
- let(:payload) do
- base_payload.merge(
- 'screen' => 'edit',
- 'class_name' => 'CoreWorkflow',
- )
- end
- it 'does not show screens for empty object' do
- expect(result[:restrict_values]['preferences::screen']).to eq([''])
- end
- it 'does not show invalid objects' do
- expect(result[:restrict_values]['object']).not_to include('CoreWorkflow')
- end
- describe 'on object Ticket' do
- let(:payload) do
- base_payload.merge(
- 'screen' => 'edit',
- 'class_name' => 'CoreWorkflow',
- 'params' => { 'object' => 'Ticket' },
- )
- end
- it 'does show screen create_middle' do
- expect(result[:restrict_values]['preferences::screen']).to include('create_middle')
- end
- it 'does show screen edit' do
- expect(result[:restrict_values]['preferences::screen']).to include('edit')
- end
- end
- describe 'on saved object Ticket' do
- let(:workflow) { create(:core_workflow, object: 'Ticket') }
- let(:payload) do
- base_payload.merge(
- 'screen' => 'edit',
- 'class_name' => 'CoreWorkflow',
- 'params' => { 'id' => workflow.id },
- )
- end
- it 'does show screen create_middle' do
- expect(result[:restrict_values]['preferences::screen']).to include('create_middle')
- end
- it 'does show screen edit' do
- expect(result[:restrict_values]['preferences::screen']).to include('edit')
- end
- end
- end
- describe '.perform - Condition - owner_id not set' do
- let!(:workflow) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'ticket.owner_id': {
- operator: 'not_set',
- },
- })
- end
- it 'does match' do
- expect(result[:matched_workflows]).to include(workflow.id)
- end
- describe 'for owner id 1' do
- let(:payload) do
- base_payload.merge(
- 'params' => { 'owner_id' => '1' },
- )
- end
- it 'does match' do
- expect(result[:matched_workflows]).to include(workflow.id)
- end
- end
- end
- describe '.perform - Condition - session.role_ids' do
- let!(:workflow) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'session.role_ids': {
- operator: 'is',
- value: [ Role.find_by(name: 'Agent').id.to_s ],
- },
- })
- end
- it 'does match' do
- expect(result[:matched_workflows]).to include(workflow.id)
- end
- describe 'for customer' do
- let!(:action_user) { create(:customer) } # rubocop:disable RSpec/LetSetup
- it 'does not match' do
- expect(result[:matched_workflows]).not_to include(workflow.id)
- end
- end
- end
- describe '.perform - Condition - session.group_ids_full' do
- let!(:workflow) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'session.group_ids_full': {
- operator: 'is',
- value: [ ticket.group.id.to_s ],
- },
- })
- end
- it 'does match' do
- expect(result[:matched_workflows]).to include(workflow.id)
- end
- describe 'for customer' do
- let!(:action_user) { create(:customer) } # rubocop:disable RSpec/LetSetup
- it 'does not match' do
- expect(result[:matched_workflows]).not_to include(workflow.id)
- end
- end
- end
- describe '.perform - Condition - session.group_ids_change' do
- let!(:workflow) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'session.group_ids_change': {
- operator: 'is',
- value: [ ticket.group.id.to_s ],
- },
- })
- end
- it 'does match' do
- expect(result[:matched_workflows]).to include(workflow.id)
- end
- describe 'for customer' do
- let!(:action_user) { create(:customer) } # rubocop:disable RSpec/LetSetup
- it 'does not match' do
- expect(result[:matched_workflows]).not_to include(workflow.id)
- end
- end
- end
- describe '.perform - Condition - session.group_ids_read' do
- let!(:workflow) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'session.group_ids_read': {
- operator: 'is',
- value: [ ticket.group.id.to_s ],
- },
- })
- end
- it 'does match' do
- expect(result[:matched_workflows]).to include(workflow.id)
- end
- describe 'for customer' do
- let!(:action_user) { create(:customer) } # rubocop:disable RSpec/LetSetup
- it 'does not match' do
- expect(result[:matched_workflows]).not_to include(workflow.id)
- end
- end
- end
- describe '.perform - Condition - session.group_ids_overview' do
- let!(:workflow) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'session.group_ids_overview': {
- operator: 'is',
- value: [ ticket.group.id.to_s ],
- },
- })
- end
- it 'does match' do
- expect(result[:matched_workflows]).to include(workflow.id)
- end
- describe 'for customer' do
- let!(:action_user) { create(:customer) } # rubocop:disable RSpec/LetSetup
- it 'does not match' do
- expect(result[:matched_workflows]).not_to include(workflow.id)
- end
- end
- end
- describe '.perform - Condition - session.group_ids_create' do
- let!(:workflow) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'session.group_ids_create': {
- operator: 'is',
- value: [ ticket.group.id.to_s ],
- },
- })
- end
- it 'does match' do
- expect(result[:matched_workflows]).to include(workflow.id)
- end
- describe 'for customer' do
- let!(:action_user) { create(:customer) } # rubocop:disable RSpec/LetSetup
- it 'does not match' do
- expect(result[:matched_workflows]).not_to include(workflow.id)
- end
- end
- end
- describe '.perform - Condition - session.permission_ids' do
- let!(:workflow) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'session.permission_ids': {
- operator: 'is',
- value: [ Permission.find_by(name: 'ticket.agent').id.to_s ],
- },
- })
- end
- it 'does match' do
- expect(result[:matched_workflows]).to include(workflow.id)
- end
- describe 'for customer' do
- let!(:action_user) { create(:customer) } # rubocop:disable RSpec/LetSetup
- it 'does not match' do
- expect(result[:matched_workflows]).not_to include(workflow.id)
- end
- end
- end
- describe '.perform - Condition - Regex match' do
- let(:payload) do
- base_payload.merge(
- 'params' => { 'title' => 'workflow ticket' },
- )
- end
- let!(:workflow) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'ticket.title': {
- operator: 'regex match',
- value: [ '^workflow' ],
- },
- })
- end
- it 'does match' do
- expect(result[:matched_workflows]).to include(workflow.id)
- end
- describe 'for invalid regex' do
- let!(:workflow) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'ticket.title': {
- operator: 'regex match',
- value: [ '^workfluw' ],
- },
- })
- end
- it 'does not match' do
- expect(result[:matched_workflows]).not_to include(workflow.id)
- end
- end
- end
- describe '.perform - Condition - Regex mismatch' do
- let(:payload) do
- base_payload.merge(
- 'params' => { 'title' => 'workflow ticket' },
- )
- end
- let!(:workflow) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'ticket.title': {
- operator: 'regex mismatch',
- value: [ '^workfluw' ],
- },
- })
- end
- it 'does match' do
- expect(result[:matched_workflows]).to include(workflow.id)
- end
- describe 'for invalid regex' do
- let!(:workflow) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'ticket.title': {
- operator: 'regex mismatch',
- value: [ '^workflow' ],
- },
- })
- end
- it 'does not match' do
- expect(result[:matched_workflows]).not_to include(workflow.id)
- end
- end
- end
- describe '.perform - Condition - Contains' do
- let(:payload) do
- base_payload.merge(
- 'params' => { 'title' => 'workflow ticket' },
- )
- end
- let!(:workflow) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'ticket.title': {
- operator: 'contains',
- value: [ 'workflow ticket', 'workflaw ticket' ],
- },
- })
- end
- it 'does match' do
- expect(result[:matched_workflows]).to include(workflow.id)
- end
- describe 'for invalid value' do
- let!(:workflow) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'ticket.title': {
- operator: 'contains',
- value: [ 'workfluw ticket', 'workflaw ticket' ],
- },
- })
- end
- it 'does not match' do
- expect(result[:matched_workflows]).not_to include(workflow.id)
- end
- end
- end
- describe '.perform - Condition - Contains not' do
- let(:payload) do
- base_payload.merge(
- 'params' => { 'title' => 'workflow ticket' },
- )
- end
- let!(:workflow) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'ticket.title': {
- operator: 'contains not',
- value: [ 'workfluw ticket', 'workflaw ticket' ],
- },
- })
- end
- it 'does match' do
- expect(result[:matched_workflows]).to include(workflow.id)
- end
- describe 'for invalid value' do
- let!(:workflow) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'ticket.title': {
- operator: 'contains not',
- value: [ 'workflow ticket', 'workflow ticket' ],
- },
- })
- end
- it 'does not match' do
- expect(result[:matched_workflows]).not_to include(workflow.id)
- end
- end
- end
- describe '.perform - Condition - Contains all' do
- let(:payload) do
- base_payload.merge(
- 'params' => { 'title' => 'workflow ticket' },
- )
- end
- let!(:workflow) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'ticket.title': {
- operator: 'contains all',
- value: [ 'workflow ticket', 'workflow ticket' ],
- },
- })
- end
- it 'does match' do
- expect(result[:matched_workflows]).to include(workflow.id)
- end
- describe 'for invalid value' do
- let!(:workflow) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'ticket.title': {
- operator: 'contains all',
- value: [ 'workflow ticket', 'workflaw ticket' ],
- },
- })
- end
- it 'does not match' do
- expect(result[:matched_workflows]).not_to include(workflow.id)
- end
- end
- end
- describe '.perform - Condition - Contains all not' do
- let(:payload) do
- base_payload.merge(
- 'params' => { 'title' => 'workflow ticket' },
- )
- end
- let!(:workflow) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'ticket.title': {
- operator: 'contains all not',
- value: [ 'workfluw ticket', 'workflaw ticket' ],
- },
- })
- end
- it 'does match' do
- expect(result[:matched_workflows]).to include(workflow.id)
- end
- describe 'for invalid value' do
- let!(:workflow) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'ticket.title': {
- operator: 'contains all not',
- value: [ 'workflow ticket', 'workflaw ticket' ],
- },
- })
- end
- it 'does not match' do
- expect(result[:matched_workflows]).not_to include(workflow.id)
- end
- end
- end
- describe '.perform - Stop after match' do
- let(:stop_after_match) { false }
- before do
- create(:core_workflow,
- object: 'Ticket',
- perform: {
- 'ticket.priority_id': {
- operator: 'hide',
- hide: 'true'
- },
- })
- create(:core_workflow,
- object: 'Ticket',
- perform: {
- 'ticket.priority_id': {
- operator: 'show',
- show: 'true'
- },
- },
- stop_after_match: stop_after_match)
- create(:core_workflow,
- object: 'Ticket',
- perform: {
- 'ticket.priority_id': {
- operator: 'hide',
- hide: 'true'
- },
- })
- end
- it 'does not stop' do
- expect(result[:visibility]['priority_id']).to eq('hide')
- end
- describe 'with stop_after_match' do
- let(:stop_after_match) { true }
- it 'does stop' do
- expect(result[:visibility]['priority_id']).to eq('show')
- end
- end
- end
- describe '.perform - Condition - Custom module' do
- let(:modules) { ['CoreWorkflow::Custom::Testa', 'CoreWorkflow::Custom::Testb', 'CoreWorkflow::Custom::Testc'] }
- let(:custom_class_false) do
- Class.new(CoreWorkflow::Custom::Backend) do
- def selected_attribute_match?
- false
- end
- end
- end
- let(:custom_class_true) do
- Class.new(CoreWorkflow::Custom::Backend) do
- def selected_attribute_match?
- true
- end
- end
- end
- let!(:workflow) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'custom.module': {
- operator: operator,
- value: modules,
- },
- })
- end
- describe 'with "match all modules" false' do
- let(:operator) { 'match all modules' }
- before do
- stub_const 'CoreWorkflow::Custom::Testa', custom_class_false
- stub_const 'CoreWorkflow::Custom::Testb', custom_class_false
- stub_const 'CoreWorkflow::Custom::Testc', custom_class_false
- end
- it 'does not match' do
- expect(result[:matched_workflows]).not_to include(workflow.id)
- end
- end
- describe 'with "match all modules" true' do
- let(:operator) { 'match all modules' }
- before do
- stub_const 'CoreWorkflow::Custom::Testa', custom_class_true
- stub_const 'CoreWorkflow::Custom::Testb', custom_class_true
- stub_const 'CoreWorkflow::Custom::Testc', custom_class_true
- end
- it 'does match' do
- expect(result[:matched_workflows]).to include(workflow.id)
- end
- end
- describe 'with "match all modules" blank' do
- let(:modules) { [] }
- let(:operator) { 'match all modules' }
- it 'does match' do
- expect(result[:matched_workflows]).to include(workflow.id)
- end
- end
- describe 'with "match one module" true' do
- let(:operator) { 'match one module' }
- before do
- stub_const 'CoreWorkflow::Custom::Testa', custom_class_false
- stub_const 'CoreWorkflow::Custom::Testb', custom_class_false
- stub_const 'CoreWorkflow::Custom::Testc', custom_class_true
- end
- it 'does match' do
- expect(result[:matched_workflows]).to include(workflow.id)
- end
- end
- describe 'with "match one module" false' do
- let(:operator) { 'match one module' }
- before do
- stub_const 'CoreWorkflow::Custom::Testa', custom_class_false
- stub_const 'CoreWorkflow::Custom::Testb', custom_class_false
- stub_const 'CoreWorkflow::Custom::Testc', custom_class_false
- end
- it 'does not match' do
- expect(result[:matched_workflows]).not_to include(workflow.id)
- end
- end
- describe 'with "match one module" blank' do
- let(:modules) { [] }
- let(:operator) { 'match one module' }
- it 'does match' do
- expect(result[:matched_workflows]).to include(workflow.id)
- end
- end
- describe 'with "match no modules" true' do
- let(:operator) { 'match no modules' }
- before do
- stub_const 'CoreWorkflow::Custom::Testa', custom_class_false
- stub_const 'CoreWorkflow::Custom::Testb', custom_class_false
- stub_const 'CoreWorkflow::Custom::Testc', custom_class_false
- end
- it 'does match' do
- expect(result[:matched_workflows]).to include(workflow.id)
- end
- end
- describe 'with "match no modules" false' do
- let(:operator) { 'match no modules' }
- before do
- stub_const 'CoreWorkflow::Custom::Testa', custom_class_true
- stub_const 'CoreWorkflow::Custom::Testb', custom_class_true
- stub_const 'CoreWorkflow::Custom::Testc', custom_class_true
- end
- it 'does not match' do
- expect(result[:matched_workflows]).not_to include(workflow.id)
- end
- end
- describe 'with "match no modules" blank' do
- let(:modules) { [] }
- let(:operator) { 'match no modules' }
- it 'does match' do
- expect(result[:matched_workflows]).to include(workflow.id)
- end
- end
- end
- describe '.perform - Select' do
- let!(:workflow1) do
- create(:core_workflow,
- object: 'Ticket',
- perform: {
- 'ticket.group_id': {
- operator: 'select',
- select: [ticket.group.id.to_s]
- },
- })
- end
- let!(:workflow2) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'ticket.group_id': {
- operator: 'is',
- value: ticket.group.id.to_s
- },
- },
- perform: {
- 'ticket.owner_id': {
- operator: 'select',
- select: [action_user.id.to_s]
- },
- })
- end
- it 'does match workflows' do
- expect(result[:matched_workflows]).to include(workflow1.id, workflow2.id)
- end
- it 'does select group' do
- expect(result[:select]['group_id']).to eq(ticket.group.id.to_s)
- end
- it 'does select owner (recursion)' do
- expect(result[:select]['owner_id']).to eq(action_user.id.to_s)
- end
- it 'does rerun 2 times (group select + owner select)' do
- expect(result[:rerun_count]).to eq(2)
- end
- end
- describe '.perform - Auto Select' do
- let!(:workflow1) do
- create(:core_workflow,
- object: 'Ticket',
- perform: {
- 'ticket.group_id': {
- operator: 'auto_select',
- auto_select: true
- },
- })
- end
- let!(:workflow2) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'ticket.group_id': {
- operator: 'is',
- value: ticket.group.id.to_s
- },
- },
- perform: {
- 'ticket.owner_id': {
- operator: 'auto_select',
- auto_select: true
- },
- })
- end
- it 'does match workflows' do
- expect(result[:matched_workflows]).to include(workflow1.id, workflow2.id)
- end
- it 'does select group' do
- expect(result[:select]['group_id']).to eq(ticket.group.id.to_s)
- end
- it 'does select owner (recursion)' do
- expect(result[:select]['owner_id']).to eq(action_user.id.to_s)
- end
- it 'does rerun 2 times (group select + owner select)' do
- expect(result[:rerun_count]).to eq(2)
- end
- describe 'with owner' do
- let(:payload) do
- base_payload.merge('params' => {
- 'group_id' => ticket.group.id.to_s,
- 'owner_id' => action_user.id.to_s,
- })
- end
- it 'does not select owner' do
- expect(result[:select]['owner_id']).to be_nil
- end
- it 'does rerun 0 times' do
- expect(result[:rerun_count]).to eq(0)
- end
- end
- end
- describe '.perform - Fill in' do
- let!(:workflow1) do
- create(:core_workflow,
- object: 'Ticket',
- perform: {
- 'ticket.group_id': {
- operator: 'select',
- select: [ticket.group.id.to_s]
- },
- })
- end
- let!(:workflow2) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'ticket.group_id': {
- operator: 'is',
- value: ticket.group.id.to_s
- },
- },
- perform: {
- 'ticket.title': {
- operator: 'fill_in',
- fill_in: 'hello'
- },
- })
- end
- it 'does match workflows' do
- expect(result[:matched_workflows]).to include(workflow1.id, workflow2.id)
- end
- it 'does select group' do
- expect(result[:select]['group_id']).to eq(ticket.group.id.to_s)
- end
- it 'does fill in title' do
- expect(result[:fill_in]['title']).to eq('hello')
- end
- it 'does rerun 1 time (group select + title fill in)' do
- expect(result[:rerun_count]).to eq(1)
- end
- end
- describe '.perform - Fill in empty' do
- let!(:workflow1) do
- create(:core_workflow,
- object: 'Ticket',
- perform: {
- 'ticket.group_id': {
- operator: 'select',
- select: [ticket.group.id.to_s]
- },
- })
- end
- let!(:workflow2) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'ticket.group_id': {
- operator: 'is',
- value: ticket.group.id.to_s
- },
- },
- perform: {
- 'ticket.title': {
- operator: 'fill_in_empty',
- fill_in_empty: 'hello'
- },
- })
- end
- it 'does match workflows' do
- expect(result[:matched_workflows]).to include(workflow1.id, workflow2.id)
- end
- it 'does select group' do
- expect(result[:select]['group_id']).to eq(ticket.group.id.to_s)
- end
- it 'does fill in title' do
- expect(result[:fill_in]['title']).to eq('hello')
- end
- it 'does rerun 1 time (group select + title fill in)' do
- expect(result[:rerun_count]).to eq(1)
- end
- describe 'with title' do
- let(:payload) do
- base_payload.merge('params' => {
- 'title' => 'ha!',
- })
- end
- it 'does not fill in title' do
- expect(result[:fill_in]['title']).to be_nil
- end
- it 'does rerun 1 times (group select)' do
- expect(result[:rerun_count]).to eq(1)
- end
- end
- end
- describe '.perform - Rerun attributes default cache bug' do
- before do
- create(:core_workflow,
- object: 'Ticket',
- perform: {
- 'ticket.group_id': {
- operator: 'select',
- select: [ticket.group.id.to_s]
- },
- })
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'ticket.group_id': {
- operator: 'is_set',
- },
- },
- perform: {
- 'ticket.owner_id': {
- operator: 'select',
- select: [action_user.id.to_s]
- },
- })
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'ticket.owner_id': {
- operator: 'not_set',
- },
- },
- perform: {
- 'ticket.priority_id': {
- operator: 'hide',
- hide: 'true'
- },
- })
- end
- it 'does not hide priority id' do
- expect(result[:visibility]['priority_id']).to eq('show')
- end
- end
- describe '.perform - Clean up params after restrict values removed selected value by set_fixed_to' do
- let(:payload) do
- base_payload.merge('params' => {
- 'owner_id' => action_user.id,
- })
- end
- before do
- create(:core_workflow,
- object: 'Ticket',
- perform: {
- 'ticket.group_id': {
- operator: 'select',
- select: [ticket.group.id.to_s]
- },
- })
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'ticket.group_id': {
- operator: 'is_set',
- },
- },
- perform: {
- 'ticket.owner_id': {
- operator: 'set_fixed_to',
- set_fixed_to: ['']
- },
- })
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'ticket.owner_id': {
- operator: 'is_set',
- },
- },
- perform: {
- 'ticket.priority_id': {
- operator: 'hide',
- hide: 'true'
- },
- })
- end
- it 'does not allow owner_id' do
- expect(result[:restrict_values]['owner_id']).not_to include(action_user.id)
- end
- it 'does not hide priority id' do
- expect(result[:visibility]['priority_id']).to eq('show')
- end
- end
- describe '.perform - Clean up params after restrict values removed selected value by remove_option' do
- let(:payload) do
- base_payload.merge('params' => {
- 'owner_id' => action_user.id,
- })
- end
- before do
- create(:core_workflow,
- object: 'Ticket',
- perform: {
- 'ticket.group_id': {
- operator: 'select',
- select: [ticket.group.id.to_s]
- },
- })
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'ticket.group_id': {
- operator: 'is_set',
- },
- },
- perform: {
- 'ticket.owner_id': {
- operator: 'remove_option',
- remove_option: [action_user.id]
- },
- })
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'ticket.owner_id': {
- operator: 'is_set',
- },
- },
- perform: {
- 'ticket.priority_id': {
- operator: 'hide',
- hide: 'true'
- },
- })
- end
- it 'does not allow owner_id' do
- expect(result[:restrict_values]['owner_id']).not_to include(action_user.id)
- end
- it 'does not hide priority id' do
- expect(result[:visibility]['priority_id']).to eq('show')
- end
- end
- describe '.perform - Clean up params after restrict values removed selected value by default attributes' do
- let(:payload) do
- base_payload.merge('params' => {
- 'owner_id' => action_user.id,
- })
- end
- before do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'ticket.owner_id': {
- operator: 'is_set',
- },
- },
- perform: {
- 'ticket.priority_id': {
- operator: 'hide',
- hide: 'true'
- },
- })
- end
- it 'does not allow owner_id' do
- expect(result[:restrict_values]['owner_id']).not_to include(action_user.id)
- end
- it 'does not hide priority id' do
- expect(result[:visibility]['priority_id']).to eq('show')
- end
- end
- describe '.perform - Default - auto selection based on only_shown_if_selectable' do
- it 'does auto select group' do
- expect(result[:select]['group_id']).not_to be_nil
- end
- it 'does auto hide group' do
- expect(result[:visibility]['group_id']).to eq('hide')
- end
- end
- describe '.perform - One field and two perform actions' do
- before do
- create(:core_workflow,
- object: 'Ticket',
- perform: {
- 'ticket.owner_id': {
- operator: %w[select set_optional],
- select: [action_user.id.to_s],
- set_optional: 'true',
- },
- })
- end
- it 'does auto select owner' do
- expect(result[:select]['owner_id']).to eq(action_user.id.to_s)
- end
- it 'does set owner optional' do
- expect(result[:mandatory]['owner_id']).to be(false)
- end
- end
- describe '.perform - Hide mobile based on user login' do
- let(:base_payload) do
- {
- 'event' => 'core_workflow',
- 'request_id' => 'default',
- 'class_name' => 'User',
- 'screen' => 'create',
- 'params' => {
- 'login' => 'nicole.special@zammad.org',
- },
- }
- end
- before do
- create(:core_workflow,
- object: 'User',
- condition_selected: { 'user.login'=>{ 'operator' => 'is', 'value' => 'nicole.special@zammad.org' } },
- perform: { 'user.mobile'=>{ 'operator' => 'hide', 'hide' => 'true' } },)
- end
- it 'does hide mobile for user' do
- expect(result[:visibility]['mobile']).to eq('hide')
- end
- end
- describe '.perform - Condition - group active is true' do
- let(:payload) do
- base_payload.merge('params' => {
- 'group_id' => Group.first.id,
- })
- end
- let!(:workflow) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: { 'group.active'=>{ 'operator' => 'is', 'value' => true } })
- end
- it 'does match' do
- expect(result[:matched_workflows]).to include(workflow.id)
- end
- end
- describe '.perform - Condition - group.assignment_timeout (Integer) matches' do
- let(:group) { create(:group, assignment_timeout: 10) }
- let(:payload) do
- base_payload.merge('params' => {
- 'group_id' => group.id,
- })
- end
- before do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: { 'group.assignment_timeout'=>{ 'operator' => 'is', 'value' => 10 } },
- perform: { 'ticket.priority_id'=>{ 'operator' => 'hide', 'hide' => 'true' } },)
- end
- it 'does match' do
- expect(result[:visibility]['priority_id']).to eq('hide')
- end
- end
- describe '.perform - Readonly' do
- let!(:workflow1) do
- create(:core_workflow,
- object: 'Ticket',
- perform: {
- 'ticket.group_id': {
- operator: 'set_readonly',
- set_readonly: 'true'
- },
- })
- end
- it 'does match workflow' do
- expect(result[:matched_workflows]).to include(workflow1.id)
- end
- it 'does set group readonly' do
- expect(result[:readonly]['group_id']).to be(true)
- end
- context 'when readonly unset' do
- let!(:workflow2) do
- create(:core_workflow,
- object: 'Ticket',
- perform: {
- 'ticket.group_id': {
- operator: 'unset_readonly',
- unset_readonly: 'true'
- },
- })
- end
- it 'does match workflows' do
- expect(result[:matched_workflows]).to include(workflow1.id, workflow2.id)
- end
- it 'does set group readonly' do
- expect(result[:readonly]['group_id']).to be(false)
- end
- end
- end
- describe 'Core Workflow "is not" operator is working unexpected #3752' do
- let(:approval_role) { create(:role) }
- let!(:workflow) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'session.role_ids': {
- operator: 'is_not',
- value: [ approval_role.id.to_s ]
- },
- })
- end
- context 'when not action user has approval role' do
- let(:action_user) { create(:agent, roles: [Role.find_by(name: 'Agent'), approval_role]) }
- it 'does not match' do
- expect(result[:matched_workflows]).not_to include(workflow.id)
- end
- end
- context 'when action user has not approval role' do
- let(:action_user) { create(:agent) }
- it 'does match' do
- expect(result[:matched_workflows]).to include(workflow.id)
- end
- end
- end
- describe 'Saved conditions break on selections without reloading #3758', db_strategy: :reset do
- let(:field_name) { SecureRandom.uuid }
- let(:screens) do
- {
- edit: {
- 'ticket.agent' => {
- shown: true,
- },
- },
- }
- end
- let!(:workflow) do
- create(:core_workflow,
- object: 'Ticket',
- condition_saved: {
- "ticket.#{field_name}": {
- operator: 'is_not',
- value: 'true',
- },
- })
- end
- let(:payload) do
- base_payload.merge('params' => { 'id' => ticket.id }, 'screen' => 'edit')
- end
- before do
- create(:object_manager_attribute_boolean, object_name: 'Ticket', name: field_name, display: field_name, screens: screens)
- ObjectManager::Attribute.migration_execute
- end
- it 'does match the workflow because saved value is false' do
- expect(result[:matched_workflows]).to include(workflow.id)
- end
- context 'when params contain boolean field true' do
- let(:payload) do
- base_payload.merge('params' => { 'id' => ticket.id, field_name => true }, 'screen' => 'edit')
- end
- it 'does match the workflow because saved value is false' do
- expect(result[:matched_workflows]).to include(workflow.id)
- end
- end
- end
- describe 'Core Workflow: Add organization condition attributes for object User #3779' do
- let(:organization) { create(:organization, note: 'hello') }
- let!(:base_payload) do
- {
- 'event' => 'core_workflow',
- 'request_id' => 'default',
- 'class_name' => 'User',
- 'screen' => 'create',
- 'params' => {},
- }
- end
- let!(:workflow) do
- create(:core_workflow,
- object: 'User',
- condition_selected: {
- 'organization.note': {
- operator: 'is',
- value: 'hello',
- },
- })
- end
- context 'when new user has no organization' do
- it 'does not match the workflow' do
- expect(result[:matched_workflows]).not_to include(workflow.id)
- end
- end
- context 'when new user is part of the organization' do
- let(:payload) do
- base_payload.merge('params' => { 'organization_id' => organization.id.to_s })
- end
- it 'does match the workflow' do
- expect(result[:matched_workflows]).to include(workflow.id)
- end
- end
- end
- describe 'Ticket owner selection is not updated if owner selection should be empty #3809' do
- let(:group_no_owners) { create(:group) }
- let(:payload) do
- base_payload.merge('params' => { 'group_id' => group_no_owners.id })
- end
- before do
- action_user.group_names_access_map = {
- group_no_owners.name => %w[create read change overview],
- }
- end
- it 'does not show any owners because no one has full permissions' do
- expect(result[:restrict_values]['owner_id']).to eq([''])
- end
- end
- describe 'If selected value is not part of the restriction of set_fixed_to it should recalculate it with the new value #3822', db_strategy: :reset do
- let(:field_name1) { SecureRandom.uuid }
- let(:screens) do
- {
- 'create_middle' => {
- 'ticket.agent' => {
- 'shown' => false,
- 'required' => false,
- }
- }
- }
- end
- let!(:workflow1) do
- create(:core_workflow,
- object: 'Ticket',
- perform: { "ticket.#{field_name1}" => { 'operator' => 'set_fixed_to', 'set_fixed_to' => ['key_3'] } })
- end
- let!(:workflow2) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- "ticket.#{field_name1}": {
- operator: 'is',
- value: 'key_3',
- },
- })
- end
- before do
- create(:object_manager_attribute_select, name: field_name1, display: field_name1, screens: screens)
- ObjectManager::Attribute.migration_execute
- end
- it 'does select key_3 as new param value and based on this executes workflow 2' do
- expect(result[:matched_workflows]).to include(workflow1.id, workflow2.id)
- end
- end
- describe 'Add clear selection action or has changed condition #3821' do
- let!(:workflow_has_changed) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'ticket.priority_id': {
- operator: 'has_changed',
- },
- })
- end
- let!(:workflow_changed_to) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'ticket.priority_id': {
- operator: 'changed_to',
- value: [ Ticket::Priority.find_by(name: '3 high').id.to_s ]
- },
- })
- end
- context 'when priority changed' do
- let(:payload) do
- base_payload.merge('last_changed_attribute' => 'priority_id', 'params' => { 'priority_id' => Ticket::Priority.find_by(name: '3 high').id.to_s })
- end
- it 'does match on condition has changed' do
- expect(result[:matched_workflows]).to include(workflow_has_changed.id)
- end
- it 'does match on condition changed to' do
- expect(result[:matched_workflows]).to include(workflow_changed_to.id)
- end
- end
- context 'when nothing changed' do
- it 'does not match on condition has changed' do
- expect(result[:matched_workflows]).not_to include(workflow_has_changed.id)
- end
- it 'does not match on condition changed to' do
- expect(result[:matched_workflows]).not_to include(workflow_changed_to.id)
- end
- end
- context 'when state changed' do
- let(:payload) do
- base_payload.merge('last_changed_attribute' => 'state_id')
- end
- it 'does not match on condition has changed' do
- expect(result[:matched_workflows]).not_to include(workflow_has_changed.id)
- end
- it 'does not match on condition changed to' do
- expect(result[:matched_workflows]).not_to include(workflow_changed_to.id)
- end
- end
- end
- describe 'Wrong core workflow execution because of missing relation defaults #4541' do
- let!(:workflow) do
- create(:core_workflow,
- object: 'Ticket',
- condition_selected: {
- 'ticket.priority_id': {
- operator: 'is',
- value: [ Ticket::Priority.find_by(name: '1 low').id.to_s ]
- },
- })
- end
- before do
- Ticket::Priority.find_by(name: '2 normal').update(note: 'Test')
- workflow
- end
- it 'does not execute the core workflow because the default priority is 2 normal and not 1 low' do
- expect(result[:matched_workflows]).not_to include(workflow.id)
- end
- end
- end
|