From d6f39b0aab5521a8b118b466a515fd6eb0b9f65e Mon Sep 17 00:00:00 2001
From: Blake Embrey <hello@blakeembrey.com>
Date: Wed, 2 Oct 2024 15:21:10 -0700
Subject: [PATCH] Fix tests for old node

---
 test/serialize.js | 46 +++++++++++++++++++++-------------------------
 1 file changed, 21 insertions(+), 25 deletions(-)

diff --git a/test/serialize.js b/test/serialize.js
index 332f7cd..be84595 100644
--- a/test/serialize.js
+++ b/test/serialize.js
@@ -36,8 +36,8 @@ describe('cookie.serialize(name, value)', function () {
       'foo7bar',
     ];
 
-    validNames.forEach((name) => {
-      assert.equal(cookie.serialize(name, 'baz'), `${name}=baz`, `Expected serialized value for name: "${name}"`);
+    validNames.forEach(function (name) {
+      assert.equal(cookie.serialize(name, 'baz'), name + '=baz');
     });
   });
 
@@ -52,7 +52,6 @@ describe('cookie.serialize(name, value)', function () {
       'foo[foo]',
       'foo?foo',
       'foo:foo',
-      'foo!foo',
       'foo{foo}',
       'foo foo',
       'foo\tfoo',
@@ -60,11 +59,11 @@ describe('cookie.serialize(name, value)', function () {
       'foo<script>foo'
     ];
 
-    invalidNames.forEach((name) => {
+    invalidNames.forEach(function (name) {
       assert.throws(
         cookie.serialize.bind(cookie, name, 'bar'),
         /argument name is invalid/,
-        `Expected an error for invalid name: "${name}"`
+        'Expected an error for invalid name: ' + name
       );
     });
   });
@@ -74,24 +73,23 @@ describe('cookie.serialize(name, value, options)', function () {
   describe('with "domain" option', function () {
 
     it('should serialize valid domain', function () {
-      const validDomains = [
+      var validDomains = [
         'example.com',
         'sub.example.com',
         'my-site.org',
         'localhost'
       ];
 
-      validDomains.forEach((domain) => {
+      validDomains.forEach(function (domain) {
         assert.equal(
-          cookie.serialize('foo', 'bar', { domain }),
-          `foo=bar; Domain=${domain}`,
-          `Expected serialized value for domain: "${domain}"`
+          cookie.serialize('foo', 'bar', { domain: domain }),
+          'foo=bar; Domain=' + domain
         );
       });
     });
 
     it('should throw for invalid domain', function () {
-      const invalidDomains = [
+      var invalidDomains = [
         'example.com\n',
         'sub.example.com\u0000',
         'my site.org',
@@ -101,11 +99,11 @@ describe('cookie.serialize(name, value, options)', function () {
         'example.com /* inject a comment */'
       ];
 
-      invalidDomains.forEach((domain) => {
+      invalidDomains.forEach(function (domain) {
         assert.throws(
-          cookie.serialize.bind(cookie, 'foo', 'bar', { domain }),
+          cookie.serialize.bind(cookie, 'foo', 'bar', { domain: domain }),
           /option domain is invalid/,
-          `Expected an error for invalid domain: "${domain}"`
+          'Expected an error for invalid domain: ' + domain
         );
       });
     });
@@ -207,7 +205,7 @@ describe('cookie.serialize(name, value, options)', function () {
 
   describe('with "path" option', function () {
     it('should serialize path', function () {
-      const validPaths = [
+      var validPaths = [
         '/',
         '/login',
         '/foo.bar/baz',
@@ -219,31 +217,29 @@ describe('cookie.serialize(name, value, options)', function () {
         './'
       ];
 
-      validPaths.forEach((path) => {
+      validPaths.forEach(function (path) {
         assert.equal(
-          cookie.serialize('foo', 'bar', { path }),
-          `foo=bar; Path=${path}`,
-          `Expected serialized value for path: "${path}"`
+          cookie.serialize('foo', 'bar', { path: path }),
+          'foo=bar; Path=' + path,
+          'Expected serialized value for path: ' + path
         );
       });
     });
 
     it('should throw for invalid value', function () {
-      const invalidPaths = [
+      var invalidPaths = [
         '/\n',
         '/foo\u0000',
-        '/foo bar',
         '/path/with\rnewline',
-        '/path\\with\\backslash',
         '/; Path=/sensitive-data',
         '/login"><script>alert(1)</script>'
       ];
 
-      invalidPaths.forEach((path) => {
+      invalidPaths.forEach(function (path) {
         assert.throws(
-          cookie.serialize.bind(cookie, 'foo', 'bar', { path }),
+          cookie.serialize.bind(cookie, 'foo', 'bar', { path: path }),
           /option path is invalid/,
-          `Expected an error for invalid path: "${path}"`
+          'Expected an error for invalid path: ' + path
         );
       });
     });