From 1d38f0097dd4f87ff3cfe4664e4cc3df258a2a85 Mon Sep 17 00:00:00 2001
From: Marco Pivetta <ocramius@gmail.com>
Date: Thu, 7 Jul 2022 17:00:12 +0200
Subject: [PATCH] Applying type improvements brought in by newer psalm version

---
 psalm-baseline.xml                          | 30 +++++++--------------
 psalm.xml.dist                              |  2 +-
 src/StringWrapper/AbstractStringWrapper.php |  2 +-
 test/SplQueueTest.php                       |  6 +++--
 4 files changed, 15 insertions(+), 25 deletions(-)

diff --git a/psalm-baseline.xml b/psalm-baseline.xml
index aaf4bc7c..029329ba 100644
--- a/psalm-baseline.xml
+++ b/psalm-baseline.xml
@@ -1,5 +1,5 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<files psalm-version="4.9.3@4c262932602b9bbab5020863d1eb22d49de0dbf4">
+<files psalm-version="4.24.0@06dd975cb55d36af80f242561738f16c5f58264f">
   <file src="src/AbstractOptions.php">
     <DocblockTypeContradiction occurrences="1">
       <code>! is_array($options) &amp;&amp; ! $options instanceof Traversable</code>
@@ -61,35 +61,17 @@
     </RedundantConditionGivenDocblockType>
   </file>
   <file src="src/ArrayUtils.php">
-    <DocblockTypeContradiction occurrences="1">
-      <code>! is_array($iterator) &amp;&amp; ! $iterator instanceof Traversable</code>
-    </DocblockTypeContradiction>
     <MixedArgumentTypeCoercion occurrences="1">
       <code>$callback</code>
     </MixedArgumentTypeCoercion>
-    <MixedArrayOffset occurrences="4">
-      <code>$array[$key]</code>
-      <code>$array[$key]</code>
-      <code>$array[$key]</code>
-      <code>$array[$key]</code>
-    </MixedArrayOffset>
-    <MixedAssignment occurrences="9">
+    <MixedAssignment occurrences="6">
       <code>$a[$key]</code>
       <code>$a[$key]</code>
       <code>$a[$key]</code>
       <code>$a[]</code>
-      <code>$array[$key]</code>
       <code>$h</code>
-      <code>$key</code>
-      <code>$value</code>
       <code>$value</code>
     </MixedAssignment>
-    <MixedInferredReturnType occurrences="1">
-      <code>array</code>
-    </MixedInferredReturnType>
-    <MixedReturnStatement occurrences="1">
-      <code>$iterator-&gt;toArray()</code>
-    </MixedReturnStatement>
   </file>
   <file src="src/ErrorHandler.php">
     <InvalidArgument occurrences="1">
@@ -174,7 +156,7 @@
   </file>
   <file src="src/Message.php">
     <DocblockTypeContradiction occurrences="2">
-      <code>! is_array($spec) &amp;&amp; ! $spec instanceof Traversable</code>
+      <code>gettype($spec)</code>
       <code>is_scalar($key)</code>
     </DocblockTypeContradiction>
     <MixedArrayOffset occurrences="1">
@@ -192,6 +174,9 @@
     <ParamNameMismatch occurrences="1">
       <code>$value</code>
     </ParamNameMismatch>
+    <RedundantConditionGivenDocblockType occurrences="1">
+      <code>is_object($spec)</code>
+    </RedundantConditionGivenDocblockType>
   </file>
   <file src="src/MessageInterface.php">
     <MissingReturnType occurrences="1">
@@ -568,6 +553,9 @@
     </MixedAssignment>
   </file>
   <file src="test/SplStackTest.php">
+    <InvalidDocblock occurrences="1">
+      <code>assertSame</code>
+    </InvalidDocblock>
     <MixedArgument occurrences="2">
       <code>$unserialized</code>
       <code>$unserialized</code>
diff --git a/psalm.xml.dist b/psalm.xml.dist
index c28af9f3..4f1921c0 100644
--- a/psalm.xml.dist
+++ b/psalm.xml.dist
@@ -1,6 +1,6 @@
 <?xml version="1.0"?>
 <psalm
-    totallyTyped="true"
+    errorLevel="1"
     xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
     xmlns="https://getpsalm.org/schema/config"
     xsi:schemaLocation="https://getpsalm.org/schema/config vendor/vimeo/psalm/config.xsd"
diff --git a/src/StringWrapper/AbstractStringWrapper.php b/src/StringWrapper/AbstractStringWrapper.php
index 355e1ea1..a96c7a46 100644
--- a/src/StringWrapper/AbstractStringWrapper.php
+++ b/src/StringWrapper/AbstractStringWrapper.php
@@ -253,7 +253,7 @@ public function strPad($input, $padLength, $padString = ' ', $padType = STR_PAD_
         $repeatCount = (int) floor($lengthOfPadding / $padStringLength);
 
         if ($padType === STR_PAD_BOTH) {
-            $repeatCountLeft = $repeatCountRight = (int) ($repeatCount - $repeatCount % 2) / 2;
+            $repeatCountLeft = $repeatCountRight = ($repeatCount - $repeatCount % 2) / 2;
 
             $lastStringLength       = $lengthOfPadding - 2 * $repeatCountLeft * $padStringLength;
             $lastStringLeftLength   = $lastStringRightLength = (int) floor($lastStringLength / 2);
diff --git a/test/SplQueueTest.php b/test/SplQueueTest.php
index 33db8261..02cc1970 100644
--- a/test/SplQueueTest.php
+++ b/test/SplQueueTest.php
@@ -14,12 +14,14 @@
 
 class SplQueueTest extends TestCase
 {
-    /** @var SplQueue */
+    /** @var SplQueue<int, non-empty-string> */
     protected $queue;
 
     protected function setUp(): void
     {
-        $this->queue = new SplQueue();
+        /** @var SplQueue<int, non-empty-string> $splQueue */
+        $splQueue    = new SplQueue();
+        $this->queue = $splQueue;
         $this->queue->push('foo');
         $this->queue->push('bar');
         $this->queue->push('baz');