Merge pull request #6721 from DFE-Digital/also-register-to-sentry-ext…

…ernal-source-import-runs-without-errors Report to Sentry ATS imports without errors
DFE-Digital · Mar 8, 2024 · 4c14df6 · 4c14df6
2 parents 182c4f2 + af2bc7b
commit 4c14df6
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/app/jobs/import_from_vacancy_source_job.rb b/app/jobs/import_from_vacancy_source_job.rb
@@ -71,9 +71,8 @@ def create_failed_imported_vacancy(vacancy)
   end
 
   def report_validation_errors
-    return if @errors.none?
+    failed_percentage = @errors.any? ? ((@errors.count.to_f / @vacancies_count) * 100).round(1) : 0
 
-    failed_percentage = ((@errors.count.to_f / @vacancies_count) * 100).round(1)
     Sentry.with_scope do |scope|
       scope.set_tags(source: @source_name)
       scope.set_context("Import failure rate", { vacancies_in_feed: @vacancies_count,