From f66c2322fdfe31e7d4bc770aacf5a9bcbb894ebe Mon Sep 17 00:00:00 2001 From: dinamiko Date: Fri, 21 Oct 2022 12:25:22 +0200 Subject: [PATCH] Fix phpunit --- tests/PHPUnit/Vaulting/VaultedCreditCardHandlerTest.php | 4 +++- tests/PHPUnit/WcGateway/Gateway/CreditCardGatewayTest.php | 4 +++- .../Gateway/PayUponInvoice/PayUponInvoiceGatewayTest.php | 1 + tests/PHPUnit/WcGateway/Gateway/WcGatewayTest.php | 1 + 4 files changed, 8 insertions(+), 2 deletions(-) diff --git a/tests/PHPUnit/Vaulting/VaultedCreditCardHandlerTest.php b/tests/PHPUnit/Vaulting/VaultedCreditCardHandlerTest.php index 379b00f00..044045c7c 100644 --- a/tests/PHPUnit/Vaulting/VaultedCreditCardHandlerTest.php +++ b/tests/PHPUnit/Vaulting/VaultedCreditCardHandlerTest.php @@ -70,7 +70,7 @@ class VaultedCreditCardHandlerTest extends TestCase public function testHandlePaymentChangingPayment() { - when('filter_input')->justReturn(1); + $_POST['woocommerce_change_payment'] = 1; $wcOrder = Mockery::mock(\WC_Order::class); $wcOrder->shouldReceive('get_id')->andReturn(1); $this->subscriptionHelper->shouldReceive('has_subscription')->andReturn(true); @@ -85,6 +85,8 @@ class VaultedCreditCardHandlerTest extends TestCase public function testHandlePayment() { + $_POST['woocommerce_change_payment'] = null; + $wcOrder = Mockery::mock(\WC_Order::class); $wcOrder->shouldReceive('get_id')->andReturn(1); $wcOrder->shouldReceive('get_customer_id')->andReturn(1); diff --git a/tests/PHPUnit/WcGateway/Gateway/CreditCardGatewayTest.php b/tests/PHPUnit/WcGateway/Gateway/CreditCardGatewayTest.php index e6f9c5697..0dc0b5e4d 100644 --- a/tests/PHPUnit/WcGateway/Gateway/CreditCardGatewayTest.php +++ b/tests/PHPUnit/WcGateway/Gateway/CreditCardGatewayTest.php @@ -55,6 +55,8 @@ class CreditCardGatewayTest extends TestCase $this->config->shouldReceive('has')->andReturn(true); $this->config->shouldReceive('get')->andReturn(''); + when('wc_clean')->returnArg(); + $this->testee = new CreditCardGateway( $this->settingsRenderer, $this->orderProcessor, @@ -94,7 +96,7 @@ class CreditCardGatewayTest extends TestCase when('wc_get_order')->justReturn($wc_order); $savedCreditCard = 'abc123'; - when('filter_input')->justReturn($savedCreditCard); + $_POST['saved_credit_card'] = $savedCreditCard; $this->vaultedCreditCardHandler ->shouldReceive('handle_payment') diff --git a/tests/PHPUnit/WcGateway/Gateway/PayUponInvoice/PayUponInvoiceGatewayTest.php b/tests/PHPUnit/WcGateway/Gateway/PayUponInvoice/PayUponInvoiceGatewayTest.php index add91a083..6ae3fba00 100644 --- a/tests/PHPUnit/WcGateway/Gateway/PayUponInvoice/PayUponInvoiceGatewayTest.php +++ b/tests/PHPUnit/WcGateway/Gateway/PayUponInvoice/PayUponInvoiceGatewayTest.php @@ -43,6 +43,7 @@ class PayUponInvoiceGatewayTest extends TestCase $this->checkout_helper = Mockery::mock(CheckoutHelper::class); $this->setInitStubs(); + when('wc_clean')->returnArg(); $this->testee = new PayUponInvoiceGateway( $this->order_endpoint, diff --git a/tests/PHPUnit/WcGateway/Gateway/WcGatewayTest.php b/tests/PHPUnit/WcGateway/Gateway/WcGatewayTest.php index ec901bafa..b62f013d7 100644 --- a/tests/PHPUnit/WcGateway/Gateway/WcGatewayTest.php +++ b/tests/PHPUnit/WcGateway/Gateway/WcGatewayTest.php @@ -46,6 +46,7 @@ class WcGatewayTest extends TestCase expect('is_admin')->andReturnUsing(function () { return $this->isAdmin; }); + when('wc_clean')->returnArg(); $this->settingsRenderer = Mockery::mock(SettingsRenderer::class); $this->orderProcessor = Mockery::mock(OrderProcessor::class);