Merge pull request #7 from worksome/disallow_resolve_function

Adds `resolve` to forbidden function all along with `app`

src/PHPStan/Laravel/DisallowAppHelperUsageRule.php

@@ -13,7 +13,10 @@
  */
 class DisallowAppHelperUsageRule implements Rule
 {
-    const FUNCTION_NAME = 'app';
+    private array $forbiddenFunctions = [
+        'app',
+        'resolve',
+    ];
 
     public function getNodeType(): string
     {
@@ -33,13 +36,13 @@ public function processNode(Node $node, Scope $scope): array
 
         $functionName = $nodeName->toString();
 
-        if ($functionName !== self::FUNCTION_NAME) {
+        if (! in_array($functionName, $this->forbiddenFunctions)) {
             return [];
         }
 
         return [
             RuleErrorBuilder::message(
-                "Usage of app helper is disallowed. Use dependency injection instead."
+                "Usage of {$functionName} helper is disallowed. Use dependency injection instead."
             )->build(),
         ];
     }

tests/PHPStan/Laravel/DisallowAppUsageRule/DisallowAppUsageRuleTest.php

@@ -16,6 +16,13 @@
             7,
         ],
     ],
+    'calls resolve helper with parameters' => [
+        __DIR__ . '/Fixture/resolve_helper_with_parameters.php.inc',
+        [
+            'Usage of resolve helper is disallowed. Use dependency injection instead.',
+            7,
+        ],
+    ],
     'calls app helper without args' => [
         __DIR__ . '/Fixture/app_helper_without_arg.php.inc',
         [
@@ -26,6 +33,9 @@
     'skips app function from other namespace' => [
         __DIR__ . '/Fixture/skip_app_function_in_namespace.php.inc',
     ],
+    'skips resolve function from other namespace' => [
+        __DIR__ . '/Fixture/skip_resolve_function_in_namespace.php.inc',
+    ],
     'skips function call on variable' => [
         __DIR__ . '/Fixture/skip_variable_call.php.inc',
     ],
@@ -36,11 +46,25 @@
             9,
         ],
     ],
+    'calls resolve helper when inside a namespace' => [
+        __DIR__ . '/Fixture/resolve_helper_in_namespace.php.inc',
+        [
+            'Usage of resolve helper is disallowed. Use dependency injection instead.',
+            9,
+        ],
+    ],
     'calls app helper with chain calls after' => [
         __DIR__ . '/Fixture/app_helper_with_chain_calls.php.inc',
         [
             'Usage of app helper is disallowed. Use dependency injection instead.',
             9,
         ],
     ],
+    'calls resolve helper with chain calls after' => [
+        __DIR__ . '/Fixture/resolve_helper_with_chain_calls.php.inc',
+        [
+            'Usage of resolve helper is disallowed. Use dependency injection instead.',
+            9,
+        ],
+    ],
 ]);

tests/PHPStan/Laravel/DisallowAppUsageRule/Fixture/resolve_helper_in_namespace.php.inc

@@ -0,0 +1,13 @@
+<?php
+
+namespace Resolve\Services;
+
+class CallsResolveHelper
+{
+    public function method()
+    {
+        return resolve('myService');
+    }
+}
+
+?>

tests/PHPStan/Laravel/DisallowAppUsageRule/Fixture/resolve_helper_with_chain_calls.php.inc

@@ -0,0 +1,13 @@
+<?php
+
+namespace Resolve\Services;
+
+class CallsResolveHelper
+{
+    public function method()
+    {
+        return \resolve('myService')->execute();
+    }
+}
+
+?>

tests/PHPStan/Laravel/DisallowAppUsageRule/Fixture/resolve_helper_with_parameters.php.inc

@@ -0,0 +1,11 @@
+<?php
+
+class CallsResolveHelper
+{
+    public function method()
+    {
+        return \resolve('myService', ['foo' => 'bar']);
+    }
+}
+
+?>

tests/PHPStan/Laravel/DisallowAppUsageRule/Fixture/skip_resolve_function_in_namespace.php.inc

@@ -0,0 +1,11 @@
+<?php
+
+class CallsOtherResolveHelper
+{
+    public function container()
+    {
+        return \MyCustom\resolve();
+    }
+}
+
+?>