Merge pull request #14 from sirbrillig/add/less-automated-tests

Alter tests to run without AbstractSniffUnitTest

Author: sirbrillig

README.md

@@ -66,3 +66,10 @@ This was forked from the excellent work in https://github.com/illusori/PHP_Codes
 ## Contributing
 
 Please open issues or PRs on this repository.
+
+To run tests, make sure composer is installed, then run:
+
+```
+composer install # you only need to do this once
+composer test
+```

VariableAnalysis/Tests/BaseTestCase.php

@@ -0,0 +1,36 @@
+<?php
+namespace VariableAnalysis\Tests;
+
+use PHPUnit\Framework\TestCase;
+use PHP_CodeSniffer\Files\LocalFile;
+use PHP_CodeSniffer\Ruleset;
+use PHP_CodeSniffer\Config;
+
+class BaseTestCase extends TestCase {
+  public function prepareLocalFileForSniffs($sniffFiles, string $fixtureFile): LocalFile {
+    $config = new Config();
+    $ruleset = new Ruleset($config);
+    if (! is_array($sniffFiles)) {
+      $sniffFiles = [$sniffFiles];
+    }
+    $ruleset->registerSniffs($sniffFiles, [], []);
+    $ruleset->populateTokenListeners();
+    return new LocalFile($fixtureFile, $ruleset, $config);
+  }
+
+  public function getLineNumbersFromMessages(array $messages): array {
+    return array_keys($messages);
+  }
+
+  public function getWarningLineNumbersFromFile(LocalFile $phpcsFile): array {
+    return $this->getLineNumbersFromMessages($phpcsFile->getWarnings());
+  }
+
+  public function getErrorLineNumbersFromFile(LocalFile $phpcsFile): array {
+    return $this->getLineNumbersFromMessages($phpcsFile->getErrors());
+  }
+
+  public function getSniffFiles(): array {
+    return [__DIR__ . '/../../VariableAnalysis/Sniffs/CodeAnalysis/VariableAnalysisSniff.php'];
+  }
+}