Skip to content

Refactor tests - #344

Merged
Nyholm merged 1 commit into
geocoder-php:masterfrom
norkunas:refactor-tests
Oct 3, 2023
Merged

Refactor tests#344
Nyholm merged 1 commit into
geocoder-php:masterfrom
norkunas:refactor-tests

Conversation

@norkunas

@norkunas norkunas commented Oct 2, 2023

Copy link
Copy Markdown
Member
  • Drop dependency on OrmTestCase and instead use DoctrineBundle for setting orm automatically instead of manually;
  • Use MockHttpClient from symfony/http-client instead of making real http calls in tests Use mock Client from php-http/mock-client;
  • Merge CI's phpunit jobs into single one;
  • Separate inlined fixtures in tests to separate classes;

@norkunas
norkunas requested a review from Nyholm October 2, 2023 10:05
@norkunas
norkunas force-pushed the refactor-tests branch 12 times, most recently from 052901e to d9d728c Compare October 2, 2023 12:10
- Drop dependency on `OrmTestCase` and instead use `DoctrineBundle` for setting orm automatically instead of manually;
- ~~Use MockHttpClient from symfony/http-client instead of making real http calls in tests~~ Use mock Client from php-http/mock-client;
- Merge CI's phpunit jobs into single one;
- Separate inlined fixtures in tests to separate classes;

@Nyholm Nyholm left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great!
Thank you

@Nyholm
Nyholm merged commit 608f883 into geocoder-php:master Oct 3, 2023
@norkunas
norkunas deleted the refactor-tests branch October 3, 2023 04:21
@mbrodala

mbrodala commented Mar 28, 2025

Copy link
Copy Markdown

This silently updated event subscribers to event listeners, see stof/StofDoctrineExtensionsBundle#450

More specifically doctrine/DoctrineBundle#1664

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.

3 participants