Merge pull request #158 from basemate/fix_issue_157

fixed image component and added missing img specs, #157

Author: jonasjabari

app/concepts/matestack/ui/core/img/img.haml

@@ -1 +1 @@
-= image_tag(@tag_attributes, ActionController::Base.helpers.asset_path(options[:path]), height: options[:height], width: options[:width], alt: options[:alt]])
+%img{@tag_attributes}

app/concepts/matestack/ui/core/img/img.rb

@@ -1,5 +1,14 @@
 module Matestack::Ui::Core::Img
   class Img < Matestack::Ui::Core::Component::Static
 
+    def setup
+      @tag_attributes.merge!({
+        src: ActionController::Base.helpers.asset_path(options[:path]),
+        height: options[:height],
+        width: options[:width],
+        alt: options[:alt]
+      })
+    end
+
   end
 end

spec/dummy/app/assets/images/matestack-logo.png

@@ -0,0 +1,20 @@
+require_relative '../../support/utils'
+include Utils
+
+describe 'Img Component', type: :feature, js: true do
+
+  it 'Renders an image tag on the page' do
+
+    class ExamplePage < Matestack::Ui::Page
+      def response
+        components {
+          img path: 'matestack-logo.png', width: 500, height: 300, alt: "logo"
+        }
+      end
+    end
+
+    visit '/example'
+
+    expect(page).to have_xpath("//img[contains(@src,'matestack-logo') and @alt='logo' and @width='500' and @height='300']")
+  end
+end