added tests, made php-cs-fixer a formatter

README.md

@@ -66,7 +66,7 @@ trunk check enable {linter}
 | Nix             | [nixpkgs-fmt]                                                                                                        |
 | package.json    | [sort-package-json]                                                                                                  |
 | Perl            | [perlcritic], [perltidy]                                                                                             |
-| PHP             | [phpstan]                                                                                                            |
+| PHP             | [php-cs-fixer], [phpstan]                                                                                            |
 | PNG             | [oxipng]                                                                                                             |
 | PowerShell      | [psscriptanalyzer]                                                                                                   |
 | Prisma          | [prisma]                                                                                                             |
@@ -143,6 +143,7 @@ trunk check enable {linter}
 [oxipng]: https://github.com/shssoichiro/oxipng#readme
 [perlcritic]: https://metacpan.org/pod/Perl::Critic
 [perltidy]: https://metacpan.org/dist/Perl-Tidy/view/bin/perltidy
+[php-cs-fixer]: https://github.com/PHP-CS-Fixer/PHP-CS-Fixer#readme
 [phpstan]: https://phpstan.org/
 [pmd]: https://pmd.github.io/
 [pragma-once]: linters/pragma-once/README.md

linters/php-cs-fixer/php-cs-fixer.test.ts

@@ -0,0 +1,7 @@
+import { linterFmtTest } from "tests";
+import { skipOS } from "tests/utils";
+
+linterFmtTest({
+  linterName: "php-cs-fixer",
+  skipTestIf: skipOS(["win32"]),
+});

linters/php-cs-fixer/plugin.yaml

@@ -0,0 +1,36 @@
+version: 0.1
+tools:
+  definitions:
+    - name: php-cs-fixer
+      runtime: php
+      package: friendsofphp/php-cs-fixer
+      known_good_version: 3.54.0
+      shims: [php-cs-fixer]
+      environment:
+        - name: PATH
+          list: ["${tool}/vendor/bin"]
+      health_checks:
+        - command: php-cs-fixer --version
+          parse_regex: PHP CS Fixer ${semver}
+lint:
+  definitions:
+    - name: php-cs-fixer
+      description: Keeps PHP code up to standards
+      tools: [php-cs-fixer]
+      known_good_version: 3.54.0
+      files: [php]
+      commands:
+        # TODO(Tyler): Do we want to define a lint command here?
+        - name: format
+          output: rewrite
+          success_codes: [0]
+          formatter: true
+          batch: true
+          in_place: true
+          run: php-cs-fixer fix --using-cache=no --show-progress=none ${target}
+      direct_configs:
+        # uses ruleset @PSR12 when a config is not present.
+        - .php-cs-fixer.dist.php
+      affects_cache:
+        - .php-cs-fixer.php
+      suggest_if: config_present

linters/php-cs-fixer/test_data/basic.in.php

@@ -0,0 +1,15 @@
+<?php declare(strict_types = 1);
+
+class HelloWorld
+{
+	private $b;
+
+	private $a;
+
+
+	public function sayHello(DateTimeImutable $date): void
+	{
+		$c=(ARRAY)array(1,2);
+		echo 'Hello, ' . $date->format('j. n. Y');
+	}
+}

linters/php-cs-fixer/test_data/php_cs_fixer_v3.54.0_basic.fmt.shot

@@ -0,0 +1,22 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`Testing formatter php-cs-fixer test basic 1`] = `
+"<?php
+
+declare(strict_types=1);
+
+class HelloWorld
+{
+    private $b;
+
+    private $a;
+
+
+    public function sayHello(DateTimeImutable $date): void
+    {
+        $c = (array)array(1,2);
+        echo 'Hello, ' . $date->format('j. n. Y');
+    }
+}
+"
+`;

tools/diff-so-fancy/diff_so_fancy.test.ts

@@ -1,5 +1,6 @@
 import { toolInstallTest } from "tests";
 
+// TODO(Tyler): tool def is missing healthcheck
 toolInstallTest({
   toolName: "diff-so-fancy",
   toolVersion: "1.4.3",

tools/difft/difft.test.ts

@@ -1,5 +1,6 @@
 import { toolInstallTest } from "tests";
 
+// TODO(Tyler): tool def is missing healthcheck
 toolInstallTest({
   toolName: "difft",
   toolVersion: "0.56.1",

tools/gk/gk.test.ts

@@ -1,5 +1,6 @@
 import { toolInstallTest } from "tests";
 
+// TODO(Tyler): tool def is missing healthcheck
 toolInstallTest({
   toolName: "gk",
   toolVersion: "1.2.2",

tools/goreleaser/goreleaser.test.ts

@@ -4,6 +4,7 @@ import { osTimeoutMultiplier } from "tests/utils";
 // This install is quite slow on some Linux machines.
 jest.setTimeout(600000 * osTimeoutMultiplier);
 
+// TODO(Tyler): tool def is missing healthcheck
 toolInstallTest({
   toolName: "goreleaser",
   toolVersion: "1.25.1",

tools/paratest/paratest.test.ts

@@ -0,0 +1,8 @@
+import { toolInstallTest } from "tests";
+import { skipOS } from "tests/utils";
+
+toolInstallTest({
+  toolName: "paratest",
+  toolVersion: "7.4.3",
+  skipTestIf: skipOS(["win32"]),
+});

tools/paratest/plugin.yaml

@@ -9,3 +9,6 @@ tools:
       environment:
         - name: PATH
           list: ["${tool}/vendor/bin"]
+      health_checks:
+        - command: paratest --version
+          parse_regex: ParaTest v${semver}

tools/phpunit/phpunit.test.ts

@@ -0,0 +1,8 @@
+import { toolInstallTest } from "tests";
+import { skipOS } from "tests/utils";
+
+toolInstallTest({
+  toolName: "phpunit",
+  toolVersion: "11.1.3",
+  skipTestIf: skipOS(["win32"]),
+});

tools/phpunit/plugin.yaml

@@ -9,3 +9,6 @@ tools:
       environment:
         - name: PATH
           list: ["${tool}/vendor/bin"]
+      health_checks:
+        - command: phpunit --version
+          parse_regex: PHPUnit ${semver}

tools/tsc/tsc.test.ts

@@ -1,6 +1,7 @@
 import { toolInstallTest } from "tests";
 import { skipOS } from "tests/utils";
 
+// TODO(Tyler): tool def is missing healthcheck
 toolInstallTest({
   toolName: "tsc",
   toolVersion: "5.2.2",

tools/yq/yq.test.ts

@@ -1,5 +1,6 @@
 import { toolInstallTest } from "tests";
 
+// TODO(Tyler): tool def is missing healthcheck
 toolInstallTest({
   toolName: "yq",
   toolVersion: "4.40.5",