Skip to content

Commit

Permalink
Provider methods needs to be static.
Browse files Browse the repository at this point in the history
  • Loading branch information
remcotolsma committed Mar 26, 2024
1 parent 5ff39ec commit 8c90b86
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 5 deletions.
4 changes: 2 additions & 2 deletions tests/src/DataHelperTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ public function test_validate_an( $value, $length, $exception = null ) {
*
* @return array<array<string|int>>
*/
public function validate_an_provider() {
public static function validate_an_provider() {
return [
// Valid.
[ '1234567890', 50 ],
Expand Down Expand Up @@ -83,7 +83,7 @@ public function test_sanitize_an( $value, $length, $expected ) {
*
* @return array<array<string|int>>
*/
public function sanitize_an_provider() {
public static function sanitize_an_provider() {
return [
// Nothing to-do.
[ '1234567890', 10, '1234567890' ],
Expand Down
2 changes: 1 addition & 1 deletion tests/src/ErrorTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ public function test_json( $file, $error_code, $error_message, $consumer_message
*
* @return array<array<string|int|null>>
*/
public function json_test_provider() {
public static function json_test_provider() {
return [
[ 'error-5001-full-authentication-required.json', 5001, 'Full authentication is required to access this resource', null ],
[ 'error-5001-invalid-or-missing-signature.json', 5001, null, 'Invalid or missing signature' ],
Expand Down
2 changes: 1 addition & 1 deletion tests/src/PaymentBrandsTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ public function test_transform( $pronamic_pay_method, $expected ) {
*
* @return array<array<string>>
*/
public function status_matrix_provider() {
public static function status_matrix_provider() {
return [
[ PaymentMethods::BANCONTACT, PaymentBrands::BANCONTACT ],
[ PaymentMethods::CREDIT_CARD, PaymentBrands::CARDS ],
Expand Down
2 changes: 1 addition & 1 deletion tests/src/StatusesTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ public function test_transform( $omnikassa_2_status, $expected ) {
*
* @return array<array<string|null>>
*/
public function status_matrix_provider() {
public static function status_matrix_provider() {
return [
[ Statuses::CANCELLED, PaymentStatus::CANCELLED ],
[ Statuses::COMPLETED, PaymentStatus::SUCCESS ],
Expand Down

0 comments on commit 8c90b86

Please sign in to comment.