공개 (답변 중복 등록)/(문의 삭제 시 답변 잔존) 수정과 동형 결함 전수 해소.
- 문의 답변 단일성: 1차(피벗 is_answered) + 2차(게시판 리스너 duplicate 마커) 이중 방어,
경합 경로에서도 사유(reply_already_exists 422)가 보존되도록 훅 반환 계약 확장
- 질문·답변·피벗 생명주기 대칭: 삭제/복원/영구삭제 훅 연동 + 고아 데이터 백필 업그레이드 스텝
- 게시판별 답글 삭제 정책(cascade/block) 신설, 정렬 화이트리스트·per_page 게이트
(전량 조회 소비처 200/999 를 수용하는 상한 999 — 조였다가 관리자 화면 2곳 422 회귀 즉시 정정)
- 결제/본인인증 플러그인 검증 계약: FormRequest 15종 승격 + 컨트롤러 catch 정합.
거래 조회 tid 상한은 저장 컬럼(varchar 100) 기준 — 해외 CBT TID(40자) 수용
- onError 토스트 고정 문구 전수 해소(67건/50파일/15확장) — 서버 실패 사유를 그대로 표면화,
change-password 오변수(response→error) 교정
- 코어 플러그인 설정 검증 안내의 항목 표시명이 스키마 라벨(ko/en) 폴백에 덮여 요청 로케일
(ja) 라벨을 잃던 회귀 정정 — 번역 등록이 있으면 그것을 우선 (7.0.7 미출시 기능 자체 결함)
- 사전 결함 정리(Icon 등가표 27·darkmode 2·validated 통일), API 문서·CHANGELOG·ja 팩 동기
- develop(60652fdc0) 리베이스: getTable 정비와 병합 — 양측 의도 보존 (충돌 18파일)
130 lines
5.1 KiB
PHP
130 lines
5.1 KiB
PHP
<?php
|
|
|
|
declare(strict_types=1);
|
|
|
|
namespace Plugins\Sirsoft\PayNicepayments\Tests\Feature\Controllers;
|
|
|
|
use Modules\Sirsoft\Ecommerce\Database\Factories\OrderFactory;
|
|
use Modules\Sirsoft\Ecommerce\Database\Factories\OrderPaymentFactory;
|
|
use Modules\Sirsoft\Ecommerce\Enums\OrderStatusEnum;
|
|
use Modules\Sirsoft\Ecommerce\Enums\PaymentMethodEnum;
|
|
use Modules\Sirsoft\Ecommerce\Enums\PaymentStatusEnum;
|
|
use Plugins\Sirsoft\PayNicepayments\Services\NicePaymentsApiService;
|
|
use Plugins\Sirsoft\PayNicepayments\Tests\PluginTestCase;
|
|
|
|
class AdminEscrowControllerTest extends PluginTestCase
|
|
{
|
|
private const TID = 'NICE_ESCROW_TID_001';
|
|
|
|
public function test_register_delivery_requires_bound_escrow_payment_and_restores_credentials(): void
|
|
{
|
|
$admin = $this->createAdminUser(['sirsoft-ecommerce.orders.update']);
|
|
$this->createPaidEscrowPayment();
|
|
|
|
$apiServiceMock = $this->createMock(NicePaymentsApiService::class);
|
|
$apiServiceMock->expects($this->once())
|
|
->method('useStoredCredentials')
|
|
->with(false, 'SRLIVE001');
|
|
$apiServiceMock->expects($this->once())
|
|
->method('registerEscrowDelivery')
|
|
->with(
|
|
self::TID,
|
|
'CJ대한통운',
|
|
'TRACK123456',
|
|
'서울시 테스트구 1',
|
|
'운영자',
|
|
)
|
|
->willReturn([
|
|
'ResultCode' => 'C000',
|
|
'ResultMsg' => '정상처리',
|
|
]);
|
|
$this->app->instance(NicePaymentsApiService::class, $apiServiceMock);
|
|
|
|
$response = $this->actingAs($admin)->postJson('/api/plugins/sirsoft-pay_nicepayments/admin/escrow/register-delivery', [
|
|
'tid' => self::TID,
|
|
'delivery_name' => 'CJ대한통운',
|
|
'tracking_number' => 'TRACK123456',
|
|
'buyer_address' => '서울시 테스트구 1',
|
|
'register_name' => '운영자',
|
|
]);
|
|
|
|
$response->assertOk()
|
|
->assertJsonPath('success', true)
|
|
->assertJsonPath('data.ResultCode', 'C000');
|
|
}
|
|
|
|
public function test_register_delivery_rejects_unbound_tid_before_api_call(): void
|
|
{
|
|
$admin = $this->createAdminUser(['sirsoft-ecommerce.orders.update']);
|
|
$this->createPaidEscrowPayment();
|
|
|
|
$apiServiceMock = $this->createMock(NicePaymentsApiService::class);
|
|
$apiServiceMock->expects($this->never())->method('useStoredCredentials');
|
|
$apiServiceMock->expects($this->never())->method('registerEscrowDelivery');
|
|
$this->app->instance(NicePaymentsApiService::class, $apiServiceMock);
|
|
|
|
$response = $this->actingAs($admin)->postJson('/api/plugins/sirsoft-pay_nicepayments/admin/escrow/register-delivery', [
|
|
'tid' => 'ATTACKER_SUPPLIED_TID',
|
|
'delivery_name' => 'CJ대한통운',
|
|
'tracking_number' => 'TRACK123456',
|
|
'buyer_address' => '서울시 테스트구 1',
|
|
'register_name' => '운영자',
|
|
]);
|
|
|
|
$response->assertStatus(422)
|
|
->assertJsonPath('success', false);
|
|
}
|
|
|
|
/**
|
|
* TID 상한(30자) 초과는 EscrowDeliveryRegisterRequest 가 표준 422 로 차단하고
|
|
* PG 배송등록 API 는 호출되지 않아야 한다 (나이스페이먼츠 TID 는 30자).
|
|
*/
|
|
public function test_register_delivery_rejects_overlong_tid(): void
|
|
{
|
|
$admin = $this->createAdminUser(['sirsoft-ecommerce.orders.update']);
|
|
$this->createPaidEscrowPayment();
|
|
|
|
$apiServiceMock = $this->createMock(NicePaymentsApiService::class);
|
|
$apiServiceMock->expects($this->never())->method('useStoredCredentials');
|
|
$apiServiceMock->expects($this->never())->method('registerEscrowDelivery');
|
|
$this->app->instance(NicePaymentsApiService::class, $apiServiceMock);
|
|
|
|
$response = $this->actingAs($admin)->postJson('/api/plugins/sirsoft-pay_nicepayments/admin/escrow/register-delivery', [
|
|
'tid' => str_repeat('T', 31),
|
|
'delivery_name' => 'CJ대한통운',
|
|
'tracking_number' => 'TRACK123456',
|
|
'buyer_address' => '서울시 테스트구 1',
|
|
'register_name' => '운영자',
|
|
]);
|
|
|
|
$response->assertStatus(422)
|
|
->assertJsonValidationErrors(['tid']);
|
|
}
|
|
|
|
private function createPaidEscrowPayment(): void
|
|
{
|
|
$order = OrderFactory::new()->create([
|
|
'order_status' => OrderStatusEnum::PAYMENT_COMPLETE,
|
|
'subtotal_amount' => 30000,
|
|
'total_amount' => 30000,
|
|
'total_due_amount' => 0,
|
|
'total_paid_amount' => 30000,
|
|
'paid_at' => now(),
|
|
]);
|
|
|
|
OrderPaymentFactory::new()->forOrder($order)->create([
|
|
'payment_status' => PaymentStatusEnum::PAID,
|
|
'payment_method' => PaymentMethodEnum::CARD,
|
|
'pg_provider' => 'nicepayments',
|
|
'is_escrow' => true,
|
|
'paid_amount_local' => 30000,
|
|
'paid_at' => now(),
|
|
'transaction_id' => self::TID,
|
|
'payment_meta' => [
|
|
'mid' => 'SRLIVE001',
|
|
'is_test_mode' => false,
|
|
],
|
|
]);
|
|
}
|
|
}
|