Fix test warnings due to usage of deprecated assertArraySubset()

Also changed app_id to strings where appropriate, in real apps they should be strings when read from environment, not ints.

Author: francislavoie

tests/ConnectionTest.php

@@ -46,7 +46,7 @@ public function successful_connections_have_the_app_attached()
         $this->pusherServer->onOpen($connection);
 
         $this->assertInstanceOf(App::class, $connection->app);
-        $this->assertSame(1234, $connection->app->id);
+        $this->assertSame('1234', $connection->app->id);
         $this->assertSame('TestKey', $connection->app->key);
         $this->assertSame('TestSecret', $connection->app->secret);
         $this->assertSame('Test App', $connection->app->name);

tests/Statistics/Controllers/WebSocketsStatisticsControllerTest.php

@@ -22,16 +22,20 @@ public function it_can_store_statistics()
 
         $this->assertCount(1, $entries);
 
-        $this->assertArraySubset($this->payload(), $entries->first()->attributesToArray());
+        $actual = $entries->first()->attributesToArray();
+        foreach ($this->payload() as $key => $value) {
+            $this->assertArrayHasKey($key, $actual);
+            $this->assertSame($value, $actual[$key]);
+        }
     }
 
     protected function payload(): array
     {
         return [
             'app_id' => config('websockets.apps.0.id'),
-            'peak_connection_count' => 1,
-            'websocket_message_count' => 2,
-            'api_message_count' => 3,
+            'peak_connection_count' => '1',
+            'websocket_message_count' => '2',
+            'api_message_count' => '3',
         ];
     }
 }

tests/TestCase.php

@@ -46,7 +46,7 @@ protected function getEnvironmentSetUp($app)
         $app['config']->set('websockets.apps', [
             [
                 'name' => 'Test App',
-                'id' => 1234,
+                'id' => '1234',
                 'key' => 'TestKey',
                 'secret' => 'TestSecret',
                 'host' => 'localhost',