refactor: clear up image code, add e2e-test

luhmann · Mar 18, 2018 · 2a1eedf · 2a1eedf
1 parent 3ce2c7a
commit 2a1eedf
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/cypress/integration/home/search_beers_spec.js b/cypress/integration/home/search_beers_spec.js
@@ -101,6 +101,11 @@ describe('Searching for beers', () => {
  });
 
  it('should display complete and correct information on a beer', () => {
+ cy
+ .get(sel('beer-card'))
+ .first()
+ .find(sel('beer-card-img'));
+
  cy
  .get(sel('beer-card'))
  .first()

diff --git a/src/Beer.re b/src/Beer.re
@@ -31,7 +31,7 @@ let make =
  | None => ReasonReact.nullElement
  | Some(img_url) =>
  <div
- className="bg-contain bg-top bg-no-repeat h-48 sm:h-auto beerImg-width"
+ className="bg-contain bg-top bg-no-repeat h-48 sm:h-auto beerImg-width test-beer-card-img"
  style=(
  ReactDOMRe.Style.make(~backgroundImage={j|url($img_url)|j}, ())
  )