Merge pull request #3 from rickycheers/fix-domain-parsing

[2.x] Correctly obtain domain when tld substring is in it

Author: adrorocker

src/Host/Host.php

@@ -38,7 +38,10 @@ public function tld(?string $tld = null): string|self
 
         $this->tld = $tld;
 
-        $root = str_replace(strval($this->tld), '', strval($this->host));
+        $escaped = str_replace('.', '\.', '.' . strval($this->tld));
+
+        /** @var string $root */
+        $root = preg_replace("/$escaped$/", '', strval($this->host));
 
         if (!validate_domain_root(trim($root, '.'))) {
             $this->isValid = false;

tests/Unit/ValidatorTest.php

@@ -62,3 +62,19 @@
     it('is not empty domain', fn () =>  expect($host->domain())->toBe('adro.com'));
     it("is same as $url", fn () =>  expect($host->original())->toBe($url));
 });
+
+describe('Root domain is correctly obtained when the tld substring is contained in it', function () {
+    $validator = getInstance();
+    $url = 'a.b.c.compass.com';
+    $host = $validator->validate($url);
+
+    it("$url is valid domain", fn () => expect($host->isValid())->toBeTrue());
+    it("from $url - compass.com is the domain", fn () => expect($host->domain())->toBe('compass.com'));
+
+    $validator = getInstance();
+    $url = 'compass.com';
+    $host = $validator->validate($url);
+
+    it("$url is valid domain", fn () => expect($host->isValid())->toBeTrue());
+    it("from $url - compass.com is the domain", fn () => expect($host->domain())->toBe('compass.com'));
+});