tests: add functional test (#75)

* feat: Removed AbstractEvent and event Uid

* fix: Fixed CS

* chore: Fixed factory name

* chore: Added 100% test coverage

Author: yohang

.dagger/src/CalendR.php

@@ -66,10 +66,39 @@ public function test(
         #[DefaultPath('.')] Directory $source,
         string $phpVersion = '8.4',
         string $dependencyVersion = 'highest',
-    ): string {
-        return $this
+        ?string $coverallsRepoToken = null,
+        ?string $ciName = null,
+        ?string $ciJobId = null,
+        ?string $ciBranch = null,
+    ): string
+    {
+        $container = $this
             ->build($source, $phpVersion, $dependencyVersion)
-            ->withExec(['php', './vendor/bin/phpunit', '--coverage-text'])
+            ->withFile('/tmp/coveralls-linux.tar.gz', dag()->http('https://coveralls.io/coveralls-linux.tar.gz'))
+            ->withExec(['tar', '-xvzf', '/tmp/coveralls-linux.tar.gz', '-C', '/usr/local/bin']);
+
+
+        $exec = ['php', './vendor/bin/phpunit', '--coverage-text'];
+        if ($coverallsRepoToken) {
+            $exec[] = '--coverage-clover=build/logs/clover.xml';
+
+            $container = $container->withEnvVariable('COVERALLS_REPO_TOKEN', $coverallsRepoToken);
+        }
+
+        if ($ciName && $ciJobId && $ciBranch) {
+            $container = $container
+                ->withEnvVariable('CI_NAME', $ciName)
+                ->withEnvVariable('CI_JOB_ID', $ciJobId)
+                ->withEnvVariable('CI_BRANCH', $ciBranch);
+        }
+
+        $container = $container->withExec($exec);
+
+        if ($coverallsRepoToken) {
+            $container = $container->withExec(['coveralls', 'report']);
+        }
+
+        return $container
             ->stdout();
     }
 

.github/workflows/ci.yml

@@ -56,7 +56,7 @@ jobs:
           with:
             version: "latest"
             verb: call
-            args: test --source=. --php-version=${{ matrix.php-versions }} --dependency-version=${{ matrix.dependencies-versions }}
+            args: test --source=. --php-version=${{ matrix.php-versions }} --dependency-version=${{ matrix.dependencies-versions }} --coveralls-repo-token=${{ secrets.COVERALLS_REPO_TOKEN }} --ci-name=gha --ci-job-id=${{ github.run_id }} --ci-branch=${{ github.ref }}
             cloud-token: ${{ secrets.DAGGER_CLOUD_TOKEN }}
 
   mutation:

README.md

@@ -5,6 +5,7 @@
 CalendR provides a clean, immutable, and iterable API to manipulate time periods (Years, Months, Weeks, Days...) and manage associated events.
 
 [![CI Status](https://github.com/yohang/CalendR/actions/workflows/ci.yml/badge.svg)](https://github.com/yohang/CalendR/actions/workflows/ci.yml)
+[![Coverage Status](https://coveralls.io/repos/github/yohang/CalendR/badge.svg?branch=master)](https://coveralls.io/github/yohang/CalendR?branch=main)
 [![Mutation testing badge](https://img.shields.io/endpoint?style=flat&url=https%3A%2F%2Fbadge-api.stryker-mutator.io%2Fgithub.com%2Fyohang%2FCalendR%2Fmain)](https://dashboard.stryker-mutator.io/reports/github.com/yohang/CalendR/main)
 
 ## ✨ Features

src/Bridge/Symfony/Bundle/DependencyInjection/CalendRExtension.php

@@ -27,7 +27,7 @@ public function load(array $configs, ContainerBuilder $container): void
 
         $defaultFirstWeekday = $config['periods']['default_first_weekday'];
         if (!($defaultFirstWeekday instanceof DayOfWeek)) {
-            $defaultFirstWeekday = DayOfWeek::from($defaultFirstWeekday);
+            $defaultFirstWeekday = DayOfWeek::from($defaultFirstWeekday); // @codeCoverageIgnore
         }
 
         $container

src/Bridge/Symfony/Bundle/DependencyInjection/Configuration.php

@@ -28,13 +28,15 @@ public function getConfigTreeBuilder(): TreeBuilder
                 ->enumFqcn(DayOfWeek::class)
                 ->defaultValue(DayOfWeek::MONDAY);
         } else {
+            // @codeCoverageIgnoreStart
             $enumNode
                 ->values(array_map(fn (DayOfWeek $dayOfWeek) => $dayOfWeek->value, DayOfWeek::cases()))
                 ->defaultValue(DayOfWeek::MONDAY->value)
                 ->validate()
                     ->ifNotInArray(array_map(static fn (DayOfWeek $d) => $d->value, DayOfWeek::cases()))
                     ->thenInvalid('Day must be be between 0 (Sunday) and 6 (Saturday)')
                 ->end();
+            // @codeCoverageIgnoreEnd
         }
 
         return $treeBuilder;

src/Bridge/Symfony/Bundle/Resources/config/services.php

@@ -5,7 +5,7 @@
 use CalendR\Bridge\Twig\CalendRExtension;
 use CalendR\Calendar;
 use CalendR\Event\EventManager;
-use CalendR\Period\PeriodPeriodFactory;
+use CalendR\Period\PeriodFactory;
 use Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator;
 
 use function Symfony\Component\DependencyInjection\Loader\Configurator\service;
@@ -16,12 +16,12 @@
             ->public();
 
     $container->services()
-        ->set(PeriodPeriodFactory::class)
+        ->set(PeriodFactory::class)
             ->public();
 
     $container->services()
         ->set(Calendar::class)
-            ->arg('$factory', service(PeriodPeriodFactory::class))
+            ->arg('$factory', service(PeriodFactory::class))
             ->arg('$eventManager', service(EventManager::class))
             ->public();
 
@@ -35,7 +35,7 @@
             ->public();
 
     $container->services()
-        ->alias('calendr.factory', PeriodPeriodFactory::class)
+        ->alias('calendr.factory', PeriodFactory::class)
             ->public();
 
     $container->services()

src/Calendar.php

@@ -11,9 +11,9 @@
 use CalendR\Period\Hour;
 use CalendR\Period\Minute;
 use CalendR\Period\Month;
+use CalendR\Period\PeriodFactory;
 use CalendR\Period\PeriodFactoryInterface;
 use CalendR\Period\PeriodInterface;
-use CalendR\Period\PeriodPeriodFactory;
 use CalendR\Period\Second;
 use CalendR\Period\Week;
 use CalendR\Period\Year;
@@ -26,7 +26,7 @@
 readonly class Calendar
 {
     public function __construct(
-        protected PeriodFactoryInterface $factory = new PeriodPeriodFactory(),
+        protected PeriodFactoryInterface $factory = new PeriodFactory(),
         private EventManager $eventManager = new EventManager(),
     ) {
     }

src/Event/Collection/IndexedCollection.php

@@ -47,10 +47,7 @@ final class IndexedCollection implements CollectionInterface, \IteratorAggregate
      */
     public function __construct(array $events = [], ?callable $callable = null)
     {
-        $this->indexFunction = (static fn (\DateTimeInterface $dateTime): string => $dateTime->format('Y-m-d'));
-        if (\is_callable($callable)) {
-            $this->indexFunction = $callable;
-        }
+        $this->indexFunction = $callable ?? static fn (\DateTimeInterface $dateTime): string => $dateTime->format('Y-m-d');
 
         foreach ($events as $event) {
             $this->add($event);

src/Period/Month.php

@@ -33,7 +33,7 @@ public function getDays(): array
 
     /**
      * Returns the first day of the first week of month.
-     * First day of week is configurable via {@link PeriodPeriodFactory}.
+     * First day of week is configurable via {@link PeriodFactory}.
      */
     public function getFirstDayOfFirstWeek(): \DateTimeImmutable
     {
@@ -51,7 +51,7 @@ public function getExtendedMonth(): PeriodInterface
 
     /**
      * Returns the last day of last week of month
-     * First day of week is configurable via {@link PeriodPeriodFactory}.
+     * First day of week is configurable via {@link PeriodFactory}.
      */
     public function getLastDayOfLastWeek(): \DateTimeImmutable
     {

src/Period/PeriodAbstract.php

@@ -110,7 +110,7 @@ public function getEnd(): \DateTimeImmutable
     protected function getFactory(): PeriodFactoryInterface
     {
         if (null === $this->factory) {
-            $this->factory = new PeriodPeriodFactory();
+            $this->factory = new PeriodFactory();
         }
 
         return $this->factory;