Remove code features unsupported by older versions of PHP LRN-43664

Author: david-scarratt-lrn

src/Services/PreHashStringFactory.php

@@ -13,7 +13,10 @@ class PreHashStringFactory
         LegacyPreHashString::class,
     ];
 
-    protected array $validServices;
+    /**
+     * @var array
+     */
+    protected /* array */ $validServices;
 
     public function __construct()
     {

src/Services/PreHashStrings/LegacyPreHashString.php

@@ -94,11 +94,15 @@ class LegacyPreHashString implements PreHashStringInterface
         'domain',
     ];
 
-    /** Service name to generate a pre-hash string for */
-    protected string $service;
+    /** Service name to generate a pre-hash string for
+     * @var string
+     */
+    protected /* string */ $service;
 
-    /** V1-compat strings need the secret to be part of the pre-hash string */
-    protected bool $v1Compat;
+    /** V1-compat strings need the secret to be part of the pre-hash string
+     * @var bool
+     */
+    protected /* bool */ $v1Compat;
 
     public function __construct(string $service, bool $v1Compat = false)
     {

tests/Request/InitTest.php

@@ -153,6 +153,9 @@ public function testNullRequestPacketAndActionGeneratesValidInit()
         $this->assertInstanceOf(Init::class, $initObject);
     }
 
+    /**
+     * @requires PHPUnit >= 9.6
+     */
     public function testMetaWithTelemetryOnlyAddsSdkProp()
     {
         list($service, $security, $secret, $request, $action) = ParamsFixture::getWorkingQuestionsApiParams();
@@ -170,6 +173,9 @@ public function testMetaWithTelemetryOnlyAddsSdkProp()
         $this->assertEquals(1, count((array) $generatedObject->meta));
     }
 
+    /**
+     * @requires PHPUnit >= 9.6
+     */
     public function testRequestWithTelemetryPreservesOtherMetaProps()
     {
         list($service, $security, $secret, $request, $action) = ParamsFixture::getWorkingQuestionsApiParams();
@@ -195,6 +201,9 @@ public function testRequestWithTelemetryPreservesOtherMetaProps()
         $this->assertObjectHasProperty('sdk', $generatedObject->meta);
     }
 
+    /**
+     * @requires PHPUnit >= 9.6
+     */
     public function testRequestWithoutTelemetryPreservesEmptyMeta()
     {
         // We disable telemetry to be able to reliably test signature generation. Added telemetry
@@ -210,6 +219,9 @@ public function testRequestWithoutTelemetryPreservesEmptyMeta()
         $this->assertObjectNotHasProperty('meta', $generatedObject);
     }
 
+    /**
+     * @requires PHPUnit >= 9.6
+     */
     public function testRequestWithoutTelemetryPreservesFilledMeta()
     {
         // We disable telemetry to be able to reliably test signature generation. Added telemetry

tests/Services/LegacySignaturesTest.php

@@ -11,7 +11,7 @@
 
 class LegacySignaturesTest extends AbstractTestCase
 {
-    private SignatureFactory $signatureFactory;
+    private $signatureFactory;
 
     public function setUp(): void
     {