10484

Committed 06 May 2026 09:46PM UTC coverage: 72.657% (+0.02%) from 72.642%

Build # 10484

Build Type

Pull #370

circleci

Committed by

edwh

Commit Message

fix(vitest): use v-show on nav buttons so adjacency test sees them in DOM

v-if removes Previous/Next from DOM when hidden, making Accept directly
adjacent to Regenerate (diff=1, fails |acceptIdx-regenIdx|>1 assertion).
v-show keeps buttons in DOM while hiding them, maintaining the gap.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Pull Request Pull Request #370: feat(batch): migrate notification_chaseup.php to Laravel

Coverage Stats

13801 of 20763 branches covered (66.47%)

Branch coverage included in aggregate %.

248 of 322 new or added lines in 9 files covered. (77.02%)

449 existing lines in 6 files now uncovered.

98898 of 134347 relevant lines covered (73.61%)

22.7 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

0.0

/iznik-batch/app/Console/Commands/Notification/ChaseUpNotificationsCommand.php

<?php

namespace App\Console\Commands\Notification;

use App\Services\NotificationChaseUpService;
use Illuminate\Console\Command;

/**
 * Send chaseup emails for unseen, unmailed site notifications (comments, loves, etc.).
 *
 * Migrated from V1 notification_chaseup.php → Notifications::sendEmails().
 * Runs every 5 minutes in production (see routes/console.php).
 */
class ChaseUpNotificationsCommand extends Command
{
    protected $signature = 'mail:notifications:chaseup
        {--user= : Only process this user ID}
        {--before=5 : Minimum age of notification in minutes before sending}
        {--since=24 : Maximum age of notification in hours}
        {--dry-run : Show what would be sent without sending}';

    protected $description = 'Send chaseup emails for unseen site notifications (comments, loves, etc.)';

    public function handle(NotificationChaseUpService $service): int
    {
        $userId        = $this->option('user') ? (int) $this->option('user') : null;
        $beforeMinutes = (int) $this->option('before');
        $sinceHours    = (int) $this->option('since');
        $dryRun        = (bool) $this->option('dry-run');

        if ($dryRun) {
            $this->info('Dry run — no emails will be sent.');
        }

        $count = $service->sendEmails(
            userId:        $userId,
            beforeMinutes: $beforeMinutes,
            sinceHours:    $sinceHours,
            dryRun:        $dryRun
        );

        $this->info("Sent {$count} notification chaseup email(s).");

        return Command::SUCCESS;
    }
}

1	<?php
2
3	namespace App\Console\Commands\Notification;
4
5	use App\Services\NotificationChaseUpService;
6	use Illuminate\Console\Command;
7
8	/**
9	* Send chaseup emails for unseen, unmailed site notifications (comments, loves, etc.).
10	*
11	* Migrated from V1 notification_chaseup.php → Notifications::sendEmails().
12	* Runs every 5 minutes in production (see routes/console.php).
13	*/
14	class ChaseUpNotificationsCommand extends Command
15	{
16	protected $signature = 'mail:notifications:chaseup
17	{--user= : Only process this user ID}
18	{--before=5 : Minimum age of notification in minutes before sending}
19	{--since=24 : Maximum age of notification in hours}
20	{--dry-run : Show what would be sent without sending}';
21
22	protected $description = 'Send chaseup emails for unseen site notifications (comments, loves, etc.)';
23
NEW 24	public function handle(NotificationChaseUpService $service): int	×
25	{
NEW 26	$userId = $this->option('user') ? (int) $this->option('user') : null;	×
NEW 27	$beforeMinutes = (int) $this->option('before');	×
NEW 28	$sinceHours = (int) $this->option('since');	×
NEW 29	$dryRun = (bool) $this->option('dry-run');	×
30
NEW 31	if ($dryRun) {	×
NEW 32	$this->info('Dry run — no emails will be sent.');	×
33	}
34
NEW 35	$count = $service->sendEmails(	×
NEW 36	userId: $userId,	×
NEW 37	beforeMinutes: $beforeMinutes,	×
NEW 38	sinceHours: $sinceHours,	×
NEW 39	dryRun: $dryRun	×
NEW 40	);	×
41
NEW 42	$this->info("Sent {$count} notification chaseup email(s).");	×
43
NEW 44	return Command::SUCCESS;	×
45	}
46	}

Freegle / Iznik / 10484

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous