From 5be9feae4fb01e868c71005c97d29b0ad3d3bc97 Mon Sep 17 00:00:00 2001
From: Dennis K <denniskraft87@googlemail.com>
Date: Thu, 24 May 2018 12:51:24 +0200
Subject: [PATCH 1/4] PWA-334 wip

---
 libraries/common/components/Image/index.jsx   |  1 +
 libraries/ui-shared/Availability/index.jsx    |  2 +-
 libraries/ui-shared/FavoritesButton/index.jsx |  2 +-
 libraries/ui-shared/TaxDisclaimer/index.jsx   | 10 ++++++----
 themes/gmd                                    |  2 +-
 5 files changed, 10 insertions(+), 7 deletions(-)

diff --git a/libraries/common/components/Image/index.jsx b/libraries/common/components/Image/index.jsx
index 37c2848765..5c98aaa5f2 100644
--- a/libraries/common/components/Image/index.jsx
+++ b/libraries/common/components/Image/index.jsx
@@ -183,6 +183,7 @@ class Image extends Component {
           style={inlineStyles}
           alt={this.props.alt}
           role="presentation"
+          data-test-id="image"
         />
       );
     }
diff --git a/libraries/ui-shared/Availability/index.jsx b/libraries/ui-shared/Availability/index.jsx
index fea944f2f7..1a5cd93c75 100644
--- a/libraries/ui-shared/Availability/index.jsx
+++ b/libraries/ui-shared/Availability/index.jsx
@@ -35,7 +35,7 @@ const Availability = ({
   }
 
   return (
-    <div className={`${className} ${style}`}>
+    <div className={`${className} ${style}`} data-test-id={`availabilityText: ${text}`}>
       {text}
     </div>
   );
diff --git a/libraries/ui-shared/FavoritesButton/index.jsx b/libraries/ui-shared/FavoritesButton/index.jsx
index 4c253b0cc4..8851b104e3 100644
--- a/libraries/ui-shared/FavoritesButton/index.jsx
+++ b/libraries/ui-shared/FavoritesButton/index.jsx
@@ -144,7 +144,7 @@ class FavoritesButton extends Component {
         aria-label={this.getLabel()}
         className={`${className} ${this.props.className}`}
         onClick={this.handleClick}
-        data-test-id="button"
+        data-test-id="favoriteButton"
       >
         <Ripple
           className={`${styles.ripple} ${this.props.rippleClassName}`}
diff --git a/libraries/ui-shared/TaxDisclaimer/index.jsx b/libraries/ui-shared/TaxDisclaimer/index.jsx
index e73c7347a3..668bca89af 100644
--- a/libraries/ui-shared/TaxDisclaimer/index.jsx
+++ b/libraries/ui-shared/TaxDisclaimer/index.jsx
@@ -9,10 +9,12 @@ import styles from './style';
  */
 const TaxDisclaimer = () => (
   showTaxDisclaimer ? (
-    <I18n.Text
-      className={styles}
-      string="product.tax_disclaimer"
-    />
+    <div data-test-id="taxDisclaimer">
+      <I18n.Text
+        className={styles}
+        string="product.tax_disclaimer"
+      />
+    </div>
   ) : null
 );
 
diff --git a/themes/gmd b/themes/gmd
index 9f13163847..4c95d7c128 160000
--- a/themes/gmd
+++ b/themes/gmd
@@ -1 +1 @@
-Subproject commit 9f131638472ca6d1ab2d4e035295b05eca53bdea
+Subproject commit 4c95d7c1285b1ca96adacd7bd7b4e6ec2804779e

From b00b947e7e89fc9742d97e16e1341cbcd00b3ce8 Mon Sep 17 00:00:00 2001
From: Dennis K <denniskraft87@googlemail.com>
Date: Fri, 25 May 2018 15:30:57 +0200
Subject: [PATCH 2/4] PWA-334 added test ids

---
 extensions/@shopgate-product-reviews      | 2 +-
 extensions/@shopgate-tracking-ga-native   | 2 +-
 libraries/ui-shared/PriceInfo/index.jsx   | 1 +
 libraries/ui-shared/RatingStars/index.jsx | 2 +-
 4 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/extensions/@shopgate-product-reviews b/extensions/@shopgate-product-reviews
index 78da5b09f4..66db998561 160000
--- a/extensions/@shopgate-product-reviews
+++ b/extensions/@shopgate-product-reviews
@@ -1 +1 @@
-Subproject commit 78da5b09f493130e25f66e739eac3be79f4e0f7f
+Subproject commit 66db998561ea364c5e833c92f6b41fd508a1e3e1
diff --git a/extensions/@shopgate-tracking-ga-native b/extensions/@shopgate-tracking-ga-native
index 47d9081866..0fe6a14394 160000
--- a/extensions/@shopgate-tracking-ga-native
+++ b/extensions/@shopgate-tracking-ga-native
@@ -1 +1 @@
-Subproject commit 47d9081866cdf67eaa9c78cef728fd39468b304a
+Subproject commit 0fe6a143942126398f3565bf0e22bb24e54a2de7
diff --git a/libraries/ui-shared/PriceInfo/index.jsx b/libraries/ui-shared/PriceInfo/index.jsx
index 280d6aa3bf..19e79103b6 100644
--- a/libraries/ui-shared/PriceInfo/index.jsx
+++ b/libraries/ui-shared/PriceInfo/index.jsx
@@ -13,6 +13,7 @@ const PriceInfo = ({ className, text }) => (
   <div
     className={`${styles} ${className}`}
     dangerouslySetInnerHTML={{ __html: text }}
+    data-test-id={`priceInfo: ${text}`}
   />
 );
 
diff --git a/libraries/ui-shared/RatingStars/index.jsx b/libraries/ui-shared/RatingStars/index.jsx
index 409b325cc2..56b03eb1da 100644
--- a/libraries/ui-shared/RatingStars/index.jsx
+++ b/libraries/ui-shared/RatingStars/index.jsx
@@ -145,7 +145,7 @@ class RatingStars extends Component {
     ];
 
     return (
-      <div className={className} aria-label={this.getTextualFinal(ratedStars)}>
+      <div className={className} aria-label={this.getTextualFinal(ratedStars)} data-test-id={`ratedStars: ${ratedStars}`}>
         <div className={styles.emptyStars}>
           {emptyStars}
         </div>

From 3965c3ce0b45c49138a0b16a49a2d8e8993833b6 Mon Sep 17 00:00:00 2001
From: Dennis K <denniskraft87@googlemail.com>
Date: Fri, 25 May 2018 17:47:51 +0200
Subject: [PATCH 3/4] PWA-334 updated snapshots

---
 .../components/Image/__snapshots__/spec.jsx.snap   |  1 +
 .../Availability/__snapshots__/spec.jsx.snap       |  3 +++
 .../FavoritesButton/__snapshots__/spec.jsx.snap    |  4 ++--
 .../RatingStars/__snapshots__/spec.jsx.snap        |  5 +++++
 .../TaxDisclaimer/__snapshots__/spec.jsx.snap      | 14 +++++++++-----
 themes/gmd                                         |  2 +-
 themes/ios11                                       |  2 +-
 7 files changed, 22 insertions(+), 9 deletions(-)

diff --git a/libraries/common/components/Image/__snapshots__/spec.jsx.snap b/libraries/common/components/Image/__snapshots__/spec.jsx.snap
index 7e337a7bab..cda76831ed 100644
--- a/libraries/common/components/Image/__snapshots__/spec.jsx.snap
+++ b/libraries/common/components/Image/__snapshots__/spec.jsx.snap
@@ -66,6 +66,7 @@ exports[`<Image /> should render placeholders if src is null 1`] = `
     <img
       alt={null}
       className="css-1trsaz6"
+      data-test-id="image"
       role="presentation"
       src="foo/bar?w=440&h=440&q=70&zc=resize&fillc=FFFFFF"
       style={Object {}}
diff --git a/libraries/ui-shared/Availability/__snapshots__/spec.jsx.snap b/libraries/ui-shared/Availability/__snapshots__/spec.jsx.snap
index 71553b0058..77f6a99cfd 100644
--- a/libraries/ui-shared/Availability/__snapshots__/spec.jsx.snap
+++ b/libraries/ui-shared/Availability/__snapshots__/spec.jsx.snap
@@ -27,6 +27,7 @@ exports[`<Availability /> should render if state is "ok" when "showWhenAvailable
 >
   <div
     className=" css-1ixzglu"
+    data-test-id="availabilityText: Available"
   >
     Available
   </div>
@@ -42,6 +43,7 @@ exports[`<Availability /> should render if state is "warning" 1`] = `
 >
   <div
     className=" css-14jedxl"
+    data-test-id="availabilityText: Only 5 left"
   >
     Only 5 left
   </div>
@@ -57,6 +59,7 @@ exports[`<Availability /> should render if state is "warning" 2`] = `
 >
   <div
     className=" css-qsvazi"
+    data-test-id="availabilityText: Out of stock"
   >
     Out of stock
   </div>
diff --git a/libraries/ui-shared/FavoritesButton/__snapshots__/spec.jsx.snap b/libraries/ui-shared/FavoritesButton/__snapshots__/spec.jsx.snap
index 240cea12ee..5d5932a3ef 100644
--- a/libraries/ui-shared/FavoritesButton/__snapshots__/spec.jsx.snap
+++ b/libraries/ui-shared/FavoritesButton/__snapshots__/spec.jsx.snap
@@ -32,7 +32,7 @@ exports[`<FavoritesButton /> should only render when no favorites set 1`] = `
       <button
         aria-label="favorites.add"
         className="css-orm6vn "
-        data-test-id="button"
+        data-test-id="favoriteButton"
         onClick={[Function]}
       >
         <Ripple
@@ -116,7 +116,7 @@ exports[`<FavoritesButton /> should render when favorites set 1`] = `
       <button
         aria-label="favorites.remove"
         className="css-orm6vn "
-        data-test-id="button"
+        data-test-id="favoriteButton"
         onClick={[Function]}
       >
         <Ripple
diff --git a/libraries/ui-shared/RatingStars/__snapshots__/spec.jsx.snap b/libraries/ui-shared/RatingStars/__snapshots__/spec.jsx.snap
index 466008e2b4..aae6ca5b0b 100644
--- a/libraries/ui-shared/RatingStars/__snapshots__/spec.jsx.snap
+++ b/libraries/ui-shared/RatingStars/__snapshots__/spec.jsx.snap
@@ -4,6 +4,7 @@ exports[`<RatingStars /> renders with value of 0 1`] = `
 <div
   aria-label="reviews.rating_stars"
   className="css-13azwyo "
+  data-test-id="ratedStars: 0"
 >
   <div
     className="css-4s85e9"
@@ -74,6 +75,7 @@ exports[`<RatingStars /> renders with value of 50 1`] = `
 <div
   aria-label="reviews.rating_stars"
   className="css-13azwyo "
+  data-test-id="ratedStars: 2.5"
 >
   <div
     className="css-4s85e9"
@@ -178,6 +180,7 @@ exports[`<RatingStars /> renders with value of 100 1`] = `
 <div
   aria-label="reviews.rating_stars"
   className="css-13azwyo "
+  data-test-id="ratedStars: 5"
 >
   <div
     className="css-4s85e9"
@@ -304,6 +307,7 @@ exports[`<RatingStars /> should change rating on click 1`] = `
 <div
   aria-label="reviews.rating_stars"
   className="css-13azwyo "
+  data-test-id="ratedStars: 5"
 >
   <div
     className="css-4s85e9"
@@ -460,6 +464,7 @@ exports[`<RatingStars /> should change rating on click 2`] = `
 <div
   aria-label="reviews.rating_stars"
   className="css-13azwyo "
+  data-test-id="ratedStars: 3.5"
 >
   <div
     className="css-4s85e9"
diff --git a/libraries/ui-shared/TaxDisclaimer/__snapshots__/spec.jsx.snap b/libraries/ui-shared/TaxDisclaimer/__snapshots__/spec.jsx.snap
index 6e0e98eb45..d6f24ea136 100644
--- a/libraries/ui-shared/TaxDisclaimer/__snapshots__/spec.jsx.snap
+++ b/libraries/ui-shared/TaxDisclaimer/__snapshots__/spec.jsx.snap
@@ -3,9 +3,13 @@
 exports[`<TaxDisclaimer /> should display null 1`] = `""`;
 
 exports[`<TaxDisclaimer /> should display the component 1`] = `
-<Translate
-  className="css-fazkyv"
-  params={Object {}}
-  string="product.tax_disclaimer"
-/>
+<div
+  data-test-id="taxDisclaimer"
+>
+  <Translate
+    className="css-fazkyv"
+    params={Object {}}
+    string="product.tax_disclaimer"
+  />
+</div>
 `;
diff --git a/themes/gmd b/themes/gmd
index 1384f12ac5..b73f2b99f3 160000
--- a/themes/gmd
+++ b/themes/gmd
@@ -1 +1 @@
-Subproject commit 1384f12ac5e9dafb281775862fda96004e238083
+Subproject commit b73f2b99f363432e93306bee48857f7b80e2c650
diff --git a/themes/ios11 b/themes/ios11
index 1e93f09b3d..0f783461c2 160000
--- a/themes/ios11
+++ b/themes/ios11
@@ -1 +1 @@
-Subproject commit 1e93f09b3d839c0a1a64e49150e86ff07f2edd3b
+Subproject commit 0f783461c2bb674e9b2a690da478f08964a57804

From 207d5ec3144e48783168d41b22231bc3dfd69cc4 Mon Sep 17 00:00:00 2001
From: Dennis K <denniskraft87@googlemail.com>
Date: Fri, 25 May 2018 18:06:31 +0200
Subject: [PATCH 4/4] PWA-334 fix for fav button selector

---
 themes/gmd   | 2 +-
 themes/ios11 | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/themes/gmd b/themes/gmd
index b73f2b99f3..53a26bc4ef 160000
--- a/themes/gmd
+++ b/themes/gmd
@@ -1 +1 @@
-Subproject commit b73f2b99f363432e93306bee48857f7b80e2c650
+Subproject commit 53a26bc4ef6c897bc042f5ff7009b3830e388343
diff --git a/themes/ios11 b/themes/ios11
index 0f783461c2..a4326f8279 160000
--- a/themes/ios11
+++ b/themes/ios11
@@ -1 +1 @@
-Subproject commit 0f783461c2bb674e9b2a690da478f08964a57804
+Subproject commit a4326f8279921d68ab26770b1d59d81866927fd9