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

Update vagrant and fix unit test #588

Merged
merged 2 commits into from
Nov 19, 2022
Merged

Update vagrant and fix unit test #588

merged 2 commits into from
Nov 19, 2022

Conversation

blackcoder87
Copy link
Member

@blackcoder87 blackcoder87 commented Nov 18, 2022

Description

  • Update Vagrant to Debian 11.
  • assertMatchesRegularExpression() is not available in older versions of PHPUnit. Vagrant and GitHub Actions use PHPUnit in version 5.7.27
  • Wrong username and password in readme for running unit tests
  1. Ilch\RouterTest::testDefaultRegexpPattern
    Error: Call to undefined method Ilch\RouterTest::assertMatchesRegularExpression()

/vagrant/tests/libraries/ilch/RouterTest.php:40
/vagrant/vendor/sminnee/phpunit/phpunit:51

Type of change

  • Bug fix (non-breaking change which fixes an issue)

@hhunderter
Wie führst du Unittests aus? Nutzt du eine neuere Version von PHPUnit?

- Update Vagrant to Debian 11.
- assertMatchesRegularExpression() is not available in older versions of phpunit
- Wrong username and password in readme for running unit tests
@hhunderter
Copy link
Member

Stimmt, ich habe zwar nicht die neuste, aber das sie gerade so php8 unterstützt ("phpunit/phpunit": "^9.5").

@blackcoder87
Copy link
Member Author

Die Unittests sollten aber unter Vagrant und GitHub Actions laufen. Testweise die lokal mal mit neueren Versionen zu testen kann natürlich nicht schaden um schonmal zu gucken wo es in Zukunft Probleme geben könnte.

Denke es wäre sinnvoll #584 zu mergen, damit wenigstens überhaupt Unittests bei Änderungen ausgeführt werden. Idealerweise sollten die Unittests für PHP 7.0 bis 8.1 ausgeführt werden, solange wir diese Versionen auch unterstützen. Unterstützung für PHP 7.0 bis 7.2 sollte man aber zeitnah abschaffen. Dann könnte man wahrscheinlich auch PHPUnit in einer neueren Version ausführen. Es wäre also sinnvoll, wenn ihr einen Artikel schreibt zum Thema "Unterstützung für PHP 7.0 bis 7.2 wird bald eingestellt".

Siehe auch:
https://www.ilch.de/index.php?news-344
#482
#529
https://www.php.net/supported-versions.php

@Mairu Mairu merged commit f9e2866 into IlchCMS:master Nov 19, 2022
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