ensure name is allocated with `toSliceClone` (#1917)

* ensure name is allocated with `toSliceClone`

* shorten toSliceClone

1 files changed, 15 insertions, 19 deletions

diff --git a/test/bun.js/node-dns.test.js b/test/bun.js/node-dns.test.js
index b3ee09d48..b84b8c9ba 100644
--- a/test/bun.js/node-dns.test.js
+++ b/test/bun.js/node-dns.test.js
@@ -25,7 +25,9 @@ test("dns.resolveSrv (_test._tcp.test.socketify.dev)", (done) => {
   dns.resolveSrv("_test._tcp.test.socketify.dev", (err, results) => {
     expect(err).toBeNull();
     expect(results instanceof Array).toBe(true);
-    expect(results[0].name).toBe("_dc-srv.130c90ab9de1._test._tcp.test.socketify.dev");
+    expect(results[0].name).toBe(
+      "_dc-srv.130c90ab9de1._test._tcp.test.socketify.dev",
+    );
     expect(results[0].priority).toBe(50);
     expect(results[0].weight).toBe(50);
     expect(results[0].port).toBe(80);
@@ -50,11 +52,9 @@ test("dns.resolveTxt (txt.socketify.dev)", (done) => {
   });
 });
 
-
 test("dns.resolveSoa (bun.sh)", (done) => {
   dns.resolveSoa("bun.sh", (err, result) => {
     expect(err).toBeNull();
-    
     expect(result.serial).toBe(2295878541);
     expect(result.refresh).toBe(10000);
     expect(result.retry).toBe(2400);
@@ -62,7 +62,6 @@ test("dns.resolveSoa (bun.sh)", (done) => {
     expect(result.minttl).toBe(3600);
     expect(result.nsname).toBe("hans.ns.cloudflare.com");
     expect(result.hostmaster).toBe("dns.cloudflare.com");
-    
     done(err);
   });
 });
@@ -71,10 +70,10 @@ test("dns.resolveNaptr (naptr.socketify.dev)", (done) => {
   dns.resolveNaptr("naptr.socketify.dev", (err, results) => {
     expect(err).toBeNull();
     expect(results instanceof Array).toBe(true);
-    expect(results[0].flags).toBe('S');
-    expect(results[0].service).toBe('test');
-    expect(results[0].regexp).toBe('');
-    expect(results[0].replacement).toBe('');
+    expect(results[0].flags).toBe("S");
+    expect(results[0].service).toBe("test");
+    expect(results[0].regexp).toBe("");
+    expect(results[0].replacement).toBe("");
     expect(results[0].order).toBe(1);
     expect(results[0].preference).toBe(12);
     done(err);
@@ -86,29 +85,27 @@ test("dns.resolveCaa (caa.socketify.dev)", (done) => {
     expect(err).toBeNull();
     expect(results instanceof Array).toBe(true);
     expect(results[0].critical).toBe(0);
-    expect(results[0].issue).toBe('bun.sh');
+    expect(results[0].issue).toBe("bun.sh");
     done(err);
   });
 });
 
-
 test("dns.resolveMx (bun.sh)", (done) => {
   dns.resolveMx("bun.sh", (err, results) => {
     expect(err).toBeNull();
     expect(results instanceof Array).toBe(true);
-    expect(results[0].priority).toBe(10);
-    expect(results[0].exchange).toBe('eforward1.registrar-servers.com');
+    const priority = results[0].priority;
+    expect(priority >= 0 && priority < 65535).toBe(true);
+    expect(results[0].exchange.includes(".registrar-servers.com")).toBe(true);
     done(err);
   });
 });
 
-
-
 test("dns.resolveNs (bun.sh) ", (done) => {
   dns.resolveNs("bun.sh", (err, results) => {
     expect(err).toBeNull();
     expect(results instanceof Array).toBe(true);
-    expect(results[0]).toBe('hans.ns.cloudflare.com');
+    expect(results[0].includes(".ns.cloudflare.com")).toBe(true);
     done(err);
   });
 });
@@ -117,7 +114,7 @@ test("dns.resolvePtr (ptr.socketify.dev)", (done) => {
   dns.resolvePtr("ptr.socketify.dev", (err, results) => {
     expect(err).toBeNull();
     expect(results instanceof Array).toBe(true);
-    expect(results[0]).toBe('bun.sh');
+    expect(results[0]).toBe("bun.sh");
     done(err);
   });
 });
@@ -126,12 +123,11 @@ test("dns.resolveCname (cname.socketify.dev)", (done) => {
   dns.resolveCname("cname.socketify.dev", (err, results) => {
     expect(err).toBeNull();
     expect(results instanceof Array).toBe(true);
-    expect(results[0]).toBe('bun.sh');
+    expect(results[0]).toBe("bun.sh");
     done(err);
   });
 });
 
-
 test("dns.lookup (example.com)", (done) => {
   dns.lookup("example.com", (err, address, family) => {
     expect(err).toBeNull();
@@ -151,4 +147,4 @@ test("dns.lookup (localhost)", (done) => {
 
     done(err);
   });
-});
\ No newline at end of file
+});