@openneuro/server 4.6.2 → 4.7.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@openneuro/server",
-  "version": "4.6.2",
+  "version": "4.7.0",
   "description": "Core service for the OpenNeuro platform.",
   "license": "MIT",
   "main": "src/server.js",
@@ -18,7 +18,7 @@
   "dependencies": {
     "@apollo/client": "3.4.17",
     "@elastic/elasticsearch": "7.15.0",
-    "@openneuro/search": "^4.6.2",
+    "@openneuro/search": "^4.7.0",
     "@passport-next/passport-google-oauth2": "^1.0.0",
     "@sentry/node": "^4.5.3",
     "apollo-server": "2.25.3",
@@ -89,7 +89,7 @@
     "babel-eslint": "^10.1.0",
     "core-js": "^3.10.1",
     "ioredis-mock": "^3.8.1",
-    "jest": "^26.6.3",
+    "jest": "27.5.1",
     "mockingoose": "2.11.0",
     "nodemon": "^2.0.7",
     "supertest": "^3.0.0",
@@ -105,5 +105,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "7049c5927d87f4bdc0d081a6c34d643cdd623b65"
+  "gitHead": "74336af513bf3128af43ad3c55e2127b3a85915b"
 }

package/src/datalad/__tests__/dataset.spec.js

@@ -19,16 +19,15 @@ describe('dataset model operations', () => {
         'findOne',
       )
     })
-    it('resolves to dataset id string', async done => {
+    it('resolves to dataset id string', async () => {
       const { id: dsId } = await createDataset(null, null, {
         affirmedDefaced: true,
         affirmedConsent: true,
       })
       expect(dsId).toHaveLength(8)
       expect(dsId.slice(0, 2)).toBe('ds')
-      done()
     })
-    it('posts to the DataLad /datasets/{dsId} endpoint', async done => {
+    it('posts to the DataLad /datasets/{dsId} endpoint', async () => {
       // Reset call count for request.post
       request.post.mockClear()
       const { id: dsId } = await createDataset(null, null, {
@@ -39,7 +38,6 @@ describe('dataset model operations', () => {
       expect(request.post).toHaveBeenCalledWith(
         expect.stringContaining(`${getDatasetWorker(dsId)}/datasets/`),
       )
-      done()
     })
   })
   describe('datasetsFilter()', () => {

package/src/datalad/__tests__/pagination.spec.js

@@ -56,14 +56,13 @@ describe('pagination model operations', () => {
         'aggregate',
       )
     })
-    it('returns a connection shaped result', async done => {
+    it('returns a connection shaped result', async () => {
       const res = await pagination.datasetsConnection({
         orderBy: { created: 'ascending' },
         limit: 5,
       })([])
       expect(res).toHaveProperty('pageInfo')
       expect(res).toHaveProperty('edges')
-      done()
     })
   })
   describe('maxLimit()', () => {

package/src/datalad/__tests__/snapshots.spec.js

@@ -31,7 +31,7 @@ describe('snapshot model operations', () => {
         'findOne',
       )
     })
-    it('posts to the DataLad /datasets/{dsId}/snapshots/{snapshot} endpoint', async done => {
+    it('posts to the DataLad /datasets/{dsId}/snapshots/{snapshot} endpoint', async () => {
       const tag = 'snapshot'
       const { id: dsId } = await createDataset(null, null, {
         affirmedDefaced: true,
@@ -47,7 +47,6 @@ describe('snapshot model operations', () => {
           `${getDatasetWorker(dsId)}/datasets/${dsId}/snapshots/${tag}`,
         ),
       )
-      done()
     })
   })
 })

package/src/graphql/__tests__/comment.spec.js

@@ -17,7 +17,7 @@ describe('comment resolver helpers', () => {
     let aId
     let commentA
     let adminUser, nonAdminUser
-    beforeEach(async done => {
+    beforeEach(async () => {
       mockingoose.resetAll()
       adminUser = {
         user: 'userid',
@@ -35,10 +35,9 @@ describe('comment resolver helpers', () => {
       mockingoose.Comment.toReturn(commentA, 'findOne')
       aId = (await Comment.findById('just getting commentA here').exec())._id
       mockingoose.Comment.toReturn([], 'find')
-      done()
     })
 
-    it('returns an array of the deleted comment ids', async done => {
+    it('returns an array of the deleted comment ids', async () => {
       const deletedIds = (
         await deleteComment({}, { commentId: aId }, adminUser)
       ).map(id => id.toString())
@@ -46,7 +45,6 @@ describe('comment resolver helpers', () => {
       // mockingoose ids are sequential, and the documents returned from each query get new ids for some reason
       //   TODO: Replace mockingoose with better library
       expect(deletedIds[0].slice(-5)).toEqual(incrementHex(aId).slice(-5))
-      done()
     })
 
     it('prevents non-admin users from deleting comments', () => {

package/src/graphql/resolvers/__tests__/dataset.spec.js

@@ -16,14 +16,13 @@ describe('dataset resolvers', () => {
     )
   })
   describe('createDataset()', () => {
-    it('createDataset mutation succeeds', async done => {
+    it('createDataset mutation succeeds', async () => {
       const { id: dsId } = await ds.createDataset(
         null,
         { affirmedDefaced: true, affirmedConsent: false },
         { user: '123456', userInfo: {} },
       )
       expect(dsId).toEqual(expect.stringMatching(/^ds[0-9]{6}$/))
-      done()
     })
   })
   describe('snapshotCreationComparison()', () => {