1680: Improve error when exporting through an account_link

Author: kkoehn

app/services/exercise_service/push_external.rb

@@ -18,7 +18,11 @@ def execute
           request.body = body
         end
 
-        response.success? ? nil : response.body
+        if response.success?
+          nil
+        else
+          response.status == 401 ? I18n.t('exercises.export_codeharbor.not_authorized') : response.body
+        end
       rescue StandardError => e
         e.message
       end

config/locales/de/exercise.yml

@@ -103,6 +103,7 @@ de:
       export_failed: 'Export ist fehlgeschlagen.<br>ID: %{id}<br>Title: %{title}<br><br>Error: %{error}'
       label: Zu CodeHarbor exportieren
       successfully_exported: 'Aufgabe wurde erfolgreich exportiert.<br>ID: %{id}<br>Title: %{title}'
+      not_authorized: Die Autorisierung mit CodeHarbor konnte nicht hergestellt werden. Ist der API-Schlüssel korrekt?
     external_users:
       statistics:
         addendum: "* Differenzen von mehr als %{delta} Minuten werden ignoriert."

config/locales/en/exercise.yml

@@ -103,6 +103,7 @@ en:
       export_failed: 'Export has failed.<br>ID: %{id}<br>Title: %{title}<br><br>Error: %{error}'
       label: Export to CodeHarbor
       successfully_exported: 'Exercise has been successfully exported.<br>ID: %{id}<br>Title: %{title}'
+      not_authorized: Authorization with could not be established with CodeHarbor. Is the API Key correct?
     external_users:
       statistics:
         addendum: "* Deltas longer than %{delta} minutes are ignored."

spec/services/exercise_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('exercises.export_codeharbor.not_authorized') }
+
+        it { is_expected.to eql response }
+      end
     end
 
     context 'when an error occurs' do