Merge pull request #665 from ejunker/main

Prevent out of memory with multiple requests in a test

src/LaravelDataServiceProvider.php

@@ -41,6 +41,10 @@ function () {
         );
 
         $this->app->beforeResolving(BaseData::class, function ($class, $parameters, $app) {
+            if ($app->has($class)) {
+                return;
+            }
+
             $app->bind(
                 $class,
                 fn ($container) => $class::from($container['request'])

tests/RequestTest.php

@@ -21,7 +21,7 @@
         ValidationException::class,
     ]);
 
-    Route::post('/example-route', function (Sim $data) {
+    Route::post('/example-route', function (SimpleData $data) {
         return ['given' => $data->string];
     });
 });
@@ -38,6 +38,18 @@
         ->assertJson(['given' => 'Hello']);
 });
 
+it('can make multiple requests', function () {
+    postJson('/example-route', [
+        'string' => 'Hello',
+    ])
+        ->assertOk();
+
+    postJson('/example-route', [
+        'string' => 'Hello',
+    ])
+        ->assertOk();
+});
+
 it('can returns a 201 response code for POST requests', function () {
     Route::post('/example-route', function () {
         return new SimpleData(request()->input('string'));