Updated to run with Laravel 5.8 and fix tests (#194)

Author: footballencarta

.travis.yml

@@ -7,10 +7,9 @@ stages:
   - test
 
 php:
-  - '5.6'
-  - '7.0'
   - '7.1'
   - '7.2'
+  - '7.3'
 
 before_script:
   - java -Djava.library.path=./DynamoDBLocal_lib -jar dynamodb_local/DynamoDBLocal.jar --port 3000 &

README.md

@@ -608,3 +608,4 @@ Author and Contributors
 * [Alexander Ward](https://github.com/cthos)
 * [Quang Ngo](https://github.com/vanquang9387)
 * [David Higgins](https://github.com/zoul0813)
+* [Damon Williams](https://github.com/footballencarta)

composer.json

@@ -4,8 +4,8 @@
     "keywords": ["laravel", "dynamodb", "aws"],
     "require": {
         "aws/aws-sdk-php": "^3.0.0",
-        "illuminate/support": "5.1.* || 5.2.* || 5.3.* || 5.4.* || 5.5.* || 5.6.* || 5.7.*",
-        "illuminate/database": "5.1.* || 5.2.* || 5.3.* || 5.4.* || 5.5.* || 5.6.* || 5.7.*"
+        "illuminate/support": "5.1.* || 5.2.* || 5.3.* || 5.4.* || 5.5.* || 5.6.* || 5.7.* || 5.8.*",
+        "illuminate/database": "5.1.* || 5.2.* || 5.3.* || 5.4.* || 5.5.* || 5.6.* || 5.7.* || 5.8.*"
     },
     "license": "MIT",
     "authors": [
@@ -19,7 +19,6 @@
             "BaoPham\\DynamoDb\\": "src/"
         }
     },
-    "minimum-stability": "dev",
     "require-dev": {
         "orchestra/testbench": "~3.0"
     },

src/RawDynamoDbQuery.php

@@ -136,6 +136,8 @@ public function count()
      * For backward compatibility, previously we use array to represent the raw query
      *
      * @var array
+     *
+     * @return array
      */
     private function internal()
     {

tests/DynamoDb/DynamoDbManagerTest.php

@@ -20,7 +20,7 @@ class DynamoDbManagerTest extends DynamoDbTestCase
      */
     protected $mockedClient;
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
 

tests/DynamoDbClientServiceTest.php

@@ -13,7 +13,7 @@
  */
 class DynamoDbClientServiceTest extends TestCase
 {
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
 
@@ -75,7 +75,7 @@ public function testUnsetDynamoDbClientService()
     /**
      * Make sure we are not leaving any values set on the DynamoDbModel
      */
-    public function tearDown()
+    public function tearDown(): void
     {
         parent::tearDown();
 

tests/DynamoDbModelTest.php

@@ -21,7 +21,7 @@ abstract class DynamoDbModelTest extends DynamoDbTestCase
      */
     protected $marshaler;
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
 

tests/DynamoDbTestCase.php

@@ -12,7 +12,7 @@
  */
 abstract class DynamoDbTestCase extends TestCase
 {
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
 

tests/Parsers/ConditionExpressionTest.php

@@ -26,7 +26,7 @@ class ConditionExpressionTest extends TestCase
      */
     private $values;
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
         $this->names = new ExpressionAttributeNames();

tests/Parsers/UpdateExpressionTest.php

@@ -18,7 +18,7 @@ class UpdateExpressionTest extends TestCase
      */
     private $names;
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
         $this->names = new ExpressionAttributeNames();