ReviewOS

also looking at this

meemalabs/laravel-meema

Apply fixes from StyleCI

#7
Merged glennmichael123 wants to merge analysis-M1yPLL into implemen-test
3 files +6 -6

Review threads live on the whole diff, not on one commit, so none are shown here - a thread's line means something in the branch's final form, and painting it into an intermediate step would put it on code it is not about.

tests/LaravelMeemaTestCase.phpmodifiedformatting only+2-2
Changes to tests/LaravelMeemaTestCase.php
@@ -8,7 +8,7 @@ use Orchestra\Testbench\TestCase;
88
99class LaravelMeemaTestCase extends TestCase
1010{
11 protected function getPackageProviders($app): array
11 protected function getPackageProviders($app): array
1212 {
1313 return [MeemaServiceProvider::class];
1414 }
@@ -28,4 +28,4 @@ class LaravelMeemaTestCase extends TestCase
2828 // let's make sure these config values are set
2929 Config::set('meema.api_key', env('MEEMA_API_KEY'));
3030 }
31}⧵
31}
tests/MediaTest.phpmodifiedformatting only+2-2
Changes to tests/MediaTest.php
@@ -1,8 +1,8 @@
11<?php
22
3use Meema\LaravelMeema\Facades\Folder;
34use Meema\LaravelMeema\Facades\Media;
45use Meema\LaravelMeema\Facades\Tag;
5use Meema\LaravelMeema\Facades\Folder;
66use Meema\LaravelMeema\Tests\LaravelMeemaTestCase;
77
88uses(LaravelMeemaTestCase::class);
@@ -120,4 +120,4 @@ it('can delete a media', function () {
120120 $response = Media::delete((int) $media[0]['media_id']);
121121
122122 $this->assertTrue(is_null($response));
123});⧵
123});
tests/TagTest.phpmodifiedformatting only+2-2
Changes to tests/TagTest.php
@@ -1,8 +1,8 @@
11<?php
22
3use Meema\LaravelMeema\Facades\Folder;
34use Meema\LaravelMeema\Facades\Media;
45use Meema\LaravelMeema\Facades\Tag;
5use Meema\LaravelMeema\Facades\Folder;
66use Meema\LaravelMeema\Tests\LaravelMeemaTestCase;
77
88uses(LaravelMeemaTestCase::class);
@@ -70,4 +70,4 @@ it('can delete a tag', function () {
7070 $response = Tag::delete($tags[0]['id']);
7171
7272 $this->assertTrue(is_null($response));
73});⧵
73});