[primitives] fix Headers#entries implementation to be consistent (#942)

* update body test

* fix entries() function implementation as undici implements it now

* refactor test helpers

* add changeset

* test iterators only if above node.js 16

* run tests against node.js 16.x if they run in edge runtime

Author: Schniz

.changeset/dirty-timers-reply.md

@@ -0,0 +1,21 @@
+---
+'@edge-runtime/primitives': minor
+---
+
+remove custom entries() iterator in favor of Undici's default implementation
+
+The difference will be that `set-cookie` headers will be emitted independently.
+
+```ts
+const headers = new Headers()
+headers.append('set-cookie', 'a=1')
+headers.append('set-cookie', 'b=2')
+
+const entries = [...headers.entries()]
+
+// previous implementation
+console.log(entries) // [["set-cookie", "a=1, b=2"]]
+
+// new implementation (undici's native implementation)
+console.log(entries) // [["set-cookie", "a=1"], ["set-cookie", "b=2"]]
+```

packages/integration-tests/package.json

@@ -9,7 +9,6 @@
   },
   "license": "MIT",
   "devDependencies": {
-    "http-body": "1.0.12",
     "multer": "1.4.5-lts.1",
     "test-listen": "1.1.0",
     "@types/test-listen": "1.1.2",

packages/integration-tests/test/abort-controller.test.ts

@@ -1,12 +1,11 @@
+import { polyfilledOrNative, guard } from './test-if'
+
 if (!globalThis.DOMException) {
   globalThis.DOMException = require('@edge-runtime/ponyfill').DOMException
 }
 
-const testOrSkip =
-  process.versions.node.split('.').map(Number)[0] > 16 ? test : test.skip
-
 describe('AbortController', () => {
-  testOrSkip('allows to abort fetch', async () => {
+  guard(it, polyfilledOrNative)('allows to abort fetch', async () => {
     expect.assertions(1)
     const controller = new AbortController()
     controller.abort()