123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232 |
- # Copyright (C) 2012-2024 Zammad Foundation, https://zammad-foundation.org/
- require 'csv'
- RSpec.shared_examples 'CanCsvImport - User specific tests', :aggregate_failures do
- describe '.csv_example' do
- context 'when no data avaiable' do
- let(:headers) do
- CSV.parse(User.csv_example).shift
- end
- it 'returns expected headers' do
- expect(headers).to start_with('id', 'login', 'firstname', 'lastname', 'email')
- expect(headers).to include('organization')
- end
- end
- end
- describe '.csv_import' do
- let(:try) { true }
- let(:delete) { false }
- let(:params) { { string: csv_string, parse_params: { col_sep: ';' }, try: try, delete: delete } }
- let(:result) { User.csv_import(**params) }
- shared_examples 'fails with error' do |errors|
- shared_examples 'checks error handling' do
- it 'returns error(s)' do
- expect(result).to include({ try: try, result: 'failed', errors: errors })
- end
- it 'does not import users' do
- # Any single failure will cause the entire import to be aborted.
- expect { result }.not_to change(User, :count)
- end
- end
- context 'with :try' do
- include_examples 'checks error handling'
- end
- context 'without :try' do
- let(:try) { false }
- include_examples 'checks error handling'
- end
- end
- context 'with empty string' do
- let(:csv_string) { '' }
- include_examples 'fails with error', ['Unable to parse empty file/string for User.']
- end
- context 'with just CSV header line' do
- let(:csv_string) { "login;firstname;lastname;email;active;\n" }
- include_examples 'fails with error', ['No records found in file/string for User.']
- end
- context 'without required lookup header' do
- let(:csv_string) { "firstname;lastname;active;\nfirstname-simple-import1;lastname-simple-import1;;true\nfirstname-simple-import2;lastname-simple-import2;false\n" }
- include_examples 'fails with error', ['No lookup column like id,login,email for User found.']
- end
- context 'with invalid id' do
- let(:csv_string) { "id;login;firstname;lastname;email;active;\n999999999;user-simple-invalid_id-import1;firstname-simple-import1;lastname-simple-import1;user-simple-invalid_id-import1@example.com;true\n;user-simple-invalid_id-import2;firstname-simple-import2;lastname-simple-import2;user-simple-invalid_id-import2@example.com;false\n" }
- include_examples 'fails with error', ["Line 1: unknown User with id '999999999'."]
- end
- context 'with readonly id' do
- let(:csv_string) { "id;login;firstname;lastname;email;active;\n1;user-simple-readonly_id-import1;firstname-simple-import1;lastname-simple-import1;user-simple-readonly_id-import1@example.com;true\n;user-simple-readonly_id-import2;firstname-simple-import2;lastname-simple-import2;user-simple-readonly_id-import2@example.com;false\n" }
- include_examples 'fails with error', ["Line 1: unable to update User with id '1'."]
- end
- context 'with invalid attributes' do
- let(:csv_string) { "login;firstname2;lastname;email\nuser-invalid-import1;firstname-invalid-import1;firstname-invalid-import1;user-invalid-import1@example.com\nuser-invalid-import2;firstname-invalid-import2;firstname-invalid-import2;user-invalid-import2@example.com\n" }
- include_examples 'fails with error', [
- "Line 1: Unable to create record - unknown attribute 'firstname2' for User.",
- "Line 2: Unable to create record - unknown attribute 'firstname2' for User.",
- ]
- end
- context 'with delete' do
- let(:csv_string) { "login;firstname;lastname;email\nuser-simple-import-fixed1;firstname-simple-import-fixed1;lastname-simple-import-fixed1;user-simple-import-fixed1@example.com\nuser-simple-import-fixed2;firstname-simple-import-fixed2;lastname-simple-import-fixed2;user-simple-import-fixed2@example.com\n" }
- let(:delete) { true }
- include_examples 'fails with error', ['Delete is not possible for User.']
- end
- context 'with duplicates' do
- let(:csv_string) { "login;firstname;lastname;email\nuser-duplicate-import1;firstname-duplicate-import1;firstname-duplicate-import1;user-duplicate-import1@example.com\nuser-duplicate-import2;firstname-duplicate-import2;firstname-duplicate-import2;user-duplicate-import2@example.com\nuser-duplicate-import2;firstname-duplicate-import3;firstname-duplicate-import3;user-duplicate-import3@example.com" }
- include_examples 'fails with error', ['Line 3: duplicate record found.']
- end
- context 'with references to nonexisting organizations' do
- let(:csv_string) { "login;firstname;lastname;email;organization\nuser-reference-import1;firstname-reference-import1;firstname-reference-import1;user-reference-import1@example.com;organization-reference-import1\nuser-reference-import2;firstname-reference-import2;firstname-reference-import2;user-reference-import2@example.com;organization-reference-import2\nuser-reference-import3;firstname-reference-import3;firstname-reference-import3;user-reference-import3@example.com;Zammad Foundation\n" }
- include_examples 'fails with error', [
- "Line 1: No lookup value found for 'organization': \"organization-reference-import1\"",
- "Line 2: No lookup value found for 'organization': \"organization-reference-import2\"",
- ]
- context 'when organizations are available' do
- before do
- create(:organization, name: 'organization-reference-import1')
- create(:organization, name: 'organization-reference-import2')
- end
- let(:try) { false }
- let(:first_user) { User.last(3).first }
- let(:second_user) { User.last(3).second }
- let(:third_user) { User.last }
- it 'returns success' do
- expect(result).to include({ try: try, result: 'success' })
- expect(result[:records].count).to be(3)
- end
- it 'does import users' do
- expect { result }.to change(User, :count).by(3)
- expect(first_user.organization.name).to eq('organization-reference-import1')
- expect(second_user.organization.name).to eq('organization-reference-import2')
- expect(third_user.organization.name).to eq('Zammad Foundation')
- end
- end
- end
- context 'with valid import data' do
- let(:csv_string) { "login;firstname;lastname;email;active;\nuser-simple-IMPORT1;firstname-simple-import1;lastname-simple-import1;user-simple-IMPORT1@example.com ;true\nuser-simple-import2;firstname-simple-import2;lastname-simple-import2;user-simple-import2@example.com;false\n" }
- let(:csv_string_without_email) { "login;firstname;lastname;email;active;\nuser-simple-IMPORT1;firstname-simple-import1;lastname-simple-import1;user-simple-IMPORT1@example.com ;true\nuser-simple-import2;firstname-simple-import2;lastname-simple-import2;;false\n" }
- let(:second_result) { User.csv_import(**params) }
- let(:second_params) { { string: second_csv_string, parse_params: { col_sep: ';' }, try: try, delete: delete } }
- context 'with :try' do
- it 'returns success' do
- expect(result).to include({ try: try, result: 'success' })
- expect(result[:records].count).to be(2)
- end
- it 'does not import users' do
- expect { result }.not_to change(User, :count)
- end
- end
- context 'without :try' do
- let(:try) { false }
- let(:first_user) { User.last(2).first }
- let(:second_user) { User.last }
- it 'returns success' do
- expect(result).to include({ try: try, result: 'success', records: have_attributes(count: 2), stats: { created: 2, updated: 0 } })
- expect(second_result).to include({ try: try, result: 'success', records: have_attributes(count: 2), stats: { created: 0, updated: 2 } })
- end
- it 'does import users' do
- expect { result }.to change(User, :count).by(2)
- expect(first_user).to have_attributes(
- login: 'user-simple-import1',
- firstname: 'firstname-simple-import1',
- lastname: 'lastname-simple-import1',
- email: 'user-simple-import1@example.com',
- active: true,
- )
- expect(second_user).to have_attributes(
- login: 'user-simple-import2',
- firstname: 'firstname-simple-import2',
- lastname: 'lastname-simple-import2',
- email: 'user-simple-import2@example.com',
- active: false,
- )
- expect { second_result }.not_to change(User, :count)
- expect(first_user.reload).to have_attributes(
- login: 'user-simple-import1',
- firstname: 'firstname-simple-import1',
- lastname: 'lastname-simple-import1',
- email: 'user-simple-import1@example.com',
- active: true,
- )
- # Email is still present, though missing in CSV.
- expect(second_user.reload).to have_attributes(
- login: 'user-simple-import2',
- firstname: 'firstname-simple-import2',
- lastname: 'lastname-simple-import2',
- email: 'user-simple-import2@example.com',
- active: false,
- )
- end
- end
- end
- context 'with roles and fixed params' do
- let(:result) { User.csv_import(**params, fixed_params: { note: 'some note' }) }
- let(:csv_string) { "login;firstname;lastname;email;roles;\nuser-role-import1;firstname-role-import1;lastname-role-import1;user-role-import1@example.com;Customer;\nuser-role-import2;firstname-role-import2;lastname-role-import2;user-role-import2@example.com;Agent\n;;;;Admin" }
- context 'with :try' do
- it 'returns success' do
- expect(result).to include({ try: try, result: 'success' })
- expect(result[:records].count).to be(2)
- end
- it 'does not import users' do
- expect { result }.not_to change(User, :count)
- end
- end
- context 'without :try' do
- let(:try) { false }
- let(:first_user) { User.last(2).first }
- let(:second_user) { User.last }
- it 'returns success' do
- expect(result).to include({ try: try, result: 'success', records: have_attributes(count: 2), stats: { created: 2, updated: 0 } })
- end
- it 'does import users with roles' do
- expect { result }.to change(User, :count).by(2)
- expect(first_user.roles.count).to be(1)
- expect(first_user.note).to eq('some note')
- expect(second_user.roles.count).to be(2)
- expect(second_user.note).to eq('some note')
- end
- end
- end
- end
- end
|