Set the job display name as just the class name

Excludes the `@handle` that the `job` key has.

Author: cgrama-ac

.gitignore

@@ -0,0 +1,3 @@
+/vendor
+.idea
+composer.lock

src/Sqs/Queue.php

@@ -76,6 +76,8 @@ public function pop($queue = null)
                 return new SqsJob($this->container, $this->sqs, $response, $this->connectionName, $queue);
             }
 
+            dd($queue, $response);
+
             return new SqsJob($this->container, $this->sqs, $queue, $response);
         }
     }
@@ -92,6 +94,7 @@ private function modifyPayload($payload, $class)
         $body = json_decode($payload['Body'], true);
 
         $body = [
+            'displayName' => $class,
             'job' => $class . '@handle',
             'data' => isset($body['data']) ? $body['data'] : $body
         ];

tests/PlainSqs/QueueTest.php

@@ -4,13 +4,75 @@
 use Aws\Sqs\SqsClient;
 use Dusterio\PlainSqs\Jobs\DispatcherJob;
 use Dusterio\PlainSqs\Sqs\Queue;
+use Illuminate\Container\Container;
+use Illuminate\Contracts\Config\Repository;
 
 /**
  * Class QueueTest
  * @package Dusterio\PlainSqs\Tests
  */
 class QueueTest extends \PHPUnit_Framework_TestCase
 {
+    /**
+     * @var \PHPUnit_Framework_MockObject_MockObject|SqsClient
+     */
+    private $sqsClientMock;
+
+    /**
+     * @var \PHPUnit_Framework_MockObject_MockObject|Container
+     */
+    private $containerMock;
+
+    /**
+     * @var \PHPUnit_Framework_MockObject_MockObject|\Illuminate\Contracts\Config\Repository
+     */
+    private $configMock;
+
+    /**
+     * @var string
+     */
+    private $version;
+
+    protected function setUp()
+    {
+        parent::setUp();
+
+        $this->sqsClientMock = $this
+            ->getMockBuilder(SqsClient::class)
+            ->disableOriginalConstructor()
+            ->setMethods(['receiveMessage'])
+            ->getMock();
+
+        $this->configMock = $this
+            ->getMockBuilder(Repository::class)
+            ->setMethods(['has', 'set', 'get', 'all', 'prepend', 'push'])
+            ->getMock();
+
+        $this->containerMock = $this
+            ->getMockBuilder(Container::class)
+            ->setMethods(['version', 'offsetGet'])
+            ->getMock();
+
+        $this->containerMock
+            ->expects($this->any())
+            ->method('offsetGet')
+            ->will($this->returnValue($this->configMock));
+
+        preg_match("#^\d.\d#", phpversion(), $match);
+
+        $this->version = '5.8.0';
+        switch ($match[0]) {
+            case '5.5':
+                $this->version = '5.0.0';
+                break;
+        }
+
+        $this->containerMock
+            ->expects($this->any())
+            ->method('version')
+            ->will($this->returnValue($this->version));
+    }
+
     /**
      * @test
      */
@@ -35,4 +97,44 @@ public function class_named_is_derived_from_queue_name()
 
         //$response = $method->invokeArgs($queue, [$job]);
     }
-}
+
+    public function testSettingJobDisplayName()
+    {
+
+        $this->sqsClientMock
+            ->expects($this->once())
+            ->method('receiveMessage')
+            ->will($this->returnValue([
+                'Messages' => [
+                    json_encode([
+                        'Body' => json_encode([
+                            'data' => [
+                                'foo' => 'bar'
+                            ]
+                        ])
+                    ])
+                ]
+            ]));
+
+
+        $this->configMock
+            ->expects($this->any())
+            ->method('get')
+            ->with('sqs-plain.handlers')
+            ->will($this->returnValue([
+                'default-queue' => 'SomeClass'
+            ]));
+
+        $queue = new Queue($this->sqsClientMock, 'default-queue');
+
+        $queue->setContainer($this->containerMock);
+
+        if ($this->version !== '5.0.0') {
+            $queue->setConnectionName('default-connection');
+        }
+
+        $result = $queue->pop();
+
+        $this->assertEquals('SomeClass', $result->resolveName());
+    }
+}