1680: Improve error when exporting through an account_link

Author: kkoehn

app/services/task_service/push_external.rb

@@ -12,7 +12,11 @@ def execute
       body = @zip.string
       begin
         response = self.class.connection.post(@account_link.push_url) {|request| request_parameters(request, body) }
-        response.success? ? nil : response.body
+        if response.success?
+          nil
+        else
+          response.status == 401 ? I18n.t('tasks.export_external_confirm.not_authorized', account_link: @account_link.name) : response.body
+        end
       rescue StandardError => e
         e
       end

config/locales/de/controllers/tasks.yml

@@ -8,6 +8,7 @@ de:
       error_alert: Die Aufgabe konnte nicht dupliziert werden.
     export_external_confirm:
       error: 'Der Export der Aufgabe (%{title}) ist fehlgeschlagen. <br> Fehler: %{error}'
+      not_authorized: Die Autorisierung mit "%{account_link}" konnte nicht hergestellt werden. Ist der API-Schlüssel korrekt?
       success: Aufgabe (%{title}) erfolgreich exportiert.
     import:
       internal_error: Beim Import dieser Aufgabe ist auf CodeHarbor ein interner Fehler aufgetreten.

config/locales/en/controllers/tasks.yml

@@ -8,6 +8,7 @@ en:
       error_alert: Task could not be duplicated
     export_external_confirm:
       error: 'Export of task (%{title}) failed. <br> Error: %{error}'
+      not_authorized: Authorization with could not be established with "%{account_link}". Is the API Key correct?
       success: Task (%{title}) successfully exported.
     import:
       internal_error: An internal error occurred on CodeHarbor while importing the exercise.

spec/services/task_service/push_external_spec.rb

@@ -53,6 +53,13 @@
 
         it { is_expected.to be response }
       end
+
+      context 'when response status is 401' do
+        let(:status) { 401 }
+        let(:response) { I18n.t('tasks.export_external_confirm.not_authorized', account_link: account_link.name) }
+
+        it { is_expected.to eq response }
+      end
     end
 
     context 'when an error occurs' do