refactor LogNonGetRequestsTest

Author: AyoobMH

tests/LogNonGetRequestsTest.php

@@ -4,35 +4,22 @@
 
 use Spatie\HttpLogger\LogNonGetRequests;
 
-class LogNonGetRequestsTest extends TestCase
-{
-    /** @var \Spatie\HttpLogger\LogNonGetRequests */
-    protected $logProfile;
+beforeEach(function () {
+    $this->logProfile = new LogNonGetRequests();
+});
 
-    public function setUp(): void
-    {
-        parent::setup();
+test('it logs post patch put delete', function () {
+    foreach (['post', 'put', 'patch', 'delete'] as $method) {
+        $request = $this->makeRequest($method, $this->uri);
 
-        $this->logProfile = new LogNonGetRequests();
+        $this->assertTrue($this->logProfile->shouldLogRequest($request), "{$method} should be logged.");
     }
+});
 
-    /** @test */
-    public function it_logs_post_patch_put_delete()
-    {
-        foreach (['post', 'put', 'patch', 'delete'] as $method) {
-            $request = $this->makeRequest($method, $this->uri);
+test('it doesnt log get head options trace', function () {
+    foreach (['get', 'head', 'options', 'trace'] as $method) {
+        $request = $this->makeRequest($method, $this->uri);
 
-            $this->assertTrue($this->logProfile->shouldLogRequest($request), "{$method} should be logged.");
-        }
+        $this->assertFalse($this->logProfile->shouldLogRequest($request), "{$method} should not be logged.");
     }
-
-    /** @test */
-    public function it_doesnt_log_get_head_options_trace()
-    {
-        foreach (['get', 'head', 'options', 'trace'] as $method) {
-            $request = $this->makeRequest($method, $this->uri);
-
-            $this->assertFalse($this->logProfile->shouldLogRequest($request), "{$method} should not be logged.");
-        }
-    }
-}
+});