test: replace test class names

CURLRequestTest is the main file.

Author: kenjis

…TTP/CURLRequestDoNotShareOptionsTest.php …tem/HTTP/CURLRequestShareOptionsTest.phptests/system/HTTP/CURLRequestDoNotShareOptionsTest.php renamed to tests/system/HTTP/CURLRequestShareOptionsTest.php tests/system/HTTP/CURLRequestDoNotShareOptionsTest.php renamed to tests/system/HTTP/CURLRequestShareOptionsTest.php

@@ -23,11 +23,14 @@
 use CURLFile;
 
 /**
+ * This test case is for the case where shareOptions is true.
+ * The shareOptions should be set to false.
+ *
  * @internal
  *
  * @group Others
  */
-final class CURLRequestDoNotShareOptionsTest extends CIUnitTestCase
+final class CURLRequestShareOptionsTest extends CIUnitTestCase
 {
     private MockCURLRequest $request;
 
@@ -45,7 +48,7 @@ protected function getRequest(array $options = []): MockCURLRequest
         $app = new App();
 
         $config               = new ConfigCURLRequest();
-        $config->shareOptions = false;
+        $config->shareOptions = true;
         Factories::injectMock('config', 'CURLRequest', $config);
 
         return new MockCURLRequest(($app), $uri, new Response($app), $options);
@@ -226,23 +229,6 @@ public function testDefaultOptionsAreSharedBetweenRequests(): void
         $this->assertSame('CodeIgniter Framework v4', $request->curl_options[CURLOPT_USERAGENT]);
     }
 
-    public function testHeaderContentLengthNotSharedBetweenRequests(): void
-    {
-        $options = [
-            'base_uri' => 'http://www.foo.com/api/v1/',
-        ];
-        $request = $this->getRequest($options);
-
-        $request->post('example', [
-            'form_params' => [
-                'q' => 'keyword',
-            ],
-        ]);
-        $request->get('example');
-
-        $this->assertNull($request->header('Content-Length'));
-    }
-
     /**
      * @backupGlobals enabled
      */
@@ -960,21 +946,6 @@ public function testApplyBodyByOptions(): void
         $this->assertSame('name=George', $request->curl_options[CURLOPT_POSTFIELDS]);
     }
 
-    public function testBodyIsResetOnSecondRequest(): void
-    {
-        $request = $this->getRequest([
-            'base_uri' => 'http://www.foo.com/api/v1/',
-            'delay'    => 100,
-        ]);
-        $request->setBody('name=George');
-        $request->setOutput('Hi there');
-
-        $request->post('answer');
-        $request->post('answer');
-
-        $this->assertArrayNotHasKey(CURLOPT_POSTFIELDS, $request->curl_options);
-    }
-
     public function testResponseHeaders(): void
     {
         $request = $this->getRequest([
@@ -1094,6 +1065,7 @@ public function testJSONData(): void
             $expected,
             $this->request->curl_options[CURLOPT_POSTFIELDS]
         );
+        $this->assertSame($expected, $this->request->getBody());
     }
 
     public function testSetJSON(): void
@@ -1112,6 +1084,8 @@ public function testSetJSON(): void
             $expected,
             $this->request->curl_options[CURLOPT_POSTFIELDS]
         );
+        $this->assertSame($expected, $this->request->getBody());
+
         $this->assertSame(
             'Content-Type: application/json',
             $this->request->curl_options[CURLOPT_HTTPHEADER][0]

tests/system/HTTP/CURLRequestTest.php

@@ -45,7 +45,7 @@ protected function getRequest(array $options = []): MockCURLRequest
         $app = new App();
 
         $config               = new ConfigCURLRequest();
-        $config->shareOptions = true;
+        $config->shareOptions = false;
         Factories::injectMock('config', 'CURLRequest', $config);
 
         return new MockCURLRequest(($app), $uri, new Response($app), $options);
@@ -226,6 +226,23 @@ public function testDefaultOptionsAreSharedBetweenRequests(): void
         $this->assertSame('CodeIgniter Framework v4', $request->curl_options[CURLOPT_USERAGENT]);
     }
 
+    public function testHeaderContentLengthNotSharedBetweenRequests(): void
+    {
+        $options = [
+            'base_uri' => 'http://www.foo.com/api/v1/',
+        ];
+        $request = $this->getRequest($options);
+
+        $request->post('example', [
+            'form_params' => [
+                'q' => 'keyword',
+            ],
+        ]);
+        $request->get('example');
+
+        $this->assertNull($request->header('Content-Length'));
+    }
+
     /**
      * @backupGlobals enabled
      */
@@ -943,6 +960,21 @@ public function testApplyBodyByOptions(): void
         $this->assertSame('name=George', $request->curl_options[CURLOPT_POSTFIELDS]);
     }
 
+    public function testBodyIsResetOnSecondRequest(): void
+    {
+        $request = $this->getRequest([
+            'base_uri' => 'http://www.foo.com/api/v1/',
+            'delay'    => 100,
+        ]);
+        $request->setBody('name=George');
+        $request->setOutput('Hi there');
+
+        $request->post('answer');
+        $request->post('answer');
+
+        $this->assertArrayNotHasKey(CURLOPT_POSTFIELDS, $request->curl_options);
+    }
+
     public function testResponseHeaders(): void
     {
         $request = $this->getRequest([
@@ -1062,7 +1094,6 @@ public function testJSONData(): void
             $expected,
             $this->request->curl_options[CURLOPT_POSTFIELDS]
         );
-        $this->assertSame($expected, $this->request->getBody());
     }
 
     public function testSetJSON(): void
@@ -1081,8 +1112,6 @@ public function testSetJSON(): void
             $expected,
             $this->request->curl_options[CURLOPT_POSTFIELDS]
         );
-        $this->assertSame($expected, $this->request->getBody());
-
         $this->assertSame(
             'Content-Type: application/json',
             $this->request->curl_options[CURLOPT_HTTPHEADER][0]