Merge pull request #43 from lynnverse/master

Map PTR queries

Author: Christian Thomas

src/Entities/DataAbstract.php

@@ -69,6 +69,10 @@ public static function createFromTypeAndString(DNSRecordType $recordType, string
             );
         }
 
+        if ($recordType->isA(DNSRecordType::TYPE_PTR)) {
+            return new PTRData(new Hostname($data));
+        }
+
         throw new InvalidArgumentException("{$data} could not be created with type {$recordType}");
     }
 

src/Entities/PTRData.php

@@ -0,0 +1,47 @@
+<?php
+
+namespace RemotelyLiving\PHPDNS\Entities;
+
+use function serialize;
+use function unserialize;
+
+final class PTRData extends DataAbstract
+{
+    private Hostname $hostname;
+
+    public function __construct(Hostname $hostname)
+    {
+        $this->hostname = $hostname;
+    }
+
+    public function __toString(): string
+    {
+        return (string)$this->hostname;
+    }
+
+    public function getHostname(): Hostname
+    {
+        return $this->hostname;
+    }
+
+    public function toArray(): array
+    {
+        return [
+            'hostname' => (string)$this->hostname,
+        ];
+    }
+
+    public function serialize(): string
+    {
+        return serialize($this->toArray());
+    }
+
+    /**
+     * @param string $serialized
+     */
+    public function unserialize($serialized): void
+    {
+        $unserialized = unserialize($serialized);
+        $this->hostname = new Hostname($unserialized['hostname']);
+    }
+}

tests/Unit/Entities/PTRDataTest.php

@@ -0,0 +1,84 @@
+<?php
+
+namespace RemotelyLiving\PHPDNS\Tests\Unit\Entities;
+
+use RemotelyLiving\PHPDNS\Entities\Hostname;
+use RemotelyLiving\PHPDNS\Entities\PTRData;
+use RemotelyLiving\PHPDNS\Tests\Unit\BaseTestAbstract;
+
+class PTRDataTest extends BaseTestAbstract
+{
+    /**
+     * @var \RemotelyLiving\PHPDNS\Entities\Hostname
+     */
+    private $hostname;
+
+    /**
+     * @var \RemotelyLiving\PHPDNS\Entities\PTRData
+     */
+    private $PTRData;
+
+    protected function setUp(): void
+    {
+        parent::setUp();
+
+        $this->hostname = new Hostname('google.com');
+        $this->PTRData = new PTRData($this->hostname);
+    }
+
+    /**
+     * @test
+     */
+    public function knowsIfEquals(): void
+    {
+        $this->assertTrue($this->PTRData->equals($this->PTRData));
+        $this->assertFalse($this->PTRData->equals(new PTRData(new Hostname('boop.com'))));
+    }
+
+    /**
+     * @test
+     */
+    public function isArrayable(): void
+    {
+        $this->assertArrayableAndEquals(
+            ['hostname' => (string)$this->hostname],
+            $this->PTRData
+        );
+    }
+
+    /**
+     * @test
+     */
+    public function isJsonSerializable(): void
+    {
+        $this->assertJsonSerializeableAndEquals(
+            ['hostname' => (string)$this->hostname],
+            $this->PTRData
+        );
+    }
+
+    /**
+     * @test
+     */
+    public function isSerializable(): void
+    {
+        $this->assertSerializable($this->PTRData);
+    }
+
+    /**
+     * @test
+     */
+    public function isStringable(): void
+    {
+        $this->assertStringableAndEquals('google.com.', $this->PTRData);
+        $this->assertEquals($this->PTRData, unserialize(serialize($this->PTRData)));
+    }
+
+    /**
+     * @test
+     */
+    public function hasBasicGetters(): void
+    {
+        $this->assertSame($this->hostname, $this->PTRData->getHostname());
+    }
+}

tests/Unit/Mappers/GoogleDNSTest.php

@@ -61,6 +61,18 @@ public function mapsAGoogleDNSRecordVariants(): void
             DNSRecord::createFromPrimitives('TXT', 'facebook.com', 365, null, 'IN', 'txtval'),
             $mappedRecord
         );
+
+        $mappedRecord = $this->mapper->mapFields([
+            'type' => 12,
+            'TTL' => 18930,
+            'name' => '8.8.8.8.in-addr.arpa.',
+            'data' => 'dns.google.'
+        ])->toDNSRecord();
+
+        $this->assertEquals(
+            DNSRecord::createFromPrimitives('PTR', '8.8.8.8.in-addr.arpa.', 18930, null, 'IN', 'dns.google.'),
+            $mappedRecord
+        );
     }
 
     /**

tests/Unit/Mappers/LocalSystemTest.php

@@ -95,6 +95,14 @@ class LocalSystemTest extends BaseTestAbstract
             'class' => 'IN',
             'weight' => 200,
             'port' => 9999,
+        ],
+        [
+            'host' => '8.8.8.8.in-addr.arpa',
+            'class' => 'IN',
+            'ttl' => 15248,
+            'type' => 'PTR',
+            'target' => 'dns.google'
+
         ]
     ];