refactor: simple

test/node-fetch/main.js

@@ -10,7 +10,7 @@ const crypto = require('crypto')
 const chaiPromised = require('chai-as-promised')
 const chaiIterator = require('chai-iterator')
 const chaiString = require('chai-string')
-const { delay } = require('../utils/timeout')
+const { timeout } = require('../utils/timeout')
 const { Blob } = require('buffer')
 
 const {
@@ -761,7 +761,7 @@ describe('node-fetch', () => {
 
   it('should collect handled errors on the body stream to reject if the body is used later', () => {
     const url = `${base}invalid-content-encoding`
-    return fetch(url).then(delay(20)).then(res => {
+    return fetch(url).then(timeout(20, { rejection: false })).then(res => {
       expect(res.headers.get('content-type')).to.equal('text/plain')
       return expect(res.text()).to.eventually.be.rejected
     })

test/node-fetch/utils/chai-timeout.js

@@ -1,11 +1,15 @@
-const { pTimeout } = require('../../utils/timeout')
+const { timeout } = require('../../utils/timeout')
 
 module.exports = ({ Assertion }, utils) => {
   utils.addProperty(Assertion.prototype, 'timeout', async function () {
     let timeouted = false
-    await pTimeout(this._obj, 150, () => {
-      timeouted = true
+    await timeout(150, {
+      promise: this._obj,
+      fallback: () => {
+        timeouted = true
+      }
     })
+
     return this.assert(
       timeouted,
       'expected promise to timeout but it was resolved',

test/utils/timeout.js

@@ -44,22 +44,6 @@ async function timeout (time, options = {}) {
   }
 }
 
-/**
- * @type {<T, R = T>(promise: Promise<T>, time: number, fallback?: () => R | Promise<R>) => Promise<Awaited<R> | T>}
- */
-function pTimeout (promise, time, fallback) {
-  return timeout(time, { promise, fallback })
-}
-
-/**
- * @param {number} time
- */
-function delay (time) {
-  return timeout(time, { rejection: false })
-}
-
 module.exports = {
-  timeout,
-  delay,
-  pTimeout
+  timeout
 }