Skip to content

Commit

Permalink
adjust test so that CNAME is excluded
Browse files Browse the repository at this point in the history
Signed-off-by: georglauterbach <[email protected]>
  • Loading branch information
georglauterbach committed Oct 13, 2024
1 parent f3055c2 commit e3d7d84
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions tests/lib/Http/Client/DnsPinMiddlewareTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -537,10 +537,11 @@ static function (RequestInterface $request, array $options) {
['nextcloud' => ['allow_local_address' => false]]
);

$this->assertCount(4, $dnsQueries);
$this->assertCount(3, $dnsQueries);
$this->assertContains('example.com' . DNS_SOA, $dnsQueries);
$this->assertContains('subsubdomain.subdomain.example.com' . DNS_A, $dnsQueries);
$this->assertContains('subsubdomain.subdomain.example.com' . DNS_AAAA, $dnsQueries);
$this->assertContains('subsubdomain.subdomain.example.com' . DNS_CNAME, $dnsQueries);
// CNAME should not be queried of A or AAAA succeeded already
$this->assertNotContains('subsubdomain.subdomain.example.com' . DNS_AAAA, $dnsQueries);
}
}

0 comments on commit e3d7d84

Please sign in to comment.