From 95ef2df20d8aaec363bc10d2a2e56107e7df580e Mon Sep 17 00:00:00 2001 From: Phil Lee Date: Mon, 27 Feb 2023 11:11:16 +0000 Subject: [PATCH] add #errors? to summary component --- ...ulk_upload_error_summary_table_component.rb | 4 ++++ ...pload_error_summary_table_component_spec.rb | 18 ++++++++++++++++++ 2 files changed, 22 insertions(+) diff --git a/app/components/bulk_upload_error_summary_table_component.rb b/app/components/bulk_upload_error_summary_table_component.rb index 510e8e7da..a3d295110 100644 --- a/app/components/bulk_upload_error_summary_table_component.rb +++ b/app/components/bulk_upload_error_summary_table_component.rb @@ -18,6 +18,10 @@ class BulkUploadErrorSummaryTableComponent < ViewComponent::Base .sort_by { |el| el[0][0].rjust(3, "0") } end + def errors? + sorted_errors.present? + end + private def display_threshold diff --git a/spec/components/bulk_upload_error_summary_table_component_spec.rb b/spec/components/bulk_upload_error_summary_table_component_spec.rb index a1b3c7247..a7468d70a 100644 --- a/spec/components/bulk_upload_error_summary_table_component_spec.rb +++ b/spec/components/bulk_upload_error_summary_table_component_spec.rb @@ -95,4 +95,22 @@ RSpec.describe BulkUploadErrorSummaryTableComponent, type: :component do ]) end end + + describe "#errors?" do + context "when there are no errors" do + it "returns false" do + expect(component).not_to be_errors + end + end + + context "when there are errors" do + before do + create(:bulk_upload_error, bulk_upload:, col: "A", row: 2, field: "field_1") + end + + it "returns true" do + expect(component).to be_errors + end + end + end end