kjdev / php-ext-zstd

Zstd Extension for PHP
MIT License
201 stars 27 forks source link

Tests: Fix confused too old/new message #30

Closed glensc closed 4 years ago

glensc commented 4 years ago

The too old/too new messages were swapped, reported too old when should have said too new.

This also makes tests consistently use PHP_VERSION_ID. It's available since 5.2.7, but was already used by tests, so this is not a new dependency. Also, the change covers only tests, so bumping min version in package.xml shouldn't be necessary.

kjdev commented 4 years ago

Thanks.