From 248654176a927037cecff133d69abd7ebc3ca22a Mon Sep 17 00:00:00 2001
From: Paul Redmond <paulrredmond@gmail.com>
Date: Sun, 29 May 2016 22:30:58 -0700
Subject: [PATCH 1/2] Add Spec for YoutubeConverter Extension

Provide a spec for the YoutubeConverter for better test coverage and to
fix regression bug reported in issue #5.

Fixes #5
---
 .../Extensions/YoutubeConverterSpec.php       | 169 ++++++++++++++++++
 src/Converter/Extensions/YoutubeConverter.php |   6 +-
 2 files changed, 174 insertions(+), 1 deletion(-)
 create mode 100644 spec/Converter/Extensions/YoutubeConverterSpec.php

diff --git a/spec/Converter/Extensions/YoutubeConverterSpec.php b/spec/Converter/Extensions/YoutubeConverterSpec.php
new file mode 100644
index 0000000..8135ebb
--- /dev/null
+++ b/spec/Converter/Extensions/YoutubeConverterSpec.php
@@ -0,0 +1,169 @@
+<?php
+
+namespace spec\Predmond\HtmlToAmp\Converter\Extensions;
+
+use Prophecy\Argument;
+use PhpSpec\ObjectBehavior;
+use Predmond\HtmlToAmp\Element;
+use League\Event\EventInterface;
+use Predmond\HtmlToAmp\ElementInterface;
+use Predmond\HtmlToAmp\Converter\Extensions\YoutubeConverter;
+
+class YoutubeConverterSpec extends ObjectBehavior
+{
+    function it_is_initializable()
+    {
+        $this->shouldHaveType(YoutubeConverter::class);
+    }
+
+    function it_can_convert_a_youtube_iframe_tag(
+        EventInterface $event,
+        ElementInterface $element,
+        Element $writeableElement,
+        Element $ampYoutubeElement
+    ) {
+        $event
+            ->stopPropagation()
+            ->shouldBeCalled()
+        ;
+        
+        $writeableElement
+            ->appendChild($ampYoutubeElement)
+            ->shouldBeCalled()
+        ;
+        
+        $element
+            ->getAttribute('src')
+            ->shouldBeCalled()
+            ->willReturn('http://www.youtube.com/embed/XGSy3_Czz8k?autoplay=1')
+        ;
+
+        $element
+            ->createWritableElement('div', [
+                'class' => 'youtube-container'
+            ])
+            ->shouldBeCalled()
+            ->willReturn($writeableElement)
+        ;
+
+        $element->replaceWith($writeableElement)->shouldBeCalled();
+
+        $element
+            ->createWritableElement(
+                'amp-youtube',
+                [
+                    'data-videoid' => 'XGSy3_Czz8k',
+                    "layout" => "responsive",
+                    "width" => "560",
+                    "height" => "315"
+                ]
+            )
+            ->shouldBeCalled()
+            ->willReturn($ampYoutubeElement)
+        ;
+
+        $this->handleIframe($event, $element);
+    }
+
+    function it_skips_converting_non_youtube_iframes(
+        EventInterface $event,
+        ElementInterface $element
+    ) {
+        $element
+            ->getAttribute('src')
+            ->shouldBeCalled()
+            ->willReturn('http://metube.com/embed/XGSy3_Czz8k')
+        ;
+
+        $event->stopPropagation()->shouldNotBeCalled();
+        $element->createWritableElement('div', [
+            'class' => 'youtube-container'
+        ])->shouldNotBeCalled();
+
+        $this->handleIframe($event, $element);
+    }
+
+    function it_converts_a_youtube_object_tag(
+        EventInterface $event,
+        ElementInterface $element,
+        Element $child,
+        Element $containerElement,
+        Element $ampYoutubeElement
+    ) {
+        $event
+            ->stopPropagation()
+            ->shouldBeCalled()
+        ;
+        
+        $element
+            ->getChildren()
+            ->shouldBeCalled()
+            ->willReturn([$child])
+        ;
+
+        $child
+            ->getAttribute('value')
+            ->shouldBeCalled()
+            ->willReturn('http://www.youtube.com/embed/XGSy3_Czz8k?autoplay=1')
+        ;
+        
+        $element
+            ->createWritableElement('div', [
+                'class' => 'youtube-container'
+            ])
+            ->shouldBeCalled()
+            ->willReturn($containerElement)
+        ;
+
+        $containerElement
+            ->appendChild($ampYoutubeElement)
+            ->shouldBeCalled()
+        ;
+        
+        $element
+            ->replaceWith($containerElement)
+            ->shouldBeCalled()
+        ;
+
+        $element
+            ->createWritableElement(
+                'amp-youtube',
+                [
+                    'data-videoid' => 'XGSy3_Czz8k',
+                    "layout" => "responsive",
+                    "width" => "560",
+                    "height" => "315"
+                ]
+            )
+            ->shouldBeCalled()
+            ->willReturn($ampYoutubeElement)
+        ;
+
+        $this->handleObject($event, $element);
+    }
+
+    function it_skips_converting_non_youtube_object_tags(
+        EventInterface $event,
+        ElementInterface $element,
+        ElementInterface $childElement
+    ) {
+        $element
+            ->getChildren()
+            ->shouldBeCalled()
+            ->willReturn([$childElement])
+        ;
+
+        $childElement
+            ->getAttribute('value')
+            ->shouldBeCalled()
+            ->willReturn('http://metube.com/embed/XGSy3_Czz8k')
+        ;
+
+        $event->stopPropagation()->shouldNotBeCalled();
+        $element->createWritableElement('div', [
+            'class' => 'youtube-container'
+        ])->shouldNotBeCalled();
+
+        $this->handleObject($event, $element);
+    }
+}
diff --git a/src/Converter/Extensions/YoutubeConverter.php b/src/Converter/Extensions/YoutubeConverter.php
index 7f00f45..1d8fe27 100644
--- a/src/Converter/Extensions/YoutubeConverter.php
+++ b/src/Converter/Extensions/YoutubeConverter.php
@@ -33,7 +33,11 @@ public function handleObject(EventInterface $event, ElementInterface $element)
         }
 
         if ($embedCode !== false) {
-            $container = $element->createWritableElement('div', 'youtube-container');
+
+            $container = $element->createWritableElement('div', [
+                'class' => 'youtube-container'
+            ]);
+
             $container->appendChild($this->createAmpTag($element, $embedCode));
             $element->replaceWith($container);
             $event->stopPropagation();

From 57b6e4e496ec5fdc0666d99d326732a038867e61 Mon Sep 17 00:00:00 2001
From: Paul Redmond <paulrredmond@gmail.com>
Date: Sun, 29 May 2016 22:34:33 -0700
Subject: [PATCH 2/2] OCD Kicking In D:

---
 spec/Converter/Extensions/YoutubeConverterSpec.php | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/spec/Converter/Extensions/YoutubeConverterSpec.php b/spec/Converter/Extensions/YoutubeConverterSpec.php
index 8135ebb..020f41a 100644
--- a/spec/Converter/Extensions/YoutubeConverterSpec.php
+++ b/spec/Converter/Extensions/YoutubeConverterSpec.php
@@ -130,9 +130,9 @@ function it_converts_a_youtube_object_tag(
                 'amp-youtube',
                 [
                     'data-videoid' => 'XGSy3_Czz8k',
-                    "layout" => "responsive",
-                    "width" => "560",
-                    "height" => "315"
+                    'layout' => 'responsive',
+                    'width' => '560',
+                    'height' => '315'
                 ]
             )
             ->shouldBeCalled()