✨ Add SlevomatCodingStandard.Functions.RequireTrailingCommaInCall (#6)

Author: dpi

PreviousNextDrupal/ruleset.xml

@@ -25,6 +25,7 @@
     </rule>
 
     <!-- SlevomatCodingStandard.Functions -->
+    <rule ref="SlevomatCodingStandard.Functions.RequireTrailingCommaInCall" />
     <rule ref="SlevomatCodingStandard.Functions.UnusedInheritedVariablePassedToClosure" />
 
     <!-- Drupal -->

tests/Sniffs/RequireTrailingCommaInCallTest.php

@@ -0,0 +1,29 @@
+<?php
+
+declare(strict_types = 1);
+
+namespace PreviousNext\CodingStandard\Tests\Sniffs;
+
+use SlevomatCodingStandard\Sniffs\Functions\RequireTrailingCommaInDeclarationSniff;
+
+/**
+ * @covers \SlevomatCodingStandard\Sniffs\Functions\RequireTrailingCommaInDeclarationSniff
+ */
+final class RequireTrailingCommaInCallTest extends Base {
+
+  public function testNoError(): void {
+    $report = self::checkFile(__DIR__ . '/fixtures/RequireTrailingCommaInCallNoError.php');
+    self::assertNoSniffErrorInFile($report);
+  }
+
+  public function testError(): void {
+    $report = self::checkFile(__DIR__ . '/fixtures/RequireTrailingCommaInCallError.php');
+    self::assertSame(1, $report->getErrorCount());
+    self::assertSniffError($report, 7, sniffName: 'SlevomatCodingStandard.Functions.RequireTrailingCommaInCall', code: RequireTrailingCommaInDeclarationSniff::CODE_MISSING_TRAILING_COMMA);
+  }
+
+  protected static function getSniffName(): string {
+    return 'SlevomatCodingStandard.Functions.RequireTrailingCommaInCall';
+  }
+
+}

tests/Sniffs/fixtures/RequireTrailingCommaInCallError.php

@@ -0,0 +1,8 @@
+<?php
+
+declare(strict_types = 1);
+
+hello(
+  'foo',
+  'bar'
+);

tests/Sniffs/fixtures/RequireTrailingCommaInCallNoError.php

@@ -0,0 +1,8 @@
+<?php
+
+declare(strict_types = 1);
+
+hello(
+  'foo',
+  'bar',
+);