Update all dependencies (#1125)

* Update all dependencies Signed-off-by: Matteo Collina <hello@matteocollina.com> * fixup Signed-off-by: Matteo Collina <hello@matteocollina.com> --------- Signed-off-by: Matteo Collina <hello@matteocollina.com>
mercurius-js · Dec 12, 2024 · 368b85d · 368b85d
1 parent 8d6c319
commit 368b85d
Show file tree

Hide file tree

Showing 11 changed files with 79 additions and 103 deletions.
diff --git a/.gitignore b/.gitignore
@@ -72,4 +72,7 @@ typings/
 # lockfiles
 package-lock.json
 yarn.lock
-pnpm-lock.yaml
+pnpm-lock.yaml
+
+# tap
+.tap
diff --git a/.taprc b/.taprc
diff --git a/lib/subscription-connection.js b/lib/subscription-connection.js
@@ -56,6 +56,7 @@ module.exports = class SubscriptionConnection {
         this.fastify.log.error(err)
         this.handleConnectionClose()
       }
+      /* c8 ignore next 3 */
     }
   }
 
@@ -168,7 +169,7 @@ module.exports = class SubscriptionConnection {
     if (this.keepAlive) {
       this.sendKeepAlive()
 
-      /* istanbul ignore next */
+      /* c8 ignore next 3 */
       this.keepAliveTimer = setInterval(() => {
         this.sendKeepAlive()
       }, this.keepAlive)

diff --git a/package.json b/package.json
@@ -31,44 +31,44 @@
     "@graphql-tools/merge": "^9.0.0",
     "@graphql-tools/schema": "^10.0.0",
     "@graphql-tools/utils": "^10.0.0",
-    "@sinonjs/fake-timers": "^11.0.0",
+    "@sinonjs/fake-timers": "^14.0.0",
     "@types/isomorphic-form-data": "^2.0.0",
     "@types/node": "^22.0.0",
     "@types/ws": "^8.2.0",
-    "autocannon": "^7.3.0",
-    "concurrently": "^8.0.1",
+    "autocannon": "^8.0.0",
+    "concurrently": "^9.0.0",
     "docsify-cli": "^4.4.3",
     "eslint": "^9.9.1",
     "fastify": "^5.0.0-alpha.4",
     "graphql": "^16.0.0",
     "graphql-tag": "^2.12.6",
     "graphql-ws": "^5.11.2",
-    "neostandard": "^0.11.4",
+    "neostandard": "^0.12.0",
     "pre-commit": "^1.2.2",
     "proxyquire": "^2.1.3",
     "semver": "^7.5.0",
-    "sinon": "^17.0.0",
+    "sinon": "^19.0.0",
     "split2": "^4.0.0",
-    "tap": "^16.3.0",
-    "tsd": "^0.30.0",
+    "tap": "^21.0.0",
+    "tsd": "^0.31.0",
     "typescript": "^5.0.2",
-    "wait-on": "^7.0.1"
+    "undici": "^7.0.0",
+    "wait-on": "^8.0.0"
   },
   "dependencies": {
     "@fastify/error": "^4.0.0",
     "@fastify/static": "^8.0.0",
     "@fastify/websocket": "^11.0.0",
     "fastify-plugin": "^5.0.0",
-    "graphql-jit": "0.8.6",
+    "graphql-jit": "0.8.7",
     "mqemitter": "^6.0.0",
     "p-map": "^4.0.0",
     "quick-lru": "^7.0.0",
     "readable-stream": "^4.0.0",
     "safe-stable-stringify": "^2.3.0",
-    "secure-json-parse": "^2.7.0",
+    "secure-json-parse": "^3.0.0",
     "single-user-cache": "^1.0.0",
     "tiny-lru": "^11.0.0",
-    "undici": "^6.19.8",
     "ws": "^8.2.2"
   },
   "tsd": {

diff --git a/tap-snapshots/test/errors.js.test.cjs b/tap-snapshots/test/errors.js.test.cjs
@@ -5,7 +5,51 @@
  * Make sure to inspect the output below.  Do not ignore changes!
  */
 'use strict'
-exports['test/errors.js TAP POST query which throws, with JIT enabled, twice > must match snapshot 1'] = `
+exports['test/errors.js > TAP > app.graphql which throws, with JIT enabled, twice > must match snapshot 1'] = `
+{
+  "errors": [
+    {
+      "message": "Bad Resolver",
+      "locations": [
+        {
+          "line": 3,
+          "column": 9
+        }
+      ],
+      "path": [
+        "bad"
+      ]
+    }
+  ],
+  "data": {
+    "bad": null
+  }
+}
+`
+
+exports['test/errors.js > TAP > app.graphql which throws, with JIT enabled, twice > must match snapshot 2'] = `
+{
+  "data": {
+    "bad": null
+  },
+  "errors": [
+    {
+      "message": "Int cannot represent non-integer value: [function bad]",
+      "locations": [
+        {
+          "line": 3,
+          "column": 9
+        }
+      ],
+      "path": [
+        "bad"
+      ]
+    }
+  ]
+}
+`
+
+exports['test/errors.js > TAP > POST query which throws, with JIT enabled, twice > must match snapshot 1'] = `
 {
   "data": {
     "bad": null
@@ -27,7 +71,7 @@ exports['test/errors.js TAP POST query which throws, with JIT enabled, twice > m
 }
 `
 
-exports['test/errors.js TAP POST query which throws, with JIT enabled, twice > must match snapshot 2'] = `
+exports['test/errors.js > TAP > POST query which throws, with JIT enabled, twice > must match snapshot 2'] = `
 {
   "data": {
     "bad": null
@@ -48,11 +92,3 @@ exports['test/errors.js TAP POST query which throws, with JIT enabled, twice > m
   ]
 }
 `
-
-exports['test/errors.js TAP app.graphql which throws, with JIT enabled, twice > must match snapshot 1'] = `
-{"errors":[{"message":"Bad Resolver","locations":[{"line":3,"column":9}],"path":["bad"]}],"data":{"bad":null}}
-`
-
-exports['test/errors.js TAP app.graphql which throws, with JIT enabled, twice > must match snapshot 2'] = `
-{"data":{"bad":null},"errors":[{"message":"Int cannot represent non-integer value: [function bad]","locations":[{"line":3,"column":9}],"path":["bad"]}]}
-`
diff --git a/tap-snapshots/test/federation.js.test.cjs b/tap-snapshots/test/federation.js.test.cjs
diff --git a/tap-snapshots/test/gateway/remote-services.js.test.cjs b/tap-snapshots/test/gateway/remote-services.js.test.cjs
diff --git a/tap-snapshots/test/routes.js.test.cjs b/tap-snapshots/test/routes.js.test.cjs
@@ -5,7 +5,7 @@
  * Make sure to inspect the output below.  Do not ignore changes!
  */
 'use strict'
-exports['test/routes.js TAP GET return 200 on resolver error > must match snapshot 1'] = `
+exports['test/routes.js > TAP > GET return 200 on resolver error > must match snapshot 1'] = `
 {
   "data": {
     "add": null
@@ -27,7 +27,7 @@ exports['test/routes.js TAP GET return 200 on resolver error > must match snapsh
 }
 `
 
-exports['test/routes.js TAP HTTP mutation with GET errors > must match snapshot 1'] = `
+exports['test/routes.js > TAP > HTTP mutation with GET errors > must match snapshot 1'] = `
 {
   "data": null,
   "errors": [
@@ -38,7 +38,12 @@ exports['test/routes.js TAP HTTP mutation with GET errors > must match snapshot
 }
 `
 
-exports['test/routes.js TAP POST return 200 on resolver error > must match snapshot 1'] = `
+exports['test/routes.js > TAP > if ide is graphiql, serve config.js with the correct endpoint > must match snapshot 1'] = `
+window.GRAPHQL_ENDPOINT = '/app/graphql';
+window.GRAPHIQL_PLUGIN_LIST = []
+`
+
+exports['test/routes.js > TAP > POST return 200 on resolver error > must match snapshot 1'] = `
 {
   "data": {
     "add": null
@@ -60,7 +65,7 @@ exports['test/routes.js TAP POST return 200 on resolver error > must match snaps
 }
 `
 
-exports['test/routes.js TAP POST return 400 on error > must match snapshot 1'] = `
+exports['test/routes.js > TAP > POST return 400 on error > must match snapshot 1'] = `
 {
   "data": null,
   "errors": [
@@ -77,12 +82,7 @@ exports['test/routes.js TAP POST return 400 on error > must match snapshot 1'] =
 }
 `
 
-exports['test/routes.js TAP if ide is graphiql, serve config.js with the correct endpoint > must match snapshot 1'] = `
-window.GRAPHQL_ENDPOINT = '/app/graphql';
-window.GRAPHIQL_PLUGIN_LIST = []
-`
-
-exports['test/routes.js TAP server should return 200 on graphql errors (if field can be null) > must match snapshot 1'] = `
+exports['test/routes.js > TAP > server should return 200 on graphql errors (if field can be null) > must match snapshot 1'] = `
 {
   "data": {
     "hello": null
@@ -104,7 +104,7 @@ exports['test/routes.js TAP server should return 200 on graphql errors (if field
 }
 `
 
-exports['test/routes.js TAP server should return 200 on graphql errors (if field can not be null) > must match snapshot 1'] = `
+exports['test/routes.js > TAP > server should return 200 on graphql errors (if field can not be null) > must match snapshot 1'] = `
 {
   "data": null,
   "errors": [

diff --git a/test/app-decorator.js b/test/app-decorator.js
@@ -628,7 +628,7 @@ test('scalar should be supported', async (t) => {
         return value
       },
       serialize (value) {
-        t.pass(value, 4)
+        t.equal(value, 4)
         return value
       },
       parseLiteral (ast) {

diff --git a/test/errors.js b/test/errors.js
@@ -775,11 +775,11 @@ test('app.graphql which throws, with JIT enabled, twice', async (t) => {
 
   let res = await app.graphql(query, null, { x: 1 })
 
-  t.matchSnapshot(JSON.stringify(res), null, 2)
+  t.matchSnapshot(JSON.stringify(res, null, 2))
 
   res = await app.graphql(query, null, { x: 1 })
 
-  t.matchSnapshot(JSON.stringify(res), null, 2)
+  t.matchSnapshot(JSON.stringify(res, null, 2))
 
   lines.end()
 

diff --git a/test/subscription.js b/test/subscription.js
@@ -17,7 +17,8 @@ t.beforeEach(({ context }) => {
   context.clock = FakeTimers.install({
     shouldClearNativeTimers: true,
     shouldAdvanceTime: true,
-    advanceTimeDelta: 40
+    advanceTimeDelta: 40,
+    toFake: ['setTimeout', 'clearTimeout', 'setImmediate', 'clearImmediate', 'setInterval', 'clearInterval', 'Date', 'hrtime', 'performance']
   })
 })