Make Dockerfile compatible with PHPUnit 9

[stephandesouza]

PHPUnit 9 only runs with PHP >=7.3, also updated to Composer 2

[codecov-io]

Codecov Report

Merging #2186 (c9eba55) into master (ae864fb) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #2186   +/-   ##
=========================================
  Coverage     86.99%   86.99%           
  Complexity      664      664           
=========================================
  Files            33       33           
  Lines          1561     1561           
=========================================
  Hits           1358     1358           
  Misses          203      203           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ae864fb...c9eba55. Read the comment docs.

[Smolevich]

ARG is param can be passed through docker build --build-arg PHP_VERSION=8.0 ... I think that changing default params is can be realized later, after big refactoring code base

[stephandesouza]

Oh, thanks!! I really forgot about build-args on Docker.

[divine]

Docker file already doesn't run correctly tests. Bumping two variables doesn't change anything.

Thanks!

[stephandesouza]

Well, ok, but I could run standalone test correctly on master with this bump.

So others might run this kind of tests too.

[divine]

Well, ok, but I could run standalone test correctly on master with this bump.

So others might run this kind of tests too.

Docker file doesn't contain any code to run tests. Could you please show me where exactly it does otherwise?

Thanks!

[stephandesouza]

Sure, with docker-compose it runs smoothly: docker-compose run tests ./vendor/bin/phpunit

[divine]

Sure, with docker-compose it runs smoothly: docker-compose run tests ./vendor/bin/phpunit

What was I meaning that when you start it with docker-compose the PHP container should run tests automatically.

There is no point in running it manually each time. @Smolevich what do you think?

Thanks!

[divine]

Thank you for your contribution! Much appreciated!