Skip to content

Commit

Permalink
Merge pull request #6 from remkade/php7-tests
Browse files Browse the repository at this point in the history
Upgrading tests to phpunit 8
  • Loading branch information
remkade authored Jul 5, 2019
2 parents 615e861 + 3e2f086 commit 219a1e9
Show file tree
Hide file tree
Showing 7 changed files with 2,589 additions and 272 deletions.
4 changes: 1 addition & 3 deletions .travis.yml
Original file line number Diff line number Diff line change
@@ -1,9 +1,7 @@
language: php
php:
- 7.3
- 7.2
- 7.1
- 7.0
- 5.5
install:
- composer install --dev
- mkdir -p build/logs
Expand Down
4 changes: 4 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,10 @@ Status
* **TODO**: Add some configuration options
* **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.

PHP Versions
------------
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.

Security
--------
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.
Expand Down
10 changes: 8 additions & 2 deletions composer.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,12 @@
{
"require-dev": {
"satooshi/php-coveralls": "dev-master",
"codeclimate/php-test-reporter": "dev-master"
"php-coveralls/php-coveralls": "^2",
"codeclimate/php-test-reporter": "^0.4",
"phpunit/phpunit": "^8"
},
"config": {
"platform": {
"php": "7.2"
}
}
}
Loading

0 comments on commit 219a1e9

Please sign in to comment.