Add extended test for loophole

CHANGELOG.md

@@ -5,11 +5,15 @@ Intended to follow [«Keep a Changelog»](https://keepachangelog.com/en/)
 
 ----
 
+## TODO
+- TODO: remove variableUnpredictable` in favour of  `variableUninferable`
+
 ## Upcoming
 
 ### Added
 - Support for attributes
 	* missing stand-alone attribute ???
+- Extended test case for Loophole (still failing)
 
 ### Removed
 * Support for PHP < 7.4

tests/AttribForceFactoryRuleTest.php

@@ -22,6 +22,12 @@ public function testLoopholeFactory(): void
         $this->analyse([__DIR__ . '/dataAttrib/LoopholeFactory.php'], []);
     }
 
+    // Sadly this remains a vector, as phpstan fails to infer the created class name
+    public function testLoopholeInvoker(): void
+    {
+        $this->analyse([__DIR__ . '/dataAttrib/LoopholeFactory.php', __DIR__ . '/dataAttrib/LoopholeInvoker.php'], []);
+    }
+
     public function testEmptyAllowedClasses(): void
     {
         $this->analyse([__DIR__ . '/dataAttrib/EmptyFactory.php'], [

tests/dataAttrib/LoopholeInvoker.php

@@ -0,0 +1,27 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Test\Ebln\PHPStan\EnforceFactory\dataAttrib;
+
+class LoopholeInvoker
+{
+    private LoopholeFactory $loopholeFactory;
+
+    public function __construct()
+    {
+        $this->loopholeFactory = new LoopholeFactory();
+    }
+
+    public function expectedFailingLoophole(): object
+    {
+        $loophole = $this->loopholeFactory->variableUninferable(true);
+
+        return $loophole;
+    }
+
+    public function expectedMissingClass()
+    {
+        return $this->loopholeFactory->variableUninferable(false);
+    }
+}