Merge pull request #2504 from acelaya-forks/remove-set-accessible

Remove all calls to ReflectionProperty::setAccessible
This commit is contained in:
Alejandro Celaya
2025-10-28 11:08:03 +01:00
committed by GitHub
4 changed files with 0 additions and 4 deletions

View File

@@ -51,7 +51,6 @@ class ShlinkTableTest extends TestCase
$ref = new ReflectionObject($instance);
$baseTable = $ref->getProperty('baseTable');
$baseTable->setAccessible(true);
self::assertInstanceOf(Table::class, $baseTable->getValue($instance));
}

View File

@@ -85,7 +85,6 @@ class ShortUrlListRepositoryTest extends DatabaseTestCase
$foo2->setVisits(new ArrayCollection($visits2));
$ref = new ReflectionObject($foo2);
$dateProp = $ref->getProperty('dateCreated');
$dateProp->setAccessible(true);
$dateProp->setValue($foo2, Chronos::now()->subDays(5));
$this->getEntityManager()->persist($foo2);

View File

@@ -98,7 +98,6 @@ class ShortUrlsFixture extends AbstractFixture implements DependentFixtureInterf
{
$ref = new ReflectionObject($shortUrl);
$dateProp = $ref->getProperty('dateCreated');
$dateProp->setAccessible(true);
$dateProp->setValue($shortUrl, Chronos::parse($date));
return $shortUrl;

View File

@@ -27,7 +27,6 @@ class EmptyResponseImplicitOptionsMiddlewareFactoryTest extends TestCase
$ref = new ReflectionObject($instance);
$prop = $ref->getProperty('responseFactory');
$prop->setAccessible(true);
/** @var ResponseFactoryInterface $value */
$value = $prop->getValue($instance);