Kydoimosandanthropic/claude-opus-4-8
e389e40843
CI / Coding Standards (pull_request) Successful in 28s
CI / Tests (PHP 8.1) (pull_request) Successful in 35s
CI / No Debug Code (pull_request) Successful in 9s
CI / Tests (PHP 8.3) (pull_request) Successful in 44s
CI / Tests (PHP 8.2) (pull_request) Successful in 47s
CI / Tests (PHP 8.5) (pull_request) Successful in 48s
CI / Static Analysis (pull_request) Successful in 52s
CI / Build Plugin Zip (pull_request) Skipped
The daily billing scan runs on request via WP-Cron and can overlap itself under concurrent traffic. Each run emailed the payments it created with no record that a notice had gone out, so two overlapping runs could send a payer two identical "Payment due" emails for one charge — read by families as being billed twice, though only one row exists. Stamp us_payments.notice_sent_at atomically before emailing: the scan now claims each payment with a conditional UPDATE ... WHERE notice_sent_at IS NULL and only notices, credits and batches the rows it won. A competing run finds them claimed and stays quiet, so exactly one notice is sent regardless of how the scan is triggered. A one-time backfill stamps existing scheduled rows on upgrade so already-noticed charges are not re-emailed. Co-authored-by: anthropic/claude-opus-4-8
491 lines
22 KiB
PHP
491 lines
22 KiB
PHP
<?php
|
|
declare(strict_types=1);
|
|
|
|
namespace Unsupervised\Schedular\Tests\Unit\Payment;
|
|
|
|
use Brain\Monkey\Functions;
|
|
use Mockery;
|
|
use Unsupervised\Schedular\Booking\BookingRepository;
|
|
use Unsupervised\Schedular\GroupClass\Enrollment;
|
|
use Unsupervised\Schedular\GroupClass\EnrollmentRepository;
|
|
use Unsupervised\Schedular\Guardian\GuardianService;
|
|
use Unsupervised\Schedular\Offering\Offering;
|
|
use Unsupervised\Schedular\Offering\OfferingRepository;
|
|
use Unsupervised\Schedular\Payment\Payment;
|
|
use Unsupervised\Schedular\Payment\PaymentDueMailer;
|
|
use Unsupervised\Schedular\Payment\PaymentService;
|
|
use Unsupervised\Schedular\Payment\ScheduledBillingRunner;
|
|
use Unsupervised\Schedular\Tests\Unit\TestCase;
|
|
|
|
class ScheduledBillingRunnerTest extends TestCase
|
|
{
|
|
private GuardianService&Mockery\MockInterface $guardians;
|
|
private PaymentService $payments;
|
|
private BookingRepository $bookings;
|
|
private EnrollmentRepository $enrollments;
|
|
private OfferingRepository $offerings;
|
|
private PaymentDueMailer $mailer;
|
|
private ScheduledBillingRunner $runner;
|
|
|
|
protected function setUp(): void
|
|
{
|
|
parent::setUp();
|
|
|
|
$this->payments = Mockery::mock(PaymentService::class);
|
|
$this->bookings = Mockery::mock(BookingRepository::class);
|
|
$this->enrollments = Mockery::mock(EnrollmentRepository::class);
|
|
$this->offerings = Mockery::mock(OfferingRepository::class);
|
|
$this->mailer = Mockery::mock(PaymentDueMailer::class);
|
|
|
|
// Defaults: nothing to bill unless a test says otherwise.
|
|
$this->bookings->shouldReceive('findUnbilledScheduledLessons')->andReturn([])->byDefault();
|
|
$this->enrollments->shouldReceive('findActiveByBillingModes')->andReturn([])->byDefault();
|
|
$this->mailer->shouldReceive('send')->andReturn(true)->byDefault();
|
|
$this->payments->shouldReceive('assignNoticeBatch')->byDefault();
|
|
// No account credit unless a test says otherwise.
|
|
$this->payments->shouldReceive('applyCredits')->andReturn([])->byDefault();
|
|
// Every payment wins its notice claim unless a test simulates an
|
|
// overlapping run that already claimed it.
|
|
$this->payments->shouldReceive('markNoticed')->andReturn(true)->byDefault();
|
|
|
|
Functions\when('wp_generate_uuid4')->justReturn('abcdef12-3456-7890-abcd-ef1234567890');
|
|
|
|
$student = Mockery::mock(\WP_User::class);
|
|
$student->user_email = '[email protected]';
|
|
Functions\when('get_userdata')->justReturn($student);
|
|
|
|
$this->guardians = Mockery::mock(GuardianService::class);
|
|
$this->guardians->shouldReceive('payerFor')->andReturnUsing(static fn (int $id): int => $id)->byDefault();
|
|
$this->guardians->shouldReceive('studentName')->andReturn('Ada')->byDefault();
|
|
|
|
$this->runner = new ScheduledBillingRunner(
|
|
$this->payments,
|
|
$this->bookings,
|
|
$this->enrollments,
|
|
$this->offerings,
|
|
$this->mailer,
|
|
$this->guardians
|
|
);
|
|
}
|
|
|
|
private function now(string $mysql): void
|
|
{
|
|
Functions\when('current_time')->justReturn($mysql);
|
|
}
|
|
|
|
private function pending(int $id, string $due): Payment
|
|
{
|
|
return new Payment(5, 3, Payment::REG_LESSON, 12, 35.00, currency: 'CAD', method: Payment::METHOD_ETRANSFER, status: Payment::STATUS_PENDING, dueDate: $due, id: $id);
|
|
}
|
|
|
|
private function lessonRow(int $id, string $mode, string $start, float $price, int $offeringId = 9): object
|
|
{
|
|
return (object) [
|
|
'id' => (string) $id,
|
|
'student_id' => '5',
|
|
'instructor_id' => '3',
|
|
'offering_id' => (string) $offeringId,
|
|
'start_dt' => $start,
|
|
'billing_mode' => $mode,
|
|
'title' => 'Piano',
|
|
'price' => (string) $price,
|
|
'currency' => 'CAD',
|
|
'etransfer_email' => '[email protected]',
|
|
];
|
|
}
|
|
|
|
public function testPrivateWeeklyBillsLessonWithin24h(): void
|
|
{
|
|
$this->now('2026-07-15 09:00:00');
|
|
$this->bookings->shouldReceive('findUnbilledScheduledLessons')
|
|
->andReturn([ $this->lessonRow(101, Offering::BILLING_WEEKLY, '2026-07-15 18:00:00', 35.0) ]);
|
|
|
|
$this->payments->shouldReceive('createForRegistration')
|
|
->once()
|
|
->with(Payment::REG_LESSON, 101, 5, 3, 35.0, 'CAD', '[email protected]', '2026-07-14', '2026-07-15', 5)
|
|
->andReturn($this->pending(500, '2026-07-14'));
|
|
|
|
$this->mailer->shouldReceive('send')->once();
|
|
|
|
$this->runner->run();
|
|
}
|
|
|
|
public function testPrivateWeeklySkipsLessonBeyond24h(): void
|
|
{
|
|
$this->now('2026-07-15 09:00:00');
|
|
$this->bookings->shouldReceive('findUnbilledScheduledLessons')
|
|
->andReturn([ $this->lessonRow(101, Offering::BILLING_WEEKLY, '2026-07-18 18:00:00', 35.0) ]);
|
|
|
|
$this->payments->shouldNotReceive('createForRegistration');
|
|
$this->mailer->shouldNotReceive('send');
|
|
|
|
$this->runner->run();
|
|
}
|
|
|
|
public function testPrivateMonthlyGroupsLessonsIntoOnePayment(): void
|
|
{
|
|
$this->now('2026-07-15 09:00:00');
|
|
$this->bookings->shouldReceive('findUnbilledScheduledLessons')->andReturn([
|
|
$this->lessonRow(201, Offering::BILLING_MONTHLY, '2026-07-07 18:00:00', 30.0),
|
|
$this->lessonRow(202, Offering::BILLING_MONTHLY, '2026-07-14 18:00:00', 30.0),
|
|
$this->lessonRow(203, Offering::BILLING_MONTHLY, '2026-07-21 18:00:00', 30.0),
|
|
]);
|
|
|
|
// One payment for the month: 3 x 30, due on the 1st, linked to the earliest.
|
|
$this->payments->shouldReceive('createForRegistration')
|
|
->once()
|
|
->with(Payment::REG_LESSON, 201, 5, 3, 90.0, 'CAD', '[email protected]', '2026-07-01', '2026-07', 5)
|
|
->andReturn($this->pending(600, '2026-07-01'));
|
|
|
|
// The other two lessons are pointed at the same payment so they are not re-billed.
|
|
$this->bookings->shouldReceive('setPaymentId')->once()->with(202, 600);
|
|
$this->bookings->shouldReceive('setPaymentId')->once()->with(203, 600);
|
|
|
|
$this->runner->run();
|
|
}
|
|
|
|
public function testPrivateMonthlySkipsFutureMonth(): void
|
|
{
|
|
$this->now('2026-07-15 09:00:00');
|
|
$this->bookings->shouldReceive('findUnbilledScheduledLessons')
|
|
->andReturn([ $this->lessonRow(301, Offering::BILLING_MONTHLY, '2026-08-04 18:00:00', 30.0) ]);
|
|
|
|
$this->payments->shouldNotReceive('createForRegistration');
|
|
|
|
$this->runner->run();
|
|
}
|
|
|
|
public function testGroupWeeklyBillsDueSessionsOnly(): void
|
|
{
|
|
$this->now('2026-07-15 09:00:00');
|
|
$enrollment = new Enrollment(offeringId: 9, studentId: 5, instructorId: 3, id: 44);
|
|
$this->enrollments->shouldReceive('findActiveByBillingModes')->andReturn([ $enrollment ]);
|
|
$this->offerings->shouldReceive('findById')->with(9)->andReturn($this->groupOffering(Offering::BILLING_WEEKLY, '2026-07-07', '2026-07-21'));
|
|
|
|
// Sessions Jul 7 (due Jul 6) and Jul 14 (due Jul 13) are due by Jul 15; Jul 21 is not.
|
|
$this->payments->shouldReceive('scheduledPaymentExists')->with(Payment::REG_ENROLLMENT, 44, '2026-07-07')->andReturn(false);
|
|
$this->payments->shouldReceive('scheduledPaymentExists')->with(Payment::REG_ENROLLMENT, 44, '2026-07-14')->andReturn(false);
|
|
|
|
$this->payments->shouldReceive('createForRegistration')
|
|
->once()
|
|
->with(Payment::REG_ENROLLMENT, 44, 5, 3, 20.0, 'CAD', null, '2026-07-06', '2026-07-07', 5)
|
|
->andReturn($this->pending(700, '2026-07-06'));
|
|
$this->payments->shouldReceive('createForRegistration')
|
|
->once()
|
|
->with(Payment::REG_ENROLLMENT, 44, 5, 3, 20.0, 'CAD', null, '2026-07-13', '2026-07-14', 5)
|
|
->andReturn($this->pending(701, '2026-07-13'));
|
|
|
|
$this->runner->run();
|
|
}
|
|
|
|
public function testGroupWeeklyDedupSkipsExistingPeriod(): void
|
|
{
|
|
$this->now('2026-07-15 09:00:00');
|
|
$enrollment = new Enrollment(offeringId: 9, studentId: 5, instructorId: 3, id: 44);
|
|
$this->enrollments->shouldReceive('findActiveByBillingModes')->andReturn([ $enrollment ]);
|
|
$this->offerings->shouldReceive('findById')->with(9)->andReturn($this->groupOffering(Offering::BILLING_WEEKLY, '2026-07-07', '2026-07-21'));
|
|
|
|
// First session already billed; only the second generates a payment.
|
|
$this->payments->shouldReceive('scheduledPaymentExists')->with(Payment::REG_ENROLLMENT, 44, '2026-07-07')->andReturn(true);
|
|
$this->payments->shouldReceive('scheduledPaymentExists')->with(Payment::REG_ENROLLMENT, 44, '2026-07-14')->andReturn(false);
|
|
|
|
$this->payments->shouldReceive('createForRegistration')
|
|
->once()
|
|
->with(Payment::REG_ENROLLMENT, 44, 5, 3, 20.0, 'CAD', null, '2026-07-13', '2026-07-14', 5)
|
|
->andReturn($this->pending(701, '2026-07-13'));
|
|
|
|
$this->runner->run();
|
|
}
|
|
|
|
/**
|
|
* A monthly group class is priced per month, not per session: the same fee
|
|
* is charged whether the class meets four times in the month or once. This
|
|
* is what the class card quotes and what the student agrees to pay.
|
|
*/
|
|
public function testGroupMonthlyBillsTheMonthlyFeeOnceHoweverManySessions(): void
|
|
{
|
|
$this->now('2026-07-15 09:00:00');
|
|
$enrollment = new Enrollment(offeringId: 9, studentId: 5, instructorId: 3, id: 44);
|
|
$this->enrollments->shouldReceive('findActiveByBillingModes')->andReturn([ $enrollment ]);
|
|
// 4 Tuesday sessions in July.
|
|
$this->offerings->shouldReceive('findById')->with(9)->andReturn($this->groupOffering(Offering::BILLING_MONTHLY, '2026-07-07', '2026-07-28'));
|
|
|
|
$this->payments->shouldReceive('scheduledPaymentExists')->with(Payment::REG_ENROLLMENT, 44, '2026-07')->andReturn(false);
|
|
|
|
// One payment of the monthly fee — not 4 x 20 — due on the 1st.
|
|
$this->payments->shouldReceive('createForRegistration')
|
|
->once()
|
|
->with(Payment::REG_ENROLLMENT, 44, 5, 3, 20.0, 'CAD', null, '2026-07-01', '2026-07', 5)
|
|
->andReturn($this->pending(800, '2026-07-01'));
|
|
|
|
$this->runner->run();
|
|
}
|
|
|
|
/**
|
|
* The per-month fee does not shrink for a short month either — a month with
|
|
* a single session is billed the same as a month with four.
|
|
*/
|
|
public function testGroupMonthlyBillsTheSameFeeForAMonthWithOneSession(): void
|
|
{
|
|
$this->now('2026-07-15 09:00:00');
|
|
$enrollment = new Enrollment(offeringId: 9, studentId: 5, instructorId: 3, id: 44);
|
|
$this->enrollments->shouldReceive('findActiveByBillingModes')->andReturn([ $enrollment ]);
|
|
// A single July session.
|
|
$this->offerings->shouldReceive('findById')->with(9)->andReturn($this->groupOffering(Offering::BILLING_MONTHLY, '2026-07-07', '2026-07-07'));
|
|
|
|
$this->payments->shouldReceive('scheduledPaymentExists')->with(Payment::REG_ENROLLMENT, 44, '2026-07')->andReturn(false);
|
|
|
|
$this->payments->shouldReceive('createForRegistration')
|
|
->once()
|
|
->with(Payment::REG_ENROLLMENT, 44, 5, 3, 20.0, 'CAD', null, '2026-07-01', '2026-07', 5)
|
|
->andReturn($this->pending(800, '2026-07-01'));
|
|
|
|
$this->runner->run();
|
|
}
|
|
|
|
public function testCompPaymentIsNotBucketed(): void
|
|
{
|
|
$this->now('2026-07-15 09:00:00');
|
|
$this->bookings->shouldReceive('findUnbilledScheduledLessons')
|
|
->andReturn([ $this->lessonRow(101, Offering::BILLING_WEEKLY, '2026-07-15 18:00:00', 35.0) ]);
|
|
|
|
// A comp student's payment comes back paid — no due notice should be sent.
|
|
$comp = new Payment(5, 3, Payment::REG_LESSON, 12, 35.00, currency: 'CAD', method: Payment::METHOD_COMP, status: Payment::STATUS_PAID, dueDate: '2026-07-14', id: 900);
|
|
$this->payments->shouldReceive('createForRegistration')->once()->andReturn($comp);
|
|
|
|
$this->mailer->shouldNotReceive('send');
|
|
|
|
$this->runner->run();
|
|
}
|
|
|
|
public function testConsolidatesAllItemsIntoOneEmailPerStudent(): void
|
|
{
|
|
$this->now('2026-07-15 09:00:00');
|
|
$this->bookings->shouldReceive('findUnbilledScheduledLessons')
|
|
->andReturn([ $this->lessonRow(101, Offering::BILLING_WEEKLY, '2026-07-15 18:00:00', 35.0) ]);
|
|
$enrollment = new Enrollment(offeringId: 9, studentId: 5, instructorId: 3, id: 44);
|
|
$this->enrollments->shouldReceive('findActiveByBillingModes')->andReturn([ $enrollment ]);
|
|
$this->offerings->shouldReceive('findById')->with(9)->andReturn($this->groupOffering(Offering::BILLING_WEEKLY, '2026-07-14', '2026-07-14'));
|
|
|
|
$this->payments->shouldReceive('scheduledPaymentExists')->andReturn(false);
|
|
$this->payments->shouldReceive('createForRegistration')->andReturn($this->pending(500, '2026-07-14'), $this->pending(501, '2026-07-13'));
|
|
|
|
// Same student billed twice in one run -> exactly one email with both items,
|
|
// and both payments tagged with one shared notice-batch reference.
|
|
$this->payments->shouldReceive('assignNoticeBatch')
|
|
->once()
|
|
->with(Mockery::on(static fn (array $ids): bool => count($ids) === 2), Mockery::type('string'));
|
|
$this->mailer->shouldReceive('send')
|
|
->once()
|
|
->with(Mockery::type(\WP_User::class), Mockery::on(static fn (array $items): bool => count($items) === 2), Mockery::type('string'), 0.0);
|
|
|
|
$this->runner->run();
|
|
}
|
|
|
|
public function testAppliesAccountCreditToTheRun(): void
|
|
{
|
|
$this->now('2026-07-15 09:00:00');
|
|
$this->bookings->shouldReceive('findUnbilledScheduledLessons')
|
|
->andReturn([ $this->lessonRow(101, Offering::BILLING_WEEKLY, '2026-07-15 18:00:00', 35.0) ]);
|
|
|
|
$payment = $this->pending(500, '2026-07-14');
|
|
$this->payments->shouldReceive('createForRegistration')->once()->andReturn($payment);
|
|
|
|
// Student holds $20 credit, applied to the one $35 charge — still $15 owing,
|
|
// so the payment stays in the notice batch and the notice quotes the credit.
|
|
$this->payments->shouldReceive('applyCredits')
|
|
->once()
|
|
->with(5, Mockery::on(static fn (array $p): bool => count($p) === 1))
|
|
->andReturn([500 => 20.0]);
|
|
$this->payments->shouldReceive('assignNoticeBatch')
|
|
->once()
|
|
->with([500], Mockery::type('string'));
|
|
$this->mailer->shouldReceive('send')
|
|
->once()
|
|
->with(Mockery::type(\WP_User::class), Mockery::type('array'), Mockery::type('string'), 20.0);
|
|
|
|
$this->runner->run();
|
|
}
|
|
|
|
public function testCreditFullyCoveringAChargeLeavesItOutOfTheBatch(): void
|
|
{
|
|
$this->now('2026-07-15 09:00:00');
|
|
$this->bookings->shouldReceive('findUnbilledScheduledLessons')
|
|
->andReturn([ $this->lessonRow(101, Offering::BILLING_WEEKLY, '2026-07-15 18:00:00', 35.0) ]);
|
|
|
|
$payment = $this->pending(500, '2026-07-14');
|
|
$this->payments->shouldReceive('createForRegistration')->once()->andReturn($payment);
|
|
|
|
// Credit covers the whole $35 charge: nothing owing, so no reconciliation
|
|
// batch and no reference on the (zero-balance) notice.
|
|
$this->payments->shouldReceive('applyCredits')->once()->andReturn([500 => 35.0]);
|
|
$this->payments->shouldReceive('assignNoticeBatch')->once()->with([], '');
|
|
$this->mailer->shouldReceive('send')
|
|
->once()
|
|
->with(Mockery::type(\WP_User::class), Mockery::type('array'), '', 35.0);
|
|
|
|
$this->runner->run();
|
|
}
|
|
|
|
/**
|
|
* A payment is emailed exactly once. WP-Cron fires on request and can run the
|
|
* scan twice concurrently; the second run reaching the send step for a
|
|
* payment already claimed by the first (markNoticed returns false) must not
|
|
* email it again, nor re-batch it. This is the double-notice regression.
|
|
*/
|
|
public function testDoesNotEmailAPaymentWhoseNoticeWasAlreadyClaimed(): void
|
|
{
|
|
$this->now('2026-07-15 09:00:00');
|
|
$this->bookings->shouldReceive('findUnbilledScheduledLessons')
|
|
->andReturn([ $this->lessonRow(101, Offering::BILLING_WEEKLY, '2026-07-15 18:00:00', 35.0) ]);
|
|
|
|
$this->payments->shouldReceive('createForRegistration')->once()->andReturn($this->pending(500, '2026-07-14'));
|
|
|
|
// The competing run already stamped notice_sent_at, so the claim loses.
|
|
$this->payments->shouldReceive('markNoticed')->with(500)->once()->andReturn(false);
|
|
|
|
// No credit, no batch, no email for an already-noticed payment.
|
|
$this->payments->shouldNotReceive('applyCredits');
|
|
$this->payments->shouldNotReceive('assignNoticeBatch');
|
|
$this->mailer->shouldNotReceive('send');
|
|
|
|
$this->runner->run();
|
|
}
|
|
|
|
/**
|
|
* When only some of a payer's charges are already claimed, the run notices
|
|
* the rest — the still-unclaimed payment is emailed and batched on its own.
|
|
*/
|
|
public function testEmailsOnlyTheStillUnclaimedPaymentsInABucket(): void
|
|
{
|
|
$this->now('2026-07-15 09:00:00');
|
|
|
|
$one = $this->lessonRow(101, Offering::BILLING_WEEKLY, '2026-07-15 18:00:00', 35.0);
|
|
$two = $this->lessonRow(102, Offering::BILLING_WEEKLY, '2026-07-15 19:00:00', 35.0);
|
|
$this->bookings->shouldReceive('findUnbilledScheduledLessons')->andReturn([$one, $two]);
|
|
|
|
$this->payments->shouldReceive('createForRegistration')
|
|
->andReturn($this->pending(500, '2026-07-14'), $this->pending(501, '2026-07-14'));
|
|
|
|
// 500 already claimed by an overlapping run; 501 is this run's to send.
|
|
$this->payments->shouldReceive('markNoticed')->with(500)->once()->andReturn(false);
|
|
$this->payments->shouldReceive('markNoticed')->with(501)->once()->andReturn(true);
|
|
|
|
$this->payments->shouldReceive('applyCredits')
|
|
->once()
|
|
->with(5, Mockery::on(static fn (array $p): bool => count($p) === 1 && (int) $p[0]->id === 501))
|
|
->andReturn([]);
|
|
$this->payments->shouldReceive('assignNoticeBatch')->once()->with([501], Mockery::type('string'));
|
|
$this->mailer->shouldReceive('send')
|
|
->once()
|
|
->with(Mockery::type(\WP_User::class), Mockery::on(static fn (array $items): bool => count($items) === 1), Mockery::type('string'), 0.0);
|
|
|
|
$this->runner->run();
|
|
}
|
|
|
|
private function groupOffering(string $mode, string $termStart, string $termEnd): Offering
|
|
{
|
|
return new Offering(
|
|
instructorId: 3,
|
|
kind: Offering::KIND_GROUP_CLASS,
|
|
title: 'Ensemble',
|
|
price: 20.0,
|
|
currency: 'CAD',
|
|
billingMode: $mode,
|
|
durationMinutes: 60,
|
|
termStart: $termStart,
|
|
termEnd: $termEnd,
|
|
classTime: '16:00:00',
|
|
id: 9,
|
|
);
|
|
}
|
|
|
|
/**
|
|
* Two children billed on the same day belong to one payer, so the guardian
|
|
* gets a single notice covering both — not one email per child — and each
|
|
* line names whose lesson it is.
|
|
*/
|
|
public function testAGuardiansChildrenShareOneNoticeWithNamedLines(): void
|
|
{
|
|
$this->now('2026-07-15 09:00:00');
|
|
|
|
$adas = $this->lessonRow(101, Offering::BILLING_WEEKLY, '2026-07-15 18:00:00', 35.0);
|
|
$alans = $this->lessonRow(102, Offering::BILLING_WEEKLY, '2026-07-15 19:00:00', 35.0);
|
|
$adas->student_id = '42';
|
|
$alans->student_id = '43';
|
|
|
|
$this->bookings->shouldReceive('findUnbilledScheduledLessons')->andReturn([$adas, $alans]);
|
|
|
|
$this->guardians->shouldReceive('payerFor')->with(42)->andReturn(5);
|
|
$this->guardians->shouldReceive('payerFor')->with(43)->andReturn(5);
|
|
$this->guardians->shouldReceive('studentName')->with(42)->andReturn('Ada');
|
|
$this->guardians->shouldReceive('studentName')->with(43)->andReturn('Alan');
|
|
|
|
$this->payments->shouldReceive('createForRegistration')
|
|
->andReturn($this->pending(500, '2026-07-14'), $this->pending(501, '2026-07-14'));
|
|
|
|
// One send, two lines, each prefixed with the child it is for.
|
|
$this->mailer->shouldReceive('send')
|
|
->once()
|
|
->with(
|
|
Mockery::type(\WP_User::class),
|
|
Mockery::on(static function (array $items): bool {
|
|
return count($items) === 2
|
|
&& str_starts_with((string) $items[0]['label'], 'Ada: ')
|
|
&& str_starts_with((string) $items[1]['label'], 'Alan: ');
|
|
}),
|
|
Mockery::type('string'),
|
|
0.0
|
|
);
|
|
|
|
$this->runner->run();
|
|
}
|
|
|
|
/**
|
|
* A student who pays for themselves gets the plain label — prefixing every
|
|
* line with their own name would be noise.
|
|
*/
|
|
public function testAStudentPayingForThemselvesGetsAnUnprefixedLabel(): void
|
|
{
|
|
$this->now('2026-07-15 09:00:00');
|
|
$this->bookings->shouldReceive('findUnbilledScheduledLessons')
|
|
->andReturn([$this->lessonRow(101, Offering::BILLING_WEEKLY, '2026-07-15 18:00:00', 35.0)]);
|
|
|
|
$this->payments->shouldReceive('createForRegistration')->andReturn($this->pending(500, '2026-07-14'));
|
|
|
|
$this->mailer->shouldReceive('send')
|
|
->once()
|
|
->with(
|
|
Mockery::type(\WP_User::class),
|
|
Mockery::on(static fn (array $items): bool => str_starts_with((string) $items[0]['label'], 'Piano — ')),
|
|
Mockery::type('string'),
|
|
0.0
|
|
);
|
|
|
|
$this->runner->run();
|
|
}
|
|
|
|
/**
|
|
* The family balance is drawn against the payer, so a credit one child
|
|
* earned can settle a sibling's charge.
|
|
*/
|
|
public function testCreditsAreAppliedAgainstThePayerNotEachStudent(): void
|
|
{
|
|
$this->now('2026-07-15 09:00:00');
|
|
|
|
$adas = $this->lessonRow(101, Offering::BILLING_WEEKLY, '2026-07-15 18:00:00', 35.0);
|
|
$adas->student_id = '42';
|
|
|
|
$this->bookings->shouldReceive('findUnbilledScheduledLessons')->andReturn([$adas]);
|
|
$this->guardians->shouldReceive('payerFor')->with(42)->andReturn(5);
|
|
$this->guardians->shouldReceive('studentName')->with(42)->andReturn('Ada');
|
|
|
|
$this->payments->shouldReceive('createForRegistration')->andReturn($this->pending(500, '2026-07-14'));
|
|
$this->payments->shouldReceive('applyCredits')->once()->with(5, Mockery::type('array'))->andReturn([]);
|
|
|
|
$this->mailer->shouldReceive('send')->once();
|
|
|
|
$this->runner->run();
|
|
}
|
|
}
|