From adf1783cd927e9b882c0de2223d9c178b50e8342 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jan=20Jakes=CC=8C?= <jan@jakes.pro>
Date: Wed, 12 Dec 2018 17:34:45 +0100
Subject: [PATCH] Add test ensuring image renderer understands dimensions both
 with & without 'px'

[MAILPOET-1680]
---
 tests/integration/Newsletter/RendererTest.php | 28 +++++++++++++++++++
 1 file changed, 28 insertions(+)

diff --git a/tests/integration/Newsletter/RendererTest.php b/tests/integration/Newsletter/RendererTest.php
index 3b0493cd77..8b01324fb8 100644
--- a/tests/integration/Newsletter/RendererTest.php
+++ b/tests/integration/Newsletter/RendererTest.php
@@ -348,6 +348,34 @@ class RendererTest extends \MailPoetTest {
     expect($rendered_image)->contains('max-width:100%');
   }
 
+  function testItAdjustImageDimensionsWithPx() {
+    $image = array(
+      'width' => '1000px',
+      'height' => '1000px',
+      'src' => 'https://example.com/image.jpg',
+      'link' => '',
+      'fullWidth' => false,
+      'alt' => 'some test alt text'
+    );
+    $rendered_image = Image::render($image, self::COLUMN_BASE_WIDTH);
+    expect($rendered_image)->contains('width="620"');
+    expect($rendered_image)->contains('max-width:620px');
+  }
+
+  function testItAdjustImageDimensionsWithoutPx() {
+    $image = array(
+      'width' => '1000',
+      'height' => '1000',
+      'src' => 'https://example.com/image.jpg',
+      'link' => '',
+      'fullWidth' => false,
+      'alt' => 'some test alt text'
+    );
+    $rendered_image = Image::render($image, self::COLUMN_BASE_WIDTH);
+    expect($rendered_image)->contains('width="620"');
+    expect($rendered_image)->contains('max-width:620px');
+  }
+
   function testItRendersText() {
     $newsletter = $this->newsletter['body'];
     $template = $newsletter['content']['blocks'][0]['blocks'][0]['blocks'][2];