Skip to content

Commit 219a1e9

Browse files
authored
Merge pull request #6 from remkade/php7-tests
Upgrading tests to phpunit 8
2 parents 615e861 + 3e2f086 commit 219a1e9

File tree

7 files changed

+2589
-272
lines changed

7 files changed

+2589
-272
lines changed

.travis.yml

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,7 @@
11
language: php
22
php:
3+
- 7.3
34
- 7.2
4-
- 7.1
5-
- 7.0
6-
- 5.5
75
install:
86
- composer install --dev
97
- mkdir -p build/logs

README.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,10 @@ Status
1010
* **TODO**: Add some configuration options
1111
* **TODO**: Add `full-stack-test.php` to make a MySQL connection and verify the whole stack loads, faster and more efficient than trying to do a full page rendering for your uptime checks.
1212

13+
PHP Versions
14+
------------
15+
This was originally written on PHP 5.3.x and so is probably still compatible with that version. However, upgrading to phpunit 8 required that I make tests require PHP 7.2+. So its probably still fine to run this on something less than 7.2, but since I can't get working tests on that version, I don't recommend it.
16+
1317
Security
1418
--------
1519
Make sure you limit access to the enpoints! You should not allow any yahoo off the internet to scan your site and look for these endpoints. I highly recommend some sort of `.htaccess` or nginx configuration settings to deny access to all but the local addresses you use for the API clients.

composer.json

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,12 @@
11
{
22
"require-dev": {
3-
"satooshi/php-coveralls": "dev-master",
4-
"codeclimate/php-test-reporter": "dev-master"
3+
"php-coveralls/php-coveralls": "^2",
4+
"codeclimate/php-test-reporter": "^0.4",
5+
"phpunit/phpunit": "^8"
6+
},
7+
"config": {
8+
"platform": {
9+
"php": "7.2"
10+
}
511
}
612
}

0 commit comments

Comments
 (0)