Fixes: #5281@1.2h; View specs and minor CR change from #50065281-workbench-import-structural-bug-reading-zip

  - CR change from #5006, severity of zip file messages :warning -> :error
  - View specs for showing import_messages

3 files changed, 44 insertions, 0 deletions

diff --git a/spec/views/companies/edit.html.erb_spec.rb b/spec/views/companies/edit.html.erb_spec.rb
index 8aaf705ab..b85b9aa8b 100644
--- a/spec/views/companies/edit.html.erb_spec.rb
+++ b/spec/views/companies/edit.html.erb_spec.rb
@@ -9,6 +9,7 @@ describe "/companies/edit", :type => :view do
   describe "form" do
     it "should render input for name" do
       render
+        require 'pry'; binding.pry
       expect(rendered).to have_selector("form") do
         with_tag "input[type=text][name='company[name]'][value=?]", company.name
       end
diff --git a/spec/views/imports/show.html.slim_spec.rb b/spec/views/imports/show.html.slim_spec.rb
new file mode 100644
index 000000000..f30202231
--- /dev/null
+++ b/spec/views/imports/show.html.slim_spec.rb
@@ -0,0 +1,42 @@
+RSpec.describe '/imports/show', type: :view do
+  let(:workbench){ create :workbench }
+  let(:workbench_import){ create :workbench_import, workbench: workbench }
+  let!( :messages ) {[
+    create(:corrupt_zip_file, import: workbench_import),
+    create(:inconsistent_zip_file, import: workbench_import),
+  ]}
+
+
+  before do
+    assign :import, workbench_import.decorate( context: {workbench: workbench} )
+    render
+  end
+
+  it 'shows the correct record...' do
+    # ... zip file name
+    expect(rendered).to have_selector('.dl-def') do
+      with_text workbench_import.file
+    end
+
+    # ... messages
+    messages.each do | message |
+      # require 'htmlbeautifier'
+      # b = HtmlBeautifier.beautify(rendered, indent: '  ')
+      # require 'pry'; binding.pry
+      expect(rendered).to have_selector('dl#import_messages dt.import_message') do
+        with_text message.criticity
+      end
+      expect(rendered).to have_selector('dl#import_messages dd.import_message') do
+        with_text rendered_message( message )
+      end
+    end
+  end
+
+
+  def rendered_message message
+    Object.new.extend(CommonHelper).tap do |helper|
+      return I18n.t(message.message_key, helper.string_keys_to_symbols( message.message_attributes )) 
+    end
+  end
+
+end
diff --git a/spec/views/vehicle_journeys/new.html.erb_spec.rb b/spec/views/vehicle_journeys/new.html.erb_spec.rb
index 546e89ac8..40b7fb049 100644
--- a/spec/views/vehicle_journeys/new.html.erb_spec.rb
+++ b/spec/views/vehicle_journeys/new.html.erb_spec.rb
@@ -9,6 +9,7 @@ describe "/vehicle_journeys/new", :type => :view do
 
   it "renders _form" do
     render
+      require 'pry'; binding.pry
     expect(view).to render_template(:partial => "_form")
   end