@fails-components/webtransport 0.3.1 → 1.0.1

package/test/session.spec.js

@@ -10,9 +10,8 @@ import { expect } from './fixtures/chai.js'
  */
 
 describe('session', function () {
-  // FIXME: sometimes there are seemingly arbitrary 5s delays in
-  // communicating with the server under node.js
-  this.timeout(30000)
+  let forceReliable = false
+  if (process.env.USE_HTTP2 === 'true') forceReliable = true
 
   /** @type {import('../lib/dom').WebTransport | undefined} */
   let client
@@ -32,7 +31,9 @@ describe('session', function () {
           algorithm: 'sha-256',
           value: readCertHash(process.env.CERT_HASH)
         }
-      ]
+      ],
+      // @ts-ignore
+      forceReliable
     })
     await client.ready
 
@@ -51,14 +52,18 @@ describe('session', function () {
             algorithm: 'sha-256',
             value: readCertHash(process.env.CERT_HASH)
           }
-        ]
+        ],
+        // @ts-ignore
+        forceReliable
       }
     )
     await client.ready
 
     const result = await client.closed
     expect(result).to.have.property('closeCode', 7)
-    expect(result).to.have.property('reason', 'this is the reason')
+    if (process.env.USE_HTTP2 !== 'true')
+      // unsupported for http2
+      expect(result).to.have.property('reason', 'this is the reason')
   })
 
   it('should error when connecting to a server that does not exist', async () => {
@@ -70,7 +75,9 @@ describe('session', function () {
         }
       ],
       quicConnectTimeout: 100,
-      webTransportConnectTimeout: 100
+      webTransportConnectTimeout: 100,
+      // @ts-ignore
+      forceReliable
     })
 
     const [closedResult, readyResult] = await Promise.all([
@@ -93,7 +100,9 @@ describe('session', function () {
           algorithm: 'sha-256',
           value: readCertHash(process.env.CERT_HASH)
         }
-      ]
+      ],
+      // @ts-ignore
+      forceReliable
     })
 
     const [closedResult, readyResult] = await Promise.all([
@@ -109,51 +118,61 @@ describe('session', function () {
       .with.property('message', 'Opening handshake failed.')
   })
 
-  it('should error when connecting with a bad certificate', async () => {
-    client = new WebTransport(`${process.env.SERVER_URL}/session_close`, {
-      serverCertificateHashes: [
-        {
-          algorithm: 'sha-256',
-          value: readCertHash(process.env.CERT_HASH + ':DE:AD:BE:EF')
-        }
-      ]
+  if (
+    process.env.USE_POLYFILL !== 'true' &&
+    process.env.USE_PONYFILL !== 'true'
+  ) {
+    // deactivated in polyfill case, no test necessary
+    it('should error when connecting with a bad certificate', async () => {
+      client = new WebTransport(`${process.env.SERVER_URL}/session_close`, {
+        serverCertificateHashes: [
+          {
+            algorithm: 'sha-256',
+            value: readCertHash(process.env.CERT_HASH + ':DE:AD:BE:EF')
+          }
+        ],
+        // @ts-ignore
+        forceReliable
+      })
+
+      const [closedResult, readyResult] = await Promise.all([
+        client.closed.catch((err) => err),
+        client.ready.catch((err) => err)
+      ])
+
+      expect(closedResult)
+        .to.be.a('WebTransportError')
+        .with.property('message', 'Opening handshake failed.')
+      expect(readyResult)
+        .to.be.a('WebTransportError')
+        .with.property('message', 'Opening handshake failed.')
     })
 
-    const [closedResult, readyResult] = await Promise.all([
-      client.closed.catch((err) => err),
-      client.ready.catch((err) => err)
-    ])
-
-    expect(closedResult)
-      .to.be.a('WebTransportError')
-      .with.property('message', 'Opening handshake failed.')
-    expect(readyResult)
-      .to.be.a('WebTransportError')
-      .with.property('message', 'Opening handshake failed.')
-  })
-
-  it('should error when connecting with the wrong certificate', async () => {
-    client = new WebTransport(`${process.env.SERVER_URL}/session_close`, {
-      serverCertificateHashes: [
-        {
-          algorithm: 'sha-256',
-          value: readCertHash(
-            'DE:AD:BE:EF:' + process.env.CERT_HASH?.substring(12)
-          )
-        }
-      ]
+    it('should error when connecting with the wrong certificate', async () => {
+      client = new WebTransport(`${process.env.SERVER_URL}/session_close`, {
+        serverCertificateHashes: [
+          {
+            algorithm: 'sha-256',
+            value: readCertHash(
+              'DE:AD:BE:EF:' + process.env.CERT_HASH?.substring(12)
+            )
+          }
+        ],
+        // @ts-ignore
+        forceReliable
+      })
+
+      const [closedResult, readyResult] = await Promise.all([
+        client.closed.catch((err) => err),
+        client.ready.catch((err) => err)
+      ])
+
+      expect(closedResult)
+        .to.be.a('WebTransportError')
+        .with.property('message', 'Opening handshake failed.')
+      expect(readyResult)
+        .to.be.a('WebTransportError')
+        .with.property('message', 'Opening handshake failed.')
     })
-
-    const [closedResult, readyResult] = await Promise.all([
-      client.closed.catch((err) => err),
-      client.ready.catch((err) => err)
-    ])
-
-    expect(closedResult)
-      .to.be.a('WebTransportError')
-      .with.property('message', 'Opening handshake failed.')
-    expect(readyResult)
-      .to.be.a('WebTransportError')
-      .with.property('message', 'Opening handshake failed.')
-  })
+  }
 })

package/test/unidirectional-streams.spec.js

@@ -12,6 +12,8 @@ import { KNOWN_BYTES, KNOWN_BYTES_LENGTH } from './fixtures/known-bytes.js'
 describe('unidirectional streams', function () {
   /** @type {import('../lib/dom').WebTransport | undefined} */
   let client
+  let forceReliable = false
+  if (process.env.USE_HTTP2 === 'true') forceReliable = true
 
   // @ts-ignore
   afterEach(async () => {
@@ -32,7 +34,9 @@ describe('unidirectional streams', function () {
               algorithm: 'sha-256',
               value: readCertHash(process.env.CERT_HASH)
             }
-          ]
+          ],
+          // @ts-ignore
+          forceReliable
         }
       )
       await client.ready
@@ -47,6 +51,7 @@ describe('unidirectional streams', function () {
 
     // the remote will close the session
     const result = await client.closed
+    client = undefined
 
     // should receive the default close info
     expect(result).to.have.property('reason', '')
@@ -63,7 +68,9 @@ describe('unidirectional streams', function () {
             algorithm: 'sha-256',
             value: readCertHash(process.env.CERT_HASH)
           }
-        ]
+        ],
+        // @ts-ignore
+        forceReliable
       }
     )
     await client.ready
@@ -77,7 +84,13 @@ describe('unidirectional streams', function () {
   })
 
   it('handles fin when paused due to backpressure', async function () {
-    this.timeout(6000)
+    let addpolyfill = 0
+    if (
+      process.env.USE_POLYFILL === 'true' ||
+      process.env.USE_PONYFILL === 'true'
+    )
+      addpolyfill = 4000
+    this.timeout(6000 + addpolyfill)
     client = new WebTransport(
       `${process.env.SERVER_URL}/unidirectional_server_delay_before_read`,
       {
@@ -86,7 +99,9 @@ describe('unidirectional streams', function () {
             algorithm: 'sha-256',
             value: readCertHash(process.env.CERT_HASH)
           }
-        ]
+        ],
+        // @ts-ignore
+        forceReliable
       }
     )
     await client.ready
@@ -110,6 +125,7 @@ describe('unidirectional streams', function () {
 
     // the remote will close the session cleanly if everything was ok
     await client.closed
+    client = undefined
 
     // should receive the default close info, not true on chromium
     // expect(result).to.have.property('reason', '')