✨ Add SlevomatCodingStandard.Functions.StaticClosure

PreviousNextDrupal/ruleset.xml

@@ -36,6 +36,7 @@
     <!-- SlevomatCodingStandard.Functions -->
     <rule ref="SlevomatCodingStandard.Functions.RequireTrailingCommaInCall" />
     <rule ref="SlevomatCodingStandard.Functions.RequireTrailingCommaInDeclaration" />
+    <rule ref="SlevomatCodingStandard.Functions.StaticClosure" />
     <rule ref="SlevomatCodingStandard.Functions.UnusedInheritedVariablePassedToClosure" />
 
     <!-- SlevomatCodingStandard.Namespaces -->

tests/Sniffs/FunctionsStaticClosureTest.php

@@ -0,0 +1,27 @@
+<?php
+
+declare(strict_types=1);
+
+namespace PreviousNext\CodingStandard\Tests\Sniffs;
+
+/**
+ * @covers \SlevomatCodingStandard\Sniffs\Functions\StaticClosureSniff
+ */
+final class FunctionsStaticClosureTest extends Base {
+
+  public function testNoError(): void {
+    $report = self::checkFile(__DIR__ . '/fixtures/FunctionsStaticClosureNoError.php');
+    self::assertNoSniffErrorInFile($report);
+  }
+
+  public function testError(): void {
+    $report = self::checkFile(__DIR__ . '/fixtures/FunctionsStaticClosureError.php');
+    self::assertSame(1, $report->getErrorCount());
+    self::assertSniffError($report, 16, sniffName: 'SlevomatCodingStandard.Functions.StaticClosure', code: 'ClosureNotStatic');
+  }
+
+  protected static function getSniffName(): string {
+    return 'SlevomatCodingStandard.Functions.StaticClosure';
+  }
+
+}

tests/Sniffs/fixtures/FunctionsStaticClosureError.php

@@ -0,0 +1,21 @@
+<?php
+
+declare(strict_types=1);
+
+namespace PreviousNext\CodingStandard\Tests\Sniffs\fixtures;
+
+/**
+ * The class.
+ */
+final class FunctionsStaticClosureError {
+
+  /**
+   * Foo.
+   */
+  public function foo(): void {
+    $a = function (): void {
+      $a = 1;
+    };
+  }
+
+}

tests/Sniffs/fixtures/FunctionsStaticClosureNoError.php

@@ -0,0 +1,21 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Sniffs\fixtures;
+
+/**
+ * The class.
+ */
+final class FunctionsStaticClosureNoError {
+
+  /**
+   * Foo.
+   */
+  public function foo(): void {
+    $a = static function (): void {
+      $a = 1;
+    };
+  }
+
+}