Merge pull request #119 from humanmade/fix-travis

Add support for returning `WP_Error` objects

.gitignore

@@ -0,0 +1,4 @@
+tests/tests/class-wp-core-cron.php
+.phpunit.result.cache
+composer.lock
+vendor/

.travis.yml

@@ -1,24 +1,25 @@
 language: php
 php:
-  - 7.1
-  - 7.2
-  - 7.3
+  - 7.4
+  - 8.0
+  - 8.1
 env:
   - WP_VERSION=latest WP_MULTISITE=0
   - WP_VERSION=latest WP_MULTISITE=1
 matrix:
   include:
-    - php: 7.3
+    - php: 8.0
       env: WP_VERSION=trunk WP_MULTISITE=0
-    - php: 7.3
+    - php: 8.0
       env: WP_VERSION=trunk WP_MULTISITE=1
+allow_failures:
+  - php: 8.1
 services:
   - mysql
 install:
   - composer install
   - bash tests/install-tests.sh wordpress_test root '' 127.0.0.1 $WP_VERSION
 script:
-  # 32656 tests are related to the pre_*_event filters which we're using already
-  - vendor/bin/phpunit --exclude-group 32656
+  - vendor/bin/phpunit
 notifications:
     email: false

composer.json

@@ -11,10 +11,15 @@
 		}
 	],
 	"require": {
-		"composer/installers": "~1.0"
+		"composer/installers": "^1.0 || ^2.0"
 	},
 	"require-dev": {
-		"humanmade/coding-standards": "dev-master",
-		"phpunit/phpunit": "7.1"
+		"phpunit/phpunit": "^7.1 || ^9.0",
+		"yoast/phpunit-polyfills": "^1.0"
+	},
+	"config": {
+		"allow-plugins": {
+			"composer/installers": true
+		}
 	}
 }

inc/class-job.php

@@ -70,6 +70,7 @@ public function save() {
 
 		self::flush_query_cache();
 		wp_cache_set( "job::{$this->id}", $this, 'cavalcade-jobs' );
+		return (bool) $result;
 	}
 
 	public function delete( $options = [] ) {