fix: few fixes after #972

nikophil · nikophil · commit 32e9868fb0d8 · 2025-08-28T18:58:50.000+02:00
diff --git a/src/Persistence/PersistentObjectFactory.php b/src/Persistence/PersistentObjectFactory.php
@@ -233,6 +233,7 @@ public function create(callable|array $attributes = []): object
 
         if ($configuration->inADataProvider()
             && Configuration::autoRefreshWithLazyObjectsIsEnabled()
+            && $this->isPersisting()
             && !$this instanceof PersistentProxyObjectFactory
         ) {
             return ProxyGenerator::wrapFactoryNativeProxy($this, $attributes);
diff --git a/tests/Integration/DataProvider/DataProviderInUnitTest.php b/tests/Integration/DataProvider/DataProviderInUnitTest.php
@@ -71,6 +71,7 @@ public static function createObjectWithPersistentObjectFactoryInDataProvider():
 
     #[Test]
     #[DataProvider('useGetterOnObjectCreatedInDataProvider')]
+    #[IgnoreDeprecations]
     public function assert_it_can_use_getter_on_object_created_in_data_provider(string $providedData, mixed $expectedData): void
     {
         self::assertEquals($expectedData, ProxyGenerator::unwrap($providedData));
diff --git a/tests/Integration/DataProvider/DataProviderWithPersistentFactoryInKernelTestCase.php b/tests/Integration/DataProvider/DataProviderWithPersistentFactoryInKernelTestCase.php
@@ -15,6 +15,7 @@
 
 use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\Attributes\IgnoreDeprecations;
+use PHPUnit\Framework\Attributes\RequiresEnvironmentVariable;
 use PHPUnit\Framework\Attributes\RequiresPhpunit;
 use PHPUnit\Framework\Attributes\RequiresPhpunitExtension;
 use PHPUnit\Framework\Attributes\Test;
@@ -110,6 +111,7 @@ public static function useGetterOnProxyObjectCreatedInDataProvider(): iterable
 
     #[Test]
     #[DataProvider('throwsExceptionWhenCreatingObjectInDataProvider')]
+    #[RequiresEnvironmentVariable('USE_PHP_84_LAZY_OBJECTS', '0')]
     public function it_throws_when_creating_persisted_object_with_non_proxy_factory_in_data_provider(?\Throwable $e,
     ): void {
         self::assertInstanceOf(\LogicException::class, $e);

Original file line number	Diff line number	Diff line change
`@@ -71,6 +71,7 @@ public static function createObjectWithPersistentObjectFactoryInDataProvider():`
`71`	`71`
`72`	`72`	`#[Test]`
`73`	`73`	`#[DataProvider('useGetterOnObjectCreatedInDataProvider')]`
	`74`	`+ #[IgnoreDeprecations]`
`74`	`75`	`public function assert_it_can_use_getter_on_object_created_in_data_provider(string $providedData, mixed $expectedData): void`
`75`	`76`	`{`
`76`	`77`	`self::assertEquals($expectedData, ProxyGenerator::unwrap($providedData));`