Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support only PHP 8.3 #66

Merged
merged 4 commits into from
Sep 18, 2024
Merged

Support only PHP 8.3 #66

merged 4 commits into from
Sep 18, 2024

Conversation

hugo-goncalves-kununu
Copy link
Collaborator

Description

The objective of this PR is to make the bundle support only PHP 8.3 and take advantage of it.

Details

BREAKING CHANGES

  • Only support PHP 8.3 - Make classes read-only when possible
    • Add types to constants
    • Upgrade PHPUnit to v11.3
      • Use static versions of asserts and expectations of invocations
    • Change CI pipeline to use PHP 8.3
    • Bump to use kununu/data-fixtures 12.0
  • Changes on AbstractTestCase and children (FixturesAwareTestCase and WebTestCase):
    • getFixturesContainer now has the Symfony ContainerInterface and not PSR interface as its return type
    • Add getServiceFromContainer
    • Add getParameterFromContainer
  • Add FixturesContainerGetterTrait to help getting common services from the test container:
    • getConnection to get a Doctrine DBAL connection
    • getCachePool to get a CacheItemPoolInterface
    • getElasticsearchClient to get an Elasticsearch client
    • getHttpClient to get a Symfony Http client
  • Update documentation

- Only support PHP 8.3
    - Make classes read-only when possible
	- Add types to constants
    - Upgrade PHPUnit to v11.3
		- Use static versions of asserts and expectations of invocations
	- Change CI pipeline to use PHP 8.3
	- Bump to use `kununu/data-fixtures` 12.0
- Changes on `AbstractTestCase` and children (`FixturesAwareTestCase` and `WebTestCase`):
	- `getFixturesContainer` now has the Symfony `ContainerInterface` and not PSR interface as its return type
	- Add `getServiceFromContainer`
	- Add `getParameterFromContainer`
- Add `FixturesContainerGetterTrait` to help getting common services from the test container:
	- `getConnection` to get a Doctrine DBAL connection
	- `getCachePool` to get a CacheItemPoolInterface
	- `getElasticsearchClient` to get an Elasticsearch client
	- `getHttpClient` to get a Symfony Http client
- Update documentation
@hugo-goncalves-kununu hugo-goncalves-kununu requested a review from a team as a code owner September 18, 2024 08:44
Copy link

@hugo-goncalves-kununu hugo-goncalves-kununu merged commit b21210c into master Sep 18, 2024
3 checks passed
@hugo-goncalves-kununu hugo-goncalves-kununu deleted the support_php83_only branch September 18, 2024 10:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant