From 5d8b3090f7f29b0b8e4f452f5cd130bf6d7c02e0 Mon Sep 17 00:00:00 2001
From: Harry Bragg <harry.bragg@graze.com>
Date: Mon, 5 Oct 2015 09:47:02 +0100
Subject: [PATCH] get timezone from string, but set locally to avoid NOTICEs

---
 src/Response/Response.php                   |  7 +------
 tests/unit/Response/ResponseFactoryTest.php | 14 +++++++-------
 tests/unit/Response/ResponseTest.php        | 18 +++++++-----------
 3 files changed, 15 insertions(+), 24 deletions(-)

diff --git a/src/Response/Response.php b/src/Response/Response.php
index 6f1d2b0..4bed777 100644
--- a/src/Response/Response.php
+++ b/src/Response/Response.php
@@ -4,7 +4,6 @@
 
 use DateTimeImmutable;
 use DateTimeInterface;
-use DateTimeZone;
 use GuzzleHttp\Message\ResponseInterface as GuzzleResponseInterface;
 use Illuminate\Support\Collection;
 
@@ -72,11 +71,7 @@ public function __construct(GuzzleResponseInterface $response)
         $this->statusCode = (int)$this->popField('statusCode');
         $this->statusReason = $this->popField('statusReason');
         $this->callId = $this->popField('callId');
-        $this->time = DateTimeImmutable::createFromFormat(
-            static::DATE_TIME_FORMAT,
-            $this->popField('time'),
-            new DateTimeZone('UTC')
-        );
+        $this->time = DateTimeImmutable::createFromFormat(static::DATE_TIME_FORMAT, $this->popField('time'));
     }
 
     /**
diff --git a/tests/unit/Response/ResponseFactoryTest.php b/tests/unit/Response/ResponseFactoryTest.php
index b878844..75f2384 100644
--- a/tests/unit/Response/ResponseFactoryTest.php
+++ b/tests/unit/Response/ResponseFactoryTest.php
@@ -2,9 +2,7 @@
 
 namespace Graze\Gigya\Test\Unit\Response;
 
-use DateTime;
 use DateTimeImmutable;
-use DateTimeZone;
 use Graze\Gigya\Exceptions\UnknownResponseException;
 use Graze\Gigya\Response\Response;
 use Graze\Gigya\Response\ResponseCollectionInterface;
@@ -19,6 +17,7 @@
 
 class ResponseFactoryTest extends TestCase
 {
+
     /**
      * @var ResponseFactory
      */
@@ -29,6 +28,11 @@ class ResponseFactoryTest extends TestCase
      */
     private $validator;
 
+    public static function setUpBeforeClass()
+    {
+        date_default_timezone_set('UTC');
+    }
+
     public function setUp()
     {
         $this->validator = m::mock('Graze\Gigya\Validation\GigyaResponseValidatorInterface');
@@ -60,11 +64,7 @@ public function testAccountModel()
         static::assertEquals("e6f891ac17f24810bee6eb533524a152", $gigyaResponse->getCallId());
         static::assertInstanceOf('DateTimeInterface', $gigyaResponse->getTime());
         static::assertEquals(
-            DateTimeImmutable::createFromFormat(
-                Response::DATE_TIME_FORMAT,
-                "2015-03-22T11:42:25.943Z",
-                new DateTimeZone('UTC')
-            ),
+            DateTimeImmutable::createFromFormat(Response::DATE_TIME_FORMAT, "2015-03-22T11:42:25.943Z"),
             $gigyaResponse->getTime()
         );
         $data = $gigyaResponse->getData();
diff --git a/tests/unit/Response/ResponseTest.php b/tests/unit/Response/ResponseTest.php
index dc59f3c..9b8656c 100644
--- a/tests/unit/Response/ResponseTest.php
+++ b/tests/unit/Response/ResponseTest.php
@@ -3,19 +3,19 @@
 namespace Graze\Gigya\Test\Unit\Response;
 
 use DateTime;
-use DateTimeZone;
 use Graze\Gigya\Response\Response;
 use Graze\Gigya\Test\TestCase;
 
 class ResponseTest extends TestCase
 {
+    public static function setUpBeforeClass()
+    {
+        date_default_timezone_set('UTC');
+    }
+
     public function testDateFormat()
     {
-        $time = DateTime::createFromFormat(
-            Response::DATE_TIME_FORMAT,
-            '2015-03-22T11:42:25.943Z',
-            new DateTimeZone('UTC')
-        );
+        $time = DateTime::createFromFormat(Response::DATE_TIME_FORMAT, '2015-03-22T11:42:25.943Z');
 
         static::assertInstanceOf('DateTimeInterface', $time);
         static::assertEquals("2015", $time->format('Y'));
@@ -30,11 +30,7 @@ public function testDateFormat()
 
     public function testOtherTimeZoneFormat()
     {
-        $time = DateTime::createFromFormat(
-            Response::DATE_TIME_FORMAT,
-            '2015-03-22T11:42:25.943+02:00',
-            new DateTimeZone('UTC')
-        );
+        $time = DateTime::createFromFormat(Response::DATE_TIME_FORMAT, '2015-03-22T11:42:25.943+02:00');
 
         static::assertInstanceOf('DateTimeInterface', $time);
         static::assertEquals("2015", $time->format('Y'));