fix: remove node-fetch, use built-in fetch

package.json

@@ -10,7 +10,6 @@
     "ci-info": "^4.0.0",
     "debug": "^4.1.1",
     "inquirer": "^8.2.1",
-    "node-fetch": "^2.6.7",
     "os-name": "^4.0.1",
     "splunk-logging": "^0.11.1"
   },
@@ -28,7 +27,6 @@
     "eslint-plugin-standard": "^4.0.1",
     "execa": "^4.0.2",
     "jest": "^29",
-    "jest-fetch-mock": "^3.0.0",
     "jest-junit": "^13.0.0",
     "memfs": "^4.6.0",
     "oclif": "^4.3.6",

src/telemetry-lib.js

@@ -9,7 +9,6 @@ OF ANY KIND, either express or implied. See the License for the specific languag
 governing permissions and limitations under the License.
 */
 
-const fetch = require('node-fetch')
 const config = require('@adobe/aio-lib-core-config')
 const osName = require('os-name')
 const inquirer = require('inquirer')

test/hooks.test.js

@@ -10,7 +10,6 @@
  * governing permissions and limitations under the License.
  */
 
-const fetch = require('node-fetch')
 const inquirer = require('inquirer')
 const config = require('@adobe/aio-lib-core-config')
 
@@ -28,7 +27,7 @@ const mockPackageJson = {
 
 describe('hook interfaces', () => {
   beforeEach(() => {
-    fetch.mockReset()
+    global.setFetchMock()
   })
 
   test('command-error', async () => {

test/index.test.js

@@ -10,7 +10,6 @@
  * governing permissions and limitations under the License.
  */
 
-const fetch = require('node-fetch')
 const TheCommand = require('../src/commands/telemetry')
 const { stdout } = require('stdout-stderr')
 
@@ -19,9 +18,8 @@ jest.mock('inquirer')
 let command
 
 beforeEach(() => {
-  fetch.resetMocks()
   command = new TheCommand([])
-  fetch.mockResponseOnce('ok')
+  global.setFetchMock()
 })
 
 test('exports a run function', async () => {

test/jest.setup.js

@@ -16,8 +16,12 @@ const { vol } = require('memfs')
 jest.setTimeout(3000)
 jest.useFakeTimers()
 
-const fetch = require('jest-fetch-mock')
-jest.setMock('node-fetch', fetch)
+global.setFetchMock = (ok = true, mockData = {}) => {
+  global.fetch = jest.fn().mockResolvedValue({
+    ok,
+    json: () => ok ? Promise.resolve(mockData) : Promise.reject(mockData)
+  })
+}
 
 vol.reset()
 

test/telemetry-lib.test.js

@@ -10,16 +10,19 @@
  * governing permissions and limitations under the License.
  */
 
-const fetch = require('node-fetch')
 const telemetryLib = require('../src/telemetry-lib')
 const config = require('@adobe/aio-lib-core-config')
 
 jest.mock('@adobe/aio-lib-core-config')
-
+jest.mock('debug', () => {
+  global.mockDebug = jest.fn()
+  return () => global.mockDebug
+})
 describe('telemetry-lib', () => {
   beforeEach(() => {
     jest.resetModules()
-    fetch.mockReset()
+    global.mockDebug.mockReset()
+    global.setFetchMock()
   })
 
   test('exports messages', async () => {
@@ -43,10 +46,28 @@ describe('telemetry-lib', () => {
   test('uses client id from config', async () => {
     config.get.mockReturnValue('clientidxyz')
     telemetryLib.init('a@4', 'binTest2')
+
     await telemetryLib.trackEvent('test-event')
+
+    expect(config.get).toHaveBeenCalledWith('binTest2-cli-telemetry.clientId')
+    expect(config.get).toHaveBeenCalledWith('binTest2-cli-telemetry.optOut', 'global')
+    expect(fetch).toHaveBeenCalledWith(expect.any(String),
+      expect.objectContaining({ body: expect.stringContaining('"clientId":"clientidxyz"') }))
+    expect(global.mockDebug).not.toHaveBeenCalledWith(expect.stringContaining('error reaching telemetry server'))
+  })
+
+  test('fetch error', async () => {
+    const error = 'some error'
+    global.fetch = jest.fn().mockRejectedValue(error)
+
+    config.get.mockReturnValue('clientidxyz')
+    telemetryLib.init('a@4', 'binTest2')
+    await telemetryLib.trackEvent('test-event')
+
     expect(config.get).toHaveBeenCalledWith('binTest2-cli-telemetry.clientId')
     expect(config.get).toHaveBeenCalledWith('binTest2-cli-telemetry.optOut', 'global')
     expect(fetch).toHaveBeenCalledWith(expect.any(String),
       expect.objectContaining({ body: expect.stringContaining('"clientId":"clientidxyz"') }))
+    expect(global.mockDebug).toHaveBeenCalledWith(expect.stringContaining('error reaching telemetry server'), error)
   })
 })