valkyrjaio / valkyrja-starter-app-php / 25969414172

0%

push

github

[Application] Update providers to use instances and align with new framework contracts (#56)

# Description

Updates all provider implementations to align with the new Valkyrja
framework contracts, which changed `ComponentProviderContract` and
`ServiceProviderContract` methods from static to instance. Renames
`RouteProvider` to `CliRouteProvider` and `HttpRouteProvider` for
clarity. Updates `ThrowableHandler::enable()` and its callers from
static to instance. Config `providers` arrays now pass instances instead
of class strings.

---

## Types of Changes

- [ ] Improvement _(non-breaking change which improves code)_
- [ ] Bug fix _(non-breaking change which fixes an issue)_
- [ ] New feature _(non-breaking change which adds functionality)_
- [ ] Deprecation _(breaking change which removes functionality)_
- [x] Breaking change _(fix or feature that would cause existing
functionality to change)_
- [ ] Documentation improvement

---

## Changes

- **`App\Cli\App`** / **`App\Http\App`** — call `(new
ThrowableHandler())->enable()` instead of static
`ThrowableHandler::enable()`
- **`App\Cli\Provider\ComponentProvider`** /
**`App\Http\Provider\ComponentProvider`** — converted all provider
methods from static to instance; `getComponentProviders()` now returns
`new CliWithHttpApplicationComponentProvider()` / `new
HttpApplicationComponentProvider()`; `getContainerProviders()` /
`getCliProviders()` / `getHttpProviders()` return instances
- **`App\Cli\Provider\DataServiceProvider`** /
**`App\Http\Provider\DataServiceProvider`** /
**`App\Cli\Provider\ServiceProvider`** /
**`App\Http\Provider\ServiceProvider`** — `publishers()` converted from
static to instance method
- **`App\Cli\Provider\RouteProvider`** →
**`App\Cli\Provider\CliRouteProvider`** — renamed for clarity; converted
`getControllerClasses()` and `getRoutes()` from static to instance
- **`App\Http\Provider\RouteProvider`** →
**`App\Http\Provider\HttpRouteProvider`** — renamed for clarity;
converted `ge... (continued)

0 of 82 new or added lines in 13 files covered. (0.0%)

0 of 297 relevant lines covered (0.0%)

0.0 hits per line