build(deps-dev): bump @types/node to v16

[bavulapati]

Description of Change

Bump @types/node to major line v16 as we are already using node v16.x.x

Re-enabling the associated test as the require PR got merged
cc: @codebytere @MarshallOfSound @dsanders11

Checklist

• PR description included and stakeholders cc'd
• npm test passes
• tests are changed or added

Release Notes

Notes: none

[bavulapati]

The test failure looks unrelated to the change.

[dsanders11]

Confirming the one build failure is unrelated.

not ok 18 app module app.requestSingleInstanceLock gets back an ack after preventing default
  AssertionError: received data null is not equal to expected data {"level":1,"testkey":"acktestvalue1","inner":{"level":2,"testkey":"acktestvalue2"}}.: expected null to deeply equal { level: 1,
    testkey: 'acktestvalue1',
    inner: { level: 2, testkey: 'acktestvalue2' } }
      at testArgumentPassing (electron\spec-main\api-app-spec.ts:269:42)
      at processTicksAndRejections (node:internal/process/task_queues:96:5)
      at Context.<anonymous> (electron\spec-main\api-app-spec.ts:308:7)

@bavulapati, I think this should also bump the version in the root package.json at the same time, so they stay in sync?

[codebytere]

@bavulapati lint failure:

../node_modules/@types/express-serve-static-core/index.d.ts(505,18): error TS2430: Interface 'Response' incorrectly extends interface 'ServerResponse'.
Property 'req' is optional in type 'Response' but required in type 'ServerResponse'.

[bavulapati]

@codebytere I've addressed the lint errors. Had to update few more types.
@dsanders11 Updated the node types in the root package.json as well.
Waiting for the CI to go all ✅

[jkleinsc]

• Closing as this was resolved via chore: update node types version #33452