Adds header to generated files

raphaelstolt · Sep 25, 2023 · 2245c04 · 2245c04
1 parent ef14f96
commit 2245c04
Show file tree

Hide file tree

Showing 7 changed files with 53 additions and 28 deletions.
diff --git a/bin/release-version b/bin/release-version
@@ -14,8 +14,9 @@ foreach ($autoloads as $autoload) {
     }
 }
 
+use Composer\Semver\Comparator;
 use Stolt\LeanPackage\Helpers\Str as OsHelper;
-use vierbergenlars\SemVer\version as SemVer;
+use PHLAK\SemVer\Version;
 
 $binApplicationName = 'lean-package-validator';
 $binFile = __DIR__ . DIRECTORY_SEPARATOR . $binApplicationName;
@@ -130,7 +131,7 @@ if ($version === null) {
 }
 
 if ($version) {
-    if (SemVer::lte($version, $currentVersion)) {
+    if (Comparator::lessThanOrEqualTo($version, $currentVersion)) {
         echo "Earlier version '{$version}' provided." . PHP_EOL;
         exit(1);
     }
@@ -156,7 +157,17 @@ if ($increment) {
         exit(1);
     }
 
-    $version = (new SemVer($currentVersion))->inc($increment)->getVersion();
+    switch ($increment) {
+        case 'major':
+            $version = (new Version($currentVersion))->incrementMajor()->__toString();
+            break;
+        case 'minor':
+            $version = (new Version($currentVersion))->incrementMinor()->__toString();
+            break;
+        case 'patch':
+            $version = (new Version($currentVersion))->incrementPatch()->__toString();
+            break;
+    }
 }
 
 if (!isset($version)) {

diff --git a/composer.json b/composer.json
@@ -53,9 +53,9 @@
     "require-dev": {
         "friendsofphp/php-cs-fixer": "^3.0",
         "mockery/mockery": "^1.0",
+        "phlak/semver": "^4.1",
         "php-mock/php-mock-phpunit": "^2.4||^1.1",
         "phpstan/phpstan": "^1.6.2",
-        "phpunit/phpunit": "8.*",
-        "vierbergenlars/php-semver": "^3.0"
+        "phpunit/phpunit": "^10.3"
     }
 }
diff --git a/phpunit.xml.dist b/phpunit.xml.dist
@@ -1,21 +1,14 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<phpunit backupGlobals="false"
-         backupStaticAttributes="false"
-         bootstrap="vendor/autoload.php"
-         colors="true"
-         convertErrorsToExceptions="true"
-         convertNoticesToExceptions="true"
-         convertWarningsToExceptions="true"
-         processIsolation="false"
-         stopOnFailure="true">
-    <testsuites>
-        <testsuite name="Lean Package Validator">
-            <directory>tests/</directory>
-        </testsuite>
-    </testsuites>
-    <filter>
-        <whitelist>
-            <directory suffix=".php">src/</directory>
-        </whitelist>
-    </filter>
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" backupGlobals="false" bootstrap="vendor/autoload.php" colors="true" processIsolation="false" stopOnFailure="true" xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.3/phpunit.xsd" cacheDirectory=".phpunit.cache" backupStaticProperties="false">
+  <coverage/>
+  <testsuites>
+    <testsuite name="Lean Package Validator">
+      <directory>tests/</directory>
+    </testsuite>
+  </testsuites>
+  <source>
+    <include>
+      <directory suffix=".php">src/</directory>
+    </include>
+  </source>
 </phpunit>
diff --git a/src/Commands/InitCommand.php b/src/Commands/InitCommand.php
@@ -90,6 +90,10 @@ protected function execute(InputInterface $input, OutputInterface $output)
         }
 
         $defaultGlobPatterns = $this->analyser->getDefaultGlobPatterns();
+        array_unshift(
+            $defaultGlobPatterns,
+            'This file was generated by the lean package validator (http://git.io/lean-package-validator).' . PHP_EOL
+        );
         $lpvFileContent = implode("\n", $defaultGlobPatterns);
 
         $bytesWritten = file_put_contents(

diff --git a/src/Commands/ValidateCommand.php b/src/Commands/ValidateCommand.php
@@ -238,6 +238,13 @@ protected function execute(InputInterface $input, OutputInterface $output)
             if ($expectedGitattributesFileContent !== '') {
                 if ($createGitattributesFile || $overwriteGitattributesFile) {
                     try {
+                        $headerContent = 'This file was partly modified by the lean package validator (http://git.io/lean-package-validator).' . PHP_EOL;
+
+                        if ($createGitattributesFile) {
+                            $headerContent = 'This file was generated by the lean package validator (http://git.io/lean-package-validator).' . PHP_EOL;
+                        }
+
+                        $expectedGitattributesFileContent = $headerContent . PHP_EOL . $expectedGitattributesFileContent;
                         $outputContent .= $this->createGitattributesFile(
                             $expectedGitattributesFileContent
                         );

diff --git a/tests/Commands/InitCommandTest.php b/tests/Commands/InitCommandTest.php
@@ -66,6 +66,8 @@ public function createsExpectedDefaultLpvFile()
 CONTENT;
 
         $expectedDefaultLpvFileContent = <<<CONTENT
+This file was generated by the lean package validator (http://git.io/lean-package-validator).
+
 .*
 *.lock
 *.txt

diff --git a/tests/Commands/ValidateCommandTest.php b/tests/Commands/ValidateCommandTest.php
@@ -598,9 +598,11 @@ public function validateOnNonExistentGitattributesFilesWithCreationOptionCreates
         ]);
 
         $expectedDisplay = <<<CONTENT
-Warning: There is no .gitattributes file present in {$this->temporaryDirectory}.
+Warning: There is no .gitattributes file present in /tmp/lpv.
 
 Created a .gitattributes file with the shown content:
+This file was generated by the lean package validator (http://git.io/lean-package-validator).
+
 * text=auto eol=lf
 
 .gitattributes export-ignore
@@ -610,7 +612,7 @@ public function validateOnNonExistentGitattributesFilesWithCreationOptionCreates
 
 CONTENT;
 
-        $this->assertSame($expectedDisplay, $commandTester->getDisplay());
+        $this->assertEquals($expectedDisplay, $commandTester->getDisplay());
         $this->assertTrue($commandTester->getStatusCode() == 0);
         $this->assertFileExists(
             WORKING_DIRECTORY . DIRECTORY_SEPARATOR . '.gitattributes'
@@ -642,6 +644,8 @@ public function validateOnNonExistentGitattributesFilesWithCreationOptionCreates
 Warning: There is no .gitattributes file present in {$this->temporaryDirectory}.
 
 Created a .gitattributes file with the shown content:
+This file was generated by the lean package validator (http://git.io/lean-package-validator).
+
 * text=auto eol=lf
 
 .gitattributes export-ignore
@@ -652,6 +656,8 @@ public function validateOnNonExistentGitattributesFilesWithCreationOptionCreates
 CONTENT;
 
         $expectedGitattributesContent = <<<CONTENT
+This file was generated by the lean package validator (http://git.io/lean-package-validator).
+
 * text=auto eol=lf
 
 .gitattributes export-ignore
@@ -660,7 +666,7 @@ public function validateOnNonExistentGitattributesFilesWithCreationOptionCreates
 
 CONTENT;
 
-        $this->assertSame($expectedDisplay, $commandTester->getDisplay());
+        $this->assertEquals($expectedDisplay, $commandTester->getDisplay());
         $this->assertTrue($commandTester->getStatusCode() == 0);
         $this->assertStringEqualsFile(
             WORKING_DIRECTORY . DIRECTORY_SEPARATOR . '.gitattributes',
@@ -854,6 +860,8 @@ public function overwriteOptionOnNonExistentGitattributesFileImplicatesCreate()
 Warning: There is no .gitattributes file present in {$this->temporaryDirectory}.
 
 Created a .gitattributes file with the shown content:
+This file was partly modified by the lean package validator (http://git.io/lean-package-validator).
+
 * text=auto eol=lf
 
 .gitattributes export-ignore
@@ -1819,7 +1827,7 @@ public function gitignoredFilesAreExcludedFromValidation()
     /**
      * @return array
      */
-    public function optionProvider()
+    public static function optionProvider()
     {
         return [
             ['--overwrite'],
-Original file line number
+Diff line change
@@ Expand Up / @@ -66,6 +66,8 @@ public function createsExpectedDefaultLpvFile() @@
     CONTENT;
             $expectedDefaultLpvFileContent = <<<CONTENT
+    This file was generated by the lean package validator (http://git.io/lean-package-validator).
     .*
     *.lock
     *.txt
@@ Expand Down @@