starred/coolify
1
0
Fork 0
mirror of https://github.com/coollabsio/coolify.git synced 2026-11-06 13:10:37 -04:00
Code Issues Packages Projects Releases Wiki Activity

test(previews): cover portless compose links in pull request comments

Browse Source 
Regression test for #11065. Since v4.3.15 (#11594) preview domains are
stored without the internal port, and every deployment regenerates them
before the ready comment is posted.
This commit is contained in:
Andras Bacsai
2026-10-24 00:44:35 +02:00
parent 14cdafdbe6
commit bb3c400459
1 changed files with 34 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/Feature/PreviewDomainPortOverridesTest.php
+34
View File
@@ -1,5 +1,7 @@
<?php
use App\Enums\ProcessStatus;
use App\Jobs\ApplicationPullRequestUpdateJob;
use App\Jobs\CheckDomainDnsJob;
use App\Jobs\ServerFilesFromServerJob;
use App\Livewire\Project\Application\PreviewDomains;
@@ -379,6 +381,38 @@ it('saves compose preview domain port overrides per service without putting the
]);
});
it('posts portless compose preview links in the pull request comment', function () {
$this->application->update([
'build_pack' => 'dockercompose',
'preview_url_template' => '{{pr_id}}.{{domain}}',
'docker_compose_raw' => "services:\n frontend:\n image: nginx:alpine\n backend:\n image: nginx:alpine\n",
'docker_compose_domains' => json_encode([
'frontend' => ['domain' => 'https://myapp.example.com:3000'],
'backend' => ['domain' => 'https://myapp.example.com:3001/backend'],
]),
]);
// A preview saved before ports moved to overrides still holds port-bearing domains.
$preview = createPreviewForPortTests($this->application->fresh(), 136, [
'docker_compose_domains' => json_encode([
'frontend' => ['domain' => 'https://136.myapp.example.com:3000'],
'backend' => ['domain' => 'https://136.myapp.example.com:3001/backend'],
]),
]);
// The deployment job regenerates the preview domains before it posts the comment.
$preview->generate_preview_fqdn_compose();
$job = new ApplicationPullRequestUpdateJob($this->application->fresh(), $preview->fresh(), ProcessStatus::FINISHED);
$links = (fn (): string => $this->getPreviewLinks())->call($job);
expect($links)->toBe('[Open frontend](https://136.myapp.example.com) | [Open backend](https://136.myapp.example.com/backend) | ')
->and($preview->fresh()->domain_port_overrides)->toBe([
'https://136.myapp.example.com' => 3000,
'https://136.myapp.example.com/backend' => 3001,
]);
});
it('copies the parent domain port override onto a generated preview domain', function () {
$this->application->update([
'fqdn' => 'https://app.example.com:8080',