✨ Add SlevomatCodingStandard.TypeHints.ReturnTypeHintSpacing (#11)

Author: dpi

PreviousNextDrupal/ruleset.xml

@@ -41,6 +41,8 @@
     <rule ref="SlevomatCodingStandard.TypeHints.ReturnTypeHint">
         <exclude name="SlevomatCodingStandard.TypeHints.ReturnTypeHint.MissingTraversableTypeHintSpecification"/>
     </rule>
+    <rule ref="SlevomatCodingStandard.TypeHints.ReturnTypeHintSpacing" />
+
     <!-- Drupal -->
     <rule ref="Drupal" />
 

tests/Sniffs/ReturnTypeHintSpacingTest.php

@@ -0,0 +1,29 @@
+<?php
+
+declare(strict_types = 1);
+
+namespace PreviousNext\CodingStandard\Tests\Sniffs;
+
+use SlevomatCodingStandard\Sniffs\TypeHints\ReturnTypeHintSpacingSniff;
+
+/**
+ * @covers \SlevomatCodingStandard\Sniffs\TypeHints\ReturnTypeHintSpacingSniff
+ */
+final class ReturnTypeHintSpacingTest extends Base {
+
+  public function testNoError(): void {
+    $report = self::checkFile(__DIR__ . '/fixtures/ReturnTypeHintSpacingNoError.php');
+    self::assertNoSniffErrorInFile($report);
+  }
+
+  public function testError(): void {
+    $report = self::checkFile(__DIR__ . '/fixtures/ReturnTypeHintSpacingError.php');
+    self::assertSame(1, $report->getErrorCount());
+    self::assertSniffError($report, 8, ReturnTypeHintSpacingSniff::CODE_WHITESPACE_BEFORE_COLON);
+  }
+
+  protected static function getSniffName(): string {
+    return 'SlevomatCodingStandard.TypeHints.ReturnTypeHintSpacing';
+  }
+
+}

tests/Sniffs/fixtures/ReturnTypeHintSpacingError.php

@@ -0,0 +1,10 @@
+<?php
+
+declare(strict_types = 1);
+
+/**
+ * A function.
+ */
+function foo() : ?int {
+  return NULL;
+}

tests/Sniffs/fixtures/ReturnTypeHintSpacingNoError.php

@@ -0,0 +1,10 @@
+<?php
+
+declare(strict_types = 1);
+
+/**
+ * A function.
+ */
+function foo(): ?int {
+  return NULL;
+}