Remove GuzzleHttp\json_encode to pass tests.

Author: andreshg112

README.md

@@ -1,8 +1,10 @@
 # Very short description of the package
 
 [![Latest Version on Packagist](https://img.shields.io/packagist/v/andreshg112/laravel-auditing-notifications.svg?style=flat-square)](https://packagist.org/packages/andreshg112/laravel-auditing-notifications)
-[![Build Status](https://img.shields.io/travis/andreshg112/laravel-auditing-notifications/master.svg?style=flat-square)](https://travis-ci.org/andreshg112/laravel-auditing-notifications)
-[![Quality Score](https://img.shields.io/scrutinizer/g/andreshg112/laravel-auditing-notifications.svg?style=flat-square)](https://scrutinizer-ci.com/g/andreshg112/laravel-auditing-notifications)
+[![Build Status](https://travis-ci.com/andreshg112/laravel-auditing-notifications.svg?branch=master)](https://travis-ci.com/andreshg112/laravel-auditing-notifications)
+[![StyleCI](https://github.styleci.io/repos/178957162/shield?branch=master)](https://github.styleci.io/repos/178957162)
+[![Scrutinizer Code Quality](https://scrutinizer-ci.com/g/andreshg112/laravel-auditing-notifications/badges/quality-score.png?b=master)](https://scrutinizer-ci.com/g/andreshg112/laravel-auditing-notifications/?branch=master)
+[![Code Coverage](https://scrutinizer-ci.com/g/andreshg112/laravel-auditing-notifications/badges/coverage.png?b=master)](https://scrutinizer-ci.com/g/andreshg112/laravel-auditing-notifications/?branch=master)
 [![Total Downloads](https://img.shields.io/packagist/dt/andreshg112/laravel-auditing-notifications.svg?style=flat-square)](https://packagist.org/packages/andreshg112/laravel-auditing-notifications)
 
 This is where your description should go. Try and limit it to a paragraph or two, and maybe throw in a mention of what PSRs you support to avoid any confusion with users and contributors.
@@ -17,13 +19,13 @@ composer require andreshg112/laravel-auditing-notifications
 
 ## Usage
 
-``` php
+```php
 // Usage description here
 ```
 
 ### Testing
 
-``` bash
+```bash
 composer test
 ```
 
@@ -41,13 +43,13 @@ If you discover any security related issues, please email [email protected]
 
 ## Credits
 
-- [Andrés Herrera García](https://github.com/andreshg112)
-- [All Contributors](../../contributors)
+-   [Andrés Herrera García](https://github.com/andreshg112)
+-   [All Contributors](../../contributors)
 
 ## License
 
 The MIT License (MIT). Please see [License File](LICENSE.md) for more information.
 
 ## Laravel Package Boilerplate
 
-This package was generated using the [Laravel Package Boilerplate](https://laravelpackageboilerplate.com).
+This package was generated using the [Laravel Package Boilerplate](https://laravelpackageboilerplate.com).

composer.json

@@ -19,11 +19,11 @@
         "php": "^7.1",
         "illuminate/support": "5.2.* || 5.3.* || 5.4.* || 5.5.* || 5.6.* || 5.7.*",
         "lab123/aws-sns": "dev-master",
-        "mockery/mockery": "^1.2",
         "owen-it/laravel-auditing": "^8.0"
     },
     "require-dev": {
         "codedungeon/phpunit-result-printer": "^0.26.1",
+        "mockery/mockery": "^1.2",
         "orchestra/testbench": "^3.5",
         "phpunit/phpunit": "^7.0"
     },

src/AuditSns.php

@@ -3,7 +3,6 @@
 namespace Andreshg112\LaravelAuditingNotifications;
 
 use Illuminate\Bus\Queueable;
-use function GuzzleHttp\json_encode;
 use Illuminate\Support\Facades\Config;
 use Lab123\AwsSns\Messages\AwsSnsMessage;
 use Illuminate\Notifications\Notification;

tests/NotificationDriverTest.php

@@ -49,21 +49,19 @@ public function readyForAuditing(): bool
         return true;
     }
 
-    public function resolveIpAddress(): string
+    protected function resolveIpAddress(): string
     {
         return '';
     }
 
-    public function resolveUrl(): string
+    protected function resolveUrl(): string
     {
         return '';
     }
 
     protected function resolveUser()
-    {
-    }
+    { }
 
-    public function resolveUserAgent()
-    {
-    }
+    protected function resolveUserAgent()
+    { }
 }