Various udpates and more tests (#235)

Author: JoshuaEstes

.github/workflows/codecov.yml

@@ -42,11 +42,11 @@ jobs:
                   XDEBUG_MODE: coverage
                   PHPUNIT_OPTIONS: --coverage-clover coverage.xml --log-junit junit.xml
             - name: Upload coverage reports to Codecov
-              uses: codecov/codecov-action@v3
-              env:
-                  CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
+              uses: codecov/codecov-action@v4
+              with:
+                  token: ${{ secrets.CODECOV_TOKEN }}
             - name: Upload test results to Codecov
               if: ${{ !cancelled() }}
               uses: codecov/test-results-action@v1
               with:
-                token: ${{ secrets.CODECOV_TOKEN }}
+                  token: ${{ secrets.CODECOV_TOKEN }}

composer.json

@@ -248,4 +248,4 @@
             "@php -r \"if(file_exists('./.git')&&file_exists('./build/hooks/pre-commit')){copy('./build/hooks/pre-commit','./.git/hooks/pre-commit');chmod('./.git/hooks/pre-commit',0755);}\""
         ]
     }
-}
+}

docs/bard/commands.md

@@ -1,4 +1,9 @@
-# Commands
+---
+title: Bard Commands
+---
+
+All commands have usage examples along with help documentation in the commands.
+To view more details, run `bard <command> -h`.
 
 ## init
 

docs/bard/overview.md

@@ -2,10 +2,19 @@
 title: Bard
 ---
 
-# Bard
-
 Bard is used to manage monorepos.
 
+## Installing
+
+Bard is meant to be installed into a "tools" folder for use with your project.
+
+```shell
+mkdir tools/bard
+composer require --working-dir=tools/bard sonsofphp/bard
+echo "composer.lock\nvendor/" > tools/bard/.gitignore
+php tools/bard/vendor/bin/bard
+```
+
 ## Usage
 
 Initialize a new bard.json file for new monorepos.

src/SonsOfPHP/Bard/Tests/Console/Command/AddCommandTest.php

@@ -21,7 +21,7 @@
 use SonsOfPHP\Bard\Console\Command\SplitCommand;
 use SonsOfPHP\Bard\Console\Command\UpdateCommand;
 use SonsOfPHP\Bard\JsonFile;
-use SonsOfPHP\Bard\Worker\File\Bard\AddPackageWorker;
+use SonsOfPHP\Bard\Operation\Bard\AddPackageOperation;
 use Symfony\Component\Console\Tester\CommandTester;
 
 #[Group('bard')]
@@ -38,7 +38,7 @@
 #[UsesClass(SplitCommand::class)]
 #[UsesClass(UpdateCommand::class)]
 #[UsesClass(JsonFile::class)]
-#[UsesClass(AddPackageWorker::class)]
+#[UsesClass(AddPackageOperation::class)]
 final class AddCommandTest extends TestCase
 {
     private Application $application;

src/SonsOfPHP/Bard/Tests/Console/Command/CopyCommandTest.php

@@ -21,7 +21,7 @@
 use SonsOfPHP\Bard\Console\Command\SplitCommand;
 use SonsOfPHP\Bard\Console\Command\UpdateCommand;
 use SonsOfPHP\Bard\JsonFile;
-use SonsOfPHP\Bard\Worker\File\Bard\AddPackageWorker;
+use SonsOfPHP\Bard\Operation\Bard\AddPackageOperation;
 use Symfony\Component\Console\Tester\CommandTester;
 
 #[Group('bard')]
@@ -38,7 +38,7 @@
 #[UsesClass(SplitCommand::class)]
 #[UsesClass(UpdateCommand::class)]
 #[UsesClass(JsonFile::class)]
-#[UsesClass(AddPackageWorker::class)]
+#[UsesClass(AddPackageOperation::class)]
 final class CopyCommandTest extends TestCase
 {
     private Application $application;

src/SonsOfPHP/Bard/Tests/Console/Command/MergeCommandTest.php

@@ -21,17 +21,17 @@
 use SonsOfPHP\Bard\Console\Command\SplitCommand;
 use SonsOfPHP\Bard\Console\Command\UpdateCommand;
 use SonsOfPHP\Bard\JsonFile;
-use SonsOfPHP\Bard\Worker\File\Composer\Package\Authors;
-use SonsOfPHP\Bard\Worker\File\Composer\Package\BranchAlias;
-use SonsOfPHP\Bard\Worker\File\Composer\Package\Funding;
-use SonsOfPHP\Bard\Worker\File\Composer\Package\Support;
-use SonsOfPHP\Bard\Worker\File\Composer\Root\ClearSection;
-use SonsOfPHP\Bard\Worker\File\Composer\Root\UpdateAutoloadDevSection;
-use SonsOfPHP\Bard\Worker\File\Composer\Root\UpdateAutoloadSection;
-use SonsOfPHP\Bard\Worker\File\Composer\Root\UpdateProvideSection;
-use SonsOfPHP\Bard\Worker\File\Composer\Root\UpdateReplaceSection;
-use SonsOfPHP\Bard\Worker\File\Composer\Root\UpdateRequireDevSection;
-use SonsOfPHP\Bard\Worker\File\Composer\Root\UpdateRequireSection;
+use SonsOfPHP\Bard\Operation\ClearSectionOperation;
+use SonsOfPHP\Bard\Operation\Composer\Package\CopyAuthorsSectionFromRootToPackageOperation;
+use SonsOfPHP\Bard\Operation\Composer\Package\CopyBranchAliasValueFromRootToPackageOperation;
+use SonsOfPHP\Bard\Operation\Composer\Package\CopyFundingSectionFromRootToPackageOperation;
+use SonsOfPHP\Bard\Operation\Composer\Package\CopySupportSectionFromRootToPackageOperation;
+use SonsOfPHP\Bard\Operation\Composer\Root\UpdateAutoloadDevSectionOperation;
+use SonsOfPHP\Bard\Operation\Composer\Root\UpdateAutoloadSectionOperation;
+use SonsOfPHP\Bard\Operation\Composer\Root\UpdateProvideSectionOperation;
+use SonsOfPHP\Bard\Operation\Composer\Root\UpdateReplaceSectionOperation;
+use SonsOfPHP\Bard\Operation\Composer\Root\UpdateRequireDevSectionOperation;
+use SonsOfPHP\Bard\Operation\Composer\Root\UpdateRequireSectionOperation;
 use Symfony\Component\Console\Tester\CommandTester;
 
 #[Group('bard')]
@@ -48,17 +48,17 @@
 #[UsesClass(SplitCommand::class)]
 #[UsesClass(UpdateCommand::class)]
 #[UsesClass(JsonFile::class)]
-#[UsesClass(Authors::class)]
-#[UsesClass(BranchAlias::class)]
-#[UsesClass(Funding::class)]
-#[UsesClass(Support::class)]
-#[UsesClass(ClearSection::class)]
-#[UsesClass(UpdateAutoloadDevSection::class)]
-#[UsesClass(UpdateAutoloadSection::class)]
-#[UsesClass(UpdateProvideSection::class)]
-#[UsesClass(UpdateReplaceSection::class)]
-#[UsesClass(UpdateRequireSection::class)]
-#[UsesClass(UpdateRequireDevSection::class)]
+#[UsesClass(CopyAuthorsSectionFromRootToPackageOperation::class)]
+#[UsesClass(CopyBranchAliasValueFromRootToPackageOperation::class)]
+#[UsesClass(CopyFundingSectionFromRootToPackageOperation::class)]
+#[UsesClass(CopySupportSectionFromRootToPackageOperation::class)]
+#[UsesClass(ClearSectionOperation::class)]
+#[UsesClass(UpdateAutoloadDevSectionOperation::class)]
+#[UsesClass(UpdateAutoloadSectionOperation::class)]
+#[UsesClass(UpdateProvideSectionOperation::class)]
+#[UsesClass(UpdateReplaceSectionOperation::class)]
+#[UsesClass(UpdateRequireSectionOperation::class)]
+#[UsesClass(UpdateRequireDevSectionOperation::class)]
 final class MergeCommandTest extends TestCase
 {
     private Application $application;

src/SonsOfPHP/Bard/Tests/Console/Command/PushCommandTest.php

@@ -21,7 +21,7 @@
 use SonsOfPHP\Bard\Console\Command\SplitCommand;
 use SonsOfPHP\Bard\Console\Command\UpdateCommand;
 use SonsOfPHP\Bard\JsonFile;
-use SonsOfPHP\Bard\Worker\File\Bard\AddPackageWorker;
+use SonsOfPHP\Bard\Operation\Bard\AddPackageOperation;
 use Symfony\Component\Console\Tester\CommandTester;
 
 #[Group('bard')]
@@ -38,7 +38,7 @@
 #[UsesClass(SplitCommand::class)]
 #[UsesClass(UpdateCommand::class)]
 #[UsesClass(JsonFile::class)]
-#[UsesClass(AddPackageWorker::class)]
+#[UsesClass(AddPackageOperation::class)]
 final class PushCommandTest extends TestCase
 {
     private Application $application;

src/SonsOfPHP/Bard/Tests/Console/Command/ReleaseCommandTest.php

@@ -21,9 +21,18 @@
 use SonsOfPHP\Bard\Console\Command\SplitCommand;
 use SonsOfPHP\Bard\Console\Command\UpdateCommand;
 use SonsOfPHP\Bard\JsonFile;
-use SonsOfPHP\Bard\Worker\File\Bard\UpdateVersionWorker;
-use SonsOfPHP\Bard\Worker\File\Composer\Package\BranchAlias;
-use SonsOfPHP\Bard\Worker\File\Composer\Root\UpdateReplaceSection;
+use SonsOfPHP\Bard\Operation\Bard\UpdateVersionOperation;
+use SonsOfPHP\Bard\Operation\ClearSectionOperation;
+use SonsOfPHP\Bard\Operation\Composer\Package\CopyAuthorsSectionFromRootToPackageOperation;
+use SonsOfPHP\Bard\Operation\Composer\Package\CopyBranchAliasValueFromRootToPackageOperation;
+use SonsOfPHP\Bard\Operation\Composer\Package\CopyFundingSectionFromRootToPackageOperation;
+use SonsOfPHP\Bard\Operation\Composer\Package\CopySupportSectionFromRootToPackageOperation;
+use SonsOfPHP\Bard\Operation\Composer\Root\UpdateAutoloadDevSectionOperation;
+use SonsOfPHP\Bard\Operation\Composer\Root\UpdateAutoloadSectionOperation;
+use SonsOfPHP\Bard\Operation\Composer\Root\UpdateProvideSectionOperation;
+use SonsOfPHP\Bard\Operation\Composer\Root\UpdateReplaceSectionOperation;
+use SonsOfPHP\Bard\Operation\Composer\Root\UpdateRequireDevSectionOperation;
+use SonsOfPHP\Bard\Operation\Composer\Root\UpdateRequireSectionOperation;
 use SonsOfPHP\Component\Version\Version;
 use Symfony\Component\Console\Tester\CommandTester;
 
@@ -41,9 +50,18 @@
 #[UsesClass(SplitCommand::class)]
 #[UsesClass(UpdateCommand::class)]
 #[UsesClass(JsonFile::class)]
-#[UsesClass(UpdateVersionWorker::class)]
-#[UsesClass(BranchAlias::class)]
-#[UsesClass(UpdateReplaceSection::class)]
+#[UsesClass(CopyAuthorsSectionFromRootToPackageOperation::class)]
+#[UsesClass(CopyBranchAliasValueFromRootToPackageOperation::class)]
+#[UsesClass(CopyFundingSectionFromRootToPackageOperation::class)]
+#[UsesClass(CopySupportSectionFromRootToPackageOperation::class)]
+#[UsesClass(ClearSectionOperation::class)]
+#[UsesClass(UpdateAutoloadDevSectionOperation::class)]
+#[UsesClass(UpdateAutoloadSectionOperation::class)]
+#[UsesClass(UpdateProvideSectionOperation::class)]
+#[UsesClass(UpdateReplaceSectionOperation::class)]
+#[UsesClass(UpdateRequireSectionOperation::class)]
+#[UsesClass(UpdateRequireDevSectionOperation::class)]
+#[UsesClass(UpdateVersionOperation::class)]
 #[UsesClass(Version::class)]
 final class ReleaseCommandTest extends TestCase
 {

src/SonsOfPHP/Bard/Tests/JsonFileTest.php

@@ -8,6 +8,7 @@
 use PHPUnit\Framework\Attributes\Group;
 use PHPUnit\Framework\TestCase;
 use SonsOfPHP\Bard\JsonFile;
+use SonsOfPHP\Bard\Operation\OperationInterface;
 
 #[Group('bard')]
 #[CoversClass(JsonFile::class)]
@@ -67,4 +68,13 @@ public function testItCanConvertUpdatedConfigToJson(): void
         $this->assertArrayHasKey('version', $json);
         $this->assertSame('1.2.4', $json['version']);
     }
+
+    public function testItWillApplyOperation(): void
+    {
+        $file = new JsonFile(__DIR__ . '/fixtures/test.json');
+        $operation = $this->createMock(OperationInterface::class);
+        $operation->expects($this->once())->method('apply')->willReturn($file);
+
+        $file->with($operation);
+    }
 }