archtechx/tenancy
1
0
Fork 0
mirror of https://github.com/archtechx/tenancy.git synced 2026-05-06 17:04:04 +00:00
Code Activity

Ensure Slack throws cURL exception in test

Browse source
This commit is contained in:
lukinovec 2026-04-13 14:10:05 +02:00
parent 89b0d1cb4b
commit 9660faf2f9
1 changed files with 20 additions and 18 deletions
Download patch file Download diff file Expand all files Collapse all files

38
tests/Bootstrappers/LogTenancyBootstrapperTest.php
View file

@ -350,6 +350,22 @@ test('slack channel uses correct webhook urls', function () {
'logging.channels.slack.level' => 'debug', // Set level to debug to keep the tests simple, since the default level here is 'critical' 'logging.channels.slack.level' => 'debug', // Set level to debug to keep the tests simple, since the default level here is 'critical'
]); ]);
$assertWebhook = function (string $expectedWebhook, string $message): void {
$thrown = false;
// Because the Slack channel uses cURL to send messages, we cannot use Http::fake() here.
// Instead, we catch the exception and check the error message which contains the actual webhook URL
// (logging always throws "Curl error (code 6): Could not resolve host: {webhook_url}").
try {
Log::channel('slack')->info($message);
} catch (Exception $e) {
$thrown = true;
expect($e->getMessage())->toContain($expectedWebhook);
}
expect($thrown)->toBeTrue();
};
$tenant1 = Tenant::create(['id' => 'tenant1', 'slackUrl' => 'tenant1-webhook']); $tenant1 = Tenant::create(['id' => 'tenant1', 'slackUrl' => 'tenant1-webhook']);
$tenant2 = Tenant::create(['id' => 'tenant2', 'slackUrl' => 'tenant2-webhook']); $tenant2 = Tenant::create(['id' => 'tenant2', 'slackUrl' => 'tenant2-webhook']);
@ -358,27 +374,13 @@ test('slack channel uses correct webhook urls', function () {
]; ];
// Test central context - should attempt to use central webhook // Test central context - should attempt to use central webhook
// Because the Slack channel uses cURL to send messages, we cannot use Http::fake() here. $assertWebhook('central-webhook', 'central');
// Instead, we catch the exception and check the error message which contains the actual webhook URL.
try {
Log::channel('slack')->info('central');
} catch (Exception $e) {
expect($e->getMessage())->toContain('central-webhook');
}
// Slack channel should attempt to use the tenant-specific webhooks // Slack channel should attempt to use the tenant-specific webhooks
tenancy()->runForMultiple([$tenant1, $tenant2], function (Tenant $tenant) { tenancy()->runForMultiple([$tenant1, $tenant2], function (Tenant $tenant) use ($assertWebhook) {
try { $assertWebhook($tenant->slackUrl, $tenant->id);
Log::channel('slack')->info($tenant->id);
} catch (Exception $e) {
expect($e->getMessage())->toContain($tenant->slackUrl);
}
}); });
// Central context, central webhook should be used again // Central context, central webhook should be used again
try { $assertWebhook('central-webhook', 'central');
Log::channel('slack')->info('central');
} catch (Exception $e) {
expect($e->getMessage())->toContain('central-webhook');
}
}); });