[CORE]Unit tests needs to change not to use container overide

eXpansionPluginPack / eXpansion2

The next generation server controller for maniaplanet using symfony components

http://mp-expansion.com/

MIT License

7 stars 3 forks source link

[CORE]Unit tests needs to change not to use container overide #107

Open oliverde8 opened 6 years ago

oliverde8 commented 6 years ago

It worked fine with SF3.2 but it's deprecated with 3.3 and even caused some issues in some tests.

No hurry as all works well, new tests should take this into consideration. This ticket is to migrate existing tests.

oliverde8 commented 6 years ago

Increasing the priority as this blocks upgrade to Symfony3.4 which is an LTS. And to symfony4 which could allow us to diminish the footprint of eXpansion relatively easily.

I am also tagging it with good first issue as it can be a nice exercise to learn to write unit tests and understand some of the core.