123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173 |
- # Copyright (C) 2012-2022 Zammad Foundation, https://zammad-foundation.org/
- require 'rails_helper'
- RSpec.describe 'Error handling', type: :request do
- shared_examples 'JSON response format' do
- let(:as) { :json }
- it { expect(response).to have_http_status(http_status) }
- it { expect(json_response).to be_a_kind_of(Hash) }
- it do
- # There is a special case where we mask technical errors and return
- # a random error code that can be easily found in the logs by an
- # administrator. However, this makes it hard to check for the exact error
- # message. Therefore we only check for the substring in this particular case
- if message == 'Please contact your administrator' || message == 'Mysql2::Error' || message == 'PG::ForeignKeyViolation'
- expect(json_response['error']).to include(message)
- else
- expect(json_response['error']).to eq(message)
- end
- end
- end
- shared_examples 'HTML response format' do
- let(:as) { :html }
- it { expect(response).to have_http_status(http_status) }
- it { expect(response.content_type).to start_with('text/html') }
- it { expect(response.body).to include('<html') }
- it { expect(response.body).to include("<title>#{title}</title>") }
- it { expect(response.body).to include("<h1>#{headline}</h1>") }
- it { expect(response.body).to include(message) }
- end
- context 'URL route does not exist' do
- before do
- get url, as: as
- end
- let(:url) { '/not_existing_url' }
- let(:message) { "No route matches [GET] #{url}" }
- let(:http_status) { :not_found }
- context 'requesting JSON' do
- include_examples 'JSON response format'
- end
- context 'requesting HTML' do
- let(:title) { '404: Not Found' }
- let(:headline) { '404: Requested resource was not found' }
- include_examples 'HTML response format'
- context 'when request ends with URL' do
- let(:url) { "//////#{message}" }
- let(:message) { 'this__website__is__closed__visit__our__new__site:_someother.com' }
- include_examples 'HTML response format'
- end
- end
- end
- context 'request is not authenticated' do
- before do
- authenticated_as(create(:agent), password: 'wrongpw')
- get '/api/v1/organizations', as: as
- end
- let(:message) { 'Invalid BasicAuth credentials' }
- let(:http_status) { :unauthorized }
- context 'requesting JSON' do
- include_examples 'JSON response format'
- end
- context 'requesting HTML' do
- let(:title) { '401: Unauthorized' }
- let(:headline) { '401: Unauthorized' }
- include_examples 'HTML response format'
- end
- end
- context 'request is forbidden' do
- before do
- get '/api/v1/organizations', as: as
- end
- let(:message) { 'Authentication required' }
- let(:http_status) { :forbidden }
- context 'requesting JSON' do
- include_examples 'JSON response format'
- end
- context 'requesting HTML' do
- let(:title) { '403: Forbidden' }
- let(:headline) { '403: Forbidden' }
- include_examples 'HTML response format'
- end
- end
- context 'exception is raised' do
- before do
- authenticated_as(create(user))
- get '/tests/raised_exception', params: { exception: exception.name, message: message }, as: as
- end
- shared_examples 'exception check' do |message, exception, http_status, title, headline|
- context "#{exception} is raised" do
- let(:exception) { exception }
- let(:http_status) { http_status }
- let(:message) { message }
- context 'requesting JSON' do
- include_examples 'JSON response format'
- end
- context 'requesting HTML' do
- let(:title) { title }
- let(:headline) { headline }
- include_examples 'HTML response format'
- end
- end
- end
- shared_examples 'handles exception' do |exception, http_status, title, headline, message = 'some error message'|
- include_examples 'exception check', message, exception, http_status, title, headline
- end
- shared_examples 'masks exception' do |exception, http_status, title, headline|
- include_examples 'exception check', 'Please contact your administrator', exception, http_status, title, headline
- end
- context 'with agent user' do
- let(:user) { :agent }
- include_examples 'handles exception', Exceptions::NotAuthorized, :unauthorized, '401: Unauthorized', '401: Unauthorized'
- include_examples 'handles exception', Exceptions::Forbidden, :forbidden, '403: Forbidden', '403: Forbidden'
- include_examples 'handles exception', Pundit::NotAuthorizedError, :forbidden, '403: Forbidden', '403: Forbidden', 'Not authorized'
- include_examples 'handles exception', ActiveRecord::RecordNotFound, :not_found, '404: Not Found', '404: Requested resource was not found'
- include_examples 'handles exception', Exceptions::UnprocessableEntity, :unprocessable_entity, '422: Unprocessable Entity', '422: The change you wanted was rejected.'
- include_examples 'masks exception', ArgumentError, :unprocessable_entity, '422: Unprocessable Entity', '422: The change you wanted was rejected.'
- include_examples 'masks exception', StandardError, :internal_server_error, '500: Something went wrong', "500: We're sorry, but something went wrong."
- end
- context 'with admin user' do
- let(:user) { :admin }
- include_examples 'handles exception', Exceptions::NotAuthorized, :unauthorized, '401: Unauthorized', '401: Unauthorized'
- include_examples 'handles exception', Exceptions::Forbidden, :forbidden, '403: Forbidden', '403: Forbidden'
- include_examples 'handles exception', Pundit::NotAuthorizedError, :forbidden, '403: Forbidden', '403: Forbidden', 'Not authorized'
- include_examples 'handles exception', ActiveRecord::RecordNotFound, :not_found, '404: Not Found', '404: Requested resource was not found'
- include_examples 'handles exception', Exceptions::UnprocessableEntity, :unprocessable_entity, '422: Unprocessable Entity', '422: The change you wanted was rejected.'
- include_examples 'handles exception', ArgumentError, :unprocessable_entity, '422: Unprocessable Entity', '422: The change you wanted was rejected.'
- include_examples 'handles exception', StandardError, :internal_server_error, '500: Something went wrong', "500: We're sorry, but something went wrong."
- end
- end
- end
|