fix: tests

Author: palpalani

phpstan.neon.dist

@@ -7,5 +7,4 @@ parameters:
         - src
     tmpDir: build/phpstan
     checkOctaneCompatibility: true
-    checkModelProperties: true
-    checkMissingIterableValueType: false
+    checkModelProperties: true

phpunit.xml.dist

@@ -3,20 +3,15 @@
     xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
     xsi:noNamespaceSchemaLocation="vendor/phpunit/phpunit/phpunit.xsd"
     backupGlobals="false"
-    backupStaticAttributes="false"
     bootstrap="vendor/autoload.php"
     colors="true"
-    convertErrorsToExceptions="true"
-    convertNoticesToExceptions="true"
-    convertWarningsToExceptions="true"
     processIsolation="false"
     stopOnFailure="false"
     executionOrder="random"
     failOnWarning="true"
     failOnRisky="true"
     failOnEmptyTestSuite="true"
     beStrictAboutOutputDuringTests="true"
-    verbose="true"
 >
     <testsuites>
         <testsuite name="palPalani Test Suite">

src/SqsQueueReaderServiceProvider.php

@@ -151,7 +151,7 @@ private function removeBatchMessages(Job $job, string $connectionName): void
 
     /**
      * @param  array<string, mixed>  $payload
-     * @return array<int, array<string, mixed>>
+     * @return list<list<array<string, mixed>>>
      */
     private function extractBatchIds(array $payload): array
     {

tests/Pest.php

@@ -0,0 +1,7 @@
+<?php
+
+declare(strict_types=1);
+
+use palPalani\SqsQueueReader\Tests\TestCase;
+
+uses(TestCase::class)->in(__DIR__);

tests/QueueTest.php

@@ -2,42 +2,45 @@
 
 declare(strict_types=1);
 
-namespace palPalani\SqsQueueReader\Tests;
-
 use palPalani\SqsQueueReader\Jobs\DispatcherJob;
 use palPalani\SqsQueueReader\Sqs\Queue;
 
-use function PHPUnit\Framework\assertTrue;
+it('can create payload for dispatcher job with default handler', function () {
+    $content = [
+        'test' => 'test',
+    ];
+
+    $job = new DispatcherJob($content);
+
+    $queue = Mockery::mock(Queue::class)
+        ->makePartial()
+        ->shouldAllowMockingProtectedMethods();
 
-/**
- * Class QueueTest
- */
-class QueueTest extends TestCase
-{
-    /**
-     * @test
-     */
-    public function class_named_is_derived_from_queue_name(): void
-    {
-        $content = [
-            'test' => 'test',
-        ];
+    $payload = $queue->createPayload($job);
+    $decodedPayload = json_decode($payload, true);
 
-        $job = new DispatcherJob($content);
+    expect($payload)->toBeString()
+        ->and($decodedPayload)->toBeArray()
+        ->and($decodedPayload['job'])->toBe('App\Jobs\SqsHandler@handle')
+        ->and($decodedPayload['data'])->toHaveKey('job')
+        ->and($decodedPayload['data'])->toHaveKey('data')
+        ->and($decodedPayload['data']['data'])->toBe($content);
+});
 
-        $queue = $this->getMockBuilder(Queue::class)
-            ->disableOriginalConstructor()
-            ->getMock();
+it('can create plain payload for dispatcher job', function () {
+    $content = [
+        'test' => 'test',
+    ];
 
-        $method = new \ReflectionMethod(
-            Queue::class,
-            'createPayload'
-        );
+    $job = new DispatcherJob($content);
+    $job->setPlain(true);
 
-        $method->setAccessible(true);
+    $queue = Mockery::mock(Queue::class)
+        ->makePartial()
+        ->shouldAllowMockingProtectedMethods();
 
-        $method->invokeArgs($queue, [$job]);
+    $payload = $queue->createPayload($job);
 
-        assertTrue(true);
-    }
-}
+    expect($payload)->toBeString()
+        ->and(json_decode($payload, true))->toBe($content);
+});